[Qemu-devel] [Bug 1739413] Re: Hotplugged vcpu does not guarantee cpu compat mode(power8) on power9 host

2018-01-05 Thread Satheesh Rajendran
Upstream patch from David Gibson @ https://lists.nongnu.org/archive/html /qemu-devel/2018-01/msg00587.html ** Changed in: qemu Status: New => In Progress -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2018-01-05 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/657006 Title: arm v7M -

[Qemu-devel] [PATCH v8 22/23] tcg/i386: Add vector operations

2018-01-05 Thread Richard Henderson
The x86 vector instruction set is extremely irregular. With newer editions, Intel has filled in some of the blanks. However, we don't get many 64-bit operations until SSE4.2, introduced in 2009. The subsequent edition was for AVX1, introduced in 2011, which added three-operand addressing, and

[Qemu-devel] [PATCH v8 12/23] tcg/optimize: Handle vector opcodes during optimize

2018-01-05 Thread Richard Henderson
Trivial move and constant propagation. Some identity and constant function folding, but nothing that requires knowledge of the size of the vector element. Signed-off-by: Richard Henderson --- tcg/optimize.c | 141

[Qemu-devel] [PATCH v8 21/23] target/arm: Use vector infrastructure for aa64 widening shifts

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index 48caba3d9f..4f15e58556 100644 ---

[Qemu-devel] [PATCH v8 11/23] tcg: Add generic vector helpers with a scalar immediate operand

2018-01-05 Thread Richard Henderson
We already have immediate shifts. Add addition, multiplication, and logical operations with an immediate. Subtraction can thus be done with negation of the constant. Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 14 tcg/tcg-op-gvec.h

[Qemu-devel] [PATCH v8 18/23] target/arm: Use vector infrastructure for aa64 constant shifts

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 386 ++--- 1 file changed, 329 insertions(+), 57 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index 8769b4505a..d8bb3bbb25

[Qemu-devel] [PATCH v8 23/23] tcg/aarch64: Add vector operations

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target.h | 30 +- tcg/aarch64/tcg-target.opc.h | 3 + tcg/aarch64/tcg-target.inc.c | 674 --- 3 files changed, 660 insertions(+), 47 deletions(-) create mode 100644

[Qemu-devel] [PATCH v8 19/23] target/arm: Use vector infrastructure for aa64 compares

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 96 ++ 1 file changed, 62 insertions(+), 34 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index d8bb3bbb25..44e44cc9f2

[Qemu-devel] [PATCH v8 06/23] tcg: Add generic vector ops for constant shifts

2018-01-05 Thread Richard Henderson
Opcodes are added for scalar and vector shifts, but considering the varied semantics of these do not expose them to the front ends. Do go ahead and provide them in case they are needed for backend expansion. Signed-off-by: Richard Henderson ---

[Qemu-devel] [PATCH v8 16/23] target/arm: Use vector infrastructure for aa64 dup/movi

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 83 +++--- 1 file changed, 34 insertions(+), 49 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index bc14c28e71..55a4902fc2

[Qemu-devel] [PATCH v8 20/23] target/arm: Use vector infrastructure for aa64 multiplies

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 171 - 1 file changed, 138 insertions(+), 33 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index 44e44cc9f2..48caba3d9f

[Qemu-devel] [PATCH v8 14/23] target/arm: Use vector infrastructure for aa64 add/sub/logic

2018-01-05 Thread Richard Henderson
Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 207 + 1 file changed, 134 insertions(+), 73 deletions(-) diff --git a/target/arm/translate-a64.c

[Qemu-devel] [PATCH v8 07/23] tcg: Add generic vector ops for comparisons

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 30 +++ tcg/tcg-op-gvec.h| 4 + tcg/tcg-op.h | 3 + tcg/tcg-opc.h| 2 + tcg/tcg.h| 1 + accel/tcg/tcg-runtime-gvec.c |

[Qemu-devel] [PATCH v8 15/23] target/arm: Use vector infrastructure for aa64 mov/not/neg

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 43 ++- 1 file changed, 38 insertions(+), 5 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index 572af456d1..bc14c28e71

[Qemu-devel] [PATCH v8 08/23] tcg: Add generic vector ops for multiplication

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 5 + tcg/tcg-op-gvec.h| 2 ++ tcg/tcg-op.h | 1 + tcg/tcg-opc.h| 1 + tcg/tcg.h| 1 + accel/tcg/tcg-runtime-gvec.c | 44

[Qemu-devel] [PATCH v8 17/23] target/arm: Use vector infrastructure for aa64 zip/uzp/trn/xtn

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 103 +++-- 1 file changed, 35 insertions(+), 68 deletions(-) diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c index 55a4902fc2..8769b4505a

[Qemu-devel] [PATCH v8 04/23] tcg: Add generic vector expanders

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- Makefile.target |2 +- accel/tcg/tcg-runtime.h | 29 + tcg/tcg-gvec-desc.h | 49 ++ tcg/tcg-op-gvec.h| 194 ++ tcg/tcg-op.h |1 + tcg/tcg-opc.h

[Qemu-devel] [PATCH v8 10/23] tcg: Add generic helpers for saturating arithmetic

2018-01-05 Thread Richard Henderson
No vector ops as yet. SSE only has direct support for 8- and 16-bit saturation; handling 32- and 64-bit saturation is much more expensive. Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 20 tcg/tcg-op-gvec.h| 10 ++

[Qemu-devel] [PATCH v8 03/23] tcg: Standardize integral arguments to expanders

2018-01-05 Thread Richard Henderson
Some functions use intN_t arguments, some use uintN_t, some just used "unsigned". To aid putting function pointers in tables, we need consistency. Signed-off-by: Richard Henderson --- tcg/tcg-op.h | 16 tcg/tcg-op.c | 42

[Qemu-devel] [PATCH v8 02/23] tcg: Add types and basic operations for host vectors

2018-01-05 Thread Richard Henderson
Nothing uses or enables them yet. Signed-off-by: Richard Henderson --- Makefile.target | 4 +- tcg/tcg-op.h | 30 + tcg/tcg-opc.h| 26 tcg/tcg.h| 56 + tcg/tcg-op-vec.c | 362

[Qemu-devel] [PATCH v8 13/23] target/arm: Align vector registers

2018-01-05 Thread Richard Henderson
Reviewed-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Richard Henderson --- target/arm/cpu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/arm/cpu.h b/target/arm/cpu.h

[Qemu-devel] [PATCH v8 09/23] tcg: Add generic vector ops for extension

2018-01-05 Thread Richard Henderson
Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 8 +++ tcg/tcg-op-gvec.h| 9 +++ tcg/tcg-op.h | 5 ++ tcg/tcg-opc.h| 5 ++ tcg/tcg.h| 2 + accel/tcg/tcg-runtime-gvec.c |

[Qemu-devel] [PATCH v8 05/23] tcg: Add generic vector ops for interleave

2018-01-05 Thread Richard Henderson
Includes zip, unzip, and transform. Signed-off-by: Richard Henderson --- accel/tcg/tcg-runtime.h | 15 ++ tcg/tcg-op-gvec.h| 17 +++ tcg/tcg-op.h | 6 + tcg/tcg-opc.h| 7 + tcg/tcg.h| 3 +

[Qemu-devel] [PATCH v8 00/23] tcg: generic vector operations

2018-01-05 Thread Richard Henderson
Minor changes since v7: * Rearranged the patches into groups for readability. * Squished the tcg/i386 patches. * Fixed an encoding bug for vmovd xmm, reg (Kirill). * Fixed expansion bugs for i386 sar, trn. Minor changes since v6: * Two bugs fixed pointed out by Kirill. * Allow for 4

[Qemu-devel] [PATCH v8 01/23] tcg: Allow multiple word entries into the constant pool

2018-01-05 Thread Richard Henderson
This will be required for storing vector constants. Signed-off-by: Richard Henderson --- tcg/tcg-pool.inc.c | 115 +++-- 1 file changed, 93 insertions(+), 22 deletions(-) diff --git a/tcg/tcg-pool.inc.c

Re: [Qemu-devel] [PATCH 6/6] hw/sd: move sdcard legacy API to "hw/sd/sdcard_legacy.h"

2018-01-05 Thread Alistair Francis
On Wed, Jan 3, 2018 at 8:24 AM, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé > --- > roughly 2 users left. Exciting! Reviewed-by: Alistair Francis Alistair > > include/hw/sd/sd.h| 17

Re: [Qemu-devel] [PATCH 2/6] hw/sd/milkymist-memcard: split realize() out of SysBusDevice init()

2018-01-05 Thread Alistair Francis
On Wed, Jan 3, 2018 at 8:23 AM, Philippe Mathieu-Daudé wrote: > Create the SDCard in the realize() function. > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis Alistair > --- > hw/sd/milkymist-memcard.c | 24

Re: [Qemu-devel] MTTCG External Halt

2018-01-05 Thread Alistair Francis
On Thu, Jan 4, 2018 at 3:08 AM, Alex Bennée wrote: > > Alistair Francis writes: > >> Hey guys, I'm super stuck with an ugly MTTCG issue and was wondering >> if anyone had any ideas. >> >> In the Xilinx fork of QEMU (based on 2.11) we have a way for

Re: [Qemu-devel] [RFC PATCH 4/6] arm/xlnx-zynq: use FDT names for the Cadence UART

2018-01-05 Thread Alistair Francis
On Thu, Jan 4, 2018 at 6:40 AM, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis Alistair > --- > hw/arm/xilinx_zynq.c | 1 + > hw/arm/xlnx-zynqmp.c | 2 +- > 2 files changed, 2

Re: [Qemu-devel] [RFC PATCH 5/6] hw/net/cadence_gem: add FDT names as alias

2018-01-05 Thread Alistair Francis
On Thu, Jan 4, 2018 at 6:40 AM, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis Alistair > --- > hw/net/cadence_gem.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff

Re: [Qemu-devel] [RFC PATCH 6/6] hw/arm/xlnx-zynq: use FDT names for the Cadence GEM

2018-01-05 Thread Alistair Francis
On Thu, Jan 4, 2018 at 6:40 AM, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis Alistair > --- > hw/arm/xilinx_zynq.c | 1 + > hw/arm/xlnx-zynqmp.c | 2 +- > 2 files changed, 2

Re: [Qemu-devel] [RFC PATCH 3/6] hw/char/cadence_uart: add FDT aliases

2018-01-05 Thread Alistair Francis
On Thu, Jan 4, 2018 at 6:40 AM, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis Alistair > --- > hw/char/cadence_uart.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff

[Qemu-devel] [PULL 01/12] virtio-9p: move unrealize/realize after virtio_9p_transport definition

2018-01-05 Thread Greg Kurz
And drop the now useless forward declaration of virtio_9p_transport. Signed-off-by: Greg Kurz --- hw/9pfs/virtio-9p-device.c | 60 ++ 1 file changed, 29 insertions(+), 31 deletions(-) diff --git a/hw/9pfs/virtio-9p-device.c

[Qemu-devel] [PULL 07/12] 9pfs: fix error path in pdu_submit()

2018-01-05 Thread Greg Kurz
If we receive an unsupported request id, we first decide to return -ENOTSUPP to the client, but since the request id causes is_read_only_op() to return false, we change the error to be -EROFS if the fsdev is read-only. This doesn't make sense since we don't know what the client asked for. This

[Qemu-devel] [PULL 08/12] 9pfs: make pdu_marshal() and pdu_unmarshal() static functions

2018-01-05 Thread Greg Kurz
They're only used by the 9p core code. Signed-off-by: Greg Kurz Reviewed-by: Eric Blake --- hw/9pfs/9p.c | 4 ++-- hw/9pfs/9p.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c index 558efb41defd..1e4ebbe57687

[Qemu-devel] [PULL 12/12] fsdev: improve error handling of backend init

2018-01-05 Thread Greg Kurz
This patch changes some error messages in the backend init code and convert backends to propagate QEMU Error objects instead of calling error_report(). One notable improvement is that the local backend now provides a more detailed error report when it fails to open the shared directory.

[Qemu-devel] [PULL 10/12] tests: virtio-9p: set DRIVER_OK before using the device

2018-01-05 Thread Greg Kurz
Signed-off-by: Greg Kurz --- tests/virtio-9p-test.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/virtio-9p-test.c b/tests/virtio-9p-test.c index ebd24b20f657..00f00f7246e9 100644 --- a/tests/virtio-9p-test.c +++ b/tests/virtio-9p-test.c @@ -75,6 +75,9 @@ static

[Qemu-devel] [PULL 11/12] fsdev: improve error handling of backend opts parsing

2018-01-05 Thread Greg Kurz
This patch changes some error messages in the backend opts parsing code and convert backends to propagate QEMU Error objects instead of calling error_report(). Signed-off-by: Greg Kurz --- fsdev/file-op-9p.h | 2 +- fsdev/qemu-fsdev.c | 4 +++- hw/9pfs/9p-handle.c | 2 +-

[Qemu-devel] [PULL 06/12] 9pfs: fix type in *_parse_opts declarations

2018-01-05 Thread Greg Kurz
To comply with the QEMU coding style. Signed-off-by: Greg Kurz Reviewed-by: Eric Blake --- hw/9pfs/9p-handle.c | 2 +- hw/9pfs/9p-local.c | 2 +- hw/9pfs/9p-proxy.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/9pfs/9p-handle.c

[Qemu-devel] [PULL 09/12] tests: virtio-9p: fix ISR dependence

2018-01-05 Thread Greg Kurz
Like other virtio tests, use the used ring APIs instead of assuming ISR being set means the request has completed. Signed-off-by: Greg Kurz Reviewed-by: Stefan Hajnoczi --- tests/virtio-9p-test.c | 30 +++--- 1 file changed, 11

[Qemu-devel] [PULL 04/12] 9pfs: fix some type definitions

2018-01-05 Thread Greg Kurz
To comply with the QEMU coding style. Signed-off-by: Greg Kurz --- hw/9pfs/9p.c | 6 +++--- hw/9pfs/9p.h | 12 ++-- hw/9pfs/virtio-9p-device.c | 2 +- hw/9pfs/xen-9p-backend.c | 2 +- 4 files changed, 11 insertions(+), 11 deletions(-)

[Qemu-devel] [PULL 05/12] 9pfs: handle: fix type definition

2018-01-05 Thread Greg Kurz
To comply with the QEMU coding style. Signed-off-by: Greg Kurz --- hw/9pfs/9p-handle.c | 48 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/hw/9pfs/9p-handle.c b/hw/9pfs/9p-handle.c index

[Qemu-devel] [PULL 02/12] 9pfs: fix XattrOperations typedef

2018-01-05 Thread Greg Kurz
To comply with the QEMU coding style. Signed-off-by: Greg Kurz --- fsdev/file-op-9p.h | 5 +++-- hw/9pfs/9p-xattr.h | 5 ++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fsdev/file-op-9p.h b/fsdev/file-op-9p.h index 474c79d003f6..05b3ef357462 100644 ---

[Qemu-devel] [PULL 03/12] fsdev: fix some type definitions

2018-01-05 Thread Greg Kurz
To comply with the QEMU coding style. Signed-off-by: Greg Kurz --- fsdev/file-op-9p.h | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/fsdev/file-op-9p.h b/fsdev/file-op-9p.h index 05b3ef357462..63d19a6dcd0e 100644 ---

[Qemu-devel] [PULL 00/12] 9p patches for 2.12 20170106

2018-01-05 Thread Greg Kurz
The following changes since commit 281f327487c9c9b1599f93c589a408bbf4a651b8: Merge remote-tracking branch 'remotes/vivier/tags/m68k-for-2.12-pull-request' into staging (2017-12-22 00:11:36 +) are available in the git repository at: https://github.com/gkurz/qemu.git tags/for-upstream

[Qemu-devel] [PATCH 0/1] Check SMT based on KVM_CAP_PPC_SMT_POSSIBLE

2018-01-05 Thread Jose Ricardo Ziviani
If one defines a P9 guest like -smp sockets=1,cores=1,threads=8 QEMU will silently changes threads to 4: (guest) # lscpu Architecture: ppc64le Byte Order:Little Endian CPU(s):4 On-line CPU(s) list: 0-3 Thread(s) per core:4 Core(s) per socket:1

[Qemu-devel] [PATCH 1/1] spapr: Check SMT based on KVM_CAP_PPC_SMT_POSSIBLE

2018-01-05 Thread Jose Ricardo Ziviani
Power9 supports 4 HW threads/core but it's possible to emulate doorbells to implement virtual SMT. KVM has the KVM_CAP_PPC_SMT_POSSIBLE which returns a bitmap with all SMT modes supported by the host. Today, QEMU forces the SMT mode based on PVR compat table, this is silently done in

[Qemu-devel] [PATCH v7 16/20] vdi: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vdi driver accordingly. Note that the TODO is already covered (the block layer guarantees bounds of its requests), and that we can remove the now-unused s->block_sectors. Signed-off-by: Eric Blake

[Qemu-devel] [PATCH v7 15/20] vdi: Avoid bitrot of debugging code

2018-01-05 Thread Eric Blake
Rework the debug define so that we always get -Wformat checking, even when debugging is disabled. Signed-off-by: Eric Blake Reviewed-by: Stefan Weil Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Vladimir Sementsov-Ogievskiy

[Qemu-devel] [PATCH v7 20/20] block: Drop unused .bdrv_co_get_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Now that all drivers have been updated to provide the byte-based .bdrv_co_block_status(), we can delete the sector-based interface. Signed-off-by: Eric Blake Reviewed-by: Vladimir

[Qemu-devel] [PATCH v7 19/20] vvfat: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vvfat driver accordingly. Note that we can rely on the block driver having already clamped limits to our block size, and simplify accordingly. Signed-off-by: Eric Blake Reviewed-by:

[Qemu-devel] [PATCH v7 10/20] qcow: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qcow driver accordingly. There is no intent to optimize based on the want_zero flag for this format. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy

[Qemu-devel] [PATCH v7 18/20] vpc: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vpc driver accordingly. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy --- v7: tweak commit message and type of 'n' [Fam] v6: no change

[Qemu-devel] [PATCH v7 12/20] qed: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qed driver accordingly, taking the opportunity to inline qed_is_allocated_cb() into its lone caller (the callback used to be important, until we switched qed to coroutines). There is no intent to optimize

[Qemu-devel] [PATCH v7 17/20] vmdk: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vmdk driver accordingly. Drop the now-unused vmdk_find_index_in_cluster(). Also, fix a pre-existing bug: if find_extent() fails (unlikely, since the block layer did a bounds check), then we must return a

[Qemu-devel] [PATCH v7 08/20] null: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the null driver accordingly. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Fam Zheng --- v6-v7: no change v5:

[Qemu-devel] [PATCH v7 14/20] sheepdog: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the sheepdog driver accordingly. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Fam Zheng Reviewed-by: Jeff

[Qemu-devel] [PATCH v7 09/20] parallels: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the parallels driver accordingly. Note that the internal function block_status() is still sector-based, because it is still in use by other sector-based functions; but that's okay because request_alignment is

[Qemu-devel] [PATCH v7 13/20] raw: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the raw driver accordingly. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Fam Zheng --- v5-v7: no change v4:

[Qemu-devel] [PATCH v7 06/20] iscsi: Switch iscsi_allocmap_update() to byte-based

2018-01-05 Thread Eric Blake
We are gradually converting to byte-based interfaces, as they are easier to reason about than sector-based. Convert all uses of the allocmap (no semantic change). Callers that already had bytes available are simpler, and callers that now scale to bytes will be easier to switch to byte-based in

[Qemu-devel] [PATCH v7 04/20] gluster: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the gluster driver accordingly. In want_zero mode, we continue to report fine-grained hole information (the caller wants as much mapping detail as possible); but when not in that mode, the caller prefers larger

[Qemu-devel] [PATCH v7 11/20] qcow2: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qcow2 driver accordingly. For now, we are ignoring the 'want_zero' hint. However, it should be relatively straightforward to honor the hint as a way to return larger *pnum values when we have consecutive

[Qemu-devel] [PATCH v7 01/20] block: Add .bdrv_co_block_status() callback

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Now that the block layer exposes byte-based allocation, it's time to tackle the drivers. Add a new callback that operates on as small as byte boundaries. Subsequent patches will then update individual drivers, then

[Qemu-devel] [PATCH v7 05/20] iscsi: Switch cluster_sectors to byte-based

2018-01-05 Thread Eric Blake
We are gradually converting to byte-based interfaces, as they are easier to reason about than sector-based. Convert all uses of the cluster size in sectors, along with adding assertions that we are not dividing by zero. Improve some comment grammar while in the area. Signed-off-by: Eric Blake

[Qemu-devel] [PATCH v7 02/20] block: Switch passthrough drivers to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the generic helpers, and all passthrough clients (blkdebug, commit, mirror, throttle) accordingly. Signed-off-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy

[Qemu-devel] [PATCH v7 07/20] iscsi: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the iscsi driver accordingly. In this case, it is handy to teach iscsi_co_block_status() to handle a NULL map and file parameter, even though the block layer passes non-NULL values, because we also call the

[Qemu-devel] [PATCH v7 00/20] add byte-based block_status driver callbacks

2018-01-05 Thread Eric Blake
There are patches floating around to add NBD_CMD_BLOCK_STATUS, but NBD wants to report status on byte granularity (even if the reporting will probably be naturally aligned to sectors or even much higher levels). I've therefore started the task of converting our block status code to report at a

[Qemu-devel] [PATCH v7 03/20] file-posix: Switch to .bdrv_co_block_status()

2018-01-05 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the file protocol driver accordingly. In want_zero mode, we continue to report fine-grained hole information (the caller wants as much mapping detail as possible); but when not in that mode, the caller prefers

[Qemu-devel] [PATCH v4 3/3] qemu: add linkspeed and duplex settings to virtio-net

2018-01-05 Thread Jason Baron via Qemu-devel
Although linkspeed and duplex can be set in a linux guest via 'ethtool -s', this requires custom ethtool commands for virtio-net by default. Introduce a new feature flag, VIRTIO_NET_F_SPEED_DUPLEX, which allows the hypervisor to export a linkspeed and duplex setting. The user can subsequently

[Qemu-devel] [PATCH net-next v4 1/3] virtio_net: propagate linkspeed/duplex settings from the hypervisor

2018-01-05 Thread Jason Baron via Qemu-devel
The ability to set speed and duplex for virtio_net is useful in various scenarios as described here: 16032be virtio_net: add ethtool support for set and get of settings However, it would be nice to be able to set this from the hypervisor, such that virtio_net doesn't require custom guest ethtool

[Qemu-devel] [PATCH v4 0/3] virtio_net: allow hypervisor to indicate linkspeed and duplex setting

2018-01-05 Thread Jason Baron via Qemu-devel
We have found it useful to be able to set the linkspeed and duplex settings from the host-side for virtio_net. This obviates the need for guest changes and settings for these fields, and does not require custom ethtool commands for virtio_net. The ability to set linkspeed and duplex is useful in

[Qemu-devel] [PATCH v4 2/3] qemu: virtio-net: use 64-bit values for feature flags

2018-01-05 Thread Jason Baron via Qemu-devel
In prepartion for using some of the high order feature bits, make sure that virtio-net uses 64-bit values everywhere. Signed-off-by: Jason Baron Cc: "Michael S. Tsirkin" Cc: Jason Wang Cc: virtio-...@lists.oasis-open.org ---

Re: [Qemu-devel] [PATCH v1 00/21] RISC-V QEMU Port Submission v1

2018-01-05 Thread Paolo Bonzini
On 05/01/2018 13:25, Fam Zheng wrote: >> >> CheckpatchFlags: --ignore-long-lines > It sounds feasible. Putting these flags after a --- line will keep commit > message clean. > > OTOH I think we should spend effort on patching checkpatch.pl to implement > this. Maybe just add something like

[Qemu-devel] [PATCH v4 11/11] [RFH] tests: Add migration compress threads tests

2018-01-05 Thread Juan Quintela
Yeap, it is still not working. trying to learn how to debug threads for guests running from the testt hardness. For some reason, compression is not working at the moment, test is disabled until I found why. Signed-off-by: Juan Quintela --- tests/migration-test.c | 51

[Qemu-devel] [PATCH v4 10/11] tests: Adjust sleeps for migration test

2018-01-05 Thread Juan Quintela
Also reorder code to not sleep when event already happened. Signed-off-by: Juan Quintela --- tests/migration-test.c | 29 ++--- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index

[Qemu-devel] [PATCH v4 06/11] tests: Add migration precopy test

2018-01-05 Thread Juan Quintela
Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- tests/migration-test.c | 44 ++-- 1 file changed, 42 insertions(+), 2 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c

[Qemu-devel] [PATCH v4 08/11] tests: Add migration xbzrle test

2018-01-05 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 67 +- 1 file changed, 66 insertions(+), 1 deletion(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index 024b6675fb..c25c9dd7b1 100644 ---

[Qemu-devel] [PATCH v4 09/11] tests: Create migrate-start-postcopy command

2018-01-05 Thread Juan Quintela
This way, it is like the rest of commands Signed-off-by: Juan Quintela --- tests/migration-test.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index c25c9dd7b1..70b3c0870b 100644 ---

[Qemu-devel] [PATCH v4 04/11] tests: Use consistent names and sizes for migration

2018-01-05 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index d81f22118b..f469235d0b 100644 --- a/tests/migration-test.c +++

[Qemu-devel] [PATCH v4 05/11] tests: Add deprecated commands migration test

2018-01-05 Thread Juan Quintela
We add deprecated commands on a new test, so we don't have to add it on normal tests. Signed-off-by: Juan Quintela --- tests/migration-test.c | 59 ++ 1 file changed, 59 insertions(+) diff --git a/tests/migration-test.c

[Qemu-devel] [PATCH v4 07/11] tests: Add basic migration precopy tcp test

2018-01-05 Thread Juan Quintela
Not sharing code from precopy/unix because we have to read back the tcp parameter. Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- tests/migration-test.c | 59 +++--- 1 file changed, 56

[Qemu-devel] [PATCH v4 03/11] tests: Consolidate accelerators declaration

2018-01-05 Thread Juan Quintela
Signed-off-by: Juan Quintela --- tests/migration-test.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index 32f3bb86a8..d81f22118b 100644 --- a/tests/migration-test.c +++

[Qemu-devel] [PATCH v4 02/11] tests: Migration ppc test was missing arguments

2018-01-05 Thread Juan Quintela
Argument file is also needed there. Signed-off-by: Juan Quintela --- tests/migration-test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index 0448bc77dc..32f3bb86a8 100644 --- a/tests/migration-test.c

[Qemu-devel] [PATCH v4 01/11] tests: Remove deprecated migration tests commands

2018-01-05 Thread Juan Quintela
We move to use migration_set_parameter() for everything. Signed-off-by: Juan Quintela --- tests/migration-test.c | 33 - 1 file changed, 8 insertions(+), 25 deletions(-) diff --git a/tests/migration-test.c b/tests/migration-test.c index

[Qemu-devel] [PATCH v4 00/11] Add make check tests for Migration

2018-01-05 Thread Juan Quintela
Hi In v4: Based-on: 20180105205109.683-1-quint...@redhat.com Changes: - rebase on top on v4 info_migrate patches - Tune sleeps to make patches fast - Create a deprecated test for deprecated commands (i.e. make peterxu happy) - create migrate_start_postcopy function - fix naming/sizes between

Re: [Qemu-devel] [PATCH v1 12/21] RISC-V HART Array

2018-01-05 Thread Eric Blake
On 01/05/2018 03:41 PM, Antony Pavlov wrote: > On Wed, 3 Jan 2018 13:44:16 +1300 > Michael Clark wrote: > >> Holds the state of a heterogenous array of RISC-V hardware threads. > > > Hmm, you use SysBusDevice, uint32_t and RISCVCPU types but there is no header > files >

Re: [Qemu-devel] [PATCH v1 19/21] SiFive Freedom E300 RISC-V Machine

2018-01-05 Thread Antony Pavlov
On Wed, 3 Jan 2018 13:44:23 +1300 Michael Clark wrote: > This provides a RISC-V Board compatible with the the SiFive E300 SDK. > The following machine is implemented: > > - 'sifive_e300'; CLINT, PLIC, UART, AON, GPIO, QSPI, PWM > ... > diff --git

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Philippe Mathieu-Daudé
Hi Eric, On Fri, Jan 5, 2018 at 12:29 PM, Eric Blake wrote: > On 01/03/2018 03:49 PM, Philippe Mathieu-Daudé wrote: [...] >> +if (ambiguous) { >> +error_setg(errp, "Path '%s' is ambiguous", qom_path); >> +} else { >> +error_set(errp,

Re: [Qemu-devel] [PATCH v1 12/21] RISC-V HART Array

2018-01-05 Thread Antony Pavlov
On Wed, 3 Jan 2018 13:44:16 +1300 Michael Clark wrote: > Holds the state of a heterogenous array of RISC-V hardware threads. ... > --- /dev/null > +++ b/include/hw/riscv/riscv_hart.h > @@ -0,0 +1,45 @@ > +/* > + * QEMU RISC-V Hart Array interface > + * > + * Copyright (c)

[Qemu-devel] [Bug 1731957] Re: qemu-kvm exits with console permission problems

2018-01-05 Thread David Moreau Simard
These might be helpful or related: - https://bugzilla.redhat.com/show_bug.cgi?id=1499800 - https://bugzilla.redhat.com/show_bug.cgi?id=1501957 ** Bug watch added: Red Hat Bugzilla #1499800 https://bugzilla.redhat.com/show_bug.cgi?id=1499800 ** Bug watch added: Red Hat Bugzilla #1501957

Re: [Qemu-devel] linux-user/mmap: Should not return NULL on guest call mmap(NULL, ...), causes crash inside glibc

2018-01-05 Thread Maximilian Riemensberger
On 05.01.18 19:38, Peter Maydell wrote: > On 5 January 2018 at 18:13, Maximilian Riemensberger > wrote: >> Hi, >> >> yesterday I hit the following problem when running an arm linux executable on >> qemu-2.10 (qemu-arm-static through binfmt_misc) >> >> 1879 >>

[Qemu-devel] [PATCH v4 1/2] migration: Create tcp_port parameter

2018-01-05 Thread Juan Quintela
It will be used to store the uri tcp_port parameter. This is the only parameter than can change and we can need to be able to connect to it. Signed-off-by: Juan Quintela -- This used to be uri parameter, but it has so many troubles to reproduce that it don't just make

[Qemu-devel] [PATCH v4 0/2] Improve info migrate output on destination

2018-01-05 Thread Juan Quintela
Hi In v4: Based-on: 20180103093834.20879-1-quint...@redhat.com - update and rebase - tcp-port is not a property that can be set anymore (peter xu) - improve comments (eric) Please, review. [v3] On top of my previous migration-for-2.11 patches. - We don't create an uri parameter, just too

[Qemu-devel] [PATCH v4 2/2] migration: Set the migration tcp port

2018-01-05 Thread Juan Quintela
We can set the port parameter as zero. This patch lets us know what port the system was choosen for us. Now we can migrate to this place. Signed-off-by: Juan Quintela -- This was migrate_set_uri(), but as we only need the tcp_port, change to that one. ---

Re: [Qemu-devel] [PATCH v3 5/6] tests: Add migration xbzrle test

2018-01-05 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > "Dr. David Alan Gilbert" wrote: > > * Juan Quintela (quint...@redhat.com) wrote: > >> Signed-off-by: Juan Quintela > >> --- > >> tests/migration-test.c | 68 > >>

[Qemu-devel] [Bug 1727250] Re: qemu-io-test 147 segfaults when configured with gcov

2018-01-05 Thread Murilo Opsfelder Araújo
I confirmed that my patch http://lists.nongnu.org/archive/html/qemu- devel/2018-01/msg00883.html fixes this bug too. ** Changed in: qemu Status: New => In Progress ** Changed in: qemu Assignee: (unassigned) => Murilo Opsfelder Araújo (mopsfelder) -- You received this bug

Re: [Qemu-devel] [PATCH v3 5/6] tests: Add migration xbzrle test

2018-01-05 Thread Juan Quintela
"Dr. David Alan Gilbert" wrote: > * Juan Quintela (quint...@redhat.com) wrote: >> Signed-off-by: Juan Quintela >> --- >> tests/migration-test.c | 68 >> ++ >> 1 file changed, 68 insertions(+) >> >> diff

Re: [Qemu-devel] [PATCH] isa/piix: move to mips

2018-01-05 Thread Philippe Mathieu-Daudé
Hi Michael, On Thu, Jan 4, 2018 at 8:21 PM, Michael S. Tsirkin wrote: > This device is only used on mips, move to the correct maintainer > section. > > Cc: Aurelien Jarno > Cc: Yongbok Kim > Cc: Hervé Poussineau

Re: [Qemu-devel] linux-user/mmap: Should not return NULL on guest call mmap(NULL, ...), causes crash inside glibc

2018-01-05 Thread Peter Maydell
On 5 January 2018 at 18:13, Maximilian Riemensberger wrote: > Hi, > > yesterday I hit the following problem when running an arm linux executable on > qemu-2.10 (qemu-arm-static through binfmt_misc) > > 1879 >

  1   2   >