[Qemu-devel] seabios for qemu 1.5

2013-04-24 Thread Gerd Hoffmann
Hi, b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd} 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning Anthony + Gerd, can we expect an rev to pc-bios/bios.bin soon to pick up these

Re: [Qemu-devel] seabios for qemu 1.5

2013-04-24 Thread Amos Kong
On Wed, Apr 24, 2013 at 07:59:36AM +0200, Gerd Hoffmann wrote: Hi, b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd} 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning Anthony +

Re: [Qemu-devel] [PATCH 1/2] cpu-exec: rid cs_base of TranslationBlock

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 03:48, liguang ha scritto: cs_base is only meaningful for target-i386/sparc, so, get rid of cs_base for other target This is really ugly, we're trying to get less target-dependent code outside target-*, not more. Also, please limit the number of people that you CC. Paolo

Re: [Qemu-devel] [PATCH v5 11/12] rdma: core logic

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 01:53, Michael R. Hines ha scritto: So, I started investigating the problem, and I noticed that whenever I disabled the limits of max_size, the throughput went back to the normal throughput (peak of 26 gbps). So, rather than change the default max_size calculation for TCP,

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread Peter Maydell
On 24 April 2013 02:48, liguang lig.f...@cn.fujitsu.com wrote: Signed-off-by: liguang lig.f...@cn.fujitsu.com I'm afraid this is definitely wrong. It has a less than helpful name, but cs_base is actually just another 32/64 bits of state that the target can use to distinguish translation blocks,

Re: [Qemu-devel] [PATCH 1/2] cpu-exec: rid cs_base of TranslationBlock

2013-04-24 Thread Aurelien Jarno
On Wed, Apr 24, 2013 at 08:36:54AM +0200, Paolo Bonzini wrote: Il 24/04/2013 03:48, liguang ha scritto: cs_base is only meaningful for target-i386/sparc, so, get rid of cs_base for other target This is really ugly, we're trying to get less target-dependent code outside target-*, not more.

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread li guang
在 2013-04-24三的 08:05 +0100,Peter Maydell写道: On 24 April 2013 02:48, liguang lig.f...@cn.fujitsu.com wrote: Signed-off-by: liguang lig.f...@cn.fujitsu.com I'm afraid this is definitely wrong. It has a less than helpful name, but cs_base is actually just another 32/64 bits of state that the

Re: [Qemu-devel] [PATCH 1/2] cpu-exec: rid cs_base of TranslationBlock

2013-04-24 Thread li guang
在 2013-04-24三的 09:11 +0200,Aurelien Jarno写道: On Wed, Apr 24, 2013 at 08:36:54AM +0200, Paolo Bonzini wrote: Il 24/04/2013 03:48, liguang ha scritto: cs_base is only meaningful for target-i386/sparc, so, get rid of cs_base for other target This is really ugly, we're trying to get less

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread Peter Maydell
On 24 April 2013 08:15, li guang lig.f...@cn.fujitsu.com wrote: 在 2013-04-24三的 08:05 +0100,Peter Maydell写道: On 24 April 2013 02:48, liguang lig.f...@cn.fujitsu.com wrote: Signed-off-by: liguang lig.f...@cn.fujitsu.com I'm afraid this is definitely wrong. It has a less than helpful name, but

Re: [Qemu-devel] [PATCH 1/2] cpu-exec: rid cs_base of TranslationBlock

2013-04-24 Thread Peter Maydell
On 24 April 2013 08:25, li guang lig.f...@cn.fujitsu.com wrote: 在 2013-04-24三的 09:11 +0200,Aurelien Jarno写道: On Wed, Apr 24, 2013 at 08:36:54AM +0200, Paolo Bonzini wrote: Il 24/04/2013 03:48, liguang ha scritto: cs_base is only meaningful for target-i386/sparc, so, get rid of cs_base

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread li guang
在 2013-04-24三的 08:28 +0100,Peter Maydell写道: On 24 April 2013 08:15, li guang lig.f...@cn.fujitsu.com wrote: 在 2013-04-24三的 08:05 +0100,Peter Maydell写道: On 24 April 2013 02:48, liguang lig.f...@cn.fujitsu.com wrote: Signed-off-by: liguang lig.f...@cn.fujitsu.com I'm afraid this is

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread Peter Maydell
On 24 April 2013 08:32, li guang lig.f...@cn.fujitsu.com wrote: I think even others want to use something like you said, it should not 'cs_base', or, it's a bad name. Yes, this is why I said has a less than helpful name. --- a/target-sparc/cpu.h +++ b/target-sparc/cpu.h @@ -715,7

Re: [Qemu-devel] [PATCH 0/3] block: block-backup live backup command

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 10:49:35AM -0600, Eric Blake wrote: On 04/23/2013 10:25 AM, Stefan Hajnoczi wrote: This series adds a new QMP command, block-backup, which takes a point-in-time snapshot of a block device. The snapshot is copied out to a target block device. A simple example is:

Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets

2013-04-24 Thread li guang
在 2013-04-24三的 08:36 +0100,Peter Maydell写道: On 24 April 2013 08:32, li guang lig.f...@cn.fujitsu.com wrote: I think even others want to use something like you said, it should not 'cs_base', or, it's a bad name. Yes, this is why I said has a less than helpful name. ---

Re: [Qemu-devel] [PATCH v6 01/20] tcg-arm: Fix local stack frame

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:33PM -0700, Richard Henderson wrote: We were not allocating TCG_STATIC_CALL_ARGS_SIZE, so this meant that any helper with more than 4 arguments would clobber the saved regs. Realizing that we're supposed to have this memory pre-allocated means we can clean up the

Re: [Qemu-devel] [PATCH v6 19/20] tcg-arm: Convert to CONFIG_QEMU_LDST_OPTIMIZATION

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:51PM -0700, Richard Henderson wrote: Move the slow path out of line, as the TODO's mention. This allows the fast path to be unconditional, which can speed up the fast path as well, depending on the core. Signed-off-by: Richard Henderson r...@twiddle.net ---

Re: [Qemu-devel] [PATCH v6 09/20] tcg-arm: Implement deposit for armv7

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:41PM -0700, Richard Henderson wrote: We have BFI and BFC available for implementing it. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 41 ++--- tcg/arm/tcg-target.h | 5 - 2 files

Re: [Qemu-devel] [PATCH v6 16/20] tcg-arm: Improve scheduling of tcg_out_tlb_read

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:48PM -0700, Richard Henderson wrote: The schedule was fully serial, with no possibility for dual issue. The old schedule had a minimal issue of 7 cycles; the new schedule has a minimal issue of 5 cycles. Signed-off-by: Richard Henderson r...@twiddle.net ---

Re: [Qemu-devel] [PATCH v6 17/20] tcg-arm: Delete the 'S' constraint

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:49PM -0700, Richard Henderson wrote: After the previous patch, 's' and 'S' are the same. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tcg/arm/tcg-target.c

Re: [Qemu-devel] [PATCH v6 20/20] tcg-arm: Remove long jump from tcg_out_goto_label

2013-04-24 Thread Aurelien Jarno
On Tue, Apr 23, 2013 at 01:46:52PM -0700, Richard Henderson wrote: Branches within a TB will always be within 16MB. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/tcg/arm/tcg-target.c

Re: [Qemu-devel] KVM call minutes for 2013-04-23

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 10:06:41AM -0600, Eric Blake wrote: On 04/23/2013 08:45 AM, Juan Quintela wrote: we can change drive_mirror to use a new command to see if there are the new features. drive-mirror changed in 1.4 to add optional buf-size parameter; right now, libvirt is forced

[Qemu-devel] [PULL] libcacard and ccid fixes

2013-04-24 Thread Alon Levy
The following changes since commit bb71623811686ce3c34ce724f073f5c5dd95f51b: Move TPM passthrough specific command line options to backend structure (2013-04-23 10:40:40 -0500) are available in the git repository at: git://people.freedesktop.org/~alon/qemu libcacard_ccid.1 for you to

[Qemu-devel] [PATCH RESEND v4 0/8] virtio-rng refactoring.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Tested-by: Cornelia Huck cornelia.h...@de.ibm.com Acked-by: Amit Shah amit.s...@redhat.com There are no modifications, just rebased. This is the last backend of the refactoring (*and must be applied at last, because it would break virtio-9p*).

[Qemu-devel] [PATCH RESEND v4 2/8] virtio-rng: add virtio-rng device.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Create virtio-rng-device which extends virtio-device, so it can be connected on virtio-bus. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/s390x/virtio-ccw.c | 3 +- hw/virtio/virtio-pci.c | 8 +--

[Qemu-devel] [PATCH RESEND v4 4/8] virtio-rng-s390: switch to the new API.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Here the virtio-rng-s390 is modified for the new API. The device virtio-rng-s390 extends virtio-s390-device as before. It creates and connects a virtio-rng during the init. The properties are not modified. Signed-off-by: KONRAD Frederic

[Qemu-devel] [PATCH RESEND v4 1/8] virtio-rng: don't use pointer for configuration.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com The configuration field must not be a pointer as it will be used for virtio-rng properties. So *conf is replaced by conf. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio/virtio-rng.c | 12 ++--

[Qemu-devel] [PATCH RESEND v4 3/8] virtio-rng-pci: switch to the new API.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Here the virtio-rng-pci is modified for the new API. The device virtio-rng-pci extends virtio-pci. It creates and connects a virtio-rng-device during the init. The properties are not changed. The virtio_pci_reset function, is removed as no longer

[Qemu-devel] [PATCH RESEND v4 6/8] virtio-rng: cleanup: init and exit functions.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This remove old init and exit function as they are no longer needed. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio/virtio-rng.c | 87 +++--- include/hw/virtio/virtio.h | 2 -- 2

[Qemu-devel] [PATCH RESEND v4 5/8] virtio-rng-ccw: switch to the new API.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Here the virtio-rng-ccw is modified for the new API. The device virtio-rng-pci extends virtio-pci. It creates and connects a virtio-rng-device during the init. The properties are not changed. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com

[Qemu-devel] [PATCH RESEND v4 8/8] virtio-rng: cleanup: use QOM casts.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com As the virtio-rng-pci, virtio-rng-s390 and virtio-rng-ccw are switched to the new API, we can use QOM casts. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com Reviewed-by: Andreas Färber afaer...@suse.de --- hw/virtio/virtio-rng.c |

[Qemu-devel] [PATCH RESEND v4 7/8] virtio-rng: cleanup: remove qdev field.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com The qdev field is no longer needed, just drop it. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio/virtio-rng.c | 2 -- include/hw/virtio/virtio-rng.h | 2 -- 2 files changed, 4 deletions(-) diff --git

Re: [Qemu-devel] [PULL 0/9] SCSI updates for 2013-04-13

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 06:56, Stefan Weil ha scritto: Hi Paolo, the latest QEMU build is broken when CONFIG_VHOST_SCSI is undefined. --verbose, please. :) $ ../configure --disable-vhost-scsi $ grep VHOST_SC */config-* config-* $ works here. Paolo

[Qemu-devel] [PATCH v4 0/7] virtio-refactoring cleanup.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This is the last part of the refactoring. The step 4 replaces the function pointers contained in VirtIODevice structure by the function pointers contained in VirtioDeviceClass. The step 5 removes virtiobindings, and use the VirtioBusClass. I

[Qemu-devel] [PATCH v4 4/7] virtio: remove the function pointer.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This remove the function pointer in VirtIODevice, and use only VirtioDeviceClass function pointer. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/9pfs/virtio-9p-device.c | 2 -- hw/block/virtio-blk.c | 5 -

[Qemu-devel] [PATCH v4 7/7] virtio: drop unused function prototypes.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This removes the unused prototypes in virtio.h. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- include/hw/virtio/virtio.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/include/hw/virtio/virtio.h

[Qemu-devel] [PATCH v4 1/7] virtio-bus: add new functions.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This add two functions: * virtio_bus_set_vdev_config. * virtio_bus_set_vdev_feature. Needed by virtio-ccw. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio/virtio-bus.c | 23 +++

[Qemu-devel] [PATCH v4 2/7] virtio-bus: make virtio_x_bus_new static.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com virtio_x_bus_new are only used in file scope. So this make them static. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/s390x/s390-virtio-bus.c | 4 +++- hw/s390x/s390-virtio-bus.h | 2 -- hw/s390x/virtio-ccw.c | 4 +++-

[Qemu-devel] [PATCH v4 3/7] virtio-pci: cleanup.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This remove the init, exit functions as they are no longer used. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio/virtio-pci.c | 54 ++ hw/virtio/virtio-pci.h | 2 -- 2 files

[Qemu-devel] [PATCH v4 5/7] virtio: remove virtiobindings.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This remove virtio-bindings, and use class instead. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/block/dataplane/virtio-blk.c | 15 - hw/net/vhost_net.c | 17 -- hw/s390x/s390-virtio-bus.c

[Qemu-devel] [PATCH v4 6/7] virtio: cleanup: init and exit function.

2013-04-24 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This clean the init and the exit functions and rename virtio_common_cleanup to virtio_cleanup. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/block/virtio-blk.c | 4 ++-- hw/char/virtio-serial-bus.c | 2 +-

Re: [Qemu-devel] Adding Disk-Level Introspection to QEMU

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 01:40:19PM -0400, Wolfgang Richter wrote: -- Wolf On Apr 23, 2013, at 1:22 PM, Eric Blake ebl...@redhat.com wrote: On 04/23/2013 11:12 AM, Wolfgang Richter wrote: I'm interested in adding introspection of disk writes to QEMU for various applications and

Re: [Qemu-devel] Adding Disk-Level Introspection to QEMU

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 03:11:26PM -0400, Wolfgang Richter wrote: On Tue, Apr 23, 2013 at 2:31 PM, Wolfgang Richter w...@cs.cmu.edu wrote: On Tue, Apr 23, 2013 at 2:21 PM, Stefan Hajnoczi stefa...@gmail.comwrote: Eric's suggestion to use NBD makes sense to me. The block-backup code can

Re: [Qemu-devel] [Xen-devel] guest cannot boot up successfully using qemu.git in Xen

2013-04-24 Thread Liu, SongtaoX
latest commit:bb71623811686ce3c34ce724f073f5c5dd95f51b in qemu.git works for this issue. guest could boot up normally. Thanks. Regards Carl(Songtao) -Original Message- From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] Sent: Monday, April 22, 2013 5:50 PM To: Liu,

Re: [Qemu-devel] Adding Disk-Level Introspection to QEMU

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 10:37, Stefan Hajnoczi ha scritto: Has there been any performance analysis of drive-mirror (impact on executing guest)? What Stefan wrote is about block-backup. drive-mirror has a limited impact on guest performance, but it doesn't pass the writes through to the channel.

Re: [Qemu-devel] [PATCH V3 5/5] block: make all steps in qmp_transaction() as callback

2013-04-24 Thread Stefan Hajnoczi
On Fri, Apr 19, 2013 at 08:57:10AM +0800, Wenchao Xia wrote: diff --git a/blockdev.c b/blockdev.c index 051be98..b336794 100644 --- a/blockdev.c +++ b/blockdev.c @@ -779,14 +779,41 @@ void qmp_blockdev_snapshot_sync(const char *device, const char *snapshot_file, /* New and old

Re: [Qemu-devel] Adding Disk-Level Introspection to QEMU

2013-04-24 Thread Paolo Bonzini
Il 23/04/2013 20:31, Wolfgang Richter ha scritto: On Tue, Apr 23, 2013 at 2:21 PM, Stefan Hajnoczi stefa...@gmail.com mailto:stefa...@gmail.com wrote: The tracing subsystem is geared towards tracepoint instrumentation rather than binary dumps. Can you share some specific

[Qemu-devel] [PATCH 2/5] console: add device link to QemuConsoles

2013-04-24 Thread Gerd Hoffmann
So it is possible to figure which qemu console displays which device. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/arm/musicpal.c|2 +- hw/display/blizzard.c|2 +- hw/display/cirrus_vga.c |4 ++-- hw/display/exynos4210_fimd.c |2 +-

[Qemu-devel] [PATCH 4/5] console: switch ppm_save to qemu_open

2013-04-24 Thread Gerd Hoffmann
... so it works with fdset. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- ui/console.c |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ui/console.c b/ui/console.c index e3ab985..3835316 100644 --- a/ui/console.c +++ b/ui/console.c @@ -269,18 +269,20 @@ static

Re: [Qemu-devel] [PATCH v4 0/7] publish etc/acpi/APIC in fw_cfg

2013-04-24 Thread Laszlo Ersek
On 04/18/13 22:22, Laszlo Ersek wrote: v4: - patches 1-6 are unchanged and carry Michael's ACK, - patch 7 is rendered dependent on a new configure switch (default off) [Michael Tsirkin]. v3: - rebased to current master 24a6e7f4, - added Michael's S-o-b to 6/7 [Eric Blake], - added Dave

Re: [Qemu-devel] [PATCH 11/11] savevm: remove backward compatibility from bdrv_snapshot_find()

2013-04-24 Thread Pavel Hrdina
On 24.4.2013 05:51, Wenchao Xia wrote: I think that my whole series is almost done and it would be quickly accepted and applied upstream. I'll send today the v2 and we will see. Pavel OK, I'll review v2 too to make it faster. Hi, Pavel Sorry for query your progress, I hope to catch

[Qemu-devel] [PATCH 3/5] console: add qemu_console_lookup_by_device

2013-04-24 Thread Gerd Hoffmann
Look up the QemuConsole for a given device, using the new link. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h |1 + ui/console.c | 19 +++ 2 files changed, 20 insertions(+) diff --git a/include/ui/console.h b/include/ui/console.h index

Re: [Qemu-devel] [PATCH v4 7/7] hw/i386: build ACPI MADT (APIC) for fw_cfg clients

2013-04-24 Thread Michael S. Tsirkin
On Fri, Apr 19, 2013 at 12:58:03PM +0200, Laszlo Ersek wrote: On 04/18/13 22:30, Michael S. Tsirkin wrote: On Thu, Apr 18, 2013 at 10:22:24PM +0200, Laszlo Ersek wrote: This patch reuses some code from SeaBIOS, which was originally under LGPLv2 and then relicensed to GPLv3 or LGPLv3, in

[Qemu-devel] [PATCH 1/5] console: qom-ify QemuConsole

2013-04-24 Thread Gerd Hoffmann
Just the minimal bits to turn QemuConsoles into Objects. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h | 15 +++ ui/console.c | 15 ++- 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/include/ui/console.h

[Qemu-devel] [PULL 0/5] console: qom-ify consoles

2013-04-24 Thread Gerd Hoffmann
Hi, Here are the console patches, targeting 1.5. It's just the QemuConsole QOM-ification and two little fixes. I'll go hold back the screendump monitor patch until the dust from the qom api discussions has settled and it is clear which route we are taking wrt new vs. extended commands. Which

[Qemu-devel] [PATCH 5/5] console: zap ds arg from register_displaychangelistener

2013-04-24 Thread Gerd Hoffmann
We don't have multiple DisplayStates any more, so passing it in as argument is not needed. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/display/qxl.c |2 +- include/ui/console.h |3 +-- ui/cocoa.m |2 +- ui/console.c | 10 +- ui/curses.c

[Qemu-devel] [PATCH] sheepdog: fix loadvm operation

2013-04-24 Thread Liu Yuan
From: Liu Yuan tailai...@taobao.com Currently the 'loadvm' opertaion works as following: 1. switch to the snapshot 2. mark current working VDI as a snapshot 3. rely on sd_create_branch to create a new working VDI based on the snapshot This works not the same as other format as QCOW2. For e.g,

Re: [Qemu-devel] [Qemu-trivial] [PATCH] configure: Undefine _FORTIFY_SOURCE prior using it

2013-04-24 Thread Markus Armbruster
Michal Privoznik mpriv...@redhat.com writes: On 06.02.2013 16:09, Paolo Bonzini wrote: Il 06/02/2013 15:49, Stefan Hajnoczi ha scritto: I don't think that's necessary. The 2nd level is the highest one [1] or [2]. It seems like in my case it's compiler who's defining the macro: $ echo int

[Qemu-devel] [PATCH 1/2] input: make QEMUPutLEDEntry + QEMUPutMouseEntry private

2013-04-24 Thread Gerd Hoffmann
There is no need for anybody outside ui/input.c to access the struct elements. Move the definitions, leaving only the typedefs in the header files. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- include/ui/console.h | 19 ++- ui/input.c | 18 ++

[Qemu-devel] [PATCH 2/2] input: introduce keyboard handler list

2013-04-24 Thread Gerd Hoffmann
Add a linked list of keyboard handlers. Added handlers will go to the head of the list. Removed handlers will be zapped from the list. The head of the list will be used for events. This fixes the keyboard-dead-after-usb-kbd-unplug issue, key events will be re-routed to the ps/2 kbd instead of

[Qemu-devel] [RESEND PATCH 0/2] input: kbd handler list

2013-04-24 Thread Gerd Hoffmann
Hi, Tiny series which adds a list for keyboard handlers, simliar to the one we have for mice. Lingering unmodified in a git branch for a while, just noticed it isn't in yet, so I'm resending it. please apply, Gerd Gerd Hoffmann (2): input: make QEMUPutLEDEntry + QEMUPutMouseEntry private

[Qemu-devel] [PATCH 2/3] vnc: Support for LED state extension

2013-04-24 Thread Lei Li
Signed-off-by: Lei Li li...@linux.vnet.ibm.com --- ui/vnc.c | 45 + ui/vnc.h |4 +++- 2 files changed, 48 insertions(+), 1 deletions(-) diff --git a/ui/vnc.c b/ui/vnc.c index f574962..44189d7 100644 --- a/ui/vnc.c +++ b/ui/vnc.c @@ -1522,6

[Qemu-devel] [PATCH 3/3] doc: document the Pseudo-encoding of LED state

2013-04-24 Thread Lei Li
Signed-off-by: Lei Li li...@linux.vnet.ibm.com --- docs/vnc-ledstate-Pseudo-encoding.txt | 40 + 1 files changed, 40 insertions(+), 0 deletions(-) create mode 100644 docs/vnc-ledstate-Pseudo-encoding.txt diff --git a/docs/vnc-ledstate-Pseudo-encoding.txt

[Qemu-devel] [PATCH 1/3] vnc: Add SCROLL lock key to kbd_leds

2013-04-24 Thread Lei Li
Signed-off-by: Lei Li li...@linux.vnet.ibm.com --- ui/vnc.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/ui/vnc.c b/ui/vnc.c index 8ee66b7..f574962 100644 --- a/ui/vnc.c +++ b/ui/vnc.c @@ -1525,10 +1525,11 @@ static void press_key(VncState *vs, int keysym) static

[Qemu-devel] [PATCH 0/3 v3] Support for LED state extension to Qemu VNC server

2013-04-24 Thread Lei Li
Hi guys, This patch series tries to add support for LED state extension to Qemu VNC server. The proposal has been sent few days ago as link below: http://lists.gnu.org/archive/html/qemu-devel/2013-04/msg01421.html The previous version as link:

Re: [Qemu-devel] [PATCH 17/21] introduce memory_region_get_address() and use it in kvm/ioapic

2013-04-24 Thread Paolo Bonzini
So how about: MemoryRegionSection memory_region_find(MemoryRegion *address_space, hwaddr addr, uint64_t size); becomes MemoryRegionSection address_space_find_region_by_addr( AddressSpace *address_space,

Re: [Qemu-devel] [PATCH 17/21] introduce memory_region_get_address() and use it in kvm/ioapic

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 12:22, Paolo Bonzini ha scritto: diff --git a/memory.c b/memory.c index c82bd12..dba0a4b 100644 --- a/memory.c +++ b/memory.c @@ -1451,15 +1451,24 @@ static FlatRange *address_space_lookup(AddressSpace *as, AddrRange addr) sizeof(FlatRange),

[Qemu-devel] question about concurrent block job

2013-04-24 Thread Zhangleiqiang
Hi, Paolo: I found qemu does not support concurrently running multiple block jobs of the same drive (limited by the bs-in_use field). But there are indeed some type of block job can run concurrently, such as running block stream job while running drive mirror, etc. Am I right ? Is it

[Qemu-devel] [PATCH] ich9: kill cmos_s3

2013-04-24 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/acpi/ich9.c | 2 +- hw/i386/pc_q35.c | 15 +-- hw/isa/lpc_ich9.c | 4 ++-- include/hw/acpi/ich9.h | 2 +- include/hw/i386/ich9.h | 2 +- 5 files changed, 6 insertions(+), 19 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] ui/gtk: Add Turkish translations

2013-04-24 Thread Andreas Färber
Anthony, Am 23.04.2013 12:04, schrieb Ozan Çağlayan: Signed-off-by: Ozan Çağlayan ozan...@gmail.com --- po/tr.po | 62 ++ 1 file changed, 62 insertions(+) create mode 100644 po/tr.po diff --git a/po/tr.po b/po/tr.po new file

Re: [Qemu-devel] [RFC][PATCH v3 00/24] instrument: Let the user wrap/override specific event tracing routines

2013-04-24 Thread Stefan Hajnoczi
On Sun, Apr 21, 2013 at 09:11:30PM +0200, Lluís Vilanova wrote: TODO: Operations 'instr_load' and 'instr_unload' are not thread safe. (qemu_cpu_kick?) TODO: Do cmdline actions have to be implemented on top of QMP routines? TODO: HMP and QMP interfaces only accept one argument to

Re: [Qemu-devel] [PATCH] ich9: kill cmos_s3

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 12:37, Hu Tao ha scritto: Signed-off-by: Hu Tao hu...@cn.fujitsu.com I had exactly the same patch in my 1.6 queue. :) Reviewed-by: Paolo Bonzini pbonz...@redhat.com --- hw/acpi/ich9.c | 2 +- hw/i386/pc_q35.c | 15 +-- hw/isa/lpc_ich9.c | 4

[Qemu-devel] [PATCH v2] microblaze: Add internal base vectors reg

2013-04-24 Thread edgar . iglesias
From: Edgar E. Iglesias edgar.igles...@gmail.com Configurable at CPU synthesis/instantiation. Signed-off-by: Edgar E. Iglesias edgar.igles...@gmail.com --- Peter Crosthwaite commented off-list that it would be a good idea to make this a device property. I agree but am a bit unsure of how it

Re: [Qemu-devel] [PATCH 2/3] vnc: Support for LED state extension

2013-04-24 Thread Gerd Hoffmann
Hi, +case VNC_ENCODING_LED_STATE: +vs-features |= VNC_FEATURE_LED_STATE_MASK; +break; I think it with the client supporting the led state extension it is probably a good idea to turn off the lock state sync logic in qemu's vnc server, i.e. add a !(vs-features

[Qemu-devel] [PATCH 1/5] pci: Simpler implementation of PCI_COMMAND_MASTER bit

2013-04-24 Thread David Gibson
In commit 1c380f9460522f32c8dd2577b2a53d518ec91c6d pci: honor PCI_COMMAND_MASTER the PCI_COMMAND_MASTER bit of the PCI command register was implemented by toggling the enable bit on a memory region alias interposed between the PCI device's dma address space and the main system memory region.

[Qemu-devel] [PATCH 3/5] vfio: Associate VFIO groups with DMAContexts

2013-04-24 Thread David Gibson
The only model so far supported for VFIO passthrough devices is the model usually used on x86, where all of the guest's RAM is mapped into the (host) IOMMU and there is no IOMMU visible in the guest. Later, however we want to also support guest visible IOMMUs. In order to do that the vfio

Re: [Qemu-devel] [PATCH 02/21] cpu: call cpu_synchronize_post_init() from CPUClass.realize() if hotplugged

2013-04-24 Thread Andreas Färber
Am 23.04.2013 17:59, schrieb Andreas Färber: Am 23.04.2013 10:29, schrieb Igor Mammedov: ... to synchronize CPU state to KVM Signed-off-by: Igor Mammedov imamm...@redhat.com Reviewed-by: Eduardo Habkost ehabk...@redhat.com --- v2: * linking kvm-stub.o to *-user target moved in separate

[Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-24 Thread David Gibson
At the moment, vfio maintains a global list of containers that are assumed to be more or less interchangeable, since they are all set up with a MemoryListener to have all of system memory mapped. However, that only makes sense if all the containers are used on devices which really do expect a dma

[Qemu-devel] [0/5] RFC: Preparations for supporting VFIO with guest IOMMUs

2013-04-24 Thread David Gibson
This patch series represents my attempt at better integration of the vfio code with qemu's DMAContext and other guest IOMMU handling code, as discussed by Alex Williamson and myself on IRC. It opens the way for using VFIO with a guest system containing an IOMMU by passing guest IOMMU operations

[Qemu-devel] [PATCH 2/5] pci: Don't create an address space object for every PCI device

2013-04-24 Thread David Gibson
Commit 817dcc5368988b023c5e1d3f1444fd370c77c6a9 pci: give each device its own address space did exactly what the comment suggests, it creates an AddressSpace object for every PCI device to represent the address space it uses for DMA. All the objects so constructed are basically identical to

[Qemu-devel] [PATCH 5/5] vfio: Only use memory listeners when appropriate

2013-04-24 Thread David Gibson
Currently, vfio registers a MemoryListener for every vfio container we create, to keep the container's mappings in sync with main system memory. That's only correct though, if the context the container is attached to represents a dma address space which actually matches main system memory -

Re: [Qemu-devel] [RFC][PATCH v3 00/24] instrument: Let the user wrap/override specific event tracing routines

2013-04-24 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Sun, Apr 21, 2013 at 09:11:30PM +0200, Lluís Vilanova wrote: TODO: Operations 'instr_load' and 'instr_unload' are not thread safe. (qemu_cpu_kick?) TODO: Do cmdline actions have to be implemented on top of QMP routines? TODO: HMP and QMP interfaces only accept

Re: [Qemu-devel] [PATCH v2 2/5] block: vhdx header for the QEMU support of VHDX images

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 10:24:21AM -0400, Jeff Cody wrote: +/* HEADER SECTION STRUCTURES */ + +/* Important note: these structures are as defined in the VHDX specification, + * including byte order and size. However, without being packed structures, + * they will not match 1:1 data

Re: [Qemu-devel] [PATCH v2 2/5] block: vhdx header for the QEMU support of VHDX images

2013-04-24 Thread Jeff Cody
On Wed, Apr 24, 2013 at 02:31:27PM +0200, Stefan Hajnoczi wrote: On Tue, Apr 23, 2013 at 10:24:21AM -0400, Jeff Cody wrote: +/* HEADER SECTION STRUCTURES */ + +/* Important note: these structures are as defined in the VHDX specification, + * including byte order and size.

Re: [Qemu-devel] [PATCH 2/5] pci: Don't create an address space object for every PCI device

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 14:01, David Gibson ha scritto: So while its true that the DMAContext handling needs to be better integrated with the MemoryRegion and AddressSpace handling, that commit wasn't actually a step in the right direction for it. Since then, the DMAContext has been extended so it can

Re: [Qemu-devel] [PATCH] ui/gtk: Add Turkish translations

2013-04-24 Thread Anthony Liguori
Andreas Färber afaer...@suse.de writes: Anthony, Am 23.04.2013 12:04, schrieb Ozan Çağlayan: Signed-off-by: Ozan Çağlayan ozan...@gmail.com --- po/tr.po | 62 ++ 1 file changed, 62 insertions(+) create mode 100644 po/tr.po

Re: [Qemu-devel] [PATCH 1/5] pci: Simpler implementation of PCI_COMMAND_MASTER bit

2013-04-24 Thread Paolo Bonzini
Il 24/04/2013 14:01, David Gibson ha scritto: In commit 1c380f9460522f32c8dd2577b2a53d518ec91c6d pci: honor PCI_COMMAND_MASTER the PCI_COMMAND_MASTER bit of the PCI command register was implemented by toggling the enable bit on a memory region alias interposed between the PCI device's dma

[Qemu-devel] [PATCH v3 1/6] vmdk: named return code.

2013-04-24 Thread Fam Zheng
Internal routines in vmdk.c previously return -1 on error and 0 on success. More return values are useful for future changes such as zeroed-grain GTE. Change all the magic `return 0` and `return -1` to macro names: * VMDK_OK 0 * VMDK_ERROR (-1) * VMDK_UNALLOC (-2) * VMDK_ZEROED (-3)

[Qemu-devel] [PATCH v3 0/6] vmdk: zeroed-grain GTE support

2013-04-24 Thread Fam Zheng
Added support for zeroed-grain GTE to VMDK according to VMDK Spec 5.0[1]. [1] Virtual Disk Format 5.0 - VMware, http://www.vmware.com/support/developer/vddk/vmdk_50_technote.pdf?src=vmdk Changes since v2: - all: Added 5/6 (vmdk: store fields of VmdkMetaData in cpu endian) - 6/6: Avoid

[Qemu-devel] [PATCH v3 6/6] vmdk: add bdrv_co_write_zeroes

2013-04-24 Thread Fam Zheng
Use special offset to write zeroes efficiently, when zeroed-grain GTE is available. If zero-write an allocated cluster, cluster is leaked because its offset pointer is overwritten by 0x1. Signed-off-by: Fam Zheng f...@redhat.com --- block/vmdk.c | 77

[Qemu-devel] [PATCH v3 3/6] vmdk: Add option to create zeroed-grain image

2013-04-24 Thread Fam Zheng
Add image create option zeroed-grain to enable zeroed-grain GTE feature of vmdk sparse extents. When this option is on, header version of newly created extent will be 2 and VMDK4_FLAG_ZERO_GRAIN flag bit will be set. Signed-off-by: Fam Zheng f...@redhat.com --- block/vmdk.c | 22

[Qemu-devel] [PATCH v3 4/6] vmdk: change magic number to macro

2013-04-24 Thread Fam Zheng
Two hard coded flag bits are changed to macros. Signed-off-by: Fam Zheng f...@redhat.com --- block/vmdk.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/block/vmdk.c b/block/vmdk.c index cc19e20..0463d3b 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -32,6 +32,7 @@

[Qemu-devel] [RESEND PATCH 1/2] qapi: introduce strList and visit_type_strList()

2013-04-24 Thread Amos Kong
Currently we can only use ['String'] to add string to a list, it contains some additional JSON structure. multicast: [ { str: 01:80:c2:00:00:21 }, { str: 00:00:00:00:00:00 } ] This patch introdued strList, we can use ['str']

[Qemu-devel] [PATCH v3 5/6] vmdk: store fields of VmdkMetaData in cpu endian

2013-04-24 Thread Fam Zheng
Previously VmdkMetaData.offset is stored little endian while other fields are cpu endian. This changes offset to cpu endian and convert before writing to image. Signed-off-by: Fam Zheng f...@redhat.com --- block/vmdk.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[Qemu-devel] [PATCH 2/2] monitor: introduce query-config-schema command

2013-04-24 Thread Amos Kong
Libvirt has no way to probe if an option or property is supported, This patch introdues a new qmp command to query configuration schema information. hmp command isn't added because it's not needed. Signed-off-by: Amos Kong ak...@redhat.com CC: Osier Yang jy...@redhat.com CC: Anthony Liguori

Re: [Qemu-devel] [PATCH 0/3] block: block-backup live backup command

2013-04-24 Thread Eric Blake
On 04/24/2013 01:41 AM, Stefan Hajnoczi wrote: 2. Sync modes like drive-mirror (top, full, none). This makes it possible to preserve the backing file chain. Based on today's phone call, it sounds like this would mean adding optional parameters to the QMP command. It's not a

Re: [Qemu-devel] [PATCH] ui/gtk: Add Turkish translations

2013-04-24 Thread Daniel P. Berrange
On Wed, Apr 24, 2013 at 07:36:25AM -0500, Anthony Liguori wrote: Andreas Färber afaer...@suse.de writes: Anthony, Am 23.04.2013 12:04, schrieb Ozan Çağlayan: Signed-off-by: Ozan Çağlayan ozan...@gmail.com --- po/tr.po | 62

Re: [Qemu-devel] [PATCH 1/5] pci: Simpler implementation of PCI_COMMAND_MASTER bit

2013-04-24 Thread David Gibson
On Wed, Apr 24, 2013 at 02:36:46PM +0200, Paolo Bonzini wrote: Il 24/04/2013 14:01, David Gibson ha scritto: In commit 1c380f9460522f32c8dd2577b2a53d518ec91c6d pci: honor PCI_COMMAND_MASTER the PCI_COMMAND_MASTER bit of the PCI command register was implemented by toggling the enable bit on

Re: [Qemu-devel] [PATCH 2/5] pci: Don't create an address space object for every PCI device

2013-04-24 Thread David Gibson
On Wed, Apr 24, 2013 at 02:34:21PM +0200, Paolo Bonzini wrote: Il 24/04/2013 14:01, David Gibson ha scritto: So while its true that the DMAContext handling needs to be better integrated with the MemoryRegion and AddressSpace handling, that commit wasn't actually a step in the right

[Qemu-devel] [PATCH v3 2/6] vmdk: add support for “zeroed‐grain” GTE

2013-04-24 Thread Fam Zheng
Introduced support for zeroed-grain GTE, as specified in Virtual Disk Format 5.0[1]. Recent VMware hosted platform products support a new “zeroed‐grain” grain table entry (GTE). The zeroed‐grain GTE returns all zeros on read. In other words, the zeroed‐grain GTE indicates that a

Re: [Qemu-devel] [PATCH v2 3/5] block: initial VHDX driver support framework - supports open and probe

2013-04-24 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 10:24:22AM -0400, Jeff Cody wrote: +if (!vhdx_checksum_is_valid(buffer, VHDX_HEADER_BLOCK_SIZE, 4) || +s-rt.signature != VHDX_RT_MAGIC) { +ret = -EINVAL; +goto fail; +} + +for (i = 0; i s-rt.entry_count; i++) { It's nice to avoid

  1   2   3   >