[Qemu-devel] Re: sparc32 do not clear interrupts when masking

2010-01-17 Thread Blue Swirl
On Sun, Jan 17, 2010 at 6:42 PM, Artyom Tarasenko wrote: > 2010/1/16 Blue Swirl : >> On Sat, Jan 16, 2010 at 8:47 AM, Blue Swirl wrote: >>> On Sat, Jan 16, 2010 at 8:16 AM, Blue Swirl wrote: On Fri, Jan 15, 2010 at 10:37 PM, Artyom Tarasenko wrote: > Don't clear interrupts on disa

[Qemu-devel] Re: sparc32 do not clear interrupts when masking

2010-01-17 Thread Artyom Tarasenko
2010/1/16 Blue Swirl : > On Sat, Jan 16, 2010 at 8:47 AM, Blue Swirl wrote: >> On Sat, Jan 16, 2010 at 8:16 AM, Blue Swirl wrote: >>> On Fri, Jan 15, 2010 at 10:37 PM, Artyom Tarasenko >>> wrote: Don't clear interrupts on disabling, because * Sun4M_SystemArchitecture_edited2.pdf doesn'

[Qemu-devel] [PATCH] [For stable-0.12] Sync OSS_GETVERSION handling with head

2010-01-17 Thread Juergen Lock
As suggested by Andreas Färber, here is a cumulative patch that syncs OSS_GETVERSION handling with head by merging the following commits: 1. oss: issue OSS_GETVERSION ioctl only when needed 6d246526ce3c145b2831285def6983f5de6190d3 2. oss: fix fragment setting 3d709fe73a77c40e263b3af6e650fd4

[Qemu-devel] [RFC] Don't send local debug output to stdout (was: pm_smbus: remove #ifdef DEBUG)

2010-01-17 Thread Stefan Weil
Isaku Yamahata schrieb: > On Wed, Jan 06, 2010 at 12:42:28PM +0100, Stefan Weil wrote: >> Isaku Yamahata schrieb: >>> remove #ifdef DEBUG by using macro. >>> >>> Signed-off-by: Isaku Yamahata >>> Acked-by: Gerd Hoffmann >>> --- >>> hw/pm_smbus.c | 21 - >>> 1 files changed, 12

Re: [Qemu-devel] [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Loïc Minier
On Sun, Jan 17, 2010, Stefan Weil wrote: > > On systems were sdl-config isn't installed, ./configure triggers this > > warning: > > ./configure: 957: sdl-config: not found > > which version did you test? > Git master has no sdl-config call at configure:957. > > But I get warning messages, too, wh

[Qemu-devel] Re: [PATCH v2 2/4] Clean-up a little bit the RW related bits of BDRV_O_FLAGS. BDRV_O_RDONLY gone (and so is BDRV_O_ACCESS). Default value for bdrv_flags (0/zero) is READ-ONLY. Need to exp

2010-01-17 Thread Michael S. Tsirkin
On Sun, Jan 17, 2010 at 04:48:13PM +0200, Naphtali Sprei wrote: > Instead of using the field 'readonly' of the BlockDriverState struct for > passing the request, > pass the request in the flags parameter to the function. > > Signed-off-by: Naphtali Sprei Many changes seem to be about passing 0

[Qemu-devel] Re: [PATCH v2 4/4] Disable fall-back to read-only when cannot open drive's file for read-write

2010-01-17 Thread Michael S. Tsirkin
On Sun, Jan 17, 2010 at 04:48:15PM +0200, Naphtali Sprei wrote: > > Signed-off-by: Naphtali Sprei > --- > block.c |6 -- > 1 files changed, 0 insertions(+), 6 deletions(-) > > diff --git a/block.c b/block.c > index 8def3c4..f90e983 100644 > --- a/block.c > +++ b/block.c > @@ -444,8 +444

[Qemu-devel] [PATCH v2 4/4] Disable fall-back to read-only when cannot open drive's file for read-write

2010-01-17 Thread Naphtali Sprei
Signed-off-by: Naphtali Sprei --- block.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/block.c b/block.c index 8def3c4..f90e983 100644 --- a/block.c +++ b/block.c @@ -444,8 +444,6 @@ int bdrv_open2(BlockDriverState *bs, const char *filename, int flags, if (

[Qemu-devel] [PATCH v2 3/4] Added drives' readonly option

2010-01-17 Thread Naphtali Sprei
Signed-off-by: Naphtali Sprei --- qemu-options.hx |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemu-options.hx b/qemu-options.hx index e2edd71..4617867 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -103,7 +103,7 @@ DEF("drive", HAS_ARG, QEMU_OPTION_drive,

[Qemu-devel] [PATCH v2 2/4] Clean-up a little bit the RW related bits of BDRV_O_FLAGS. BDRV_O_RDONLY gone (and so is BDRV_O_ACCESS). Default value for bdrv_flags (0/zero) is READ-ONLY. Need to explici

2010-01-17 Thread Naphtali Sprei
Instead of using the field 'readonly' of the BlockDriverState struct for passing the request, pass the request in the flags parameter to the function. Signed-off-by: Naphtali Sprei --- block.c | 31 +-- block.h |2 -- block/raw-posix.c |

[Qemu-devel] [PATCH v2 1/4] Make CDROM a read-only drive

2010-01-17 Thread Naphtali Sprei
Signed-off-by: Naphtali Sprei --- vl.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/vl.c b/vl.c index 06cb40d..76ef8ca 100644 --- a/vl.c +++ b/vl.c @@ -2233,6 +2233,13 @@ DriveInfo *drive_init(QemuOpts *opts, void *opaque, } (void)bdrv_set_read

[Qemu-devel] [PATCH v2 0/4] Modifications to the drives' readonly attribute

2010-01-17 Thread Naphtali Sprei
This is version 2, to replace previous set. Addresses all Kevin comments. Naphtali Sprei (4): Make CDROM a read-only drive Clean-up a little bit the RW related bits of BDRV_O_FLAGS. BDRV_O_RDONLY gone (and so is BDRV_O_ACCESS). Default value for bdrv_flags (0/zero) is READ-ONLY. Need

[Qemu-devel] Re: [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Måns Rullgård
Stefan Weil writes: > Måns Rullgård schrieb: >> Loïc Minier writes: >> >> >>> Hi >>> >>> On systems were sdl-config isn't installed, ./configure triggers this >>> warning: >>> ./configure: 957: sdl-config: not found >>> >>> The attached patch fixes the warning for me. >>> >>>

Re: [Qemu-devel] Re: [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Stefan Weil
Måns Rullgård schrieb: > Loïc Minier writes: > > >> Hi >> >> On systems were sdl-config isn't installed, ./configure triggers this >> warning: >> ./configure: 957: sdl-config: not found >> >> The attached patch fixes the warning for me. >> >>Thanks, >> -- >> Loïc Minier >> >

Re: [Qemu-devel] [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Stefan Weil
Loïc Minier schrieb: > Hi > > On systems were sdl-config isn't installed, ./configure triggers this > warning: > ./configure: 957: sdl-config: not found > > The attached patch fixes the warning for me. > > Thanks, Hi, which version did you test? Git master has no sdl-config call at configure:957.

Re: [Qemu-devel] Re: [PATCH] rtl8139: fix clang reporting unused assignment of VLAN tagging data

2010-01-17 Thread Simon Horman
On Fri, Jan 15, 2010 at 08:08:53AM +0100, Paolo Bonzini wrote: > > >diff --git a/hw/rtl8139.c b/hw/rtl8139.c > >index 1f4f585..f04dd54 100644 > >--- a/hw/rtl8139.c > >+++ b/hw/rtl8139.c > >@@ -1909,6 +1909,7 @@ static int rtl8139_cplus_transmit_one(RTL8139State *s) > > > > cpu_physical_memory

[Qemu-devel] Re: [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Måns Rullgård
Loïc Minier writes: > Hi > > On systems were sdl-config isn't installed, ./configure triggers this > warning: > ./configure: 957: sdl-config: not found > > The attached patch fixes the warning for me. > >Thanks, > -- > Loïc Minier > > From 94876939db7f46cf8d920e289d0d4f929d3b7

[Qemu-devel] Re: New kvm-related qemu patch queue

2010-01-17 Thread Avi Kivity
On 01/11/2010 05:30 PM, Anthony Liguori wrote: On 01/10/2010 06:02 AM, Avi Kivity wrote: In order to improve qemu.git kvm integration quality wrt performance, features, and reliability Marcelo and I will begin to maintain a patch queue based on qemu.git containing kvm-related patches. We will

[Qemu-devel] [PATCH] Check for sdl-config before calling it

2010-01-17 Thread Loïc Minier
Hi On systems were sdl-config isn't installed, ./configure triggers this warning: ./configure: 957: sdl-config: not found The attached patch fixes the warning for me. Thanks, -- Loïc Minier >From 94876939db7f46cf8d920e289d0d4f929d3b7df1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q

[Qemu-devel] [PATCH] Add -static earlier to LDFLAGS for compile_prog()

2010-01-17 Thread Loïc Minier
Hi When configure qemu with --static, it might autodetect support for some features by looking at available shared libraries instead of static libraries. e.g. if you have libbluetooth.so, bluez support will be turned on, but the build will fail at link stage. Setting LDFLAGS earlier

[Qemu-devel] [PATCH] Fix missing symbols in .rela.plt sections

2010-01-17 Thread Loïc Minier
Hi there, Static builds of qemu on x86-64 (and probably i386) fail with: gcc -I/home/lool/git/savannah/qemu/slirp -Werror -m64 -Wold-style-definition -Wold-style-declaration -I. -I/home/lool/git/savannah/qemu -U_FORTIFY_SOURCE -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Ws

Re: [Qemu-devel] [PATCH] linux-user: adapt uname machine to emulated CPU

2010-01-17 Thread Loïc Minier
On Thu, Jan 14, 2010, Aurelien Jarno wrote: > This patch look ok, but is missing a Signed-off-by: Attaching a rebased patch with a Signed-off-by. [ I didn't retest, but it applied without any merge issue and there was only two (arch-specific) commits in linux-user since the previous version. ]

[Qemu-devel] Re: [PATCH] qemu-iotests: Test qemu-img rebase

2010-01-17 Thread Christoph Hellwig
Looks good, thanks!

[Qemu-devel] [PATCH] block: fix cache flushing in bdrv_commit

2010-01-17 Thread Christoph Hellwig
On Thu, Jan 14, 2010 at 11:21:15AM +0100, Kevin Wolf wrote: > Anthony, you seem to have missed the v2 patch that considered my review > comments. Can you please apply the diff between v1 and v2 on top? Here is the differences in patch form: --- From: Christoph Hellwig Subject: block: fix cache f

[Qemu-devel] Re: [PATCH] fix linux-user microblaze ELF_ARCH definition

2010-01-17 Thread Edgar E. Iglesias
On Sun, Jan 17, 2010 at 01:15:05AM -0500, Mike Frysinger wrote: > Signed-off-by: Mike Frysinger > --- > linux-user/elfload.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) Applied, thanks. > > diff --git a/linux-user/elfload.c b/linux-user/elfload.c > index 9b65005..abc5332 10