Re: [Qemu-devel] [PATCH] hostmem-file: add a property 'notrunc' to avoid data corruption

2016-10-20 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] hostmem-file: add a property 'notrunc' to avoid data corruption Type: series Message-id:

[Qemu-devel] [PATCH] hostmem-file: add a property 'notrunc' to avoid data corruption

2016-10-20 Thread Haozhong Zhang
If a file is used as the backend of memory-backend-file and its size is not identical to the property 'size', the file will be truncated. For a file used as the backend of vNVDIMM, its data is expected to be persistent and the truncation may corrupt the existing data. A property 'notrunc' is

Re: [Qemu-devel] [PATCH 01/13] pseries: Remove unused callbacks from sPAPR VIO bus state

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:11, David Gibson wrote: > The original QOMification of the spapr VIO devices in 3954d33 "spapr: > convert to QEMU Object Model (v2)" moved some callbacks from the > VIOsPAPRBus structure to the VIOsPAPRDeviceClass. Except, that it > forgot to actually remove them from the

Re: [Qemu-devel] [PATCH v14 17/21] numa: convert to use QObjectInputVisitor for -numa

2016-10-20 Thread Markus Armbruster
"Daniel P. Berrange" writes: > Switch away from using OptsVisitor to parse the -numa > argument processing. This enables use of the modern > list syntax for specifying CPUs. e.g. the old syntax > > -numa node,nodeid=0,cpus=0-3,cpus=8-11,mem=107 > > is equivalent to > >

Re: [Qemu-devel] [PATCH v14 16/21] hmp: support non-scalar properties with object_add

2016-10-20 Thread Markus Armbruster
"Daniel P. Berrange" writes: > The current object_add HMP syntax only allows for > creation of objects with scalar properties, or a list > with a fixed scalar element type. Objects which have > properties that are represented as structs in the QAPI > schema cannot be created

Re: [Qemu-devel] [PATCH 02/13] pseries: Split device tree construction from device tree load

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:11, David Gibson wrote: > spapr_finalize_fdt() both finishes building the device tree for the guest > and loads it into guest memory. For future cleanups, it's going to be > more convenient to do these two things separately. The loading portion is > pretty trivial, so we move it

Re: [Qemu-devel] my attempt at new QEMU website

2016-10-20 Thread Thomas Huth
On 19.10.2016 12:53, Paolo Bonzini wrote: > Hi all, > > another small PSA. :) I have created a small but hopefully useful > example of a new QEMU website at http://qemu-bonzini.rhcloud.com/. The > site aims at providing and also a home for blog posts about QEMU. Looks good to me! Two comments,

Re: [Qemu-devel] [PATCH 1/2] script/clean-includes: added duplicate #include check

2016-10-20 Thread Anand J
send a new patch. please check http://patchwork.ozlabs.org/patch/684533/ Thanks, Anand On Thu, Oct 20, 2016 at 2:38 PM, Thomas Huth wrote: > On 18.10.2016 14:38, Anand J wrote: > > > > > > On Tue, Oct 18, 2016 at 4:03 PM, Anand J >

Re: [Qemu-devel] [PATCH 2/2] clean-up: removed duplicate #includes

2016-10-20 Thread Anand J
send a new patch. please check http://patchwork.ozlabs.org/patch/684532/ Thanks, Anand On Thu, Oct 20, 2016 at 2:41 PM, Thomas Huth wrote: > On 18.10.2016 14:39, Anand J wrote: > > > > > > On Tue, Oct 18, 2016 at 4:03 PM, Anand J >

[Qemu-devel] [PATCH 1/2] tests/stubs: Add a dummy, silent monitor

2016-10-20 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" We need an easy way to silence error_report's that come up in meant-to-fail test cases. The easiest way to do that is to create a monitor instance, and since our stubbed monitor_printf's are slent this causes the errors to disappear.

[Qemu-devel] [PATCH 2/2] test-vmstate: Silence expected errors

2016-10-20 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" test-vmstate has some expected failure tests (from explicitly truncating an input stream); these trigger errors in the migration code that are now reported; silence these errors. Signed-off-by: Dr. David Alan Gilbert ---

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert" writes: > > The problem is that our way of propagating errors actively discourages > > people from adding errors and you're left with lots of useless invented > > errno's. > > error_report makes it

[Qemu-devel] [PATCH v5 3/3] exec: call cpu_exec_exit() from a CPU unrealize common function

2016-10-20 Thread Laurent Vivier
As cpu_exec_exit() mirrors the cpu_exec_realizefn(), rename it as cpu_exec_unrealizefn(). Create and register a cpu_common_unrealizefn() function for the CPU device class and call cpu_exec_unrealizefn() from this function. Remove cpu_exec_exit() from cpu_common_finalize() (which mirrors init,

[Qemu-devel] [PATCH v4 2/5] qapi: auto generate enum value strings

2016-10-20 Thread Lin Ma
Automatically generate enum value strings that containing the acceptable values. (Borrowed Daniel's code.) Signed-off-by: Lin Ma --- scripts/qapi-types.py | 2 ++ scripts/qapi.py | 9 + 2 files changed, 11 insertions(+) diff --git a/scripts/qapi-types.py

[Qemu-devel] [PATCH v4 3/5] qapi: add test case for the generated enum value str

2016-10-20 Thread Lin Ma
Signed-off-by: Lin Ma --- tests/test-qmp-commands.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/tests/test-qmp-commands.c b/tests/test-qmp-commands.c index 81cbe54..9cd61b2 100644 --- a/tests/test-qmp-commands.c +++ b/tests/test-qmp-commands.c @@ -262,6

Re: [Qemu-devel] [Qemu-ppc] (no subject)

2016-10-20 Thread Greg Kurz
On Thu, 20 Oct 2016 17:59:09 +1100 Nicholas Piggin wrote: > Date: Thu, 20 Oct 2016 17:38:24 +1100 > Subject: [PATCH 0/3] ppc: system reset interrupt fixes and new hcall IPI > > Hi, > > We are implementing this new unmaskable IPI hcall for crash dumping > and debugging. I had

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

2016-10-20 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 | 40 ++- include/hw/virtio/virtio-crypto.h | 11

Re: [Qemu-devel] CVE-2016-5403 results in a bug

2016-10-20 Thread yue
i basicly understand the code, i have it done next week. do you have any suggestion? At 2016-10-19 16:58:21, "Yadi Hu" wrote: >On 2016年10月18日 17:40, Stefan Hajnoczi wrote: >> On Tue, Oct 18, 2016 at 10:24 AM, yue wrote: >>> i apply this

Re: [Qemu-devel] [Qemu-ppc] [PATCH 1/3] ppc: fix MSR_ME handling for system reset interrupt

2016-10-20 Thread Cédric Le Goater
On 10/20/2016 08:59 AM, Nicholas Piggin wrote: > Power ISA specifies ME bit handling for system reset interrupt: > > if the interrupt occurred while the thread was in power-saving > mode, set to 1; otherwise not altered > > Signed-off-by: Nicholas Piggin Reviewed-by:

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Daniel P. Berrange (berra...@redhat.com) wrote: >> On Thu, Oct 20, 2016 at 10:55:52AM +0200, Markus Armbruster wrote: >> > "Dr. David Alan Gilbert" writes: >> > >> > > * Daniel P. Berrange (berra...@redhat.com)

Re: [Qemu-devel] qemu master tests/vmstate prints "Failed to load simple/primitive:b_1" etc

2016-10-20 Thread Dr. David Alan Gilbert
* Paolo Bonzini (pbonz...@redhat.com) wrote: > > > On 18/10/2016 11:54, Dr. David Alan Gilbert wrote: > > I've not quite figured it out but we're linked against the stubs/ monitor > > code rather than the real monitor code; and the stubs monitor_vprintf > > discards > > and the stubs

Re: [Qemu-devel] [PATCH v7 13/35] tcg: Add atomic helpers

2016-10-20 Thread Alex Bennée
Richard Henderson writes: > Add all of cmpxchg, op_fetch, fetch_op, and xchg. > Handle both endian-ness, and sizes up to 8. > Handle expanding non-atomically, when emulating in serial. > > Reviewed-by: Emilio G. Cota > Reviewed-by: Alex Bennée

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Thu, Oct 20, 2016 at 11:38 AM Daniel P. Berrange > wrote: > >> On Wed, Oct 19, 2016 at 07:06:16PM +0100, Dr. David Alan Gilbert wrote: >> > * Daniel P. Berrange (berra...@redhat.com) wrote: [...] >> > > If we

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Daniel P. Berrange
On Thu, Oct 20, 2016 at 12:01:08PM +0100, Dr. David Alan Gilbert wrote: > * Markus Armbruster (arm...@redhat.com) wrote: > > "Dr. David Alan Gilbert" writes: > > > > The problem is that our way of propagating errors actively discourages > > > people from adding errors and

[Qemu-devel] [PATCH v4 5/5] object: Add 'help' option for all available backends and properties

2016-10-20 Thread Lin Ma
'-object help' prints available user creatable backends. '-object $typename,help' prints relevant properties. Signed-off-by: Lin Ma --- include/qom/object_interfaces.h | 2 ++ qemu-options.hx | 7 +- qom/object_interfaces.c | 55

[Qemu-devel] [PATCH v4 1/5] qom: Add interface check in object_class_is_abstract

2016-10-20 Thread Lin Ma
Signed-off-by: Lin Ma --- qom/object.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/qom/object.c b/qom/object.c index 7a05e35..4096645 100644 --- a/qom/object.c +++ b/qom/object.c @@ -747,7 +747,11 @@ ObjectClass *object_get_class(Object *obj) bool

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

2016-10-20 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

Re: [Qemu-devel] [PATCH 1/4] tests/test-vmstate.c: Add vBuffer test

2016-10-20 Thread Dr. David Alan Gilbert
* Halil Pasic (pa...@linux.vnet.ibm.com) wrote: > From: Guenther Hutzl > > The unit test test-vmstate.c is missing tests for some of the complex > vmstate macros. This patch adds a new test for VMSTATE_VBUFFER > and VMSTATE_VBUFFER_ALLOC_UINT32. The added test does not

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

2016-10-20 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 | 290

Re: [Qemu-devel] [PATCH 2/4] tests/test-vmstate.c: prove VMStateField.start broken

2016-10-20 Thread Dr. David Alan Gilbert
* Halil Pasic (pa...@linux.vnet.ibm.com) wrote: > > > On 10/18/2016 08:32 PM, Dr. David Alan Gilbert wrote: > >> > "The idea is to remove .start support and this patch should > >> > be reverted, as soon this happens, or even better just > >> > dropped. If however dropping the support for .start

Re: [Qemu-devel] [PATCH 4/4] migration: drop unused VMStateField.start

2016-10-20 Thread Dr. David Alan Gilbert
* Halil Pasic (pa...@linux.vnet.ibm.com) wrote: > The member VMStateField.start was solely used to implement the partial > data migration for VBUFFER data (basically provide migration for a > sub-buffer). However the implementation of this feature seems broken to > me, but this goes unnoticed

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

2016-10-20 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 +-

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Dr. David Alan Gilbert
* Daniel P. Berrange (berra...@redhat.com) wrote: > On Thu, Oct 20, 2016 at 10:55:52AM +0200, Markus Armbruster wrote: > > "Dr. David Alan Gilbert" writes: > > > > > * Daniel P. Berrange (berra...@redhat.com) wrote: > > >> On Wed, Oct 19, 2016 at 02:16:05PM +0200, Markus

[Qemu-devel] [PATCH 0/2] Silence test-vmstate errors

2016-10-20 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" In a1771070e7 I added an error message during migration loading so you could see the field that failed to load. Unfortunately this triggers during the expected-failure case testing in tests/test-vmstate. This silences that error by providing a

[Qemu-devel] alpha platform is missing files after initrd load

2016-10-20 Thread Dennis Luehring
qemu: 2.7.x (git head) platform: Alpha (Clipper) kernel: 4.7.0 gcc: 6.1 i don't know if its an qemu oder linux kernel problem i've got an ~360MB big_initrd.cpio and it sometimes happen(seems so) that there are files missing after the kernel loaded the initrd this does not happen with the same

Re: [Qemu-devel] [PULL v1 00/10] Merge qio 2016/10/19

2016-10-20 Thread Daniel P. Berrange
On Thu, Oct 20, 2016 at 11:44:47AM +0100, Peter Maydell wrote: > On 19 October 2016 at 12:17, Daniel P. Berrange wrote: > > The following changes since commit e8ddc2eae5ccc41f0815e5c43e70cb04a7e67e2e: > > > > Merge remote-tracking branch

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Daniel P. Berrange
On Thu, Oct 20, 2016 at 12:42:01PM +0200, Markus Armbruster wrote: > "Dr. David Alan Gilbert" writes: > > > * Daniel P. Berrange (berra...@redhat.com) wrote: > >> On Thu, Oct 20, 2016 at 10:55:52AM +0200, Markus Armbruster wrote: > >> > >> Our code has increasingly

[Qemu-devel] [PATCH v5 2/3] exec: move cpu_exec_init() calls to realize functions

2016-10-20 Thread Laurent Vivier
Modify all CPUs to call it from XXX_cpu_realizefn() function. Remove all the cannot_destroy_with_object_finalize_yet as unsafe references have been moved to cpu_exec_realizefn(). (tested with QOM command provided by commit 4c315c27) for arm: Setting of cpu->mp_affinity is moved from

Re: [Qemu-devel] [PATCH v4 11/13] pc: add 'etc/boot-cpus' fw_cfg file for machine with more than 255 CPUs

2016-10-20 Thread Igor Mammedov
On Wed, 19 Oct 2016 16:29:29 -0200 Eduardo Habkost wrote: > On Wed, Oct 19, 2016 at 05:18:38PM +0200, Igor Mammedov wrote: > > On Wed, 19 Oct 2016 11:15:46 -0200 > > Eduardo Habkost wrote: > > > > > On Wed, Oct 19, 2016 at 02:05:41PM +0200, Igor

[Qemu-devel] [PATCH v5 1/3] exec: split cpu_exec_init()

2016-10-20 Thread Laurent Vivier
Put in cpu_exec_initfn() what initializes the CPU, and leave in cpu_exec_init() what adds it to the environment. As cpu_exec_initfn() is called by all XX_cpu_initfn(), call it directly in cpu_common_initfn(). cpu_exec_init() is now a realize function, it will be renamed to cpu_exec_realizefn()

[Qemu-devel] [PATCH v5 0/3] Split cpu_exec_init() into an init and a realize part

2016-10-20 Thread Laurent Vivier
Since commit 42ecaba ("target-i386: Call cpu_exec_init() on realize"), , commit 6dd0f83 ("target-ppc: Move cpu_exec_init() call to realize function"), and commit c6644fc ("s390x/cpu: Get rid of side effects when creating a vcpu"), cpu_exec_init() has been moved to realize function for some

[Qemu-devel] [PATCH v4 0/5] object: Add 'help' option for all available backends and properties

2016-10-20 Thread Lin Ma
V3->V4: * drop the code about manually define interface 'user-creatable' as abstract. * add test case for the generated enum value str. * split the code about adding descriptions to class properties into a separate patch. * minor changes in user_creatable_help_func to ignore interface

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Markus Armbruster
"Daniel P. Berrange" writes: > On Thu, Oct 20, 2016 at 12:01:08PM +0100, Dr. David Alan Gilbert wrote: >> * Markus Armbruster (arm...@redhat.com) wrote: >> > "Dr. David Alan Gilbert" writes: >> >> > > The problem is that our way of propagating errors

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

2016-10-20 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 +--

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

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

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Markus Armbruster
"Daniel P. Berrange" writes: > On Thu, Oct 20, 2016 at 12:42:01PM +0200, Markus Armbruster wrote: >> "Dr. David Alan Gilbert" writes: >> >> > * Daniel P. Berrange (berra...@redhat.com) wrote: >> >> On Thu, Oct 20, 2016 at 10:55:52AM +0200, Markus

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

2016-10-20 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

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

2016-10-20 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

Re: [Qemu-devel] [PATCHv3 08/11] tests: Clean up IO handling in ide-test

2016-10-20 Thread Laurent Vivier
On 20/10/2016 05:43, David Gibson wrote: > ide-test uses many explicit inb() / outb() operations for its IO, which > means it's not portable to non-x86 platforms. This cleans it up to use > the libqos PCI accessors instead. > > Signed-off-by: David Gibson > --- >

Re: [Qemu-devel] [Qemu-ppc] [PATCH 1/3] ppc: fix MSR_ME handling for system reset interrupt

2016-10-20 Thread Greg Kurz
On Thu, 20 Oct 2016 17:59:10 +1100 Nicholas Piggin wrote: > Power ISA specifies ME bit handling for system reset interrupt: > > if the interrupt occurred while the thread was in power-saving > mode, set to 1; otherwise not altered > > Signed-off-by: Nicholas Piggin

Re: [Qemu-devel] [Bug 1625295] Re: qemu-arm dies with libarmmem inside ld.so.preload

2016-10-20 Thread Stu
Yup, got it - cheers :) On 20 October 2016 at 08:41, Peter Maydell wrote: > NB that commit 9b6a3ea7a69959416 had a bug (it broke exception return to > Thumb code), so you should also make sure you have commit fb0e8e79a9d77 > which fixes that bug. > > -- > You received

Re: [Qemu-devel] [PULL v1 00/10] Merge qio 2016/10/19

2016-10-20 Thread Peter Maydell
On 19 October 2016 at 12:17, Daniel P. Berrange wrote: > The following changes since commit e8ddc2eae5ccc41f0815e5c43e70cb04a7e67e2e: > > Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into > staging (2016-10-18 09:29:44 +0100) > > are available in

[Qemu-devel] [PATCH v4 4/5] backends: add description for enum class properties

2016-10-20 Thread Lin Ma
Signed-off-by: Lin Ma --- backends/hostmem.c | 4 crypto/secret.c| 4 crypto/tlscreds.c | 4 net/filter.c | 4 4 files changed, 16 insertions(+) diff --git a/backends/hostmem.c b/backends/hostmem.c index 4256d24..25f303d 100644 ---

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

2016-10-20 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 | 351

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

2016-10-20 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

Re: [Qemu-devel] [PATCH 04/13] pseries: Make spapr_create_fdt_skel() get information from machine state

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > Currently spapr_create_fdt_skel() takes a bunch of individual parameters > for various things it will put in the device tree. Some of these can > already be taken directly from sPAPRMachineState. This patch alters it so > that all of them can be taken

[Qemu-devel] [PATCH] audio: intel-hda: check stream entry count during transfer

2016-10-20 Thread P J P
From: Prasad J Pandit Intel HDA emulator uses stream of buffers during DMA data transfers. Each entry has buffer length and buffer pointer position, which are used to derive bytes to 'copy'. If this length and buffer pointer were to be same, 'copy' could be set to

Re: [Qemu-devel] [PATCH v4 1/4] fdc: Add a floppy qbus

2016-10-20 Thread KONRAD Frederic
Hi, Le 20/10/2016 à 09:55, Kevin Wolf a écrit : This adds a qbus to the floppy controller that should contain the floppy drives eventually. At the moment it just exists and is empty. Signed-off-by: Kevin Wolf Reviewed-by: John Snow --- hw/block/fdc.c | 40

Re: [Qemu-devel] my attempt at new QEMU website

2016-10-20 Thread Daniel P. Berrange
On Wed, Oct 19, 2016 at 03:54:22PM -0400, Programmingkid wrote: > > On Wed, Oct 19, 2016 at 12:53:55PM +0200, Paolo Bonzini wrote: > >> Hi all, > >> > >> another small PSA. :) I have created a small but hopefully useful > >> example of a new QEMU website at http://qemu-bonzini.rhcloud.com/. The

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Daniel P. Berrange
On Wed, Oct 19, 2016 at 07:06:16PM +0100, Dr. David Alan Gilbert wrote: > * Daniel P. Berrange (berra...@redhat.com) wrote: > > On Wed, Oct 19, 2016 at 02:16:05PM +0200, Markus Armbruster wrote: > > > "Daniel P. Berrange" writes: > > > > > > > On Wed, Oct 19, 2016 at

Re: [Qemu-devel] [PATCH 00/12] virtio: cleanup ioeventfd start/stop

2016-10-20 Thread Cornelia Huck
On Wed, 19 Oct 2016 22:44:18 +0200 Paolo Bonzini wrote: > On 19/10/2016 17:38, Cornelia Huck wrote: > > - failures are in qom-test for aarch64: > > TEST: tests/qom-test... (pid=23997) > > /aarch64/qom/integratorcp: OK > >

Re: [Qemu-devel] [PULL v1 0/6] Merge qcrypto 2016/10/19

2016-10-20 Thread Daniel P. Berrange
Hi Peter, Please drop this merge request. It seems to cause a regression on old Ubuntu 12.04 which I'm investigating. I'll send a v2 when I've determined the problem. On Wed, Oct 19, 2016 at 11:02:42AM +0100, Daniel P. Berrange wrote: > The following changes since commit

Re: [Qemu-devel] [GIT PULL] pseries: Update SLOF firmware image to 20161019

2016-10-20 Thread Peter Maydell
On 19 October 2016 at 00:15, Alexey Kardashevskiy wrote: > The following changes since commit bd56ff33ed174cbe825a3b9929399ca804ce0f27: > > pseries: Update SLOF firmware image to 20161019 (2016-10-19 10:05:26 +1100) > > are available in the git repository at: > >

Re: [Qemu-devel] [PATCH v9 01/12] vfio: Mediated device Core driver

2016-10-20 Thread Jike Song
On 10/18/2016 05:22 AM, Kirti Wankhede wrote: > diff --git a/drivers/vfio/mdev/mdev_core.c b/drivers/vfio/mdev/mdev_core.c > new file mode 100644 > index ..7db5ec164aeb > --- /dev/null > +++ b/drivers/vfio/mdev/mdev_core.c > @@ -0,0 +1,372 @@ > +/* > + * Mediated device Core Driver > +

Re: [Qemu-devel] [PATCH 08/13] spapr: Consolidate construction of /chosen device tree node

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > For historical reasons, building the /chosen node in the guest device tree > is split across several places and includes both parts which write the DT > sequentially and others which use random access functions. > > This patch consolidates construction

[Qemu-devel] [PATCH v4 1/4] fdc: Add a floppy qbus

2016-10-20 Thread Kevin Wolf
This adds a qbus to the floppy controller that should contain the floppy drives eventually. At the moment it just exists and is empty. Signed-off-by: Kevin Wolf Reviewed-by: John Snow --- hw/block/fdc.c | 40 +++- 1 file

[Qemu-devel] [PATCH v4 3/4] fdc: Move qdev properties to FloppyDrive

2016-10-20 Thread Kevin Wolf
This makes the FloppyDrive qdev object actually useful: Now that it has all properties that don't belong to the controller, you can actually use '-device floppy' and get a working result. Command line semantics is consistent with CD-ROM drives: By default you get a single empty floppy drive. You

[Qemu-devel] [PATCH v4 2/4] fdc: Add a floppy drive qdev

2016-10-20 Thread Kevin Wolf
Floppy controllers automatically create two floppy drive devices in qdev now. (They always created two drives, but managed them only internally.) Signed-off-by: Kevin Wolf Reviewed-by: John Snow --- hw/block/fdc.c | 151

Re: [Qemu-devel] [PATCH v4 2/4] fdc: Add a floppy drive qdev

2016-10-20 Thread KONRAD Frederic
Le 20/10/2016 à 09:55, Kevin Wolf a écrit : Floppy controllers automatically create two floppy drive devices in qdev now. (They always created two drives, but managed them only internally.) Signed-off-by: Kevin Wolf Reviewed-by: John Snow ---

Re: [Qemu-devel] [PATCH 10/13] spapr: Move /event-sources construction to spapr_build_fdt()

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > The /event-sources device tree node is built from spapr_create_fdt_skel(). > As part of consolidating device tree construction to reset time, this moves > it to spapr_build_fdt(). > > Signed-off-by: David Gibson > --- >

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

2016-10-20 Thread Kevin Wolf
Am 20.10.2016 um 10:25 hat Alberto Garcia geschrieben: > On Wed 19 Oct 2016 07:11:20 PM CEST, Kevin Wolf wrote: > >> 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

Re: [Qemu-devel] [Qemu-ppc] [PATCH 3/3] ppc/spapr: implement H_SIGNAL_SYS_RESET

2016-10-20 Thread Thomas Huth
On 20.10.2016 08:59, Nicholas Piggin wrote: > The H_SIGNAL_SYS_RESET hcall allows a guest CPU to raise a system > reset exception on other CPUs in the same guest. > > Signed-off-by: Nicholas Piggin > --- > hw/ppc/spapr_hcall.c | 42 ++

Re: [Qemu-devel] [PATCH v4 RESEND 3/3] IOMMU: enable intel_iommu map and unmap notifiers

2016-10-20 Thread Peter Xu
On Mon, Oct 17, 2016 at 06:44:24PM +0300, Aviv B.D wrote: [...] > @@ -2000,8 +2065,10 @@ static void vtd_iommu_notify_flag_changed(MemoryRegion > *iommu, >IOMMUNotifierFlag new) > { > VTDAddressSpace *vtd_as = container_of(iommu,

Re: [Qemu-devel] [PATCH v14 20/21] net: convert to QObjectInputVisitor for -net/-netdev parsing

2016-10-20 Thread Markus Armbruster
"Daniel P. Berrange" writes: > The -net/-netdev command line parsing code uses OptsVisitor > for parsing options to populate NetLegacy or NetDev struct > respectively. Although those structs have nesting, the > OptsVisitor flattens them, so we must enable compatibility >

[Qemu-devel] [Bug 1625295] Re: qemu-arm dies with libarmmem inside ld.so.preload

2016-10-20 Thread Peter Maydell
NB that commit 9b6a3ea7a69959416 had a bug (it broke exception return to Thumb code), so you should also make sure you have commit fb0e8e79a9d77 which fixes that bug. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] qemu master tests/vmstate prints "Failed to load simple/primitive:b_1" etc

2016-10-20 Thread Halil Pasic
On 10/18/2016 10:03 AM, Markus Armbruster wrote: > Paolo Bonzini writes: > >> > On 17/10/2016 21:15, Dr. David Alan Gilbert wrote: >>> >> * Peter Maydell (peter.mayd...@linaro.org) wrote: >>> On 17 October 2016 at 19:51, Dr. David Alan Gilbert >>>

[Qemu-devel] Intel HDA and ALSA audio driver hogging event loop

2016-10-20 Thread Stefan Hajnoczi
Hi Gerd, A RHEL 7.2 guest sometimes hangs when I play back a wav file on the Intel HDA emulated sound card using the QEMU ALSA audio driver. I can only trigger this with the GTK UI but I think that may be a timing issue. The GTK UI has a GSource .prepare() function that calls recvmsg(2) on a

Re: [Qemu-devel] [PATCH 12/13] spapr: Consolidate construction of /vdevice device tree node

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > Construction of the /vdevice node (and its children) is divided between > spapr_create_fdt_skel() (at init time), which creates the base node, and > spapr_populate_vdevice() (at reset time) which creates the nodes for each > individual virtual device. >

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Daniel P. Berrange (berra...@redhat.com) wrote: >> On Wed, Oct 19, 2016 at 02:16:05PM +0200, Markus Armbruster wrote: >> > "Daniel P. Berrange" writes: >> > >> > > On Wed, Oct 19, 2016 at 11:05:53AM +0100, Dr. David

Re: [Qemu-devel] [PATCH 13/13] spapr: Remove spapr_create_fdt_skel()

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > For historical reasons construction of the guest device tree in spapr is > divided between spapr_create_fdt_skel() which is called at init time, and > spapr_build_fdt() which runs at reset time. Over time, more and more > things have needed to be moved

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Daniel P. Berrange
On Thu, Oct 20, 2016 at 10:55:52AM +0200, Markus Armbruster wrote: > "Dr. David Alan Gilbert" writes: > > > * Daniel P. Berrange (berra...@redhat.com) wrote: > >> On Wed, Oct 19, 2016 at 02:16:05PM +0200, Markus Armbruster wrote: > >> > "Daniel P. Berrange"

Re: [Qemu-devel] Intel HDA and ALSA audio driver hogging event loop

2016-10-20 Thread P J P
Hello Gerd, +-- On Thu, 20 Oct 2016, Stefan Hajnoczi wrote --+ | A RHEL 7.2 guest sometimes hangs when I play back a wav file on the | Intel HDA emulated sound card using the QEMU ALSA audio driver. | | intel_hda_xfer() transfers samples between the emulated card and the | QEMU audio

Re: [Qemu-devel] Intel HDA and ALSA audio driver hogging event loop

2016-10-20 Thread Stefan Hajnoczi
On Thu, Oct 20, 2016 at 10:08 AM, P J P wrote: > Hello Gerd, > > +-- On Thu, 20 Oct 2016, Stefan Hajnoczi wrote --+ > | A RHEL 7.2 guest sometimes hangs when I play back a wav file on the > | Intel HDA emulated sound card using the QEMU ALSA audio driver. > | > |

[Qemu-devel] [PATCH] m25p80: add support for the mx66l1g45g

2016-10-20 Thread Cédric Le Goater
Signed-off-by: Cédric Le Goater --- hw/block/m25p80.c |1 + 1 file changed, 1 insertion(+) Index: qemu-aspeed.git/hw/block/m25p80.c === --- qemu-aspeed.git.orig/hw/block/m25p80.c +++

Re: [Qemu-devel] [PATCH 07/13] spapr: Move construction of /interrupt-controller fdt node

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > Currently the device tree node for the XICS interrupt controller is in > spapr_create_fdt_skel(). As part of consolidating device tree construction > to reset time, this moves it to a function called from spapr_build_fdt(). > > In addition we move the

Re: [Qemu-devel] [PATCH 08/13] spapr: Consolidate construction of /chosen device tree node

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > For historical reasons, building the /chosen node in the guest device tree > is split across several places and includes both parts which write the DT > sequentially and others which use random access functions. > > This patch consolidates construction

[Qemu-devel] [PATCH v4 0/4] fdc: Use separate qdev device for drives

2016-10-20 Thread Kevin Wolf
We have been complaining for a long time about how the floppy controller and floppy drives are combined in a single qdev device and how this makes the device awkward to work with because it behaves different from all other block devices. The latest reason to complain was when I noticed that using

Re: [Qemu-devel] Infinite IRQ injection when trying windows 10

2016-10-20 Thread Dr. David Alan Gilbert
* //-//ani Nemati (hani.nem...@gmail.com) wrote: > /usr/bin/qemu-system-x86_64 -name win10 -S -machine > pc-i440fx-2.6,accel=kvm,usb=off -cpu Haswell -m 2048 -realtime mlock=off > -smp 2,sockets=2,cores=1,threads=1 -uuid > 9b3b910d-f3d1-494e-9471-943fc511d994 -no-user-config -nodefaults -chardev

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

2016-10-20 Thread Alberto Garcia
On Wed 19 Oct 2016 07:11:20 PM CEST, Kevin Wolf wrote: >> 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

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Daniel P. Berrange
On Wed, Oct 19, 2016 at 10:51:07PM +0200, Paolo Bonzini wrote: > > > On 19/10/2016 19:01, Dr. David Alan Gilbert wrote: > > * Paolo Bonzini (pbonz...@redhat.com) wrote: > >> > >> > >> On 18/10/2016 16:01, Daniel P. Berrange wrote: > > > > I already use error_report's in places in

Re: [Qemu-devel] [PATCH 09/13] pseries: Consolidate construction of /rtas device tree node

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > For historical reasons construction of the /rtas node in the device > tree (amongst others) is split into several places. In particular > it's split between spapr_create_fdt_skel(), spapr_build_fdt() and > spapr_rtas_device_tree_setup(). > > In fact, as

Re: [Qemu-devel] [PATCH 11/13] spapr: Move /hypervisor node construction to fdt_build_fdt()

2016-10-20 Thread Thomas Huth
On 20.10.2016 07:12, David Gibson wrote: > Currently the /hypervisor device tree node is constructed in > spapr_create_fdt_skel(). As part of consolidating device tree construction > to reset time, move it to a function called from spapr_build_fdt(). > > Signed-off-by: David Gibson

Re: [Qemu-devel] [PATCH v7 00/35] cmpxchg-based emulation of atomics

2016-10-20 Thread Alex Bennée
Richard Henderson writes: > I believe we're about ready for merge. > > Changes since v6: > * atomic_add-bench update (Emilio). > * Use atomic_load(__nocheck) prior to the cmpxchg in the > reverse-endian atomic addition routines (Emilio). > * tcg temp free-after-free

Re: [Qemu-devel] [PATCH 2/2] clean-up: removed duplicate #includes

2016-10-20 Thread Thomas Huth
On 18.10.2016 14:39, Anand J wrote: > > > On Tue, Oct 18, 2016 at 4:03 PM, Anand J > wrote: > > Some files contain multiple #includes of the same header file. > Removed most of those unnecessary duplicate entries using >

Re: [Qemu-devel] [PATCH v4 1/4] fdc: Add a floppy qbus

2016-10-20 Thread Kevin Wolf
Am 20.10.2016 um 10:06 hat KONRAD Frederic geschrieben: > Hi, > > Le 20/10/2016 à 09:55, Kevin Wolf a écrit : > >This adds a qbus to the floppy controller that should contain the floppy > >drives eventually. At the moment it just exists and is empty. > > > >Signed-off-by: Kevin Wolf

Re: [Qemu-devel] [PATCH] audio: intel-hda: check stream entry count during transfer

2016-10-20 Thread Stefan Hajnoczi
On Thu, Oct 20, 2016 at 01:10:24PM +0530, P J P wrote: > From: Prasad J Pandit > > Intel HDA emulator uses stream of buffers during DMA data > transfers. Each entry has buffer length and buffer pointer > position, which are used to derive bytes to 'copy'. If this > length

Re: [Qemu-devel] [Problem] qemu crash when vhost_net_start

2016-10-20 Thread Xulei (Stone)
> > Hi > > - Original Message - > > Hi, all > > Recently, I have a VM with a vhost-user network card created by qemu 2.6.0. > > Once, I restart OpenVSwitch service > > and start this VM in the same time. I found qemu may probably crash > > with following stack: > > > > (gdb) bt > > #0

[Qemu-devel] [PATCH v4 4/4] qemu-iotests: Test creating floppy drives

2016-10-20 Thread Kevin Wolf
This tests the different supported methods to create floppy drives and how they interact. Signed-off-by: Kevin Wolf --- tests/qemu-iotests/172 | 242 + tests/qemu-iotests/172.out | 1170 tests/qemu-iotests/group |

Re: [Qemu-devel] chardev's and fd's in monitors

2016-10-20 Thread Marc-André Lureau
Hi On Thu, Oct 20, 2016 at 11:38 AM Daniel P. Berrange wrote: > On Wed, Oct 19, 2016 at 07:06:16PM +0100, Dr. David Alan Gilbert wrote: > > * Daniel P. Berrange (berra...@redhat.com) wrote: > > > On Wed, Oct 19, 2016 at 02:16:05PM +0200, Markus Armbruster wrote: > > > >

Re: [Qemu-devel] [PATCH 1/2] script/clean-includes: added duplicate #include check

2016-10-20 Thread Thomas Huth
On 18.10.2016 14:38, Anand J wrote: > > > On Tue, Oct 18, 2016 at 4:03 PM, Anand J > wrote: > > Added script to check duplicate #include entries. This check will scan > and print the files in which duplicate #include entries

  1   2   3   4   >