Re: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread Stefan Weil
Am 07.09.2016 um 23:29 schrieb Peter Maydell: On 7 September 2016 at 20:13, Stefan Weil wrote: The old log messages are implemented by conditional compilation and not available by default. The new log messages can be enabled either by a command line option (-d unimp) or in

Re: [Qemu-devel] [PATCH v1 00/14] virtio-crypto: introduce framework and device emulation

2016-09-07 Thread no-reply
Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH v1 00/14] virtio-crypto: introduce framework and device emulation Type: series Message-id:

Re: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread Fam Zheng
On Wed, 09/07 22:50, Stefan Weil wrote: > Am 07.09.2016 um 22:17 schrieb no-re...@patchew.org: > > Hi, > > > > Your series failed automatic build test. Please find the testing commands > > and > > their output below. If you have docker installed, you can probably > > reproduce it > > locally. >

Re: [Qemu-devel] [virtio-dev] Re: [PATCH v1 11/14] virtio-crypto: add control queue handler

2016-09-07 Thread Gonglei (Arei)
> -Original Message- > From: virtio-...@lists.oasis-open.org [mailto:virtio-...@lists.oasis-open.org] > On Behalf Of Michael S. Tsirkin > Sent: Thursday, September 08, 2016 12:30 PM > To: Gonglei (Arei) > Subject: [virtio-dev] Re: [PATCH v1 11/14] virtio-crypto: add control queue >

Re: [Qemu-devel] [PATCH v1 11/14] virtio-crypto: add control queue handler

2016-09-07 Thread Michael S. Tsirkin
On Thu, Sep 08, 2016 at 11:42:33AM +0800, Gonglei wrote: > +info->cipher_alg = virtio_ldl_p(vdev, _para->algo); > +info->key_len = virtio_ldl_p(vdev, _para->keylen); > +info->direction = virtio_ldl_p(vdev, _para->op); There is no reason to use the virtio mixed endian-ness for modern

Re: [Qemu-devel] [RFC PATCH v0] spapr: Introduce sPAPRCPUCoreClass

2016-09-07 Thread Bharata B Rao
On Thu, Sep 08, 2016 at 11:51:32AM +1000, David Gibson wrote: > On Fri, Sep 02, 2016 at 03:06:38PM +0530, Bharata B Rao wrote: > > Each spapr cpu core type defines an instance_init routine which just > > populates the CPU class name. This can be done in the class_init > > commonly for all core

[Qemu-devel] [PATCH v1 04/14] crypto: add symetric algorithms support

2016-09-07 Thread Gonglei
This patch include three parts, the first is define the session structure and the second is the opertion structure, whose properties are needed to finish the symetric algorithms. The third part defines some function pointers. Signed-off-by: Gonglei --- crypto/crypto.c

[Qemu-devel] [PATCH v1 02/14] crypto: introduce crypto queue handler

2016-09-07 Thread Gonglei
crypto queue is a gallery used for executing crypto operation, which supports both synchronization and asynchronization. The thoughts stolen from net/queue.c Signed-off-by: Gonglei --- crypto/Makefile.objs | 1 + crypto/crypto-queue.c | 205

[Qemu-devel] [PATCH v1 08/14] virtio-crypto-pci: add virtio crypto pci support

2016-09-07 Thread Gonglei
This patch adds virtio-crypto-pci, which is the pci proxy for the virtio crypto device. Signed-off-by: Gonglei --- hw/virtio/Makefile.objs | 1 + hw/virtio/virtio-crypto-pci.c | 71 +++ hw/virtio/virtio-pci.h| 16

[Qemu-devel] [PATCH v1 09/14] virtio-crypto: add virtio crypto realization modle

2016-09-07 Thread Gonglei
Introduce the virtio crypto realization modle, I'll finish the core code in the following patches. The thoughts came from virtio net realization. Signed-off-by: Gonglei --- hw/core/qdev-properties-system.c | 86 ++ hw/virtio/Makefile.objs | 1 +

[Qemu-devel] [PATCH v1 11/14] virtio-crypto: add control queue handler

2016-09-07 Thread Gonglei
Realize the Symmetric algos session creation handler, including plain cipher and chainning algorithms. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 172 ++ 1 file changed, 172 insertions(+) diff --git

[Qemu-devel] [PATCH v1 10/14] virtio-crypto: set capacity of crypto legacy hardware

2016-09-07 Thread Gonglei
Set the crypto legacy hardware's capacity according to the backend peer cryptodev's capacity. We only support only one queue at present. Virtio crypto device is a kind of crypto legacy hardware. Signed-off-by: Gonglei --- crypto/crypto.c | 17

[Qemu-devel] [PATCH v1 14/14] virtio-crypto: add data virtqueue processing handler

2016-09-07 Thread Gonglei
At present, we only support cipher and algorithm chainning. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 364 ++ 1 file changed, 364 insertions(+) diff --git a/hw/virtio/virtio-crypto.c b/hw/virtio/virtio-crypto.c

[Qemu-devel] [PATCH v1 13/14] virtio-crypto: get correct input data address for each request

2016-09-07 Thread Gonglei
We MUST use the original hva for input data, but not the copyed address, otherwise the guest can't get the results. Fix a non-initial problem in an exception case as well. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 40 +---

[Qemu-devel] [PATCH v1 07/14] virtio-crypto: introduce virtio-crypto.h

2016-09-07 Thread Gonglei
This patch introduces the header of virtio crypto emulation. Signed-off-by: Gonglei --- include/hw/virtio/virtio-crypto.h | 84 ++ include/standard-headers/linux/virtio_crypto.h | 34 +-- 2 files changed, 101 insertions(+),

[Qemu-devel] [PATCH v1 00/14] virtio-crypto: introduce framework and device emulation

2016-09-07 Thread Gonglei
This patch series realize the framework and emulation of a new virtio crypto device, which is similar with virtio net device. - I introduce the cryptodev backend as the client of virtio crypto device which can be realized by different methods, such as cryptodev-linux in my series,

[Qemu-devel] [PATCH v1 12/14] virtio-crypto: add destroy session logic

2016-09-07 Thread Gonglei
Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 36 +--- 1 file changed, 33 insertions(+), 3 deletions(-) diff --git a/hw/virtio/virtio-crypto.c b/hw/virtio/virtio-crypto.c index b3ad974..0888725 100644 ---

[Qemu-devel] [PATCH v1 01/14] crypto: introduce cryptodev backend and crypto legacy hardware

2016-09-07 Thread Gonglei
cryptodev backend is used to realize the active work for virtual crypto device. CryptoLegacyHW device is a cryptographic hardware device seen by the virtual machine. The relationship between cryptodev backend and legacy hadware as follow: crypto_legacy_hw device (1)--->(n) cryptodev client

[Qemu-devel] [PATCH v1 05/14] crypto: add cryptodev-linux as a cryptodev backend

2016-09-07 Thread Gonglei
Cryptodev-linux is a device that allows access to Linux kernel cryptographic drivers; thus allowing of userspace applications to take advantage of hardware accelerators. Cryptodev-linux is implemented as a standalone module that requires no dependencies other than a stock linux kernel. The

[Qemu-devel] [PATCH v1 06/14] crypto: add internal handle logic layer

2016-09-07 Thread Gonglei
This patch add some transfering layer or functions, such as session creation, closing etc, which will be used by subsequent virtio crypto device patches. Signed-off-by: Gonglei --- crypto/crypto.c | 68 + 1 file

[Qemu-devel] [PATCH v1 03/14] crypto: add cryptoLegacyHW stuff

2016-09-07 Thread Gonglei
In previous patch, we define CryptoLegacyHWOptions in qapi-schema.json. we introduce the new/delete funciton about crypto legacy hardware device. Note: legacy cryptographic device support muliple queue. Signed-off-by: Gonglei --- crypto/crypto.c | 74

[Qemu-devel] KVM Forum 2016 videos now online

2016-09-07 Thread Pranith Kumar
FYI, The KVM Forum 2016 videos are now online on youtube. You can find them here: https://www.youtube.com/playlist?list=PLW3ep1uCIRfzQoZ0SlniYE8nz1ZRobjH7 Thanks, -- Pranith

Re: [Qemu-devel] [PATCH v7 2/4] vfio: VFIO driver for mediated devices

2016-09-07 Thread Jike Song
On 08/25/2016 05:22 PM, Dong Jia wrote: > On Thu, 25 Aug 2016 09:23:53 +0530 > Kirti Wankhede wrote: > > [...] > > Dear Kirti, > > I just rebased my vfio-ccw patches to this series. > With a little fix, which was pointed it out in my reply to the #3 > patch, it works

Re: [Qemu-devel] [PATCH v7 2/4] vfio: VFIO driver for mediated devices

2016-09-07 Thread Jike Song
On 08/26/2016 10:13 PM, Kirti Wankhede wrote: > > > On 8/25/2016 2:52 PM, Dong Jia wrote: >> On Thu, 25 Aug 2016 09:23:53 +0530 >> Kirti Wankhede wrote: >> >> [...] >> >> Dear Kirti, >> >> I just rebased my vfio-ccw patches to this series. >> With a little fix, which was

Re: [Qemu-devel] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic

2016-09-07 Thread David Gibson
On Tue, Sep 06, 2016 at 03:17:56PM +0200, Laurent Vivier wrote: > And add support for ppc64. > > Signed-off-by: Laurent Vivier Some of my coments may be obsoleted by the discussion with Greg. > --- > v2: > - remove useless parenthesis, inline > > tests/Makefile.include

Re: [Qemu-devel] [PATCH v4 1/3] qtest: replace strtoXX() by qemu_strtoXX()

2016-09-07 Thread David Gibson
On Tue, Sep 06, 2016 at 03:17:55PM +0200, Laurent Vivier wrote: > Signed-off-by: Laurent Vivier Reviewed-by: David Gibson > --- > v4: > - add this patch in the series to change all strtoXX() in qtest.c > > qtest.c | 49

Re: [Qemu-devel] [RFC PATCH v0] spapr: Introduce sPAPRCPUCoreClass

2016-09-07 Thread David Gibson
On Fri, Sep 02, 2016 at 03:06:38PM +0530, Bharata B Rao wrote: > Each spapr cpu core type defines an instance_init routine which just > populates the CPU class name. This can be done in the class_init > commonly for all core types which simplifies the registration. > This is inspired by how

Re: [Qemu-devel] add gcc compiler version to the browsable output in patchew?

2016-09-07 Thread Fam Zheng
On Wed, 09/07 16:27, Claudio Fontana wrote: > > Hi, > > it would be useful I think to know exactly which gcc version is running in > patchew, > it can help when trying to reproduce a problem which does not seem to trigger > locally. Hi, I have a work in progress series for QEMU to include

Re: [Qemu-devel] [RFC v4 23/28] atomic: introduce cmpxchg_bool

2016-09-07 Thread Richard Henderson
On 08/11/2016 08:24 AM, Alex Bennée wrote: +#define atomic_bool_cmpxchg(ptr, old, new) \ +({ \ +typeof(*ptr) _old = (old), _new = (new);\ Needs updating for

Re: [Qemu-devel] [PATCH v16 03/16] tests: Add ptimer tests

2016-09-07 Thread Dmitry Osipenko
On 08.09.2016 02:04, Peter Maydell wrote: > On 7 September 2016 at 23:32, Dmitry Osipenko wrote: >> Thank you for the suggestion, I'll s/Author:/Copyright (C) 2016,/. The >> consistency with other files is arguable, because that variant I choose by >> looking in several files

Re: [Qemu-devel] [PATCH v16 03/16] tests: Add ptimer tests

2016-09-07 Thread Peter Maydell
On 7 September 2016 at 23:32, Dmitry Osipenko wrote: > Thank you for the suggestion, I'll s/Author:/Copyright (C) 2016,/. The > consistency with other files is arguable, because that variant I choose by > looking in several files and picking the most popular. Most files with an

Re: [Qemu-devel] [PATCH v4 RFC] block/vxhs: Initial commit to add Veritas HyperScale VxHS block device support

2016-09-07 Thread ashish mittal
Hi Jeff, Thank you for the comments and corrections. I have submitted a v5 patch after incorporating most of your review comments. Please find details in-line. Thanks, Ashish On Tue, Aug 30, 2016 at 10:35 AM, Jeff Cody wrote: > > First-pass-over-the-code review: > > On Mon,

Re: [Qemu-devel] [PATCH v16 03/16] tests: Add ptimer tests

2016-09-07 Thread Dmitry Osipenko
On 08.09.2016 00:32, Eric Blake wrote: > On 09/07/2016 08:22 AM, Dmitry Osipenko wrote: >> Ptimer is a generic countdown timer helper that is used by many timer >> device models as well as by the QEMU core. Add QTests for the ptimer. >> >> Signed-off-by: Dmitry Osipenko >> --- >

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Alex Williamson
On Wed, 7 Sep 2016 23:36:28 +0530 Kirti Wankhede wrote: > On 9/7/2016 10:14 PM, Alex Williamson wrote: > > On Wed, 7 Sep 2016 21:45:31 +0530 > > Kirti Wankhede wrote: > > > >> On 9/7/2016 2:58 AM, Alex Williamson wrote: > >>> On Wed, 7 Sep 2016

Re: [Qemu-devel] [PATCH v3] Move max-bandwidth and downtime-limit into migrate_set_parameter for both hmp and qmp

2016-09-07 Thread Eric Blake
On 09/06/2016 08:39 AM, Ashijeet Acharya wrote: > Mark old-commands for speed and downtime as deprecated. > Move max-bandwidth and downtime-limit into migrate-set-parameters for > setting maximum migration speed and expected downtime limit parameters > respectively. > Change downtime units to

Re: [Qemu-devel] [PATCH v6 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-09-07 Thread Paulina Szubarczyk
On 09/07/2016 10:56 PM, Stefano Stabellini wrote: On Wed, 7 Sep 2016, Paulina Szubarczyk wrote: Copy data operated on during request from/to local buffers to/from the grant references. Before grant copy operation local buffers must be allocated what is done by calling ioreq_init_copy_buffers.

Re: [Qemu-devel] [PATCH v2 3/7] ppc/pnv: Add XSCOM infrastructure

2016-09-07 Thread Benjamin Herrenschmidt
On Wed, 2016-09-07 at 17:47 +0200, Cédric Le Goater wrote: >  > +static uint64_t pnv_lpc_xscom_mr_read(void *opaque, hwaddr addr, > unsigned size) > +{ > +XScomDevice *xd = XSCOM_DEVICE(opaque); > +uint64_t val = 0; > + > +pnv_lpc_xscom_read(xd, 0, xscom_to_pcb_addr(xd->chip_type, >

Re: [Qemu-devel] [PATCH v2 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-09-07 Thread Benjamin Herrenschmidt
On Wed, 2016-09-07 at 14:13 +0200, Cédric Le Goater wrote: > On 09/07/2016 01:08 PM, Benjamin Herrenschmidt wrote: > > > > On Wed, 2016-09-07 at 12:50 +0200, Cédric Le Goater wrote: > > > > > > This is a bit broader than Ben's patch which used > > > PPC_SEGMENT_64B.  > > > it's basically

Re: [Qemu-devel] [PATCH v16 03/16] tests: Add ptimer tests

2016-09-07 Thread Eric Blake
On 09/07/2016 08:22 AM, Dmitry Osipenko wrote: > Ptimer is a generic countdown timer helper that is used by many timer > device models as well as by the QEMU core. Add QTests for the ptimer. > > Signed-off-by: Dmitry Osipenko > --- > --- /dev/null > +++

Re: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread Peter Maydell
On 7 September 2016 at 20:13, Stefan Weil wrote: > The old log messages are implemented by conditional compilation > and not available by default. > > The new log messages can be enabled either by a command line option > (-d unimp) or in the QEMU monitor (log unimp). > >

[Qemu-devel] [PATCH v2] iscsi: Fix divide-by-zero regression on raw SG devices

2016-09-07 Thread Eric Blake
When qemu uses iscsi devices in sg mode, iscsilun->block_size is left at 0. Prior to commits cf081fca and similar, when block limits were tracked in sectors, this did not matter: various block limits were just left at 0. But when we started scaling by block size, this caused SIGFPE. Then, in a

[Qemu-devel] [PATCH 1/3] iscsi: extract find_iscsi_opts() helper function

2016-09-07 Thread Stefan Hajnoczi
Extract a function that looks up the -iscsi object to avoid duplicating this code. Signed-off-by: Stefan Hajnoczi --- block/iscsi.c | 82 ++- 1 file changed, 36 insertions(+), 46 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] 9pfs: add support for IO limits to 9p-local driver

2016-09-07 Thread Eric Blake
On 09/07/2016 07:39 AM, Pradeep wrote: > Uses throttling APIs to limit I/O bandwidth and number of operations on the > devices which use 9p-local driver. > > Signed-off-by: Pradeep > fsdev/file-op-9p.h | 3 + > fsdev/qemu-fsdev-opts.c | 52 + >

[Qemu-devel] [PATCH 3/3] iscsi: add iscsi= option

2016-09-07 Thread Stefan Hajnoczi
An iSCSI block driver instance is implicitly matched with an -iscsi object that has id=. If no -iscsi object with that id exists then the first -iscsi object is used. This patch adds a -drive ...,iscsi= option so the relationship between the block driver instance and an -iscsi object can be

[Qemu-devel] [PULL 15/18] target-alpha: Generate fence op

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-13-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- target-alpha/translate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[Qemu-devel] [PATCH 2/3] iscsi: simplify -iscsi QemuOpts fetching

2016-09-07 Thread Stefan Hajnoczi
Instead of looking up the -iscsi option multiple times, look it up once and pass the QemuOpts pointer around. This results in shorter, simpler code. Note that this patch also take advantage of the fact that qemu_opt_get(NULL, "foo") returns NULL. Therefore it's not necessary to check that opts

[Qemu-devel] [PULL 17/18] target-i386: Generate fences for x86

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-15-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- target-i386/translate.c | 8 1 file changed, 8 insertions(+) diff --git

[Qemu-devel] [PULL 13/18] tcg/tci: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Cc: Stefan Weil Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-11-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/tci/tcg-target.inc.c | 3 +++ tci.c

[Qemu-devel] [PATCH 0/3] iscsi: add -drive ..., iscsi= parameter

2016-09-07 Thread Stefan Hajnoczi
Add a parameter to associate an iSCSI block driver instance with an -iscsi object. This is more powerful than relying on the implicit target IQN naming convention since that only allows one -iscsi object per target. Stefan Hajnoczi (3): iscsi: extract find_iscsi_opts() helper function iscsi:

[Qemu-devel] [PULL 16/18] target-aarch64: Generate fences for aarch64

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-14-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- target-arm/translate-a64.c | 14 +- 1 file changed, 13 insertions(+), 1

[Qemu-devel] [PULL 14/18] target-arm: Generate fences in ARMv7 frontend

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-12-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- target-arm/translate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[Qemu-devel] [PULL 11/18] tcg/s390: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Cc: Alexander Graf Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-9-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/s390/tcg-target.inc.c | 11 +++ 1 file

[Qemu-devel] [PULL 18/18] tcg: Optimize fence instructions

2016-09-07 Thread Richard Henderson
From: Pranith Kumar This commit optimizes fence instructions. Two optimizations are currently implemented: (1) unnecessary duplicate fence instructions, and (2) merging weaker fences into a stronger fence. [rth: Merge tcg_optimize_mb back into tcg_optimize, so that we

[Qemu-devel] [PULL 07/18] tcg/arm: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Cc: Andrzej Zaborowski Cc: Peter Maydell Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-5-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson

[Qemu-devel] [PULL 12/18] tcg/sparc: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-10-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/sparc/tcg-target.inc.c | 13 + 1 file changed, 13 insertions(+)

[Qemu-devel] [PULL 09/18] tcg/mips: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-7-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/mips/tcg-target.inc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[Qemu-devel] [PULL 04/18] Introduce TCGOpcode for memory barrier

2016-09-07 Thread Richard Henderson
From: Pranith Kumar This commit introduces the TCGOpcode for memory barrier instruction. This opcode takes an argument which is the type of memory barrier which should be generated. Signed-off-by: Pranith Kumar Message-Id:

[Qemu-devel] [PULL 10/18] tcg/ppc: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-8-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/ppc/tcg-target.inc.c | 21 + 1 file changed, 21 insertions(+)

[Qemu-devel] [PULL 05/18] tcg/i386: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Generate a 'lock orl $0,0(%esp)' instruction for ordering instead of mfence which has similar ordering semantics. Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-3-bobby.pr...@gmail.com> Signed-off-by: Richard

[Qemu-devel] [PULL 06/18] tcg/aarch64: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Cc: Claudio Fontana Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-4-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target.inc.c | 24

[Qemu-devel] [PULL 00/18] tcg queued patches

2016-09-07 Thread Richard Henderson
Three unrelated patches and Pranith's memory barrier patch sets. The alignment patch is in support of Sparc's ldf instructions: 8 and 16-byte memory operations that require only 4-byte alignment. It's just as easy to support this kind of misalignment as any other. As mentioned in the commit, we'd

[Qemu-devel] [PULL 08/18] tcg/ia64: Add support for fence

2016-09-07 Thread Richard Henderson
From: Pranith Kumar Cc: Aurelien Jarno Signed-off-by: Pranith Kumar Message-Id: <20160714202026.9727-6-bobby.pr...@gmail.com> Signed-off-by: Richard Henderson --- tcg/ia64/tcg-target.inc.c | 5 + 1 file

[Qemu-devel] [PULL 01/18] tcg: Support arbitrary size + alignment

2016-09-07 Thread Richard Henderson
Previously we allowed fully unaligned operations, but not operations that are aligned but with less alignment than the operation size. In addition, arm32, ia64, mips, and sparc had been omitted from the previous overalignment patch, which would have led to that alignment being enforced.

[Qemu-devel] [PULL 03/18] cpu-exec: Check -dfilter for -d cpu

2016-09-07 Thread Richard Henderson
Signed-off-by: Richard Henderson --- cpu-exec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cpu-exec.c b/cpu-exec.c index 5d9710a..e7f851c 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -147,7 +147,8 @@ static inline tcg_target_ulong cpu_tb_exec(CPUState

[Qemu-devel] [PULL 02/18] tcg: Merge GETPC and GETRA

2016-09-07 Thread Richard Henderson
The return address argument to the softmmu template helpers was confused. In the legacy case, we wanted to indicate that there is no return address, and so passed in NULL. However, we then immediately subtracted GETPC_ADJ from NULL, resulting in a non-zero value, indicating the presence of an

Re: [Qemu-devel] [PATCH v6 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-09-07 Thread Stefano Stabellini
On Wed, 7 Sep 2016, Paulina Szubarczyk wrote: > Copy data operated on during request from/to local buffers to/from > the grant references. > > Before grant copy operation local buffers must be allocated what is > done by calling ioreq_init_copy_buffers. For the 'read' operation, > first, the qemu

Re: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread Stefan Weil
Am 07.09.2016 um 22:17 schrieb no-re...@patchew.org: Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. [...] GTESTER tests/test-string-output-visitor GTESTER

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-07 Thread Laine Stump
On 09/07/2016 03:39 PM, Marcel Apfelbaum wrote: On 09/07/2016 08:55 PM, Laine Stump wrote: On 09/07/2016 04:06 AM, Marcel Apfelbaum wrote: On 09/07/2016 09:21 AM, Gerd Hoffmann wrote: Hi, ports, if that's allowed). For example: - 1-32 ports needed: use root ports only - 33-64 ports

Re: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread no-reply
Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write Type: series Message-id:

Re: [Qemu-devel] seccomp missing calls in 2.7.0?

2016-09-07 Thread Brian Rak
--- src_clean/qemu-seccomp.c2016-09-02 11:34:22.0 -0400 +++ src/qemu-seccomp.c2016-09-06 11:28:23.189162653 -0400 @@ -65,6 +65,7 @@ { SCMP_SYS(prctl), 245 }, { SCMP_SYS(signalfd), 245 }, { SCMP_SYS(getrlimit), 245 }, +{ SCMP_SYS(getrusage), 245 }, {

Re: [Qemu-devel] [PATCH V2] virtio-pci: reduce modern_mem_bar size

2016-09-07 Thread Marcel Apfelbaum
On 09/07/2016 06:05 PM, Michael S. Tsirkin wrote: On Wed, Sep 07, 2016 at 06:02:25PM +0300, Marcel Apfelbaum wrote: Currently each VQ Notification Virtio Capability is allocated on a different page. The idea is to enable split drivers within guests, however there are no known plans to do that.

Re: [Qemu-devel] [PATCH v2 3/7] ppc/pnv: Add XSCOM infrastructure

2016-09-07 Thread Benjamin Herrenschmidt
On Wed, 2016-09-07 at 17:55 +0200, Cédric Le Goater wrote: > > yes. To hack my way through again, I have added a memory region under > the XScomDevice, but we should have a list like the ranges[] because of  > the PHB3 PCBQs. You have the parent region in the chip. Then each device can create

Re: [Qemu-devel] virtio ssd caching

2016-09-07 Thread Richard W.M. Jones
On Wed, Sep 07, 2016 at 03:36:38PM -0400, Stefan Hajnoczi wrote: > On Tue, Sep 06, 2016 at 12:44:39PM +0430, Aarian P. Aleahmad wrote: > > I am looking for a solution to cache a VM on a SSD without the guest > > knowing about it. We can do it by caching the disk on which the VM's disk > > is

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-07 Thread Marcel Apfelbaum
On 09/07/2016 08:55 PM, Laine Stump wrote: On 09/07/2016 04:06 AM, Marcel Apfelbaum wrote: On 09/07/2016 09:21 AM, Gerd Hoffmann wrote: Hi, ports, if that's allowed). For example: - 1-32 ports needed: use root ports only - 33-64 ports needed: use 31 root ports, and one switch with 2-32

Re: [Qemu-devel] virtio ssd caching

2016-09-07 Thread Stefan Hajnoczi
On Tue, Sep 06, 2016 at 12:44:39PM +0430, Aarian P. Aleahmad wrote: > I am looking for a solution to cache a VM on a SSD without the guest > knowing about it. We can do it by caching the disk on which the VM's disk > is lcoated with a SSD or we can do it through the hypervisor (i.e. Qemu, > KVM,

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-07 Thread Marcel Apfelbaum
On 09/07/2016 07:08 PM, Alex Williamson wrote: On Wed, 7 Sep 2016 11:06:45 +0300 Marcel Apfelbaum wrote: On 09/07/2016 09:21 AM, Gerd Hoffmann wrote: Hi, ports, if that's allowed). For example: - 1-32 ports needed: use root ports only - 33-64 ports needed: use 31

Re: [Qemu-devel] [PULL 00/26] Leak patches

2016-09-07 Thread Eric Blake
On 09/07/2016 01:57 PM, Marc-André Lureau wrote: >>> I'm afraid this doesn't build with our minimum glib version: >>> >>> /Users/pm215/src/qemu-for-merges/tests/libqtest.c:771:42: error: expected >>> ')' >>> (GTestFixtureFunc) fn, (GTestFixtureFunc) >>>

[Qemu-devel] [PATCH] memory: Use standard log messages for unassigned memory read / write

2016-09-07 Thread Stefan Weil
The old log messages are implemented by conditional compilation and not available by default. The new log messages can be enabled either by a command line option (-d unimp) or in the QEMU monitor (log unimp). Signed-off-by: Stefan Weil --- The new code is very useful when

[Qemu-devel] [PATCH v5 RFC] block/vxhs: Initial commit to add Veritas HyperScale VxHS block device support

2016-09-07 Thread Ashish Mittal
This patch adds support for a new block device type called "vxhs". Source code for the library that this code loads can be downloaded from: https://github.com/MittalAshish/libqnio.git Sample command line using JSON syntax: ./qemu-system-x86_64 -name instance-0008 -S -vnc 0.0.0.0:0 -k en-us

Re: [Qemu-devel] [PULL 00/26] Leak patches

2016-09-07 Thread Marc-André Lureau
Hi - Original Message - > Hi > > - Original Message - > > On 6 September 2016 at 13:26, Marc-André Lureau > > wrote: > > > The following changes since commit > > > 1fd66154fdf8305e6668a96046a22b863b4d7320: > > > > > > Merge remote-tracking branch

Re: [Qemu-devel] [PATCH v5 02/20] qapi.py: add a simple #ifdef conditional

2016-09-07 Thread Eric Blake
On 09/07/2016 08:40 AM, Markus Armbruster wrote: >> Yet another option is to add 'ifdef' keys to the definitions >> themselves, i.e. >> >> { 'command': 'query-gic-capabilities', 'returns': ['GICCapability'], >> 'ifdef': 'TARGET_ARM' } Seems like it might be useful;

Re: [Qemu-devel] [PATCH v3] qqq: module for synchronizing with a simulation

2016-09-07 Thread Greg Kurz
On Wed, 7 Sep 2016 13:53:18 -0400 "James J. Nutaro" wrote: > This patch adds an interface for pacing the execution of > QEMU to match its rtc with an external simulation clock. > Its aim is to permit QEMU to be used as a module within a > larger simulation system. This

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Neo Jia
On Wed, Sep 07, 2016 at 07:27:19PM +0100, Daniel P. Berrange wrote: > On Wed, Sep 07, 2016 at 11:17:39AM -0700, Neo Jia wrote: > > On Wed, Sep 07, 2016 at 10:44:56AM -0600, Alex Williamson wrote: > > > On Wed, 7 Sep 2016 21:45:31 +0530 > > > Kirti Wankhede wrote: > > > > >

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Daniel P. Berrange
On Wed, Sep 07, 2016 at 11:17:39AM -0700, Neo Jia wrote: > On Wed, Sep 07, 2016 at 10:44:56AM -0600, Alex Williamson wrote: > > On Wed, 7 Sep 2016 21:45:31 +0530 > > Kirti Wankhede wrote: > > > > > To hot-plug mdev device to a domain in which there is already a mdev > > >

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-07 Thread Laine Stump
On 09/07/2016 03:04 AM, Gerd Hoffmann wrote: Hi, Side note for usb: In practice you don't want to use the tons of uhci/ehci controllers present in the original q35 but plug xhci into one of the pcie root ports instead (unless your guest doesn't support xhci). I've wondered about that

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Neo Jia
On Wed, Sep 07, 2016 at 10:44:56AM -0600, Alex Williamson wrote: > On Wed, 7 Sep 2016 21:45:31 +0530 > Kirti Wankhede wrote: > > > To hot-plug mdev device to a domain in which there is already a mdev > > device assigned, mdev device should be created with same group number

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Kirti Wankhede
On 9/7/2016 10:14 PM, Alex Williamson wrote: > On Wed, 7 Sep 2016 21:45:31 +0530 > Kirti Wankhede wrote: > >> On 9/7/2016 2:58 AM, Alex Williamson wrote: >>> On Wed, 7 Sep 2016 01:05:11 +0530 >>> Kirti Wankhede wrote: >>> On 9/6/2016 11:10

Re: [Qemu-devel] [PATCH 3/3] virtio-balloon: fix stats vq migration

2016-09-07 Thread Stefan Hajnoczi
On Wed, Sep 07, 2016 at 05:20:49PM +0200, Ladi Prosek wrote: > The statistics virtqueue is not migrated properly because virtio-balloon > does not include s->stats_vq_elem in the migration stream. > > After migration the statistics virtqueue hangs because the host never > completes the last

Re: [Qemu-devel] [PATCH 0/3] virtio-balloon: stats vq fixes

2016-09-07 Thread Stefan Hajnoczi
On Wed, Sep 07, 2016 at 05:20:46PM +0200, Ladi Prosek wrote: > It is not > necessary to check balloon_stats_supported. Pushing buffers without > negotiating stats support would be a driver bug. The common codepath > does not check it either. This part made me nervous because QEMU is never allowed

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-07 Thread Laine Stump
On 09/07/2016 04:06 AM, Marcel Apfelbaum wrote: On 09/07/2016 09:21 AM, Gerd Hoffmann wrote: Hi, ports, if that's allowed). For example: - 1-32 ports needed: use root ports only - 33-64 ports needed: use 31 root ports, and one switch with 2-32 downstream ports I expect you rarely need

[Qemu-devel] [PATCH v3] qqq: module for synchronizing with a simulation

2016-09-07 Thread James J. Nutaro
This patch adds an interface for pacing the execution of QEMU to match its rtc with an external simulation clock. Its aim is to permit QEMU to be used as a module within a larger simulation system. This revision (v3) corrects formatting errors detected by Patchew. Signed-off-by: James J. Nutaro

Re: [Qemu-devel] [PATCH 1/3] virtio-balloon: discard virtqueue element on reset

2016-09-07 Thread Stefan Hajnoczi
On Wed, Sep 07, 2016 at 05:20:47PM +0200, Ladi Prosek wrote: > The one pending element is being freed but not discarded on device > reset, which causes svq->inuse to creep up, eventually hitting the > "Virtqueue size exceeded" error. > > Properly discarding the element on device reset makes sure

Re: [Qemu-devel] [PATCH v4 00/14] tcg: Add support for fence generation

2016-09-07 Thread Richard Henderson
On 07/14/2016 01:20 PM, Pranith Kumar wrote: Pranith Kumar (14): Introduce TCGOpcode for memory barrier tcg/i386: Add support for fence tcg/aarch64: Add support for fence tcg/arm: Add support for fence tcg/ia64: Add support for fence tcg/mips: Add support for fence tcg/ppc: Add

[Qemu-devel] [PATCH 2/2] virtio-pci: error out when both legacy and modern modes are disabled

2016-09-07 Thread Greg Kurz
From: Greg Kurz Without presuming if we got there because of a user mistake or some more subtle bug in the tooling, it really does not make sense to implement a non-functional device. Signed-off-by: Greg Kurz Reviewed-by: Marcel Apfelbaum

[Qemu-devel] [PATCH 1/2] virtio-9p: print error message and exit instead of BUG_ON()

2016-09-07 Thread Greg Kurz
Calling assert() really makes sense when hitting a genuine bug, which calls for a fix in QEMU. However, when something goes wrong because the guest sends a malformed message, it is better to write down a more meaningul error message and exit. Signed-off-by: Greg Kurz ---

[Qemu-devel] [PATCH 0/2] virtio: error report fixes in 9P and PCI

2016-09-07 Thread Greg Kurz
This series brings error reporting to virtio 9P in case a malformed element is returned by virtqueue_pop(). While here, I also re-post a patch for virtio PCI that fell through the cracks and missed the 2.7 merge window. --- Greg Kurz (2): virtio-9p: print error message and exit instead of

Re: [Qemu-devel] [PATCH v6 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-09-07 Thread Anthony PERARD
On Wed, Sep 07, 2016 at 12:41:00PM +0200, Paulina Szubarczyk wrote: > Copy data operated on during request from/to local buffers to/from > the grant references. > > Before grant copy operation local buffers must be allocated what is > done by calling ioreq_init_copy_buffers. For the 'read'

Re: [Qemu-devel] [RFC v2 0/4] adding mdev bus and vfio support

2016-09-07 Thread Alex Williamson
On Wed, 07 Sep 2016 14:42:58 +0800 Jike Song wrote: > On 09/07/2016 11:38 AM, Neo Jia wrote: > > On Wed, Sep 07, 2016 at 10:22:26AM +0800, Jike Song wrote: > >> On 09/02/2016 11:03 PM, Alex Williamson wrote: > >>> On Fri, 2 Sep 2016 16:16:08 +0800 > >>> Jike Song

Re: [Qemu-devel] [PATCH v2] target-i386: Use struct X86XSaveArea in fpu_helper.c

2016-09-07 Thread Richard Henderson
On 07/06/2016 01:35 PM, Richard Henderson wrote: This avoids a double hand-full of magic numbers in the xsave and xrstor helper functions. Signed-off-by: Richard Henderson --- target-i386/cpu.c| 7 ++- target-i386/cpu.h| 10 + target-i386/fpu_helper.c

Re: [Qemu-devel] [PATCH v7 0/4] Add Mediated device support

2016-09-07 Thread Alex Williamson
On Wed, 7 Sep 2016 21:45:31 +0530 Kirti Wankhede wrote: > On 9/7/2016 2:58 AM, Alex Williamson wrote: > > On Wed, 7 Sep 2016 01:05:11 +0530 > > Kirti Wankhede wrote: > > > >> On 9/6/2016 11:10 PM, Alex Williamson wrote: > >>> On Sat, 3 Sep 2016

  1   2   3   >