[Qemu-devel] [Bug 1579306] Re: usb-uas does not work in Windows (10) guest

2016-12-08 Thread Tom Yan
I think they are two separate issues in usb-uas and usb-host respectively. I probably should not have bring in the usb-host case here but create another report for it. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] [PATCH 14/14] tests: Add test-image-lock

2016-12-08 Thread Fam Zheng
On Fri, 12/02 17:30, Max Reitz wrote: > > +static struct CompatData { > > +bool write_1; > > +bool share_1; > > +bool write_2; > > +bool share_2; > > +bool compatible; > > +} compat_data[] = { > > +/* Write 1, Share 1, Write 2, Share 2, Compatible. */ > > +{ RO,

[Qemu-devel] [PATCH for-2.9 V3 3/3] Add a new qmp command to do checkpoint, get replication error

2016-12-08 Thread Zhang Chen
We can call this qmp command to do checkpoint outside of qemu. Like Xen colo need this function. Signed-off-by: Zhang Chen --- docs/qmp-commands.txt | 24 migration/colo.c | 10 ++ qapi-schema.json | 22

[Qemu-devel] [PATCH for-2.9 V3 0/3] Add new qmp commands to suppurt Xen COLO

2016-12-08 Thread Zhang Chen
Xen COLO depend on qemu COLO replication function. So, We need new qmp commands for Xen to use qemu replication. Corresponding libxl patches already in xen.git. Commit ID: ed37ef1f91c20f0ab162ce60f8c38400b917fa64 COLO: introduce new API to prepare/start/do/get_error/stop replication

[Qemu-devel] [PATCH for-2.9 V3 1/3] Migration: Don't load vmdesc when xen is enabled

2016-12-08 Thread Zhang Chen
Xen doesn't need this. Signed-off-by: Zhang Chen --- migration/savevm.c | 4 1 file changed, 4 insertions(+) diff --git a/migration/savevm.c b/migration/savevm.c index 0363372..bec6c7e 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -1973,6

[Qemu-devel] [PATCH for-2.9 V3 2/3] Add a new qmp command to start/stop replication

2016-12-08 Thread Zhang Chen
We can call this qmp command to start/stop replication outside of qemu. Like Xen colo need this function. Signed-off-by: Zhang Chen --- docs/qmp-commands.txt | 18 ++ migration/colo.c | 23 +++ qapi-schema.json | 19

[Qemu-devel] [Bug 1579306] Re: usb-uas does not work in Windows (10) guest

2016-12-08 Thread Jack Coulter
Just echoing Tom Yan's comment; I've tried passing through what appears to be the very same device (if not, at least the same UAS-SATA bridge) to a VM in its entirety. I can confirm the same result - with an otherwise identical domain configuration, Linux guests correctly use the UAS driver: /:

Re: [Qemu-devel] [PATCH v1 1/9] target-ppc: implement lxvl instruction

2016-12-08 Thread Nikunj A Dadhania
David Gibson writes: > [ Unknown signature status ] > On Wed, Dec 07, 2016 at 11:54:54PM +0530, Nikunj A Dadhania wrote: >> lxvl: Load VSX Vector with Length >> >> Little/Big-endian Storage: >> +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ >>

Re: [Qemu-devel] [PATCH v1 3/9] target-ppc: implement stxvl instruction

2016-12-08 Thread Nikunj A Dadhania
David Gibson writes: > [ Unknown signature status ] > On Wed, Dec 07, 2016 at 11:54:56PM +0530, Nikunj A Dadhania wrote: >> stxvl: Store VSX Vector with Length >> >> Vector (8-bit elements) in BE: >> +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ >>

Re: [Qemu-devel] [PATCH for-2.9] cryptodev: remove single-DES support in cryptodev

2016-12-08 Thread Gonglei (Arei)
> > From: Daniel P. Berrange [mailto:berra...@redhat.com] > Sent: Wednesday, December 07, 2016 5:07 PM > To: longpeng > Cc: Gonglei (Arei); ebl...@redhat.com; Wubin (H); Zhoujian (jay, Euler); > qemu-devel@nongnu.org > Subject: Re: [PATCH for-2.9] cryptodev: remove single-DES support in cryptodev

Re: [Qemu-devel] [PATCH for-2.9] cryptodev: add 3des-ede support

2016-12-08 Thread Gonglei (Arei)
> -Original Message- > From: longpeng > Sent: Thursday, December 08, 2016 10:52 AM > To: Gonglei (Arei) > Cc: Wubin (H); Zhoujian (jay, Euler); qemu-devel@nongnu.org; longpeng > Subject: [PATCH for-2.9] cryptodev: add 3des-ede support > > This patch add 3des-ede support for cryptodev.

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

2016-12-08 Thread Li, Liang Z
> 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 Andrea and David feel pretty strongly that we should remove the > bitmap, unless

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

2016-12-08 Thread Dave Hansen
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 Andrea and David feel pretty strongly that we should remove the bitmap, unless we have some

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

2016-12-08 Thread Li, Liang Z
> > 1. Current patches do a hypercall for each order in the allocator. > >This is inefficient, but independent from the underlying data > >structure in the ABI, unless bitmaps are in play, which they aren't. > > 2. Should we have bitmaps in the ABI, even if they are not in use by the > >

Re: [Qemu-devel] [PATCH for-2.9 V2 2/3] Add a new qmp command to start/stop replication

2016-12-08 Thread Zhang Chen
On 12/08/2016 11:38 PM, Eric Blake wrote: On 12/08/2016 01:55 AM, Zhang Chen wrote: We can call this qmp command to start/stop replication outside of qemu. Like Xen colo need this function. Signed-off-by: Zhang Chen --- docs/qmp-commands.txt | 17

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

2016-12-08 Thread Li, Liang Z
> Subject: Re: [PATCH kernel v5 0/5] Extend virtio-balloon for fast > (de)inflating > & fast live migration > > On 12/07/2016 05:35 AM, Li, Liang Z wrote: > >> Am 30.11.2016 um 09:43 schrieb Liang Li: > >> IOW in real examples, do we have really large consecutive areas or > >> are all pages just

[Qemu-devel] [for-2.9 0/5] Hash Page Table resizing for TCG pseries guests

2016-12-08 Thread David Gibson
This series implements the host side of the PAPR ACR to allow runtime resizing of the Hashed Page Table (HPT) for pseries guests. Exercising this feature requires a guest OS which is also aware of it. Patches to implement the guest side in Linux have just been submitted upstream:

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

2016-12-08 Thread David Gibson
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 it by default for spapr-2.9, but leaves it disabled

[Qemu-devel] [for-2.9 5/5] pseries: Use smaller default hash page tables when guest can resize

2016-12-08 Thread David Gibson
We've now implemented a PAPR extension allowing PAPR guest to resize their hash page table (HPT) during runtime. This patch makes use of that facility to allocate smaller HPTs by default. Specifically when a guest is aware of the HPT resize facility, qemu sizes the HPT to the initial memory size,

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

2016-12-08 Thread David Gibson
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 controlling whether this new facility is available, and

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

2016-12-08 Thread David Gibson
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 --- a/hw/ppc/spapr.c +++

[Qemu-devel] [for-2.9 3/5] pseries: Implement HPT resizing

2016-12-08 Thread David Gibson
This patch implements hypercalls allowing a PAPR guest to resize its own hash page table. This will eventually allow for more flexible memory hotplug. The implementation is partially asynchronous, handled in a special thread running the hpt_prepare_thread() function. The state of a pending

Re: [Qemu-devel] [RFC PATCH 11/13] intel_iommu: provide its own replay() callback

2016-12-08 Thread Peter Xu
On Thu, Dec 08, 2016 at 11:01:53AM +0800, Lan Tianyu wrote: > On 2016年12月06日 18:36, Peter Xu wrote: > > +/** > > + * vtd_page_walk - walk specific IOVA range, and call the hook > > + * > > + * @ce: context entry to walk upon > > + * @start: IOVA address to start the walk > > + * @end: size of the

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

2016-12-08 Thread Gonglei (Arei)
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 usually have a special memset > >>> variation for

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 5/6] target-ppc: Implement bcdtrunc. instruction

2016-12-08 Thread joserz
On Fri, Dec 09, 2016 at 08:06:14AM +1100, David Gibson wrote: > On Thu, Dec 08, 2016 at 12:07:07AM -0200, Jose Ricardo Ziviani wrote: > > bcdtrunc.: Decimal integer truncate. Given a BCD number in vrb and the > > number of bytes to truncate in vra, the return register will have vrb > > with such

[Qemu-devel] [PATCH risu] A script to run concurrently the master and the apprentice

2016-12-08 Thread Laurent Vivier
This expect script will generate the .out file and then start the apprentice in a chroot and the master on the reference hardware. The list of patterns is given on the command line. As my reference system is very slow, ssh can take more than 2 minutes, this script starts a ssh and a chroot once,

Re: [Qemu-devel] [PATCH v1 9/9] target-ppc: Add xxperm and xxpermr instructions

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:55:02PM +0530, Nikunj A Dadhania wrote: > From: Bharata B Rao > > xxperm: VSX Vector Permute > xxpermr: VSX Vector Permute Right-indexed > > Signed-off-by: Bharata B Rao > Signed-off-by: Nikunj A Dadhania

Re: [Qemu-devel] [PATCH v1 8/9] target-ppc: implement xscpsgnqp instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:55:01PM +0530, Nikunj A Dadhania wrote: > xscpsgnqp: VSX Scalar Copy Sign Quad-Precision > > Signed-off-by: Nikunj A Dadhania > Reviewed-by: Richard Henderson Merged to ppc-for-2.9 > --- >

Re: [Qemu-devel] [PATCH v1 6/9] target-ppc: implement xxinsertw instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:54:59PM +0530, Nikunj A Dadhania wrote: > xxinsertw: VSX Vector Insert Word > > Signed-off-by: Nikunj A Dadhania > Reviewed-by: Richard Henderson > --- > target-ppc/helper.h | 1 + >

Re: [Qemu-devel] [PATCH v1 1/9] target-ppc: implement lxvl instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:54:54PM +0530, Nikunj A Dadhania wrote: > lxvl: Load VSX Vector with Length > > Little/Big-endian Storage: > +---+---+---+---+---+---+---+---+---+---+---+---+---+---+--+--+ > |“T”|“h”|“i”|“s”|“ ”|“i”|“s”|“ ”|“a”|“ ”|“T”|“E”|“S”|“T”|FF|FF| >

Re: [Qemu-devel] [PATCH v3 5/6] target-ppc: Implement bcdtrunc. instruction

2016-12-08 Thread David Gibson
On Thu, Dec 08, 2016 at 12:07:07AM -0200, Jose Ricardo Ziviani wrote: > bcdtrunc.: Decimal integer truncate. Given a BCD number in vrb and the > number of bytes to truncate in vra, the return register will have vrb > with such bits truncated. > > Signed-off-by: Jose Ricardo Ziviani

Re: [Qemu-devel] [PATCH v1 7/9] target-ppc: implement xsnegqp instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:55:00PM +0530, Nikunj A Dadhania wrote: > xsnegqp: VSX Scalar Negate Quad-Precision > > Signed-off-by: Nikunj A Dadhania Merged to ppc-for-2.9. > --- > target-ppc/translate/vsx-impl.inc.c | 4 > target-ppc/translate/vsx-ops.inc.c | 1

Re: [Qemu-devel] [PATCH v1 5/9] target-ppc: implement xxextractuw instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:54:58PM +0530, Nikunj A Dadhania wrote: > xxextractuw: VSX Vector Extract Unsigned Word > > Signed-off-by: Nikunj A Dadhania > --- > target-ppc/helper.h | 1 + > target-ppc/int_helper.c | 31

Re: [Qemu-devel] [PATCH v1 3/9] target-ppc: implement stxvl instruction

2016-12-08 Thread David Gibson
On Wed, Dec 07, 2016 at 11:54:56PM +0530, Nikunj A Dadhania wrote: > 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| >

[Qemu-devel] [Bug 613529] Re: qemu does not accept regular disk geometry

2016-12-08 Thread Thomas Huth
Which version of QEMU have you been using for your tests? Can you still reproduce this problem with the latest version of QEMU? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] migration: Introduce a new "--only-migratable" option

2016-12-08 Thread John Snow
On 12/08/2016 02:59 PM, Dr. David Alan Gilbert wrote: > * Ashijeet Acharya (ashijeetacha...@gmail.com) wrote: >> Hi Dave, >> >> I have added the compatibility of this option for both command line >> and hotplug via qmp and hmp. Although, please confirm that making use >> of "device_add" is the

Re: [Qemu-devel] [PATCH v4 23/64] tcg: Allow an operand to be matching or a constant

2016-12-08 Thread Alex Bennée
Richard Henderson writes: > On 12/08/2016 09:19 AM, Alex Bennée wrote: >> >> Richard Henderson writes: >> >>> This allows an output operand to match an input operand >>> only when the input operand needs a register. >>> >>> Signed-off-by: Richard Henderson

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

2016-12-08 Thread Weiwei Jia
Hi everyone, I am testing the timeslice of vCPU thread in QEMU/KVM. In principle, the timeslice should be stable under following workload but it is unstable after I do experiments with following workload. I appreciate it if you can give me some suggestions. Thanks in advance. Workload settings:

[Qemu-devel] [RFC] qmp: query-device-slots command

2016-12-08 Thread Eduardo Habkost
This adds a new command to QMP: query-device-slots. It will allow management software to query possible slots where devices can be plugged. This implementation of the command will return: * Multiple PCI slots per bus, in the case of PCI buses; * One slot per bus in the case of the other buses; *

Re: [Qemu-devel] [PATCH] migration: re-active images when migration fails to complete

2016-12-08 Thread Dr. David Alan Gilbert
* Hailiang Zhang (zhang.zhanghaili...@huawei.com) wrote: > Hi, > > On 2016/12/6 23:24, Dr. David Alan Gilbert wrote: > > * Kevin Wolf (kw...@redhat.com) wrote: > > > Am 19.11.2016 um 12:43 hat zhanghailiang geschrieben: > > > > commit fe904ea8242cbae2d7e69c052c754b8f5f1ba1d6 fixed a case > > > >

Re: [Qemu-devel] migration: Introduce a new "--only-migratable" option

2016-12-08 Thread Dr. David Alan Gilbert
* Ashijeet Acharya (ashijeetacha...@gmail.com) wrote: > Hi Dave, > > I have added the compatibility of this option for both command line > and hotplug via qmp and hmp. Although, please confirm that making use > of "device_add" is the only way of hotplugging devices into a QEMU > instance. Hmm;

[Qemu-devel] [Bug 1592590] Re: Prevent qemu-img resize from causing "Active L1 table too large"

2016-12-08 Thread Max Reitz
Thanks for the report, sorry for the late reply: This has been fixed in commit 84c26520d3c1c9ff4a10455748139463278816d5 (included in the 2.7.0 release). ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which

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

2016-12-08 Thread Max Reitz
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 usually have a special memset >>> variation for doing this. That is because compilers may choose to >>> "optimize" memset(p, ...) + free(p) to just the free(p). Having

[Qemu-devel] [QEMU PATCH v16 4/4] migration: add error_report

2016-12-08 Thread Jianjun Duan
Added error_report where version_ids do not match in vmstate_load_state. Signed-off-by: Jianjun Duan --- migration/vmstate.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/migration/vmstate.c b/migration/vmstate.c index 2f9d4ba..8ddd230 100644 ---

[Qemu-devel] [QEMU PATCH v16 3/4] tests/migration: Add test for QTAILQ migration

2016-12-08 Thread Jianjun Duan
Add a test for QTAILQ migration to tests/test-vmstate.c. Signed-off-by: Jianjun Duan --- tests/test-vmstate.c | 147 +++ 1 file changed, 147 insertions(+) diff --git a/tests/test-vmstate.c b/tests/test-vmstate.c index

[Qemu-devel] [QEMU PATCH v16 1/4] migration: extend VMStateInfo

2016-12-08 Thread Jianjun Duan
Current migration code cannot handle some data structures such as QTAILQ in qemu/queue.h. Here we extend the signatures of put/get in VMStateInfo so that customized handling is supported. put now will return int type. Signed-off-by: Jianjun Duan ---

[Qemu-devel] [QEMU PATCH v16 2/4] migration: migrate QTAILQ

2016-12-08 Thread Jianjun Duan
Currently we cannot directly transfer a QTAILQ instance because of the limitation in the migration code. Here we introduce an approach to transfer such structures. We created VMStateInfo vmstate_info_qtailq for QTAILQ. Similar VMStateInfo can be created for other data structures such as list.

[Qemu-devel] [QEMU PATCH v16 0/4] migration: migrate QTAILQ

2016-12-08 Thread Jianjun Duan
Hi all, I addressed some review comments. Comments are welcome. v16: - Moved a variable to global in test-vmstate.c following Dave's suggestion. Previous versions are: v15: - Improved error messages. (link: http://lists.nongnu.org/archive/html/qemu-devel/2016-12/msg01059.html) v14:

Re: [Qemu-devel] [PATCH v4 42/64] tcg/arm: Handle ctz and clz opcodes

2016-12-08 Thread Richard Henderson
On 12/08/2016 09:56 AM, Alex Bennée wrote: +if (c && a2 == 32) { +tcg_out_dat_reg(s, COND_AL, INSN_CLZ, a0, 0, a1, 0); +break; +} Why the early break instead of else leg? +tcg_out_dat_imm(s, COND_AL, ARITH_CMP, 0, a1, 0); +

Re: [Qemu-devel] [RFC PATCH 1/3] tcg: Release tb_lock in the order acquired

2016-12-08 Thread Paolo Bonzini
On 07/12/2016 16:38, Pranith Kumar wrote: > > Hi Alex, > > Alex Bennée writes: > >> >> Do you have any numbers for this? The main reason being we are trying to >> avoid bouncing the lock too much and while this is cleaner it could >> cause more contention. > > I did not really consider

Re: [Qemu-devel] [PATCH v4 42/64] tcg/arm: Handle ctz and clz opcodes

2016-12-08 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson > --- > tcg/arm/tcg-target.h | 4 ++-- > tcg/arm/tcg-target.inc.c | 27 +++ > 2 files changed, 29 insertions(+), 2 deletions(-) > > diff --git a/tcg/arm/tcg-target.h

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

2016-12-08 Thread Andrew Jones
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 spec-order (OCD kicked in...) v7: - add

[Qemu-devel] migration: Introduce a new "--only-migratable" option

2016-12-08 Thread Ashijeet Acharya
Hi Dave, I have added the compatibility of this option for both command line and hotplug via qmp and hmp. Although, please confirm that making use of "device_add" is the only way of hotplugging devices into a QEMU instance. With this sorted out, there is one special case left where the device

[Qemu-devel] [PATCH kvm-unit-tests v8 08/10] arm/arm64: add initial gicv3 support

2016-12-08 Thread Andrew Jones
Reviewed-by: Alex Bennée Reviewed-by: Eric Auger Signed-off-by: Andrew Jones --- v8: few sysreg framework and new delay rebase changes v7: split lib/arm/gic.c into gic-v2/3.c [Eric] v6: - added comments [Alex] - added stride

[Qemu-devel] [PATCH kvm-unit-tests v8 06/10] arm/arm64: gicv2: add an IPI test

2016-12-08 Thread Andrew Jones
Reviewed-by: Eric Auger Signed-off-by: Andrew Jones --- v8: - fix check_spurious print arguments and change the printf to a report_info - remove the self-ipi test issued when no subtest is given, i.e. require a subtest to be given - use

[Qemu-devel] [PATCH kvm-unit-tests v8 10/10] arm/arm64: gic: don't just use zero

2016-12-08 Thread Andrew Jones
Instead of using cpu0 and irq=0 for the IPI test, use something a bit more interesting. To make sure we can still run the test with the minimal number of cpus (2), we need to use sender=1. As the irq in the test (an SGI) can only be 0 to 15 and there's really no difference, then use irq=1.

[Qemu-devel] [PATCH kvm-unit-tests v8 05/10] arm/arm64: add initial gicv2 support

2016-12-08 Thread Andrew Jones
Add some gicv2 support. This just adds init and enable functions, allowing unit tests to start messing with it. Reviewed-by: Andre Przywara Reviewed-by: Eric Auger Signed-off-by: Andrew Jones --- v6: added comments (register

[Qemu-devel] [PATCH kvm-unit-tests v8 02/10] arm/arm64: smp: support more than 8 cpus

2016-12-08 Thread Andrew Jones
By adding support for launching with gicv3 we can break the 8 vcpu limit. This patch adds support to smp code and also selects the vgic model corresponding to the host. The vgic model may also be manually selected by adding e.g. -machine gic-version=3 to extra_params. Reviewed-by: Alex Bennée

[Qemu-devel] [PATCH kvm-unit-tests v8 04/10] arm/arm64: irq enable/disable

2016-12-08 Thread Andrew Jones
Reviewed-by: Alex Bennée Reviewed-by: Eric Auger Signed-off-by: Andrew Jones --- lib/arm/asm/processor.h | 10 ++ lib/arm64/asm/processor.h | 10 ++ 2 files changed, 20 insertions(+) diff --git

[Qemu-devel] [PATCH kvm-unit-tests v8 00/10] arm/arm64: add gic framework

2016-12-08 Thread Andrew Jones
v8: - Main change is rebasing to Wei's sysreg framework, which is part of his PMU series, which I've applied to arm/next. That rebase leads to dropping the first two patches of the v7 series, expecting get_mpidr() to return u64 (with future patches), a rework of the delay routines

[Qemu-devel] [PATCH kvm-unit-tests v8 01/10] arm/arm64: yield on cpu_relax

2016-12-08 Thread Andrew Jones
In many tests one or more cpus wait for events from other cpus. However, with TCG, if the event triggering cpus then continue without first informing TCG it should schedule other cpus, then those other cpus may never get scheduled, and never see their events. This is because the TCG scheduler

[Qemu-devel] [PATCH kvm-unit-tests v8 03/10] arm/arm64: add some delay routines

2016-12-08 Thread Andrew Jones
Allow a thread to wait some specified amount of time. Can specify in cycles, usecs, and msecs. Signed-off-by: Andrew Jones --- v8: rewrote basing on new sysreg framework. Also decided delay functions warrant their own files (delay.[ch]) --- arm/Makefile.common |

[Qemu-devel] [PATCH kvm-unit-tests v8 07/10] libcflat: add IS_ALIGNED() macro, and page sizes

2016-12-08 Thread Andrew Jones
From: Peter Xu These macros will be useful to do page alignment checks. Reviewed-by: Andre Przywara Reviewed-by: Eric Auger Signed-off-by: Peter Xu [drew: also added SZ_64K and changed to shifts]

Re: [Qemu-devel] [PATCH v4 23/64] tcg: Allow an operand to be matching or a constant

2016-12-08 Thread Richard Henderson
On 12/08/2016 09:19 AM, Alex Bennée wrote: Richard Henderson writes: This allows an output operand to match an input operand only when the input operand needs a register. Signed-off-by: Richard Henderson It's hard to offer anything more than a

Re: [Qemu-devel] [PATCH v4 38/64] target-arm: Use clz opcode

2016-12-08 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > target-arm/helper-a64.c| 10 -- > target-arm/helper-a64.h| 2 -- > target-arm/helper.c| 5 - >

Re: [Qemu-devel] [Qemu-ppc] [QEMU PATCH v14 3/4] tests/migration: Add test for QTAILQ migration

2016-12-08 Thread Jianjun Duan
On 12/08/2016 01:03 AM, Dr. David Alan Gilbert wrote: > * Jianjun Duan (du...@linux.vnet.ibm.com) wrote: >> >> >> On 12/07/2016 11:49 AM, Dr. David Alan Gilbert wrote: >>> * Jianjun Duan (du...@linux.vnet.ibm.com) wrote: Add a test for QTAILQ migration to tests/test-vmstate.c. >>> >>> Yes,

Re: [Qemu-devel] [PATCH v4 24/64] tcg: Add clz and ctz opcodes

2016-12-08 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > tcg-runtime.c| 20 +++ > tcg/README | 8 +++ > tcg/aarch64/tcg-target.h | 4 ++ >

Re: [Qemu-devel] [PATCH v4 23/64] tcg: Allow an operand to be matching or a constant

2016-12-08 Thread Alex Bennée
Richard Henderson writes: > This allows an output operand to match an input operand > only when the input operand needs a register. > > Signed-off-by: Richard Henderson It's hard to offer anything more than a mechanical review for this as the constraints

Re: [Qemu-devel] [QEMU, PATCH] x86: implement la57 paging mode

2016-12-08 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [QEMU, PATCH] x86: implement la57 paging mode Type: series Message-id: 20161208162150.148763-2-kirill.shute...@linux.intel.com === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

Re: [Qemu-devel] [RFC PATCH 6/6] iscsi: Add blockdev-add support

2016-12-08 Thread Eric Blake
On 12/08/2016 07:23 AM, Kevin Wolf wrote: > This adds blockdev-add support for iscsi devices. > > Signed-off-by: Kevin Wolf > --- > block/iscsi.c| 14 ++ > qapi/block-core.json | 74 > > 2 files changed, 78

[Qemu-devel] [QEMU, PATCH] x86: implement la57 paging mode

2016-12-08 Thread Kirill A. Shutemov
The new paging more is extension of IA32e mode with more additional page table level. It brings support of 57-bit vitrual address space (128PB) and 52-bit physical address space (4PB). The structure of new page table level is identical to pml4. The feature is enumerated with CPUID.(EAX=07H,

Re: [Qemu-devel] [Nbd] [PATCH v3] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-08 Thread Alex Bligh
> On 8 Dec 2016, at 15:59, Eric Blake wrote: > > We should use similar wording to whatever we already say about what a > client would see when reading data cleared by NBD_CMD_TRIM. After all, > the status of STATE_HOLE set and STATE_ZERO clear is what you logically > get

Re: [Qemu-devel] [Nbd] [PATCH v3] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-08 Thread Eric Blake
On 12/08/2016 08:40 AM, Alex Bligh wrote: >>> + metadata context is the basic "exists at all" metadata context. >>> >>> Disagree. You're saying that if a server supports metadata contexts >>> at all, it must support this one. >> >> No, I'm trying to say that this metadata context exposes whether

Re: [Qemu-devel] [PATCH for-2.9 v2] crypto: add 3des-ede support when using libgcrypt/nettle

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 09:45:13AM -0600, Eric Blake wrote: > On 12/07/2016 08:33 PM, Longpeng(Mike) wrote: > > Libgcrypt and nettle support 3des-ede, so this patch add 3des-ede > > support when using libgcrypt or nettle. > > > > Signed-off-by: Longpeng(Mike) > > --- > > >

Re: [Qemu-devel] [PATCH v3 0/3] Add HAX support

2016-12-08 Thread Vincent Palatin
On Thu, Dec 8, 2016 at 4:31 PM, Stefan Weil wrote: > > Am 08.12.2016 um 10:34 schrieb Yu Ning: > > As a HAXM developer at Intel, I just want to come out and show our > > support for Vincent's upstreaming effort. We'd love to see HAXM support > > code land in upstream QEMU, and

Re: [Qemu-devel] [PATCH for-2.9 v2] crypto: add 3des-ede support when using libgcrypt/nettle

2016-12-08 Thread Eric Blake
On 12/07/2016 08:33 PM, Longpeng(Mike) wrote: > Libgcrypt and nettle support 3des-ede, so this patch add 3des-ede > support when using libgcrypt or nettle. > > Signed-off-by: Longpeng(Mike) > --- > +++ b/qapi/crypto.json > @@ -63,6 +63,7 @@ > # @aes-192: AES with 192 bit

Re: [Qemu-devel] qemu -netdev tun/tap support can't handle macvtap type devices

2016-12-08 Thread Peter Maydell
On 8 December 2016 at 01:52, Jason Wang wrote: > On 2016年12月07日 22:32, Peter Maydell wrote: >> It's particularly confusing in this case that fd= doesn't work with >> queues=, because the user isn't trying to pass multiple fds, just >> the one is fine. > > > Since fd can only

Re: [Qemu-devel] [PATCH for-2.9 V2 2/3] Add a new qmp command to start/stop replication

2016-12-08 Thread Eric Blake
On 12/08/2016 01:55 AM, Zhang Chen wrote: > We can call this qmp command to start/stop replication outside of qemu. > Like Xen colo need this function. > > Signed-off-by: Zhang Chen > --- > docs/qmp-commands.txt | 17 + > migration/colo.c | 22

Re: [Qemu-devel] [PATCH v3 0/3] Add HAX support

2016-12-08 Thread Stefan Weil
Am 08.12.2016 um 10:34 schrieb Yu Ning: > As a HAXM developer at Intel, I just want to come out and show our > support for Vincent's upstreaming effort. We'd love to see HAXM support > code land in upstream QEMU, and will do what is necessary to make that > happen. > > We've been working on the

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

2016-12-08 Thread Eric Blake
On 12/07/2016 08:28 PM, Gonglei (Arei) wrote: >> As far as I'm aware, other projects usually have a special memset >> variation for doing this. That is because compilers may choose to >> "optimize" memset(p, ...) + free(p) to just the free(p). Having a > > Actually, I googled this, but I didn't

Re: [Qemu-devel] [PATCH v3 0/3] Add HAX support

2016-12-08 Thread Yu Ning
As a HAXM developer at Intel, I just want to come out and show our support for Vincent's upstreaming effort. We'd love to see HAXM support code land in upstream QEMU, and will do what is necessary to make that happen. We've been working on the issues that Vincent raised and have made some

[Qemu-devel] [Bug 1626972] Re: QEMU memfd_create fallback mechanism change for security drivers

2016-12-08 Thread Antonio Messina
Hi all, I am facing this issue too, and although I can confirm the patch can be easily backported to Trusty (we run Mitaka on Trusty), some of our customers have VMs started with the old qemu and I cannot live migrate anymore or update qemu without stopping and starting the VM. Do you have any

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

2016-12-08 Thread Eric Blake
On 12/07/2016 08:33 PM, Gonglei (Arei) wrote: >> Sensitive information is sometimes also held in mlocked pages to prevent >> it being swapped to disk, but qemu in general is not currently taking >> that level of precaution (see also commit 8813800b). >> >>> >>> Let's zeroize the memory of

Re: [Qemu-devel] [RFC PATCH 4/6] iscsi: Add header-digest option

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:09PM +0100, Kevin Wolf wrote: > This was previously only available with -iscsi. Again, after this patch, > the -iscsi option only takes effect if an URL is given. New users are > supposed to use the new driver-specific option. > > Signed-off-by: Kevin Wolf

Re: [Qemu-devel] [RFC PATCH 1/6] iscsi: Split URL into individual options

2016-12-08 Thread Kevin Wolf
Am 08.12.2016 um 15:10 hat Daniel P. Berrange geschrieben: > On Thu, Dec 08, 2016 at 02:23:06PM +0100, Kevin Wolf wrote: > > This introduces a .bdrv_parse_filename handler for iscsi which parses an > > URL if given and translates it to individual options. > > > > Signed-off-by: Kevin Wolf

Re: [Qemu-devel] [RFC PATCH 1/6] iscsi: Split URL into individual options

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:06PM +0100, Kevin Wolf wrote: > This introduces a .bdrv_parse_filename handler for iscsi which parses an > URL if given and translates it to individual options. > > Signed-off-by: Kevin Wolf > --- > block/iscsi.c | 189 >

Re: [Qemu-devel] [kvm-unit-tests PATCH v14 3/5] arm: Add PMU test

2016-12-08 Thread Andrew Jones
On Tue, Dec 06, 2016 at 12:05:26PM -0600, Wei Huang wrote: > From: Christopher Covington > > Beginning with a simple sanity check of the control register, add > a unit test for the ARM Performance Monitors Unit (PMU). > > Signed-off-by: Christopher Covington

Re: [Qemu-devel] QEMU Advent Calendar

2016-12-08 Thread Martin Steigerwald
Am Donnerstag, 8. Dezember 2016, 09:14:06 CET schrieb Thomas Huth: > On 07.12.2016 23:15, Martin Steigerwald wrote: […] > > Considered > > > > http://aros.sourceforge.net/ > > > > or some of its distributions? > > Yes, just yesterday, somebody else suggested that to me. So far, I only > thought

Re: [Qemu-devel] [RFC PATCH 6/6] iscsi: Add blockdev-add support

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:11PM +0100, Kevin Wolf wrote: > This adds blockdev-add support for iscsi devices. > > Signed-off-by: Kevin Wolf > --- > block/iscsi.c| 14 ++ > qapi/block-core.json | 74 > > 2

Re: [Qemu-devel] [RFC PATCH 3/6] iscsi: Add initiator-name option

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:08PM +0100, Kevin Wolf wrote: > This was previously only available with -iscsi. Again, after this patch, > the -iscsi option only takes effect if an URL is given. New users are > supposed to use the new driver-specific option. > > Signed-off-by: Kevin Wolf

Re: [Qemu-devel] [RFC PATCH 0/6] iscsi: Add blockdev-add support

2016-12-08 Thread Kevin Wolf
Am 08.12.2016 um 14:55 hat Daniel P. Berrange geschrieben: > On Thu, Dec 08, 2016 at 02:23:05PM +0100, Kevin Wolf wrote: > > This adds blockdev-add support to the iscsi block driver. > > > > Note that this is only compile tested at this point. Jeff is going to > > take over from here and bring

Re: [Qemu-devel] [Nbd] [PATCH v3] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-08 Thread Alex Bligh
Wouter, >> +- `NBD_OPT_META_CONTEXT` (10) >> + >> +Return a list of `NBD_REP_META_CONTEXT` replies, one per context, >> +followed by an `NBD_REP_ACK`. If a server replies to such a request >> +with no error message, clients >> >> "*the* server" / "*the* cient" >> >> Perhaps only an

Re: [Qemu-devel] [RFC PATCH 5/6] iscsi: Add timeout option

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:10PM +0100, Kevin Wolf wrote: > This was previously only available with -iscsi. Again, after this patch, > the -iscsi option only takes effect if an URL is given. New users are > supposed to use the new driver-specific option. > > All -iscsi options have a

Re: [Qemu-devel] [PATCH v3] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-08 Thread Alex Bligh
> On 8 Dec 2016, at 06:58, Vladimir Sementsov-Ogievskiy > wrote: > > An idea: let's not use uppercase. Why to shout the namespace? 'base' and 'x-' > would be better I think. BASE and X- will provoke all user-defined namespaces > be in uppercase too and a lot of

Re: [Qemu-devel] [RFC PATCH 0/6] iscsi: Add blockdev-add support

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:05PM +0100, Kevin Wolf wrote: > This adds blockdev-add support to the iscsi block driver. > > Note that this is only compile tested at this point. Jeff is going to > take over from here and bring the series to a mergable state. > > Kevin Wolf (6): > iscsi: Split

Re: [Qemu-devel] [Qemu-block] Meeting notes on -blockdev, dynamic backend reconfiguration

2016-12-08 Thread Fam Zheng
On Thu, 12/08 13:46, Markus Armbruster wrote: > Fam Zheng writes: > > > On Wed, 12/07 10:48, Kevin Wolf wrote: > >> > If so I think there is no race to worry about, mirror-filter should go > >> > away only after a QMP command. > >> > >> Currently, a mirror job goes away

Re: [Qemu-devel] [RFC PATCH 2/6] iscsi: Handle -iscsi user/password in bdrv_parse_filename()

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:23:07PM +0100, Kevin Wolf wrote: > This splits the logic in the old parse_chap() function into a part that > parses the -iscsi options into the new driver-specific options, and > another part that actually applies those options (called apply_chap() > now). > > Note that

Re: [Qemu-devel] [Qemu-block] [PATCH 0/2] Remove need for -iscsi argument

2016-12-08 Thread Daniel P. Berrange
On Thu, Dec 08, 2016 at 02:12:04PM +0100, Kevin Wolf wrote: > Am 08.12.2016 um 13:41 hat Daniel P. Berrange geschrieben: > > This small series removes the need for using the -iscsi argument > > by directly supporting all the -iscsi options against the block > > device config eg instead of > > > >

[Qemu-devel] [RFC PATCH 5/6] iscsi: Add timeout option

2016-12-08 Thread Kevin Wolf
This was previously only available with -iscsi. Again, after this patch, the -iscsi option only takes effect if an URL is given. New users are supposed to use the new driver-specific option. All -iscsi options have a corresponding driver-specific option for the iscsi block driver now.

[Qemu-devel] [RFC PATCH 3/6] iscsi: Add initiator-name option

2016-12-08 Thread Kevin Wolf
This was previously only available with -iscsi. Again, after this patch, the -iscsi option only takes effect if an URL is given. New users are supposed to use the new driver-specific option. Signed-off-by: Kevin Wolf --- block/iscsi.c | 32 +++- 1

  1   2   >