Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Stefan Hajnoczi
On Tue, Apr 5, 2011 at 7:03 PM, Lyu Mitnick mitnick@gmail.com wrote: Hello Stefan, Kevin I don't want to diversify my GSoC applications. I thought I should do my best of supporting fully VHD and add asynchronous io into block/vpc.c via coroutines. I noticed that there is splitting hard

Re: [Qemu-devel] Re: [PATCH 00/10] pci: pci_register_bar_simple

2011-04-06 Thread Avi Kivity
On 04/05/2011 09:06 PM, Blue Swirl wrote: The more complicated BARs cannot be described declaratively (at least without a lot of complicated infrastructure). They can switch from RAM to MMIO mappings at runtime, and have different sub-regions. Subregions should be possible, but I agree

[Qemu-devel] [PATCH v2] Fix integer overflow in block migration bandwidth calculation

2011-04-06 Thread Avishay Traeger
block_mig_state.reads is an int, and multiplying by BLOCK_SIZE yielded a negative number, resulting in a negative bandwidth (running on a 32-bit machine). Change order to avoid. Signed-off-by: Avishay Traeger avis...@il.ibm.com --- block-migration.c |2 +- 1 files changed, 1 insertions(+),

[Qemu-devel] Re: spice in kvm-autotest [was: Re: KVM call minutes for Apr 5]

2011-04-06 Thread Alon Levy
On Tue, Apr 05, 2011 at 03:03:00PM -0300, Lucas Meneghel Rodrigues wrote: On Tue, 2011-04-05 at 20:08 +0300, Alon Levy wrote: On Tue, Apr 05, 2011 at 01:27:48PM -0300, Lucas Meneghel Rodrigues wrote: On Tue, 2011-04-05 at 19:01 +0300, Alon Levy wrote: So I was basically talking about

Re: [Qemu-devel] Re: spice in kvm-autotest [was: Re: KVM call minutes for Apr 5]

2011-04-06 Thread Alon Levy
On Tue, Apr 05, 2011 at 03:32:35PM -0500, Anthony Liguori wrote: On 04/05/2011 01:44 PM, Cleber Rosa wrote: On 04/05/2011 03:25 PM, Lucas Meneghel Rodrigues wrote: On Tue, 2011-04-05 at 13:08 -0500, Anthony Liguori wrote: On 04/05/2011 12:08 PM, Alon Levy wrote: The thing about WHQL is that

Re: [Qemu-devel] [PATCH v2 3/3] raw-posix: Re-open host CD-ROM after media change

2011-04-06 Thread Amit Shah
On (Tue) 05 Apr 2011 [12:17:30], Avi Kivity wrote: Guest kernel bug: CDROM change event missed, so the the revalidate call isn't made, which causes stale data (like disc size) to be used on newer media. qemu bug: We don't handle the GET_EVENT_STATUS_NOTIFICATION command from guests (which

Re: [Qemu-devel] [PATCH] Minor spelling mistakes in hw/usb-ccid.c

2011-04-06 Thread Alon Levy
On Tue, Apr 05, 2011 at 07:54:04AM +1000, Brad Hards wrote: Hi, While looking at David Gibson's build-fix for hw/usb-ccid.c, I noticed a spello in a comment on the following (unchanged) line. This patch fixes that, and a couple of other spellos, in that file. ACK. Thanks. Brad -

[Qemu-devel] Re: [PATCH V3] floppy: save and restore DIR register

2011-04-06 Thread Juan Quintela
Jason Wang jasow...@redhat.com wrote: Paolo Bonzini writes: On 04/01/2011 08:22 AM, Jason Wang wrote: + +if (drive-bs == NULL) { +return 1; Is it okay to return 1 here? Have you tested the case when both the source and the target drives have no floppy?

Re: [Qemu-devel] [PATCH] cpu-all.h: Remove unnecessary target-specific ifdef for CPU_QuadU

2011-04-06 Thread Peter Maydell
On 5 April 2011 23:15, Laurent Vivier laur...@vivier.eu wrote: Le mardi 05 avril 2011 à 22:54 +0100, Peter Maydell a écrit : On 5 April 2011 22:48, Laurent Vivier laur...@vivier.eu wrote: Le lundi 04 avril 2011 à 12:09 +0100, Peter Maydell a écrit : -#if defined(TARGET_SPARC) ||

[Qemu-devel] Re: [PATCH V3] floppy: save and restore DIR register

2011-04-06 Thread Juan Quintela
Jason Wang jasow...@redhat.com wrote: We need to keep DIR register unchanged across migration, but currently it depends on the media_changed flags from block layer. Since we do not save/restore it and the bdrv_open() called in dest node may set the media_changed flag when trying to open floppy

[Qemu-devel] Re: [PATCH 4/5] NBD device: Separate out parsing configuration and opening sockets.

2011-04-06 Thread Kevin Wolf
Am 22.02.2011 16:44, schrieb Nick Thomas: We also change the way the file parameter is parsed so IPv6 IP addresses can be used, e.g.: drive=nbd:[::1]:5000 Signed-off-by: Nick Thomas n...@bytemark.co.uk --- block/nbd.c | 157 ++- 1

[Qemu-devel] [PATCH v4] floppy: save and restore DIR register

2011-04-06 Thread Jason Wang
We need to keep DIR register unchanged across migration, but currently it depends on the media_changed flags from block layer. Since we do not save/restore it and the bdrv_open() called in dest node may set the media_changed flag when trying to open floppy image, guest driver may think the floppy

[Qemu-devel] Re: [PATCH 2/6] trace: generalize the property concept in the trace-events file

2011-04-06 Thread Stefan Hajnoczi
On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: @@ -168,7 +168,7 @@ The st_change_trace_event_state() function can be used to enable or disable trac  events at runtime inside QEMU:     #include trace.h - + Please leave the trailing whitespace, it's a workaround for wiki

[Qemu-devel] Re: [PATCH 3/6] trace-state: always use the nop backend on events with the disable keyword

2011-04-06 Thread Stefan Hajnoczi
On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: Any event with the keyword/property disable generates an empty trace event using the nop backend, regardless of the current backend. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu ---  docs/tracing.txt  |    3 +++  

Re: [Qemu-devel] Re: [PATCH 00/10] pci: pci_register_bar_simple

2011-04-06 Thread Michael S. Tsirkin
On Tue, Apr 05, 2011 at 09:02:45PM +0300, Blue Swirl wrote: On Tue, Apr 5, 2011 at 12:26 AM, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Apr 05, 2011 at 12:05:08AM +0300, Blue Swirl wrote: On Mon, Apr 4, 2011 at 10:44 PM, Michael S. Tsirkin m...@redhat.com wrote: On Mon, Apr 04,

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Lyu Mitnick
Hello Stefan, splitting hard disk images feature depends on win32 file system. I am wondering whether it is suitable to implement in raw-win32.c?? My idea is as follows: (1) adding an array to store file descriptor related in BDRVRawState (2) detect all splitted files in raw_open(). ie. consider

[Qemu-devel] Re: [PATCH] NBD: Convert the NBD driver to use the AIO interface.

2011-04-06 Thread Kevin Wolf
Am 24.02.2011 17:49, schrieb Nick Thomas: This preserves the previous behaviour where the NBD server is unavailable or goes away during guest execution, but switches the NBD backend to present the AIO interface instead of the sync IO interface. We also split read write requests into 1 MiB

[Qemu-devel] Re: [PATCH 5/6] trace-state: [simple] add -trace events argument to control initial state

2011-04-06 Thread Stefan Hajnoczi
On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: When using the simple tracing backend, all events are in disabled state by default. The -trace events argument can be used to provide a file with a list of trace event names that will be enabled prior to starting execution. This

[Qemu-devel] Re: [PATCH 6/6] trace: enable all events

2011-04-06 Thread Stefan Hajnoczi
On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: Given that all events with programmatically-controlled state are disabled by default, we can delete the disable property from all events. As of now, the only backend that will generate high ammounts of (possibly) undesired output

[Qemu-devel] [Bug 749522] Re: qemu-system-arm reads wrong entry in L1 page table for cortex-a8

2011-04-06 Thread Peter Maydell
Why do you think this is wrong? The ARM ARM says that the physical address of the first level page table descriptor is (ignoring TTBCR.N) the concatenation of bits [31..14] of the TTBR with bits [31..20] of the MVA and two zero bits. In this code 'table' is initially the [31..14] bits of the TTBR,

[Qemu-devel] Re: [RFC][PATCH 0/6] trace-state: make the behaviour of disable consistent across all backends

2011-04-06 Thread Stefan Hajnoczi
On Tue, Apr 5, 2011 at 2:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: This patch defines the disable trace event state to always use the nop backend. As a side-effect, all events are now enabled (without disable) by default,

[Qemu-devel] Re: [PATCH v4] floppy: save and restore DIR register

2011-04-06 Thread Juan Quintela
Jason Wang jasow...@redhat.com wrote: We need to keep DIR register unchanged across migration, but currently it depends on the media_changed flags from block layer. Since we do not save/restore it and the bdrv_open() called in dest node may set the media_changed flag when trying to open floppy

[Qemu-devel] Re: [PATCH v4] floppy: save and restore DIR register

2011-04-06 Thread Paolo Bonzini
On 04/06/2011 02:13 PM, Juan Quintela wrote: Jason Wangjasow...@redhat.com wrote: We need to keep DIR register unchanged across migration, but currently it depends on the media_changed flags from block layer. Since we do not save/restore it and the bdrv_open() called in dest node may set the

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Stefan Hajnoczi
On Wed, Apr 6, 2011 at 12:13 PM, Lyu Mitnick mitnick@gmail.com wrote: Hello Stefan, splitting hard disk images feature depends on win32 file system. I am wondering whether it is suitable to implement in raw-win32.c?? No, raw-win32.c is not the right place to do this. If I create a split

[Qemu-devel] [Bug 752476] [NEW] monitor command mouse_button 1 moves mouse

2011-04-06 Thread Bernhard M. Wiedemann
Public bug reported: via the qemu -monitor interface, it is possible to move and click the mouse using mouse_move 2 1 mouse_button 1 but the mouse_button command always moves the mouse to (0,0) making it rather unusable to (auto-)trigger any widgets in the VM from the outside. Would be

[Qemu-devel] [PATCH 0/2] Correct transitions for cd change state

2011-04-06 Thread Amit Shah
Hello, These two patches fix the cd media size change bugs. The test scenario is: 1. create an iso image from a file 2. create a second iso image from a bigger file 3. mount 1st cd in guest 4. unmount it 5. change cd via qemu monitor 6. mount 2nd cd 7. copy file on cd to local disk -- shows

[Qemu-devel] [PATCH 1/2] cdrom: Allow the TEST_UNIT_READY command after a cdrom change

2011-04-06 Thread Amit Shah
We restrict the commands that a guest can send us after a cdrom change event. The current list includes REQUEST_SENSE and INQUIRY commands. Guests can also issue TEST_UNIT_READY to inquire for the status, so allow this command as well. Signed-off-by: Amit Shah amit.s...@redhat.com ---

[Qemu-devel] [PATCH 2/2] cdrom: Make disc change event visible to guests

2011-04-06 Thread Amit Shah
Commit 93c8cfd9e67a62711b86f4c93747566885eb7928 tried to send a 'no disc' event after a cdrom change so that guests notice a cd change event between two 'cd present' states. However, we don't go from 'cd present' - 'no cd' - 'cd present' as the SENSE_UNIT_ATTENTION sense_key is written over by

Re: [Qemu-devel] [PATCH] Minor spelling mistakes in hw/usb-ccid.c

2011-04-06 Thread Stefan Hajnoczi
On Wed, Apr 6, 2011 at 9:15 AM, Alon Levy al...@redhat.com wrote: On Tue, Apr 05, 2011 at 07:54:04AM +1000, Brad Hards wrote: Hi, While looking at David Gibson's build-fix for hw/usb-ccid.c, I noticed a spello in a comment on the following (unchanged) line. This patch fixes that, and a

[Qemu-devel] [PATCH] trivial: fix comment around qdev_init_nofail()

2011-04-06 Thread Michael Tokarev
Resending (previous attempt was at Wed, 30 Mar 2011). In previous life qdev_init_nofail() used to call hw_error() which did register dump and other scary things. Now it calls error_report() and does a regular exit(1). Fix the comment to match reality. Signed-off-by: Michael Tokarev

Re: [Qemu-devel] [PATCH 1/3] arm: basic support for ARMv4/ARMv4T emulation

2011-04-06 Thread Peter Maydell
On 4 April 2011 14:38, Dmitry Eremin-Solenikov dbarysh...@gmail.com wrote: Currently target-arm/ assumes at least ARMv5 core. Add support for handling also ARMv4/ARMv4T. This changes the following instructions: Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com Reviewed-by: Peter

[Qemu-devel] Re: [regression] configure: add opengl detection

2011-04-06 Thread Benjamin Poirier
On Mon, Apr 4, 2011 at 6:13 PM, Michael Walle mich...@walle.cc wrote: Hi Benjamin, Let me know if you need more info. what happens if you configure with ./configure --target-list=x86_64-softmmu --disable-opengl Works as usual. The problem I'm facing stems from linking to libGL and memory

Re: [Qemu-devel] Re: [PATCH 5/6] trace-state: [simple] add -trace events argument to control initial state

2011-04-06 Thread Lluís
Stefan Hajnoczi writes: +            if (len 1) {              /* skip empty lines */ +                line[len - 1] = '\0'; +                if (!st_change_trace_event_state(line, true)) { The build breaks when --enable-trace-backend != simple because this code is outside an #ifdef

[Qemu-devel] Re: [RFC][PATCH 0/6] trace-state: make the behaviour of disable consistent across all backends

2011-04-06 Thread Fabien Chouteau
On 04/06/2011 01:42 PM, Stefan Hajnoczi wrote: On Tue, Apr 5, 2011 at 2:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Mon, Apr 4, 2011 at 10:49 PM, Lluís xscr...@gmx.net wrote: This patch defines the disable trace event state to always use the nop backend. As a side-effect, all events

[Qemu-devel] How long does it normally take patches to make their way into git?

2011-04-06 Thread John Haxby
Hello, How long does it normally take patches to make their way into git? I only ask because [PATCH 1/2][REPOST] Introduce a new 'connected' xendev op called when Connected. [PATCH 2/2][REPOST] Move the xenfb pointer handler to the connected method were (reposted) just under a month ago

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Christoph Hellwig
On Wed, Apr 06, 2011 at 07:41:37AM +0100, Stefan Hajnoczi wrote: Yes, split images are not supported by block/vpc.c. I'm working on that feature for vmdk right now, which is quite interesting as vmdk really is two different format both claiming to be vmdk, of which one is simply a text file

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Kevin Wolf
Am 06.04.2011 17:06, schrieb Christoph Hellwig: On Wed, Apr 06, 2011 at 07:41:37AM +0100, Stefan Hajnoczi wrote: Yes, split images are not supported by block/vpc.c. I'm working on that feature for vmdk right now, which is quite interesting as vmdk really is two different format both claiming

[Qemu-devel] QEMU functionality for emulating Solaris 8 SPARC

2011-04-06 Thread Harris, Brit
Maybe I am not looking in the right place, but I was wondering what functionality has been tested or is known with running a QEMU emulated Solaris 8 SPARC OS on a Linux 64-bit intel platform. When I look at downloads I see a sparc-test-0.2.tar.gz QEMU disk image. I assume that running Solaris

[Qemu-devel] Re: [PATCH v4] floppy: save and restore DIR register

2011-04-06 Thread Kevin Wolf
Am 06.04.2011 12:34, schrieb Jason Wang: We need to keep DIR register unchanged across migration, but currently it depends on the media_changed flags from block layer. Since we do not save/restore it and the bdrv_open() called in dest node may set the media_changed flag when trying to open

Re: [Qemu-devel] QEMU functionality for emulating Solaris 8 SPARC

2011-04-06 Thread Artyom Tarasenko
On Wed, Apr 6, 2011 at 5:10 PM, Harris, Brit brit.har...@boeing.com wrote: Maybe I am not looking in the right place, but I was wondering what functionality has been tested or is known with running a QEMU emulated Solaris 8 SPARC OS on a Linux 64-bit intel platform. The kernel can be booted

[Qemu-devel] Re: [PATCH v2] Fix integer overflow in block migration bandwidth calculation

2011-04-06 Thread Kevin Wolf
Am 06.04.2011 09:45, schrieb Avishay Traeger: block_mig_state.reads is an int, and multiplying by BLOCK_SIZE yielded a negative number, resulting in a negative bandwidth (running on a 32-bit machine). Change order to avoid. Signed-off-by: Avishay Traeger avis...@il.ibm.com Thanks,

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Christoph Hellwig
On Wed, Apr 06, 2011 at 05:15:29PM +0200, Kevin Wolf wrote: Am 06.04.2011 17:06, schrieb Christoph Hellwig: On Wed, Apr 06, 2011 at 07:41:37AM +0100, Stefan Hajnoczi wrote: Yes, split images are not supported by block/vpc.c. I'm working on that feature for vmdk right now, which is quite

Re: [Qemu-devel] [PATCH V11 11/15] Introduce qemu_put_ram_ptr

2011-04-06 Thread John Baboval
This seems to leak entries on the map cache locked entries list. Patch below: Signed-off-by: John Babovaljbaboval@jbaboval-desktop.(none) --- exec.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/exec.c b/exec.c index 1f88d6a..f18cb7d 100644 --- a/exec.c +++

Re: [Qemu-devel] [PATCH 1/3] arm: basic support for ARMv4/ARMv4T emulation

2011-04-06 Thread Dmitry Eremin-Solenikov
On 4/6/11, Peter Maydell peter.mayd...@linaro.org wrote: On 4 April 2011 14:38, Dmitry Eremin-Solenikov dbarysh...@gmail.com wrote: Currently target-arm/ assumes at least ARMv5 core. Add support for handling also ARMv4/ARMv4T. This changes the following instructions: Signed-off-by: Dmitry

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Luiz Capitulino
On Mon, 04 Apr 2011 08:05:48 -0500 Anthony Liguori anth...@codemonkey.ws wrote: On 04/04/2011 07:54 AM, Avi Kivity wrote: On 04/04/2011 01:59 PM, Daniel P. Berrange wrote: Interesting that with HMP you need to specify a single CPU index, but with QMP it is injecting to all CPUs at once. Is

Re: [Qemu-devel] [PATCH 1/3] arm: basic support for ARMv4/ARMv4T emulation

2011-04-06 Thread Peter Maydell
On 6 April 2011 18:41, Dmitry Eremin-Solenikov dbarysh...@gmail.com wrote: On 4/6/11, Peter Maydell peter.mayd...@linaro.org wrote: On 4 April 2011 14:38, Dmitry Eremin-Solenikov dbarysh...@gmail.com wrote: Currently target-arm/ assumes at least ARMv5 core. Add support for handling also

Re: [Qemu-devel] [PATCH] Few new QMP features

2011-04-06 Thread Luiz Capitulino
On Tue, 22 Mar 2011 15:55:56 +0300 Dmitry Konishchev konishc...@gmail.com wrote: Hi! I use QEMU via QMP and I've discovered that for some tasks there is no proper way to do them via QMP. I've written few patches: I'm going to respond you without looking at the patches. * One of them modifies

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Luiz Capitulino
On Wed, 06 Apr 2011 13:03:37 -0500 Anthony Liguori anth...@codemonkey.ws wrote: On 04/06/2011 12:47 PM, Luiz Capitulino wrote: On Mon, 04 Apr 2011 08:05:48 -0500 Anthony Liguorianth...@codemonkey.ws wrote: On 04/04/2011 07:54 AM, Avi Kivity wrote: On 04/04/2011 01:59 PM, Daniel P.

Re: [Qemu-devel] GSoC: Xen.org project to implement PV driver frontend for SeaBIOS

2011-04-06 Thread Luiz Capitulino
On Fri, 1 Apr 2011 17:36:08 +0100 Ian Campbell ian.campb...@citrix.com wrote: Hi, (I was unsure what the appropriate mailing list for this was, apologies for the cross post) You may or may not be aware that Xen intends to switch to using SeaBIOS as part of switching to using upstream

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Anthony Liguori
On 04/06/2011 12:47 PM, Luiz Capitulino wrote: On Mon, 04 Apr 2011 08:05:48 -0500 Anthony Liguorianth...@codemonkey.ws wrote: On 04/04/2011 07:54 AM, Avi Kivity wrote: On 04/04/2011 01:59 PM, Daniel P. Berrange wrote: Interesting that with HMP you need to specify a single CPU index, but

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Jan Kiszka
On 2011-04-06 20:08, Luiz Capitulino wrote: On Wed, 06 Apr 2011 13:03:37 -0500 Anthony Liguori anth...@codemonkey.ws wrote: On 04/06/2011 12:47 PM, Luiz Capitulino wrote: On Mon, 04 Apr 2011 08:05:48 -0500 Anthony Liguorianth...@codemonkey.ws wrote: On 04/04/2011 07:54 AM, Avi Kivity

Re: [Qemu-devel] GSoC 2011: S3 Trio, AHCI

2011-04-06 Thread Luiz Capitulino
On Tue, 05 Apr 2011 15:36:12 +0200 Roland Elek elek.rol...@gmail.com wrote: Dear Qemu developers, First, I'd like to reintroduce myself, as my university and official duties prevented me from being active in the community since last year. I am Roland Elek, a student from Hungary, and a

[Qemu-devel] [PATCH] virtio-blk: fail unaligned requests

2011-04-06 Thread Christoph Hellwig
Like all block drivers virtio-blk should not allow small than block size granularity access. But given that the protocol specifies a byte unit length field we currently accept such requests, which cause qemu to abort() in lower layers. Add checks to the main read and write handlers to catch

[Qemu-devel] [RFC][PATCH v2 00/11] trace-state: make the behaviour of disable consistent across all backends

2011-04-06 Thread Lluís
This patch defines the disable trace event state to always use the nop backend. As a side-effect, all events are now enabled (without disable) by default, as all backends (except stderr) have programmatic support for dynamically (de)activating each trace event. In order to make this true, the

[Qemu-devel] [PATCH v2 04/11] trace: [trace-events] fix print formats in some events

2011-04-06 Thread Lluís
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- trace-events |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/trace-events b/trace-events index 06efdb7..9fcc3bb 100644 --- a/trace-events +++ b/trace-events @@ -253,8 +253,8 @@ disable leon3_set_irq(int intno) Set

[Qemu-devel] [PATCH v2 05/11] trace: [simple] minor code fixes on conditional compilation

2011-04-06 Thread Lluís
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- monitor.c |4 +--- qemu-config.c |4 ++-- simpletrace.h |7 --- vl.c |7 +-- 4 files changed, 8 insertions(+), 14 deletions(-) diff --git a/monitor.c b/monitor.c index 2ba1362..377424e 100644 ---

[Qemu-devel] [PATCH v2 07/11] trace-state: always use the nop backend on events with the disable keyword

2011-04-06 Thread Lluís
Any event with the keyword/property disable generates an empty trace event using the nop backend, regardless of the current backend. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 25 +++-- scripts/tracetool | 15 ++- 2 files

[Qemu-devel] [PATCH v2 10/11] trace-state: [stderr] add support for dynamically enabling/disabling events

2011-04-06 Thread Lluís
This includes all the control interfaces already provided by the simple backend (i.e., command line, programmatic and monitor). Signed-off-by: Fabien Chouteau chout...@adacore.com Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |8 +++- configure |3

[Qemu-devel] [PATCH v2 01/11] minor whitespace/indentation fixes

2011-04-06 Thread Lluís
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- configure | 30 +- hmp-commands.hx | 28 monitor.c | 14 ++-- vl.c| 64 --- 4 files changed, 68

[Qemu-devel] [PATCH v2 09/11] trace-state: [simple] add -trace events argument to control initial state

2011-04-06 Thread Lluís
When using the simple tracing backend, all events are in disabled state by default. The -trace events argument can be used to provide a file with a list of trace event names that will be enabled prior to starting execution. This saves the user from manually toggling event states through the

[Qemu-devel] [PATCH v2 02/11] docs/tracing.txt: minor documentation fixes

2011-04-06 Thread Lluís
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/docs/tracing.txt b/docs/tracing.txt index f15069c..34b0901 100644 --- a/docs/tracing.txt +++ b/docs/tracing.txt @@ -26,14 +26,14 @@ for

[Qemu-devel] [PATCH v2 08/11] trace-state: [simple] disable all trace points by default

2011-04-06 Thread Lluís
Note that this refers to the backend-specific state (whether the output must be generated), not the event disabled property (which always uses the nop backend). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool |9 ++--- trace-events |3 --- 2 files

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Luiz Capitulino
On Wed, 06 Apr 2011 20:17:47 +0200 Jan Kiszka jan.kis...@siemens.com wrote: On 2011-04-06 20:08, Luiz Capitulino wrote: On Wed, 06 Apr 2011 13:03:37 -0500 Anthony Liguori anth...@codemonkey.ws wrote: On 04/06/2011 12:47 PM, Luiz Capitulino wrote: On Mon, 04 Apr 2011 08:05:48 -0500

[Qemu-devel] [PATCH v2 03/11] trace: [ust] fix generation of 'trace.c' on events without args

2011-04-06 Thread Lluís
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/tracetool b/scripts/tracetool index 412f695..d88cb43 100755 --- a/scripts/tracetool +++ b/scripts/tracetool @@ -338,6 +338,7 @@

[Qemu-devel] [PATCH v2 06/11] trace: generalize the property concept in the trace-events file

2011-04-06 Thread Lluís
This adds/modifies the following functions: * get_name: Get _only_ the event name * has_property: Return whether an event has a property (keyword before the event name) Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt |6 ++-- scripts/tracetool | 73

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Peter Maydell
On 6 April 2011 20:00, Luiz Capitulino lcapitul...@redhat.com wrote: On Wed, 06 Apr 2011 20:17:47 +0200 Jan Kiszka jan.kis...@siemens.com wrote: HMP is currently x86-only, thus it's probably OK to model it after some PC feature (though I don't know if there aren't NMI buttons with BP-only

Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

2011-04-06 Thread Anthony Liguori
On 04/06/2011 02:27 PM, Peter Maydell wrote: Right, but honestly speaking, I don't know how this works for other arches. So, the best thing to do is to have a general design that can be used by any architecture. Of course that we can also add a new command later if needed. Well, I'm not sure

Re: [Qemu-devel] GSoC: Xen.org project to implement PV driver frontend for SeaBIOS

2011-04-06 Thread Ian Campbell
On Wed, 2011-04-06 at 19:13 +0100, Luiz Capitulino wrote: On Fri, 1 Apr 2011 17:36:08 +0100 Ian Campbell ian.campb...@citrix.com wrote: Hi, (I was unsure what the appropriate mailing list for this was, apologies for the cross post) You may or may not be aware that Xen intends to

Re: [Qemu-devel] How long does it normally take patches to make their way into git?

2011-04-06 Thread Stefan Hajnoczi
On Wed, Apr 6, 2011 at 3:59 PM, John Haxby john.ha...@oracle.com wrote: Hello, How long does it normally take patches to make their way into git?  I only ask because  [PATCH 1/2][REPOST] Introduce a new 'connected' xendev op called when Connected.  [PATCH 2/2][REPOST] Move the xenfb

Re: [Qemu-devel] Re: [PATCH 5/6] trace-state: [simple] add -trace events argument to control initial state

2011-04-06 Thread Stefan Hajnoczi
On Wed, Apr 6, 2011 at 3:15 PM, Lluís xscr...@gmx.net wrote: Stefan Hajnoczi writes: +            if (len 1) {              /* skip empty lines */ +                line[len - 1] = '\0'; +                if (!st_change_trace_event_state(line, true)) { The build breaks when

[Qemu-devel] [PATCH RFC 4/5] vhost: optimize out no-change assignment

2011-04-06 Thread Michael S. Tsirkin
Cirrus VGA (at least) calls register memory region with the same values again and again. The registration in vhost-net slows this a lot, optimize by checking that the same data is already registered. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/vhost.c | 54

[Qemu-devel] [PATCH RFC 0/5] kvm/vhost: enable durty logging during memory registration

2011-04-06 Thread Michael S. Tsirkin
Currently, vga cards that allocate vga ram, register it as regular ram, and then request dirty logging from kvm (which is required for this hack to function correctly). Both these operations involve memory slot update and flush in kvm and in vhost which is a slow operation. This was observed to

[Qemu-devel] [PATCH RFC 1/5] cpu: add set_memory flag to request dirty logging

2011-04-06 Thread Michael S. Tsirkin
Pass the flag to all cpu notifiers, doing nothing at this point. Will be used by follow-up patches. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- cpu-common.h | 22 +- exec.c | 14 -- hw/vhost.c |3 ++- kvm-all.c|3 ++- 4 files

Re: [Qemu-devel] [PATCH] trivial: fix comment around qdev_init_nofail()

2011-04-06 Thread Stefan Hajnoczi
On Wed, Apr 6, 2011 at 2:51 PM, Michael Tokarev m...@tls.msk.ru wrote: Resending (previous attempt was at Wed, 30 Mar 2011). In previous life qdev_init_nofail() used to call hw_error() which did register dump and other scary things.  Now it calls error_report() and does a regular exit(1).  

[Qemu-devel] [PATCH RFC 5/5] cirrus_vga: flag on-device ram for dirty logging

2011-04-06 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/cirrus_vga.c | 16 +++- 1 files changed, 11 insertions(+), 5 deletions(-) diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c index bdf4c8b..7212849 100644 --- a/hw/cirrus_vga.c +++ b/hw/cirrus_vga.c @@ -2489,7 +2489,9 @@ static

[Qemu-devel] [PATCH RFC 3/5] vhost: skip memory which needs dirty logging

2011-04-06 Thread Michael S. Tsirkin
vhost doesn't support write logging (except for migration), anyway. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/vhost.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/hw/vhost.c b/hw/vhost.c index dc3d0e2..257e3dd 100644 --- a/hw/vhost.c +++ b/hw/vhost.c

Re: [Qemu-devel] Re: [PATCH 5/6] trace-state: [simple] add -trace events argument to control initial state

2011-04-06 Thread Lluís
Stefan Hajnoczi writes: Hmmm... why don't simply conditionally call st_init (put it into an #ifdef) and remove the #else in simpletrace.h. I've looked at it and it's not called from anywhere else. The benefit to stubbing out these functions is that callers don't have #ifdefs. And caller

[Qemu-devel] Info Request!

2011-04-06 Thread Ariel Portilla
To whom it Might Concern. I am interested in hiring VPS services in a windows supported platform, the requirements are minimal, I just want to install a mass mailer software for marketing campains with proved custumers and remotely administrate it. If you provide this services please send me

[Qemu-devel] [PATCH] linux-user: improve traces

2011-04-06 Thread Laurent Vivier
Add trace details for getpid(), kill(), _llseek(), rt_sigaction(), rt_sigprocmask(), clone(). Signed-off-by: Laurent Vivier laur...@vivier.eu --- linux-user/strace.c| 161 linux-user/strace.list | 12 ++-- 2 files changed, 167

Re: [Qemu-devel] GSoC 2011: S3 Trio, AHCI

2011-04-06 Thread Paul Brook
Last year, I was also interested in working on S3 Trio emulation. This year, the same idea is on the ideas list. The hardware is pretty thoroughly documented through source code and textual documentation, and I'm already familiar with adding PCI devices to Qemu, so I do see a rough

Re: [Qemu-devel] GSoC - QCOW2 - QED image converter

2011-04-06 Thread Lyu Mitnick
Hello Christoph, 2011/4/6 Christoph Hellwig h...@lst.de On Wed, Apr 06, 2011 at 05:15:29PM +0200, Kevin Wolf wrote: Am 06.04.2011 17:06, schrieb Christoph Hellwig: On Wed, Apr 06, 2011 at 07:41:37AM +0100, Stefan Hajnoczi wrote: Yes, split images are not supported by block/vpc.c.

[Qemu-devel] [PATCH] hw: improve multiboot module loading

2011-04-06 Thread ralf
Multiboot modules couldn't be loaded when there are spaces between the filename and ','. Those spaces can simply be killed. Signed-off-by: --- diff --git a/hw/multiboot.c b/hw/multiboot.c index 0d2bfb4..27eb159 100644 --- a/hw/multiboot.c +++ b/hw/multiboot.c @@ -267,6 +267,9 @@ int

Re: [Qemu-devel] [PATCH] hw: improve multiboot module loading

2011-04-06 Thread Alexander Graf
On 07.04.2011, at 02:19, r...@humppa.name wrote: Multiboot modules couldn't be loaded when there are spaces between the filename and ','. Those spaces can simply be killed. Signed-off-by: --- diff --git a/hw/multiboot.c b/hw/multiboot.c index 0d2bfb4..27eb159 100644 --- a/hw/multiboot.c

Re: [Qemu-devel] [PATCH] hw: improve multiboot module loading

2011-04-06 Thread Brad Hards
On Thu, 7 Apr 2011 10:19:01 am r...@humppa.name wrote: Signed-off-by: Looks like something may be wrong with your git config Brad

[Qemu-devel] [PATCH 3/5] virtio-9p: fixed LE-to-host conversion bug when QEMU is called from guest

2011-04-06 Thread David Gibson
From: Alexey Kardashevskiy a...@ozlabs.ru The 9p code already contains an attempt at the necessary endian conversions, but it's broken. The code which does conversion from host to guest does it correctly and this code was copied to the function which does guest to host conversion. However the

[Qemu-devel] [PATCH 1/5] ppce500_mpc8544ds: Fix compile with --enable-debug and --disable-kvm

2011-04-06 Thread David Gibson
From: Alexey Kardashevskiy a...@ozlabs.ru When configured with --enable-debug, the makefile does not use any optimization and compilation of hw/ppce500_mpc8544ds.c fails because gcc does not remove the never called without kvm function mpc8544_copy_soc_cell(), which will fail to link without the

[Qemu-devel] [PATCH 4/5] virtio-balloon: fixed endianness bug in the config space

2011-04-06 Thread David Gibson
From: Alexey Kardashevskiy a...@ozlabs.ru The specification for the virtio balloon device requres that the values in the config space be encoded little-endian. This differs from most virtio things, where guest-native endian is the norm. Currently, the qemu virtio-balloon code correctly makes

[Qemu-devel] Assorted trivial bugfixes

2011-04-06 Thread David Gibson
This series contains 5 essentially unrelated trivial bugfixes for qemu code. Most are aimed at eventually getting virtio devices working on the pseries target, the others are just annoying build bugs.

[Qemu-devel] [PATCH 5/5] virtio-serial: Fix endianness bug in the config space

2011-04-06 Thread David Gibson
From: Alexey Kardashevskiy a...@ozlabs.ru The virtio serial specification requres that the values in the config space are encoded in native endian of the guest. The qemu virtio-serial code did not do conversion to the guest endian format what caused problems when host and guest use different

[Qemu-devel] [PATCH 2/5] spapr_llan: Fix warning when compiled with -dDEBUG

2011-04-06 Thread David Gibson
From: Alexey Kardashevskiy a...@ozlabs.ru Compiling with the DEBUG macro causes leaves hw/spapr_llan.c with an unused variable, which is treated as an error in the qemu build. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Signed-off-by: David Gibson da...@gibson.dropbear.id.au ---

[Qemu-devel] [PATCH v2 0/2] Correct transitions for cd change state

2011-04-06 Thread Amit Shah
Hello, These two patches fix the cd media size change bugs. The test scenario is: 1. create an iso image from a file 2. create a second iso image from a bigger file 3. mount 1st cd in guest 4. unmount it 5. change cd via qemu monitor 6. mount 2nd cd 7. copy file on cd to local disk -- shows

[Qemu-devel] [PATCH v2 1/2] cdrom: Allow the TEST_UNIT_READY command after a cdrom change

2011-04-06 Thread Amit Shah
We restrict the commands that a guest can send us after a cdrom change event. The current list includes REQUEST_SENSE and INQUIRY commands. Guests can also issue TEST_UNIT_READY to inquire for the status, so allow this command as well. Signed-off-by: Amit Shah amit.s...@redhat.com ---

[Qemu-devel] [PATCH] configure: add --version flag

2011-04-06 Thread Mike Frysinger
Standard autoconf scripts include a --version flag so people can easily query things. Add this to qemu's configure so it too can integrate with build systems that have standard autotool helpers. Signed-off-by: Mike Frysinger vap...@gentoo.org --- configure |2 ++ 1 files changed, 2

[Qemu-devel] [PATCH v2 2/2] cdrom: Make disc change event visible to guests

2011-04-06 Thread Amit Shah
Commit 93c8cfd9e67a62711b86f4c93747566885eb7928 tried to send a 'no disc' event after a cdrom change so that guests notice a cd change event between two 'cd present' states. However, we don't go from 'cd present' - 'no cd' - 'cd present' as the SENSE_UNIT_ATTENTION sense_key is written over by

Re: [Qemu-devel] [PATCH RFC 0/5] kvm/vhost: enable durty logging during memory registration

2011-04-06 Thread Brad Hards
On Thu, 7 Apr 2011 06:41:35 am Michael S. Tsirkin wrote: As a solution, this adds an explicit flag that will enable dirty logging directly when registering the ram. kvm then needs a single system call to update tables for vga ram, vhost-net can simply ignore it. I'm not very familiar with QEMU