[Qemu-devel] [PATCH 07/15] xen: Prepare xendev qtail to be shared with frontends

2016-10-04 Thread Emil Condrea
* move xendevs qtail to xen_pvdev.c * change xen_be_get_xendev to use a new function: xen_pv_insert_xendev Signed-off-by: Emil Condrea --- hw/xen/xen_backend.c | 51 +- hw/xen/xen_pvdev.c| 57 +++

[Qemu-devel] [PATCH 03/15] xen: Create a new file xen_pvdev.c

2016-10-04 Thread Emil Condrea
The purpose of the new file is to store generic functions shared by frontend and backends such as xenstore operations, xendevs. Signed-off-by: Quan Xu Signed-off-by: Emil Condrea --- hw/xen/Makefile.objs | 2 +- hw/xen/xen_backend.c | 126 +---

Re: [Qemu-devel] [PATCH v2] vfio: Add support for mmapping sub-page MMIO BARs

2016-10-04 Thread Yongji Xie
On 2016/10/1 0:05, Paolo Bonzini wrote: On 14/09/2016 00:55, Alex Williamson wrote: Paolo, as the reigning memory API expert, do you see any issues with this? Expanding the size of a container MemoryRegion and the contained mmap'd subregion and manipulating priorities so that we don't interfer

[Qemu-devel] [PATCH 14/15] xen: Rename xen_be_del_xendev

2016-10-04 Thread Emil Condrea
Prepare xen_be_del_xendev to be shared with frontends: * xen_be_del_xendev -> xen_pv_del_xendev Signed-off-by: Emil Condrea --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a

[Qemu-devel] [PATCH 11/15] xen: Rename xen_be_send_notify

2016-10-04 Thread Emil Condrea
Prepare xen_be_send_notify to be shared with frontends: * xen_be_send_notify -> xen_pv_send_notify Signed-off-by: Emil Condrea --- hw/block/xen_disk.c| 4 ++-- hw/char/xen_console.c | 4 ++-- hw/display/xenfb.c | 8 hw/net/xen_nic.c | 4 ++-- hw/usb/xen-u

[Qemu-devel] [PATCH 10/15] xen: Rename xen_be_unbind_evtchn

2016-10-04 Thread Emil Condrea
Prepare xen_be_unbind_evtchn to be shared with frontends: * xen_be_unbind_evtchn -> xen_pv_unbind_evtchn Signed-off-by: Emil Condrea --- hw/block/xen_disk.c| 2 +- hw/char/xen_console.c | 2 +- hw/display/xenfb.c | 2 +- hw/net/xen_nic.c | 2 +- hw/usb/xen-usb.c

[Qemu-devel] [PATCH 09/15] xen: Rename xen_be_printf to xen_pv_printf

2016-10-04 Thread Emil Condrea
Prepare xen_be_printf to be used by both backend and frontends: * xen_be_printf -> xen_pv_printf Signed-off-by: Emil Condrea --- hw/block/xen_disk.c| 58 +++--- hw/char/xen_console.c | 10 hw/display/xenfb.c | 42

[Qemu-devel] [PATCH 13/15] xen: Rename xen_be_find_xendev

2016-10-04 Thread Emil Condrea
Prepare xen_be_find_xendev to be shared with frontends: * xen_be_find_xendev -> xen_pv_find_xendev Signed-off-by: Emil Condrea --- hw/display/xenfb.c | 4 ++-- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 4 files changed, 5 ins

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 1/6] libqos: add PPC64 PCI support

2016-10-04 Thread Greg Kurz
On Tue, 4 Oct 2016 11:24:54 +1100 David Gibson wrote: > On Mon, Oct 03, 2016 at 04:03:14PM +0200, Greg Kurz wrote: > > On Mon, 3 Oct 2016 13:23:27 +0200 > > Cédric Le Goater wrote: > > > > > On 09/29/2016 07:27 AM, David Gibson wrote: > > > > On Wed, Sep 28, 2016 at 08:51:28PM +0200, Lauren

Re: [Qemu-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)

2016-10-04 Thread Emil Condrea
Anthony, I've split the reorganization and code style issues in a separate patch series which I've posted today: http://markmail.org/message/feu6u3f7py5vt77v Thanks On Mon, Jul 25, 2016 at 5:09 PM, Anthony PERARD wrote: > On Sun, Jul 10, 2016 at 02:47:31PM +0300, Emil Condrea wrote: >> Emil Co

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 1/6] libqos: add PPC64 PCI support

2016-10-04 Thread Greg Kurz
On Tue, 4 Oct 2016 08:58:35 +0200 Greg Kurz wrote: > On Tue, 4 Oct 2016 11:24:54 +1100 > David Gibson wrote: > > > On Mon, Oct 03, 2016 at 04:03:14PM +0200, Greg Kurz wrote: > > > On Mon, 3 Oct 2016 13:23:27 +0200 > > > Cédric Le Goater wrote: > > > > > > > On 09/29/2016 07:27 AM, David

Re: [Qemu-devel] top(1) utility implementation in QEMU

2016-10-04 Thread Markus Armbruster
Fam Zheng writes: > On Fri, 09/30 19:08, Markus Armbruster wrote: >> Fam Zheng writes: >> >> > On Mon, 09/26 17:28, Daniel P. Berrange wrote: >> >> On Mon, Sep 26, 2016 at 07:14:33PM +0530, prashanth sunder wrote: >> >> > Hi All, >> >> > >> >> > Summary of the discussion and different approach

Re: [Qemu-devel] [PATCH v12 2/2] docs: Add a generic loader explanation document

2016-10-04 Thread Markus Armbruster
Alistair Francis writes: > On Thu, Sep 29, 2016 at 10:36 PM, Markus Armbruster wrote: >> Alistair Francis writes: >> >>> On Thu, Sep 29, 2016 at 2:24 AM, Markus Armbruster >>> wrote: Alistair Francis writes: > Signed-off-by: Alistair Francis > Reviewed-by: Peter Maydell >

Re: [Qemu-devel] [PATCH V3] hw/iommu: Fix problems reported by Coverity scan

2016-10-04 Thread Paolo Bonzini
On 04/10/2016 06:03, David Kiarie wrote: > Signed-off-by: David Kiarie > --- > hw/i386/amd_iommu.c | 22 +- > 1 file changed, 13 insertions(+), 9 deletions(-) > > diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c > index 023de52..47b79d9 100644 > --- a/hw/i386/amd_iomm

Re: [Qemu-devel] [PATCH 01/12] virtio: add VIRTIO_DEF_DEVICE_VMSD macro

2016-10-04 Thread Halil Pasic
On 10/03/2016 05:24 PM, Paolo Bonzini wrote: > > > On 03/10/2016 15:34, Halil Pasic wrote: >> Hi Paolo, >> >> I'm sorry, but I do not get it quite yet, or more exactly I have the >> feeling I did not manage to bring my point over. So I will try with >> more details. >> >> On 10/03/2016 01:29 PM

Re: [Qemu-devel] [PATCH] hw/iommu: Fix problems reported by Coverity scan

2016-10-04 Thread Markus Armbruster
Stefan Weil writes: > Hi, > > On 10/01/16 17:57, David Kiarie wrote: >> Signed-off-by: David Kiarie >> --- >> hw/i386/amd_iommu.c | 12 >> 1 file changed, 8 insertions(+), 4 deletions(-) >> >> diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c >> index 023de52..815d45f 100644 >

Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed

2016-10-04 Thread Paolo Bonzini
On 04/10/2016 08:43, Emil Condrea wrote: > xen_be_frontend_changed -> xen_fe_frontend_changed This is not correct. The front-end is implemented in the guest domain, while the back-end is implemented in the dom0 or stubdom. This function processes *in the backed* a notification that the fronten

Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed

2016-10-04 Thread Paolo Bonzini
On 04/10/2016 10:06, Paolo Bonzini wrote: > > > On 04/10/2016 08:43, Emil Condrea wrote: >> xen_be_frontend_changed -> xen_fe_frontend_changed > > This is not correct. The front-end is implemented in the guest domain, > while the back-end is implemented in the dom0 or stubdom. > > This funct

Re: [Qemu-devel] [Bug 1626972] Re: [PATCH] util: secure memfd_create fallback mechanism

2016-10-04 Thread Daniel P. Berrange
On Mon, Oct 03, 2016 at 04:15:55PM -0300, Rafael David Tinoco wrote: > Yes, definitely. Check this: [snip] So in that case, I think we must add ability to specify an explicit path that apps can use *regardles* of whether memfd support exists or not. > > On Oct 03, 2016, at 15:46, Rafael David Ti

Re: [Qemu-devel] [PATCH v4 03/35] exec: Avoid direct references to Int128 parts

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > exec.c| 4 ++-- > include/qemu/int128.h | 10 ++ > 2 files changed, 12 insertions(+), 2 deletions(-) > > diff --git a/exec.c b/exec.c > index c8389f9..cc2997e 100644 > ---

Re: [Qemu-devel] [PATCH v4 05/35] int128: Add int128_make128

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Allows Int128 to be used more generally, rather than having to > begin with 64-bit inputs and accumulate. > > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > include/qemu/int128.h | 20 +++- > 1 file changed, 15 insertions(+), 5 d

Re: [Qemu-devel] [PATCH v4 04/35] int128: Use __int128 if available

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > include/qemu/int128.h | 135 > +- > tests/test-int128.c | 22 > 2 files changed, 145 insertions(+), 12 deletions(-) > > diff --git a/inc

Re: [Qemu-devel] [PATCH v4 08/35] cputlb: Replace SHIFT with DATA_SIZE

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > cputlb.c | 16 > softmmu_template.h | 7 ++- > 2 files changed, 10 insertions(+), 13 deletions(-) > > diff --git a/cputlb.c b/cputlb.c > index 3c99c34..5575b73 10064

Re: [Qemu-devel] [PATCH v4 09/35] cputlb: Move probe_write out of softmmu_template.h

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > cputlb.c | 21 + > softmmu_template.h | 23 --- > 2 files changed, 21 insertions(+), 23 deletions(-) > > diff --git a/cputlb.c b/cputlb.c > index 55

[Qemu-devel] [PATCH] block: use bdrv_add_before_write_notifier

2016-10-04 Thread Paolo Bonzini
Register the notifier using the specific API for block devices. Signed-off-by: Paolo Bonzini --- block/write-threshold.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/block/write-threshold.c b/block/write-threshold.c index cc2ca71..0bd1a01 100644 --- a/block/write-thresho

Re: [Qemu-devel] [SeaBIOS] [PATCH 5/5] [wip] sercon: initial split-output implementation

2016-10-04 Thread Gerd Hoffmann
Hi, > Interesting. I'm curious how the memory scan works, because I didn't > think there was any way to find the vga entry point except from the > int10 vector. Run the init code in emulator? > Were you looking to include this series in SeaBIOS v1.10? We can > either delay the release or pus

Re: [Qemu-devel] [PATCH v4 10/35] cputlb: Remove includes from softmmu_template.h

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > We already include exec/address-spaces.h and exec/memory.h in > cputlb.c; the include of qemu/timer.h appears to be a fossil. > > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > softmmu_template.h | 4 > 1 file changed, 4 deletions(-) > > d

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Kevin Wolf
Am 03.10.2016 um 13:07 hat Tomáš Golembiovský geschrieben: > > > > > +if (((bs->drv != &bdrv_file) || !bs->read_only) && > > > > > > > > Why the check against bdrv_file ? > > > > > > To limit it only to files. Maybe there is better way to do that? The > > > devices have a nasty habit to

Re: [Qemu-devel] [PATCH v5 12/20] trace: don't abort qemu if ftrace can't be initialized

2016-10-04 Thread Daniel P. Berrange
On Mon, Oct 03, 2016 at 02:59:02PM +0100, Stefan Hajnoczi wrote: > On Wed, Sep 28, 2016 at 02:08:15PM +0100, Daniel P. Berrange wrote: > > If the ftrace backend is compiled into QEMU, any attempt > > to start QEMU while non-root will fail due to the > > inability to open /sys/kernel/debug/tracing/t

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Daniel P. Berrange
On Tue, Oct 04, 2016 at 10:57:49AM +0200, Kevin Wolf wrote: > Am 03.10.2016 um 13:07 hat Tomáš Golembiovský geschrieben: > > > > > > +if (((bs->drv != &bdrv_file) || !bs->read_only) && > > > > > > > > > > Why the check against bdrv_file ? > > > > > > > > To limit it only to files. Maybe

Re: [Qemu-devel] VNC - broken initial resolution

2016-10-04 Thread Gerd Hoffmann
On Mo, 2016-10-03 at 19:58 +0200, Gerhard Wiesinger wrote: > Hello, > > Since the last update (git log 3b71ec8..c5d128f) it looks like that the > inital resolution via VNC is broken. > > E.g. after connecting seabios screen is not the typical text resolution > but a lot larger than it should be

Re: [Qemu-devel] [SeaBIOS] [PATCH 5/5] [wip] sercon: initial split-output implementation

2016-10-04 Thread Igor Mammedov
On Tue, 04 Oct 2016 10:49:41 +0200 Gerd Hoffmann wrote: > Hi, > > > Interesting. I'm curious how the memory scan works, because I > > didn't think there was any way to find the vga entry point except > > from the int10 vector. > > Run the init code in emulator? > > > Were you looking to inc

Re: [Qemu-devel] backup notifier fail policy

2016-10-04 Thread Stefan Hajnoczi
On Mon, Oct 03, 2016 at 02:07:34PM -0400, John Snow wrote: > > > On 10/03/2016 09:11 AM, Stefan Hajnoczi wrote: > > On Fri, Sep 30, 2016 at 09:59:16PM +0300, Vladimir Sementsov-Ogievskiy > > wrote: > > > On 30.09.2016 20:11, Vladimir Sementsov-Ogievskiy wrote: > > > > Hi all! > > > > > > > > Pl

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Kevin Wolf
Am 02.10.2016 um 21:13 hat Tomáš Golembiovský geschrieben: > Added two new options 'offset' and 'size'. This makes it possible to use > only part of the file as a device. This can be used e.g. to limit the > access only to single partition in a disk image or use a disk inside a > tar archive (like

Re: [Qemu-devel] [PATCH v2 1/5] msix_init: assert programming error

2016-10-04 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> Alex Williamson writes: >> >> > On Thu, 29 Sep 2016 15:11:27 +0200 >> > Markus Armbruster wrote: >> > >> >> Alex Williamson writes: >> >> >> >> > On Tue, 13 Sep 2016 08:16:20 +0200 >> >> > Markus Armbruster

Re: [Qemu-devel] backup notifier fail policy

2016-10-04 Thread Kevin Wolf
Am 04.10.2016 um 11:23 hat Stefan Hajnoczi geschrieben: > On Mon, Oct 03, 2016 at 02:07:34PM -0400, John Snow wrote: > > > > > > On 10/03/2016 09:11 AM, Stefan Hajnoczi wrote: > > > On Fri, Sep 30, 2016 at 09:59:16PM +0300, Vladimir Sementsov-Ogievskiy > > > wrote: > > > > On 30.09.2016 20:11, V

Re: [Qemu-devel] [PATCH v4 05/13] virtio-crypto: add virtio crypto device emulation

2016-10-04 Thread Stefan Hajnoczi
On Wed, Sep 28, 2016 at 04:25:44PM +0800, Gonglei wrote: > 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 This patch conta

Re: [Qemu-devel] [PATCH v4 07/13] virtio-crypto: set capacity of algorithms supported

2016-10-04 Thread Stefan Hajnoczi
On Wed, Sep 28, 2016 at 04:25:46PM +0800, Gonglei wrote: > static void virtio_crypto_get_config(VirtIODevice *vdev, uint8_t *config) > { > - > +VirtIOCrypto *c = VIRTIO_CRYPTO(vdev); > +struct virtio_crypto_config crypto_cfg; > + > +crypto_cfg.status = c->status; > +crypto_cfg.max

[Qemu-devel] [Bug 1629618] Re: QEMU causes host hang / reset on PPC64EL

2016-10-04 Thread T. Huth
Hi Timothy, sounds like this is a KVM (i.e. kernel) problem which might not directly be related to QEMU. So you likely should report this bug in the kernel bug tracker of your distro instead. Anyway, I've got some questions: 1) Can you always reproduce this problem after a while, or does it occur

Re: [Qemu-devel] [RFC 0/4] QOM class properties - do we need them?

2016-10-04 Thread Kevin Wolf
Am 29.09.2016 um 12:23 hat Andreas Färber geschrieben: > Am 29.09.2016 um 12:21 schrieb Daniel P. Berrange: > > On Thu, Sep 29, 2016 at 12:12:32PM +0200, Andreas Färber wrote: > >> Am 29.09.2016 um 10:14 schrieb Daniel P. Berrange: > >>> Practically all instances properties should become class prop

Re: [Qemu-devel] [PATCH v4 08/13] virtio-crypto: add control queue handler

2016-10-04 Thread Stefan Hajnoczi
On Wed, Sep 28, 2016 at 04:25:47PM +0800, Gonglei wrote: > -static void virtio_crypto_handle_ctrl(VirtIODevice *vdev, VirtQueue *vq) > +static inline int virtio_crypto_vq2q(int queue_index) > +{ > +return queue_index; > +} Please document this function. I think it takes a virtqueue index and

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Paolo Bonzini
On 04/10/2016 11:24, Kevin Wolf wrote: > You also need to update the options for the blockdev-add QMP command in > qapi/block-core.json. The relevant type is BlockdevOptionsFile. Is it? Or should he define a new BlockdevOptionsRaw, and point to it in the BlockdevOptions union instead of "'raw':

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

2016-10-04 Thread Stefan Hajnoczi
On Wed, Sep 28, 2016 at 04:25:39PM +0800, Gonglei wrote: > 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 final

Re: [Qemu-devel] [PULL 0/3] UUID fixes and one docker patch

2016-10-04 Thread Peter Maydell
On 2 October 2016 at 02:52, Fam Zheng wrote: > The following changes since commit c640f2849ee8775fe1bbd7a2772610aa77816f9f: > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into > staging (2016-09-28 23:02:56 +0100) > > are available in the git repository at: > > g...@gith

Re: [Qemu-devel] [PATCH v3 1/8] apic: add global apic_get_class()

2016-10-04 Thread Igor Mammedov
On Mon, 3 Oct 2016 13:03:33 -0300 Eduardo Habkost wrote: > On Fri, Sep 30, 2016 at 06:10:06PM +0200, Radim Krčmář wrote: > > Every configuration has only up to one APIC class and we'll be extending > > the class with a function that can be called without an instanced > > object, so a direct acces

Re: [Qemu-devel] [PATCH v4 11/35] cputlb: Move most of iotlb code out of line

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > Saves 2k code size off of a cold path. > > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > cputlb.c | 37 + > softmmu_template.h | 52 ++-- > 2 files c

[Qemu-devel] [PATCH v2 1/1] qemu-nbd: add the option to use pre-created server socket

2016-10-04 Thread Denis V. Lunev
From: Denis Plotnikov The NBD server socket was created by qemu-nbd code. This could lead to the race issue when the management layer started qemu-nbd server and allowed a client to connect to the server, the client tried to connect to the server but failed because qemu-nbd had not managed to ope

Re: [Qemu-devel] [PATCH v3 2/8] apic: add send_msi() to APICCommonClass

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:07 +0200 Radim Krčmář wrote: > The MMIO based interface to APIC doesn't work well with MSIs that have > upper address bits set (remapped x2APIC MSIs). A specialized interface > is a quick and dirty way to avoid the shortcoming. > > Signed-off-by: Radim Krčmář Reviewed

Re: [Qemu-devel] [PATCH v4 12/35] cputlb: Tidy some macros

2016-10-04 Thread Alex Bennée
Richard Henderson writes: > TGT_LE and TGT_BE are not size dependent and do not need to be > redefined. The others are no longer used at all. > > Signed-off-by: Richard Henderson Reviewed-by: Alex Bennée > --- > cputlb.c | 8 > softmmu_template.h | 22 ---

Re: [Qemu-devel] [PATCH v2 1/1] qemu-nbd: add the option to use pre-created server socket

2016-10-04 Thread Daniel P. Berrange
On Tue, Oct 04, 2016 at 02:03:01PM +0300, Denis V. Lunev wrote: > From: Denis Plotnikov > > The NBD server socket was created by qemu-nbd code. This could lead to the > race issue when the management layer started qemu-nbd server and allowed > a client to connect to the server, the client tried t

Re: [Qemu-devel] [PATCH v3 3/8] intel_iommu: pass whole remapped addresses to apic

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:08 +0200 Radim Krčmář wrote: > The MMIO interface to APIC only allowed 8 bit addresses, which is not > enough for 32 bit addresses from EIM remapping. > Intel stored upper 24 bits in the high MSI address, so use the same > technique. The technique is also used in KVM MSI

Re: [Qemu-devel] [PATCH v2 1/5] msix_init: assert programming error

2016-10-04 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert" writes: > > > * Markus Armbruster (arm...@redhat.com) wrote: > >> Alex Williamson writes: > >> > >> > On Thu, 29 Sep 2016 15:11:27 +0200 > >> > Markus Armbruster wrote: > >> > > >> >> Alex Williamson writes: > >> >> >

[Qemu-devel] [PATCH v2] scripts: qmp: Introduce vcpu pinning helper script

2016-10-04 Thread Maxime Coquelin
This python script calls 'query-cpus' QMP command to retrieve vCPUs thread IDs. Thread IDs are then used by taskset to pin vCPUs to physical CPUs passed in command line. In case more vCPUs are present than the number of CPUs assigned in command line, multiple vCPUs get pinned to physical CPUs. If

Re: [Qemu-devel] [PATCH v2 1/1] qemu-nbd: add the option to use pre-created server socket

2016-10-04 Thread Denis V. Lunev
On 10/04/2016 02:03 PM, Denis V. Lunev wrote: > From: Denis Plotnikov > > The NBD server socket was created by qemu-nbd code. This could lead to the > race issue when the management layer started qemu-nbd server and allowed > a client to connect to the server, the client tried to connect to the >

[Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator

2016-10-04 Thread Laurent Vivier
By default, QEMU uses 'tcg' or the one provided with the "accel" property. But sometime, user wants to use a real accelerator without knowing if he really can, with, for instance accel=kvm:tcg. In this case, and if the accelerator is not available we have a noisy "XXX accelerator not found". By a

[Qemu-devel] [PATCH 0/2] introduce "accel=best"

2016-10-04 Thread Laurent Vivier
This series introduces a new parameter to "accel" to ask to use the best one. This allows to remove ugly "kvm accelerator not found" from the "make check" logs. Laurent Vivier (2): accel: allows to select the "best" accelerator tests: use accel=best instead of accel=kvm:tcg accel.c

Re: [Qemu-devel] [PATCH v3 8/8] target-i386/kvm: cache the return value of kvm_enable_x2apic()

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:13 +0200 Radim Krčmář wrote: > Assume that KVM would have returned the same on subsequent runs. > Abstract the memoizaiton pattern into macros. s/memoi/memori/i Throughout whole patch > > Signed-off-by: Radim Krčmář > --- > target-i386/kvm.c | 21 +++-

[Qemu-devel] [PATCH 2/2] tests: use accel=best instead of accel=kvm:tcg

2016-10-04 Thread Laurent Vivier
This avoids to have logs polluted by "kvm accelerator not found". Signed-off-by: Laurent Vivier --- tests/bios-tables-test.c | 2 +- tests/postcopy-test.c| 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c index 7e2

Re: [Qemu-devel] [PATCH 0/2] introduce "accel=best"

2016-10-04 Thread Laurent Vivier
Sorry I didn't use the good email address for Stefano, cc the one from MAINTAINERS now... Laurent On 04/10/2016 13:30, Laurent Vivier wrote: > This series introduces a new parameter to "accel" to > ask to use the best one. > > This allows to remove ugly "kvm accelerator not found" > from the "ma

Re: [Qemu-devel] [PULL 0/2] x86 fixes

2016-10-04 Thread Peter Maydell
On 3 October 2016 at 20:08, Eduardo Habkost wrote: > The following changes since commit c5d128ffeb5357df1ea3e6de0c13b3d6a09f6064: > > Merge remote-tracking branch 'remotes/borntraeger/tags/s390x-20160927' into > staging (2016-09-30 23:45:56 +0100) > > are available in the git repository at: > >

Re: [Qemu-devel] [PATCH v3 4/8] intel_iommu: redo configuraton check in realize

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:09 +0200 Radim Krčmář wrote: > * there no point in configuring the device if realization is going to > fail, so move the check to the beginning, > * create a separate function for the check, > * use error_setg() instead error_report(). Reviewed-by: Igor Mammedov > S

Re: [Qemu-devel] [PATCH v4 02/17] vfio/pci: Pass an error object to vfio_populate_vga

2016-10-04 Thread Markus Armbruster
Eric Auger writes: > Pass an error object to prepare for the same operation in > vfio_populate_device. Eventually this contributes to the migration > to VFIO-PCI realize. > > We now report an error on vfio_get_region_info failure. > > vfio_probe_igd_bar4_quirk is not involved in the migration to

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Tomáš Golembiovský
On Tue, 4 Oct 2016 10:57:49 +0200 Kevin Wolf wrote: > Am 03.10.2016 um 13:07 hat Tomáš Golembiovský geschrieben: > > > > > > +if (((bs->drv != &bdrv_file) || !bs->read_only) && > > > > > > > > > > Why the check against bdrv_file ? > > > > > > > > To limit it only to files. Maybe t

Re: [Qemu-devel] backup notifier fail policy

2016-10-04 Thread Kevin Wolf
Am 04.10.2016 um 12:41 hat Denis V. Lunev geschrieben: > On 10/04/2016 12:34 PM, Kevin Wolf wrote: > > Am 04.10.2016 um 11:23 hat Stefan Hajnoczi geschrieben: > >> On Mon, Oct 03, 2016 at 02:07:34PM -0400, John Snow wrote: > >>> > >>> On 10/03/2016 09:11 AM, Stefan Hajnoczi wrote: > On Fri, Se

Re: [Qemu-devel] [PATCH v4 07/17] vfio/pci: Pass an error object to vfio_pci_igd_opregion_init

2016-10-04 Thread Markus Armbruster
Eric Auger writes: > Pass an error object to prepare for migration to VFIO-PCI realize. > > In vfio_probe_igd_bar4_quirk, simply report the error. > > Signed-off-by: Eric Auger > --- > hw/vfio/pci-quirks.c | 10 +- > hw/vfio/pci.c| 3 +-- > hw/vfio/pci.h| 3 ++- > 3 fi

Re: [Qemu-devel] [PATCH] raw-posix: add 'offset' and 'size' options

2016-10-04 Thread Kevin Wolf
Am 04.10.2016 um 12:12 hat Paolo Bonzini geschrieben: > > > On 04/10/2016 11:24, Kevin Wolf wrote: > > You also need to update the options for the blockdev-add QMP command in > > qapi/block-core.json. The relevant type is BlockdevOptionsFile. > > Is it? Or should he define a new BlockdevOptionsR

Re: [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator

2016-10-04 Thread Daniel P. Berrange
On Tue, Oct 04, 2016 at 01:30:47PM +0200, Laurent Vivier wrote: > By default, QEMU uses 'tcg' or the one provided with the "accel" > property. > > But sometime, user wants to use a real accelerator without knowing > if he really can, with, for instance accel=kvm:tcg. > In this case, and if the acc

Re: [Qemu-devel] [PATCH 0/2] introduce "accel=best"

2016-10-04 Thread Andrew Jones
On Tue, Oct 04, 2016 at 01:30:46PM +0200, Laurent Vivier wrote: > This series introduces a new parameter to "accel" to > ask to use the best one. > > This allows to remove ugly "kvm accelerator not found" > from the "make check" logs. > > Laurent Vivier (2): > accel: allows to select the "best"

Re: [Qemu-devel] [PATCH v4 08/17] vfio: Pass an Error object to vfio_connect_container

2016-10-04 Thread Markus Armbruster
Eric Auger writes: > The error is currently simply reported in vfio_get_group. Don't > bother too much with the prefix which will be handled at upper level, > later on. > > Signed-off-by: Eric Auger > --- > hw/vfio/common.c | 39 --- > 1 file changed, 24 inse

Re: [Qemu-devel] [PATCH 0/2] introduce "accel=best"

2016-10-04 Thread Daniel P. Berrange
On Tue, Oct 04, 2016 at 02:00:46PM +0200, Andrew Jones wrote: > On Tue, Oct 04, 2016 at 01:30:46PM +0200, Laurent Vivier wrote: > > This series introduces a new parameter to "accel" to > > ask to use the best one. > > > > This allows to remove ugly "kvm accelerator not found" > > from the "make ch

[Qemu-devel] [PATCH] qtest: add read/write accessors with a specific endianness

2016-10-04 Thread Cédric Le Goater
Some test scenarios require to access memory regions using a specific endianness, such as a device region, but the current qtest memory accessors are done in native endian, which means that the values are byteswapped in qtest if the endianness of the guest and the host are different. To maintain t

Re: [Qemu-devel] [PATCH v3 7/8] intel_iommu: keep buggy EIM enabled in 2.7 machine type

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:12 +0200 Radim Krčmář wrote: > QEMU 2.7 allowed EIM even in configurations that were forbidden in the > last patch because they were not working, like old KVM or userspace > APIC. In order to keep backward compatibility, we again allow guests to > misbehave in non-obvio

Re: [Qemu-devel] [Bug 1626972] Re: [PATCH] util: secure memfd_create fallback mechanism

2016-10-04 Thread Rafael David Tinoco
Let me work on it. I'll get back soon. Tks Daniel. > On Oct 04, 2016, at 05:36, Daniel P. Berrange wrote: > > On Mon, Oct 03, 2016 at 04:15:55PM -0300, Rafael David Tinoco wrote: >> Yes, definitely. Check this: > > [snip] > > So in that case, I think we must add ability to specify an explici

Re: [Qemu-devel] [PATCH 0/2] introduce "accel=best"

2016-10-04 Thread Andrew Jones
On Tue, Oct 04, 2016 at 01:07:28PM +0100, Daniel P. Berrange wrote: > On Tue, Oct 04, 2016 at 02:00:46PM +0200, Andrew Jones wrote: > > On Tue, Oct 04, 2016 at 01:30:46PM +0200, Laurent Vivier wrote: > > > This series introduces a new parameter to "accel" to > > > ask to use the best one. > > > >

Re: [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator

2016-10-04 Thread Laurent Vivier
On 04/10/2016 14:00, Daniel P. Berrange wrote: > On Tue, Oct 04, 2016 at 01:30:47PM +0200, Laurent Vivier wrote: >> By default, QEMU uses 'tcg' or the one provided with the "accel" >> property. >> >> But sometime, user wants to use a real accelerator without knowing >> if he really can, with, for

Re: [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator

2016-10-04 Thread Peter Maydell
On 4 October 2016 at 13:00, Daniel P. Berrange wrote: > On Tue, Oct 04, 2016 at 01:30:47PM +0200, Laurent Vivier wrote: >> By default, QEMU uses 'tcg' or the one provided with the "accel" >> property. >> >> But sometime, user wants to use a real accelerator without knowing >> if he really can, wit

Re: [Qemu-devel] [PATCH v3 5/8] intel_iommu: add OnOffAuto intr_eim as "eim" property

2016-10-04 Thread Igor Mammedov
On Fri, 30 Sep 2016 18:10:10 +0200 Radim Krčmář wrote: > The default (auto) emulates the current behavior. > > Signed-off-by: Radim Krčmář Reviewed-by: Igor Mammedov > --- > v3: > * use error_setg [Paolo] > * shorten the code [Peter] > --- > hw/i386/intel_iommu.c | 24 +

Re: [Qemu-devel] [PATCH] qtest: add read/write accessors with a specific endianness

2016-10-04 Thread Peter Maydell
On 4 October 2016 at 13:17, Cédric Le Goater wrote: > Some test scenarios require to access memory regions using a specific > endianness, such as a device region, but the current qtest memory > accessors are done in native endian, which means that the values are > byteswapped in qtest if the endia

Re: [Qemu-devel] [Bug 1626972] Re: [PATCH] util: secure memfd_create fallback mechanism

2016-10-04 Thread Marc-André Lureau
Hi Rafael, Daniel, On Tue, Oct 4, 2016 at 4:22 PM Rafael David Tinoco < rafael.tin...@canonical.com> wrote: > Let me work on it. I'll get back soon. > > thanks for working on it, before that I have a few questions: Tks Daniel. > > > On Oct 04, 2016, at 05:36, Daniel P. Berrange > wrote: > > > >

Re: [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator

2016-10-04 Thread Paolo Bonzini
On 04/10/2016 13:30, Laurent Vivier wrote: > But sometime, user wants to use a real accelerator without knowing > if he really can, with, for instance accel=kvm:tcg. > In this case, and if the accelerator is not available we > have a noisy "XXX accelerator not found". > > By allowing the user to

[Qemu-devel] [PULL 13/27] vmstateify tsc210x

2016-10-04 Thread Peter Maydell
From: "Dr. David Alan Gilbert" I'm now saving all 3 of the pll entries; only 2 were saved before. There are a couple of times that were previously stored as offsets from 'now' calculated before saving; with vmstate it's easier to store the 'now' and fix it up on reload. Signed-off-by: Dr. David

[Qemu-devel] [PULL 15/27] hw/intc/arm_gic(v3)_kvm: Initialize gsi routing

2016-10-04 Thread Peter Maydell
From: Eric Auger Advertise gsi routing and set up irqchip routing entries for GIC SPIs. This is not mandated as long as MSI routing is not used (because the kernel sets a default irqchip routing table). However once MSI routing gets used (for VIRTIO-PCI vhost for example), the first call to KVM_

[Qemu-devel] [PULL 18/27] kvm-all: Pass requester ID to MSI routing functions

2016-10-04 Thread Peter Maydell
From: Pavel Fedin Introduce global kvm_msi_use_devid flag plus associated kvm_msi_devid_required() macro. Passes the device ID, if needed, while building the MSI route entry. Device IDs are required by the ARM GICv3 ITS (IRQ remapping function is based on this information). Signed-off-by: Pavel

Re: [Qemu-devel] [Bug 1626972] Re: [PATCH] util: secure memfd_create fallback mechanism

2016-10-04 Thread Daniel P. Berrange
On Tue, Oct 04, 2016 at 12:39:17PM +, Marc-André Lureau wrote: > Hi Rafael, Daniel, > > On Tue, Oct 4, 2016 at 4:22 PM Rafael David Tinoco < > rafael.tin...@canonical.com> wrote: > > > Let me work on it. I'll get back soon. > > > > > thanks for working on it, before that I have a few question

[Qemu-devel] [PULL 17/27] target-arm: move gicv3_class_name from machine to kvm_arm.h

2016-10-04 Thread Peter Maydell
From: Eric Auger Machine.c contains code related to migration. Let's move gicv3_class_name to kvm_arm.h instead. Signed-off-by: Eric Auger Reviewed-by: Peter Maydell Message-id: 1474616617-366-4-git-send-email-eric.au...@redhat.com Suggested-by: Peter Maydell Reviewed-by: Peter Maydell Signe

[Qemu-devel] [PULL 14/27] hw/arm/virt: add 2.8 machine type

2016-10-04 Thread Peter Maydell
From: Andrew Jones Signed-off-by: Andrew Jones Message-id: 1474641676-25017-1-git-send-email-drjo...@redhat.com Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- hw/arm/virt.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/hw/arm/virt.c b/hw

[Qemu-devel] [PULL 06/27] STM32F205: Connect the ADC devices

2016-10-04 Thread Peter Maydell
From: Alistair Francis Connect the ADC devices to the STM32F205 SoC. Signed-off-by: Alistair Francis Message-id: 6214eda399da7b47014f6f895be25323d52dbc9e.1474742262.git.alist...@alistair23.me Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- hw/arm/stm32f205_soc.c | 35 +++

[Qemu-devel] [PULL 00/27] target-arm queue

2016-10-04 Thread Peter Maydell
4 11:28:30 +0100) are available in the git repository at: git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20161004 for you to fetch changes up to 9b6a3ea7a699594162ed3d11e4e04b98568dc5c0: target-arm: Correctly handle 'sub pc, pc, 1' for ARMv6 (2016-1

[Qemu-devel] [PULL 16/27] hw/intc/arm_gicv3_its: Implement ITS base class

2016-10-04 Thread Peter Maydell
From: Pavel Fedin This is the basic skeleton for both KVM and software-emulated ITS. Since we already prepare status structure, we also introduce complete VMState description. But, because we currently have no migratable implementations, we also set unmigratable flag. Signed-off-by: Pavel Fedin

[Qemu-devel] [PULL 11/27] hw/arm: Fix Integrator/CM initialization

2016-10-04 Thread Peter Maydell
From: Jakub Jermar Initialization of a class instance cannot depend on its own properties as these are not yet set. Move parts of integratorcm_init() that depend on the "memsz" property to the newly added integratorcm_realize(). This fixes: https://bugs.launchpad.net/qemu/+bug/1624726 Signed-o

[Qemu-devel] [PULL 05/27] irq: Add a new irq device that allows the ORing of lines

2016-10-04 Thread Peter Maydell
From: Alistair Francis Signed-off-by: Alistair Francis Message-id: 52e5d361e3b5a0ea8554aca73ee65ae2b586112e.1474742262.git.alist...@alistair23.me Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- hw/core/Makefile.objs | 1 + hw/core/or-irq.c | 107 +++

[Qemu-devel] [PULL 20/27] arm/virt: Add ITS to the virt board

2016-10-04 Thread Peter Maydell
From: Pavel Fedin If supported by the configuration, ITS will be added automatically. This patch also renames v2m_phandle to msi_phandle because it's now used by both MSI implementations. Signed-off-by: Pavel Fedin Signed-off-by: Eric Auger Reviewed-by: Peter Maydell Message-id: 1474616617-3

[Qemu-devel] [PULL 09/27] mainstone: Fix incorrect key mapping for Enter key.

2016-10-04 Thread Peter Maydell
From: Vijay Kumar B According to the manual the (5, 5) corresponds to backspace key, and not Enter key. Linux kernel maps (5, 4) to the enter key. Fixing it up to match the mapping in the Linux kernel. Signed-off-by: Vijay Kumar B. Reviewed-by: Deepak S. Message-id: 1475063033-8176-2-git-send-

[Qemu-devel] [PULL 01/27] STM32F205: Remove the individual device variables

2016-10-04 Thread Peter Maydell
From: Alistair Francis Cleanup the individual DeviceState and SysBusDevice variables to re-use the same variable for each device. Signed-off-by: Alistair Francis Reviewed-by: Peter Crosthwaite Message-id: fc5d75a57d320b69704df2c1146ff0fd482e4a88.1474742262.git.alist...@alistair23.me Signed-of

[Qemu-devel] [PULL 19/27] hw/intc/arm_gicv3_its: Implement support for in-kernel ITS emulation

2016-10-04 Thread Peter Maydell
From: Pavel Fedin The ITS control frame is in-kernel emulated while accesses to the GITS_TRANSLATER are mediated through the KVM_SIGNAL_MSI ioctl (MSI direct MSI injection advertised by the CAP_SIGNAL_MSI capability) the kvm_gsi_direct_mapping is explicitly set to false to emphasize the differen

[Qemu-devel] [PULL 12/27] vmstateify tsc2005

2016-10-04 Thread Peter Maydell
From: "Dr. David Alan Gilbert" I've converted the fields in it's main data structure to fixed size types in ways that look sane. Signed-off-by: Dr. David Alan Gilbert Message-id: 1474977735-10156-2-git-send-email-dgilb...@redhat.com Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell ---

[Qemu-devel] [PULL 07/27] STM32F205: Connect the SPI devices

2016-10-04 Thread Peter Maydell
From: Alistair Francis Connect the SPI devices to the STM32F205 SoC. Signed-off-by: Alistair Francis Reviewed-by: Peter Crosthwaite Message-id: d05849120420f8db0d9aa053bd23134c33cd9180.1474742262.git.alist...@alistair23.me Signed-off-by: Peter Maydell --- hw/arm/stm32f205_soc.c | 22

[Qemu-devel] [PULL 22/27] ARM: Virt: ACPI: Add GIC ITS description in ACPI MADT table

2016-10-04 Thread Peter Maydell
From: Shannon Zhao If GIC ITS is supported, add description in ACPI MADT table, then guest could use ITS when booting with ACPI. Signed-off-by: Shannon Zhao Signed-off-by: Eric Auger Message-id: 1474616617-366-9-git-send-email-eric.au...@redhat.com Signed-off-by: Peter Maydell --- hw/arm/vir

[Qemu-devel] [PULL 02/27] STM32F2xx: Display PWM duty cycle from timer

2016-10-04 Thread Peter Maydell
From: Alistair Francis If correctly configured allow the STM32F2xx timer to print out the PWM duty cycle information. Signed-off-by: Alistair Francis Reviewed-by: Peter Crosthwaite Message-id: cdb59039a25e061615713a94b40797baa12ea9f9.1474742262.git.alist...@alistair23.me Signed-off-by: Peter

[Qemu-devel] [PULL 08/27] MAINTAINERS: Add Alistair to the maintainers list

2016-10-04 Thread Peter Maydell
From: Alistair Francis Add Alistair Francis as the maintainer for the Netduino 2 and SMM32F205 SoC. Signed-off-by: Alistair Francis Reviewed-by: Peter Crosthwaite Message-id: 5a46ccf398b050a41cc3b3d0e94bcff4ce2d85e0.1474742262.git.alist...@alistair23.me Signed-off-by: Peter Maydell --- MAIN

  1   2   3   4   >