[Qemu-devel] 回复: Re: 回复: Re: Which part of qemu responds to ACPI control method?

2013-07-04 Thread bobooscar
Thank you laszlo. however, after I got DSDT.dsl, I found that there is no “_PTS” method, even if “_TTS” “_GTS”.   Then I go through all the acpi tables, still found no PTS/TTS methods:     acpidump acpidump.out     acpixtract -a acpidump.out     for file in `ls |grep dat`; do iasl -a $file; done

Re: [Qemu-devel] [PATCH v3 2/2] net: introduce command to query rx-filter information

2013-07-04 Thread Markus Armbruster
Amos Kong ak...@redhat.com writes: On Tue, Jul 02, 2013 at 03:27:12PM +0200, Markus Armbruster wrote: Amos Kong ak...@redhat.com writes: On Tue, Jul 02, 2013 at 11:05:56AM +0200, Markus Armbruster wrote: Amos Kong ak...@redhat.com writes: [...] This interface is abstract in the sense

Re: [Qemu-devel] [PATCH 3/4] qemu-char: Register ring buffer driver with correct name ringbuf

2013-07-04 Thread Markus Armbruster
Luiz Capitulino lcapitul...@redhat.com writes: On Thu, 27 Jun 2013 16:22:09 +0200 Markus Armbruster arm...@redhat.com wrote: The driver is new in 1.4, with the documented name ringbuf. However, it's actual name is the completely undocumented memory. Screwed up in commit 3949e59. Fix code

Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/17] ppc64: Enable QEMU to run on POWER 8 DD1 chip.

2013-07-04 Thread Benjamin Herrenschmidt
On Thu, 2013-07-04 at 07:54 +0200, Andreas Färber wrote: Am 27.06.2013 08:45, schrieb Alexey Kardashevskiy: From: Prerna Saxena pre...@linux.vnet.ibm.com This patch enables QEMU to launch VM guests on POWER8 chip. I have tested this to work with BML kernel on P8 dd1 chip.

Re: [Qemu-devel] meaningless to compare irqfd's msi message with new msi message in virtio_pci_vq_vector_unmask

2013-07-04 Thread Zhanghaoyu (A)
I searched vector_irqfd globally, no place found to set/change irqfd's msi message, only irqfd's virq or users member may be changed in kvm_virtio_pci_vq_vector_use, kvm_virtio_pci_vq_vector_release, etc. So I think it's meaningless to do below check in virtio_pci_vq_vector_unmask, if

Re: [Qemu-devel] [RFC V8 01/24] qcow2: Add journal specification.

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 02:53:27PM +0200, Benoît Canet wrote: By the way, I don't know much about journalling techniques. So I'm asking you these questions so that either you can answer them straight away or because they might warrant a look at existing journal implementations like: I

Re: [Qemu-devel] [PATCH] Xen PV Device

2013-07-04 Thread Andreas Färber
Am 03.07.2013 18:37, schrieb Stefano Stabellini: On Wed, 3 Jul 2013, Paul Durrant wrote: This patch introduces a new Xen PV PCI device which will act as a new binding point for PV drivers for Xen. The device has parameterized vendor-id, device-id and revision to allow to be configured as a

[Qemu-devel] [Bug 1197663] [NEW] qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB irrespective of the volume size

2013-07-04 Thread chandrashekar shastri
Public bug reported: qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB irrespective of the volume size. Kernel Version: 3.10.0-rc5+ Libvirt Version: 1.0.6 Qemu Version: 1.5.50 Steps to reproduce the issue: 1. Create a qcow2 voulme using the command qemu-img create -f qcow2

[Qemu-devel] [Bug 1192499] Re: virsh migration copy-storage-all fails with Unable to read from monitor: Connection reset by peer

2013-07-04 Thread chandrashekar shastri
Moving to qemu component as qemu is crashing based on the inputs from Michal Privoznik Bugzilla : Bug 979411 - virsh migration copy-storage-all fails with Unable to read from monitor: Connection reset by peer ** Project changed: libvirt = qemu ** Bug watch added: Red Hat Bugzilla #979411

Re: [Qemu-devel] [PATCH] qom: Use atomics for object refcounting

2013-07-04 Thread liu ping fan
On Thu, Jul 4, 2013 at 1:43 PM, Andreas Färber afaer...@suse.de wrote: Am 04.07.2013 06:46, schrieb liu ping fan: On Thu, Jul 4, 2013 at 12:36 AM, Andreas Färber afaer...@suse.de wrote: Am 03.07.2013 03:23, schrieb liu ping fan: [...] It would be nice to get CC'ed on such proposals. :) I

[Qemu-devel] [Bug 1197663] Re: qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB irrespective of the volume size

2013-07-04 Thread chandrashekar shastri
** Also affects: fedora Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1197663 Title: qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-04 Thread Kevin Wolf
Am 03.07.2013 um 17:59 hat Anthony Liguori geschrieben: Kevin Wolf kw...@redhat.com writes: Am 02.07.2013 um 19:06 hat Anthony Liguori geschrieben: Eric Blake ebl...@redhat.com writes: On 07/02/2013 08:51 AM, Anthony Liguori wrote: Amos Kong ak...@redhat.com writes: Introduces

Re: [Qemu-devel] [PATCH V1 1/2] Implement sync modes for drive-backup.

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 20:14, Ian Main ha scritto: Should the source be bs for MIRROR_SYNC_MODE_NONE? Also in this case you may want to default the format to qcow2 instead of bs's format. I'm not sure that it matters what the source is for NONE. Since we are copying all new writes, whether they

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 18:06, Anthony Liguori ha scritto: Paolo Bonzini pbonz...@redhat.com writes: Il 03/07/2013 14:54, Anthony Liguori ha scritto: So, qapi-schema.json has to be readable/writable _mostly_ by humans. That it is valid JSON is little more than a curious accident, because I can

Re: [Qemu-devel] [PATCH v2 3/4] ide: Set BSY bit during FLUSH

2013-07-04 Thread Kevin Wolf
Am 03.07.2013 um 22:02 hat Alex Williamson geschrieben: On Wed, 2013-06-05 at 15:17 +0200, Kevin Wolf wrote: From: Andreas Färber afaer...@suse.de The implementation of the ATA FLUSH command invokes a flush at the block layer, which may on raw files on POSIX entail a synchronous

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 17:59, Anthony Liguori ha scritto: For instance: { command: foo, arguments: { name: str, id: int }, optional: { bar: bool }, defaults: { bar: false } } This is still not a dictionary that QAPI is able to describe. Paolo

Re: [Qemu-devel] [PATCH 12/23] ide: Convert FLUSH CACHE to ide_cmd_table handler

2013-07-04 Thread Kevin Wolf
Am 03.07.2013 um 23:51 hat Alex Williamson geschrieben: On Wed, 2013-07-03 at 15:41 -0600, Alex Williamson wrote: On Mon, 2013-06-24 at 11:10 +0200, Stefan Hajnoczi wrote: From: Kevin Wolf kw...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com Signed-off-by: Stefan Hajnoczi

Re: [Qemu-devel] [PATCH 01/17] cow: make reads go at a decent speed

2013-07-04 Thread Paolo Bonzini
Il 04/07/2013 04:20, Fam Zheng ha scritto: On Wed, 07/03 16:34, Paolo Bonzini wrote: Do not do two reads for each sector; load each sector of the bitmap and use bitmap operations to process it. Writes are still dog slow! Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- block/cow.c |

[Qemu-devel] [PATCH] Makefile: disable parallel build with dtc

2013-07-04 Thread Michael S. Tsirkin
Sometimes I get this error when building with -j 4: ar: two different operation options specified make[1]: *** [libfdt/libfdt.a] Error 1 make: *** [subdir-dtc] Error 2 dtc make does not seem to support parallel make. Force non-parallel build to fix this. Signed-off-by: Michael S. Tsirkin

Re: [Qemu-devel] [PATCH] qom: Use atomics for object refcounting

2013-07-04 Thread Paolo Bonzini
Il 02/07/2013 18:36, Anthony Liguori ha scritto: Paolo Bonzini pbonz...@redhat.com writes: Il 02/07/2013 16:47, Anthony Liguori ha scritto: Jan Kiszka jan.kis...@siemens.com writes: Objects can soon be referenced/dereference outside the BQL. So we need to use atomics in object_ref/unref.

Re: [Qemu-devel] [PATCH 10/17] block: define get_block_status return value

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 23:04, Peter Lieven ha scritto: Define the return value of get_block_status. Bits 0, 1, 2 and 8-62 are valid; bit 63 (the sign bit) is reserved for errors. Bits 3-7 are left for future extensions. Is Bit 8 not also reserved for future use? BDRV_SECTOR_BITS is 9. Right.

Re: [Qemu-devel] [PATCH 02/17] cow: make writes go at a less indecent speed

2013-07-04 Thread Paolo Bonzini
Il 04/07/2013 04:40, Fam Zheng ha scritto: On Wed, 07/03 16:34, Paolo Bonzini wrote: Only sync once per write, rather than once per sector. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- block/cow.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff

Re: [Qemu-devel] [PATCH 12/17] qemu-img: add a map subcommand

2013-07-04 Thread Paolo Bonzini
Il 04/07/2013 07:34, Fam Zheng ha scritto: +if ((e-flags (BDRV_BLOCK_DATA|BDRV_BLOCK_ZERO)) == BDRV_BLOCK_DATA) { +printf(%lld %lld %d %lld\n, + (long long) e-start, (long long) e-length, + e-depth, (long long) e-offset); +

Re: [Qemu-devel] [PATCH] Xen PV Device

2013-07-04 Thread Paul Durrant
-Original Message- From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] Sent: 03 July 2013 17:38 To: Paul Durrant Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org; Stefano Stabellini; afaer...@suse.de Subject: Re: [PATCH] Xen PV Device On Wed, 3 Jul 2013, Paul

Re: [Qemu-devel] [PATCH 11/17] block: return get_block_status data and flags for formats

2013-07-04 Thread Paolo Bonzini
Il 04/07/2013 05:22, Fam Zheng ha scritto: +case VMDK_OK: + /* TODO: might return offset if the extents are in bs-file. */ + ret = BDRV_BLOCK_DATA; if (extent-file == bs-file) { ret |= BDRV_BLOCK_OFFSET_VALID | offset; } Thanks. :) Paolo

Re: [Qemu-devel] [PATCH] Xen PV Device

2013-07-04 Thread Paul Durrant
-Original Message- Like Anthony wrote before, All rights reserved contradicts what's written below. Like I said, it's part of all BSD licenses that I can find. It's certainly in the template on the OSI website and the FreeBSD license for instance. Aside from this, it looks OK

Re: [Qemu-devel] [PATCH v4 0/9] Make 'dump-guest-memory' dump in kdump-compressed format

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 03:39:51PM +0800, Qiao Nuohan wrote: On 07/01/2013 07:45 PM, Stefan Hajnoczi wrote: In flatten format, data will be write to dumpfile block by block, and uses the following structure to indicate the offset and size of a data block. struct makedumpfile_data_header {

Re: [Qemu-devel] [PATCH 3/3] PPC PReP: can run without bios image

2013-07-04 Thread Julio Guerra
No conclusion was finally done about the new option proposal to load roms files. It really would be handy. -- Julio Guerra

Re: [Qemu-devel] [PATCH 12/17] qemu-img: add a map subcommand

2013-07-04 Thread Fam Zheng
On Thu, 07/04 10:16, Paolo Bonzini wrote: Il 04/07/2013 07:34, Fam Zheng ha scritto: +if ((e-flags (BDRV_BLOCK_DATA|BDRV_BLOCK_ZERO)) == BDRV_BLOCK_DATA) { +printf(%lld %lld %d %lld\n, + (long long) e-start, (long long) e-length, +

Re: [Qemu-devel] [PATCH] Citrix PV Bus device

2013-07-04 Thread Michael S. Tsirkin
On Tue, Jul 02, 2013 at 12:10:17PM +0100, Peter Maydell wrote: On 2 July 2013 11:57, Paul Durrant paul.durr...@citrix.com wrote: -Original Message- From: Paolo Bonzini [mailto:pbonz...@redhat.com] So the reason to place the device here is TARGET_PAGE_SIZE... We really need a way

Re: [Qemu-devel] [PATCH v5] Add timestamp to error_report()

2013-07-04 Thread Stefan Hajnoczi
On Thu, Jul 04, 2013 at 02:57:13AM +, Seiji Aguchi wrote: -Original Message- From: Stefan Hajnoczi [mailto:stefa...@gmail.com] Sent: Wednesday, July 03, 2013 5:14 AM To: Seiji Aguchi Cc: qemu-devel@nongnu.org; aligu...@us.ibm.com; berra...@redhat.com; kw...@redhat.com;

Re: [Qemu-devel] PVFS2 Block Driver Support

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 11:41:08AM -0400, Timothy Scott wrote: In testing my block driver implementation, I am receiving the following error when trying to run an orangefs protocol with a qcow2 image format: +Header extension too large +qemu-io: can't open device pvfs2:/... +no file

Re: [Qemu-devel] [PATCH] hw/9pfs: Fix potential memory leak and avoid reuse of freed memory

2013-07-04 Thread M. Mohan Kumar
Stefan Weil s...@weilnetz.de writes: The leak was reported by cppcheck. Function proxy_init also calls g_free for ctx-fs_root. Avoid reuse of this memory by setting ctx-fs_root to NULL. Signed-off-by: Stefan Weil s...@weilnetz.de Reviewed-by: M. Mohan Kumar mo...@in.ibm.com --- Hi, I'm

Re: [Qemu-devel] [Bug 1187529] [PATCH] Update mappings after PCI bridge live migration or save-restore.

2013-07-04 Thread Michael S. Tsirkin
On Wed, Jul 03, 2013 at 11:04:16AM -0400, Don Koch wrote: From: Don Koch dk...@verizon.com Update mappings for PCI bridge after live migration. Signed-off-by: Don Koch dk...@verizon.com --- This fixes bug 1187529: devices on a PCI bridge stop working after migration. Thanks, this looks

Re: [Qemu-devel] [libvirt] best way to provide disk storage for vm without shared storage system

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 05:31:44PM +0400, Vasiliy Tolstov wrote: Now i provide ext4 fs for qcow2 images (raid1 with two sata disks). Now i don't need live migration (but may need it in feature). What is the best way to provide disks to vm in case of performance, ability to create backups (i

Re: [Qemu-devel] [PULL v2 00/21] pci,kvm,misc enhancements

2013-07-04 Thread Michael S. Tsirkin
On Fri, Jun 28, 2013 at 12:44:17PM -0500, Anthony Liguori wrote: Markus Armbruster arm...@redhat.com writes: Michael S. Tsirkin m...@redhat.com writes: pvpanic: fix fwcfg for big endian hosts Umm, 1+10+9 is 20, but the pull is for 21 patches; what's going on here? Funny :-)

Re: [Qemu-devel] [PATCH v6] add timestamp to error_report()

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 11:02:46PM -0400, Seiji Aguchi wrote: [Issue] When we offer a customer support service and a problem happens in a customer's system, we try to understand the problem by comparing what the customer reports with message logs of the customer's system. In this case, we

[Qemu-devel] [PATCH v3 01/18] range: add Range structure

2013-07-04 Thread Michael S. Tsirkin
Sometimes we need to pass ranges around, add a handy structure for this purpose. Note: memory.c defines its own concept of AddrRange structure for working with 128 addresses. It's necessary there for doing range math. This is not needed for most users: struct Range is much simpler, and is only

[Qemu-devel] [PATCH v3 02/18] pci: store PCI hole ranges in guestinfo structure

2013-07-04 Thread Michael S. Tsirkin
Will be used to pass hole ranges to guests. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/i386/pc.c | 46 +- hw/i386/pc_piix.c | 14 +- hw/i386/pc_q35.c | 6 +- hw/pci-host/q35.c | 8

[Qemu-devel] [PATCH v3 04/18] pc_piix: cleanup init compat handling

2013-07-04 Thread Michael S. Tsirkin
Make sure 1.4 calls 1.5, 1.3 calls 1.4 etc. This way it's enough to add enough new compat hook in a single place in piix. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/i386/pc_piix.c | 18 -- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git

[Qemu-devel] [PULL v3 00/18] pci,misc enhancements

2013-07-04 Thread Michael S. Tsirkin
Changes from v2: - rebased to origin/master - fixed up botched posting The following changes since commit ab8bf29078e0ab8347e2ff8b4e5542f7a0c751cf: Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2013-07-03 08:37:00 -0500) are available in the git repository at:

[Qemu-devel] [PATCH v3 07/18] pvpanic: initialization cleanup

2013-07-04 Thread Michael S. Tsirkin
Avoid use of static variables: PC systems initialize pvpanic device through pvpanic_init, so we can simply create the fw_cfg file at that point. This also makes it possible to skip device creation completely if fw_cfg is not there, e.g. for xen - so the ports it reserves are not discoverable by

[Qemu-devel] [PATCH v3 03/18] pc: pass PCI hole ranges to Guests

2013-07-04 Thread Michael S. Tsirkin
Guest currently has to jump through lots of hoops to guess the PCI hole ranges. It's fragile, and makes us change BIOS each time we add a new chipset. Let's report the window in a ROM file, to make BIOS do exactly what QEMU intends. Signed-off-by: Michael S. Tsirkin m...@redhat.com ---

[Qemu-devel] [PATCH v3 06/18] MAINTAINERS: s/Marcelo/Paolo/

2013-07-04 Thread Michael S. Tsirkin
Marcelo doesn't maintain kvm anymore, Paolo is taking over the job. Update MAINTAINERS to stop flooding Marcelo with mail. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index

[Qemu-devel] [PATCH v3 08/18] pvpanic: fix fwcfg for big endian hosts

2013-07-04 Thread Michael S. Tsirkin
Convert port number to little endian when exposing it in fw cfg. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/misc/pvpanic.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/misc/pvpanic.c b/hw/misc/pvpanic.c index 83ed226..792d8e4 100644 ---

[Qemu-devel] [PATCH v3 12/18] pci: Use helper to find device's root bus in pci_find_domain()

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au Currently pci_find_domain() performs two functions - it locates the PCI root bus above the given bus, then looks up that root bus's domain number. This patch adds a helper function to perform the first task, finding the root bus for a given PCI

[Qemu-devel] [PATCH v3 10/18] pci: Move pci_read_devaddr to pci-hotplug-old.c

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au pci_read_devaddr() is only used by the legacy functions for the old PCI hotplug interface in pci-hotplug-old.c. So we move the function there, and make it static. Signed-off-by: David Gibson da...@gibson.dropbear.id.au Signed-off-by: Michael S.

[Qemu-devel] [PATCH v3 05/18] e1000: cleanup process_tx_desc

2013-07-04 Thread Michael S. Tsirkin
From: Andrew Jones drjo...@redhat.com Coverity complains about two overruns in process_tx_desc(). The complaints are false positives, but we might as well eliminate them. The problem is that hdr is defined as an unsigned int, but then used to offset an array of size 65536, and another of size 256

[Qemu-devel] [PATCH v3 09/18] pci: Cleanup configuration for pci-hotplug.c

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au pci-hotplug.c and the CONFIG_PCI_HOTPLUG variable which controls its compilation are misnamed. They're not about PCI hotplug in general, but rather about the pci_add/pci_del interface which are now deprecated in favour of the more general

[Qemu-devel] [PATCH v3 13/18] pci: Replace pci_find_domain() with more general pci_root_bus_path()

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au pci_find_domain() is used in a number of places where we want an id for a whole PCI domain (i.e. the subtree under a PCI root bus). The trouble is that many platforms may support multiple independent host bridges with no hardware supplied notion of

[Qemu-devel] [PATCH v3 15/18] pci: Add root bus parameter to pci_nic_init()

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au At present, pci_nic_init() and pci_nic_init_nofail() assume that they will only create a NIC under the primary PCI root. As we add support for multiple PCI roots, that may no longer be the case. This patch adds a root bus parameter to

[Qemu-devel] [PATCH v3 11/18] pci: Abolish pci_find_root_bus()

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au pci_find_root_bus() takes a domain parameter. Currently PCI root buses with domain other than 0 can't be created, so this is more or less a long winded way of retrieving the main PCI root bus. Numbered domains don't actually properly cover the (non

[Qemu-devel] [PATCH v3 16/18] pci: Simpler implementation of primary PCI bus

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au Currently pci_find_primary_bus() searches the list of root buses for one with domain 0. But since host buses are always registered with domain 0, this just amounts to finding the only PCI host bus. The only remaining users of pci_find_primary_bus()

Re: [Qemu-devel] [PATCH] Makefile: disable parallel build with dtc

2013-07-04 Thread Peter Maydell
On 4 July 2013 09:06, Michael S. Tsirkin m...@redhat.com wrote: Sometimes I get this error when building with -j 4: ar: two different operation options specified make[1]: *** [libfdt/libfdt.a] Error 1 make: *** [subdir-dtc] Error 2 dtc make does not seem to support parallel make. Force

[Qemu-devel] [PATCH v3 14/18] pci: Add root bus argument to pci_get_bus_devfn()

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au pci_get_bus_devfn() interprets a full PCI address string to give a PCIBus * and device/function number within that bus. Currently it assumes it is working on an address under the primary PCI root bus. This patch extends it to allow the caller to

[Qemu-devel] [PATCH v3 18/18] pci: Fold host_buses list into PCIHostState functionality

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au The host_buses list is an odd structure - a list of pointers to PCI root buses existing in parallel to the normal qdev tree structure. This patch removes it, instead putting the link pointers into the PCIHostState structure, which have a 1:1

[Qemu-devel] [PATCH 1/1] hw/9pfs: Fix memory leak in error path

2013-07-04 Thread M. Mohan Kumar
From: M. Mohan Kumar mo...@in.ibm.com Fix few more memory leaks in virtio-9p-device.c detected using valgrind. Signed-off-by: M. Mohan Kumar mo...@in.ibm.com --- hw/9pfs/virtio-9p-device.c | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git

Re: [Qemu-devel] [PATCH v3 00/14] tcg: remainder and tcg-arm updates

2013-07-04 Thread Claudio Fontana
On 03.07.2013 23:29, Richard Henderson wrote: Changes v2-v3: * Add myself to tcg maintainers, as per afaerber's suggestion. * Fix rebase error wrt aarch64, as per claudio. * Include tcg-arm unwind patch set; no point in half measures. r~ The following changes since commit

[Qemu-devel] [PATCH v5] Xen PV Device

2013-07-04 Thread Paul Durrant
Introduces a new Xen PV PCI device which will act as a binding point for PV drivers for Xen. The device has parameterized vendor-id, device-id and revision to allow to be configured as a binding point for any vendor's PV drivers. Signed-off-by: Paul Durrant paul.durr...@citrix.com Cc: Stefano

[Qemu-devel] [PATCH v3 17/18] pci: Remove domain from PCIHostBus

2013-07-04 Thread Michael S. Tsirkin
From: David Gibson da...@gibson.dropbear.id.au There are now no users of the domain field of PCIHostBus, so remove it from the structure, and as a parameter from the pci_host_bus_register() function which sets it. Signed-off-by: David Gibson da...@gibson.dropbear.id.au Signed-off-by: Michael S.

Re: [Qemu-devel] [PATCH] Makefile: disable parallel build with dtc

2013-07-04 Thread Andreas Färber
Am 04.07.2013 11:17, schrieb Peter Maydell: On 4 July 2013 09:06, Michael S. Tsirkin m...@redhat.com wrote: Sometimes I get this error when building with -j 4: ar: two different operation options specified make[1]: *** [libfdt/libfdt.a] Error 1 make: *** [subdir-dtc] Error 2 dtc make does

Re: [Qemu-devel] [PATCH] Makefile: disable parallel build with dtc

2013-07-04 Thread Michael S. Tsirkin
On Thu, Jul 04, 2013 at 10:17:46AM +0100, Peter Maydell wrote: On 4 July 2013 09:06, Michael S. Tsirkin m...@redhat.com wrote: Sometimes I get this error when building with -j 4: ar: two different operation options specified make[1]: *** [libfdt/libfdt.a] Error 1 make: *** [subdir-dtc]

Re: [Qemu-devel] [PATCH] Makefile: disable parallel build with dtc

2013-07-04 Thread Peter Maydell
On 4 July 2013 10:45, Andreas Färber afaer...@suse.de wrote: Am 04.07.2013 11:17, schrieb Peter Maydell: Also, can you confirm that you haven't got an environment that sets ARFLAGS to something weird (including ) ? I did confirm that my environment does not have ARFLAGS set; I believe the

[Qemu-devel] [PATCH V4 06/10] NUMA: split out the common range parser

2013-07-04 Thread Wanlong Gao
Since cpus parser and hostnode parser have the common range parser part, split it out to the common range parser to avoid the duplicate code. Reviewed-by: Bandan Das b...@redhat.com Signed-off-by: Wanlong Gao gaowanl...@cn.fujitsu.com --- vl.c | 89

[Qemu-devel] [PATCH V4 01/10] NUMA: Support multiple CPU ranges on -numa option

2013-07-04 Thread Wanlong Gao
From: Bandan Das b...@redhat.com This allows us to use the cpus property multiple times to specify multiple cpu (ranges) to the -numa option : -numa node,cpus=1,cpus=2,cpus=4 or -numa node,cpus=1-3,cpus=5 Note that after this patch, the defalut suffix of -numa node,mem=N will no longer be M. So

[Qemu-devel] [PATCH V4 00/10] Add support for binding guest numa nodes to host numa nodes

2013-07-04 Thread Wanlong Gao
As you know, QEMU can't direct it's memory allocation now, this may cause guest cross node access performance regression. And, the worse thing is that if PCI-passthrough is used, direct-attached-device uses DMA transfer between device and qemu process. All pages of the guest will be pinned by

[Qemu-devel] [PATCH V4 10/10] NUMA: show host memory policy info in info numa command

2013-07-04 Thread Wanlong Gao
Show host memory policy of nodes in the info numa monitor command. After this patch, the monitor command info numa will show the information like following if the host numa support is enabled: (qemu) info numa 2 nodes node 0 cpus: 0 node 0 size: 1024 MB node 0 mempolicy:

[Qemu-devel] [PATCH V4 03/10] NUMA: Add Linux libnuma detection

2013-07-04 Thread Wanlong Gao
Add detection of libnuma (mostly contained in the numactl package) to the configure script. Can be enabled or disabled on the command line, default is use if available. Signed-off-by: Andre Przywara andre.przyw...@amd.com Signed-off-by: Wanlong Gao gaowanl...@cn.fujitsu.com --- configure | 32

[Qemu-devel] [PATCH V4 09/10] NUMA: add hmp command set-mpol

2013-07-04 Thread Wanlong Gao
Add hmp command set-mpol to set host memory policy for a guest NUMA node. Then we can also set node's memory policy using the monitor command like: (qemu) set-mpol 0 mem-policy=membind,mem-hostnode=0-1 Signed-off-by: Wanlong Gao gaowanl...@cn.fujitsu.com --- hmp-commands.hx | 16

[Qemu-devel] [PATCH V4 04/10] NUMA: parse guest numa nodes memory policy

2013-07-04 Thread Wanlong Gao
The memory policy setting format is like: mem-policy={membind|interleave|preferred},mem-hostnode=[+|!]{all|N-N} And we are adding this setting as a suboption of -numa, the memory policy then can be set like following: -numa node,nodeid=0,mem=1024,cpus=0,mem-policy=membind,mem-hostnode=0-1 -numa

[Qemu-devel] [PATCH V4 02/10] NUMA: Add numa_info structure to contain numa nodes info

2013-07-04 Thread Wanlong Gao
Add the numa_info structure to contain the numa nodes memory, VCPUs information and the future added numa nodes host memory policies. Signed-off-by: Andre Przywara andre.przyw...@amd.com Signed-off-by: Wanlong Gao gaowanl...@cn.fujitsu.com --- cpus.c | 2 +- hw/i386/pc.c

[Qemu-devel] [PATCH V4 07/10] NUMA: set guest numa nodes memory policy

2013-07-04 Thread Wanlong Gao
Set the guest numa nodes memory policies using the mbind(2) system call node by node. After this patch, we are able to set guest nodes memory policies through the QEMU options, this arms to solve the guest cross nodes memory access performance issue. And as you all know, if PCI-passthrough is

[Qemu-devel] [PATCH V4 05/10] NUMA: handle Error in cpus, mpol and hostnode parser

2013-07-04 Thread Wanlong Gao
As Paolo pointed out that, handle Error in mpol and hostnode parser will make it easier to be used for example in mem-hotplug in the future. And this will be used later in set-mpol QMP command. Also handle Error in cpus parser to be consistent with others. Signed-off-by: Wanlong Gao

[Qemu-devel] [PATCH V4 08/10] NUMA: add qmp command set-mpol to set memory policy for NUMA node

2013-07-04 Thread Wanlong Gao
The QMP command let it be able to set node's memory policy through the QMP protocol. The qmp-shell command is like: set-mpol nodeid=0 mem-policy=membind mem-hostnode=0-1 Signed-off-by: Wanlong Gao gaowanl...@cn.fujitsu.com --- cpus.c | 54

Re: [Qemu-devel] [RFC V8 01/24] qcow2: Add journal specification.

2013-07-04 Thread Benoît Canet
Simple is good. Even for deduplication alone, I think data integrity is critical - otherwise we risk stale dedup metadata pointing to clusters that are unallocated or do not contain the right data. So the journal will probably need to follow techniques for commits/checksums. I agree that

Re: [Qemu-devel] 回复: Re: 回复: Re: Which part of qemu responds to ACPI control method?

2013-07-04 Thread Laszlo Ersek
On 07/04/13 08:05, bobooscar wrote: Thank you laszlo. however, after I got DSDT.dsl, I found that there is no “_PTS” method, even if “_TTS” “_GTS”. Then I go through all the acpi tables, still found no PTS/TTS methods: acpidump acpidump.out acpixtract -a acpidump.out for file

Re: [Qemu-devel] [PATCH v3 01/14] tcg: Add myself to general TCG maintainership

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: Signed-off-by: Richard Henderson r...@twiddle.net I think this is definitely a good idea; thanks! Acked-by: Peter Maydell peter.mayd...@linaro.org -- PMM

Re: [Qemu-devel] [PATCH v3 02/14] tcg: Split rem requirement from div requirement

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: There are several hosts with only a div insn. Remainder is computed manually from the quotient and inputs. We can do this generically. Signed-off-by: Richard Henderson r...@twiddle.net --- a/tcg/tci/tcg-target.h +++

Re: [Qemu-devel] [PATCH v3 06/14] tcg: Allow non-constant control macros

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: This allows TCG_TARGET_HAS_* to be a variable rather than a constant, which allows easier support for differing ISA levels for the host. The effect of this is that TCG_OPF_NOT_PRESENT means if set, op is definitely not present; if

Re: [Qemu-devel] [PATCH v3 07/14] tcg: Simplify logic using TCG_OPF_NOT_PRESENT

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: Expand the definition of not present to include should not be present. This means we can simplify the logic surrounding the generic tcg opcodes for which the host backend ought not be providing definitions. Signed-off-by: Richard

Re: [Qemu-devel] [PATCH v3 03/14] tcg-arm: Don't implement rem

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: Signed-off-by: Richard Henderson r...@twiddle.net Reviewed-by: Peter Maydell peter.mayd...@linaro.org -- PMM

Re: [Qemu-devel] [PATCH v3 08/14] tcg-arm: Make use of conditional availability of opcodes for divide

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: We can now detect and use divide instructions at runtime, rather than having to restrict their availability to compile-time. Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/arm/tcg-target.c | 16 ++--

Re: [Qemu-devel] [PATCH v3 09/14] tcg-arm: Simplify logic in detecting the ARM ISA in use

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: -#if defined(__ARM_ARCH_7__) || \ -defined(__ARM_ARCH_7A__) || \ -defined(__ARM_ARCH_7EM__) || \ -defined(__ARM_ARCH_7M__) || \ -defined(__ARM_ARCH_7R__) -#define USE_ARMV7_INSTRUCTIONS +/* The __ARM_ARCH

Re: [Qemu-devel] [PATCH 6/9] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 14:33, Libaiqing ha scritto: Hi asias, I got the rootcause:guest was installed on raw img with lvm partition,which vhost does not support. You mean LVM in the host or the guest? I guess in the host, but I'd rather make sure because otherwise you've found a bug. Paolo

Re: [Qemu-devel] [PATCH v3 12/14] tcg: Move the CIE and FDE header definitions to common code

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: These will necessarily be the same layout for all hosts. This limits the amount of boilerplate required to implement jit debug for a host. Signed-off-by: Richard Henderson r...@twiddle.net Reviewed-by: Peter Maydell

Re: [Qemu-devel] [PATCH v3 14/14] tcg-arm: Implement tcg_register_jit

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: Allows unwinding past the code_gen_buffer. Signed-off-by: Richard Henderson r...@twiddle.net Reviewed-by: Peter Maydell peter.mayd...@linaro.org -- PMM

Re: [Qemu-devel] [PATCH v3 11/14] tcg: Fix high_pc fields in .debug_info

2013-07-04 Thread Peter Maydell
On 3 July 2013 22:29, Richard Henderson r...@twiddle.net wrote: I don't think the debugger actually looks at this for anything, using the correct .debug_frame contents, but might as well get it all correct. Signed-off-by: Richard Henderson r...@twiddle.net Reviewed-by: Peter Maydell

Re: [Qemu-devel] [PATCH] highbank: add initial Calxeda Midway A15 support

2013-07-04 Thread Peter Maydell
On 28 June 2013 12:59, Andre Przywara andre.przyw...@calxeda.com wrote: From: Rob Herring rob.herr...@calxeda.com While the Calxeda Midway part is actually a bit more than a Highbank with A15s, for QEMU's purposes this view is sufficient. So to allow both emulation with that chip as well as

[Qemu-devel] [Bug 1197663] Re: qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB irrespective of the volume size

2013-07-04 Thread chandrashekar shastri
I tried booting with the qemu and observed the same thing. qemu-system-x86_64 /home/images/rhel64-64.qcow2 -drive if=none,id=hd,file=/home/images/virtio-scsi11.img -device virtio-scsi- pci,id=scsi --enable-kvm -device scsi-hd,drive=hd -m 2000 After creating the filesystem tried running the

[Qemu-devel] [Bug 1197663] Re: qcow2 [virtio-scsi] devices when mapped to the guest shows as 0MB irrespective of the volume size

2013-07-04 Thread chandrashekar shastri
** Attachment added: Screen shot https://bugs.launchpad.net/fedora/+bug/1197663/+attachment/3724283/+files/Screenshot2.png -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1197663 Title: qcow2

Re: [Qemu-devel] [PATCH V3 4/9] qmp: add internal snapshot support in qmp_transaction

2013-07-04 Thread Stefan Hajnoczi
On Thu, Jun 27, 2013 at 10:41:43AM +0800, Wenchao Xia wrote: +/* check whether a snapshot with name exist, no need to check id, since + name will be checked later to make sure it does not mess up with id. */ +ret = bdrv_snapshot_find_by_id_and_name(bs, NULL, name, sn, errp); +

Re: [Qemu-devel] [PATCHv2 02/11] iscsi: read unmap info from block limits vpd page

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 23:23, Peter Lieven ha scritto: BDC is not used. I had an implementation that sent multiple descriptors out, but at least for my storage the maximum unmap counts not for each descriptors, but for all together. So in this case we do not need the field at all. I forgot to remove

Re: [Qemu-devel] [PATCH V3 5/9] qmp: add interface blockdev-snapshot-internal-sync

2013-07-04 Thread Stefan Hajnoczi
On Thu, Jun 27, 2013 at 10:41:44AM +0800, Wenchao Xia wrote: diff --git a/qapi-schema.json b/qapi-schema.json index 2547a7d..fba9b15 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -1738,6 +1738,28 @@ '*mode': 'NewImageMode'} } ## +#

Re: [Qemu-devel] [PATCH V3 6/9] qmp: add interface blockdev-snapshot-delete-internal-sync

2013-07-04 Thread Stefan Hajnoczi
On Thu, Jun 27, 2013 at 10:41:45AM +0800, Wenchao Xia wrote: diff --git a/blockdev.c b/blockdev.c index ce89f83..35fffd6 100644 --- a/blockdev.c +++ b/blockdev.c @@ -790,6 +790,67 @@ void qmp_blockdev_snapshot_internal_sync(const char *device, snapshot, errp); }

Re: [Qemu-devel] [PATCH V3 0/9] add internal snapshot support at block device level

2013-07-04 Thread Stefan Hajnoczi
On Wed, Jul 03, 2013 at 09:52:10AM +0800, Wenchao Xia wrote: Any comments for this version? I'm happy with the code and left comments on error messages and documentation.

Re: [Qemu-devel] [Xen-devel] [PATCH] libxl: Spice usbredirection support for upstream qemu

2013-07-04 Thread Stefano Stabellini
Please don't use HTML in emails On Thu, 4 Jul 2013, Fabio Fantoni wrote: Il 04/07/2013 12:32, Wei Liu ha scritto: On Thu, Jul 04, 2013 at 12:16:43PM +0200, Fabio Fantoni wrote: Il 04/07/2013 12:12, Wei Liu ha scritto: On Thu, Jul 04, 2013 at 12:05:59PM +0200, Fabio Fantoni wrote:

Re: [Qemu-devel] [PATCH V16 0/7] replace QEMUOptionParameter with QemuOpts parser

2013-07-04 Thread Stefan Hajnoczi
On Tue, Jun 18, 2013 at 05:31:52PM +0800, Dong Xu Wang wrote: These patches will replace QEMUOptionParameter with QemuOpts. Change logs please go to each patch's commit message. Dong Xu Wang (7): add def_value_str in QemuOptDesc struct and rewrite qemu_opts_print avoid duplication of

Re: [Qemu-devel] [PATCH v5 10/11] qemu-ga: Install Windows VSS provider on `qemu-ga -s install'

2013-07-04 Thread Paolo Bonzini
Il 03/07/2013 18:19, Tomoki Sekiyama ha scritto: On 7/3/13 11:58 , Paolo Bonzini pbonz...@redhat.com wrote: Il 03/07/2013 17:49, Tomoki Sekiyama ha scritto: -return ga_install_service(path, log_filepath, fixed_state_dir); +if (ga_install_vss_provider()) { +

Re: [Qemu-devel] [PATCH 1/2] Refine and export infinite loop checking in collect_image_info_list()

2013-07-04 Thread Stefan Hajnoczi
On Fri, Jun 28, 2013 at 02:37:52PM -0600, Eric Blake wrote: On 06/27/2013 01:38 AM, Xu Wang wrote: +filenames = g_hash_table_new_full(g_str_hash, str_equal_func, NULL, NULL); + +/* If backing file exists, filename will insert into hash table and seek + * the whole

[Qemu-devel] [RFC PATCH] elfload: load PIE executables to right address

2013-07-04 Thread Timo Teräs
PIE images are ET_DYN images. Check first for pinterp_name to make sure the main executable always is loaded to correct place. See below for current behaviour of PIE executables: Reserved 0x7f00 bytes of guest address space host mmap_min_addr=0x1000 guest_base 0x7f7cb41d5000 startend

  1   2   3   >