Re: [Qemu-devel] [PATCH v2 08/11] migration: remove check against colo support

2017-07-18 Thread Zhang Chen
On 07/17/2017 04:26 PM, Peter Xu wrote: Since commit a15215f3 ("build: remove --enable-colo/--disable-colo"), colo is always supported. We don't need any colo_supported() now since it is always true. Removing any extra code that depends on it. CC: Paolo Bonzini CC: Hailiang Zhang Signed-off-

Re: [Qemu-devel] [PATCH v5 03/10] target/arm: optimize aarch64 rev16() using extract op

2017-07-18 Thread Aurelien Jarno
On 2017-07-17 20:25, Richard Henderson wrote: > On 07/17/2017 06:55 PM, Philippe Mathieu-Daudé wrote: > > Aurelien Jarno denoted this function could be implemented more effectively > > using > > the aarch32 rev16() pattern. > >[http://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg03180.h

Re: [Qemu-devel] [PATCH] pci: honor PCI_COMMAND_MEMORY

2017-07-18 Thread Dmitry Fleytman
On 17 Jul 2017, at 19:59 PM, Michael S. Tsirkin wrote: > > On Mon, Jul 17, 2017 at 03:48:03PM +0300, Dmitry Fleytman wrote: >> Am I understand correctly that there are no special cases for >> IDE controllers, i.e. bus master bit must be set by SW same >> way as for other PCI devices? > > Bus mas

Re: [Qemu-devel] Hacks for building on gcc 7 / Fedora 26

2017-07-18 Thread Daniel P. Berrange
On Mon, Jul 17, 2017 at 12:29:08PM -0500, Eric Blake wrote: > On 07/17/2017 11:46 AM, Dr. David Alan Gilbert wrote: > > >> +++ w/hw/usb/bus.c > >> @@ -407,8 +407,9 @@ void usb_register_companion(const char *masterbus, > >> USBPort *ports[], > >> void usb_port_location(USBPort *downstream, USBPort

Re: [Qemu-devel] [PATCH] Unified Socket Transport

2017-07-18 Thread Daniel P. Berrange
On Tue, Jul 18, 2017 at 07:06:27AM +0100, anton.iva...@cambridgegreys.com wrote: > From: Anton Ivanov > > 1. Creates a common recvmmsg backend for socket transports > 2. Migrates L2TPv3 to the new backend > 3. Adds gre socket transport - a vm can attach directly to GRE > 4. Adds raw socket transp

Re: [Qemu-devel] [PULL v1 0/7] MMIO Exec pull request

2017-07-18 Thread KONRAD Frederic
On 07/17/2017 07:27 PM, Edgar E. Iglesias wrote: On Mon, Jul 17, 2017 at 11:33 PM, Peter Maydell wrote: On 14 June 2017 at 18:45, Edgar E. Iglesias wrote: From: "Edgar E. Iglesias" Paolo suggested offline that we send a pull request for this series. Here it is, I've run it through my test

Re: [Qemu-devel] [Qemu-ppc] [PULL 06/21] spapr: Add a "no HPT" encoding to HTAB migration stream

2017-07-18 Thread Thomas Huth
On 18.07.2017 05:37, David Gibson wrote: > On Mon, Jul 17, 2017 at 09:54:45PM +0200, Laurent Vivier wrote: >> On 30/06/2017 12:46, David Gibson wrote: >>> From: Bharata B Rao >>> >>> Add a "no HPT" encoding (using value -1) to the HTAB migration >>> stream (in the place of HPT size) when the guest

Re: [Qemu-devel] [PATCH] Unified Socket Transport

2017-07-18 Thread Anton Ivanov
OK. I can split it into series before next submission. A. On 18/07/17 08:27, Daniel P. Berrange wrote: On Tue, Jul 18, 2017 at 07:06:27AM +0100, anton.iva...@cambridgegreys.com wrote: From: Anton Ivanov 1. Creates a common recvmmsg backend for socket transports 2. Migrates L2TPv3 to the ne

Re: [Qemu-devel] [PATCH v2 00/27] target/sh4 improvements

2017-07-18 Thread Aurelien Jarno
On 2017-07-06 16:20, Richard Henderson wrote: > This fixes two problems with atomic operations on sh4, > including an attempt at supporting the user-space atomics > technique used by most sh-linux-user binaries. > > Changes since v1: > * Rebase on Aurelien's recent sh4 patchset. > * Patch 3,5

Re: [Qemu-devel] [PATCH] spapr: make default PHB optionnal

2017-07-18 Thread Andrea Bolognani
On Wed, 2017-07-12 at 17:09 +0530, Shivaprasad G Bhat wrote: > Setting numa_node for emulated devices is the benefit for now. On x86, I > figured there is > no way to set the numa_node for the root controller and the emulated > devices sitting there > all have numa_node set to -1. Only the devices

[Qemu-devel] [PATCH] spapr/htab: fix savevm

2017-07-18 Thread Laurent Vivier
Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream") allows to migrate an empty HPT, but doesn't mark correctly the end of the migration stream. The end condition (value returned by htab_save_iterate() and htab_save_complete()) should be 1, whereas in 3a38429 they return 0.

Re: [Qemu-devel] [PATCH 2/2] s390x/cpumodel: z14 cpu models

2017-07-18 Thread Cornelia Huck
On Mon, 17 Jul 2017 18:58:26 +0200 Christian Borntraeger wrote: > From: "Jason J. Herne" > > This patch introduces the CPU model for z14, along with all base and > optional features. > > Signed-off-by: Jason J. Herne > Acked-by: Christian Borntraeger > Signed-off-by: Christian Borntraeger >

Re: [Qemu-devel] [PULL 0/7] Tracing patches

2017-07-18 Thread Peter Maydell
On 17 July 2017 at 13:11, Stefan Hajnoczi wrote: > The following changes since commit 4871b51b9241b10f4fd8e04bbb21577886795e25: > > vmgenid-test: use boot-sector infrastructure (2017-07-14 17:03:03 +0100) > > are available in the git repository at: > > git://github.com/stefanha/qemu.git tags/t

Re: [Qemu-devel] [PATCH 2/2] s390x/cpumodel: z14 cpu models

2017-07-18 Thread Christian Borntraeger
On 07/17/2017 06:58 PM, Christian Borntraeger wrote: > From: "Jason J. Herne" > > This patch introduces the CPU model for z14, along with all base and > optional features. > > Signed-off-by: Jason J. Herne > Acked-by: Christian Borntraeger > Signed-off-by: Christian Borntraeger > --- > targe

Re: [Qemu-devel] [PATCH v3 06/10] migration: check global params for validity

2017-07-18 Thread Juan Quintela
Peter Xu wrote: > Adding validity check for the migration parameters passed in via global > properties. > > Signed-off-by: Peter Xu Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH v3 10/10] migration: check global caps for validity

2017-07-18 Thread Juan Quintela
Peter Xu wrote: > Checks validity for all the capabilities that we enabled with command > line. > > Signed-off-by: Peter Xu Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH 1/2] vfio/ccw: allocate irq info with the right size

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 03:49:25 +0200 Dong Jia Shi wrote: > From: Jing Zhang > > When allocating memory for the vfio_irq_info parameter of the > VFIO_DEVICE_GET_IRQ_INFO ioctl, we used the wrong size. Let's > fix it by using the right size. > > Reviewed-by: Dong Jia Shi > Signed-off-by: Jing Zha

Re: [Qemu-devel] [PATCH v1 00/23] Current Travis Queue

2017-07-18 Thread Alex Bennée
Fam Zheng writes: > On Mon, 07/17 15:48, Alex Bennée wrote: >> Hi, >> >> This is the current status of the travis/next patch queue. The >> includes updates from Paolo to allow parallelism while testing in the >> docker environment. I've extended the travis image so we can actually >> run our tra

Re: [Qemu-devel] [PATCH 2/2] vfio/ccw: fix initialization of the Object DeviceState pointer in the common base-device

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 03:49:26 +0200 Dong Jia Shi wrote: > Commit 7da624e2 ("vfio: Test realized when using VFIOGroup.device_list > iterator") introduced a pointer to the Object DeviceState in the VFIO > common base-device and skiped non-realized devices as we iterate s/skiped/skipped/ I'll fix t

[Qemu-devel] [PATCH v6 1/8] tpm-backend: Remove unneeded member variable from backend class

2017-07-18 Thread Amarnath Valluri
TPMDriverOps inside TPMBackend is not required, as it is supposed to be a class member. The only possible reason for keeping in TPMBackend was, to get the backend type in tpm.c where dedicated backend api, tpm_backend_get_type() is present. Signed-off-by: Amarnath Valluri Reviewed-by: Marc-André

[Qemu-devel] [PATCH v6 4/8] tpm-backend: Made few interface methods optional

2017-07-18 Thread Amarnath Valluri
This allows backend implementations left optional interface methods. For mandatory methods assertion checks added. Took the opportunity to remove unused methods: tpm_backend_get_type() tpm_backend_get_desc() Signed-off-by: Amarnath Valluri Reviewed-by: Marc-André Lureau Reviewed-by: Stefan

[Qemu-devel] [PATCH v6 5/8] tmp backend: Add new api to read backend TpmInfo

2017-07-18 Thread Amarnath Valluri
TPM configuration options are backend implementation details and shall not be part of base TPMBackend object, and these shall not be accessed directly outside of the class, hence added a new interface method, get_tpm_options() to TPMDriverOps., which shall be implemented by the derived classes to r

[Qemu-devel] [PATCH v6 3/8] tpm-backend: Initialize and free data members in it's own methods

2017-07-18 Thread Amarnath Valluri
Initialize and free TPMBackend data members in it's own instance_init() and instance_finalize methods. Took the opportunity to remove unneeded destroy() method from TpmDriverOps interface as TPMBackend is a Qemu Object, we can use object_unref() inplace of tpm_backend_destroy() to free the backend

[Qemu-devel] [PATCH v6 2/8] tpm-backend: Move thread handling inside TPMBackend

2017-07-18 Thread Amarnath Valluri
Move thread handling inside TPMBackend, this way backend implementations need not to maintain their own thread life cycle, instead they needs to implement 'handle_request()' class method that always been called from a thread. This change made tpm_backend_int.h kind of useless, hence removed it. S

[Qemu-devel] [PATCH v6 0/8] Provide support for the software TPM emulator

2017-07-18 Thread Amarnath Valluri
Briefly, Theses set of patches introduces: - new TPM backend driver to support software TPM emulators(swtpm(1)). - and few supported fixes/enhancements/cleanup to existing tpm backend code. The similar idea was initiated earliar(2) by Stefan Berger(CCed) with slightly different approach, u

Re: [Qemu-devel] [PATCH 1/2] vfio/ccw: allocate irq info with the right size

2017-07-18 Thread Dong Jia Shi
* Cornelia Huck [2017-07-18 10:32:13 +0200]: > On Tue, 18 Jul 2017 03:49:25 +0200 > Dong Jia Shi wrote: > > > From: Jing Zhang > > > > When allocating memory for the vfio_irq_info parameter of the > > VFIO_DEVICE_GET_IRQ_INFO ioctl, we used the wrong size. Let's > > fix it by using the right

[Qemu-devel] [PATCH v6 7/8] tpm-passthrough: move reusable code to utils

2017-07-18 Thread Amarnath Valluri
Signed-off-by: Amarnath Valluri Reviewed-by: Stefan Berger --- hw/tpm/tpm_passthrough.c | 64 hw/tpm/tpm_util.c| 25 +++ hw/tpm/tpm_util.h| 4 +++ 3 files changed, 34 insertions(+), 59 deletions(-) diff --git a/hw

[Qemu-devel] [PATCH v6 6/8] tpm-backend: Move realloc_buffer() implementation to tpm-tis model

2017-07-18 Thread Amarnath Valluri
buffer reallocation is very unlikely to be backend specific. Hence move inside the tis. Signed-off-by: Amarnath Valluri Reviewed-by: Stefan Berger --- backends/tpm.c | 9 - hw/tpm/tpm_passthrough.c | 12 hw/tpm/tpm_tis.c | 14 -- i

Re: [Qemu-devel] [PATCH v5 8/8] tpm: Added support for TPM emulator

2017-07-18 Thread Amarnath Valluri
Thank Stefan for your comments, i sent the newer version of the patchset by addressing all your comments. - Amarnath On Wed, 2017-07-12 at 14:24 -0400, Stefan Berger wrote: > On 06/05/2017 08:45 AM, Amarnath Valluri wrote: > > This change introduces a new TPM backend driver that can communicate wi

[Qemu-devel] [PATCH v6 8/8] tpm: Added support for TPM emulator

2017-07-18 Thread Amarnath Valluri
This change introduces a new TPM backend driver that can communicate with swtpm(software TPM emulator) using unix domain socket interface. Swtpm uses two unix sockets, one for plain TPM commands and responses, and one for out-of-band control messages. The swtpm and associated tools can be found h

Re: [Qemu-devel] [PATCH v6 0/8] Provide support for the software TPM emulator

2017-07-18 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Type: series Message-id: 1500367747-8992-1-git-send-email-amarnath.vall...@intel.com Subject: [Qemu-devel] [PATCH v6 0/8] Provide support for the software TPM emulator === TEST SCRIPT BEGIN === #!/bin/bash # Testing

Re: [Qemu-devel] [PATCH 1/2] vfio/ccw: allocate irq info with the right size

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 16:49:43 +0800 Dong Jia Shi wrote: > * Cornelia Huck [2017-07-18 10:32:13 +0200]: > > > On Tue, 18 Jul 2017 03:49:25 +0200 > > Dong Jia Shi wrote: > > > > > From: Jing Zhang > > > > > > When allocating memory for the vfio_irq_info parameter of the > > > VFIO_DEVICE_GET

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

2017-07-18 Thread Thomas Huth
On 18.07.2017 01:17, Fam Zheng wrote: > On Mon, 07/17 11:41, Thomas Huth wrote: >> On 17.07.2017 08:35, Fam Zheng wrote: >>> Hi all, >>> >>> Today I've included a fourth type of the automatic patchew replies: FreeBSD. >>> >>> So far we have these tests running by patchew on each patch series: >>> >

[Qemu-devel] [PULL 2/2] s390x/cpumodel: z14 cpu models

2017-07-18 Thread Christian Borntraeger
From: "Jason J. Herne" This patch introduces the CPU model for z14, along with all base and optional features. Signed-off-by: Jason J. Herne Acked-by: Christian Borntraeger Signed-off-by: Christian Borntraeger --- target/s390x/cpu_models.c | 1 + target/s390x/gen-features.c | 37 +

[Qemu-devel] [PULL 0/2] two more s390 changes

2017-07-18 Thread Christian Borntraeger
4:17 +0100) are available in the git repository at: git://github.com/borntraeger/qemu.git tags/s390x-20170718 for you to fetch changes up to cc28a5949bb54df070c38ab505ee2c14385e53d0: s390x/cpumodel: z14 cpu models (2017-07-18 10:5

[Qemu-devel] [PULL 1/2] linux header sync against v4.13-rc1

2017-07-18 Thread Christian Borntraeger
Signed-off-by: Christian Borntraeger --- linux-headers/asm-x86/kvm_para.h | 1 + linux-headers/linux/kvm.h| 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/linux-headers/asm-x86/kvm_para.h b/linux-headers/asm-x86/kvm_para.h index 3a53979..cefa127 100644 --- a/linux-h

Re: [Qemu-devel] [Qemu-block] [PATCH 5/7] block: add throttle block filter driver

2017-07-18 Thread Manos Pitsidianakis
On Fri, Jul 14, 2017 at 12:45:19PM +0300, Manos Pitsidianakis wrote: +static BlockDriver bdrv_throttle = { +.format_name= "throttle", +.protocol_name = "throttle", +.instance_size = sizeof(ThrottleGroupMember), +

Re: [Qemu-devel] [PATCH 0/2] vfio-ccw bugfixs

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 03:49:24 +0200 Dong Jia Shi wrote: > Dear Conny, > > Here we got two bugfix patches for vfio-ccw: > - fix commit 7da624e2 which missed to initialize a new introduced > pointer for the vfio-ccw case > - fix a memory allocation that used a wrong size for an irq info > varia

Re: [Qemu-devel] [PATCH] spapr/htab: fix savevm

2017-07-18 Thread Thomas Huth
On 18.07.2017 10:05, Laurent Vivier wrote: > Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream") > allows to migrate an empty HPT, but doesn't mark correctly the > end of the migration stream. > > The end condition (value returned by htab_save_iterate() and > htab_save_comp

Re: [Qemu-devel] [PULL 00/14] Net patches

2017-07-18 Thread Peter Maydell
On 17 July 2017 at 13:21, Jason Wang wrote: > The following changes since commit 6632f6ff96f0537fc34cdc00c760656fc62e23c5: > > Merge remote-tracking branch > 'remotes/famz/tags/block-and-testing-pull-request' into staging (2017-07-17 > 11:46:36 +0100) > > are available in the git repository at

Re: [Qemu-devel] [PULL 2/2] s390x/cpumodel: z14 cpu models

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 11:12:24 +0200 Christian Borntraeger wrote: > From: "Jason J. Herne" > > This patch introduces the CPU model for z14, along with all base and > optional features. > > Signed-off-by: Jason J. Herne > Acked-by: Christian Borntraeger > Signed-off-by: Christian Borntraeger >

Re: [Qemu-devel] [RFC PATCH 7/8] VFIO: Add new IOCTL for IOMMU TLB invalidate propagation

2017-07-18 Thread Jean-Philippe Brucker
On 17/07/17 23:45, Alex Williamson wrote: [..] >>> >>> How does a user learn which model(s) are supported by the interface? >>> How do they learn which ops are supported? Perhaps a good use for one of >>> those >>> flag bits in the outer data structure is "probe". >> >> My initial plan to user

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

2017-07-18 Thread Peter Maydell
On 18 July 2017 at 10:11, Thomas Huth wrote: > Could the test be rewritten to provide a proper timeout handling > instead? Tests should clearly fail after a while instead of hanging > forever... > Or maybe we could add some magic that the troublesome tests are not > executed if a certain environme

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

2017-07-18 Thread Daniel P. Berrange
On Tue, Jul 18, 2017 at 10:37:30AM +0100, Peter Maydell wrote: > On 18 July 2017 at 10:11, Thomas Huth wrote: > > Could the test be rewritten to provide a proper timeout handling > > instead? Tests should clearly fail after a while instead of hanging > > forever... > > Or maybe we could add some m

Re: [Qemu-devel] [PATCH v6 12/18] crypto: introduce some common functions for af_alg backend

2017-07-18 Thread Daniel P. Berrange
On Fri, Jul 14, 2017 at 02:04:05PM -0400, longpeng.m...@gmail.com wrote: > From: "Longpeng(Mike)" > > The AF_ALG socket family is the userspace interface for linux > crypto API, this patch adds af_alg family support and some common > functions for af_alg backend. It'll be used by afalg-backend cr

Re: [Qemu-devel] [PATCH v6 00/18] crypto: add afalg-backend support

2017-07-18 Thread Daniel P. Berrange
On Fri, Jul 14, 2017 at 02:03:53PM -0400, longpeng.m...@gmail.com wrote: > From: "Longpeng(Mike)" > > The AF_ALG socket family is the userspace interface for linux > crypto API, users can use it to access hardware accelerators. > > This patchset adds a afalg-backend for qemu crypto subsystem. QE

Re: [Qemu-devel] [PATCH 06/29] migration/block: use QEMU_IS_ALIGNED macro

2017-07-18 Thread Juan Quintela
Philippe Mathieu-Daudé wrote: > Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci > > Signed-off-by: Philippe Mathieu-Daudé > --- > migration/block.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/migration/block.c b/migration/block.c > inde

Re: [Qemu-devel] [PATCH 11/29] s390x/sclp: use QEMU_IS_ALIGNED macro

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 03:09:47 -0300 Philippe Mathieu-Daudé wrote: > Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci > > Signed-off-by: Philippe Mathieu-Daudé > --- > hw/s390x/sclp.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/s

Re: [Qemu-devel] [PATCH 05/29] block: use QEMU_IS_ALIGNED macro

2017-07-18 Thread Juan Quintela
Philippe Mathieu-Daudé wrote: > Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH] spapr/htab: fix savevm

2017-07-18 Thread Laurent Vivier
On 18/07/2017 11:33, Thomas Huth wrote: > On 18.07.2017 10:05, Laurent Vivier wrote: >> Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream") >> allows to migrate an empty HPT, but doesn't mark correctly the >> end of the migration stream. >> >> The end condition (value returne

Re: [Qemu-devel] [PATCH 11/29] s390x/sclp: use QEMU_IS_ALIGNED macro

2017-07-18 Thread Cornelia Huck
On Tue, 18 Jul 2017 11:58:40 +0200 Cornelia Huck wrote: > On Tue, 18 Jul 2017 03:09:47 -0300 > Philippe Mathieu-Daudé wrote: > > > Applied using the Coccinelle semantic patch > > scripts/coccinelle/use_osdep.cocci > > > > Signed-off-by: Philippe Mathieu-Daudé > > --- > > hw/s390x/sclp.c | 4

Re: [Qemu-devel] Status and RFC of patchew testings on QEMU

2017-07-18 Thread Daniel P. Berrange
On Tue, Jul 18, 2017 at 07:28:50AM +0800, Fam Zheng wrote: > On Mon, 07/17 12:39, Kevin Wolf wrote: > > > > Q3: What other tests do maintainers want? Different hosts? Different > > > configure > > > combinations? > > > > Would running qemu-iotests (at least the 'quick' group) be possible or > >

[Qemu-devel] [PATCH 0/3] scripts/qemu.py small fixes

2017-07-18 Thread Amador Pahim
First commit fixes the 'is_running()' method, which is not working currently. Second commit includes the qemu command line and its output when there's an exception during the launch() and the VM is not started. Last commit renames self._args to self.args. The leading underscore represents that th

Re: [Qemu-devel] [PATCH v6 4/8] tpm-backend: Made few interface methods optional

2017-07-18 Thread Marc-André Lureau
Hi On Tue, Jul 18, 2017 at 10:48 AM Amarnath Valluri < amarnath.vall...@intel.com> wrote: > This allows backend implementations left optional interface methods. > For mandatory methods assertion checks added. > > Took the opportunity to remove unused methods: > tpm_backend_get_type() > tpm_ba

[Qemu-devel] [PATCH 2/3] qemu.py: include debug information on launch error

2017-07-18 Thread Amador Pahim
When launching a VM, if an exception happens and the VM is not initiated, it is useful to see the qemu command line that was executed and the output of that command. Before the patch: >>> VM = qemu.QEMUMachine('../aarch64-softmmu/qemu-system-aarch64') >>> VM.launch() Traceback (most r

[Qemu-devel] [PATCH 1/3] qemu.py: fix is_running()

2017-07-18 Thread Amador Pahim
Current implementation is broken. It does not really test if the child process is running. The Popen.returncode will only be set after by a poll(), wait() or communicate(). If the Popen fails to launch a VM, the Popen.returncode will not turn to None by itself. Instead of using Popen.returncode,

[Qemu-devel] [PATCH v2] spapr/htab: fix savevm

2017-07-18 Thread Laurent Vivier
Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream") allows to migrate an empty HPT, but doesn't mark correctly the end of the migration stream. The end condition (value returned by htab_save_iterate()) should be 1, whereas in 3a38429 it returns 0. The problem can be reprodu

[Qemu-devel] [PATCH 3/3] qemu.py: make 'args' public

2017-07-18 Thread Amador Pahim
Let's make args public so users can extend it without felling like abusing the internal API. Signed-off-by: Amador Pahim --- scripts/qemu.py | 12 ++-- tests/qemu-iotests/iotests.py | 18 +- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/sc

[Qemu-devel] [PULL v1 04/18] crypto: cipher: introduce qcrypto_cipher_ctx_new for builtin-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Extracts qcrypto_cipher_ctx_new() from qcrypto_cipher_new() for builtin-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/cipher-builtin.c | 101 ++-- 1 file change

[Qemu-devel] [PULL v1 02/18] crypto: cipher: introduce qcrypto_cipher_ctx_new for gcrypt-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Extracts qcrypto_cipher_ctx_new() from qcrypto_cipher_new() for gcrypt-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/cipher-gcrypt.c | 50 +- 1 file changed

[Qemu-devel] [PULL v1 01/18] crypto: cipher: introduce context free function

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Refactors the qcrypto_cipher_free(), splits it into two parts. One is gcrypt/nettle__cipher_free_ctx() to free the special context. This makes code more clear, what's more, it would be used by the later patch. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-of

[Qemu-devel] [PULL v1 00/18] Merge crypto 201/07/18

2017-07-18 Thread Daniel P. Berrange
The following changes since commit 6c6076662d98c068059983d411cb2a8987ba5670: Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2017-07-14 12:16:09 +0100) are available in the git repository at: git://github.com/berrange/qemu tags/pull-qcrypto-2017-07-18-1 for y

[Qemu-devel] [PULL v1 12/18] crypto: introduce some common functions for af_alg backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" The AF_ALG socket family is the userspace interface for linux crypto API, this patch adds af_alg family support and some common functions for af_alg backend. It'll be used by afalg-backend crypto latter. Signed-off-by: Longpeng(Mike) Maintainer: modified to report an err

[Qemu-devel] [PULL v1 03/18] crypto: cipher: introduce qcrypto_cipher_ctx_new for nettle-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Extracts qcrypto_cipher_ctx_new() from qcrypto_cipher_new() for nettle-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/cipher-nettle.c | 41 + 1 file changed, 29 inse

[Qemu-devel] [PULL v1 05/18] crypto: cipher: add cipher driver framework

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" 1) makes the public APIs in cipher-nettle/gcrypt/builtin static, and rename them with "nettle/gcrypt/builtin" prefix. 2) introduces cipher framework, including QCryptoCipherDriver and new public APIs. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-b

[Qemu-devel] [PULL v1 07/18] crypto: hmac: move crypto/hmac.h into include/crypto/

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Moves crypto/hmac.h into include/crypto/, likes cipher.h and hash.h Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- {crypto => include/crypto}/hmac.h | 0 1 file changed, 0 insertions(+), 0 d

[Qemu-devel] [PULL v1 06/18] crypto: hash: add hash driver framework

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" 1) makes the public APIs in hash-nettle/gcrypt/glib static, and rename them with "nettle/gcrypt/glib" prefix. 2) introduces hash framework, including QCryptoHashDriver and new public APIs. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P.

[Qemu-devel] [PULL v1 15/18] crypto: hmac: add af_alg-backend hmac support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Adds afalg-backend hmac support: introduces some private APIs firstly, and then intergrates them into qcrypto_hmac_afalg_driver. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/hash-afalg.c | 104 +--

[Qemu-devel] [PULL v1 09/18] crypto: hmac: introduce qcrypto_hmac_ctx_new for nettle-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Extracts qcrypto_hmac_ctx_new() from qcrypto_hmac_new() for nettle-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/hmac-nettle.c | 34 -- 1 file changed, 24 insertions(+), 10

[Qemu-devel] [PULL v1 08/18] crypto: hmac: introduce qcrypto_hmac_ctx_new for gcrypt-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" 1) Fix a handle-leak problem in qcrypto_hmac_new(), didn't free ctx->handle if gcry_mac_setkey fails. 2) Extracts qcrypto_hmac_ctx_new() from qcrypto_hmac_new() for gcrypt-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. B

Re: [Qemu-devel] [PATCH/s390-next 3/3] s390x/flic: migrate ais states

2017-07-18 Thread Juan Quintela
Halil Pasic wrote: > On 07/13/2017 05:11 PM, Dr. David Alan Gilbert wrote: >> * Christian Borntraeger (borntrae...@de.ibm.com) wrote: >>> On 07/13/2017 04:49 PM, Dr. David Alan Gilbert wrote: * Halil Pasic (pa...@linux.vnet.ibm.com) wrote: > @Dave: > There are a couple of questions I'm gonna

[Qemu-devel] [PULL v1 10/18] crypto: hmac: introduce qcrypto_hmac_ctx_new for glib-backend

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Extracts qcrypto_hmac_ctx_new() from qcrypto_hmac_new() for glib-backend impls. Reviewed-by: Gonglei Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/hmac-glib.c | 34 -- 1 file changed, 24 insertions(+), 10 del

[Qemu-devel] [PULL v1 11/18] crypto: hmac: add hmac driver framework

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" 1) makes the public APIs in hmac-nettle/gcrypt/glib static, and rename them with "nettle/gcrypt/glib" prefix. 2) introduces hmac framework, including QCryptoHmacDriver and new public APIs. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto

[Qemu-devel] [PULL v1 13/18] crypto: cipher: add afalg-backend cipher support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Adds afalg-backend cipher support: introduces some private APIs firstly, and then intergrates them into qcrypto_cipher_afalg_driver. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/Makefile.objs | 1 + crypto/afalgpriv.h| 9 ++ crypto

[Qemu-devel] [PULL v1 16/18] tests: crypto: add cipher speed benchmark support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Now we have two qcrypto backends, libiary-backend and afalg-backend, but which one is faster? This patch add a cipher speed benchmark, it helps us to measure the performance by using "make check-speed" or using "./tests/benchmark-crypto-cipher" directly. Signed-off-by: Lon

[Qemu-devel] [PULL v1 14/18] crypto: hash: add afalg-backend hash support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" Adds afalg-backend hash support: introduces some private APIs firstly, and then intergrates them into qcrypto_hash_afalg_driver. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- crypto/Makefile.objs | 1 + crypto/afalgpriv.h | 1 + crypto/hash-a

[Qemu-devel] [PULL v1 17/18] tests: crypto: add hash speed benchmark support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" This patch add a hash speed benchmark, it helps us to measure the performance by using "make check-speed" or using "./tests/benchmark-crypto-hash" directly. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- tests/.gitignore | 1 + tests/M

[Qemu-devel] [PULL v1 18/18] tests: crypto: add hmac speed benchmark support

2017-07-18 Thread Daniel P. Berrange
From: "Longpeng(Mike)" This patch add a hmac speed benchmark, it helps us to measure the performance by using "make check-speed" or using "./tests/benchmark-crypto-hmac" directly. Signed-off-by: Longpeng(Mike) Signed-off-by: Daniel P. Berrange --- tests/.gitignore | 1 + tests/M

Re: [Qemu-devel] [PATCH v6 5/8] tmp backend: Add new api to read backend TpmInfo

2017-07-18 Thread Marc-André Lureau
Hi On Tue, Jul 18, 2017 at 1:49 AM, Amarnath Valluri wrote: > TPM configuration options are backend implementation details and shall not be > part of base TPMBackend object, and these shall not be accessed directly > outside > of the class, hence added a new interface method, get_tpm_options() t

[Qemu-devel] [PULL v1 1/1] io: simplify qio_channel_attach_aio_context

2017-07-18 Thread Daniel P. Berrange
From: Paolo Bonzini If properly preceded by qio_channel_detach_aio_context, this function really has nothing to do except setting ioc->ctx. Signed-off-by: Paolo Bonzini Signed-off-by: Daniel P. Berrange --- io/channel.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --

[Qemu-devel] [PULL v1 0/1] Merge I/O 2017/07/18

2017-07-18 Thread Daniel P. Berrange
The following changes since commit 9964e96dccf7f7c936ee854a795415d19b60: Merge remote-tracking branch 'jasowang/tags/net-pull-request' into staging (2017-05-23 15:01:31 +0100) are available in the git repository at: git://github.com/berrange/qemu tags/pull-qio-2017-07-18-1 for you to f

Re: [Qemu-devel] [PULL 00/18] target-arm queue

2017-07-18 Thread Peter Maydell
On 17 July 2017 at 13:44, Peter Maydell wrote: > ARM queue for 2.10 soft freeze... > > thanks > -- PMM > > The following changes since commit 6632f6ff96f0537fc34cdc00c760656fc62e23c5: > > Merge remote-tracking branch > 'remotes/famz/tags/block-and-testing-pull-request' into staging (2017-07-17

Re: [Qemu-devel] [PATCH v2] spapr/htab: fix savevm

2017-07-18 Thread Thomas Huth
On 18.07.2017 12:16, Laurent Vivier wrote: > Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream") > allows to migrate an empty HPT, but doesn't mark correctly the > end of the migration stream. > > The end condition (value returned by htab_save_iterate()) > should be 1, where

[Qemu-devel] [PULL 01/32] docker: allow customizing Travis global_env variables

2017-07-18 Thread Alex Bennée
From: Paolo Bonzini This is useful so that we can do builds at higher than -j3 when running travis.py locally. Signed-off-by: Paolo Bonzini Signed-off-by: Alex Bennée diff --git a/tests/docker/travis.py b/tests/docker/travis.py index 8dcc964da4..703a7fde85 100755 --- a/tests/docker/travis.py

[Qemu-devel] [PULL 03/32] docker: include python-yaml in travis.docker

2017-07-18 Thread Alex Bennée
Although the upstream Travis images don't need this library our "travis-lite" scripts are written in python. This allows us to do: make docker-travis@travis J=10 and approximate a travis run on their default image. Signed-off-by: Alex Bennée Reviewed-by: Fam Zheng Signed-off-by: Alex Bennée

[Qemu-devel] [PULL 06/32] docker: add common packages to debian base

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/dockerfiles/debian9.docker b/tests/docker/dockerfiles/debian9.docker index c74f71283c..6b6bf41421 100644 --- a/tests/docker/dockerfiles/debian9.docker +++ b/tests/docker/doc

[Qemu-devel] [PULL 07/32] docker: debian/s390x no more in unstable, now available in Stretch

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index 72bb52ad7b..84dce49655 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -59,6 +59,7 @@

[Qemu-devel] [PULL 02/32] travis: move make -j flag out of script

2017-07-18 Thread Alex Bennée
From: Paolo Bonzini Because global environment variables can be overridden when .travis.yml is processed by the docker-travis target, the effect of this patch is that docker-travis now obeys the "J=n" option. Signed-off-by: Paolo Bonzini Signed-off-by: Alex Bennée diff --git a/.travis.yml b/.

[Qemu-devel] [PULL 10/32] docker: add debian/armel based on Stretch

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index 84dce49655..db66b6f802 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -56,6 +56,7 @@

[Qemu-devel] [PULL 12/32] docker: add debian/amd64 based on Stretch

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé By itself this doesn't add much to our coverage. However later patches will extend this image to include more bleeding edge libraries which are not yet widely available in distros. Signed-off-by: Philippe Mathieu-Daudé [AJB: extend commit msg] Signed-off-by: Alex Be

[Qemu-devel] [PULL 28/32] docker: add MXE (M cross environment) base image for MinGW-w64

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé see http://mxe.cc/ Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index 5a8283674a..e993e149e7 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.inclu

[Qemu-devel] [PULL 04/32] docker: add debug tools to travis.docker

2017-07-18 Thread Alex Bennée
When a test fails/hangs you don't want the hassle of getting the debug tools installed. Lets install them on our image by default so we can debug when we need to. Signed-off-by: Alex Bennée Reviewed-by: Fam Zheng Signed-off-by: Alex Bennée diff --git a/tests/docker/dockerfiles/travis.docker b

[Qemu-devel] [PULL 20/32] docker: add qemu:debian-jessie based on outdated jessie release

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/dockerfiles/debian8.docker b/tests/docker/dockerfiles/debian8.docker new file mode 100644 index 00..3d09b4b462 --- /dev/null +++ b/tests/docker/dockerfiles/debian8.d

[Qemu-devel] [PULL 09/32] docker: optimize debian9 base image

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/dockerfiles/debian9.docker b/tests/docker/dockerfiles/debian9.docker index 6b6bf41421..056e5389cc 100644 --- a/tests/docker/dockerfiles/debian9.docker +++ b/tests/docker/doc

[Qemu-devel] [PULL 29/32] shippable: add win32/64 targets

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/.shippable.yml b/.shippable.yml index 53b43b349f..dd4bbc84b1 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -7,6 +7,10 @@ env: matrix: - IMAGE=debian-amd64 TARGET_LIST

[Qemu-devel] [PULL 05/32] docker: rename debian stable -> 9 (Stretch)

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé We'll also want to support some older Debian combinations for architectures that didn't make the Debian 9 cut. Signed-off-by: Philippe Mathieu-Daudé [AJB: extend commit msg] Signed-off-by: Alex Bennée diff --git a/tests/docker/Makefile.include b/tests/docker/Makef

[Qemu-devel] [PULL 19/32] shippable: add x86_64 targets

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé [AJB: minor tweak to name] Signed-off-by: Alex Bennée diff --git a/.shippable.yml b/.shippable.yml index 189d193da5..1f92ace8f7 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -5,6 +5,8 @@ env: global: - LC_ALL=C

[Qemu-devel] [PULL 11/32] docker: add debian/ppc64el based on Stretch

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include index db66b6f802..553465ef6d 100644 --- a/tests/docker/Makefile.include +++ b/tests/docker/Makefile.include @@ -60,6 +60,7 @@

[Qemu-devel] [PULL 23/32] shippable: add powerpc target

2017-07-18 Thread Alex Bennée
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alex Bennée Signed-off-by: Alex Bennée diff --git a/.shippable.yml b/.shippable.yml index 1f92ace8f7..fa54df6cff 100644 --- a/.shippable.yml +++ b/.shippable.yml @@ -18,6 +18,8 @@ env: # mips64el-softmmu disa

Re: [Qemu-devel] [PATCH 29/29] tests/qapi: use ARRAY_SIZE macro

2017-07-18 Thread Marc-André Lureau
On Mon, Jul 17, 2017 at 11:10 PM, Philippe Mathieu-Daudé wrote: > Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Marc-André Lureau > --- > tests/test-string-output-visitor.c | 2 +- > 1 file changed, 1 ins

  1   2   3   4   5   6   >