Re: [Qemu-devel] [PATCH-v2 0/2] Add support for vhost-scsi-pci

2013-03-29 Thread Asias He
On Fri, Mar 29, 2013 at 01:08:14AM +, Nicholas A. Bellinger wrote: From: Nicholas Bellinger n...@linux-iscsi.org Hi QEMU folks, The following is the patch-v2 series to support vhost-scsi-pci within the upstream QEMU tree. This includes the refactoring of existing virtio-scsi code

Re: [Qemu-devel] [PATCH-v2 0/2] Add support for vhost-scsi-pci

2013-03-29 Thread Nicholas A. Bellinger
On Fri, 2013-03-29 at 14:39 +0800, Asias He wrote: On Fri, Mar 29, 2013 at 01:08:14AM +, Nicholas A. Bellinger wrote: From: Nicholas Bellinger n...@linux-iscsi.org Hi QEMU folks, The following is the patch-v2 series to support vhost-scsi-pci within the upstream QEMU tree.

Re: [Qemu-devel] [PATCH-v2 0/2] Add support for vhost-scsi-pci

2013-03-29 Thread Asias He
On Thu, Mar 28, 2013 at 11:48:59PM -0700, Nicholas A. Bellinger wrote: On Fri, 2013-03-29 at 14:39 +0800, Asias He wrote: On Fri, Mar 29, 2013 at 01:08:14AM +, Nicholas A. Bellinger wrote: From: Nicholas Bellinger n...@linux-iscsi.org Hi QEMU folks, The following is the

Re: [Qemu-devel] [PATCH] memory: reverse the priority when commit

2013-03-29 Thread liu ping fan
On Thu, Mar 28, 2013 at 10:44 PM, Peter Maydell peter.mayd...@linaro.org wrote: On 28 March 2013 08:08, Liu Ping Fan pingf...@linux.vnet.ibm.com wrote: In theroy, the high priority should commit earlier, it is the same reason as the sequence of Lower = earlier (during add), later (during del)

Re: [Qemu-devel] [PATCH v15.2 4/6] pvpanic: add document of pvpanic

2013-03-29 Thread Hu Tao
On Thu, Mar 28, 2013 at 06:16:32AM -0600, Eric Blake wrote: On 03/28/2013 12:16 AM, Hu Tao wrote: Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- docs/specs/pvpanic.txt | 37 + 1 file changed, 37 insertions(+) create mode 100644

Re: [Qemu-devel] [PATCH v9 24/24] hw/arm: add FTSPI020 SPI flash controller support

2013-03-29 Thread Kuo-Jung Su
2013/3/29 Peter Crosthwaite peter.crosthwa...@xilinx.com: Hi Kuo-Jung, On Mon, Mar 25, 2013 at 10:10 PM, Kuo-Jung Su dant...@gmail.com wrote: From: Kuo-Jung Su dant...@faraday-tech.com The FTSPI020 is an integrated SPI Flash controller which supports up to 4 flash chips. Signed-off-by:

Re: [Qemu-devel] [PATCH v15.2 4/6] pvpanic: add document of pvpanic

2013-03-29 Thread Hu Tao
On Thu, Mar 28, 2013 at 10:57:57AM -0400, Paolo Bonzini wrote: - Messaggio originale - Da: Eric Blake ebl...@redhat.com A: Hu Tao hu...@cn.fujitsu.com Cc: qemu-devel qemu-devel@nongnu.org, Paolo Bonzini pbonz...@redhat.com Inviato: Giovedì, 28 marzo 2013 13:16:32 Oggetto:

Re: [Qemu-devel] [RFC PATCH v2 2/4] net: resolve race of tap backend and its peer

2013-03-29 Thread liu ping fan
On Thu, Mar 28, 2013 at 10:34 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, Mar 28, 2013 at 03:55:53PM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Signed-off-by: Liu Ping Fan pingf...@linux.vnet.ibm.com --- net/tap.c |7 ++- 1 files changed, 6

Re: [Qemu-devel] [RFC PATCH v2 3/4] net: port hub onto glib

2013-03-29 Thread liu ping fan
On Thu, Mar 28, 2013 at 10:47 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, Mar 28, 2013 at 03:55:54PM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Attach each hubport with a GSource. Currently the GSource is attached to default main context, and the hub

Re: [Qemu-devel] [PATCH v9 10/24] hw/arm: add FTDMAC020 AHB DMA support

2013-03-29 Thread Kuo-Jung Su
2013/3/29 Peter Crosthwaite peter.crosthwa...@xilinx.com: Hi Kuo-Jung, On Mon, Mar 25, 2013 at 10:09 PM, Kuo-Jung Su dant...@gmail.com wrote: From: Kuo-Jung Su dant...@faraday-tech.com The Faraday FTDMAC020 provides eight configurable channels for the memory-to-memory, memory-to-peripheral,

Re: [Qemu-devel] [PATCH 07/12] target-i386: Add ICC_BUS and attach apic, kvmvapic and cpu to it

2013-03-29 Thread li guang
在 2013-03-28四的 11:55 +0100,Igor Mammedov写道: On Wed, 27 Mar 2013 11:57:53 +0100 Paolo Bonzini pbonz...@redhat.com wrote: Il 21/03/2013 15:28, Igor Mammedov ha scritto: Introduce hot-pluggable ICC_BUS and convert APIC devices from SysBusDevice to ICCDevice wich has ICC_BUS as default

Re: [Qemu-devel] [PATCH] target-moxie: Fix pointer-to-integer conversion (MinGW-w64)

2013-03-29 Thread Stefan Weil
Am 28.03.2013 20:00, schrieb Anthony Green: Hi Stefan, This change is fine by me. It's not clear to me, as the author/maintainer of the moxie port, what my responsibility/authority is. Do I simply reply to patches like this with... Looks good to me. Signed-off-by: Anthony Green

[Qemu-devel] [PATCH] rbd: add an asynchronous flush

2013-03-29 Thread Josh Durgin
The existing bdrv_co_flush_to_disk implementation uses rbd_flush(), which is sychronous and causes the main qemu thread to block until it is complete. This results in unresponsiveness and extra latency for the guest. Fix this by using an asynchronous version of flush. This was added to librbd

Re: [Qemu-devel] [PATCH uq/master v2 0/2] Add some tracepoints for clarification of the cause of troubles

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 05:24, Kazuya Saito ha scritto: This series adds tracepoints for helping us clarify the cause of troubles. Virtualization on Linux is composed of some components such as qemu, kvm, libvirt, and so on. So it is very important to clarify firstly and swiftly the cause of troubles is

Re: [Qemu-devel] [PATCH 07/12] target-i386: Add ICC_BUS and attach apic, kvmvapic and cpu to it

2013-03-29 Thread Igor Mammedov
On Fri, 29 Mar 2013 15:22:47 +0800 li guang lig.f...@cn.fujitsu.com wrote: 在 2013-03-28四的 11:55 +0100,Igor Mammedov写道: On Wed, 27 Mar 2013 11:57:53 +0100 Paolo Bonzini pbonz...@redhat.com wrote: Il 21/03/2013 15:28, Igor Mammedov ha scritto: Introduce hot-pluggable ICC_BUS and

Re: [Qemu-devel] [PATCH V3 WIP 3/3] disable vhost_verify_ring_mappings check

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 03:53, Nicholas A. Bellinger ha scritto: On Thu, 2013-03-28 at 06:13 -0400, Paolo Bonzini wrote: I think it's the right thing to do, but maybe not the right place to do this, need to reset after all IO is done, before ring memory is write protected. Our emails are crossing each

[Qemu-devel] [PATCH v16 2/6] add a new qevent: QEVENT_GUEST_PANICKED

2013-03-29 Thread Hu Tao
This event will be emited when qemu detects guest panic. Signed-off-by: Wen Congyang we...@cn.fujitsu.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- QMP/qmp-events.txt| 14 ++ include/monitor/monitor.h | 1 + monitor.c | 1 + 3 files changed, 16

[Qemu-devel] [PATCH v16 5/6] pc_piix, pc_q35: export fw_cfg

2013-03-29 Thread Hu Tao
This lets other devices pass informations to seabios. Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/i386/pc_piix.c | 7 --- hw/i386/pc_q35.c | 9 ++--- hw/pc.h | 6 ++ 3 files changed, 16 insertions(+), 6 deletions(-) diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c

[Qemu-devel] [PATCH v16 6/6] pvpanic: pass configurable ioport to seabios

2013-03-29 Thread Hu Tao
This lets seabios patch the corresponding SSDT entry. Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/fw_cfg.h | 2 ++ hw/pvpanic.c | 14 ++ 2 files changed, 16 insertions(+) diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h index 05c8df1..07cc941 100644 --- a/hw/fw_cfg.h +++ b/hw/fw_cfg.h

[Qemu-devel] [PATCH v16 3/6] introduce a new qom device to deal with panicked event

2013-03-29 Thread Hu Tao
pvpanic device is used to send guest panic event from guest to qemu. When guest panic happens, pvpanic device driver will write a event number to IO port 0x505(which is the IO port occupied by pvpanic device, by default). On receiving the event, pvpanic device will pause guest cpu(s), and send a

[Qemu-devel] [PATCH v16 4/6] pvpanic: add document of pvpanic

2013-03-29 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- docs/specs/pvpanic.txt | 37 + 1 file changed, 37 insertions(+) create mode 100644 docs/specs/pvpanic.txt diff --git a/docs/specs/pvpanic.txt b/docs/specs/pvpanic.txt new file mode 100644 index 000..d72d667

[Qemu-devel] [PATCH v16 1/6] add a new runstate: RUN_STATE_GUEST_PANICKED

2013-03-29 Thread Hu Tao
The guest will be in this state when it is panicked. Signed-off-by: Wen Congyang we...@cn.fujitsu.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- include/sysemu/sysemu.h | 1 + qapi-schema.json| 5 - qmp.c | 3 +-- vl.c| 13 +++-- 4

[Qemu-devel] [PATCH v16 0/6] Add pvpanic device to deal with guest panic event

2013-03-29 Thread Hu Tao
This series introduces a new simulated device, pvpanic, to notify qemu when guest panic event happens. Along with this series, there are two patches to add seabios ACPI driver and kernel ACPI driver for the device, respectively. Tested with: - qemu(kvm)/qemu(tcg) - qemu piix/q35 - default

[Qemu-devel] [PATCH v16] Add pvpanic device driver

2013-03-29 Thread Hu Tao
pvpanic device is used to notify host(qemu) when guest panic happens. Signed-off-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- src/acpi.c| 3 +++ src/ssdt-misc.dsl | 46 ++ 2 files changed, 49

[Qemu-devel] [PATCH v16] pvpanic: pvpanic device driver

2013-03-29 Thread Hu Tao
pvpanic device is a qemu simulated device through which guest panic event is sent to host. Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- ref: http://lists.nongnu.org/archive/html/qemu-devel/2013-03/msg05297.html drivers/platform/x86/Kconfig | 7 +++ drivers/platform/x86/Makefile | 2 +

Re: [Qemu-devel] [PATCHv2 0/6] ARM dump-guest-memory support

2013-03-29 Thread Rabin Vincent
2013/3/25 Andreas Färber afaer...@suse.de This still does not address the architectural issue that I brought up. I guess you mean the CPUArchState stuff? AFAICS Wen Congyang (the author of the dump code) had some answers/questions for you:

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

2013-03-29 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com This is the next part of virtio-refactoring. Basically it creates virtio-serial device which extends virtio-device. Then a virtio-serial can be connected on a virtio-bus. virtio-serial-pci, virtio-serial-s390 and virtio-serial-ccw are created too,

[Qemu-devel] [PATCH v4 2/7] virtio-serial-pci: switch to the new API.

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

[Qemu-devel] [PATCH v4 5/7] virtio-serial: cleanup: init and exit functions.

2013-03-29 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-serial-bus.c | 94 +++--- 1 file changed, 21 insertions(+), 73

[Qemu-devel] [PATCH v4 1/7] virtio-serial: add the virtio-serial device.

2013-03-29 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com Create virtio-serial which extends virtio-device, so it can be connected on virtio-bus. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio-pci.c| 2 +- hw/virtio-serial-bus.c | 95

[Qemu-devel] [PATCH v4 3/7] virtio-serial-s390: switch to the new API.

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

[Qemu-devel] [PATCH v4 6/7] virtio-serial: cleanup: use QOM casts.

2013-03-29 Thread fred . konrad
From: KONRAD Frederic fred.kon...@greensocs.com As the virtio-serial-pci and virtio-serial-s390 are switched to the new API, we can use QOM casts. Signed-off-by: KONRAD Frederic fred.kon...@greensocs.com --- hw/virtio-serial-bus.c | 50 ++

[Qemu-devel] [PATCH v4 4/7] virtio-serial-ccw: switch to the new API.

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

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

2013-03-29 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-serial-bus.c | 2 -- hw/virtio-serial.h | 2 -- 2 files changed, 4 deletions(-) diff --git a/hw/virtio-serial-bus.c

Re: [Qemu-devel] [PATCH v16 5/6] pc_piix, pc_q35: export fw_cfg

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 09:16, Hu Tao ha scritto: diff --git a/hw/pc.h b/hw/pc.h index 8e1dd4c..d42b76b 100644 --- a/hw/pc.h +++ b/hw/pc.h @@ -145,6 +145,12 @@ PCIBus *i440fx_init(PCII440FXState **pi440fx_state, int *piix_devfn, extern PCIDevice *piix4_dev; int piix4_init(PCIBus *bus, ISABus

Re: [Qemu-devel] [PATCH v16] Add pvpanic device driver

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 09:18, Hu Tao ha scritto: pvpanic device is used to notify host(qemu) when guest panic happens. Signed-off-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- src/acpi.c| 3 +++ src/ssdt-misc.dsl | 46

Re: [Qemu-devel] [PATCH 1/7] virtio-console: Also throttle when less was written then requested

2013-03-29 Thread Amit Shah
On (Thu) 28 Mar 2013 [14:28:11], Hans de Goede wrote: This is necessary so that we get properly woken up to write the rest. Signed-off-by: Hans de Goede hdego...@redhat.com Acked-by: Amit Shah amit.s...@redhat.com --- hw/virtio-console.c | 5 +++-- 1 file changed, 3 insertions(+), 2

Re: [Qemu-devel] [PATCH v16] pvpanic: pvpanic device driver

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 09:34, Hu Tao ha scritto: pvpanic device is a qemu simulated device through which guest panic event is sent to host. Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- ref: http://lists.nongnu.org/archive/html/qemu-devel/2013-03/msg05297.html drivers/platform/x86/Kconfig |

Re: [Qemu-devel] [PATCH 03/20] char: add IOWatchPoll support

2013-03-29 Thread Amit Shah
On (Tue) 05 Mar 2013 [23:21:18], Amit Shah wrote: From: Anthony Liguori aligu...@us.ibm.com This is a special GSource that supports CharDriverState style poll callbacks. For reviewability and bisectability, this code is #if 0'd out in this patch to avoid unused warnings since all of the

Re: [Qemu-devel] PPC: Which instruction to fetch at Power-On?

2013-03-29 Thread Fabien Chouteau
On 03/28/2013 05:04 PM, Alexander Graf wrote: On 28.03.2013, at 16:10, Fabien Chouteau wrote: On 03/28/2013 12:46 PM, Alexander Graf wrote: On 28.03.2013, at 12:42, Fabien Chouteau wrote: While working on a patch to remove env-hreset_excp_prefix, I'm looking at which instruction should

Re: [Qemu-devel] [PATCH v2 3/3] target-i386: add AES-NI instructions

2013-03-29 Thread Aurelien Jarno
On Fri, Mar 29, 2013 at 12:46:49AM +0100, Edgar E. Iglesias wrote: On Fri, Mar 29, 2013 at 12:36:29AM +0100, Aurelien Jarno wrote: Signed-off-by: Aurelien Jarno aurel...@aurel32.net Hi Aurelien, Nice patches! A question - The sbox tables are already available but in a different format

Re: [Qemu-devel] [PATCH v3 14/20] tcg-arm: Cleanup goto_tb handling

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 01:48:42PM -0700, Richard Henderson wrote: On 2013-03-28 13:09, Aurelien Jarno wrote: I already proposed such a patch, but it seems to improve things only on some specific cases (kernel boot), while increasing the I/D cache and TLB pressure. See

[Qemu-devel] [PATCH 1/2] PPC: Remvove env-hreset_excp_prefix

2013-03-29 Thread Fabien Chouteau
This value is not needed if we use correctly the MSR[IP] bit. excp_prefix is always 0x, except when the MSR[IP] bit is implemented and set to 1, in that case excp_prefix is 0xfff0. The handling of MSR[IP] was already implemented but not used at reset because the value of env-msr was

[Qemu-devel] [PATCH 2/2] PPC: fix hreset_vector for 60x, 7x0, 7x5, G2, MPC8xx, MPC5xx, 7400 and 7450

2013-03-29 Thread Fabien Chouteau
According to the different user's manuals, the vector offset for system reset (both /HRESET and /SRESET) is 0x00100. This patch may break support of some executables, as the power-on start address may change. For a specific board, if the power-on start address is different than HRESET vector

Re: [Qemu-devel] [PATCH 03/20] char: add IOWatchPoll support

2013-03-29 Thread Anthony Liguori
Amit Shah amit.s...@redhat.com writes: On (Tue) 05 Mar 2013 [23:21:18], Amit Shah wrote: From: Anthony Liguori aligu...@us.ibm.com This is a special GSource that supports CharDriverState style poll callbacks. For reviewability and bisectability, this code is #if 0'd out in this patch to

Re: [Qemu-devel] [Xen-devel] frequently ballooning results in qemu exit

2013-03-29 Thread Stefano Stabellini
On Mon, 25 Mar 2013, Hanweidong wrote: We fixed this issue by below patch which computed the correct size for test_bits(). qemu_get_ram_ptr() and qemu_safe_ram_ptr() will call xen_map_cache() with size is 0 if the requested address is in the RAM. Then xen_map_cache() will pass the size 0 to

Re: [Qemu-devel] [PATCH 03/20] char: add IOWatchPoll support

2013-03-29 Thread Amit Shah
On (Fri) 29 Mar 2013 [07:24:07], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Tue) 05 Mar 2013 [23:21:18], Amit Shah wrote: From: Anthony Liguori aligu...@us.ibm.com This is a special GSource that supports CharDriverState style poll callbacks. For

[Qemu-devel] win32 qemu binaries

2013-03-29 Thread Michael W. Bombardieri
Hi, Thanks for your work on Qemu! My friend Prashant Satish has provided Qemu binaries for 1.4.0 hosted here: http://www.omledom.com/ Currently there is no support for SDL in the build but more binary releases are to come. Would it be possible for this mirror to be added to:

[Qemu-devel] Qemu+kvm+usb-device

2013-03-29 Thread shegolev_a...@mail.ru
Hello!   I use qemu + kvm on my home server (Ubuntu 12..0.4 Server).   There is a problem when I reconnect usb-device was not detected in the virtual machine.   Doing so: 1) connect usb-device to the parent machine 2) usb-device can be seen in the parent machine 3) connect this usb-device to the

Re: [Qemu-devel] [SeaBIOS] [PATCH v16] Add pvpanic device driver

2013-03-29 Thread Kevin O'Connor
On Fri, Mar 29, 2013 at 04:18:44PM +0800, Hu Tao wrote: pvpanic device is used to notify host(qemu) when guest panic happens. Thanks. However, we're planning a move of ACPI tables from SeaBIOS to QEMU. I think this should wait until after the move. -Kevin

Re: [Qemu-devel] Block I/O optimizations

2013-03-29 Thread Loic Dachary
Hi Stefan, This is excellent ! Thank you very much :-) Cheers On 03/13/2013 04:21 PM, Stefan Hajnoczi wrote: On Thu, Feb 21, 2013 at 9:11 AM, Stefan Hajnoczi stefa...@gmail.com wrote: On Mon, Feb 18, 2013 at 7:19 PM, Loic Dachary l...@dachary.org wrote: I recently tried to figure out the

Re: [Qemu-devel] [SeaBIOS] [PATCH v16] Add pvpanic device driver

2013-03-29 Thread Paolo Bonzini
Il 29/03/2013 14:33, Kevin O'Connor ha scritto: On Fri, Mar 29, 2013 at 04:18:44PM +0800, Hu Tao wrote: pvpanic device is used to notify host(qemu) when guest panic happens. Thanks. However, we're planning a move of ACPI tables from SeaBIOS to QEMU. I think this should wait until after the

Re: [Qemu-devel] vNVRAM / blobstore design

2013-03-29 Thread Stefan Berger
On 03/28/2013 01:39 PM, Michael S. Tsirkin wrote: On Thu, Mar 28, 2013 at 12:27:45PM -0500, Anthony Liguori wrote: Stefan Berger stef...@linux.vnet.ibm.com writes: On 03/27/2013 03:12 PM, Stefan Berger wrote: On 03/27/2013 02:27 PM, Anthony Liguori wrote: Stefan Berger

Re: [Qemu-devel] [PATCH 03/20] char: add IOWatchPoll support

2013-03-29 Thread Anthony Liguori
Amit Shah amit.s...@redhat.com writes: On (Fri) 29 Mar 2013 [07:24:07], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Tue) 05 Mar 2013 [23:21:18], Amit Shah wrote: From: Anthony Liguori aligu...@us.ibm.com This is a special GSource that supports CharDriverState

[Qemu-devel] [PATCH v4 02/11] block: add error parameter to del_existing_snapshots()

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- savevm.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/savevm.c b/savevm.c index 77c5291..dc1f4a4 100644 --- a/savevm.c +++ b/savevm.c @@ -2224,7 +2224,7 @@ static int

[Qemu-devel] [PATCH v4 00/11] convert savevm to use qapi and introduce qmp command

2013-03-29 Thread Pavel Hrdina
This patch series convert the savevm command into qapi and introduce QMP command vm-snapshot-save. It also rewrite error report for functions used by this command. The last patch introduce new functionality of savevm that you cannot override existing snapshot without using 'force' parameter. If

[Qemu-devel] [PATCH v4 03/11] savevm: add error parameter to qemu_savevm_state_begin()

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- include/sysemu/sysemu.h | 3 ++- migration.c | 2 +- savevm.c| 5 +++-- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/include/sysemu/sysemu.h

[Qemu-devel] [PATCH v4 11/11] savevm: add force parameter to HMP command and return snapshot info

2013-03-29 Thread Pavel Hrdina
HMP command savevm now takes extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- hmp-commands.hx | 16

[Qemu-devel] [PATCH v4 01/11] block: add error parameter to bdrv_snapshot_create() and related functions

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- block.c | 22 -- block/qcow2-snapshot.c| 9 - block/qcow2.h | 4 +++- block/rbd.c | 8 ++-- block/sheepdog.c |

[Qemu-devel] [PATCH v4 04/11] savevm: add error parameter to qemu_savevm_state_iterate()

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- include/sysemu/sysemu.h | 2 +- migration.c | 2 +- savevm.c| 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/sysemu/sysemu.h

[Qemu-devel] [PATCH v4 05/11] savevm: add error parameter to qemu_savevm_state_complete()

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- include/sysemu/sysemu.h | 2 +- migration.c | 2 +- savevm.c| 5 +++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/include/sysemu/sysemu.h

[Qemu-devel] [PATCH v4 06/11] savevm: add error parameter to qemu_savevm_state()

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- savevm.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/savevm.c b/savevm.c index 7598934..3c1ac9e 100644 --- a/savevm.c +++ b/savevm.c @@ -1910,7 +1910,7 @@ void

[Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Pavel Hrdina
QMP command vm-snapshot-save has also extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina phrd...@redhat.com --- hmp-commands.hx | 4 ++-- hmp.c | 11

[Qemu-devel] [PATCH v4 10/11] savevm: update return value from qemu_savevm_state

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- savevm.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/savevm.c b/savevm.c index 6ac4ece..75f64d1 100644 --- a/savevm.c +++ b/savevm.c @@ -1910,7 +1910,7 @@ void

[Qemu-devel] [PATCH v4 08/11] qemu-img: introduce qemu_img_handle_error

2013-03-29 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- qemu-img.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/qemu-img.c b/qemu-img.c index 21d02bf..d5f81cc 100644 --- a/qemu-img.c +++ b/qemu-img.c @@ -322,6 +322,17

[Qemu-devel] [PATCH v4 09/11] block: update return value from bdrv_snapshot_create

2013-03-29 Thread Pavel Hrdina
If we provide error message we don't have to also provide return value because we could check if there is any error message or not. Signed-off-by: Pavel Hrdina phrd...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- block.c | 24 ++--

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Christophe Lyon
Hi, May I ping for this small patch? Thanks, Christophe On 19 February 2013 17:28, Christophe Lyon christophe.l...@linaro.org wrote: Hi, Here is a small patch for the /proc/self/maps output (Linux user mode), which adds a space after the last number in the case where there is no filename

Re: [Qemu-devel] [RFC PATCH v2 2/4] net: resolve race of tap backend and its peer

2013-03-29 Thread Stefan Hajnoczi
On Fri, Mar 29, 2013 at 8:21 AM, liu ping fan qemul...@gmail.com wrote: On Thu, Mar 28, 2013 at 10:34 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, Mar 28, 2013 at 03:55:53PM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Signed-off-by: Liu Ping Fan

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Peter Maydell
On 29 March 2013 14:33, Christophe Lyon christophe.l...@linaro.org wrote: Hi, May I ping for this small patch? Hi Christophe; I'm afraid the patch can't be applied because it's missing a signed-off by. (Also if it was sent inline in the usual format rather than as an attachment it would be

Re: [Qemu-devel] [Qemu-trivial] [PATCH] hw/i386/pc: reject to boot a wrong header magic kernel

2013-03-29 Thread Stefan Hajnoczi
On Fri, Mar 29, 2013 at 1:38 AM, li guang lig.f...@cn.fujitsu.com wrote: 在 2013-03-28四的 10:42 +0100,Stefan Hajnoczi写道: On Wed, Mar 27, 2013 at 02:10:31PM +0800, liguang wrote: if head magic is missing or wrong unexpectedly, we'd better to reject booting. e.g. I make a mistake to boot a

Re: [Qemu-devel] [PATCH v3 14/20] tcg-arm: Cleanup goto_tb handling

2013-03-29 Thread Richard Henderson
On 2013-03-28 23:50, Aurelien Jarno wrote: On Thu, Mar 28, 2013 at 01:48:42PM -0700, Richard Henderson wrote: On 2013-03-28 13:09, Aurelien Jarno wrote: I already proposed such a patch, but it seems to improve things only on some specific cases (kernel boot), while increasing the I/D cache and

Re: [Qemu-devel] vNVRAM / blobstore design

2013-03-29 Thread Anthony Liguori
Stefan Berger stef...@linux.vnet.ibm.com writes: On 03/28/2013 01:39 PM, Michael S. Tsirkin wrote: On Thu, Mar 28, 2013 at 12:27:45PM -0500, Anthony Liguori wrote: Stefan Berger stef...@linux.vnet.ibm.com writes: On 03/27/2013 03:12 PM, Stefan Berger wrote: On 03/27/2013 02:27 PM, Anthony

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Christophe Lyon
Hi, Sorry I wasn't aware of such restrictions (I'm used to gcc/binutils conventions), and my thunderbird does display my patch as if it were inline :-) So trying a cut-and-paste approach in gmail, for this one character patch: /proc/self/maps: Add a space at end of line when there is no

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Eric Blake
On 03/29/2013 09:13 AM, Christophe Lyon wrote: Hi, Sorry I wasn't aware of such restrictions (I'm used to gcc/binutils conventions), and my thunderbird does display my patch as if it were inline :-) So trying a cut-and-paste approach in gmail, for this one character patch: Still not

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Christophe Lyon
Add a space at end of line when there is no filename to print, to conform to linux kernel format. Signed-off-by: Christophe Lyon christophe.l...@linaro.org --- linux-user/syscall.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c

Re: [Qemu-devel] [PATCH 03/20] char: add IOWatchPoll support

2013-03-29 Thread Amit Shah
On (Fri) 29 Mar 2013 [09:03:10], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Fri) 29 Mar 2013 [07:24:07], Anthony Liguori wrote: Amit Shah amit.s...@redhat.com writes: On (Tue) 05 Mar 2013 [23:21:18], Amit Shah wrote: From: Anthony Liguori aligu...@us.ibm.com

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Eric Blake
On 03/29/2013 08:12 AM, Pavel Hrdina wrote: QMP command vm-snapshot-save has also extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina phrd...@redhat.com --- hmp-commands.hx

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Pavel Hrdina
On 29.3.2013 17:12, Eric Blake wrote: On 03/29/2013 08:12 AM, Pavel Hrdina wrote: QMP command vm-snapshot-save has also extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina

[Qemu-devel] [PATCH] qemu-char: rewrite io_channel_send_all and drop the '_all' suffix

2013-03-29 Thread Anthony Liguori
The current code is oddly written and have equally odd semantics. Despite the '_all' suffix, upon EAGAIN the result will be a partial write but instead of returning the partial write, we return EAGAIN. Change the behavior to write as much as we can until we get an EAGAIN returning a partial write

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

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 08:32:57AM -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 v3 01/20] tcg-arm: Use bic to implement and with constant

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 08:32:42AM -0700, Richard Henderson wrote: This greatly improves the code we can produce for deposit without armv7 support. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 52 ++--

Re: [Qemu-devel] [PATCH v3 02/20] tcg-arm: Handle negated constant arguments to and/sub

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 08:32:43AM -0700, Richard Henderson wrote: This greatly improves code generation for addition of small negative constants. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 41 +++-- 1 file changed, 35

Re: [Qemu-devel] [PATCH v3 03/20] tcg-arm: Allow constant first argument to sub

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 08:32:44AM -0700, Richard Henderson wrote: This allows the generation of RSB instructions. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git

Re: [Qemu-devel] [PATCH v3 04/20] tcg-arm: Use tcg_out_dat_rIN for compares

2013-03-29 Thread Aurelien Jarno
On Thu, Mar 28, 2013 at 08:32:45AM -0700, Richard Henderson wrote: This allows us to emit CMN instructions. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff

[Qemu-devel] [PATCH] vmware_vga: do not cache depth and bypp

2013-03-29 Thread Alex DAMIAN
From: Alexandru DAMIAN alexandru.dam...@intel.com Do not cache depth and bypp information in the device state. This resolves a bug where Xorg video-vmare driver refuses to start up because the depth value read is the one cached from the device start (default 32 from ui/console.c) and it is not

[Qemu-devel] [PATCH] w32: Fix build with older gcc (unresolved symbol)

2013-03-29 Thread Stefan Weil
The cross i586-mingw32msvc-gcc 4.4.4 from Debian Squeeze does not support __sync_val_compare_and_swap by default. Using -march=i686 fixes that and should also result in better code. Signed-off-by: Stefan Weil s...@weilnetz.de --- Maybe this modification is also needed for native gcc-4.4 and

Re: [Qemu-devel] [Xen-devel] [PATCHv3] QEMU(upstream): Disable xen's use of O_DIRECT by default as it results in crashes.

2013-03-29 Thread Stefano Stabellini
On Wed, 20 Mar 2013, Paolo Bonzini wrote: Il 20/03/2013 09:33, Alex Bligh ha scritto: Stefano, --On 19 March 2013 15:13:29 + Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: Therefore I think that the current change is not safe, but it is pretty easy to make it safe.

Re: [Qemu-devel] vNVRAM / blobstore design

2013-03-29 Thread Kenneth Goldman
One thing I'd like to get clarity about is the following corner-case. A user supplies some VM image as persistent storage for the TPM. It contains garbage. How do we handle this case? Does the TPM then just start writing its state into this image or do we want to have some layer in place

Re: [Qemu-devel] [PATCH 14/42] migration: prepare to access s-state outside critical sections

2013-03-29 Thread Stefan Weil
Am 25.03.2013 12:02, schrieb Paolo Bonzini: - Messaggio originale - Da: Gerd Hoffmann kra...@redhat.com A: Stefan Hajnoczi stefa...@gmail.com Cc: Paolo Bonzini pbonz...@redhat.com, qemu-devel qemu-devel@nongnu.org, Orit Wasserman owass...@redhat.com, Juan Quintela

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread Peter Maydell
On 29 March 2013 16:01, Christophe Lyon christophe.l...@linaro.org wrote: Add a space at end of line when there is no filename to print, to conform to linux kernel format. Signed-off-by: Christophe Lyon christophe.l...@linaro.org Reviewed-by: Peter Maydell peter.mayd...@linaro.org ---

Re: [Qemu-devel] [PATCH] w32: Fix build with older gcc (unresolved symbol)

2013-03-29 Thread Peter Maydell
On 29 March 2013 17:20, Stefan Weil s...@weilnetz.de wrote: Maybe this modification is also needed for native gcc-4.4 and older on Linux i386. If yes, we can move the new script code out of the MinGW conditional code. There are at least a few people who want to run on older CPUs still (though

[Qemu-devel] [PATCH v2] rbd: add an asynchronous flush

2013-03-29 Thread Josh Durgin
The existing bdrv_co_flush_to_disk implementation uses rbd_flush(), which is sychronous and causes the main qemu thread to block until it is complete. This results in unresponsiveness and extra latency for the guest. Fix this by using an asynchronous version of flush. This was added to librbd

Re: [Qemu-devel] [PATCH V10 08/17] block: move qmp_query_block() and bdrv_query_info() to block/qapi.c

2013-03-29 Thread Eric Blake
On 03/22/2013 08:19 AM, Wenchao Xia wrote: This is a code move patch, except in qmp_query_block bdrv_next(bs) is used instead of direct traverse of global array 'bdrv_states'. Mixing code motion and a code change isn't always the best, but at least you were honest about it. I don't know how

[Qemu-devel] [PATCH] microblaze: Add support for the sleep insn

2013-03-29 Thread edgar . iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-microblaze/translate.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/target-microblaze/translate.c b/target-microblaze/translate.c index

Re: [Qemu-devel] [Patch] fix /proc/self/maps output

2013-03-29 Thread H. Peter Anvin
On 03/29/2013 09:01 AM, Christophe Lyon wrote: Add a space at end of line when there is no filename to print, to conform to linux kernel format. Signed-off-by: Christophe Lyon christophe.l...@linaro.org --- linux-user/syscall.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [Qemu-devel] [PATCH V10 10/17] qmp: add recursive member in ImageInfo

2013-03-29 Thread Eric Blake
On 03/22/2013 08:19 AM, Wenchao Xia wrote: New member *backing-image is added to reflect the backing chain status. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- block/qapi.c |6 +- qapi-schema.json |5 - 2 files changed, 9 insertions(+), 2 deletions(-)

Re: [Qemu-devel] [PATCH V10 12/17] hmp: add function hmp_info_snapshots()

2013-03-29 Thread Eric Blake
On 03/22/2013 08:19 AM, Wenchao Xia wrote: This function will simply call qmp interface qmp_query_snapshots() added in last commit and then dump information in monitor console. To get snapshot info, Now qemu and qemu-img both call block layer function bdrv_query_snapshot_info_list() in

[Qemu-devel] [PATCH] tweak lookup_feature() in target-i386/cpu.c

2013-03-29 Thread Michael W. Bombardieri
Hi, The following patch removes variable 'found' from lookup_feature(). We can just return true/false (found/not found) directly. Does this look OK? - Michael diff --git a/target-i386/cpu.c b/target-i386/cpu.c index a0640db..9147aaf 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@

[Qemu-devel] [PATCH 0/5] target-i386: add PCLMULQDQ and AES-NI instructions

2013-03-29 Thread Aurelien Jarno
This patch series adds the PCLMULQDQ and AES-NI instructions to the x86 emulation. Along with the SSE4.1 and SSE4.2 series, this brings the instructions emulation to the level of a Westmere CPU. It has been tested with the valgrind testsuite and with the kernel autotest. Changes v1 - v2: - Patch

[Qemu-devel] [PATCH 1/5] target-i386: add pclmulqdq instruction

2013-03-29 Thread Aurelien Jarno
Reviewed-by: Richard Henderson r...@twiddle.net Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- target-i386/cpu.c| 19 +-- target-i386/ops_sse.h| 24 target-i386/ops_sse_header.h |5 + target-i386/translate.c |

  1   2   >