[Qemu-devel] [questions] savevm|loadvm

2010-03-29 Thread Wenhao Xu
Hi, all, I am working with switching QEMU from running in KVM mode to QEMU emulatoin mode dynamically. Intuitively, if the snapshot created using savevm in kvm mode can be used by the loadvm command in QEMU emulator mode, the switchment could makes use of this. I tried to do so. However, it

[Qemu-devel] Re: [PATCH 4/4] Add virtio disk identification support

2010-03-29 Thread john cooper
Rusty Russell wrote: > On Thu, 25 Mar 2010 04:04:02 pm john cooper wrote: >> Return serial string to the guest application via >> ioctl driver call. > > This is quite nice. Minor nits: > >> +if (cmd == 'VBID') { >> +void *usr_data = (void __user *)data; > > void __user *usr_data

[Qemu-devel] KVM call agenda for Mar 30

2010-03-29 Thread Chris Wright
- vhost-blk Please send in any additional agenda items you are interested in covering. thanks, -chris

Re: [Qemu-devel] GSoC projects about AHCI and S3 Trio

2010-03-29 Thread Natalia Portillo
Hi, El 30/03/2010, a las 01:55, Alexander Graf escribió: Hi Roland, On 30.03.2010, at 01:52, Roland Elek wrote: Dear Qemu developers, I am a university student from Hungary interested in contributing to Qemu through Google Summer of Code. I am interested in emulation, and two projects fr

[Qemu-devel] Re: [PATCH 4/4] Add virtio disk identification support

2010-03-29 Thread Rusty Russell
On Thu, 25 Mar 2010 04:04:02 pm john cooper wrote: > Return serial string to the guest application via > ioctl driver call. This is quite nice. Minor nits: > + if (cmd == 'VBID') { > + void *usr_data = (void __user *)data; void __user *usr_data; > + char *id_str; >

Re: [Qemu-devel] Re: [PATCH 1/5] linux-user/ia64: workaround ia64 strangenesses

2010-03-29 Thread Aurelien Jarno
On Tue, Mar 30, 2010 at 01:00:39AM +0100, Jamie Lokier wrote: > Aurelien Jarno wrote: > > On Mon, Mar 29, 2010 at 11:36:50AM +0200, Paolo Bonzini wrote: > > > > > >> +#ifdef __ia64 > > >> +sigprocmask(SIG_SETMASK, (sigset_t *)&uc->uc_sigmask, NULL); > > >> +#else > > >> sigprocmas

[Qemu-devel] about porting a new device

2010-03-29 Thread daniel tian
Hi, guys: I wanna port framebuffer LCD device on mips machine. And I wrote the code based on malta machine in qemu-0.11.1. The lcd device is simple, which has a fixed Framebuffer address and always 32bit bpp. Of course this is my imaginary graphic card. and qemu will deplay the frambebuff

Re: [Qemu-devel] [PATCH] usb-bus: fix no params

2010-03-29 Thread TeLeMan
On Mon, Mar 29, 2010 at 20:16, Kevin Wolf wrote: > Am 27.03.2010 13:47, schrieb Aurelien Jarno: >> On Fri, Mar 19, 2010 at 12:59:24PM +0800, TeLeMan wrote: >>> The "params" is never NULL and the usb hid devices have no params. >> >> This looks plainly wrong. With your patch, usb devices which don'

[Qemu-devel] [PATCHv2] usb-bus: fix no params

2010-03-29 Thread TeLeMan
After commit 702f3e0fb52c124c07f215426eeadb70a716643f, the params is nerver NULL. It should check *params instead of params to determine whether the params is empty. Signed-off-by: TeLeMan --- hw/usb-bus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/usb-bus.c b/h

Re: [Qemu-devel] GSoC projects about AHCI and S3 Trio

2010-03-29 Thread Alexander Graf
Hi Roland, On 30.03.2010, at 01:52, Roland Elek wrote: > Dear Qemu developers, > > I am a university student from Hungary interested in contributing to Qemu > through Google Summer of Code. > I am interested in emulation, and two projects from the ideas page in > particular. One of them is AH

Re: [Qemu-devel] Re: [PATCH 1/5] linux-user/ia64: workaround ia64 strangenesses

2010-03-29 Thread Jamie Lokier
Aurelien Jarno wrote: > On Mon, Mar 29, 2010 at 11:36:50AM +0200, Paolo Bonzini wrote: > > > >> +#ifdef __ia64 > >> +sigprocmask(SIG_SETMASK, (sigset_t *)&uc->uc_sigmask, NULL); > >> +#else > >> sigprocmask(SIG_SETMASK,&uc->uc_sigmask, NULL); > >> +#endif > > > > Any reason for th

[Qemu-devel] GSoC projects about AHCI and S3 Trio

2010-03-29 Thread Roland Elek
Dear Qemu developers, I am a university student from Hungary interested in contributing to Qemu through Google Summer of Code. I am interested in emulation, and two projects from the ideas page in particular. One of them is AHCI emulation. Can I kindly ask you what were the hardest points that mad

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread Anthony Liguori
On 03/29/2010 03:54 PM, Avi Kivity wrote: On 03/29/2010 11:42 PM, Anthony Liguori wrote: For individual device models or host services, I think (3) is probably the worst model overall. I personally think that (1) is better in the long run but ultimately would need an existence proof to compar

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread jvrao
Anthony Liguori wrote: > On 03/29/2010 03:31 PM, Avi Kivity wrote: >> On 03/29/2010 06:00 PM, Anthony Liguori wrote: >>> >>> In qemu, we tend to prefer state-machine based code. >>> >>> There are a few different models that we have discussed at different >>> points: >>> >>> 1) state machines with a

[Qemu-devel] Re: [PATCH v3 1/1] Shared memory uio_pci driver

2010-03-29 Thread Avi Kivity
On 03/28/2010 10:48 PM, Cam Macdonell wrote: On Sat, Mar 27, 2010 at 11:48 AM, Avi Kivity wrote: On 03/26/2010 07:14 PM, Cam Macdonell wrote: I'm not familiar with the uio internals, but for the interface, an ioctl() on the fd to assign an eventfd to an MSI vector. Similar t

Re: [Qemu-devel] Qemu-mips64

2010-03-29 Thread Stefan Weil
EHSAN UL HAQ schrieb: > Hi, > I have an executable file for mips64,how can i run this file using Qemu?? > Thanks, There is currently no user mode emulation for mips64, so you cannot run it directly. If your executable is a Linux executable, you might run it in a malta mips64 system emulation runn

[Qemu-devel] accessing acpi (mcfg) table from qemu

2010-03-29 Thread sanjay chandra
Hi, I need to access mcfg table from qemu, please provide some pointers how I can access the table. Thanks, Sanjay

[Qemu-devel] Qemu-mips64

2010-03-29 Thread EHSAN UL HAQ
Hi, I have an executable file for mips64,how can i run this file using Qemu?? Thanks, _ Hotmail: Trusted email with Microsoft’s powerful SPAM protection. https://signup.live.com/signup.aspx?

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread Avi Kivity
On 03/29/2010 11:42 PM, Anthony Liguori wrote: For individual device models or host services, I think (3) is probably the worst model overall. I personally think that (1) is better in the long run but ultimately would need an existence proof to compare against (2). (2) looks appealing until y

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread Anthony Liguori
On 03/29/2010 03:31 PM, Avi Kivity wrote: On 03/29/2010 06:00 PM, Anthony Liguori wrote: In qemu, we tend to prefer state-machine based code. There are a few different models that we have discussed at different points: 1) state machines with a thread pool to make blocking functions asynchr

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread Avi Kivity
On 03/29/2010 06:00 PM, Anthony Liguori wrote: In qemu, we tend to prefer state-machine based code. There are a few different models that we have discussed at different points: 1) state machines with a thread pool to make blocking functions asynchronous (what we have today) 2) co-operativ

[Qemu-devel] Re: [PATCH v2 0/4] monitor: Convert do_set_link() to QObject, QError

2010-03-29 Thread Luiz Capitulino
On Fri, 26 Mar 2010 09:07:07 +0100 Markus Armbruster wrote: > PATCH 3/4 changes syntax of set_link's second argument from up|down to > on|off. I feel that the argument needs to be boolean in QMP, and this > is the simplest way to get it. Luiz likes this approach. The change > doesn't affect li

Re: [Qemu-devel] Check format specifiers for fprintf like function pointers

2010-03-29 Thread Stefan Weil
Stefan Weil schrieb: > fprintf like function pointers are used for log output, especially > for cpu / fpu register dumps. > > When I examined wrong values for an x86_64 target on a i386 host, > I noticed that it was caused by wrong format specifiers and that > there are more errors of this kind. >

[Qemu-devel] [PATCH 14/14] tcg: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- tcg/tcg.c |6 ++ tcg/tcg.h |3 +-- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index 3294743..a40b797 100644 --- a/tcg/tcg.c +++ b/tcg/tcg.c @@ -2087,8 +2087,7 @@ int tcg_gen_code_search_pc(TCGContext *s, uint8_t

[Qemu-devel] [PATCH 13/14] target-s390: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-s390x/translate.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-s390x/translate.c b/target-s390x/translate.c index 44dfa65..b4b5537 100644 --- a/target-s390x/translate.c +++ b/target-s390x/translate.c @@ -24,7 +24,7 @@ #inc

[Qemu-devel] [PATCH 12/14] target-sparc: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-sparc/helper.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/target-sparc/helper.c b/target-sparc/helper.c index 1f0f7d4..d1914d9 100644 --- a/target-sparc/helper.c +++ b/target-sparc/helper.c @@ -1261,7 +1261,7 @@ static con

[Qemu-devel] [PATCH 11/14] target-sh4: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-sh4/cpu.h |2 +- target-sh4/translate.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/target-sh4/cpu.h b/target-sh4/cpu.h index f8b1680..78602af 100644 --- a/target-sh4/cpu.h +++ b/target-sh4/cpu.h @@ -168,7 +168,7 @@ int

[Qemu-devel] [PATCH 10/14] target-ppc: Use fprintf_function and fix wrong format specifiers

2010-03-29 Thread Stefan Weil
* The register dump was wrong for XER register. * cpu_ppc_load_tbl returns uint64_t, so use PRIx64. * Print space before DECR, but not at end of line. Signed-off-by: Stefan Weil --- target-ppc/cpu.h|2 +- target-ppc/translate.c | 12 ++-- target-ppc/translate_init.

[Qemu-devel] [PATCH 09/14] target-mips: Use fprintf_function and fix wrong format specifiers

2010-03-29 Thread Stefan Weil
Removed unused declaration of fpu_dump_state. These compiler warnings still have to be fixed: cc1: warnings being treated as errors /x/target-mips/translate.c: In function 'fpu_dump_state': /x/target-mips/translate.c:9585: error: format '%08x' expects type 'unsigned int', but argument 6 has type

[Qemu-devel] [PATCH 08/14] target-microblaze: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-microblaze/translate.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-microblaze/translate.c b/target-microblaze/translate.c index ca54e2c..8b6a184 100644 --- a/target-microblaze/translate.c +++ b/target-microblaze/translate.

[Qemu-devel] [PATCH 07/14] target-m68k: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-m68k/cpu.h |2 +- target-m68k/helper.c|2 +- target-m68k/translate.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-m68k/cpu.h b/target-m68k/cpu.h index b2f37ec..27d4e8f 100644 --- a/target-m68k/cpu.h +++ b/

[Qemu-devel] [PATCH 06/14] target-i386: Use fprintf_function and fix dump of DR registers

2010-03-29 Thread Stefan Weil
The dump was wrong for 32 bit hosts with 64 bit target. Signed-off-by: Stefan Weil --- target-i386/cpu.h|3 +-- target-i386/cpuid.c |3 +-- target-i386/helper.c | 12 +++- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/target-i386/cpu.h b/target-i386/cpu.h

[Qemu-devel] [PATCH 05/14] target-cris: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-cris/cpu.h |2 +- target-cris/translate.c |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-cris/cpu.h b/target-cris/cpu.h index 063a240..a45870b 100644 --- a/target-cris/cpu.h +++ b/target-cris/cpu.h @@ -266,6 +266,

[Qemu-devel] [PATCH 04/14] target-arm: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-arm/cpu.h |2 +- target-arm/helper.c|2 +- target-arm/translate.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-arm/cpu.h b/target-arm/cpu.h index 3892db4..7c9d5ca 100644 --- a/target-arm/cpu.h +++ b/target

[Qemu-devel] [PATCH 03/14] target-alpha: Use fprintf_function

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- target-alpha/helper.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-alpha/helper.c b/target-alpha/helper.c index 46335cd..0476066 100644 --- a/target-alpha/helper.c +++ b/target-alpha/helper.c @@ -537,7 +537,7 @@ void do_interrupt

[Qemu-devel] [PATCH 02/14] Use fprint_function and fix wrong format specifiers

2010-03-29 Thread Stefan Weil
Signed-off-by: Stefan Weil --- exec.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/exec.c b/exec.c index 0916208..6be15c4 100644 --- a/exec.c +++ b/exec.c @@ -3964,8 +3964,7 @@ void cpu_io_recompile(CPUState *env, void *retaddr) #if !defined(CONFIG_USER_ONL

[Qemu-devel] [PATCH 01/14] Add new data type for fprintf like function pointers

2010-03-29 Thread Stefan Weil
The compiler should check the arguments for these functions. gcc can do this, but only if the function pointer's prototype includes the __attribute__ flag. As the necessary declaration is a bit lengthy, we use a new data type 'fprintf_function'. It is not easy to find a single header file which

[Qemu-devel] Check format specifiers for fprintf like function pointers

2010-03-29 Thread Stefan Weil
fprintf like function pointers are used for log output, especially for cpu / fpu register dumps. When I examined wrong values for an x86_64 target on a i386 host, I noticed that it was caused by wrong format specifiers and that there are more errors of this kind. I could fix some (see list below)

Re: [Qemu-devel] [PATCH] Elo touchpad 10 bytes emulator

2010-03-29 Thread Ricardo Ribalda Delgado
Hello Paul Hmmm, you are right, it should be part of the calibration (userland). As Dmtry says, the output from the chip should be from 96-4000. I have fixed the max values to that. The previous code was done to emulate an old privative app that uses an elo touchscreen. I managed to reverse engin

[Qemu-devel] [PATCH] Elo touchpad 10 bytes emulator v2

2010-03-29 Thread Ricardo Ribalda Delgado
New char device emulating an Elo serial touchpad. -Emulate id and touch packets -Absolute Output limited to 96-4000 --- Makefile.objs |2 +- hw/elo.c| 153 +++ hw/elo.h|2 + hw/serial.c |2 +- qemu-char.c |

Re: [Qemu-devel] [PATCH] Fix busted driftfix option

2010-03-29 Thread Blue Swirl
On 3/30/10, Zachary Amsden wrote: > On 03/26/10 23:14, Blue Swirl wrote: > > On 3/26/10, Zachary Amsden wrote: > > > >> For some reason, this uses CONFIG_TARGET_I386 instead of TARGET_I386, so > >> the code is dead. > >> > > The code is also broken: it references undefined variable 'buf' >

Re: [Qemu-devel] [PATCH] Fix busted driftfix option

2010-03-29 Thread Zachary Amsden
On 03/26/10 23:14, Blue Swirl wrote: > On 3/26/10, Zachary Amsden wrote: > >> For some reason, this uses CONFIG_TARGET_I386 instead of TARGET_I386, so >> the code is dead. >> > The code is also broken: it references undefined variable 'buf' > instead of 'value'. > Sorry, that wasn't t

Re: [Qemu-devel] [PATCH] Elo touchpad 10 bytes emulator

2010-03-29 Thread Dmitry Zhurikhin
Ricardo Ribalda Delgado wrote: TODO: The output of the touchpad should be in the range of the resolution. But I don't know a clean way to get the screen resolution. Any help will be very wellcomed Hello. Looking at the Linux kernel driver it seems to consider reported coordinates to be in the r

Re: [Qemu-devel] Re: [PATCH 1/5] linux-user/ia64: workaround ia64 strangenesses

2010-03-29 Thread Aurelien Jarno
On Mon, Mar 29, 2010 at 11:36:50AM +0200, Paolo Bonzini wrote: > >> +#ifdef __ia64 >> +sigprocmask(SIG_SETMASK, (sigset_t *)&uc->uc_sigmask, NULL); >> +#else >> sigprocmask(SIG_SETMASK,&uc->uc_sigmask, NULL); >> +#endif > > Any reason for the ifdef? > It is not strictly needed, a

[Qemu-devel] [PATCH 01/10] Elo touchpad 10 bytes emulator

2010-03-29 Thread Ricardo Ribalda Delgado
From: Ricardo Ribalda Delgado New char device emulating an Elo serial touchpad. TODO: The output of the touchpad should be in the range of the resolution. But I don't know a clean way to get the screen resolution. Any help will be very wellcomed --- Please, be nice it is my first patch to the

Re: [Qemu-devel] [PATCH] Elo touchpad 10 bytes emulator

2010-03-29 Thread Paul Brook
> New char device emulating an Elo serial touchpad. > > TODO: The output of the touchpad should be in the range of the > resolution. But I don't know a clean way to get the screen resolution. > Any help will be very wellcomed Are you sure? I don't see how real hardware would be able to do that.

[Qemu-devel] [PATCH] Elo touchpad 10 bytes emulator

2010-03-29 Thread Ricardo Ribalda Delgado
New char device emulating an Elo serial touchpad. TODO: The output of the touchpad should be in the range of the resolution. But I don't know a clean way to get the screen resolution. Any help will be very wellcomed --- Please. Ignore the old 01/10 Makefile.objs |2 +- hw/elo.c|

Re: [Qemu-devel] [PATCH -V3 09/32] virtio-9p: Implement P9_TWRITE/ Thread model in QEMU

2010-03-29 Thread Anthony Liguori
On 03/29/2010 01:36 AM, jvrao wrote: Aneesh Kumar K.V wrote: From: Anthony Liguori We have implemented all the vfs calls in state machine model so that we are prepared for the model where the VCPU thread(s) does the initial work until it needs to block then it submits that work (via

[Qemu-devel] [PATCH v2 1/7] qemu-config: qemu_read_config_file() reads the normal config file

2010-03-29 Thread Kevin Wolf
Introduce a new function qemu_read_config_file which reads the VM configuration from a config file. Unlike qemu_config_parse it doesn't take a open file but a filename and reduces code duplication as a side effect. Signed-off-by: Kevin Wolf --- qemu-config.c | 15 +++ qemu-config.h

[Qemu-devel] [PATCH v2 4/7] blkdebug: Inject errors

2010-03-29 Thread Kevin Wolf
Add a mechanism to inject errors instead of passing requests on. With no further patches applied, you can use it by setting inject_errno in gdb. Signed-off-by: Kevin Wolf --- block/blkdebug.c | 81 ++ 1 files changed, 81 insertions(+), 0 dele

[Qemu-devel] [PATCH v2 2/7] qemu-config: Make qemu_config_parse more generic

2010-03-29 Thread Kevin Wolf
qemu_config_parse gets the option groups as a parameter now instead of hardcoding the VM configuration groups. This way it can be used for other configurations, too. Signed-off-by: Kevin Wolf --- qemu-config.c | 18 -- qemu-config.h |2 +- 2 files changed, 13 insertions(+),

[Qemu-devel] [PATCH v2 3/7] blkdebug: Basic request passthrough

2010-03-29 Thread Kevin Wolf
This isn't doing anything interesting. It creates the blkdebug block driver as a protocol which just passes everything through to raw. Signed-off-by: Kevin Wolf --- Makefile.objs|2 +- block/blkdebug.c | 104 ++ 2 files changed, 105 in

[Qemu-devel] [PATCH v2 0/7] blkdebug

2010-03-29 Thread Kevin Wolf
This patch series introduces a new block driver which acts as a protocol and whose purpose it is to fail requests. To be more precise, I want it to fail in configurable places, so that qemu-iotests can be extended with tests for the error paths (for example for the case when something with metadata

[Qemu-devel] [PATCH v2 5/7] Make qemu-config available for tools

2010-03-29 Thread Kevin Wolf
To be able to use config files for blkdebug, we need to make these functions available in the tools. This involves moving two functions that can only be built in the context of the emulator. Signed-off-by: Kevin Wolf --- Makefile.objs|4 ++-- hw/qdev-properties.c | 19 +

[Qemu-devel] [PATCH v2 6/7] blkdebug: Add events and rules

2010-03-29 Thread Kevin Wolf
Block drivers can trigger a blkdebug event whenever they reach a place where it could be useful to inject an error for testing/debugging purposes. Rules are read from a blkdebug config file and describe which action is taken when an event is triggered. For now this is only injecting an error (with

[Qemu-devel] [PATCH v2 7/7] qcow2: Trigger blkdebug events

2010-03-29 Thread Kevin Wolf
This adds blkdebug events to qcow2 to allow injecting I/O errors in specific places. Signed-off-by: Kevin Wolf --- block.h| 44 block/blkdebug.c | 42 ++ block/qcow2-cluster.c | 15 +

Re: [Qemu-devel] [PATCH -v2 02/22] vrtio-9p: Implement P9_TVERSION for 9P

2010-03-29 Thread Anthony Liguori
On 03/29/2010 02:01 AM, Aneesh Kumar K. V wrote: On Fri, 26 Mar 2010 11:15:47 -0500, Anthony Liguori wrote: On 03/16/2010 04:15 AM, Aneesh Kumar K.V wrote: From: Anthony Liguori [ki...@linux.vnet.ibm.com: malloc to qemu_malloc coversion] Signed-off-by: Anthony Liguori Signed-off-b

Re: [Qemu-devel] QEMU 0.12.3 and SCSI boot

2010-03-29 Thread Gerd Hoffmann
On 03/29/10 15:51, Kevin Wolf wrote: It actually searches the queue in case tag != s->current->tag, and it should most likely do the same for s->current == NULL ... Attached patch makes the rom boot for me. Yes, works for me. And it seems to work reliably, unlike the 0.12.x version. Oh. The

Re: [Qemu-devel] QEMU 0.12.3 and SCSI boot

2010-03-29 Thread Kevin Wolf
Am 29.03.2010 15:41, schrieb Gerd Hoffmann: > >> Tried the same with current git master and it segfaults. This segfault >> was introduced in af12ac98 (lsi: have lsi_request for the whole life >> time of the request): >> >> #0 0x0052e2d3 in lsi_command_complete (bus=0xca22f8, reason=1, >>

[Qemu-devel] [PATCH] lsi: fix segfault in lsi_command_complete

2010-03-29 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- hw/lsi53c895a.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index a332401..525f3ca 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -679,7 +679,7 @@ static void lsi_command_complete(SCSIBus *bus

Re: [Qemu-devel] QEMU 0.12.3 and SCSI boot

2010-03-29 Thread Gerd Hoffmann
Tried the same with current git master and it segfaults. This segfault was introduced in af12ac98 (lsi: have lsi_request for the whole life time of the request): #0 0x0052e2d3 in lsi_command_complete (bus=0xca22f8, reason=1, tag=0, arg=512) at /home/kwolf/source/qemu/hw/lsi53c895a.c:69

Re: [Qemu-devel] Re: [PATCH 10/15] virtio-serial: Add QMP events for failed port/device add

2010-03-29 Thread Luiz Capitulino
On Sat, 27 Mar 2010 13:33:22 +0530 Amit Shah wrote: > On (Fri) Mar 26 2010 [14:52:36], Luiz Capitulino wrote: > > > > My suggestion for the immediate term is to do what we have been doing > > > > so > > > > far, ie. call it VIRTIO_SERIAL_ADD. Worst case here is: we add a new way > > > > to grou

[Qemu-devel] Re: [PATCH 3/3] monitor: Convert do_migrate() to QError

2010-03-29 Thread Luiz Capitulino
On Mon, 29 Mar 2010 14:38:35 +0200 Markus Armbruster wrote: > Luiz Capitulino writes: > > > On Thu, 25 Mar 2010 20:30:33 +0100 > > Markus Armbruster wrote: > > > >> Luiz Capitulino writes: > >> > >> > On Thu, 25 Mar 2010 18:37:25 +0100 > >> > Markus Armbruster wrote: > >> > > >> > [...] > >

[Qemu-devel] [PATCH] smc91c111: allow access to reserved register

2010-03-29 Thread Lars Munch
Some drivers seems to access the reserved register in bank 0 so allow and ignore these accesses. Signed-off-by: Lars Munch --- hw/smc91c111.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/hw/smc91c111.c b/hw/smc91c111.c index a2ef299..e4a2447 100644 --- a/hw/smc91c1

[Qemu-devel] [PATCH] smc91c111: mask register offset

2010-03-29 Thread Lars Munch
this fixes the smc91c111 emulation which has been broken for gumstix and mainstone and maybe others since the "MMIO callback interface changes" 8da3ff180974732fc4272cb4433fef85c1822961 where commited, see: http://thread.gmane.org/gmane.comp.emulators.qemu/33607/focus=35194 Signed-off-by: Lars Mun

Re: [Qemu-devel] QEMU 0.12.3 and SCSI boot

2010-03-29 Thread Kevin Wolf
Am 27.03.2010 10:38, schrieb Gerhard Wiesinger: > Hello, > > I'm having trouble booting from SCSI adapter 53C895a and e.g. INT13h OS > like MS-DOS 6.22. > > I downloaded and installed the option ROM with -option-rom 8xx_64.rom: > http://www.lsi.com/DistributionSystem/AssetDocument/files/support/

[Qemu-devel] Re: [PATCH 3/3] monitor: Convert do_migrate() to QError

2010-03-29 Thread Markus Armbruster
Luiz Capitulino writes: > On Thu, 25 Mar 2010 20:30:33 +0100 > Markus Armbruster wrote: > >> Luiz Capitulino writes: >> >> > On Thu, 25 Mar 2010 18:37:25 +0100 >> > Markus Armbruster wrote: >> > >> > [...] >> > >> >> >> @@ -86,12 +86,13 @@ int do_migrate(Monitor *mon, const QDict *qdict, >>

Re: [Qemu-devel] [PATCH] usb-bus: fix no params

2010-03-29 Thread Kevin Wolf
Am 27.03.2010 13:47, schrieb Aurelien Jarno: > On Fri, Mar 19, 2010 at 12:59:24PM +0800, TeLeMan wrote: >> The "params" is never NULL and the usb hid devices have no params. > > This looks plainly wrong. With your patch, usb devices which don't > accept parameters, will accept and ignore them. >

Re: [Qemu-devel] [PATCH 3/3] qemu-nbd: Improve error reporting

2010-03-29 Thread Kevin Wolf
Am 28.03.2010 19:07, schrieb Ryota Ozaki: > - use err(3) instead of errx(3) if errno is available > to report why failed > - let fail prior to daemon(3) if opening a nbd file > is likely to fail after daemonizing to avoid silent > failure exit > - add missing 'ret = 1' when unix_socket_outgoi

Re: [Qemu-devel] [PATCH 2/3] qemu-nbd: Extend read-only option to nbd device file

2010-03-29 Thread Kevin Wolf
Am 28.03.2010 19:07, schrieb Ryota Ozaki: > This patch allows to operate on nbd device file > without write permission for the file if read-only > option is specified. > > Signed-off-by: Ryota Ozaki The help for -r should be changed, too. Currently it says: -r, --read-only export read-only

Re: [Qemu-devel] [PATCH 1/3] qemu-nbd: Fix coding style

2010-03-29 Thread Kevin Wolf
Am 28.03.2010 19:07, schrieb Ryota Ozaki: > Follow "Every indented statement is braced; even if the block > contains just one statement." described in CODING_STYLE. > > Signed-off-by: Ryota Ozaki Usually we fix the coding style in places that need to be changed anyway, but avoid touching things

Re: [Qemu-devel] [PATCH] qemu-img: add FUSE-based image access

2010-03-29 Thread Alexander Graf
On 29.03.2010, at 11:37, Jan Kiszka wrote: > Alexander Graf wrote: >> On 29.03.2010, at 09:46, Jan Kiszka wrote: >> >>> Christoph Hellwig wrote: On Thu, Mar 25, 2010 at 06:52:59PM +0100, Jan Kiszka wrote: > This adds the "map" subcommand to qemu-img. It is able to expose the raw > c

Re: [Qemu-devel] [PATCH] qemu-img: add FUSE-based image access

2010-03-29 Thread Jan Kiszka
Alexander Graf wrote: > On 29.03.2010, at 09:46, Jan Kiszka wrote: > >> Christoph Hellwig wrote: >>> On Thu, Mar 25, 2010 at 06:52:59PM +0100, Jan Kiszka wrote: This adds the "map" subcommand to qemu-img. It is able to expose the raw content of a disk image via a FUSE filesystem. Both th

[Qemu-devel] Re: [PATCH 1/5] linux-user/ia64: workaround ia64 strangenesses

2010-03-29 Thread Paolo Bonzini
+#ifdef __ia64 +sigprocmask(SIG_SETMASK, (sigset_t *)&uc->uc_sigmask, NULL); +#else sigprocmask(SIG_SETMASK,&uc->uc_sigmask, NULL); +#endif Any reason for the ifdef? Paolo

Re: [Qemu-devel] [PATCH] qemu-img: add FUSE-based image access

2010-03-29 Thread Alexander Graf
On 29.03.2010, at 09:46, Jan Kiszka wrote: > Christoph Hellwig wrote: >> On Thu, Mar 25, 2010 at 06:52:59PM +0100, Jan Kiszka wrote: >>> This adds the "map" subcommand to qemu-img. It is able to expose the raw >>> content of a disk image via a FUSE filesystem. Both the whole disk can >>> be acces

Re: [Qemu-devel] [RFC][PATCH 6/7] blkdebug: Add events and rules

2010-03-29 Thread Kevin Wolf
Am 28.03.2010 15:12, schrieb Christoph Hellwig: > On Mon, Mar 15, 2010 at 06:08:34PM +0100, Kevin Wolf wrote: >> +fprintf(stderr, "bdrv_debug_event: %d\n", event); > > Is this supposed to be in the final version or a leftover debugging aid? It's not there in the final version (which I have al

Re: [Qemu-devel] [PATCH] qemu-img: add FUSE-based image access

2010-03-29 Thread Jan Kiszka
Christoph Hellwig wrote: > On Thu, Mar 25, 2010 at 06:52:59PM +0100, Jan Kiszka wrote: >> This adds the "map" subcommand to qemu-img. It is able to expose the raw >> content of a disk image via a FUSE filesystem. Both the whole disk can >> be accessed, e.g. to run partitioning tools against it, as

Re: [Qemu-devel] Question about memory micro operations in Qemu 0.12.x

2010-03-29 Thread Alexander
Thank you for your answer. I still have some questions. 27.03.2010 12:49, Stuart Brady пишет: On Fri, Mar 26, 2010 at 11:23:30PM +0300, coo...@gmail.com wrote: Hello. in qemu 0.9.x there was a special file with micro-operations, which implemented access to memory. For example for arm archi

Re: [Qemu-devel] [PATCH -v2 02/22] vrtio-9p: Implement P9_TVERSION for 9P

2010-03-29 Thread Aneesh Kumar K. V
On Fri, 26 Mar 2010 11:15:47 -0500, Anthony Liguori wrote: > On 03/16/2010 04:15 AM, Aneesh Kumar K.V wrote: > > From: Anthony Liguori > > > > [ki...@linux.vnet.ibm.com: malloc to qemu_malloc coversion] > > > > Signed-off-by: Anthony Liguori > > Signed-off-by: Aneesh Kumar K.V > > --- > > hw/vi