Re: [Qemu-devel] Interactive Boot Menu: New Fields in IPLB

2017-12-08 Thread Christian Borntraeger
On 12/08/2017 12:56 AM, Collin L. Walling wrote: > Hello, > > I'd like to bring up a formal discussion regarding the utilization of > reserved space > in the IPL Parameter Blocks for s390x guests. Particularly the following: > >  * How should we approach "claiming" bytes so that we do not obst

Re: [Qemu-devel] [virtio-dev] [PATCH v3 0/7] Vhost-pci for inter-VM communication

2017-12-08 Thread Stefan Hajnoczi
On Fri, Dec 8, 2017 at 6:43 AM, Wei Wang wrote: > On 12/08/2017 07:54 AM, Michael S. Tsirkin wrote: >> >> On Thu, Dec 07, 2017 at 06:28:19PM +, Stefan Hajnoczi wrote: >>> >>> On Thu, Dec 7, 2017 at 5:38 PM, Michael S. Tsirkin >>> wrote: On Thu, Dec 07, 2017 at 05:29:14PM +, Stef

Re: [Qemu-devel] [PATCH v3 19/50] qapi: add 'if' to enum members

2017-12-08 Thread Markus Armbruster
A bit more detail in the commit message would make this patch easier to review. Marc-André Lureau writes: > Signed-off-by: Marc-André Lureau > --- > scripts/qapi.py | 39 > - > tests/Makefile.include | 1 - > tests/qapi

Re: [Qemu-devel] [PATCH v2 for-2.12 1/5] s390x/kvm: factor out build_channel_report_mcic() into cpu.h

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 16:31:37 +0100 David Hildenbrand wrote: > We'll need it later on in two places. Refactor it to just indicate the > validity bits. While at it, introduce a define for the used CR14 bit (we'll > also need later on). > > Signed-off-by: David Hildenbrand > --- > target/s390x/cp

[Qemu-devel] [Rfc PATCH] ui: Enable SDL to use dma-buf for direct guest framebuffer rendering

2017-12-08 Thread Tina Zhang
This RFC patch is based on Qemu UI on-going work of rendering guest framebuffer through dma-buf interface, to show a protype of SDL with dma-buf interface. Against branch: work/intel-vgpu Signed-off-by: Tina Zhang --- include/ui/sdl2.h | 11 ui/sdl2-gl.c | 84 +

Re: [Qemu-devel] [Qemu-block] [PATCH for-2.12 0/4] qmp dirty bitmap API

2017-12-08 Thread Kashyap Chamarthy
On Thu, Dec 07, 2017 at 06:33:04PM +0100, Kevin Wolf wrote: > Am 07.12.2017 um 12:56 hat Kashyap Chamarthy geschrieben: [...] > > So from the above interesting discussion, it seems like Kevin is leaning > > towards a single job type that offers 'stream', 'commit', 'backup', and > > 'mirror' funct

Re: [Qemu-devel] [PATCH v2 for-2.12 2/5] s390x/tcg: fix and cleanup mcck injection

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 16:31:38 +0100 David Hildenbrand wrote: > The architecture mode indication wasn't stored. The split of certain > 64bit fields was unnecessary. Also, the complete clock comparator, not > just bit 0-55 (starting at byte 1) was stored. > > We now generate a proper MCIC via the s

Re: [Qemu-devel] [Rfc PATCH] ui: Enable SDL to use dma-buf for direct guest framebuffer rendering

2017-12-08 Thread Zhang, Tina
This patch is using SDL_GL_GetCurrentContext() to get a context and meet an issue with SDL_GL_CONTEXT_EGL which is deprecated since SDL 2.1. Before SDL 2.1, SDL_GL_CONTEXT_EGL can be used with SDL_GL_SetAttribute, so that an EGLContext can be created with SDL_GL_GetCurrentContext(). Since SDL_GL

Re: [Qemu-devel] [PATCH 09/12] crypto: fix stack-buffer-overflow error

2017-12-08 Thread Daniel P. Berrange
On Fri, Dec 08, 2017 at 01:58:22AM +0100, Marc-André Lureau wrote: > ASAN complains about: > > ==8856==ERROR: AddressSanitizer: stack-buffer-overflow on address > 0x7ffd8a1fe168 at pc 0x561136cb4451 bp 0x7ffd8a1fe130 sp 0x7ffd8a1fd8e0 > READ of size 16 at 0x7ffd8a1fe168 thread T0 > #0 0x56113

Re: [Qemu-devel] [Qemu-block] [PATCH v2] qemu-img: Document --force-share / -U

2017-12-08 Thread Stefan Hajnoczi
On Fri, Dec 08, 2017 at 09:44:56AM +0800, Fam Zheng wrote: > +@item --force-share (-U) > + > +If specified, @code{qemu-img} will open the image with shared permissions, > +which makes it less likely to conflict with a running guest's permissions due > +to image locking. For example, this can be use

Re: [Qemu-devel] [PATCH v5 01/23] memattrs: add debug attribute

2017-12-08 Thread Peter Maydell
On 7 December 2017 at 21:20, Brijesh Singh wrote: > On 12/06/2017 04:03 PM, Peter Maydell wrote: >> For instance, if a device gets a debug=1 transaction >> should it refuse to do things like read-clears-bits >> semantics or other side-effects you wouldn't expect >> of debugger accesses? >> > > Sor

Re: [Qemu-devel] [PATCH 08/12] tests: fix migration-test leak

2017-12-08 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@redhat.com) wrote: > Direct leak of 12 byte(s) in 2 object(s) allocated from: > #0 0x7f50d403c850 in malloc (/lib64/libasan.so.4+0xde850) > #1 0x7f50d1ddf98f in vasprintf (/lib64/libc.so.6+0x8098f) > > Signed-off-by: Marc-André Lureau There's an iden

Re: [Qemu-devel] [PATCH 07/12] readline: add a free function

2017-12-08 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@redhat.com) wrote: > Fixes leaks such as: > > Direct leak of 2 byte(s) in 1 object(s) allocated from: > #0 0x7eff58beb850 in malloc (/lib64/libasan.so.4+0xde850) > #1 0x7eff57942f0c in g_malloc ../glib/gmem.c:94 > #2 0x7eff579431cf in g_malloc_n ..

Re: [Qemu-devel] question on virtio ccw

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 19:18:44 +0200 "Michael S. Tsirkin" wrote: > I see this in ccw in qemu: > > if (virtio_set_status(vdev, status) == 0) { > if (vdev->status == 0) { > virtio_ccw_reset_virtio(dev, vdev); > } > if (st

Re: [Qemu-devel] [PATCH 1/2] virtio-blk: make queue size configurable

2017-12-08 Thread Stefan Hajnoczi
On Wed, Dec 06, 2017 at 01:54:00PM -0600, Mark Kanda wrote: > Depending on the configuration, it can be beneficial to adjust the virtio-blk > queue size to something other than the current default of 128. Add a new > property to make the queue size configurable. > > Signed-off-by: Mark Kanda > Re

Re: [Qemu-devel] Interactive Boot Menu: New Fields in IPLB

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 18:56:16 -0500 "Collin L. Walling" wrote: > Hello, > > I'd like to bring up a formal discussion regarding the utilization of > reserved space > in the IPL Parameter Blocks for s390x guests. Particularly the following: > > * How should we approach "claiming" bytes so that w

Re: [Qemu-devel] [PATCH 2/2] virtio-blk: reject configs with logical block size > physical block size

2017-12-08 Thread Stefan Hajnoczi
On Wed, Dec 06, 2017 at 01:54:01PM -0600, Mark Kanda wrote: > virtio-blk logical block size should never be larger than physical block > size because it doesn't make sense to have such configurations. QEMU doesn't > have a way to effectively express this condition; the best it can do is > report th

Re: [Qemu-devel] Interactive Boot Menu: New Fields in IPLB

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 18:56:16 -0500 "Collin L. Walling" wrote: >                       --- The Fields We Need To Store --- > > Currently, we're utilizing 3 bytes out of a 77 byte reserved field in > the IPLB for > CCW and SCSI. We do this because need a way to pass the Qemu command > line opti

Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/9] blockdev: fix QMP 'transaction' with IOThreads

2017-12-08 Thread Stefan Hajnoczi
On Wed, Dec 06, 2017 at 02:45:41PM +, Stefan Hajnoczi wrote: > v2: > * Use StrOrNull for x-blockdev-set-iothread iothread argument [eblake] Thanks for the reviews, Eric. I will wait for one more block layer person (Kevin, Paolo, or Fam?) to review before merging. Stefan signature.asc Desc

Re: [Qemu-devel] [PATCH v5 01/23] memattrs: add debug attribute

2017-12-08 Thread Edgar E. Iglesias
On Fri, Dec 08, 2017 at 09:55:26AM +, Peter Maydell wrote: > On 7 December 2017 at 21:20, Brijesh Singh wrote: > > On 12/06/2017 04:03 PM, Peter Maydell wrote: > >> For instance, if a device gets a debug=1 transaction > >> should it refuse to do things like read-clears-bits > >> semantics or o

Re: [Qemu-devel] iSER transport name is not good

2017-12-08 Thread Dr. David Alan Gilbert
* Charles Kelimod (lichs...@gmail.com) wrote: > Hi Dave and Sahlberg, > > for this type of name:iser:, I completely agree with > you. > my issue is when I create an vm from libvirt, the qemu command line should > be: > -drive > file.driver=iser,file.portal=xx.xx.xx.xx:3260,file.target=iqn.xxx,

Re: [Qemu-devel] iSER transport name is not good

2017-12-08 Thread Daniel P. Berrange
On Fri, Dec 08, 2017 at 03:27:27PM +0800, Charles Kelimod wrote: > Hi Dave and Sahlberg, > > for this type of name:iser:, I completely agree with > you. > my issue is when I create an vm from libvirt, the qemu command line should > be: > -drive > file.driver=iser,file.portal=xx.xx.xx.xx:3260,f

Re: [Qemu-devel] [PATCH v3 1/6] migration: free result string

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Signed-off-by: Juan Quintela > Reported-by: Peter Xu Reviewed-by: Dr. David Alan Gilbert (same as Marc-André's) > --- > tests/migration-test.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/migration-test.c b/t

Re: [Qemu-devel] [PATCH v3 2/6] tests: migration test deprecated commands

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We now test the deprecated commands everytime that we test the new > commands. This makes unnecesary to add tests for deprecated commands. > > Signed-off-by: Juan Quintela > --- > tests/migration-test.c | 32 > 1 fi

[Qemu-devel] [PATCH 0/5] 9pfs/fsdev: trivial code cleanups

2017-12-08 Thread Greg Kurz
This series just do some cleanup to comply with the QEMU coding style. No functional change. -- Greg --- Greg Kurz (5): virtio-9p: move unrealize/realize after virtio_9p_transport definition 9pfs: fix XattrOperations typedef fsdev: fix some type definitions 9pfs: fix some

[Qemu-devel] [PATCH 1/5] virtio-9p: move unrealize/realize after virtio_9p_transport definition

2017-12-08 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 b/hw/9pfs/virtio-9p-device.

[Qemu-devel] [PATCH 2/5] 9pfs: fix XattrOperations typedef

2017-12-08 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 --- a/fsdev/file-op-

[Qemu-devel] [PATCH 5/5] 9pfs: handle: fix type definition

2017-12-08 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 9875f1894cc5..65b12de230c1 100644 --- a/

[Qemu-devel] [PATCH 3/5] fsdev: fix some type definitions

2017-12-08 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 --- a/fsdev/file-op-9p.h +++ b

[Qemu-devel] [PATCH 4/5] 9pfs: fix some type definitions

2017-12-08 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(-) diff --git

Re: [Qemu-devel] [PATCH v2 0/6] block: Fix BlockDriver callbacks in bdrv_drain_all_begin()

2017-12-08 Thread Stefan Hajnoczi
On Wed, Dec 06, 2017 at 11:53:03AM +0100, Kevin Wolf wrote: > I was looking into the drain functions in order to develop them a bit in > the direction that Fam suggested, to unify the code between bdrv_drain() > and bdrv_drain_all() a bit more, and maybe to find a place to take > coroutine locks fo

Re: [Qemu-devel] [PATCH] qdev: drop unused #include "sysemu/iothread.h"

2017-12-08 Thread Stefan Hajnoczi
On Tue, Dec 05, 2017 at 01:39:54PM +, Stefan Hajnoczi wrote: > Commit 1351d1ec89eabebc9fdff20451a62c413d7accc1 ("qdev: drop iothread > property type") forgot to remove this include. > > Signed-off-by: Stefan Hajnoczi > --- > hw/core/qdev-properties-system.c | 1 - > 1 file changed, 1 deletio

[Qemu-devel] [PATCH 1/5] compiler: add a helper for C99 inline functions

2017-12-08 Thread Paolo Bonzini
C99 inline functions are useful whenever a function provide an abstraction that is generally expected to be inlined away, but yet a function pointer might be needed. One such case is function passed to GCC's cleanup attribute. Unfortunately, C99 inline functions clash a bit with -Wredundant-decls

[Qemu-devel] [RFC PATCH 0/5] Scoped locks using attribute((cleanup))

2017-12-08 Thread Paolo Bonzini
This is an attempt to make a C API that resembles the C++ std::unique_lock (mostly untested). The idea is that you can write QEMU_LOCK_GUARD(QemuMutex, guard_name, &some_mutex); instead of qemu_mutex_lock(&some_mutex); ... out: qemu_mutex_unlock(&some_mutex); and the mutex will

[Qemu-devel] [PATCH 5/5] thread-pool: convert to use lock guards

2017-12-08 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- util/thread-pool.c | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/util/thread-pool.c b/util/thread-pool.c index 610646d131..06ada38376 100644 --- a/util/thread-pool.c +++ b/util/thread-pool.c @@ -78,7 +78,7 @@ stat

[Qemu-devel] [PATCH 3/5] qemu-timer: convert to use lock guards

2017-12-08 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- util/qemu-timer.c | 84 +++ 1 file changed, 42 insertions(+), 42 deletions(-) diff --git a/util/qemu-timer.c b/util/qemu-timer.c index 82d56507a2..7a99e0e336 100644 --- a/util/qemu-timer.c +++ b/util/qemu-timer.

[Qemu-devel] [PATCH 2/5] lock-guard: add scoped lock implementation

2017-12-08 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- include/qemu/coroutine.h | 4 ++ include/qemu/lock-guard.h | 99 +++ include/qemu/thread.h | 7 util/Makefile.objs| 1 + util/qemu-thread.c| 17 5 files changed, 128 insertions(+) c

[Qemu-devel] [PATCH 4/5] qht: convert to use lock guards

2017-12-08 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- util/qht.c | 59 +-- 1 file changed, 29 insertions(+), 30 deletions(-) diff --git a/util/qht.c b/util/qht.c index ff4d2e6974..95ac32f990 100644 --- a/util/qht.c +++ b/util/qht.c @@ -248,18 +248,18 @@ void qh

Re: [Qemu-devel] Debugging io deadlock

2017-12-08 Thread Stefan Hajnoczi
On Tue, Dec 05, 2017 at 07:35:36PM +0100, BALATON Zoltan wrote: > On Tue, 5 Dec 2017, Stefan Hajnoczi wrote: > > On Mon, Dec 04, 2017 at 08:22:48PM +0100, BALATON Zoltan wrote: > I don't really know what to look for > so if anyone has any more ideas then please share. I don't have a specific idea,

Re: [Qemu-devel] [PATCH v3 3/6] tests: Add migration precopy test

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Signed-off-by: Juan Quintela OK, although you seem to have chosen a bandwidth 10x what I use in the postcopy test but for the same reason. Reviewed-by: Dr. David Alan Gilbert > --- > tests/migration-test.c | 44 ++

Re: [Qemu-devel] [PATCH v3] rcu: reduce more than 7MB heap memory by malloc_trim()

2017-12-08 Thread Yang Zhong
On Wed, Dec 06, 2017 at 10:48:45AM +0100, Paolo Bonzini wrote: > On 06/12/2017 10:26, Yang Zhong wrote: > > Hello Paolo, > > > > The best option is only trim one time after guest kernel bootup or VM > > bootup, and as for > > hotplug/unhotplug operations during the VM running, the trim stil

Re: [Qemu-devel] [PATCH v3 4/6] tests: Add basic migration precopy tcp test

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > 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 > +++--- >

Re: [Qemu-devel] About the light VM solution!

2017-12-08 Thread Yang Zhong
On Wed, Dec 06, 2017 at 03:11:55PM +, Stefan Hajnoczi wrote: > On Tue, Dec 05, 2017 at 02:35:42PM +0100, Paolo Bonzini wrote: > > On 05/12/2017 13:06, Stefan Hajnoczi wrote: > > > On Tue, Dec 05, 2017 at 02:33:13PM +0800, Yang Zhong wrote: > > Referring to your slides, the remaining points for

Re: [Qemu-devel] [PATCH RFC 0/2] s390x: cut down on unattached devices

2017-12-08 Thread Cornelia Huck
On Thu, 7 Dec 2017 18:15:57 +0100 Halil Pasic wrote: > On 12/07/2017 06:06 PM, Cornelia Huck wrote: > > On Thu, 7 Dec 2017 18:01:46 +0100 > [..] > Regarding the discussion about whether the QOM tree is API and what > exploiters like libvirt should do, Halil asked me to chip in. >

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

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Signed-off-by: Juan Quintela > --- > tests/migration-test.c | 68 > ++ > 1 file changed, 68 insertions(+) > > diff --git a/tests/migration-test.c b/tests/migration-test.c > index 841c89e28a..41dee78a9

Re: [Qemu-devel] [PATCH RFC 1/2] s390x/css: attach css bridge

2017-12-08 Thread Cornelia Huck
On Tue, 28 Nov 2017 14:46:47 +0100 Cornelia Huck wrote: > Logically, the css bridge should be attached to the machine. > > Signed-off-by: Cornelia Huck > --- > hw/s390x/css-bridge.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/hw/s390x/css-bridge.c b/hw/s390x/css-bridge.c > inde

Re: [Qemu-devel] [PATCH v3 6/6] [RFH] tests: Add migration compress threads tests

2017-12-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > 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. How does it fail? Dave > Signed-o

Re: [Qemu-devel] [Qemu-block] [PATCH v2] qemu-img: Document --force-share / -U

2017-12-08 Thread Kevin Wolf
Am 08.12.2017 um 10:54 hat Stefan Hajnoczi geschrieben: > On Fri, Dec 08, 2017 at 09:44:56AM +0800, Fam Zheng wrote: > > +@item --force-share (-U) > > + > > +If specified, @code{qemu-img} will open the image with shared permissions, > > +which makes it less likely to conflict with a running guest's

Re: [Qemu-devel] [Qemu-arm] [PATCH] hw/arm/virt: support 4 serial ports

2017-12-08 Thread Peter Maydell
On 26 October 2017 at 18:31, Jason A. Donenfeld wrote: > On Thu, Oct 26, 2017 at 6:41 PM, Peter Maydell > wrote: >> Those are primarily about migration compatibility -- you should >> be able to live migrate a guest from QEMU 2.6's virt-2.6 machine to >> QEMU 2.7's virt-2.6 machine. That's a stri

[Qemu-devel] [PATCH 0/2] iscsi fixes

2017-12-08 Thread Peter Lieven
fix a bug leaving the allocmap in a wrong state if an UNMAP fails and improve the logging of iscsi failures. Peter Lieven (2): block/iscsi: dont leave allocmap in an invalid state on UNMAP failure block/iscsi: only report an iSCSI Failure if we don't handle it gracefully block/iscsi.c |

[Qemu-devel] [PATCH 1/2] block/iscsi: dont leave allocmap in an invalid state on UNMAP failure

2017-12-08 Thread Peter Lieven
we forgot to set the allocmap to invalid if an UNMAP call fails. Cc: qemu-sta...@nongnu.org Signed-off-by: Peter Lieven --- block/iscsi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/block/iscsi.c b/block/iscsi.c index 4683f3b..c532ec7 100644 --- a/block/iscsi.c ++

[Qemu-devel] [PATCH 2/2] block/iscsi: only report an iSCSI Failure if we don't handle it gracefully

2017-12-08 Thread Peter Lieven
we currently report an "iSCSI Failure" in iscsi_co_generic_cb if the task hasn't completed with SCSI_STATUS_GOOD. However, we expect a failure in some cases and handle it gracefully. This is the case for misaligned UNMAPs and WRITESAME10/16 calls without UNMAP. In this case a failure in the logs ca

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

2017-12-08 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Fri, Dec 01, 2017 at 01:57:49PM +0100, Juan Quintela wrote: > > 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 > > >

[Qemu-devel] [PATCH] docs: update information for TLS certificate management

2017-12-08 Thread Daniel P. Berrange
The current docs for TLS assume only VNC is using TLS. Some of the information is also outdated (ie lacking subject alt name info for certs). Rewrite it to more accurately reflect the current situation. Signed-off-by: Daniel P. Berrange --- qemu-doc.texi | 368 +++

Re: [Qemu-devel] Interactive Boot Menu: New Fields in IPLB

2017-12-08 Thread David Hildenbrand
On 08.12.2017 00:56, Collin L. Walling wrote: > Hello, > > I'd like to bring up a formal discussion regarding the utilization of > reserved space > in the IPL Parameter Blocks for s390x guests. Particularly the following: > > * How should we approach "claiming" bytes so that we do not obstruct

[Qemu-devel] [PATCH 1/3] ide: pass IDEState to trim AIO callback

2017-12-08 Thread Anton Nefedov
It will be needed to handle invalid requests Signed-off-by: Anton Nefedov --- hw/ide/core.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/hw/ide/core.c b/hw/ide/core.c index 471d0c9..02a6b2c 100644 --- a/hw/ide/core.c +++ b/hw/ide/core.c @@ -383,7 +383,7 @@ s

[Qemu-devel] [PATCH 0/3] ide: abort TRIM operation for invalid range

2017-12-08 Thread Anton Nefedov
Started from the separate series discussion (trim statistics) , see http://lists.nongnu.org/archive/html/qemu-devel/2017-12/msg01059.html There is no range check for IDE trim requests now. Such request will likely be rejected by the block layer and count as failed and not an invalid/aborted operat

[Qemu-devel] [PATCH 2/3] ide: move ide_sect_range_ok() up

2017-12-08 Thread Anton Nefedov
to use it without a forward declaration in the commit to follow Signed-off-by: Anton Nefedov --- hw/ide/core.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/hw/ide/core.c b/hw/ide/core.c index 02a6b2c..23c71fa 100644 --- a/hw/ide/core.c +++ b/hw/i

[Qemu-devel] [PATCH 3/3] ide: abort TRIM operation for invalid range

2017-12-08 Thread Anton Nefedov
ATA8-ACS3, 7.9 DATA SET MANAGEMENT - 06h, DMA 7.9.5 Error Outputs If the Trim bit is set to one and: a) the device detects an invalid LBA Range Entry; or b) count is greater than IDENTIFY DEVICE data word 105 (see 7.16.7.55), then the device shall return command ab

Re: [Qemu-devel] [PATCH RFC 0/2] s390x: cut down on unattached devices

2017-12-08 Thread Halil Pasic
On 12/08/2017 12:42 PM, Cornelia Huck wrote: [..] >> In general, I kind of agree with Halil. Unless somewhere in QEMU it is >> documented that the QOM tree is not guaranteed to be stable for >> exploiters, I'd consider is part of the API. libvirt does use at least >> some hardcode

Re: [Qemu-devel] [PATCH] qemu-io: fix EOF Ctrl-D handling in qemu-io readline code

2017-12-08 Thread Dr. David Alan Gilbert
* Daniel P. Berrange (berra...@redhat.com) wrote: > qemu-io puts the TTY into non-canonical mode, which means no EOF processing is > done and thus getchar() will never return the EOF constant. Instead we have to > check for an explicit Ctrl-D, aka 0x4, to detect EOF and exit the qemu-io > shell. T

Re: [Qemu-devel] [PATCH v2 for-2.12 3/5] s390x/tcg: implement SET CLOCK PROGRAMMABLE FIELD

2017-12-08 Thread David Hildenbrand
On 08.12.2017 01:00, Richard Henderson wrote: > On 12/07/2017 07:31 AM, David Hildenbrand wrote: >> +DEF_HELPER_FLAGS_1(sckpf, TCG_CALL_NO_RWG, void, env) > ... >> +/* Set Tod Programmable Field */ >> +void HELPER(sckpf)(CPUS390XState *env) >> +{ >> +uint32_t val = env->regs[0]; >> + >> +if

Re: [Qemu-devel] [PATCH v2 for-2.12 4/5] s390x/tcg: indicate value of TODPR in STCKE

2017-12-08 Thread David Hildenbrand
On 08.12.2017 01:02, Richard Henderson wrote: > On 12/07/2017 07:31 AM, David Hildenbrand wrote: >> +/* 16 bit value store in an uint32_t (only valid bits set) */ >> +tcg_gen_ld32u_i64(todpr, cpu_env, offsetof(CPUS390XState, todpr)); > > Any reason not to use a uint16_t and use tcg_gen_ld1

Re: [Qemu-devel] [PATCH v2 for-2.12 01/10] s390x/tcg: ASI/ASGI are atomic with interlocked-acccess facility 1

2017-12-08 Thread David Hildenbrand
On 08.12.2017 00:41, Richard Henderson wrote: > On 12/07/2017 08:53 AM, David Hildenbrand wrote: >> +static ExitStatus op_asi(DisasContext *s, DisasOps *o) >> +{ >> +o->in1 = tcg_temp_new_i64(); >> +/* Perform the atomic addition in memory. */ >> +tcg_gen_atomic_fetch_add_i64(o->in1, o-

Re: [Qemu-devel] [PATCH v2 for-2.12 02/10] s390x/tcg: ALSI/ALSGI are atomic with interlocked-acccess facility 1

2017-12-08 Thread David Hildenbrand
On 08.12.2017 00:43, Richard Henderson wrote: > On 12/07/2017 08:53 AM, David Hildenbrand wrote: >> We can simply reuse our ASI implementation. Only the way CC is >> calculated differs. >> >> Signed-off-by: David Hildenbrand >> --- >> target/s390x/insn-data.def | 4 ++-- >> 1 file changed, 2 inse

Re: [Qemu-devel] [PATCH v2 for-2.12 4/5] s390x/tcg: indicate value of TODPR in STCKE

2017-12-08 Thread Christian Borntraeger
On 12/08/2017 01:45 PM, David Hildenbrand wrote: > On 08.12.2017 01:02, Richard Henderson wrote: >> On 12/07/2017 07:31 AM, David Hildenbrand wrote: >>> +/* 16 bit value store in an uint32_t (only valid bits set) */ >>> +tcg_gen_ld32u_i64(todpr, cpu_env, offsetof(CPUS390XState, todpr)); >

Re: [Qemu-devel] in a device or CPU instance init/realize, can I rely on something having the BQL or equivalent?

2017-12-08 Thread Igor Mammedov
On Thu, 7 Dec 2017 17:26:53 + Peter Maydell wrote: > On 7 December 2017 at 17:13, Peter Maydell wrote: > > On 7 December 2017 at 17:07, Eduardo Habkost wrote: > >> On Thu, Dec 07, 2017 at 04:53:59PM +, Peter Maydell wrote: > >>> On 7 December 2017 at 16:48, Igor Mammedov wrote: >

Re: [Qemu-devel] in a device or CPU instance init/realize, can I rely on something having the BQL or equivalent?

2017-12-08 Thread Peter Maydell
On 8 December 2017 at 13:16, Igor Mammedov wrote: > TBH: > I do not recall why we have x86 max/host cpu types do feature > loading at realize time instead of at class init like the rest > of static cpu types. class init is too early, IIRC -- it's before KVM has been set up at all. thanks -- P

Re: [Qemu-devel] [PATCH v2 0/9] blockdev: fix QMP 'transaction' with IOThreads

2017-12-08 Thread Kevin Wolf
Am 06.12.2017 um 15:45 hat Stefan Hajnoczi geschrieben: > v2: > * Use StrOrNull for x-blockdev-set-iothread iothread argument [eblake] > > (This is for QEMU 2.12 because this bug is not -rc4 critical) > > Previously AioContext was held across QMP 'transaction' in an attempt to > achieve bdrv_dra

[Qemu-devel] [PATCH v2] qemu-io: fix EOF Ctrl-D handling in qemu-io readline code

2017-12-08 Thread Daniel P. Berrange
qemu-io puts the TTY into non-canonical mode, which means no EOF processing is done and thus getchar() will never return the EOF constant. Instead we have to query the TTY attributes to determine the configured EOF character (usually Ctrl-D / 0x4), and then explicitly check for that value. This fix

Re: [Qemu-devel] [PATCH] qemu-io: fix EOF Ctrl-D handling in qemu-io readline code

2017-12-08 Thread Daniel P. Berrange
On Fri, Dec 08, 2017 at 12:15:21PM +, Dr. David Alan Gilbert wrote: > * Daniel P. Berrange (berra...@redhat.com) wrote: > > qemu-io puts the TTY into non-canonical mode, which means no EOF processing > > is > > done and thus getchar() will never return the EOF constant. Instead we have > > to

Re: [Qemu-devel] [PATCH v3 4/4] ivshmem: Disable irqfd on device reset

2017-12-08 Thread Markus Armbruster
Ladi Prosek writes: > The effects of ivshmem_enable_irqfd() was not undone on device reset. > > This manifested as: > ivshmem_add_kvm_msi_virq: Assertion `!s->msi_vectors[vector].pdev' failed. > > when irqfd was enabled before reset and then enabled again after reset, making > ivshmem_enable_irqf

Re: [Qemu-devel] [PATCH 12/12] WIP ucontext: annotate coroutine stack for ASAN

2017-12-08 Thread Stefan Hajnoczi
On Fri, Dec 08, 2017 at 01:58:25AM +0100, Marc-André Lureau wrote: > Not strictly necessary, but it may help ASAN and remove some false > positives. > > Sadly, this annotation produces an ASAN error: > > $ tests/test-coroutine -p /basic/lifecycle > /basic/lifecycle: ==27655==WARNING: ASan doesn't

Re: [Qemu-devel] About [PULL 20/25] block: Guard against NULL bs->drv

2017-12-08 Thread Max Reitz
On 2017-12-06 10:12, Kevin Wolf wrote: > Am 06.12.2017 um 08:28 hat Kangjie Xi geschrieben: >> Hi, >> >> I encountered a qemu-nbd segfault, finally I found it was caused by >> NULL bs-drv, which is located in block/io.c function bdrv_co_flush >> line 2377: >> >> https://git.qemu.org/?p=qemu.git;a=

Re: [Qemu-devel] [PATCH v3 4/4] ivshmem: Disable irqfd on device reset

2017-12-08 Thread Ladi Prosek
On Fri, Dec 8, 2017 at 2:36 PM, Markus Armbruster wrote: > Ladi Prosek writes: > >> The effects of ivshmem_enable_irqfd() was not undone on device reset. >> >> This manifested as: >> ivshmem_add_kvm_msi_virq: Assertion `!s->msi_vectors[vector].pdev' failed. >> >> when irqfd was enabled before res

Re: [Qemu-devel] [PATCH v7 for-2.12 24/25] block/curl: Implement bdrv_refresh_filename()

2017-12-08 Thread Max Reitz
On 2017-12-05 11:31, Alberto Garcia wrote: > On Mon 04 Dec 2017 07:26:02 PM CET, Max Reitz wrote: +static void curl_refresh_filename(BlockDriverState *bs) +{ +BDRVCURLState *s = bs->opaque; + +if (!s->sslverify || s->cookie || +s->username || s->passwor

Re: [Qemu-devel] About [PULL 20/25] block: Guard against NULL bs->drv

2017-12-08 Thread Kevin Wolf
Am 08.12.2017 um 14:39 hat Max Reitz geschrieben: > On 2017-12-06 10:12, Kevin Wolf wrote: > > Am 06.12.2017 um 08:28 hat Kangjie Xi geschrieben: > >> Hi, > >> > >> I encountered a qemu-nbd segfault, finally I found it was caused by > >> NULL bs-drv, which is located in block/io.c function bdrv_co

Re: [Qemu-devel] [PATCH v7 for-2.12 24/25] block/curl: Implement bdrv_refresh_filename()

2017-12-08 Thread Daniel P. Berrange
On Fri, Dec 08, 2017 at 02:47:52PM +0100, Max Reitz wrote: > On 2017-12-05 11:31, Alberto Garcia wrote: > > On Mon 04 Dec 2017 07:26:02 PM CET, Max Reitz wrote: > +static void curl_refresh_filename(BlockDriverState *bs) > +{ > +BDRVCURLState *s = bs->opaque; > + > +

Re: [Qemu-devel] [PATCH v3 20/50] qapi-event: add 'if' condition to generated enum

2017-12-08 Thread Markus Armbruster
Marc-André Lureau writes: > Add condition to QAPIEvent enum members based on the event 'if'. > > Signed-off-by: Marc-André Lureau > --- > scripts/qapi-event.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/qapi-event.py b/scripts/qapi-event.py > index 38f42648

Re: [Qemu-devel] [PATCH v7 for-2.12 05/25] block: Respect backing bs in bdrv_refresh_filename

2017-12-08 Thread Max Reitz
On 2017-12-05 14:27, Alberto Garcia wrote: > On Mon 20 Nov 2017 09:09:44 PM CET, Max Reitz wrote: >> @@ -5016,6 +5016,7 @@ void bdrv_refresh_filename(BlockDriverState *bs) >> >> opts = qdict_new(); >> has_open_options = append_open_options(opts, bs); >> +has_open_options

Re: [Qemu-devel] About [PULL 20/25] block: Guard against NULL bs->drv

2017-12-08 Thread Max Reitz
On 2017-12-08 14:51, Kevin Wolf wrote: > Am 08.12.2017 um 14:39 hat Max Reitz geschrieben: >> On 2017-12-06 10:12, Kevin Wolf wrote: >>> Am 06.12.2017 um 08:28 hat Kangjie Xi geschrieben: Hi, I encountered a qemu-nbd segfault, finally I found it was caused by NULL bs-drv, which

Re: [Qemu-devel] [PATCH v3 20/50] qapi-event: add 'if' condition to generated enum

2017-12-08 Thread Markus Armbruster
Markus Armbruster writes: > Marc-André Lureau writes: > >> Add condition to QAPIEvent enum members based on the event 'if'. >> >> Signed-off-by: Marc-André Lureau >> --- >> scripts/qapi-event.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/scripts/qapi-event.py b

Re: [Qemu-devel] [PATCH v2] msix: don't mask already masked vectors on reset

2017-12-08 Thread Marcel Apfelbaum
- Original Message - > From: "Ladi Prosek" > To: qemu-devel@nongnu.org > Cc: mar...@redhat.com, ge...@hostfission.com, m...@redhat.com, "alex > williamson" > Sent: Friday, December 8, 2017 8:31:13 AM > Subject: [PATCH v2] msix: don't mask already masked vectors on reset > > msix_mask_

Re: [Qemu-devel] [PATCH v7 for-2.12 24/25] block/curl: Implement bdrv_refresh_filename()

2017-12-08 Thread Max Reitz
On 2017-12-08 14:53, Daniel P. Berrange wrote: > On Fri, Dec 08, 2017 at 02:47:52PM +0100, Max Reitz wrote: >> On 2017-12-05 11:31, Alberto Garcia wrote: >>> On Mon 04 Dec 2017 07:26:02 PM CET, Max Reitz wrote: >> +static void curl_refresh_filename(BlockDriverState *bs) >> +{ >> +BD

[Qemu-devel] [PATCH] get_phys_addr_pmsav7: Support AP=0b111 for v7M

2017-12-08 Thread Peter Maydell
For PMSAv7, the v7A/R Arm ARM defines that setting AP to 0b111 is an UNPREDICTABLE reserved combination. However, for v7M this value is documented as having the same behaviour as 0b110: read-only for both privileged and unprivileged. Accept this value on an M profile core rather than treating it as

[Qemu-devel] [PATCH] hw/intc/armv7m: Support byte and halfword accesses to CFSR

2017-12-08 Thread Peter Maydell
The Configurable Fault Status Register for ARMv7M and v8M is supposed to be byte and halfword accessible, but we were only implementing word accesses. Add support for the other access sizes, which are used by the Zephyr RTOS. Signed-off-by: Peter Maydell Reported-by: Andy Gross --- hw/intc/armv

[Qemu-devel] [PATCH] docs: Document -object tls-creds-x509 priority=xxx

2017-12-08 Thread Christophe Fergeau
This was added in 13f1243, but is missing from qemu-options.hx Signed-off-by: Christophe Fergeau --- qemu-options.hx | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/qemu-options.hx b/qemu-options.hx index f11c4ac960..118784ceb7 100644 --- a/qemu-options.hx +++ b/q

Re: [Qemu-devel] [PATCH] docs: Document -object tls-creds-x509 priority=xxx

2017-12-08 Thread Daniel P. Berrange
On Fri, Dec 08, 2017 at 03:14:30PM +0100, Christophe Fergeau wrote: > This was added in 13f1243, but is missing from qemu-options.hx > > Signed-off-by: Christophe Fergeau > --- > qemu-options.hx | 11 ++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/qemu-options.hx

Re: [Qemu-devel] [Qemu-block] [PATCH for-2.12 0/4] qmp dirty bitmap API

2017-12-08 Thread Max Reitz
On 2017-12-07 23:47, John Snow wrote: > > > On 12/07/2017 06:56 AM, Kashyap Chamarthy wrote: >> On Tue, Nov 21, 2017 at 07:10:19PM -0500, John Snow wrote: >>> On 11/21/2017 12:23 PM, Kevin Wolf wrote: >> >> [...] # Snip lot of good discussion. >> On another note, I would double check before

[Qemu-devel] [PATCH] tests: virtio-9p: fix ISR dependence

2017-12-08 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 --- tests/virtio-9p-test.c | 30 +++--- 1 file changed, 11 insertions(+), 19 deletions(-) diff --git a/tests/virtio-9p-test.c b/t

Re: [Qemu-devel] [PATCH 3/5] qemu-timer: convert to use lock guards

2017-12-08 Thread Stefan Hajnoczi
On Fri, Dec 08, 2017 at 11:55:51AM +0100, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > util/qemu-timer.c | 84 > +++ > 1 file changed, 42 insertions(+), 42 deletions(-) > > diff --git a/util/qemu-timer.c b/util/qemu-timer.c > in

Re: [Qemu-devel] [PATCH 4/5] qht: convert to use lock guards

2017-12-08 Thread Stefan Hajnoczi
On Fri, Dec 08, 2017 at 11:55:52AM +0100, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > util/qht.c | 59 +-- > 1 file changed, 29 insertions(+), 30 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP si

Re: [Qemu-devel] [virtio-dev] [PATCH v3 0/7] Vhost-pci for inter-VM communication

2017-12-08 Thread Michael S. Tsirkin
On Fri, Dec 08, 2017 at 06:08:05AM +, Stefan Hajnoczi wrote: > On Thu, Dec 7, 2017 at 11:54 PM, Michael S. Tsirkin wrote: > > On Thu, Dec 07, 2017 at 06:28:19PM +, Stefan Hajnoczi wrote: > >> On Thu, Dec 7, 2017 at 5:38 PM, Michael S. Tsirkin wrote: > >> > On Thu, Dec 07, 2017 at 05:29:14

Re: [Qemu-devel] [PATCH v2] iotests: fix 197 for vpc

2017-12-08 Thread Kevin Wolf
Am 05.12.2017 um 02:08 hat John Snow geschrieben: > VPC has some difficulty creating geometries of particular size. > However, we can indeed force it to use a literal one, so let's > do that for the sake of test 197, which is testing some specific > offsets. > > Signed-off-by: John Snow > Reviewe

Re: [Qemu-devel] [Qemu-block] [PATCH] blockdev-backup: enable non-root nodes for backup

2017-12-08 Thread Max Reitz
On 2017-12-05 01:48, John Snow wrote: > > > On 12/04/2017 05:21 PM, Max Reitz wrote: >> On 2017-12-04 23:15, John Snow wrote: >>> >>> >>> On 12/01/2017 02:41 PM, Max Reitz wrote: ((By the way, I don't suppose that's how it should work... But I don't suppose that we want propagation of

Re: [Qemu-devel] [RFC] vhost: check if ring mapping is still valid when building memmap

2017-12-08 Thread Igor Mammedov
On Thu, 7 Dec 2017 18:17:51 + "Dr. David Alan Gilbert" wrote: > * Igor Mammedov (imamm...@redhat.com) wrote: > > vhost_verify_ring_mappings() were used to verify that > > rings are still accessible and related memory hasn't > > been moved after flatview is updated. > > > > It were doing chec

Re: [Qemu-devel] [PATCH for 2.11] target/sh4/translate.c: fix TCG leak during gusa sequence

2017-12-08 Thread Alex Bennée
Peter Maydell writes: > On 6 December 2017 at 15:28, Alex Bennée wrote: >> This fixes bug #1735384 while running java under qemu-sh4. When debug >> was enabled it showed a problem with TCG temps. Once fixed I was able >> to run java -version normally. >> > > I don't think this merits holding up

Re: [Qemu-devel] in a device or CPU instance init/realize, can I rely on something having the BQL or equivalent?

2017-12-08 Thread Igor Mammedov
On Fri, 8 Dec 2017 13:19:27 + Peter Maydell wrote: > On 8 December 2017 at 13:16, Igor Mammedov wrote: > > TBH: > > I do not recall why we have x86 max/host cpu types do feature > > loading at realize time instead of at class init like the rest > > of static cpu types. > > class init i

Re: [Qemu-devel] [PATCH] vhost: fix crash on virtio_error while device stop

2017-12-08 Thread Ilya Maximets
On 07.12.2017 20:27, Michael S. Tsirkin wrote: > On Thu, Dec 07, 2017 at 09:39:36AM +0300, Ilya Maximets wrote: >> On 06.12.2017 19:45, Michael S. Tsirkin wrote: >>> On Wed, Dec 06, 2017 at 04:06:18PM +0300, Ilya Maximets wrote: In case virtio error occured after vhost_dev_close(), qemu will c

[Qemu-devel] [PATCH 0/3] hw/arm/virt: Add another UART

2017-12-08 Thread Peter Maydell
This patchset adds a second UART to the ARM virt board, which can be useful in some cases where you'd like a second comms channel but don't want have to deal with PCI to get it. Patch 3 attempts to add it to the ACPI tables, but this is entirely untested and pure guesswork, since I don't really kn

  1   2   3   >