Re: [Qemu-devel] [PATCH for-2.9 v2] virtio-crypto: zeroize the key material before free

2016-12-09 Thread Gonglei (Arei)
> > > On 09.12.2016 02:42, Gonglei (Arei) wrote: > > Hi, > > > >> > >> Subject: Re: [Qemu-devel] [PATCH for-2.9 v2] virtio-crypto: zeroize the key > >> material before free > >> > >> On 08.12.2016 16:23, Eric Blake wrote: > >>> On 12/07/2016 08:28 PM, Gonglei (Arei) wrote: > >>> > > As far as

Re: [Qemu-devel] [PATCH v6 1/2] sparc: fix a building error reported by kbuild

2016-12-09 Thread Sam Ravnborg
Hi Gonglei. On Thu, Dec 08, 2016 at 12:37:08PM +0800, Gonglei wrote: > >> arch/sparc/include/asm/topology_64.h:44:44: > error: implicit declaration of function 'cpu_data' > [-Werror=implicit-function-declaration] > > #define topology_physical_package_id(cpu) (cpu_data(cpu).proc_id) >

Re: [Qemu-devel] [PATCH 3/8] 9pfs: fix P9_NOTAG and P9_NOFID macros

2016-12-09 Thread Eric Blake
On 12/09/2016 03:28 AM, Greg Kurz wrote: > The u16 and u32 types don't exist in QEMU common headers. It never broke > build because these two macros aren't use by the current code, but this > is about to change with the future addition of functional tests for 9P. > > This patch convert the types t

Re: [Qemu-devel] [PATCH v3 1/1] migration: disallow migrate_add_blocker during migration

2016-12-09 Thread John Snow
On 12/09/2016 12:39 PM, Dr. David Alan Gilbert wrote: > * John Snow (js...@redhat.com) wrote: >> If a migration is already in progress and somebody attempts >> to add a migration blocker, this should rightly fail. >> >> Add an errp parameter and a retcode return value to migrate_add_blocker. >> >

[Qemu-devel] Reproducible crash on PCIe hotplug

2016-12-09 Thread Eduardo Habkost
Using latest qemu.git master: $ qemu-system-x86_64 -machine q35 -readconfig docs/q35-chipset.cfg -monitor stdio QEMU 2.7.93 monitor - type 'help' for more information (qemu) device_add e1000e,bus=ich9-pcie-port-4,addr=00 (qemu) device_add e1000e,bus=ich9-pcie-port-4,addr=08 Segmentation

Re: [Qemu-devel] [RFC for-2.8] machine: Convert abstract typename on compat_props to subclass names

2016-12-09 Thread Eduardo Habkost
On Wed, Dec 07, 2016 at 02:39:25PM +0100, Greg Kurz wrote: > On Tue, 6 Dec 2016 17:31:59 -0200 > Eduardo Habkost wrote: > > On Tue, Dec 06, 2016 at 05:19:52PM -0200, Eduardo Habkost wrote: > > > On Tue, Dec 06, 2016 at 06:50:47PM +0100, Greg Kurz wrote: > > > > Since commit "9a4c0e220d8a hw/virt

Re: [Qemu-devel] [PATCH 2/2] 9p: v9fs new readpages.

2016-12-09 Thread Edward Shishkin
Hello Alexander, Thank you for the comments. Please, find my answers below. On 10/25/2016 04:13 PM, Alexander Graf wrote: On 10/10/2016 07:24 PM, Edward Shishkin wrote: Modify v9fs private ->readpages() method of address_space operations for merging pages into long 9p messages. Signed-off-by

Re: [Qemu-devel] [PATCH 1/2] 9p: v9fs add writepages.

2016-12-09 Thread Edward Shishkin
On 10/25/2016 04:01 PM, Alexander Graf wrote: On 10/10/2016 07:24 PM, Edward Shishkin wrote: Add a v9fs private ->writepages() method of address_space operations for merging pages into long 9p messages. Signed-off-by: Edward Shishkin --- fs/9p/v9fs.c | 46 +++ fs/9p/v9fs.h |

[Qemu-devel] [PATCH v3 1/2] tests: check-qom-proplist: add checks for cmdline-created objects

2016-12-09 Thread Michael Roth
check-qom-proplist originally added tests for verifying that object-creation helpers object_new_with_{props,propv} behaved in similar fashion to the "traditional" method involving setting each individual property separately after object creation rather than via a single call. Another similar "help

[Qemu-devel] [PATCH for-2.9] qom: Make all interface types abstract

2016-12-09 Thread Eduardo Habkost
"qom-list-types abstract=false" currently returns all interface types, as if they were not abstract. Fix this by making sure all interface types are abstract. All interface types have instance_size == 0, so we can use it to set abstract=true on Signed-off-by: Eduardo Habkost --- qom/object.c

Re: [Qemu-devel] [PATCH 19/20] xtensa: Move CPU files to target/ folder

2016-12-09 Thread Max Filippov
On Fri, Dec 9, 2016 at 4:17 AM, Thomas Huth wrote: > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 2 +- > {target-xtensa => target/xtensa}/Makefile.objs| 0 > {target-xtensa => target/xtensa}/core-dc232b.c| 0 > {target-xt

[Qemu-devel] [PATCH v3 2/2] monitor: fix object_del for command-line-created objects

2016-12-09 Thread Michael Roth
Currently objects specified on the command-line are only partially cleaned up when 'object_del' is issued in either HMP or QMP: the object itself is fully finalized, but the QemuOpts are not removed. This results in the following behavior: x86_64-softmmu/qemu-system-x86_64 -monitor stdio \ -

[Qemu-devel] [PATCH v3 0/2] Fixes/tests for hmp_object_del()

2016-12-09 Thread Michael Roth
hmp_object_del() followed by a subsequent hmp_object_add() can trigger a duplicate ID error if the previous object shared the same ID and was added via the command-line. Please see patch 2/2 for more details. This patchset fixes the issue in question and adds some general unit tests for object cre

Re: [Qemu-devel] [PATCH 15/21] qcow2: add .bdrv_can_store_dirty_bitmap

2016-12-09 Thread Vladimir Sementsov-Ogievskiy
09.12.2016 20:28, Max Reitz wrote: On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: Realize .bdrv_can_store_dirty_bitmap interface. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/qcow2-bitmap.c | 40 block/qcow2.c| 1 + block

Re: [Qemu-devel] [PATCH 13/21] qcow2: add .bdrv_store_persistent_dirty_bitmaps()

2016-12-09 Thread Vladimir Sementsov-Ogievskiy
09.12.2016 20:05, Max Reitz wrote: On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: Realize block bitmap storing interface, to allow qcow2 images store persistent bitmaps. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/qcow2-bitmap.c | 451 +

Re: [Qemu-devel] [PATCH 17/21] qmp: add autoload parameter to block-dirty-bitmap-add

2016-12-09 Thread Max Reitz
On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: > Optional. Default is false. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Signed-off-by: Denis V. Lunev > --- > blockdev.c| 18 -- > docs/qmp-commands.txt | 4 > qapi/block-core.json | 6 +- >

Re: [Qemu-devel] [PATCH v3 0/1] migration: disallow migrate_add_blocker during migration

2016-12-09 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH v3 0/1] migration: disallow migrate_add_blocker during migration Type: series Message-id: 20161209172547.31550-1-js...@redhat.com === TEST SCRIPT BEGIN === #!/bin/bash

Re: [Qemu-devel] [PATCH v3 1/1] migration: disallow migrate_add_blocker during migration

2016-12-09 Thread Dr. David Alan Gilbert
* John Snow (js...@redhat.com) wrote: > If a migration is already in progress and somebody attempts > to add a migration blocker, this should rightly fail. > > Add an errp parameter and a retcode return value to migrate_add_blocker. > > Signed-off-by: John Snow > --- > block/qcow.c

Re: [Qemu-devel] [PATCH 16/21] qmp: add persistent flag to block-dirty-bitmap-add

2016-12-09 Thread Max Reitz
On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: > Add optional 'persistent' flag to qmp command block-dirty-bitmap-add. > Default is false. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Signed-off-by: Denis V. Lunev > --- > blockdev.c| 17 - > docs/qmp-c

Re: [Qemu-devel] [PATCH 15/21] qcow2: add .bdrv_can_store_dirty_bitmap

2016-12-09 Thread Max Reitz
On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: > Realize .bdrv_can_store_dirty_bitmap interface. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-bitmap.c | 40 > block/qcow2.c| 1 + > block/qcow2.h| 4 >

Re: [Qemu-devel] [PATCH kvm-unit-tests v8 09/10] arm/arm64: gicv3: add an IPI test

2016-12-09 Thread Andrew Jones
On Fri, Dec 09, 2016 at 04:08:00PM +, Andre Przywara wrote: > On 08/12/16 17:50, Andrew Jones wrote: > > +u32 gicv3_iar_irqnr(u32 iar) > > +{ > > + return iar; > > I am probably a bit paranoid here, but the spec says that the interrupt > ID is in bits[23:0] only (at most). Indeed, I'll add

[Qemu-devel] [PATCH v3 1/1] migration: disallow migrate_add_blocker during migration

2016-12-09 Thread John Snow
If a migration is already in progress and somebody attempts to add a migration blocker, this should rightly fail. Add an errp parameter and a retcode return value to migrate_add_blocker. Signed-off-by: John Snow --- block/qcow.c | 6 +- block/vdi.c | 6 +

[Qemu-devel] [PATCH v3 0/1] migration: disallow migrate_add_blocker during migration

2016-12-09 Thread John Snow
As discussed on-list, this may be a necessary building block for a run-time flag that prevents us from getting into any situations where we configure a non-migratable QEMU. I'd like extra attention to be paid to the virtio-gpu, vhost, and ivshmem changes as I am not perfectly confident in those.

Re: [Qemu-devel] [PATCH v4 61/64] qemu/host-utils.h: Reduce the operation count in the fallback ctpop

2016-12-09 Thread Richard Henderson
On 12/09/2016 06:41 AM, Alex Bennée wrote: > +struct bitcnt_test_data sixtyfour_bit_data[] = { > +{ { .w64 = 0x0001 }, .popct=1 }, Thanks. Merged with added ULL for the 64-bit data. r~

Re: [Qemu-devel] [PATCH 03/19] check-help: add some hint to run a single test

2016-12-09 Thread Alex Bennée
Pranith Kumar writes: > From: Marc-André Lureau > > It's possible to run the test directly, tests/test-foo, but then > recompilation is not triggered and test environment variables aren't set > for you. > > Signed-off-by: Marc-André Lureau > Signed-off-by: Pranith Kumar Reviewed-by: Alex Ben

Re: [Qemu-devel] [PATCH 04/19] tests/tcg: comment unused code

2016-12-09 Thread Alex Bennée
Pranith Kumar writes: > From: Marc-André Lureau > > I didn't remove it, as suggest by checkpatch, because it looks like it > may eventually be restored back. I'm not so sure, especially with having full source control. Either fix it or delete it please. > > Signed-off-by: Marc-André Lureau >

Re: [Qemu-devel] [PATCH 02/19] linux-user: fix tcg/mmap test

2016-12-09 Thread Alex Bennée
Pranith Kumar writes: > From: Marc-André Lureau > > tests/tcg/mmap test fails with values other than default target page > size. When creating a map beyond EOF, extra anonymous pages are added up > to the target page boundary. Currently, this operation is performed only > when qemu_real_host_pa

Re: [Qemu-devel] [PATCH 14/21] block: add bdrv_can_store_dirty_bitmap

2016-12-09 Thread Max Reitz
On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: > This will be needed to check some restrictions before making bitmap > persistent in qmp-block-dirty-bitmap-add (this functionality will be > added by future patch) > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block.c

Re: [Qemu-devel] [for-2.9 2/5] pseries: Stubs for HPT resizing

2016-12-09 Thread Michael Roth
Quoting David Gibson (2016-12-08 20:23:11) > This introduces stub implementations of the H_RESIZE_HPT_PREPARE and > H_RESIZE_HPT_COMMIT hypercalls which we hope to add in a PAPR > extension to allow run time resizing of a guest's hash page table. It > also adds a new machine property for controlli

Re: [Qemu-devel] [PATCH 13/21] qcow2: add .bdrv_store_persistent_dirty_bitmaps()

2016-12-09 Thread Max Reitz
On 22.11.2016 18:26, Vladimir Sementsov-Ogievskiy wrote: > Realize block bitmap storing interface, to allow qcow2 images store > persistent bitmaps. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-bitmap.c | 451 > +++ > block/

Re: [Qemu-devel] [PATCH 01/19] linux-user: fix settime old value location

2016-12-09 Thread Alex Bennée
Pranith Kumar writes: > From: Marc-André Lureau > > old_value is the 4th argument of timer_settime(), not the 2nd. > > Signed-off-by: Marc-André Lureau > Signed-off-by: Pranith Kumar Reviewed-by: Alex Bennée but this should really be sent in another series. > --- > linux-user/syscall.c |

Re: [Qemu-devel] [PATCH 01/20] Makefile: Allow CPU targets to reside in target/ folder, too

2016-12-09 Thread Thomas Huth
On 09.12.2016 13:24, Laurent Vivier wrote: > Le 09/12/2016 à 13:17, Thomas Huth a écrit : >> To be able to compile the CPU targets from within a subfolder >> of the target/ folder, we've got to adapt the Makefile.target >> a little bit first. After this change, target CPUs can either >> reside in a

Re: [Qemu-devel] [PATCH v4 62/64] tcg: Use ctpop to generate ctz if needed

2016-12-09 Thread Richard Henderson
On 12/09/2016 08:07 AM, Alex Bennée wrote: > > Richard Henderson writes: > >> Particularly when andc is also available, this is two insns >> shorter than using clz to compute ctz. >> >> Signed-off-by: Richard Henderson >> --- >> tcg/tcg-op.c | 107 >> --

Re: [Qemu-devel] dpdk/vpp and cross-version migration for vhost

2016-12-09 Thread Daniel P. Berrange
On Fri, Dec 09, 2016 at 05:45:13PM +0100, Maxime Coquelin wrote: > > > On 12/09/2016 03:42 PM, Daniel P. Berrange wrote: > > On Fri, Dec 09, 2016 at 02:35:58PM +0100, Maxime Coquelin wrote: > > > ++Daniel for libvirt > > > > > > On 11/24/2016 07:31 AM, Yuanhan Liu wrote: > > > > > > > > > > > As

Re: [Qemu-devel] [PATCH 04/20] alpha: Move CPU files to target/ folder

2016-12-09 Thread Richard Henderson
On 12/09/2016 04:17 AM, Thomas Huth wrote: > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 2 +- > hw/alpha/alpha_sys.h | 2 +- > {target-alpha => target/alpha}/Makefile.objs | 0 > {target-alpha => target/alpha}/STATUS| 0 > {ta

Re: [Qemu-devel] dpdk/vpp and cross-version migration for vhost

2016-12-09 Thread Maxime Coquelin
On 12/09/2016 03:42 PM, Daniel P. Berrange wrote: On Fri, Dec 09, 2016 at 02:35:58PM +0100, Maxime Coquelin wrote: ++Daniel for libvirt On 11/24/2016 07:31 AM, Yuanhan Liu wrote: As version here is an opaque string for libvirt and qemu, anything can be used - but I suggest either a list of

Re: [Qemu-devel] [PATCH kernel v5 0/5] Extend virtio-balloon for fast (de)inflating & fast live migration

2016-12-09 Thread Andrea Arcangeli
Hello, On Fri, Dec 09, 2016 at 05:35:45AM +, Li, Liang Z wrote: > > On 12/08/2016 08:45 PM, Li, Liang Z wrote: > > > What's the conclusion of your discussion? It seems you want some > > > statistic before deciding whether to ripping the bitmap from the ABI, > > > am I right? > > > > I think

[Qemu-devel] [PATCH 2/4] hw/arm/virt: Merge VirtBoardInfo and VirtMachineState

2016-12-09 Thread Peter Maydell
One of the purposes of VirtBoardInfo was to hold various bits of state about the board. Now we have MachineState and the subclass VirtMachineState to do this. Fold the VirtBoardInfo into VirtMachineState rather than having some flags in one struct and some in another with no useful way to get betwe

[Qemu-devel] [PATCH 4/4] hw/arm/virt: Don't incorrectly claim architectural timer to be edge-triggered

2016-12-09 Thread Peter Maydell
The architectural timers in ARM CPUs all have level triggered interrupts (unless you're using KVM on a host kernel before 4.4, which misimplemented them as edge-triggered). We were incorrectly describing them in the device tree as edge triggered. This can cause problems for guest kernels in 4.8 be

[Qemu-devel] [PATCH 3/4] hw/arm/virt: Rename 'vbi' variables to 'vms'

2016-12-09 Thread Peter Maydell
Rename all the variables which used to be VirtBoardInfo* and are now VirtMachineState* so their names are in line with the type being used. Apart from the removal of the line 'VirtMachineState *vbi = vms;' this commit is purely a search-and-replace of 'vbi' with 'vms'. Signed-off-by: Peter Maydel

[Qemu-devel] [PATCH 1/4] hw/arm/virt: add 2.9 machine type

2016-12-09 Thread Peter Maydell
Signed-off-by: Peter Maydell --- include/hw/compat.h | 3 +++ hw/arm/virt.c | 19 +-- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/include/hw/compat.h b/include/hw/compat.h index 0f06e11..f8b8354 100644 --- a/include/hw/compat.h +++ b/include/hw/compat.h

[Qemu-devel] [PATCH 0/4] hw/arm/virt: 2.9 machtype, fix timer dt info

2016-12-09 Thread Peter Maydell
This patchset fixes an error in how we were reporting the architectural timers in the device tree for the virt board. Old KVM host kernels (before 4.4) implemented these timers as edge-triggered, which led us to decide to report them in the DT as edge triggered. However in newer kernels this was fi

Re: [Qemu-devel] [PATCH for-2.8 2/2] tests: check-qom-proplist: add checks for cmdline-created objects

2016-12-09 Thread Michael Roth
Quoting Markus Armbruster (2016-12-07 05:10:47) > "Daniel P. Berrange" writes: > > > On Tue, Dec 06, 2016 at 02:15:00PM -0600, Michael Roth wrote: > >> check-qom-proplist originally added tests for verifying that > >> object-creation helpers object_new_with_{props,propv} behaved in > >> similar f

Re: [Qemu-devel] [PATCH for-2.8 1/2] monitor: fix object_del for command-line-created objects

2016-12-09 Thread Michael Roth
Quoting Markus Armbruster (2016-12-07 04:36:20) > Michael Roth writes: > > > Currently objects specified on the command-line are only partially > > cleaned up when 'object_del' is issued in either HMP or QMP: the > > object itself is fully finalized, but the QemuOpts are not removed. > > This res

Re: [Qemu-devel] [PATCH v4 00/64] tcg 2.9 patch queue

2016-12-09 Thread Alex Bennée
Richard Henderson writes: > This is a combination of two patch sets that have had previous > revisions, as well as some new patches. I wanted to post this > all together since Alex was having trouble with prerequisites. > > The full tree is at > > git://github.com/rth7680/qemu.git tcg-2.9 OK

Re: [Qemu-devel] [PATCH v4 62/64] tcg: Use ctpop to generate ctz if needed

2016-12-09 Thread Alex Bennée
Richard Henderson writes: > Particularly when andc is also available, this is two insns > shorter than using clz to compute ctz. > > Signed-off-by: Richard Henderson > --- > tcg/tcg-op.c | 107 > --- > 1 file changed, 65 insertions(+), 4

Re: [Qemu-devel] [PATCH kvm-unit-tests v8 09/10] arm/arm64: gicv3: add an IPI test

2016-12-09 Thread Andre Przywara
Hi, On 08/12/16 17:50, Andrew Jones wrote: > Signed-off-by: Andrew Jones > > --- > v8: > - keep the gic_common_ops concept completely local to >lib/arm/gic.c by instead exposing the more useful >concept of gic-specific functions > - sysreg rebase changes > - ordered ICC registers in s

Re: [Qemu-devel] [PATCH v4 04/64] tcg/aarch64: Implement field extraction opcodes

2016-12-09 Thread Alex Bennée
Richard Henderson writes: > On 12/06/2016 04:24 AM, Alex Bennée wrote: >>> > +case INDEX_op_extract_i64: >>> > +case INDEX_op_extract_i32: >>> > +tcg_out_ubfm(s, ext, a0, a1, a2, a2 + args[3] - 1); >>> > +break; >>> > + >>> > +case INDEX_op_sextract_i64: >>> > +ca

[Qemu-devel] Any QEMU 2.8 release blockers?

2016-12-09 Thread Stefan Hajnoczi
Hi folks, How is QEMU 2.8.0-rc3 holding up? I'm not aware of release blockers. QEMU 2.8.0 will be released on Tuesday, December 13th if nothing comes up. It will be the -rc3 source tree. Stefan

Re: [Qemu-devel] [PATCH RFC 0/1] Allow storing the qcow2 L2 cache in disk

2016-12-09 Thread Alberto Garcia
On Fri 09 Dec 2016 03:18:23 PM CET, Kevin Wolf wrote: >> I have been making some tests with exactly that scenario and the >> results look good: storing the cache in disk gives roughly the same >> performance as storing it in memory. >> >> |-++--++---

Re: [Qemu-devel] [PATCH v4 61/64] qemu/host-utils.h: Reduce the operation count in the fallback ctpop

2016-12-09 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée As it looked like we were messing about with the Hackers Delight algorithms I thought it might be worth defending any changes with some unit tests. Feel free to include the unit test bellow: --8<--

Re: [Qemu-devel] dpdk/vpp and cross-version migration for vhost

2016-12-09 Thread Daniel P. Berrange
On Fri, Dec 09, 2016 at 02:35:58PM +0100, Maxime Coquelin wrote: > ++Daniel for libvirt > > On 11/24/2016 07:31 AM, Yuanhan Liu wrote: > > > > > > > > > As version here is an opaque string for libvirt and qemu, > > > > > > > > > > > > > > >>anything can be used - but I suggest either a list > > >

[Qemu-devel] [PATCH] hw/intc/arm_gicv3_common: fix aff3 in typer

2016-12-09 Thread Andrew Jones
Signed-off-by: Andrew Jones --- hw/intc/arm_gicv3_common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/intc/arm_gicv3_common.c b/hw/intc/arm_gicv3_common.c index 0f8c4b86e023..0aa9b9ca6655 100644 --- a/hw/intc/arm_gicv3_common.c +++ b/hw/intc/arm_gicv3_common.c @@ -20

Re: [Qemu-devel] [PATCH 11/20] sparc: Move CPU files to target/ folder

2016-12-09 Thread Artyom Tarasenko
Acked-By: Artyom Tarasenko (Forgot to hit "Reply to all" in the previous mail) > On Fri, Dec 9, 2016 at 1:17 PM, Thomas Huth wrote: >> Signed-off-by: Thomas Huth >> --- >> MAINTAINERS | 2 +- >> Makefile.objs | 2 +- >> {targe

Re: [Qemu-devel] [PATCH RFC 0/1] Allow storing the qcow2 L2 cache in disk

2016-12-09 Thread Max Reitz
On 09.12.2016 14:47, Alberto Garcia wrote: > Hi all, > > as we all know, one of the main things that can make the qcow2 format > slow is the need to load entries from the L2 table in order to map a > guest offset (on the virtual disk) to a host offset (on the qcow2 > image). > > We have an L2 cac

Re: [Qemu-devel] [PATCH RFC 0/1] Allow storing the qcow2 L2 cache in disk

2016-12-09 Thread Kevin Wolf
Am 09.12.2016 um 14:47 hat Alberto Garcia geschrieben: > Hi all, > > as we all know, one of the main things that can make the qcow2 format > slow is the need to load entries from the L2 table in order to map a > guest offset (on the virtual disk) to a host offset (on the qcow2 > image). > > We ha

Re: [Qemu-devel] [PATCH 02/36] qdict: Add convenience helpers for wrapped puts

2016-12-09 Thread Alberto Garcia
On Wed 30 Nov 2016 08:44:20 PM CET, Eric Blake wrote: > Quite a few users of qdict_put() were manually wrapping a > non-QObject. We can make such call-sites shorter, by providing > common macros to do the tedious work. Also shorten nearby > qdict_put_obj(,,QOBJECT()) sequences. > > Signed-off-by:

Re: [Qemu-devel] [PATCH for-2.9 v2] virtio-crypto: zeroize the key material before free

2016-12-09 Thread Max Reitz
On 09.12.2016 02:42, Gonglei (Arei) wrote: > Hi, > >> >> Subject: Re: [Qemu-devel] [PATCH for-2.9 v2] virtio-crypto: zeroize the key >> material before free >> >> On 08.12.2016 16:23, Eric Blake wrote: >>> On 12/07/2016 08:28 PM, Gonglei (Arei) wrote: >>> > As far as I'm aware, other projects

[Qemu-devel] [PATCH RFC 1/1] qcow2: Allow storing the qcow2 L2 cache in disk

2016-12-09 Thread Alberto Garcia
In scenarios where we have a large qcow2 file stored in a slow storage backend, we can save memory by storing its L2 cache in a faster drive rather than keeping it in RAM. Signed-off-by: Alberto Garcia --- block/qcow2-cache.c | 56 + block/qcow

[Qemu-devel] [PATCH RFC 0/1] Allow storing the qcow2 L2 cache in disk

2016-12-09 Thread Alberto Garcia
Hi all, as we all know, one of the main things that can make the qcow2 format slow is the need to load entries from the L2 table in order to map a guest offset (on the virtual disk) to a host offset (on the qcow2 image). We have an L2 cache to deal with this, and as long as the cache is big enoug

Re: [Qemu-devel] dpdk/vpp and cross-version migration for vhost

2016-12-09 Thread Maxime Coquelin
++Daniel for libvirt On 11/24/2016 07:31 AM, Yuanhan Liu wrote: As version here is an opaque string for libvirt and qemu, > > > > > >>anything can be used - but I suggest either a list > > > > > >>of values defining the interface, e.g. > > > > > >>any_layout=on,max_ring=256 > > > > > >>or a vers

Re: [Qemu-devel] [for-2.9 4/5] pseries: Enable HPT resizing for 2.9

2016-12-09 Thread Laurent Vivier
On 09/12/2016 03:23, David Gibson wrote: > We've now implemented a PAPR extensions which allows PAPR guests (i.e. > "pseries" machine type) to resize their hash page table during runtime. > > However, that extension is only enabled if explicitly chosen on the > command line. This patch enables

Re: [Qemu-devel] [for-2.9 1/5] pseries: Add pseries-2.9 machine type

2016-12-09 Thread Laurent Vivier
On 09/12/2016 03:23, David Gibson wrote: > Signed-off-by: David Gibson > --- > hw/ppc/spapr.c | 23 +-- > include/hw/compat.h | 3 +++ > 2 files changed, 24 insertions(+), 2 deletions(-) > > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c > index cfadc46..0f25e83 100644

Re: [Qemu-devel] Timeslice of vCPU thread in QEMU/KVM is not stable

2016-12-09 Thread Weiwei Jia
Hi Pankaj Gupta, Thanks for your reply. I have found the problem after debug Linux Kernel. The problem is once there is I/O thread upon vCPU2 thread of VM1, there will be some mutex (synchronization) produced so that it will be preempted by vCPU2 thread of VM2. After I set "/proc/sys/kernel/sched_

Re: [Qemu-devel] [PATCH 03/20] m68k: Move CPU files to target/ folder

2016-12-09 Thread Laurent Vivier
Le 09/12/2016 à 13:17, Thomas Huth a écrit : > Signed-off-by: Thomas Huth > --- > MAINTAINERS| 2 +- > include/hw/m68k/mcf.h | 2 +- > {target-m68k => target/m68k}/Makefile.objs | 0 > {target-m68k => target/m68k}/cpu-qom.h | 0 > {target-m

Re: [Qemu-devel] [PATCH 07/20] i386: Move CPU files to target/ folder

2016-12-09 Thread Eduardo Habkost
On Fri, Dec 09, 2016 at 01:17:37PM +0100, Thomas Huth wrote: > Signed-off-by: Thomas Huth Acked-by: Eduardo Habkost > --- > MAINTAINERS | 4 ++-- > Makefile.objs| 2 +- > hw/i386/acpi-build.c

[Qemu-devel] [PATCH 19/20] xtensa: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-xtensa => target/xtensa}/Makefile.objs| 0 {target-xtensa => target/xtensa}/core-dc232b.c| 0 {target-xtensa => target/xtensa}/core-dc232b/core-isa.h | 0 {target-xte

[Qemu-devel] [PATCH 20/20] Makefile.target: Targets now have to reside in the target/ folder

2016-12-09 Thread Thomas Huth
Now that all target CPU folders have been moved to target/, we do not need to support the old naming scheme of target-xxx anymore. Signed-off-by: Thomas Huth --- Makefile.target | 4 1 file changed, 4 deletions(-) diff --git a/Makefile.target b/Makefile.target index 90b25ae..1da789c 100644

[Qemu-devel] [PATCH 15/20] openrisc: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-openrisc => target/openrisc}/Makefile.objs | 0 {target-openrisc => target/openrisc}/cpu.c | 0 {target-openrisc => target/openrisc}/cpu.h | 0 {target-openrisc =>

Re: [Qemu-devel] [PATCH 10/20] s390x: Move CPU files to target/ folder

2016-12-09 Thread Cornelia Huck
On Fri, 9 Dec 2016 13:17:40 +0100 Thomas Huth wrote: > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 8 > Makefile.objs | 2 +- > {target-s390x => target/s390x}/Makefile.objs | 2 +- > {target-s390x => t

[Qemu-devel] [PATCH 13/20] lm32: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS| 2 +- {target-lm32 => target/lm32}/Makefile.objs | 0 {target-lm32 => target/lm32}/README| 0 {target-lm32 => target/lm32}/TODO | 0 {target-lm32 => target/lm32}/cpu-qom.h | 0 {target-lm32 => target/

[Qemu-devel] [PATCH 10/20] s390x: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 8 Makefile.objs | 2 +- {target-s390x => target/s390x}/Makefile.objs | 2 +- {target-s390x => target/s390x}/arch_dump.c| 0 {target-s390x => target/s390x}/

Re: [Qemu-devel] [PATCH 10/20] s390x: Move CPU files to target/ folder

2016-12-09 Thread Christian Borntraeger
On 12/09/2016 01:17 PM, Thomas Huth wrote: > Signed-off-by: Thomas Huth grep does not seem to find any other place. Reviewed-by: Christian Borntraeger > --- > MAINTAINERS | 8 > Makefile.objs | 2 +- > {target-

[Qemu-devel] [PATCH 09/20] mips: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 4 ++-- include/hw/mips/cpudevs.h | 2 +- {target-mips => target/mips}/Makefile.objs| 0 {target-mips => target/mips}/TODO | 0 {target-mips => target/mips}/cpu-qom.h| 0 {ta

Re: [Qemu-devel] [PATCH 13/20] lm32: Move CPU files to target/ folder

2016-12-09 Thread Michael Walle
Am 2016-12-09 13:17, schrieb Thomas Huth: Signed-off-by: Thomas Huth --- MAINTAINERS| 2 +- {target-lm32 => target/lm32}/Makefile.objs | 0 {target-lm32 => target/lm32}/README| 0 {target-lm32 => target/lm32}/TODO | 0 {target-lm32 => target/lm32

[Qemu-devel] [PATCH 05/20] arm: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS| 4 ++-- Makefile.objs | 2 +- hw/arm/strongarm.h | 2 +- hw/arm/virt-acpi-build.c | 2 +- include/hw/arm/arm.h | 2 +- include/h

Re: [Qemu-devel] [PATCH 17/20] tricore: Move CPU files to target/ folder

2016-12-09 Thread Bastian Koppelmann
On 12/09/2016 01:17 PM, Thomas Huth wrote: > Signed-off-by: Thomas Huth > --- > MAINTAINERS | 2 +- > {target-tricore => target/tricore}/Makefile.objs | 0 > {target-tricore => target/tricore}/cpu-qom.h | 0 > {target-tricore => target/tricore}

[Qemu-devel] [PATCH 08/20] microblaze: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-microblaze => target/microblaze}/Makefile.objs | 0 {target-microblaze => target/microblaze}/cpu-qom.h | 0 {target-microblaze => target/microblaze}/cpu.c | 0

[Qemu-devel] [PATCH 17/20] tricore: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-tricore => target/tricore}/Makefile.objs | 0 {target-tricore => target/tricore}/cpu-qom.h | 0 {target-tricore => target/tricore}/cpu.c | 0 {target-tricore => target/trico

[Qemu-devel] [PATCH 03/20] m68k: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS| 2 +- include/hw/m68k/mcf.h | 2 +- {target-m68k => target/m68k}/Makefile.objs | 0 {target-m68k => target/m68k}/cpu-qom.h | 0 {target-m68k => target/m68k}/cpu.c | 0 {target-m68k => targ

Re: [Qemu-devel] [PATCH 20/20] Makefile.target: Targets now have to reside in the target/ folder

2016-12-09 Thread Laurent Vivier
Le 09/12/2016 à 13:17, Thomas Huth a écrit : > Now that all target CPU folders have been moved to target/, we > do not need to support the old naming scheme of target-xxx > anymore. > > Signed-off-by: Thomas Huth > --- > Makefile.target | 4 > 1 file changed, 4 deletions(-) > > diff --git

[Qemu-devel] [PATCH 07/20] i386: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 4 ++-- Makefile.objs| 2 +- hw/i386/acpi-build.c | 2 +- hw/i386/kvm/apic.c | 2 +- hw/intc/ioapic.c

[Qemu-devel] [PATCH 16/20] sh4: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- hw/sh4/shix.c| 2 +- include/hw/sh4/sh.h | 2 +- {target-sh4 => target/sh4}/Makefile.objs | 0 {target-sh4 => target/sh4}/README.sh4| 2 +- {target-sh4 => target/sh

[Qemu-devel] [PATCH v2 5/6] target-ppc: implement xxextractuw instruction

2016-12-09 Thread Nikunj A Dadhania
xxextractuw: VSX Vector Extract Unsigned Word Signed-off-by: Nikunj A Dadhania --- target-ppc/helper.h | 1 + target-ppc/int_helper.c | 21 + target-ppc/translate/vsx-impl.inc.c | 27 +++ target-ppc/translate/vsx-ops.inc.c

Re: [Qemu-devel] [PATCH 01/20] Makefile: Allow CPU targets to reside in target/ folder, too

2016-12-09 Thread Laurent Vivier
Le 09/12/2016 à 13:17, Thomas Huth a écrit : > To be able to compile the CPU targets from within a subfolder > of the target/ folder, we've got to adapt the Makefile.target > a little bit first. After this change, target CPUs can either > reside in a target/xxx folder or continue to use the target-

[Qemu-devel] [PATCH 06/20] ppc: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 4 ++-- Makefile.objs | 2 +- hw/ppc/fdt.c| 2 +- hw/ppc/pnv.c| 2 +- hw/ppc/pnv_core.c

[Qemu-devel] [PATCH 12/20] cris: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- disas/cris.c | 4 +--- {target-cris => target/cris}/Makefile.objs| 0 {target-cris => target/cris}/cpu-qom.h| 0 {target-cris => target/cris}/cpu.c| 0 {ta

[Qemu-devel] [PATCH 02/20] tilegx: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- {target-tilegx => target/tilegx}/Makefile.objs | 0 {target-tilegx => target/tilegx}/cpu.c | 0 {target-tilegx => target/tilegx}/cpu.h | 0 {target-tilegx => target/tilegx}/helper.c| 0 {target-tilegx => target/tilegx}/helper.h|

[Qemu-devel] [PATCH 18/20] unicore32: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-unicore32 => target/unicore32}/Makefile.objs | 0 {target-unicore32 => target/unicore32}/cpu-qom.h | 0 {target-unicore32 => target/unicore32}/cpu.c | 0 {target-unicore32 => target

[Qemu-devel] [PATCH 04/20] alpha: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- hw/alpha/alpha_sys.h | 2 +- {target-alpha => target/alpha}/Makefile.objs | 0 {target-alpha => target/alpha}/STATUS| 0 {target-alpha => target/alpha}/cpu-qom.h | 0 {target-al

[Qemu-devel] [PATCH v2 6/6] target-ppc: implement xxinsertw instruction

2016-12-09 Thread Nikunj A Dadhania
xxinsertw: VSX Vector Insert Word Signed-off-by: Nikunj A Dadhania --- target-ppc/helper.h | 1 + target-ppc/int_helper.c | 21 + target-ppc/translate/vsx-impl.inc.c | 5 +++-- target-ppc/translate/vsx-ops.inc.c | 1 + 4 files changed, 26 inser

[Qemu-devel] [PATCH v2 4/6] target-ppc: implement stxvll instructions

2016-12-09 Thread Nikunj A Dadhania
stxvll: Store VSX Vector Left-justified with Length Vector (8-bit elements) in BE/LE: +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ |“T”|“h”|“i”|“s”|“ ”|“i”|“s”|“ ”|“a”|“ ”|“T”|“E”|“S”|“T”|00|00| +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ Storing 14 bytes wo

[Qemu-devel] [PATCH 14/20] moxie: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- {target-moxie => target/moxie}/Makefile.objs | 0 {target-moxie => target/moxie}/cpu.c | 0 {target-moxie => target/moxie}/cpu.h | 0 {target-moxie => target/moxie}/helper.c | 0 {target-moxie

[Qemu-devel] [PATCH for-2.9 00/20] Move target-* CPU file into a target/ folder

2016-12-09 Thread Thomas Huth
After my RFC last week that did not move all folder yet, here's now a patch series that moves all target-* folders into one subfolder: We've currently got 18 architectures in QEMU, and thus 18 target-xxx folders in the root folder of the QEMU source tree. More architectures (e.g. RISC-V, AVR) are

[Qemu-devel] [PATCH v2 3/6] target-ppc: implement stxvl instruction

2016-12-09 Thread Nikunj A Dadhania
stxvl: Store VSX Vector with Length Vector (8-bit elements) in BE: +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ |“T”|“h”|“i”|“s”|“ ”|“i”|“s”|“ ”|“a”|“ ”|“T”|“E”|“S”|“T”|00|00| +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ Vector (8-bit elements) in LE: +--+--+

[Qemu-devel] [PATCH 01/20] Makefile: Allow CPU targets to reside in target/ folder, too

2016-12-09 Thread Thomas Huth
To be able to compile the CPU targets from within a subfolder of the target/ folder, we've got to adapt the Makefile.target a little bit first. After this change, target CPUs can either reside in a target/xxx folder or continue to use the target-xxx scheme. The latter will be disabled once all targ

[Qemu-devel] [PATCH 11/20] sparc: Move CPU files to target/ folder

2016-12-09 Thread Thomas Huth
Signed-off-by: Thomas Huth --- MAINTAINERS | 2 +- Makefile.objs | 2 +- {target-sparc => target/sparc}/Makefile.objs | 0 {target-sparc => target/sparc}/TODO | 0 {target-sparc => target/sparc}/asi.h | 0 {targ

[Qemu-devel] [PATCH v2 2/6] target-ppc: implement lxvll instruction

2016-12-09 Thread Nikunj A Dadhania
lxvll: Load VSX Vector Left-justified with Length Little/Big-endian Storage: +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ |“T”|“h”|“i”|“s”|“ ”|“i”|“s”|“ ”|“a”|“ ”|“T”|“E”|“S”|“T”|FF|FF| +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ Loading 14 bytes to vector (

[Qemu-devel] [PATCH v2 1/6] target-ppc: implement lxvl instruction

2016-12-09 Thread Nikunj A Dadhania
lxvl: Load VSX Vector with Length Little/Big-endian Storage: +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ |“T”|“h”|“i”|“s”|“ ”|“i”|“s”|“ ”|“a”|“ ”|“T”|“E”|“S”|“T”|FF|FF| +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ Loading 14 bytes results in: Vector (8-bit

[Qemu-devel] [PATCH v2 ppc-for-2.9 0/6] POWER9 TCG enablements - part9

2016-12-09 Thread Nikunj A Dadhania
This series contains 6 new instructions for POWER9 ISA3.0 VSX Vector Insert/Extract Word VSX Load/Store with length Although, lxvl/lxvll and stxvl/stxvll can be combined as single patch, have left it for detailed commit log for instruction explanation. Changelog: v1: * Combine helpers o

  1   2   >