Re: [Qemu-devel] [PATCH v4 2/3] utils: Add helper to read arm MIDR_EL1 register

2016-10-28 Thread Vijay Kilari
On Thu, Oct 27, 2016 at 9:33 PM, Peter Maydell wrote: > On 25 October 2016 at 13:12, wrote: >> From: Vijaya Kumar K >> >> Add helper API to read MIDR_EL1 registers to fetch >> cpu identification information. This helps in >> adding errata's and architecture specific features. >> >> This is impl

[Qemu-devel] [PATCH v13 08/19] block: Block all intermediate nodes in commit_active_start()

2016-10-28 Thread Alberto Garcia
When block-commit is launched without the top parameter, it uses internally a mirror block job. In that case all intermediate nodes between the active and base nodes must be blocked as well. Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- block/mirror.c | 8 1 file changed, 8

[Qemu-devel] [PATCH v13 06/19] block: Check blockers in all nodes involved in a block-commit job

2016-10-28 Thread Alberto Garcia
qmp_block_commit() checks for op blockers in the active and destination (base) images. However all nodes between top_bs and base are also involved, and they are removed from the chain afterwards. In addition to that, if top_bs is not the active layer then top_bs's overlay also needs to be checked

[Qemu-devel] [PATCH v13 03/19] block: Add block_job_add_bdrv()

2016-10-28 Thread Alberto Garcia
When a block job is created on a certain BlockDriverState, operations are blocked there while the job exists. However, some block jobs may involve additional BDSs, which must be blocked separately when the job is created and unblocked manually afterwards. This patch adds block_job_add_bdrv(), that

[Qemu-devel] [PATCH v13 15/19] qemu-iotests: Test block-stream and block-commit in parallel

2016-10-28 Thread Alberto Garcia
As with test_stream_parallel(), we allow mixing block-stream and block-commit operations in the same backing chain as long as there's no overlap among the involved nodes. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 30 ++ tests/qemu-iotests/030.out

[Qemu-devel] [PATCH v13 04/19] block: Use block_job_add_bdrv() in mirror_start_job()

2016-10-28 Thread Alberto Garcia
Use block_job_add_bdrv() instead of blocking all operations in mirror_start_job() and unblocking them in mirror_exit(). Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- block/mirror.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/block/mirror.c b/block/mirror.c

[Qemu-devel] [PATCH v13 01/19] block: Add bdrv_drain_all_{begin, end}()

2016-10-28 Thread Alberto Garcia
bdrv_drain_all() doesn't allow the caller to do anything after all pending requests have been completed but before block jobs are resumed. This patch splits bdrv_drain_all() into _begin() and _end() for that purpose. It also adds aio_{disable,enable}_external() calls to disable external clients in

[Qemu-devel] [PATCH v13 05/19] block: Use block_job_add_bdrv() in backup_start()

2016-10-28 Thread Alberto Garcia
Use block_job_add_bdrv() instead of blocking all operations in backup_start() and unblocking them in backup_run(). Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- block/backup.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/block/backup.c b/block/backup.c ind

[Qemu-devel] [PATCH v13 13/19] qemu-iotests: Test block-stream operations in parallel

2016-10-28 Thread Alberto Garcia
This test case checks that it's possible to launch several stream operations in parallel in the same snapshot chain, each one involving a different set of nodes. Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- tests/qemu-iotests/030 | 80 +++

[Qemu-devel] [PATCH v13 07/19] block: Block all nodes involved in the block-commit operation

2016-10-28 Thread Alberto Garcia
After a successful block-commit operation all nodes between top and base are removed from the backing chain, and top's overlay needs to be updated to point to base. Because of that we should prevent other block jobs from messing with them. This patch blocks all operations in these nodes in commit_

[Qemu-devel] [PATCH v13 12/19] qemu-iotests: Test streaming to an intermediate layer

2016-10-28 Thread Alberto Garcia
This adds test_stream_intermediate(), similar to test_stream() but streams to the intermediate image instead. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 21 - tests/qemu-iotests/030.out | 4 ++-- 2 files changed, 22 insertions(+), 3 deletions(-) diff --g

[Qemu-devel] [PATCH v13 09/19] block: Support streaming to an intermediate layer

2016-10-28 Thread Alberto Garcia
This makes sure that the image we are streaming into is open in read-write mode during the operation. Operation blockers are also set in all intermediate nodes, since they will be removed from the chain afterwards. Finally, this also unblocks the stream operation in backing files. Signed-off-by:

[Qemu-devel] [PATCH v13 11/19] docs: Document how to stream to an intermediate layer

2016-10-28 Thread Alberto Garcia
Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- docs/live-block-ops.txt | 36 +--- 1 file changed, 25 insertions(+), 11 deletions(-) diff --git a/docs/live-block-ops.txt b/docs/live-block-ops.txt index a257087..2211d14 100644 --- a/docs/live-block-ops.t

[Qemu-devel] [PATCH v13 19/19] qemu-iotests: Test the 'base-node' parameter of 'block-stream'

2016-10-28 Thread Alberto Garcia
The block-stream command has traditionally used the 'base' parameter to indicate the image to copy the data from. This test checks that the 'base-node' parameter can also be used for the same purpose. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/030 | 37 +

[Qemu-devel] [PATCH v13 16/19] qemu-iotests: Add iotests.supports_quorum()

2016-10-28 Thread Alberto Garcia
There's many tests that need Quorum support in order to run. At the moment each test implements its own check to see if Quorum is enabled. This patch centralizes all those checks in a new function called iotests.supports_quorum(). Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- tests/

[Qemu-devel] [PATCH v13 02/19] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-28 Thread Alberto Garcia
When a BlockDriverState is about to be reopened it can trigger certain operations that need to write to disk. During this process a different block job can be woken up. If that block job completes and also needs to call bdrv_reopen() it can happen that it needs to do it on the same BlockDriverState

[Qemu-devel] [PATCH v13 14/19] qemu-iotests: Test overlapping stream and commit operations

2016-10-28 Thread Alberto Garcia
These test cases check that it's not possible to perform two block-stream or block-commit operations if there are nodes involved in both. Signed-off-by: Alberto Garcia Reviewed-by: Kevin Wolf --- tests/qemu-iotests/030 | 89 ++ tests/qemu-iotests/

[Qemu-devel] [PATCH v13 17/19] qemu-iotests: Test streaming to a Quorum child

2016-10-28 Thread Alberto Garcia
Quorum children are special in the sense that they're not directly attached to a block backend but they're not used as backing images either. However the intermediate block streaming code supports streaming to them. This is a test case for that scenario. Signed-off-by: Alberto Garcia Reviewed-by:

[Qemu-devel] [PATCH v13 00/19] Support streaming to an intermediate layer

2016-10-28 Thread Alberto Garcia
Hi, thank you Kevin and Eric for your comments. I addressed them all in this series. Berto v13: - Patch 10: explain that the 'base-node' parameter can be used to detect whether we support streaming to an intermediate node. - Patch 12: Revert the blkdebug removal, it is indeed necessary to ke

[Qemu-devel] [PATCH v13 18/19] block: Add 'base-node' parameter to the 'block-stream' command

2016-10-28 Thread Alberto Garcia
The way to specify the node from which to copy data in the block-stream operation is by using the 'base' parameter. This parameter however takes a file name, not a node name. Since we want to be able to perform this operation using only node names, this patch adds a new 'base-node' parameter. Sig

[Qemu-devel] [PATCH v13 10/19] block: Add QMP support for streaming to an intermediate layer

2016-10-28 Thread Alberto Garcia
This patch makes the 'device' parameter of the 'block-stream' command accept a node name that is not a root node. The presence of this feature can't be directly tested with introspection; soon we'll introduce a 'base-node' parameter whose presence can be checked for this purpose. In addition to th

Re: [Qemu-devel] [PATCH v10 10/19] vfio iommu: Add blocking notifier to notify DMA_UNMAP

2016-10-28 Thread Jike Song
On 10/27/2016 05:29 AM, Kirti Wankhede wrote: > Added blocking notifier to IOMMU TYPE1 driver to notify vendor drivers > about DMA_UNMAP. > Exported two APIs vfio_register_notifier() and vfio_unregister_notifier(). > Vendor driver should register notifer using these APIs. > Vendor driver should use

Re: [Qemu-devel] [PATCH] curses: build with -std=gnu99

2016-10-28 Thread Samuel Thibault
Markus Armbruster, on Fri 28 Oct 2016 08:51:20 +0200, wrote: > Samuel Thibault writes: > > > Peter Maydell, on Thu 27 Oct 2016 17:52:17 +0100, wrote: > >> http://www.thecodingforums.com/threads/wchar_t-is-useless.806149/#post-4398211 > > > > UURrgll... So we can't use L'\u23bd' on such systems, i

[Qemu-devel] [PATCH v3] block/vxhs: Add Veritas HyperScale VxHS block device support

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

Re: [Qemu-devel] [PATCH v2] block/vxhs: Add Veritas HyperScale VxHS block device support

2016-10-28 Thread ashish mittal
I have submitted v3 with the QAPI schema changes. Regards, Ashish On Thu, Oct 27, 2016 at 2:38 AM, ashish mittal wrote: > Hi Daniel, > >>> >I think this version still does not address Daniel's concerns regarding a >>> >QAPI schema for vxhs. >>> >>> We are working on QAPI schema changes and will

[Qemu-devel] [RFC PATCH qemu] spapr_pci: Create PCI-express root bus by default

2016-10-28 Thread Alexey Kardashevskiy
At the moment sPAPR PHB creates a root buf of TYPE_PCI_BUS type. This means that vfio-pci devices attached to it (and this is a default behaviour) hide PCIe extended capabilities as the bus does not pass a pci_bus_is_express(pdev->bus) check. This changes adds a default PCI bus type property to sP

[Qemu-devel] [PATCH v10 05/12] virtio-crypto: add virtio crypto device emulation

2016-10-28 Thread Gonglei
Introduce the virtio crypto realization, I'll finish the core code in the following patches. The thoughts came from virtio net realization. For more information see: http://qemu-project.org/Features/VirtioCrypto Signed-off-by: Gonglei --- hw/virtio/Makefile.objs | 1 + hw/

[Qemu-devel] [PATCH v10 03/12] virtio-crypto: introduce virtio_crypto.h

2016-10-28 Thread Gonglei
Introduce the virtio_crypto.h which follows virtio-crypto specification. Signed-off-by: Gonglei --- include/standard-headers/linux/virtio_crypto.h | 429 + 1 file changed, 429 insertions(+) create mode 100644 include/standard-headers/linux/virtio_crypto.h diff --git a/i

[Qemu-devel] [PATCH v10 12/12] virtio-crypto: add myself as virtio-crypto and cryptodev backends maintainer

2016-10-28 Thread Gonglei
This patch includes two parts: Cryptodev Backends and virtio-crypto stuff. I can maintain cryptodev backends which introduced by myself. For virtio-crypto stuff, I can share the work with Michael (The whole virtio supporter). Signed-off-by: Gonglei --- MAINTAINERS | 13 + 1 file chan

[Qemu-devel] [PATCH v10 01/12] cryptodev: introduce cryptodev backend interface

2016-10-28 Thread Gonglei
cryptodev backend interface is used to realize the active work for virtual crypto device. This patch only add the framework, doesn't include specific operations. Signed-off-by: Gonglei --- backends/Makefile.objs | 2 + backends/cryptodev.c | 176 +

[Qemu-devel] [PATCH v10 07/12] virtio-crypto: set capacity of algorithms supported

2016-10-28 Thread Gonglei
Expose the capacity of algorithms supported by virtio crypto device to the frontend driver using pci configuration space. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 43 +++ include/hw/virtio/virtio-crypto.h | 18 2 files ch

Re: [Qemu-devel] [PATCH v5 30/33] target-arm/cpu: don't reset TLB structures, use cputlb to do it

2016-10-28 Thread Alex Bennée
Richard Henderson writes: > On 10/27/2016 08:10 AM, Alex Bennée wrote: >> cputlb owns the TLB entries and knows how to safely update them in >> MTTCG. >> >> Signed-off-by: Alex Bennée >> --- >> target-arm/cpu.c | 6 ++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/target-arm/cpu.c

[Qemu-devel] [PATCH v10 08/12] virtio-crypto: add control queue handler

2016-10-28 Thread Gonglei
Realize the symmetric algorithm control queue handler, including plain cipher and chainning algorithms. Currently the control queue is used to create and close session for symmetric algorithm. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 299 +++

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

2016-10-28 Thread Gonglei
Hi, The virtio crypto is a virtual crypto device as well as a kind of virtual hardware accelerator for virtual machines. The encryption and decryption requests are placed in the data queue and handled by the real crypto accelerators finally. The second queue is the control queue used to create or

[Qemu-devel] [PATCH v10 11/12] virtio-crypto: using bh to handle dataq's requests

2016-10-28 Thread Gonglei
Make crypto operations are executed asynchronously, so that other QEMU threads and monitor couldn't be blocked at the virtqueue handling context. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 55 +-- include/hw/virtio/virtio-crypto.h | 8

Re: [Qemu-devel] [PATCH] curses: build with -std=gnu99

2016-10-28 Thread Markus Armbruster
Samuel Thibault writes: > Markus Armbruster, on Fri 28 Oct 2016 08:51:20 +0200, wrote: >> Samuel Thibault writes: >> >> > Peter Maydell, on Thu 27 Oct 2016 17:52:17 +0100, wrote: >> >> http://www.thecodingforums.com/threads/wchar_t-is-useless.806149/#post-4398211 >> > >> > UURrgll... So we can'

[Qemu-devel] [PATCH v10 02/12] cryptodev: add symmetric algorithm operation stuff

2016-10-28 Thread Gonglei
This patch adds session operation and crypto operation stuff in the cryptodev backend, including function pointers and corresponding structures. Signed-off-by: Gonglei --- backends/cryptodev.c | 45 ++ include/sysemu/cryptodev.h | 149 ++

[Qemu-devel] [PATCH v10 09/12] virtio-crypto: add data queue processing handler

2016-10-28 Thread Gonglei
Introduces VirtIOCryptoReq structure to store crypto request so that we can easily support asynchronous crypto operation in the future. At present, we only support cipher and algorithm chaining. Signed-off-by: Gonglei --- hw/virtio/virtio-crypto.c | 358 +

[Qemu-devel] [PULL 17/18] target-m68k: immediate ops manage word and byte operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 57 ++--- 1 file changed, 35 insertions(+), 22 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 57ac2e5..ee0ffe3 100644 --- a/t

[Qemu-devel] [PULL 10/18] target-m68k: and can manage word and byte operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index a6aaf5e..3f7db83 100644 --- a/target-m68k/translate.c +++ b/targe

[Qemu-devel] [PATCH v10 10/12] cryptodev: introduce an unified wrapper for crypto operation

2016-10-28 Thread Gonglei
We use an opaque point to the VirtIOCryptoReq which can support different packets based on different algorithms. Signed-off-by: Gonglei --- backends/cryptodev.c | 28 ++-- hw/virtio/virtio-crypto.c | 10 +- include/sysemu/cryptodev.h | 13 +++-- 3 f

[Qemu-devel] [PULL 01/18] target-m68k: add bkpt instruction

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier --- target-m68k/translate.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 6c6173a..a128b67 100644 --- a/target-m68k/translate.c +++ b/target-m68k/translate.c @@ -1639,6 +1639,11 @@ DISAS_INSN(swap)

[Qemu-devel] [PULL 16/18] target-m68k: cmp manages word and bytes operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 0448569..57ac2e5 100644 --- a/target-m68k/translate.c +++ b/targ

[Qemu-devel] [PULL 11/18] target-m68k: suba/adda can manage word operand

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 3f7db83..b82ebf3 100644 --- a/target-m68k/translate.c +++ b/target-m68k/transl

[Qemu-devel] [PATCH v10 04/12] cryptodev: introduce a new cryptodev backend

2016-10-28 Thread Gonglei
The new cryptodev backend named cryptodev-builtin, which realized by QEMU cipher APIs. These APIs can be backed by either nettle or gcrypt. Signed-off-by: Gonglei --- backends/Makefile.objs | 1 + backends/cryptodev-builtin.c | 361 +++ qemu-option

[Qemu-devel] [PULL 07/18] target-m68k: add addressing modes to not

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index fd6631d..031f7eb 100644 --- a/target-m68k/translate.c +++ b/target-

[Qemu-devel] [PULL 03/18] target-m68k: add exg ops

2016-10-28 Thread Laurent Vivier
Suggested-by: Richard Henderson Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 0d3111d..b407623 100644 --- a/targ

[Qemu-devel] [PATCH v10 06/12] virtio-crypto-pci: add virtio crypto pci support

2016-10-28 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 | 77 +++ hw/virtio/virtio-pci.h| 15 + 3 files changed,

[Qemu-devel] [PULL 13/18] target-m68k: introduce byte and word cc_ops

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/cpu.h | 6 +-- target-m68k/helper.c| 25 ++--- target-m68k/translate.c | 131 +++- 3 files changed, 93 insertions(+), 69 deletions(-) diff --git a/target-m68

[Qemu-devel] [PULL 14/18] target-m68k: add addressing modes to neg

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index aa09bd4..f7e6920 100644 --- a/target-m68k/translate.c +++ b/t

[Qemu-devel] [PULL 18/18] MAINTAINERS: update M68K entry

2016-10-28 Thread Laurent Vivier
Add myself to be the M68K maintainer. Signed-off-by: Laurent Vivier Reviewed-by: Thomas Huth --- MAINTAINERS | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index b01fec0..7d73748 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -133,7 +133,8 @@ F: i

[Qemu-devel] [PULL 05/18] target-m68k: add dbcc

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 25 + 1 file changed, 25 insertions(+) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index e595673..1836a22 100644 --- a/target-m68k/translate.c +++ b/target-m68k/

[Qemu-devel] [PULL 09/18] target-m68k: or can manage word and byte operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 9734d05..a6aaf5e 100644 --- a/target-m68k/translate.c +++ b/target-m

[Qemu-devel] [PULL 00/18] M68k part2 patches

2016-10-28 Thread Laurent Vivier
The following changes since commit 835f3d24b42fcbeca5c49048994a4e5d0fe905c5: Merge remote-tracking branch 'remotes/kraxel/tags/pull-audio-20161027-1' into staging (2016-10-27 17:24:29 +0100) are available in the git repository at: g...@github.com:vivier/qemu-m68k.git tags/m68k-part2-pull-re

[Qemu-devel] [PULL 02/18] target-m68k: add linkl

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 26 +- 1 file changed, 21 insertions(+), 5 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index a128b67..0d3111d 100644 --- a/target-m68k/translate.c +

[Qemu-devel] [PULL 06/18] target-m68k: Inline addx, subx, negx

2016-10-28 Thread Laurent Vivier
From: Richard Henderson Signed-off-by: Richard Henderson And add opcodes for 680x0 Signed-off-by: Laurent Vivier --- target-m68k/helper.c| 40 -- target-m68k/helper.h| 2 - target-m68k/translate.c | 196 +++- 3 files changed, 178

[Qemu-devel] [PATCH v3] util/mmap-alloc: check parameter before using

2016-10-28 Thread Cao jin
Also refactor a little bit for readability Signed-off-by: Cao jin --- util/mmap-alloc.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c index 5a85aa3..2f55f5e 100644 --- a/util/mmap-alloc.c +++ b/util/mmap-alloc.c @@ -12

[Qemu-devel] [PULL 04/18] target-m68k: add addressing modes to scc

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index b407623..e595673 100644 --- a/target-m68

[Qemu-devel] [PULL 15/18] target-m68k: add/sub manage word and byte operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 73 +++-- 1 file changed, 40 insertions(+), 33 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index f7e6920..0448569 100644 --- a/t

[Qemu-devel] [PULL 08/18] target-m68k: eor can manage word and byte operands

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 031f7eb..9734d05 100644 --- a/target-m68k/translate.c +++ b/target-m6

Re: [Qemu-devel] [PATCH v5 30/33] target-arm/cpu: don't reset TLB structures, use cputlb to do it

2016-10-28 Thread Alex Bennée
Peter Maydell writes: > On 28 October 2016 at 09:38, Alex Bennée wrote: >> >> Richard Henderson writes: >>> And don't we (or if not, shouldn't we) >>> handle the tlb_flush generically for reset? >> >> Probably. tlb_flush seems to be one of those things liberally sprinkled >> in the arch code f

Re: [Qemu-devel] [PATCH v4 2/3] utils: Add helper to read arm MIDR_EL1 register

2016-10-28 Thread Peter Maydell
On 28 October 2016 at 08:00, Vijay Kilari wrote: > On Thu, Oct 27, 2016 at 9:33 PM, Peter Maydell > wrote: >> On 25 October 2016 at 13:12, wrote: >>> From: Vijaya Kumar K >>> >>> Add helper API to read MIDR_EL1 registers to fetch >>> cpu identification information. This helps in >>> adding er

Re: [Qemu-devel] [PATCH v2] trace: Fix 'char **' compilation error in simple backend

2016-10-28 Thread Peter Maydell
On 26 October 2016 at 14:18, Eric Blake wrote: > On 10/25/2016 10:50 PM, Fam Zheng wrote: >> Currently, the generated function body will do "strlen(arg)" but the >> argument could be 'char **' or 'char * const *'. Avoid that by excluding >> such cases in is_string check. >> >> Reported by patchew'

[Qemu-devel] [PULL 12/18] target-m68k: some bit ops cleanup

2016-10-28 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- target-m68k/translate.c | 34 +++--- 1 file changed, 15 insertions(+), 19 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index b82ebf3..cf9b228 100644 --- a/target-m68k/tran

Re: [Qemu-devel] [PATCH v5 30/33] target-arm/cpu: don't reset TLB structures, use cputlb to do it

2016-10-28 Thread Peter Maydell
On 28 October 2016 at 09:38, Alex Bennée wrote: > > Richard Henderson writes: >> And don't we (or if not, shouldn't we) >> handle the tlb_flush generically for reset? > > Probably. tlb_flush seems to be one of those things liberally sprinkled > in the arch code for all sorts of things but certain

Re: [Qemu-devel] [RFC] powernv: CPU compatibility modes don't make sense for powernv

2016-10-28 Thread Greg Kurz
On Fri, 28 Oct 2016 13:05:01 +1100 David Gibson wrote: > powernv has some code (derived from the spapr equivalent) used in device > tree generation which depends on the CPU's compatibility mode / logical > PVR. However, compatibility modes don't make sense on powernv - at least > not as a proper

[Qemu-devel] [Bug 1637447] [NEW] VNC/RFB: QEMU reports incorrect name (length)

2016-10-28 Thread cardamon
Public bug reported: If the name of a machine (as set with the -name argument) has a length longer than 1024, (RFB) VNC clients will not receive a correct RFB ServerInit message. I suspect this is the problem: https://github.com/qemu/qemu/blob/master/ui/vnc.c#L2463 The return value of snprintf

Re: [Qemu-devel] [PATCH] curses: build with -std=gnu99

2016-10-28 Thread Gerd Hoffmann
Hi, > Before we uglify our code to work around (egregious!) defects in certain > compilers, we should find out whether and where exactly these defects > exist, and whether it's worth targeting the defective compilers. Using iconv is on the table _anyway_, to properly support "-display curses,ch

Re: [Qemu-devel] [PATCH] x86/cpuid:add AVX512_4VNNIW and AVX512_4FMAPS features.

2016-10-28 Thread Paolo Bonzini
On 28/10/2016 11:15, He Chen wrote: > From: Luwei Kang > > The spec can be found in Intel Software Developer Manual or in > Instruction Set Extensions Programming Reference. > > Signed-off-by: Luwei Kang > Signed-off-by: Piotr Luc > --- > target-i386/cpu.c | 19 ++- > target

Re: [Qemu-devel] [PULL 0/1] virtio-gpu: fix memory leak in virtio_gpu_resource_create_2d

2016-10-28 Thread Peter Maydell
On 27 October 2016 at 15:33, Gerd Hoffmann wrote: > Hi, > > vga patch queue with a virtio-gpu memory leak fix.# > > please pull, > Gerd > > The following changes since commit ede0cbeb7892bdf4a19128853a3a3c61a17fb068: > > Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2016-10-25'

Re: [Qemu-devel] [PATCH v2 0/2] less confusing block file names

2016-10-28 Thread Laszlo Ersek
On 10/27/16 21:48, Eric Blake wrote: > Based on a suggestion here: > https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg00350.html > > v1 was here, and got some R-b but also some discussion: > https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg04618.html > > Since then: > patch 1: pic

Re: [Qemu-devel] [Qemu-ppc] [RFC PATCH qemu] spapr_pci: Create PCI-express root bus by default

2016-10-28 Thread Greg Kurz
On Fri, 28 Oct 2016 18:56:40 +1100 Alexey Kardashevskiy wrote: > At the moment sPAPR PHB creates a root buf of TYPE_PCI_BUS type. > This means that vfio-pci devices attached to it (and this is > a default behaviour) hide PCIe extended capabilities as > the bus does not pass a pci_bus_is_express(p

Re: [Qemu-devel] [PATCH v4 2/3] utils: Add helper to read arm MIDR_EL1 register

2016-10-28 Thread Vijay Kilari
On Fri, Oct 28, 2016 at 2:33 PM, Peter Maydell wrote: > On 28 October 2016 at 08:00, Vijay Kilari wrote: >> On Thu, Oct 27, 2016 at 9:33 PM, Peter Maydell >> wrote: >>> On 25 October 2016 at 13:12, wrote: From: Vijaya Kumar K Add helper API to read MIDR_EL1 registers to fetch

[Qemu-devel] [Bug 1637447] Re: VNC/RFB: QEMU reports incorrect name (length)

2016-10-28 Thread Daniel Berrange
The right fix here is to switch to use g_strdup_printf and avoid a fixed length stack buffer entirely. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1637447 Title: VNC/RFB: QEMU reports incorrect n

[Qemu-devel] [Bug 1637447] Re: VNC/RFB: QEMU reports incorrect name (length)

2016-10-28 Thread Jakob van Kruijssen
** Description changed: If the name of a machine (as set with the -name argument) has a length longer than 1024, (RFB) VNC clients will not receive a correct RFB ServerInit message. I suspect this is the problem: - https://github.com/qemu/qemu/blob/master/ui/vnc.c#L2463 + https://git

Re: [Qemu-devel] [PATCH v2 1/1] block/gluster: memory usage: use one glfs instance per volume

2016-10-28 Thread Niels de Vos
On Thu, Oct 27, 2016 at 08:54:50PM +0530, Prasanna Kumar Kalever wrote: > Currently, for every drive accessed via gfapi we create a new glfs > instance (call glfs_new() followed by glfs_init()) which could consume > memory in few 100 MB's, from the table below it looks like for each > instance ~300

Re: [Qemu-devel] [PATCH v13 00/19] Support streaming to an intermediate layer

2016-10-28 Thread Kevin Wolf
Am 28.10.2016 um 09:08 hat Alberto Garcia geschrieben: > Hi, > > thank you Kevin and Eric for your comments. I addressed them all in > this series. Thanks, applied to the block branch. Kevin

[Qemu-devel] [Bug 497273] Re: winxp.64 fails to install in -rc2 with kvm

2016-10-28 Thread Thomas Huth
Fedora 11, QEMU 0.12 and Windows XP are pretty much outdated nowadays, so I guess nobody will take care of this ticket anymore - sorry! So I'm closing this as "Won't fix" now. (Anyway, if you still can reproduce this problem with the latest version of QEMU, feel free to open the bug again) ** Chan

[Qemu-devel] [Bug 584514] Re: Qemu-KVM 0.12.4 Guest entered Paused State

2016-10-28 Thread Thomas Huth
QEMU 0.12 is pretty old nowadays - is this issue still reproducible with the latest version of QEMU? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs

Re: [Qemu-devel] [PULL 0/1] seabios: update to 1.10.0 release.

2016-10-28 Thread Peter Maydell
On 27 October 2016 at 15:52, Gerd Hoffmann wrote: > Hi, > > $subject says all. Details are in the commit message. Beside the usual > set of improvements and bugfixes this brings support for more than 255 > vcpus. > > please pull, > Gerd > > The following changes since commit ede0cbeb7892bdf4

[Qemu-devel] [Bug 589231] Re: cirrus vga is very slow in qemu-kvm-0.12

2016-10-28 Thread Thomas Huth
QEMU 0.12 is pretty much outdated - has this been fixed in a newer version of QEMU? I.e. do you think we can close this bug nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU

Re: [Qemu-devel] [RFC] powernv: CPU compatibility modes don't make sense for powernv

2016-10-28 Thread David Gibson
On Fri, Oct 28, 2016 at 11:32:43AM +0200, Greg Kurz wrote: > On Fri, 28 Oct 2016 13:05:01 +1100 > David Gibson wrote: > > > powernv has some code (derived from the spapr equivalent) used in device > > tree generation which depends on the CPU's compatibility mode / logical > > PVR. However, compa

[Qemu-devel] pci-assign fails with read error on config-space file

2016-10-28 Thread Henning Schild
Hey, i am running an unusual setup where i assign pci devices behind the back of libvirt. I have two options to do that: 1. a wrapper script for qemu that takes care of suid-root and appends arguments for pci-assign 2. virsh qemu-monitor-command ... 'device_add pci-assign...' I know i should prob

Re: [Qemu-devel] [PATCH V8 0/2] Add option to configure guest vPMU

2016-10-28 Thread Peter Maydell
On 27 October 2016 at 21:53, Wei Huang wrote: > On 10/27/2016 12:15 PM, Peter Maydell wrote: >> Right, but at the moment the commit message says there's no >> PMU in TCG at all. What's missing that causes there not to be one? >> (I think maybe we just don't report it in the feature register, >> wh

[Qemu-devel] [PULL 0/6] ui patch queue.

2016-10-28 Thread Gerd Hoffmann
2.8 freeze deadline. please pull, Gerd The following changes since commit db4df20de86c6e8ecd6c9f042c029ffb9f9cddac: trace: Fix 'char **' compilation error in simple backend (2016-10-27 19:24:15 +0100) are available in the git repository at: git://git.kraxel.org/qemu tags/pull-ui

[Qemu-devel] [PULL 6/6] curses: Use cursesw instead of curses

2016-10-28 Thread Gerd Hoffmann
From: Samuel Thibault Use ncursesw package instead of curses on non-mingw, and check a few functions. Also take cflags from pkg-config, since cursesw headers may be in a separate, non-default directory. Signed-off-by: Samuel Thibault Message-id: 20161015195308.20473-3-samuel.thiba...@ens-lyon.o

[Qemu-devel] [PULL 3/6] gtk: fix compilation warning with gtk 3.22.2

2016-10-28 Thread Gerd Hoffmann
From: Alberto Garcia gdk_screen_get_width() is deprecated since gtk 3.22.2, use gdk_monitor_get_geometry() instead if it's available. Signed-off-by: Alberto Garcia Message-id: 20161026152108.12364-1-be...@igalia.com Signed-off-by: Gerd Hoffmann --- ui/gtk.c | 23 +-- 1 fil

[Qemu-devel] [PULL 4/6] ui/gtk: Fix non-working DELETE key

2016-10-28 Thread Gerd Hoffmann
From: Thomas Huth GTK generates key events for the delete key with key->string[0] = 0x7f ... but this does not work right with the readline_handle_byte() function in util/readline.c, since this treats the keycode 127 as backspace. So let's add a special case for the GTK delete key to make this ke

[Qemu-devel] [PULL 2/6] Defer BrlAPI tty acquisition to when guest starts using device

2016-10-28 Thread Gerd Hoffmann
From: Samuel Thibault We do not want to catch the BrlAPI input/ouput immediately, but only when the guest has started discussing withour virtual device. This notably fixes input before the guest driver has started. Signed-off-by: Samuel Thibault Signed-off-by: Gerd Hoffmann --- backends/baum

[Qemu-devel] [PULL 5/6] curses: fix left/right arrow translation

2016-10-28 Thread Gerd Hoffmann
From: Samuel Thibault In default VGA font, left/right arrow are glyphs 0x1a and 0x1b, not 0x0a and 0x0b. Signed-off-by: Samuel Thibault Message-id: 20161015195308.20473-2-samuel.thiba...@ens-lyon.org Signed-off-by: Gerd Hoffmann --- ui/curses.c | 4 ++-- 1 file changed, 2 insertions(+), 2 del

[Qemu-devel] [PULL 1/6] Add dots keypresses support to the baum braille device

2016-10-28 Thread Gerd Hoffmann
From: Samuel Thibault Signed-off-by: Samuel Thibault Signed-off-by: Gerd Hoffmann --- backends/baum.c | 224 +++- 1 file changed, 125 insertions(+), 99 deletions(-) diff --git a/backends/baum.c b/backends/baum.c index 919844e..dedc520 100644

Re: [Qemu-devel] [PATCH 1/2] hw/arm/spitz: Fix reset handling

2016-10-28 Thread Peter Maydell
On 27 October 2016 at 20:47, Guenter Roeck wrote: > Using the CPU reset handler for resets triggered by writing into > gpio pins other than GPIO01 is not appropriate and does not work, > since the reset triggered by writing into GPIO01 is configurable. > Use a separate reset handler for spitz to r

[Qemu-devel] [PATCH] qxl: Only emit QXL_INTERRUPT_CLIENT_MONITORS_CONFIG on config changes

2016-10-28 Thread Christophe Fergeau
Currently if the client keeps sending the same monitor config to QEMU/spice-server, QEMU will always raise a QXL_INTERRUPT_CLIENT_MONITORS_CONFIG regardless of whether there was a change or not. Guest-side (with fedora 25), the kernel QXL KMS driver will also forward the event to user-space without

Re: [Qemu-devel] [PATCH v10 10/19] vfio iommu: Add blocking notifier to notify DMA_UNMAP

2016-10-28 Thread Kirti Wankhede
On 10/28/2016 1:03 PM, Jike Song wrote: > On 10/27/2016 05:29 AM, Kirti Wankhede wrote: >> >> +/* >> + * This function finds pfn in domain->external_addr_space->pfn_list for >> given >> + * iova range. If pfn exist, notify pfn to registered notifier list. On >> + * receiving notifier callback,

Re: [Qemu-devel] [PATCH v10 10/19] vfio iommu: Add blocking notifier to notify DMA_UNMAP

2016-10-28 Thread Alex Williamson
On Fri, 28 Oct 2016 15:33:58 +0800 Jike Song wrote: > On 10/27/2016 05:29 AM, Kirti Wankhede wrote: > > Added blocking notifier to IOMMU TYPE1 driver to notify vendor drivers > > about DMA_UNMAP. > > Exported two APIs vfio_register_notifier() and vfio_unregister_notifier(). > > Vendor driver shou

Re: [Qemu-devel] [PATCH] tcg: correct 32-bit tcg_gen_ld8s_i64 sign-extension

2016-10-28 Thread Peter Maydell
On 28 October 2016 at 00:35, Joseph Myers wrote: > The version of tcg_gen_ld8s_i64 for 32-bit systems does a load into > the low part of the return value - then attempts a sign extension into > the high part, but wrongly sets the high part to a sign extension of > itself rather than of the low par

Re: [Qemu-devel] [PATCH] qxl: Only emit QXL_INTERRUPT_CLIENT_MONITORS_CONFIG on config changes

2016-10-28 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Type: series Subject: [Qemu-devel] [PATCH] qxl: Only emit QXL_INTERRUPT_CLIENT_MONITORS_CONFIG on config changes Message-id: 20161028122822.18920-1-cferg...@redhat.com === TEST SCRIPT BEGIN === #!/b

[Qemu-devel] [PATCH v4 0/2] allow blockdev-add for NFS

2016-10-28 Thread Ashijeet Acharya
Previously posted series patches: v3: https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg06903.html v2: https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg05844.html v1: https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg04487.html This series adds blockdev-add support for NFS bl

[Qemu-devel] [PATCH v4 2/2] qapi: allow blockdev-add for NFS

2016-10-28 Thread Ashijeet Acharya
Introduce new object 'BlockdevOptionsNFS' in qapi/block-core.json to support blockdev-add for NFS network protocol driver. Also make a new struct NFSServer to support tcp connection. Signed-off-by: Ashijeet Acharya --- qapi/block-core.json | 77 +--

[Qemu-devel] [PATCH v4 1/2] block/nfs: Introduce runtime_opts in NFS

2016-10-28 Thread Ashijeet Acharya
Make NFS block driver use various fine grained runtime_opts. Set .bdrv_parse_filename() to nfs_parse_filename() and introduce two new functions nfs_parse_filename() and nfs_parse_uri() to help parsing the URI. Add a new option "server" which then accepts a new struct NFSServer. "host" is supported

  1   2   3   >