Re: [Qemu-devel] [PULL 00/18] ppc patch queue 2013-09-02

2013-09-03 Thread Aurelien Jarno
On Mon, Sep 02, 2013 at 10:11:15AM +0200, Alexander Graf wrote: Hi Blue / Aurelien / Anthony, This is my current patch queue for ppc. Please pull. Done, thanks. -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurel...@aurel32.net http://www.aurel32.net

Re: [Qemu-devel] [PULL 00/29] Three tcg patch sets

2013-09-03 Thread Aurelien Jarno
On Mon, Sep 02, 2013 at 09:28:45AM -0700, Richard Henderson wrote: Aurelien has now reviewed three tcg related patch sets. This is a pull request for all of them, fixing a trivial patch conflict in the process. Done, thanks. -- Aurelien Jarno GPG: 1024D/F1BCDB73

[Qemu-devel] [PATCH 2/4] tcg/optimize: fix known-zero bits for right shift ops

2013-09-03 Thread Aurelien Jarno
32-bit versions of sar and shr ops should not propagate known-zero bits from the unused 32 high bits. For sar it could even lead to wrong code being generated. Cc: Richard Henderson r...@twiddle.net Cc: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Aurelien Jarno aurel...@aurel32.net ---

[Qemu-devel] [PATCH 0/4] tcg/optimize: fixes and improvements

2013-09-03 Thread Aurelien Jarno
This patchset first fixes known-zero bits optimization so that it is actually used, and does some further optimizations for 32-bit ops and unsigned loads. Aurelien Jarno (4): tcg/optimize: fix know-zero bits optimization tcg/optimize: fix known-zero bits for right shift ops tcg/optimize:

[Qemu-devel] [PATCH 3/4] tcg/optimize: improve known-zero bits for 32-bit ops

2013-09-03 Thread Aurelien Jarno
The shl_i32 op might set some bits of the unused 32 high bits of the mask. Fix that by clearing the unused 32 high bits for all 32-bit ops except load/store which operate on tl values. Cc: Richard Henderson r...@twiddle.net Cc: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Aurelien Jarno

[Qemu-devel] [PATCH 4/4] tcg/optimize: add known-zero bits compute for load ops

2013-09-03 Thread Aurelien Jarno
Cc: Richard Henderson r...@twiddle.net Cc: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- tcg/optimize.c | 13 + 1 file changed, 13 insertions(+) diff --git a/tcg/optimize.c b/tcg/optimize.c index b1f736b..044f456 100644 ---

[Qemu-devel] [PATCH 1/4] tcg/optimize: fix know-zero bits optimization

2013-09-03 Thread Aurelien Jarno
Known-zero bits optimization is a great idea that helps to generate more optimized code. However the current implementation is basically useless as the computed mask is not saved. Fix this to make it really working. Cc: Richard Henderson r...@twiddle.net Cc: Paolo Bonzini pbonz...@redhat.com

[Qemu-devel] [PATCH] kvm: fix traces to use %x instead of %d

2013-09-03 Thread Alexey Kardashevskiy
KVM request types are normally defined using hex constants but QEMU traces print decimal values instead what is not very convinient. This changes the request type format from %d to %x. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- trace-events | 6 +++--- 1 file changed, 3

[Qemu-devel] [PATCH] exec: avoid tcg_commit when kvm_enabled

2013-09-03 Thread liguang
Signed-off-by: liguang lig.f...@cn.fujitsu.com --- exec.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/exec.c b/exec.c index 3ca9381..4509daa 100644 --- a/exec.c +++ b/exec.c @@ -1824,7 +1824,9 @@ static void memory_map_init(void)

[Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread liguang
Signed-off-by: liguang lig.f...@cn.fujitsu.com --- cputlb.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/cputlb.c b/cputlb.c index 977c0ca..08e50e0 100644 --- a/cputlb.c +++ b/cputlb.c @@ -169,21 +169,6 @@ static inline ram_addr_t

[Qemu-devel] [PATCH] gitignore: ignore files generated by GNU GLOBAL

2013-09-03 Thread liguang
Signed-off-by: liguang lig.f...@cn.fujitsu.com --- .gitignore |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index d2c5c2f..97d7a2f 100644 --- a/.gitignore +++ b/.gitignore @@ -108,4 +108,7 @@ pc-bios/s390-ccw/s390-ccw.img cscope.* tags TAGS

Re: [Qemu-devel] [PATCH v4 2/5] qcow2-cluster: Expand zero clusters

2013-09-03 Thread Max Reitz
Am 02.09.2013 17:13, schrieb Kevin Wolf: Am 02.09.2013 um 12:04 hat Max Reitz geschrieben: Add functionality for expanding zero clusters. This is necessary for downgrading the image version to one without zero cluster support. For non-backed images, this function may also just discard zero

Re: [Qemu-devel] [PATCH v4 5/5] qemu-iotest: qcow2 image option amendment

2013-09-03 Thread Max Reitz
Am 02.09.2013 17:36, schrieb Kevin Wolf: Am 02.09.2013 um 12:04 hat Max Reitz geschrieben: Add tests for qemu-img amend on qcow2 image files. Signed-off-by: Max Reitz mre...@redhat.com --- tests/qemu-iotests/061 | 178 +++ tests/qemu-iotests/061.out | 349

Re: [Qemu-devel] [PATCH 0/4] tcg/optimize: fixes and improvements

2013-09-03 Thread Paolo Bonzini
Il 03/09/2013 08:27, Aurelien Jarno ha scritto: This patchset first fixes known-zero bits optimization so that it is actually used, and does some further optimizations for 32-bit ops and unsigned loads. Aurelien Jarno (4): tcg/optimize: fix know-zero bits optimization tcg/optimize: fix

Re: [Qemu-devel] [PULL v2 22/26] qcow2-refcount: Move OFLAG_COPIED checks

2013-09-03 Thread Kevin Wolf
Am 02.09.2013 um 15:52 hat Stefan Hajnoczi geschrieben: On Mon, Sep 2, 2013 at 10:49 AM, Kevin Wolf kw...@redhat.com wrote: From: Max Reitz mre...@redhat.com Move the OFLAG_COPIED checks out of check_refcounts_l1 and check_refcounts_l2 and after the actual refcount checks/fixes (since the

Re: [Qemu-devel] [PATCH] kvm: fix traces to use %x instead of %d

2013-09-03 Thread Paolo Bonzini
Il 03/09/2013 08:43, Alexey Kardashevskiy ha scritto: KVM request types are normally defined using hex constants but QEMU traces print decimal values instead what is not very convinient. This changes the request type format from %d to %x. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru

Re: [Qemu-devel] [PATCH] gitignore: ignore files generated by GNU GLOBAL

2013-09-03 Thread Peter Maydell
On 3 September 2013 08:06, liguang lig.f...@cn.fujitsu.com wrote: Signed-off-by: liguang lig.f...@cn.fujitsu.com --- .gitignore |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index d2c5c2f..97d7a2f 100644 --- a/.gitignore +++

Re: [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread Paolo Bonzini
Il 03/09/2013 09:05, liguang ha scritto: Signed-off-by: liguang lig.f...@cn.fujitsu.com --- cputlb.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/cputlb.c b/cputlb.c index 977c0ca..08e50e0 100644 --- a/cputlb.c +++ b/cputlb.c @@ -169,21 +169,6

[Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable

2013-09-03 Thread mrezanin
From: Miroslav Rezanina mreza...@redhat.com Use usb_legacy_register handling to create bt-dongle device. This allows to disable usb-bt-dongle device using CONFIG_BLUETOOTH option. Signed-off-by: Miroslav Rezanina mreza...@redhat.com --- hw/bt/core.c | 25 hw/bt/hci.c

[Qemu-devel] [RFC PATCH] spapr: support time base offset migration

2013-09-03 Thread Alexey Kardashevskiy
This allows guests to have a different timebase origin from the host. This is needed for migration, where a guest can migrate from one host to another and the two hosts might have a different timebase origin. However, the timebase seen by the guest must not go backwards, and should go forwards

Re: [Qemu-devel] [PATCH v3 00/29] tcg-aarch64 improvements

2013-09-03 Thread Richard W.M. Jones
On Mon, Sep 02, 2013 at 10:54:34AM -0700, Richard Henderson wrote: I'm not sure if I posted v2 or not, but my branch is named -3, therefore this is v3. ;-) The jumbo fixme patch from v1 has been split up. This has been updated for the changes in the tlb helpers over the past few weeks.

Re: [Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable

2013-09-03 Thread Gerd Hoffmann
diff --git a/hw/bt/core.c b/hw/bt/core.c index 49012e0..ef27b15 100644 --- a/hw/bt/core.c +++ b/hw/bt/core.c @@ -119,3 +119,28 @@ void bt_device_done(struct bt_device_s *dev) *p = dev-next; } + +static struct bt_vlan_s { +struct bt_scatternet_s net; +int id; +

Re: [Qemu-devel] [PATCHv2] spice-core: Use g_strdup_printf instead of snprintf

2013-09-03 Thread Gerd Hoffmann
On Mo, 2013-09-02 at 15:41 +0200, Christophe Fergeau wrote: Several places in spice-core.c were using either g_malloc+snprintf or snprintf+g_strdup to achieve the same result as g_strdup_printf. Added to spice patch queue. thanks, Gerd

Re: [Qemu-devel] [PATCH v3 00/29] tcg-aarch64 improvements

2013-09-03 Thread Laurent Desnogues
On Tue, Sep 3, 2013 at 9:37 AM, Richard W.M. Jones rjo...@redhat.com wrote: On Mon, Sep 02, 2013 at 10:54:34AM -0700, Richard Henderson wrote: I'm not sure if I posted v2 or not, but my branch is named -3, therefore this is v3. ;-) The jumbo fixme patch from v1 has been split up. This has

Re: [Qemu-devel] [libvirt] [PATCH 3/5] qemu: add usb-bot support from disks points of view

2013-09-03 Thread Gerd Hoffmann
On Mo, 2013-09-02 at 13:57 +0100, Daniel P. Berrange wrote: On Mon, Sep 02, 2013 at 05:38:42PM +0800, Guannan Ren wrote: usb-bot only supports 16 luns(0~15) and they must be contiguous, (using lun 0 and 2 without 1 doesn't work). In this case qemu doesn't throw an error, we can not find the

Re: [Qemu-devel] [PATCH v3 0/8] block: drive-backup live backup command

2013-09-03 Thread Stefan Hajnoczi
On Mon, Sep 02, 2013 at 02:57:23PM +0200, Benoît Canet wrote: I don't see the point of using hashes. Using hashes means that at least one extra read will be done on the target to compute the candidate target hash. It's bad for a cloud provider where IOs count is a huge cost. Another

[Qemu-devel] [PULL 2/6] s390/dump: zero out padding bytes in notes sections

2013-09-03 Thread Christian Borntraeger
The prstatus of an s390x dump contains several padding areas. Zero out these bytes to make reading the notes section easier with a hexdump. Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com --- target-s390x/arch_dump.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Qemu-devel] [PULL 3/6] s390/ipl: Fix waiting for virtio processing

2013-09-03 Thread Christian Borntraeger
From: Cornelia Huck cornelia.h...@de.ibm.com The guest side must not manipulate the index for the used buffers. Instead, remember the state of the used buffer locally and wait until it has moved. Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com Acked-by: Alexander Graf ag...@suse.de

[Qemu-devel] [PULL 4/6] s390/ipl: Update the s390-ccw.img rom

2013-09-03 Thread Christian Borntraeger
Rebuild of the virtio-ccw rom containing these patches: 1. s390/ipl: Fix waiting for virtio processing Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com --- pc-bios/s390-ccw.img | Bin 9432 - 9336 bytes 1 file changed, 0 insertions(+), 0 deletions(-) diff --git a/pc-bios/s390-ccw.img

[Qemu-devel] [PULL 5/6] s390/cpu: Make setcc() function available to other files

2013-09-03 Thread Christian Borntraeger
From: Thomas Huth th...@linux.vnet.ibm.com Moved the setcc() function to cpu.h so that it can be used by other files, too. It now also does not modify the kvm state anymore since this gets updated during kvm_arch_put_registers() anyway. Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com

[Qemu-devel] [PULL 0/6] s390: cleanups and fixes

2013-09-03 Thread Christian Borntraeger
Alex, the next bunch of patches for s390. If there are no complaints, I will send out a pull request soon. ---snip--- The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07: Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500) are

[Qemu-devel] [PULL 1/6] s390/kvm: Add check for priviledged SCLP handler

2013-09-03 Thread Christian Borntraeger
From: Thomas Huth th...@linux.vnet.ibm.com The SCLP instruction is priviledged, so we should make sure that we generate an exception when it is called from the problem state. Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com Signed-off-by: Christian Borntraeger borntrae...@de.ibm.com ---

[Qemu-devel] [PULL 6/6] s390/ioinst: Moved the CC setting to the IO instruction handlers

2013-09-03 Thread Christian Borntraeger
From: Thomas Huth th...@linux.vnet.ibm.com The IO instruction handlers now take care of setting the CC value on their own, so that the confusing return code magic in kvm_handle_css_inst() is not needed anymore. Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com Reviewed-by: Cornelia Huck

Re: [Qemu-devel] [PATCH v3 00/29] tcg-aarch64 improvements

2013-09-03 Thread Peter Maydell
On 3 September 2013 08:37, Richard W.M. Jones rjo...@redhat.com wrote: Is there a way yet to compile and run a 'qemu-system-aarch64'? [on a regular x86-64 host] The code for this has not yet been written :-) The patchset I posted will build a qemu-system-aarch64 but with no actual 64 bit CPUs

[Qemu-devel] [PATCH v6] kvm irqfd: support direct msimessage to irq translation

2013-09-03 Thread Alexey Kardashevskiy
On PPC64 systems MSI Messages are translated to system IRQ in a PCI host bridge. This is already supported for emulated MSI/MSIX but not for irqfd where the current QEMU allocates IRQ numbers from irqchip and maps MSIMessages to IRQ in the host kernel. This adds a new direct mapping flag which

[Qemu-devel] [PATCH v5 1/6] block: Image file option amendment

2013-09-03 Thread Max Reitz
This patch adds the amend option to qemu-img which allows changing image options on existing image files. It also adds the generic bdrv implementation which is basically just a wrapper for the image format specific function. Signed-off-by: Max Reitz mre...@redhat.com --- block.c

[Qemu-devel] [PATCH v5 0/6] block/qcow2: Image file option amendment

2013-09-03 Thread Max Reitz
This series adds support to qemu-img, block and qcow2 for amending image options on existing image files. Depends on: - option: Add assigned flag to QEMUOptionParameter - qcow2-refcount: Snapshot update for zero clusters (series, v3) - Add metadata overlap checks (series, v5) v5: - added a

[Qemu-devel] [PATCH v5 6/6] qemu-iotest: qcow2 image option amendment

2013-09-03 Thread Max Reitz
Add tests for qemu-img amend on qcow2 image files. Signed-off-by: Max Reitz mre...@redhat.com --- tests/qemu-iotests/061 | 178 +++ tests/qemu-iotests/061.out | 349 + tests/qemu-iotests/group | 1 + 3 files changed, 528

[Qemu-devel] [PATCH v5 4/6] qcow2: Save refcount order in BDRVQcowState

2013-09-03 Thread Max Reitz
Save the image refcount order in BDRVQcowState. This will be relevant for future code supporting different refcount orders than four and also for code that needs to verify a certain refcount order for an opened image. Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2.c | 3 ++-

[Qemu-devel] [PATCH v5 5/6] qcow2: Implement bdrv_amend_options

2013-09-03 Thread Max Reitz
Implement bdrv_amend_options for compat, size, backing_file, backing_fmt and lazy_refcounts. Downgrading images from compat=1.1 to compat=0.10 is achieved through handling all incompatible flags accordingly, clearing all compatible and autoclear flags and expanding all zero clusters.

[Qemu-devel] [PATCH v5 2/6] qcow2-cache: Empty cache

2013-09-03 Thread Max Reitz
Add a function for emptying a cache, i.e., flushing it and marking all elements invalid. Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2-cache.c | 18 ++ block/qcow2.h | 2 ++ 2 files changed, 20 insertions(+) diff --git a/block/qcow2-cache.c

[Qemu-devel] [PATCH v5 3/6] qcow2-cluster: Expand zero clusters

2013-09-03 Thread Max Reitz
Add functionality for expanding zero clusters. This is necessary for downgrading the image version to one without zero cluster support. For non-backed images, this function may also just discard zero clusters instead of truly expanding them. Signed-off-by: Max Reitz mre...@redhat.com ---

Re: [Qemu-devel] [PATCH v6] kvm irqfd: support direct msimessage to irq translation

2013-09-03 Thread Michael S. Tsirkin
On Tue, Sep 03, 2013 at 06:08:25PM +1000, Alexey Kardashevskiy wrote: On PPC64 systems MSI Messages are translated to system IRQ in a PCI host bridge. This is already supported for emulated MSI/MSIX but not for irqfd where the current QEMU allocates IRQ numbers from irqchip and maps

Re: [Qemu-devel] [PATCH v4 5/5] qemu-iotest: qcow2 image option amendment

2013-09-03 Thread Kevin Wolf
Am 02.09.2013 um 12:04 hat Max Reitz geschrieben: Add tests for qemu-img amend on qcow2 image files. Signed-off-by: Max Reitz mre...@redhat.com --- tests/qemu-iotests/061 | 178 +++ tests/qemu-iotests/061.out | 349 +

[Qemu-devel] [PATCH] linux-headers: update to 3.11

2013-09-03 Thread Alexey Kardashevskiy
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- I need this update as VFIO on PPC64/pseries got in upstream kernel and this is required by VFIO-SPAPR bits in QEMU. Others may find this update useful too :) --- linux-headers/asm-arm64/kvm.h | 168

Re: [Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable

2013-09-03 Thread Paolo Bonzini
Il 03/09/2013 09:26, mreza...@redhat.com ha scritto: From: Miroslav Rezanina mreza...@redhat.com Use usb_legacy_register handling to create bt-dongle device. This allows to disable usb-bt-dongle device using CONFIG_BLUETOOTH option. Signed-off-by: Miroslav Rezanina mreza...@redhat.com ---

Re: [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread Andreas Färber
Am 03.09.2013 09:22, schrieb Paolo Bonzini: Il 03/09/2013 09:05, liguang ha scritto: Signed-off-by: liguang lig.f...@cn.fujitsu.com --- cputlb.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/cputlb.c b/cputlb.c index 977c0ca..08e50e0 100644 ---

Re: [Qemu-devel] [PATCH] exec: avoid tcg_commit when kvm_enabled

2013-09-03 Thread Andreas Färber
Am 03.09.2013 08:59, schrieb liguang: Signed-off-by: liguang lig.f...@cn.fujitsu.com --- exec.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/exec.c b/exec.c index 3ca9381..4509daa 100644 --- a/exec.c +++ b/exec.c @@ -1824,7 +1824,9 @@ static void

Re: [Qemu-devel] [PATCH v5 0/6] block/qcow2: Image file option amendment

2013-09-03 Thread Kevin Wolf
Am 03.09.2013 um 10:09 hat Max Reitz geschrieben: This series adds support to qemu-img, block and qcow2 for amending image options on existing image files. Depends on: - option: Add assigned flag to QEMUOptionParameter - qcow2-refcount: Snapshot update for zero clusters (series, v3) -

Re: [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread Paolo Bonzini
Il 03/09/2013 10:35, Andreas Färber ha scritto: Am 03.09.2013 09:22, schrieb Paolo Bonzini: Il 03/09/2013 09:05, liguang ha scritto: Signed-off-by: liguang lig.f...@cn.fujitsu.com --- cputlb.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/cputlb.c

Re: [Qemu-devel] [RFC PATCH] spapr: support time base offset migration

2013-09-03 Thread Andreas Färber
Am 03.09.2013 09:31, schrieb Alexey Kardashevskiy: This allows guests to have a different timebase origin from the host. This is needed for migration, where a guest can migrate from one host to another and the two hosts might have a different timebase origin. However, the timebase seen by

Re: [Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable

2013-09-03 Thread Miroslav Rezanina
- Original Message - From: Gerd Hoffmann kra...@redhat.com To: mreza...@redhat.com Cc: qemu-devel@nongnu.org Sent: Tuesday, September 3, 2013 9:41:11 AM Subject: Re: [Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable diff --git a/hw/bt/core.c b/hw/bt/core.c index

Re: [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread Peter Maydell
On 3 September 2013 09:35, Andreas Färber afaer...@suse.de wrote: I also don't understand why qemu-trivial is suddenly picking up Stefan's arm translation patch, it used to be for unmaintained areas only. But arm is not my problem. Yeah, I wasn't expecting that either. But I'd reviewed it and

Re: [Qemu-devel] [PATCH] w32: Fix access to host devices (regression)

2013-09-03 Thread Kevin Wolf
Am 01.09.2013 um 22:59 hat Stefan Weil geschrieben: QEMU failed to open host devices like \\.\PhysicalDrive0 (first hard disk) since some time (commit 8a79380b8ef1b02d2abd705dd026a18863b09020?). Those devices use hdev_open which did not use the latest API for options. This resulted in a

Re: [Qemu-devel] [PATCH v2] kvm: warn if num cpus is greater than num recommended

2013-09-03 Thread Gleb Natapov
On Fri, Aug 23, 2013 at 03:24:37PM +0200, Andrew Jones wrote: The comment in kvm_max_vcpus() states that it's using the recommended procedure from the kernel API documentation to get the max number of vcpus that kvm supports. It is, but by always returning the maximum number supported. The

Re: [Qemu-devel] [RFC PATCH] spapr: support time base offset migration

2013-09-03 Thread Alexey Kardashevskiy
On 09/03/2013 06:42 PM, Andreas Färber wrote: Am 03.09.2013 09:31, schrieb Alexey Kardashevskiy: This allows guests to have a different timebase origin from the host. This is needed for migration, where a guest can migrate from one host to another and the two hosts might have a different

Re: [Qemu-devel] [RFC PATCH] spapr: support time base offset migration

2013-09-03 Thread Andreas Färber
Am 03.09.2013 11:07, schrieb Alexey Kardashevskiy: On 09/03/2013 06:42 PM, Andreas Färber wrote: Am 03.09.2013 09:31, schrieb Alexey Kardashevskiy: diff --git a/target-ppc/machine.c b/target-ppc/machine.c index 12e1512..d1ffc7f 100644 --- a/target-ppc/machine.c +++ b/target-ppc/machine.c

[Qemu-devel] [PATCHv3 0/2] Make usb-bt-dongle configurable

2013-09-03 Thread mrezanin
From: Miroslav Rezanina mreza...@redhat.com There's group of options allowing enable/disable usb devices. However, CONFIG_USB_BLUETOOTH can't be removed as there's dependency in vl.c file. This serie allow CONFIG_USB_BLUETOOTH to be disabled. v3: - split patch to two parts v2: - patch

[Qemu-devel] [PATCHv3 1/2] Preparation for usb-bt-dongle conditional build

2013-09-03 Thread mrezanin
From: Miroslav Rezanina mreza...@redhat.com To allow disable usb-bt-dongle device using CONFIG_BLUETOOTH option, some of functions in vl.c file has to be made accessible in dev-bluetooth.c. This is pure code moving. Signed-off-by: Miroslav Rezanina mreza...@redhat.com --- hw/bt/core.c| 23

[Qemu-devel] [PATCHv3 2/2] Remove dev-bluetooth.c dependency from vl.c

2013-09-03 Thread mrezanin
From: Miroslav Rezanina mreza...@redhat.com Use usb_legacy_register handling to create bt-dongle device and remove code dependency from vl.c so CONFIG_USB_BLUETOOTH can be disabled. Signed-off-by: Miroslav Rezanina mreza...@redhat.com --- hw/usb/Makefile.objs | 3 --- hw/usb/dev-bluetooth.c

Re: [Qemu-devel] [PATCH] qemu-iotests: Adjust test result 039

2013-09-03 Thread Stefan Hajnoczi
On Mon, Sep 02, 2013 at 04:36:15PM +0200, Max Reitz wrote: The moved OFLAG_COPIED check in qcow2_check_refcounts results in a different output from test 039 (mismatches are now found after the general refcount check (as far as any remain)). This patch adjusts the expected test result

Re: [Qemu-devel] [PATCH] linux-headers: update to 3.11

2013-09-03 Thread Peter Maydell
On 3 September 2013 09:27, Alexey Kardashevskiy a...@ozlabs.ru wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- I need this update as VFIO on PPC64/pseries got in upstream kernel and this is required by VFIO-SPAPR bits in QEMU. Others may find this update useful too :) ---

Re: [Qemu-devel] [PATCH] linux-headers: update to 3.11

2013-09-03 Thread Alexey Kardashevskiy
On 09/03/2013 07:29 PM, Peter Maydell wrote: On 3 September 2013 09:27, Alexey Kardashevskiy a...@ozlabs.ru wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- I need this update as VFIO on PPC64/pseries got in upstream kernel and this is required by VFIO-SPAPR bits in QEMU. Others

Re: [Qemu-devel] [libvirt] [PATCH 3/5] qemu: add usb-bot support from disks points of view

2013-09-03 Thread Daniel P. Berrange
On Tue, Sep 03, 2013 at 09:51:52AM +0200, Gerd Hoffmann wrote: On Mo, 2013-09-02 at 13:57 +0100, Daniel P. Berrange wrote: On Mon, Sep 02, 2013 at 05:38:42PM +0800, Guannan Ren wrote: usb-bot only supports 16 luns(0~15) and they must be contiguous, (using lun 0 and 2 without 1 doesn't

Re: [Qemu-devel] [PATCHv7] add qemu-img convert -n option (skip target volume creation)

2013-09-03 Thread Stefan Hajnoczi
On Mon, Sep 02, 2013 at 07:07:24PM +0100, Alex Bligh wrote: From: Alexandre Derumier aderum...@odiso.com Add a -n option to skip volume creation on qemu-img convert. This is useful for targets such as rbd / ceph, where the target volume may already exist; we cannot always rely on qemu-img

Re: [Qemu-devel] [PATCH] kvm: fix traces to use %x instead of %d

2013-09-03 Thread Andreas Färber
Am 03.09.2013 08:43, schrieb Alexey Kardashevskiy: KVM request types are normally defined using hex constants but QEMU traces print decimal values instead what is not very convinient. which is, convenient -- pointing it out early since the KVM guys don't fix patches once in their queue. Patch

Re: [Qemu-devel] [PATCH] w32: Fix access to host devices (regression)

2013-09-03 Thread Stefan Hajnoczi
On Sun, Sep 01, 2013 at 10:59:25PM +0200, Stefan Weil wrote: QEMU failed to open host devices like \\.\PhysicalDrive0 (first hard disk) since some time (commit 8a79380b8ef1b02d2abd705dd026a18863b09020?). Those devices use hdev_open which did not use the latest API for options. This resulted

Re: [Qemu-devel] [PATCH 1/4] tcg/optimize: fix know-zero bits optimization

2013-09-03 Thread Andreas Färber
FWIW $subject has a typo. While at it... Am 03.09.2013 08:27, schrieb Aurelien Jarno: Known-zero bits optimization is a great idea that helps to generate more optimized code. However the current implementation is basically useless as the computed mask is not saved. Fix this to make it

Re: [Qemu-devel] [PATCH] w32: Fix access to host devices (regression)

2013-09-03 Thread Andreas Färber
Am 03.09.2013 10:51, schrieb Kevin Wolf: Am 01.09.2013 um 22:59 hat Stefan Weil geschrieben: QEMU failed to open host devices like \\.\PhysicalDrive0 (first hard disk) since some time (commit 8a79380b8ef1b02d2abd705dd026a18863b09020?). Those devices use hdev_open which did not use the latest

Re: [Qemu-devel] [PATCH] aio / timers: fix build of test/test-aio.c on non-linux platforms

2013-09-03 Thread Stefan Hajnoczi
On Thu, Aug 29, 2013 at 05:48:16PM +0100, Alex Bligh wrote: tests/test-aio.c used pipe2 which is Linux only. Use qemu_pipe and qemu_set_nonblock for portabillity. Addition of O_CLOEXEC is a harmless bonus. Signed-off-by: Alex Bligh a...@alex.org.uk --- tests/test-aio.c | 11 +--

Re: [Qemu-devel] [PATCHv2] aio / timers: use g_usleep() not sleep()

2013-09-03 Thread Stefan Hajnoczi
On Thu, Aug 29, 2013 at 11:32:14PM +0100, Alex Bligh wrote: sleep() apparently doesn't exist under mingw. Use g_usleep for portability. Signed-off-by: Alex Bligh a...@alex.org.uk --- tests/test-aio.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Thanks, applied to my

Re: [Qemu-devel] [PATCH v3 0/2] Fix unassigned memory and I/O access handling

2013-09-03 Thread Andreas Färber
Am 02.09.2013 18:43, schrieb Jan Kiszka: Repost of what missed the 1.6 release. Jan Kiszka (2): memory: Provide separate handling of unassigned io ports accesses Revert memory: Return -1 again on reads from unsigned regions These were: Tested-by: Andreas Färber andreas.faer...@web.de

Re: [Qemu-devel] [PATCH v5 00/21] AArch64 preparation patchset

2013-09-03 Thread Andreas Färber
Am 01.07.2013 19:34, schrieb Peter Maydell: I've left the 'global cpu_env variable' patch alone since there wasn't any consensus about what the best approach was; it's a pretty minor (and textually localized) thing anyway so easy to fix in future if it becomes an actual problem. Ack. Andreas

Re: [Qemu-devel] [PATCH] linux-headers: update to 3.11

2013-09-03 Thread Jan Kiszka
On 2013-09-03 11:32, Alexey Kardashevskiy wrote: On 09/03/2013 07:29 PM, Peter Maydell wrote: On 3 September 2013 09:27, Alexey Kardashevskiy a...@ozlabs.ru wrote: Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru --- I need this update as VFIO on PPC64/pseries got in upstream kernel and

Re: [Qemu-devel] [PATCH] exec: check offset_within_address_space for register subpage

2013-09-03 Thread Paolo Bonzini
Il 29/08/2013 12:21, Hu Tao ha scritto: If offset_within_address_space falls in a page, then we register a subpage. So check offset_within_address_space rather than offset_within_region. Cc: Paolo Bonzini pbonz...@redhat.com Cc: Richard Henderson r...@twiddle.net Cc: Andreas Färber

Re: [Qemu-devel] [PATCH 1/2] qem-xen: add later wakeup logic when qemu wakeup

2013-09-03 Thread Anthony PERARD
On 01/09/13 10:51, Liu, Jinsong wrote: From 86ad3bb83a984ad7bbc00b81d6a0bfc1abc543ca Mon Sep 17 00:00:00 2001 From: Liu Jinsong jinsong@intel.com Date: Sun, 1 Sep 2013 23:39:14 +0800 Subject: [PATCH 1/2] qemu-xen: add later wakeup logic when qemu wakeup Currently HVM S3 has a bug coming

Re: [Qemu-devel] [PATCH 2/2] qemu-xen: add qemu xen logic for HVM S3 resume

2013-09-03 Thread Anthony PERARD
On 01/09/13 10:54, Liu, Jinsong wrote: From e7d4bd70eae8da131dc3ff2cec70cb2c7b6575a9 Mon Sep 17 00:00:00 2001 From: Liu Jinsong jinsong@intel.com Date: Mon, 2 Sep 2013 00:39:20 +0800 Subject: [PATCH 2/2] qemu-xen: add qemu xen logic for HVM S3 resume This patch is qemu-xen patch 2 to

Re: [Qemu-devel] [PATCH 1/2] qem-xen: add later wakeup logic when qemu wakeup

2013-09-03 Thread Liu, Jinsong
Anthony PERARD wrote: On 01/09/13 10:51, Liu, Jinsong wrote: From 86ad3bb83a984ad7bbc00b81d6a0bfc1abc543ca Mon Sep 17 00:00:00 2001 From: Liu Jinsong jinsong@intel.com Date: Sun, 1 Sep 2013 23:39:14 +0800 Subject: [PATCH 1/2] qemu-xen: add later wakeup logic when qemu wakeup

Re: [Qemu-devel] [qemu-devel]question on virtqueue_get_avail_bytes

2013-09-03 Thread Amit Shah
On (Mon) 19 Aug 2013 [16:30:54], Stefan Hajnoczi wrote: On Mon, Aug 19, 2013 at 05:28:44PM +0800, yinyin wrote: Hi,all: in func virtqueue_get_avail_bytes, when found a indirect desc, we need loop over it. /* loop over the indirect descriptor table */

Re: [Qemu-devel] [qemu-devel]question on virtqueue_get_avail_bytes

2013-09-03 Thread Michael S. Tsirkin
On Tue, Sep 03, 2013 at 04:40:21PM +0530, Amit Shah wrote: On (Mon) 19 Aug 2013 [16:30:54], Stefan Hajnoczi wrote: On Mon, Aug 19, 2013 at 05:28:44PM +0800, yinyin wrote: Hi,all: in func virtqueue_get_avail_bytes, when found a indirect desc, we need loop over it. /*

Re: [Qemu-devel] [PATCH v4 0/3] bugs fix for hpet

2013-09-03 Thread Paolo Bonzini
Il 02/09/2013 09:06, Liu Ping Fan ha scritto: note: I rebase it onto Stefan's net-next tree, since pc-1.7 has already been defined there. v4: use standard compat property to set hpet's interrupt compatibility v3: change hpet interrupt capablity on board's demand Liu Ping Fan

Re: [Qemu-devel] [Qemu-trivial] [PATCH] cputlb: remove dead function tlb_update_dirty

2013-09-03 Thread Michael Tokarev
03.09.2013 12:35, Andreas Färber wrote: I also don't understand why qemu-trivial is suddenly picking up Stefan's arm translation patch, it used to be for unmaintained areas only. But arm is not my problem. Which patch you're talking about? Is it target-arm: Report unimplemented opcodes

Re: [Qemu-devel] [PATCHv2] Make usb-bt-dongle configurable

2013-09-03 Thread Gerd Hoffmann
This (and some other bits) are pure code motion from vl.c, correct? Can you split this into a separate patch please? That'll simplify the review o the actual code changes. Yes, this is pure code motion. I'll split the code to separate patches. It also doesn't make much sense to

Re: [Qemu-devel] [libvirt] [PATCH 3/5] qemu: add usb-bot support from disks points of view

2013-09-03 Thread Gerd Hoffmann
Hi, How does this work if we start off a guest with 3 disks attached to the usb-bot SCSI controller. Then hot-unplug the 2nd disk. You can't hotplug individual luns anyway. How does hotplug/unplug work in the context of usb-bot ? AFAIK we need to be able to run

[Qemu-devel] [KVM] segmentation fault happened when reboot VM after hot-uplug virtio NIC

2013-09-03 Thread Zhanghaoyu (A)
Hi, all Segmentation fault happened when reboot VM after hot-unplug virtio NIC, which can be reproduced 100%. See similar bug report to https://bugzilla.redhat.com/show_bug.cgi?id=988256 test environment: host: SLES11SP2 (kenrel version: 3.0.58) qemu: 1.5.1, upstream-qemu (commit

Re: [Qemu-devel] [KVM] segmentation fault happened when reboot VM after hot-uplug virtio NIC

2013-09-03 Thread Gleb Natapov
On Tue, Sep 03, 2013 at 12:06:33PM +, Zhanghaoyu (A) wrote: Hi, all Segmentation fault happened when reboot VM after hot-unplug virtio NIC, which can be reproduced 100%. See similar bug report to https://bugzilla.redhat.com/show_bug.cgi?id=988256 test environment: host: SLES11SP2

[Qemu-devel] [PATCH 01/38] qdev: document assumption that unrealize is followed by finalize

2013-09-03 Thread Paolo Bonzini
This becomes important when undoing realize's initializations is split in two places (unrealize and exit). The way to fix this could be to split realize further into alloc (done once) and realize (can be undone). Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- include/hw/qdev-core.h | 4

[Qemu-devel] [PATCH 07/38] tpci200: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/char/tpci200.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/char/tpci200.c b/hw/char/tpci200.c index d9e17b2..948a188 100644 --- a/hw/char/tpci200.c +++ b/hw/char/tpci200.c @@ -613,8 +613,9 @@ static int

[Qemu-devel] [PATCH 03/38] ac97: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/audio/ac97.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/audio/ac97.c b/hw/audio/ac97.c index 01b4dfb..04ae601 100644 --- a/hw/audio/ac97.c +++ b/hw/audio/ac97.c @@ -1390,8 +1390,9 @@ static int ac97_initfn

[Qemu-devel] [PATCH 06/38] serial: reclaim memory in instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/char/serial-pci.c | 24 1 file changed, 24 insertions(+) diff --git a/hw/char/serial-pci.c b/hw/char/serial-pci.c index aec6705..6dd34d3 100644 --- a/hw/char/serial-pci.c +++ b/hw/char/serial-pci.c @@ -131,6 +131,14

[Qemu-devel] [PATCH 12/38] ide/piix: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/ide/piix.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/hw/ide/piix.c b/hw/ide/piix.c index e6e6c0b..ddd72c1 100644 --- a/hw/ide/piix.c +++ b/hw/ide/piix.c @@ -200,9 +200,9 @@ PCIDevice

[Qemu-devel] [PATCH 04/38] es1370: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/audio/es1370.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/audio/es1370.c b/hw/audio/es1370.c index adb66ce..b68fb84 100644 --- a/hw/audio/es1370.c +++ b/hw/audio/es1370.c @@ -1044,8 +1044,9 @@ static int

[Qemu-devel] [PATCH 13/38] ide/via: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/ide/via.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/ide/via.c b/hw/ide/via.c index e5fb297..0b63bee 100644 --- a/hw/ide/via.c +++ b/hw/ide/via.c @@ -191,9 +191,9 @@ static int vt82c686b_ide_initfn(PCIDevice

[Qemu-devel] [PATCH 25/38] pcie_aer: pcie_aer_exit really frees stuff

2013-09-03 Thread Paolo Bonzini
Rename it to pcie_aer_free, and move it together with other freeing functions. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/pci-bridge/ioh3420.c| 2 +- hw/pci-bridge/xio3130_downstream.c | 2 +- hw/pci-bridge/xio3130_upstream.c | 2 +- hw/pci/pcie_aer.c

[Qemu-devel] [PATCH 18/38] eepro100: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/net/eepro100.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/net/eepro100.c b/hw/net/eepro100.c index ffa60d5..9d45aa4 100644 --- a/hw/net/eepro100.c +++ b/hw/net/eepro100.c @@ -1840,8 +1840,9 @@ static void

[Qemu-devel] [PATCH 08/38] pci-assign: reclaim memory in instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/i386/kvm/pci-assign.c | 8 1 file changed, 8 insertions(+) diff --git a/hw/i386/kvm/pci-assign.c b/hw/i386/kvm/pci-assign.c index 011764f..9d0ff3f 100644 --- a/hw/i386/kvm/pci-assign.c +++ b/hw/i386/kvm/pci-assign.c @@ -1829,6

[Qemu-devel] [PATCH 15/38] pci-testdev: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/misc/pci-testdev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/misc/pci-testdev.c b/hw/misc/pci-testdev.c index ca53b3f..10bf145 100644 --- a/hw/misc/pci-testdev.c +++ b/hw/misc/pci-testdev.c @@ -280,9 +280,9

[Qemu-devel] [PATCH 31/38] esp: use instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/scsi/esp-pci.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/scsi/esp-pci.c b/hw/scsi/esp-pci.c index d7ec173..127868d 100644 --- a/hw/scsi/esp-pci.c +++ b/hw/scsi/esp-pci.c @@ -374,9 +374,9 @@ static int

[Qemu-devel] [PATCH 14/38] ivshmem: reclaim memory in instance_finalize instead of exit

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/misc/ivshmem.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c index 2838866..46d8c27 100644 --- a/hw/misc/ivshmem.c +++ b/hw/misc/ivshmem.c @@ -784,17 +784,23 @@

[Qemu-devel] [PATCH 23/38] shpc: split shpc_free from shpc_cleanup

2013-09-03 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/pci-bridge/pci_bridge_dev.c | 2 ++ hw/pci/shpc.c | 8 +++- include/hw/pci/shpc.h | 1 + 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/hw/pci-bridge/pci_bridge_dev.c

  1   2   3   >