[Qemu-devel] [RFC PATCH 4/5] eepro100: Add a dev field to eeprom new/free functions

2010-06-14 Thread Alex Williamson
This allows us to create a more meaningful savevm string. Signed-off-by: Alex Williamson alex.william...@redhat.com --- hw/eepro100.c |4 ++-- hw/eeprom93xx.c |8 hw/eeprom93xx.h |4 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/hw/eepro100.c

[Qemu-devel] [RFC PATCH 5/5] virtio-net: Incorporate a DeviceState pointer and let savevm track instances

2010-06-14 Thread Alex Williamson
Stuff a pointer to the DeviceState into the VirtIONet structure so that we can easily remove the vmstate entry later. Also, let vmstate track the instance number (it should always be zero internally since the device path should now be unique). Signed-off-by: Alex Williamson

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Markus Armbruster
Alex Williamson alex.william...@redhat.com writes: qdev_get_dev_path() is intended to be the canonical utility for creating a string representing the qdev hierarchy of a device. The path consists of bus and device names as well as identified properties of the immediate parent bus and device.

[Qemu-devel] [PATCH] hpet: Init capability register only once

2010-06-14 Thread Jan Kiszka
From: Jan Kiszka jan.kis...@siemens.com The capability register is read-only from guest POV, so we do not need to update it on reset. Signed-off-by: Jan Kiszka jan.kis...@siemens.com --- hw/hpet.c |9 + 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/hw/hpet.c

[Qemu-devel] Re: [RFC PATCH 0/5] Introduce canonical device hierarchy string

2010-06-14 Thread Gerd Hoffmann
Hi, My premise with this attempt is that we walk the hierarchy and use the names to create the base of the path. As we get to the device, particularly to the parent bus of the device, we need to start looking at properties to ensure uniqueness. You'll need that for every bus along the way

[Qemu-devel] [PATCHv2] pass info about hpets to seabios.]

2010-06-14 Thread Gleb Natapov
Currently HPET ACPI table is created regardless of whether qemu actually created hpet device. This may confuse some guests that don't check that hpet is functional before using it. Solve this by passing info about hpets in qemu to seabios via fw config interface. Additional benefit is that seabios

[Qemu-devel] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Gleb Natapov
Load hpet info for HPET ACPI table from qemu instead of using hardcoded values. Use hardcoded values anyway if old qemu is detected. Signed-off-by: Gleb Natapov g...@redhat.com diff --git a/src/acpi.c b/src/acpi.c index 0559443..864f1a8 100644 --- a/src/acpi.c +++ b/src/acpi.c @@ -469,7 +469,7

Re: [Qemu-devel] [PATCH v2 2/7] ioapic: convert to qdev

2010-06-14 Thread Markus Armbruster
Blue Swirl blauwir...@gmail.com writes: Convert to qdev. Signed-off-by: Blue Swirl blauwir...@gmail.com --- hw/apic.h|2 -- hw/ioapic.c | 45 ++--- hw/pc.h |4 +++- hw/pc_piix.c | 19 ++- 4 files changed, 51

Re: [Qemu-devel] [PATCH 2/2] Return usb device to host on exit

2010-06-14 Thread Gerd Hoffmann
@@ -1066,6 +1077,7 @@ USBDevice *usb_host_device_open(const char *devname) qdev_prop_set_uint32(dev-qdev, vendorid, filter.vendor_id); qdev_prop_set_uint32(dev-qdev, productid, filter.product_id); qdev_init_nofail(dev-qdev); +atexit(usb_host_cleanup); return dev;

[Qemu-devel] [PATCH 0/5] [QEMU-KVM]: Add BSG backstore using struct sg_io_v4

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org Quick resend with project subject for cover letter.. Greetings Gerd, Hannes and co, This series adds initial support for a hw/scsi-bsg.c backstore for scsi-bus compatible HBA emulation in QEMU-KVM on Linux hosts supporting the BSG driver. This code

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-14 Thread Markus Armbruster
Blue Swirl blauwir...@gmail.com writes: Make APICState completely private to apic.c by using DeviceState in external APIs. Could you explain why this is an improvement?

[Qemu-devel] [PATCH 4/5] [scsi]: Add BSG support for scsi-bus and scsi-disk

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates hw/scsi-bus.c:scsi_bus_legacy_add_drive() to check for the scsi-bsg backstore. It also updates hw/scsi-disk.c:scsi_disk_initfn() to check for when bdrv_is_bsg() is present and we need to fail for the fileio backed scsi-disk code.

[Qemu-devel] [Bug 524447] Re: virsh save is very slow

2010-06-14 Thread Dave Walker
Changing to libvirt as commentary here, and on the upstream bug report by Cole indicate a fix has been commit that improves this performance. ** Package changed: qemu-kvm (Ubuntu) = libvirt (Ubuntu) ** Also affects: libvirt via https://bugzilla.redhat.com/show_bug.cgi?id=599091 Importance:

[Qemu-devel] [PATCH 3/5] [block]: Add paio_submit_len() non sector sized AIO

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds posix-aio-compat.c:paio_submit_len(), which is a identical to paio_submit() expect that in expected nb_len instead of nb_sectors (* 512) so that it can be used by BSG AIO for write()/read() of struct sg_io_v4. Signed-off-by: Nicholas

Re: [Qemu-devel] [PATCH] qemu-iotests: qcow2 error path tests

2010-06-14 Thread Kevin Wolf
Am 14.06.2010 11:38, schrieb Christoph Hellwig: On Fri, Jun 04, 2010 at 07:35:24PM +0200, Kevin Wolf wrote: This adds test cases for qcow2 error paths (using blkdebug) Thanks, applied. What's the plan for getting the fixes this tests into mainline? The test passes in the block branch, but

[Qemu-devel] [PATCH 0/5] *** SUBJECT HERE ***

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org Greetings Gerd, Hannes and co, This series adds initial support for a hw/scsi-bsg.c backstore for scsi-bus compatible HBA emulation in QEMU-KVM on Linux hosts supporting the BSG driver. This code is available from the scsi-bsg branch in the

[Qemu-devel] [PATCH 5/5] [scsi-bsg]: Add initial support for BSG based SCSIDeviceInfo

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds initial support for using the Linux BSG interface with write/read vectored AIO as a QEMU backstore (SCSIDeviceInfo) with hw/scsi-bus.c compatible HBA emulation. So far it has been tested with x86_64 host and guest using hw/megasas.c

[Qemu-devel] [PATCH 1/5] [block]: Add top level BSG support

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds top level BSG support to QEMU-KVM block and adds the BDS_* prefixed defines for SG_IO and BSG. It adds the BDS_SCSI_GENERIC and BDS_BSG assignments in block/raw-posix.c:hdev_open() using S_ISCHR() and major(st.st_rdev) in order to

[Qemu-devel] [PATCH 2/5] [block]: Add BSG qemu_open() in block/raw.c:raw_open()

2010-06-14 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds a BSG specific qemu_open() call in block/raw.c:raw_open() that saves the opened file descriptor for BSG AIO into BlockDriverState-fd. It also adds the reverse close() call to block/raw.c:raw_close() Signed-off-by: Nicholas A.

Re: [Qemu-devel] [PATCH] qemu-iotests: qcow2 error path tests

2010-06-14 Thread Christoph Hellwig
On Fri, Jun 04, 2010 at 07:35:24PM +0200, Kevin Wolf wrote: This adds test cases for qcow2 error paths (using blkdebug) Thanks, applied. What's the plan for getting the fixes this tests into mainline?

Re: [Qemu-devel] [Bug 524447] Re: virsh save is very slow

2010-06-14 Thread Michael Tokarev
14.06.2010 13:37, Dave Walker wrote: Changing to libvirt as commentary here, and on the upstream bug report by Cole indicate a fix has been commit that improves this performance. Um. This is not that simple, apparently. I did some tests after this bug were discussed/mentioned last time, and

[Qemu-devel] [Bug 524447] Re: virsh save is very slow

2010-06-14 Thread Dave Walker
Re-introducing qemu-kvm, as commentary on qemu-devel mailing list suggest there could be a timing concern meaning poor performance. Leaving Libvirt on this report, as upstream libvirt have quoted improved performance adjusting the block size for dd. However, Qemu feel that the real issue is in

[Qemu-devel] Re: [PATCH 3/5] [block]: Add paio_submit_len() non sector sized AIO

2010-06-14 Thread Christoph Hellwig
On Mon, Jun 14, 2010 at 02:44:31AM -0700, Nicholas A. Bellinger wrote: From: Nicholas Bellinger n...@linux-iscsi.org This patch adds posix-aio-compat.c:paio_submit_len(), which is a identical to paio_submit() expect that in expected nb_len instead of nb_sectors (* 512) so that it can be used

[Qemu-devel] [PATCH] SeaBIOS: Fix bvprintf() to respect padding for hex printing.

2010-06-14 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Fix bvprintf to respect space padding when printing hex numbers and the caller specifies alignment without zero padding, eg. %2x as opposed to %02x Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- src/output.c | 27 +--

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Alex Williamson
On Mon, 2010-06-14 at 08:39 +0200, Markus Armbruster wrote: Alex Williamson alex.william...@redhat.com writes: qdev_get_dev_path() is intended to be the canonical utility for creating a string representing the qdev hierarchy of a device. The path consists of bus and device names as well

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Paul Brook
/main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be something like: /main-system-bus/piix4-pcihost/pci.0/_09.0 Could you explain why you add identified properties of the immediate parent bus and device? They

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Jan Kiszka
Alex Williamson wrote: On Mon, 2010-06-14 at 08:39 +0200, Markus Armbruster wrote: Alex Williamson alex.william...@redhat.com writes: qdev_get_dev_path() is intended to be the canonical utility for creating a string representing the qdev hierarchy of a device. The path consists of bus and

[Qemu-devel] ARM/system mode/stdin

2010-06-14 Thread Christophe LYON
Hello, I am trying to use qemu-system-arm (0.12.3) to execute an ARM bare machine program (not a Linux kernel), and I have some trouble when the program in question tries to read from stdin. The program does use ARM semihosting to communicate with the host. Here is the sample code:

[Qemu-devel] Q35 qemu repository?

2010-06-14 Thread Matthew Garrett
Hi there, I'm currently doing some work on tidying up Linux's PCIe ASPM support, and one thing that would be useful would be to be able to instrument reads and writes made by Windows to PCIe space. I noticed that you've been working on Q35 support for qemu - is there a public repository which

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/12/2010 06:14 AM, Juan Quintela wrote: Anthony Liguorianth...@codemonkey.ws wrote: On 06/11/2010 09:30 AM, Luiz Capitulino wrote: On Thu, 10 Jun 2010 12:44:55 +0200 Juan Quintelaquint...@redhat.com wrote: I think we've more or less agreed that

[Qemu-devel] Re: [SeaBIOS] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Kevin O'Connor
On Mon, Jun 14, 2010 at 11:30:53AM +0300, Gleb Natapov wrote: Load hpet info for HPET ACPI table from qemu instead of using hardcoded values. Use hardcoded values anyway if old qemu is detected. The current code does a lot of mixing of qemu provided and seabios provided data to build the acpi

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Luiz Capitulino
On Mon, 14 Jun 2010 08:58:19 -0500 Anthony Liguori aligu...@linux.vnet.ibm.com wrote: For 0.13, we need to focus on introducing the least disruptive change that addresses the fundamental requirement--allow clients to avoid a polling loop for determining when migration ends. Having a single

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 09:24 AM, Luiz Capitulino wrote: On Mon, 14 Jun 2010 08:58:19 -0500 Anthony Liguorialigu...@linux.vnet.ibm.com wrote: For 0.13, we need to focus on introducing the least disruptive change that addresses the fundamental requirement--allow clients to avoid a polling loop for

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Luiz Capitulino
On Sat, 12 Jun 2010 13:20:54 +0200 Juan Quintela quint...@redhat.com wrote: Luiz Capitulino lcapitul...@redhat.com wrote: On Fri, 11 Jun 2010 09:38:42 -0500 Anthony Liguori anth...@codemonkey.ws wrote: 1. QMP only returns the response when the command is finished, eg: C: {

[Qemu-devel] [PATCH 1/2] qcow2: Simplify image creation

2010-06-14 Thread Kevin Wolf
Instead of doing lots of magic for setting up initial refcount blocks and stuff create a minimal (inconsistent) image, open it and initialize the rest with regular qcow2 functions. This is a complete rewrite of the image creation function. The old implementating is #ifdef'd out and will be

[Qemu-devel] [PATCH 2/2] qcow2: Remove old image creation function

2010-06-14 Thread Kevin Wolf
They have been #ifdef'd out by the previous patch. Signed-off-by: Kevin Wolf kw...@redhat.com --- block/qcow2.c | 224 - 1 files changed, 0 insertions(+), 224 deletions(-) diff --git a/block/qcow2.c b/block/qcow2.c index acb850c..6f26564

[Qemu-devel] [PATCH 1/2] Remove unused DEBUG defines from hw/msix.c

2010-06-14 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Remove unused DEBUG defines from hw/msix.c to avoid having anything define the word DEBUG without any additions such as MSIX_DEBUG. Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- hw/msix.c |9 - 1 files changed, 0 insertions(+), 9

[Qemu-devel] [PATCH 2/2] Change #define DEBUG to #define E1000_DEBUG in hw/e1000.c

2010-06-14 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Change #define DEBUG to #define E1000_DEBUG in hw/e1000.c to make it possible to build QEMU with -DDEBUG Signed-off-by: Jes Sorensen jes.soren...@redhat.com --- hw/e1000.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[Qemu-devel] [RESENT PATCH 2/2] vnc: sync lock modifier state on connect.

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- vnc.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/vnc.c b/vnc.c index b25b6a1..039fb21 100644 --- a/vnc.c +++ b/vnc.c @@ -2300,6 +2300,7 @@ static void vnc_connect(VncDisplay *vd, int csock) if

[Qemu-devel] [RESENT PATCH 1/2] switch keyboard led state notification to notifiers.

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- console.h | 11 +++ input.c | 37 - vnc.c | 13 - vnc.h |2 +- 4 files changed, 28 insertions(+), 35 deletions(-) diff --git a/console.h b/console.h index

[Qemu-devel] [RfC PATCH] add pflib: PixelFormat conversion library.

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- Makefile.objs |1 + pflib.c | 204 + pflib.h |6 ++ 3 files changed, 211 insertions(+), 0 deletions(-) create mode 100644 pflib.c create mode 100644 pflib.h diff

[Qemu-devel] [RESENT PATCH] Add exit notifiers.

2010-06-14 Thread Gerd Hoffmann
Hook up any cleanup work which needs to be done here. Advantages over using atexit(3): (1) You get passed in a pointer to the notifier. If you embed that into your state struct you can use container_of() to get get your state info. (2) You can unregister, say when un-plugging a

[Qemu-devel] [RESENT PATCH 3/3] Fix and simplify gui timer logic.

2010-06-14 Thread Gerd Hoffmann
Kill nographic timer. Have a global gui_timer instead. Have the gui timer enabled unconditionally. We need a timer running anyway for mmio flush, so the whole have-gui-timer-only-when-needed logic is pretty pointless. It also simplifies displaylisteners coming and going at runtime, we don't

[Qemu-devel] [RESENT PATCH 2/3] add unregister_displaychangelistener

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- console.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/console.h b/console.h index 3a80dca..a0da498 100644 --- a/console.h +++ b/console.h @@ -227,6 +227,11 @@ static inline void

Re: [Qemu-devel] [RfC PATCH] add pflib: PixelFormat conversion library.

2010-06-14 Thread Anthony Liguori
On 06/14/2010 10:25 AM, Gerd Hoffmann wrote: Signed-off-by: Gerd Hoffmannkra...@redhat.com --- Makefile.objs |1 + pflib.c | 204 + pflib.h |6 ++ 3 files changed, 211 insertions(+), 0 deletions(-) create mode

[Qemu-devel] [RESENT PATCH 1/3] QLIST-ify display change listeners.

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- console.h | 72 +++ hw/xenfb.c |2 +- vl.c |9 ++- 3 files changed, 42 insertions(+), 41 deletions(-) diff --git a/console.h b/console.h index cac959f..3a80dca 100644

[Qemu-devel] [RfC PATCH] Fix vnc memory corruption with width = 1400

2010-06-14 Thread Gerd Hoffmann
vnc assumes that the screen width is a multiple of 16 in several places. If this is not the case vnc will overrun buffers, corrupt memory, make qemu crash. This is the minimum fix for this bug. It makes sure we don't overrun the scanline, thereby fixing the segfault. The rendering is *not*

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Alex Williamson
On Mon, 2010-06-14 at 14:09 +0100, Paul Brook wrote: /main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be something like: /main-system-bus/piix4-pcihost/pci.0/_09.0 Ok, I can easily come up with:

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Paul Brook
On Mon, 2010-06-14 at 14:09 +0100, Paul Brook wrote: /main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be something like: /main-system-bus/piix4-pcihost/pci.0/_09.0 Ok, I can easily come up with:

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Luiz Capitulino lcapitul...@redhat.com wrote: On Sat, 12 Jun 2010 13:20:54 +0200 Juan Quintela quint...@redhat.com wrote: Both. What does it report in the source? That migration has started :) Nothing else, nothing less. Think again multiple monitors and/or audit. Why can't we just

[Qemu-devel] [PATCH v2] add pflib: PixelFormat conversion library.

2010-06-14 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann kra...@redhat.com --- Makefile.objs |1 + pflib.c | 213 + pflib.h |6 ++ 3 files changed, 220 insertions(+), 0 deletions(-) create mode 100644 pflib.c create mode 100644 pflib.h diff

Re: [Qemu-devel] [PATCH v6 4/6] Add support for marking memory to not be migrated. On migration, memory is checked for the NO_MIGRATION_FLAG.

2010-06-14 Thread Anthony Liguori
On 06/04/2010 04:45 PM, Cam Macdonell wrote: This is useful for devices that do not want to take memory regions data with them on migration. --- arch_init.c | 28 cpu-all.h|2 ++ cpu-common.h |2 ++ exec.c | 12 4 files

Re: [Qemu-devel] Re: [PATCH v6 0/6] Inter-VM Shared Memory Device with migration support

2010-06-14 Thread Anthony Liguori
On 06/11/2010 05:03 PM, Cam Macdonell wrote: Hi Anthony, Is my implementation of master/peer roles acceptable? Yes, it looks good. I realize with Alex's RAMList changes I may need to modify my patch, but is the approach of marking memory non-migratable an acceptable implementation?

Re: [Qemu-devel] [PATCH v6 6/6] the stand-alone shared memory server for inter-VM shared memory

2010-06-14 Thread Anthony Liguori
On 06/04/2010 04:45 PM, Cam Macdonell wrote: this code is a standalone server which will pass file descriptors for the shared memory region and eventfds to support interrupts between guests using inter-VM shared memory. --- contrib/ivshmem-server/Makefile | 16 ++

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Jan Kiszka
Alex Williamson wrote: On Mon, 2010-06-14 at 14:09 +0100, Paul Brook wrote: /main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be something like: /main-system-bus/piix4-pcihost/pci.0/_09.0 Ok, I can easily come

Re: [Qemu-devel] [PATCH 00/10][PULL]: QMP/Monitor queue

2010-06-14 Thread Anthony Liguori
On 06/11/2010 02:58 PM, Luiz Capitulino wrote: Hi Anthony, The following QMP/Monitor patches have been sent to the list and look good to me. I have also tested most of them. The changes (since 0e2029a063405091ee34170ef71aa321715e4357) are available in the following repository:

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Anthony Liguori aligu...@linux.vnet.ibm.com wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulinolcapitul...@redhat.com wrote: The monitor that did it knows it, nobody else knows it. At destination time, I guess you agree this is important, i.e. the management app knows that

Re: [Qemu-devel] Re: [SeaBIOS] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Gleb Natapov
On Mon, Jun 14, 2010 at 03:40:16PM +0100, Jamie Lokier wrote: Gleb Natapov wrote: On Mon, Jun 14, 2010 at 09:54:25AM -0400, Kevin O'Connor wrote: Could we just have qemu build the hpet tables and pass them through to seabios? Perhaps using the qemu_cfg_acpi_additional_tables() method.

Re: [Qemu-devel] [PATCH v6 4/6] Add support for marking memory to not be migrated. On migration, memory is checked for the NO_MIGRATION_FLAG.

2010-06-14 Thread Anthony Liguori
On 06/14/2010 11:08 AM, Cam Macdonell wrote: On Mon, Jun 14, 2010 at 9:51 AM, Anthony Liguorianth...@codemonkey.ws wrote: On 06/04/2010 04:45 PM, Cam Macdonell wrote: This is useful for devices that do not want to take memory regions data with them on migration. --- arch_init.c |

[Qemu-devel] ARM Cortex-M3 Resets

2010-06-14 Thread Christopher Johnson
I am attempting to run FreeRTOS under qemu-system-arm 0.12. I am compiling from source. At the current time arm-test works fine. It uses a boot loader with the expectation that the PC=0 after Reset is de-asserted. The CORTEXT-M3 reference states: NVIC resets, holds core in reset NVIC clears

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 11:02 AM, Juan Quintela wrote: Anthony Liguorialigu...@linux.vnet.ibm.com wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulinolcapitul...@redhat.com wrote: The monitor that did it knows it, nobody else knows it. At destination time, I

Re: [Qemu-devel] [PATCH v6 4/6] Add support for marking memory to not be migrated. On migration, memory is checked for the NO_MIGRATION_FLAG.

2010-06-14 Thread Cam Macdonell
On Mon, Jun 14, 2010 at 9:51 AM, Anthony Liguori anth...@codemonkey.ws wrote: On 06/04/2010 04:45 PM, Cam Macdonell wrote: This is useful for devices that do not want to take memory regions data with them on migration. ---  arch_init.c  |   28  cpu-all.h    |    

Re: [Qemu-devel] [PATCH 15/35] tcg-s390: Query instruction extensions that are installed.

2010-06-14 Thread Richard Henderson
On 06/13/2010 03:23 PM, Alexander Graf wrote: On 13.06.2010, at 18:44, Aurelien Jarno wrote: Is KVM in 31-bit mode actually functional? I'm not aware of anything preventing it to be. But I honestly haven't tried. As long as all hypercall parameters stay within the first 32/31 bits, things

Re: [Qemu-devel] [PATCH] qemu-option: Fix uninitialized value in append_option_parameter

2010-06-14 Thread Anthony Liguori
On 06/11/2010 03:19 AM, Kevin Wolf wrote: When dest is NULL, i.e. a new copy of the list is created, we don't get a properly terminated list after the realloc. Initialize it as an empty list. Signed-off-by: Kevin Wolfkw...@redhat.com Applied. Thanks. Regards, Anthony Liguori ---

Re: [Qemu-devel] [PATCH] Make netdev_del delete the netdev even when it's in use

2010-06-14 Thread Anthony Liguori
On 06/11/2010 07:21 AM, Markus Armbruster wrote: To hot-unplug guest and host part of a network device, you do: device_del NIC-ID netdev_del NETDEV-ID For PCI devices, device_del merely tells ACPI to unplug the device. The device goes away for real only after the guest processed the

Re: [Qemu-devel] [PATCH v2] ram_blocks: Convert to a QLIST

2010-06-14 Thread Anthony Liguori
On 06/11/2010 12:11 PM, Alex Williamson wrote: This makes the RAM block list easier to manipulate. Also incorporate relevant variables into the RAMList struct. Signed-off-by: Alex Williamsonalex.william...@redhat.com Acked-by: Chris Wrightchr...@redhat.com Applied. Thanks. Regards,

Re: [Qemu-devel] [PATCH] hpet: Init capability register only once

2010-06-14 Thread Anthony Liguori
On 06/14/2010 01:40 AM, Jan Kiszka wrote: From: Jan Kiszkajan.kis...@siemens.com The capability register is read-only from guest POV, so we do not need to update it on reset. Signed-off-by: Jan Kiszkajan.kis...@siemens.com Applied. Thanks. Regards, Anthony Liguori --- hw/hpet.c |

Re: [Qemu-devel] [PATCH 1/2] Remove unused DEBUG defines from hw/msix.c

2010-06-14 Thread Anthony Liguori
On 06/14/2010 10:05 AM, jes.soren...@redhat.com wrote: From: Jes Sorensenjes.soren...@redhat.com Remove unused DEBUG defines from hw/msix.c to avoid having anything define the word DEBUG without any additions such as MSIX_DEBUG. Signed-off-by: Jes Sorensenjes.soren...@redhat.com Applied.

Re: [Qemu-devel] [PATCHv2] pass info about hpets to seabios.]

2010-06-14 Thread Anthony Liguori
On 06/14/2010 03:29 AM, Gleb Natapov wrote: Currently HPET ACPI table is created regardless of whether qemu actually created hpet device. This may confuse some guests that don't check that hpet is functional before using it. Solve this by passing info about hpets in qemu to seabios via fw config

Re: [Qemu-devel] [PATCH v2] add pflib: PixelFormat conversion library.

2010-06-14 Thread Anthony Liguori
On 06/14/2010 10:46 AM, Gerd Hoffmann wrote: Signed-off-by: Gerd Hoffmannkra...@redhat.com --- Makefile.objs |1 + pflib.c | 213 + pflib.h |6 ++ 3 files changed, 220 insertions(+), 0 deletions(-) create mode

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Alex Williamson
On Mon, 2010-06-14 at 18:00 +0200, Jan Kiszka wrote: Alex Williamson wrote: On Mon, 2010-06-14 at 14:09 +0100, Paul Brook wrote: /main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be something like:

[Qemu-devel] [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Paolo Bonzini
lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the spec: [The PMJCTL] bit controls which decision mechanism is used when jumping on phase

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Jan Kiszka
Alex Williamson wrote: On Mon, 2010-06-14 at 18:00 +0200, Jan Kiszka wrote: Alex Williamson wrote: On Mon, 2010-06-14 at 14:09 +0100, Paul Brook wrote: /main-system-bus/pci.0,addr=09.0/virtio-blk-pci There's a device missing between the main system bus and the pci bus. Should be

Re: [Qemu-devel] [PATCH 0/5] Add '-device help' output for device params and help text

2010-06-14 Thread Anthony Liguori
On 06/08/2010 12:21 AM, Amit Shah wrote: On (Mon) Jun 07 2010 [11:09:32], Anthony Liguori wrote: On 05/31/2010 07:41 AM, Amit Shah wrote: Hello, This patch series adds support to specify some descriptive help text to qdev device parameters. This series adds some help text to the

Re: [Qemu-devel] [PATCH] monitor: Add force option support to pci_del command

2010-06-14 Thread Anthony Liguori
On 06/09/2010 09:27 AM, Gerd Hoffmann wrote: Hi, This make sense when you mistakenly add a pci device on a -s -S scenario, like the scenario described on the following bug: https://bugs.launchpad.net/qemu/+bug/544367. It doesn't IMHO. When ACPI-based hotplug support is present on the

Re: [Qemu-devel] [PATCH] Add optional dump of default config file paths..

2010-06-14 Thread Anthony Liguori
On 06/09/2010 03:05 AM, john cooper wrote: This patch adds the ability to determine the build-configured runtime config file paths from the command line. After support for cpu model definitions were added to the default runtime target- config file, testing of this feature has tripped over an

[Qemu-devel] Re: [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Jan Kiszka
Paolo Bonzini wrote: lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the spec: [The PMJCTL] bit controls which decision mechanism is used

Re: [Qemu-devel] [PATCH] Add optional dump of default config file paths..

2010-06-14 Thread Daniel P. Berrange
On Mon, Jun 14, 2010 at 12:01:42PM -0500, Anthony Liguori wrote: On 06/09/2010 03:05 AM, john cooper wrote: This patch adds the ability to determine the build-configured runtime config file paths from the command line. After support for cpu model definitions were added to the default runtime

Re: [Qemu-devel] [PATCH] Fix and simplify gui timer logic.

2010-06-14 Thread Anthony Liguori
On 06/08/2010 08:18 AM, Gerd Hoffmann wrote: On 06/08/10 13:50, Paul Brook wrote: Kill nographic timer. Have a global gui_timer instead. Have the gui timer enabled unconditionally. We need a timer running anyway for mmio flush, so the whole have-gui-timer-only-when-needed logic is pretty

Re: [Qemu-devel] Re: [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Michal Novotny
On 06/14/2010 07:05 PM, Jan Kiszka wrote: Paolo Bonzini wrote: lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the spec: [The PMJCTL]

[Qemu-devel] [PATCH v2] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Paolo Bonzini
lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the spec: [The PMJCTL] bit controls which decision mechanism is used when jumping on phase

Re: [Qemu-devel] Re: [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Jan Kiszka
Michal Novotny wrote: On 06/14/2010 07:05 PM, Jan Kiszka wrote: Paolo Bonzini wrote: lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the

Re: [Qemu-devel] Re: [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Michal Novotny
On 06/14/2010 07:05 PM, Jan Kiszka wrote: Paolo Bonzini wrote: lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it breaks Windows 64-bit guests. This is the text from the spec: [The PMJCTL]

Re: [Qemu-devel] Re: [PATCH] lsi53c895a: fix Phase Mismatch Jump

2010-06-14 Thread Michal Novotny
On 06/14/2010 07:31 PM, Jan Kiszka wrote: Michal Novotny wrote: On 06/14/2010 07:05 PM, Jan Kiszka wrote: Paolo Bonzini wrote: lsi_bad_phase has a bug in the choice of pmjad1/pmjad2. This does not matter with Linux guests because it uses just one routine for both, but it

Re: [Qemu-devel] [PATCH v2 2/7] ioapic: convert to qdev

2010-06-14 Thread Blue Swirl
On Mon, Jun 14, 2010 at 9:33 AM, Markus Armbruster arm...@redhat.com wrote: Blue Swirl blauwir...@gmail.com writes: Convert to qdev. Signed-off-by: Blue Swirl blauwir...@gmail.com ---  hw/apic.h    |    2 --  hw/ioapic.c  |   45 ++---  hw/pc.h      

Re: [Qemu-devel] [PATCH 15/35] tcg-s390: Query instruction extensions that are installed.

2010-06-14 Thread Alexander Graf
Am 14.06.2010 um 18:20 schrieb Richard Henderson r...@twiddle.net: On 06/13/2010 03:23 PM, Alexander Graf wrote: On 13.06.2010, at 18:44, Aurelien Jarno wrote: Is KVM in 31-bit mode actually functional? I'm not aware of anything preventing it to be. But I honestly haven't tried. As long as

Re: [Qemu-devel] [PATCH v2 7/7] apic: qdev conversion cleanup

2010-06-14 Thread Blue Swirl
On Mon, Jun 14, 2010 at 9:36 AM, Markus Armbruster arm...@redhat.com wrote: Blue Swirl blauwir...@gmail.com writes: Make APICState completely private to apic.c by using DeviceState in external APIs. Could you explain why this is an improvement? Outside of apic.c, there is no need to access

Re: [Qemu-devel] [PATCH 5/5] [scsi-bsg]: Add initial support for BSG based SCSIDeviceInfo

2010-06-14 Thread Blue Swirl
On Mon, Jun 14, 2010 at 9:44 AM, Nicholas A. Bellinger n...@linux-iscsi.org wrote: From: Nicholas Bellinger n...@linux-iscsi.org This patch adds initial support for using the Linux BSG interface with write/read vectored AIO as a QEMU backstore (SCSIDeviceInfo) with hw/scsi-bus.c compatible

Re: [Qemu-devel] [PATCH] Add optional dump of default config file paths..

2010-06-14 Thread john cooper
Anthony Liguori wrote: On 06/09/2010 03:05 AM, john cooper wrote: This patch adds the ability to determine the build-configured runtime config file paths from the command line. After support for cpu model definitions were added to the default runtime target- config file, testing of this

[Qemu-devel] Re: [SeaBIOS] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Kevin O'Connor
On Mon, Jun 14, 2010 at 05:51:27PM +0300, Avi Kivity wrote: On 06/14/2010 05:09 PM, Gleb Natapov wrote: Could we just have qemu build the hpet tables and pass them through to seabios? Perhaps using the qemu_cfg_acpi_additional_tables() method. Possible, and I considered that. I personally

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Anthony Liguori aligu...@linux.vnet.ibm.com wrote: On 06/14/2010 11:02 AM, Juan Quintela wrote: Anthony Liguorialigu...@linux.vnet.ibm.com wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulinolcapitul...@redhat.com wrote: The monitor that did it

Re: [Qemu-devel] [RFC PATCH 1/5] qdev: Create qdev_get_dev_path()

2010-06-14 Thread Alex Williamson
On Mon, 2010-06-14 at 18:49 +0200, Jan Kiszka wrote: Alex Williamson wrote: On Mon, 2010-06-14 at 18:00 +0200, Jan Kiszka wrote: And instead of introducing another hierarchy level with the bus address, I would also prefer to add this as prefix or suffix to the device name, e.g.

[Qemu-devel] [PATCH] block: fix a warning and possible truncation

2010-06-14 Thread Blue Swirl
Fix a warning from OpenBSD gcc (3.3.5 (propolice)): /src/qemu/block.c: In function `bdrv_info_stats_bs': /src/qemu/block.c:1548: warning: long long int format, long unsigned int arg (arg 6) There may be also truncation effects. Signed-off-by: Blue Swirl blauwir...@gmail.com --- Alternatively

[Qemu-devel] Re: [SeaBIOS] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Gleb Natapov
On Mon, Jun 14, 2010 at 02:25:21PM -0400, Kevin O'Connor wrote: On Mon, Jun 14, 2010 at 05:51:27PM +0300, Avi Kivity wrote: On 06/14/2010 05:09 PM, Gleb Natapov wrote: Could we just have qemu build the hpet tables and pass them through to seabios? Perhaps using the

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 01:35 PM, Juan Quintela wrote: Anthony Liguorialigu...@linux.vnet.ibm.com wrote: On 06/14/2010 11:02 AM, Juan Quintela wrote: Anthony Liguorialigu...@linux.vnet.ibm.com wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz

Re: [Qemu-devel] [PATCH] Add optional dump of default config file paths..

2010-06-14 Thread Anthony Liguori
On 06/14/2010 12:59 PM, john cooper wrote: Anthony Liguori wrote: On 06/09/2010 03:05 AM, john cooper wrote: This patch adds the ability to determine the build-configured runtime config file paths from the command line. After support for cpu model definitions were added to the

[Qemu-devel] [PATCH] stop cpus before forking.

2010-06-14 Thread Glauber Costa
This patch fixes a bug that happens with kvm, irqchip-in-kernel, while adding a netdev. Despite the situations of reproduction being specific to kvm, I believe this fix is pretty generic, and fits here. Specially if we ever want to have our own irqchip in kernel too. The problem happens after the

Re: [Qemu-devel] Fwd: [PATCH 1/4] Make configure find uuid functions in Mac OS X by looking into libSystem.B

2010-06-14 Thread Anthony Liguori
On 06/03/2010 04:05 PM, C.W. Betts wrote: Since this didn't seem to get to the mailing list, I'm forwarding it. It's lacking a Signed-off-by and as a quoted message, it's not possible to apply it. Regards, Anthony Liguori Begin forwarded message: *From: *C.W. Betts

Re: [Qemu-devel] [PATCH] stop cpus before forking.

2010-06-14 Thread Anthony Liguori
On 06/14/2010 02:27 PM, Glauber Costa wrote: This patch fixes a bug that happens with kvm, irqchip-in-kernel, while adding a netdev. Despite the situations of reproduction being specific to kvm, I believe this fix is pretty generic, and fits here. Specially if we ever want to have our own

[Qemu-devel] Re: [SeaBIOS] [PATCHv2] load hpet info for HPET ACPI table from qemu

2010-06-14 Thread Anthony Liguori
On 06/14/2010 01:25 PM, Kevin O'Connor wrote: On Mon, Jun 14, 2010 at 05:51:27PM +0300, Avi Kivity wrote: On 06/14/2010 05:09 PM, Gleb Natapov wrote: Could we just have qemu build the hpet tables and pass them through to seabios? Perhaps using the qemu_cfg_acpi_additional_tables()

  1   2   >