Re: [Qemu-devel] [PATCH] msix: fix interrupt aggregation problem at the passthrough of NVMe SSD

2019-04-10 Thread Zhuangyanying
> -Original Message- > From: Michael S. Tsirkin [mailto:m...@redhat.com] > Sent: Tuesday, April 09, 2019 11:04 PM > To: Zhuangyanying > Cc: marcel.apfelb...@gmail.com; qemu-devel@nongnu.org; Gonglei (Arei) > > Subject: Re: [PATCH] msix: fix interrupt aggregation problem at the > passth

[Qemu-devel] [Bug 1823831] Re: BSD bootloader halts with hypervisor.framework

2019-04-10 Thread Chen Zhang
Git bisect shows that 92d5f1a4147c3722b5e9a8bcfb7dc261b7a8b855 is the first bad commit. Author: Paolo Bonzini Date: Tue Aug 21 15:31:24 2018 +0200 target/i386: unify masking of interrupts Interrupt handling depends on various flags in env->hflags or env->hflags2, and the exact

Re: [Qemu-devel] [PATCH] docs: replace min-glib with fedora

2019-04-10 Thread Stefano Garzarella
On Tue, Apr 09, 2019 at 05:44:35PM +0200, Marc-André Lureau wrote: > min-glib.docker was removed in commit > e7b3af81597db1a6b55f2c15d030d703c6b2c6ac ("glib: bump min required > glib library version to 2.40"). > > Cc: Daniel P. Berrangé > Signed-off-by: Marc-André Lureau > --- > docs/devel/test

Re: [Qemu-devel] [PATCH] configure: Automatically fall back to TCI on non-release architectures

2019-04-10 Thread Thomas Huth
On 09/04/2019 21.46, Stefan Weil wrote: [...] > The known problems with the current implementation are [...] > * Calling conventions. The current implementation works on many hosts, > but for example not on Sparc Is there also a problem with the sparc *target* (i.e. not only sparc hosts)? TCI does

Re: [Qemu-devel] [PATCH v2 for-4.0?] aio-posix: ensure poll mode is left when aio_notify is called

2019-04-10 Thread Sergio Lopez
Paolo Bonzini writes: > With aio=thread, adaptive polling makes latency worse rather than > better, because it delays the execution of the ThreadPool's > completion bottom half. > > event_notifier_poll() does run while polling, detecting that > a bottom half was scheduled by a worker thread, but

Re: [Qemu-devel] [PATCH] configure: Automatically fall back to TCI on non-release architectures

2019-04-10 Thread Thomas Huth
On 10/04/2019 08.07, Thomas Huth wrote: > On 09/04/2019 21.46, Stefan Weil wrote: >> On 05.04.19 11:16, Philippe Mathieu-Daudé wrote: >>> On 4/5/19 11:02 AM, Daniel P. Berrangé wrote: On Fri, Apr 05, 2019 at 10:47:54AM +0200, Philippe Mathieu-Daudé wrote: Do the various crashes that you i

Re: [Qemu-devel] [PATCH v5 4/6] dax: check synchronous mapping is supported

2019-04-10 Thread Jan Kara
On Wed 10-04-19 09:38:24, Pankaj Gupta wrote: > This patch introduces 'daxdev_mapping_supported' helper > which checks if 'MAP_SYNC' is supported with filesystem > mapping. It also checks if corresponding dax_device is > synchronous. Virtio pmem device is asynchronous and > does not not support VM_

Re: [Qemu-devel] [PATCH v5 3/6] libnvdimm: add dax_dev sync flag

2019-04-10 Thread Jan Kara
On Wed 10-04-19 09:38:23, Pankaj Gupta wrote: > @@ -64,6 +65,10 @@ static inline bool dax_write_cache_enabled(struct > dax_device *dax_dev) > { > return false; > } > +static inline bool dax_synchronous(struct dax_device *dax_dev) > +{ > + return true; > +} Is there a need to define da

Re: [Qemu-devel] [PATCH v5 4/6] dax: check synchronous mapping is supported

2019-04-10 Thread Pankaj Gupta
> > This patch introduces 'daxdev_mapping_supported' helper > > which checks if 'MAP_SYNC' is supported with filesystem > > mapping. It also checks if corresponding dax_device is > > synchronous. Virtio pmem device is asynchronous and > > does not not support VM_SYNC. > > > > Suggested-by: Jan K

Re: [Qemu-devel] [PATCH] migration: savevm: fix error code with migration blockers

2019-04-10 Thread Dr. David Alan Gilbert
* Cole Robinson (crobi...@redhat.com) wrote: > The only caller that checks the error code is looking for != 0, > so returning false is incorrect. > > Fixes: 5aaac467938 "migration: savevm: consult migration blockers" > > Signed-off-by: Cole Robinson Thanks, this was in 3.1.0 so isn't a 4.0 regr

Re: [Qemu-devel] [PATCH v5 3/6] libnvdimm: add dax_dev sync flag

2019-04-10 Thread Pankaj Gupta
> > On Wed 10-04-19 09:38:23, Pankaj Gupta wrote: > > @@ -64,6 +65,10 @@ static inline bool dax_write_cache_enabled(struct > > dax_device *dax_dev) > > { > > return false; > > } > > +static inline bool dax_synchronous(struct dax_device *dax_dev) > > +{ > > + return true; > > +} > > Is t

Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: use variable size of flash device to save memory

2019-04-10 Thread Xiang Zheng
Hi Kevin, On 2019/4/9 16:28, Kevin Wolf wrote: > Am 09.04.2019 um 08:01 hat Markus Armbruster geschrieben: >> László's last sentence below is "This really needs the attention of the >> block people." Cc'ing some. >> >> Laszlo Ersek writes: >> >>> On 04/08/19 15:43, Xiang Zheng wrote: O

Re: [Qemu-devel] [PATCH v4 17/20] console: make screendump asynchronous

2019-04-10 Thread Gerd Hoffmann
> +static void qmp_screendump_finish(QemuConsole *con, struct qmp_screendump > *dump) > +{ > +Error *err = NULL; > +DisplaySurface *surface; > +Monitor *prev_mon = cur_mon; Why this is needed? > +/* > + * FIXME: async save with coroutine? it would have to copy or > +

Re: [Qemu-devel] [PATCH v4 16/20] console: add graphic_hw_update_done()

2019-04-10 Thread Gerd Hoffmann
On Tue, Apr 09, 2019 at 06:10:05PM +0200, Marc-André Lureau wrote: > Add a function to be called when a graphic update is done. > > Declare the QXL renderer as async: render_update_cookie_num counts the > number of outstanding updates, and graphic_hw_update_done() is called > when it reaches none.

Re: [Qemu-devel] [PATCH v4 8/8] hw/arm/boot: Expose the PC-DIMM nodes in the DT

2019-04-10 Thread Shameerali Kolothum Thodi
> -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: 09 April 2019 16:09 > To: Shameerali Kolothum Thodi ; > qemu-devel@nongnu.org; qemu-...@nongnu.org; eric.au...@redhat.com; > imamm...@redhat.com > Cc: peter.mayd...@linaro.org; shannon.zha...@gmail.com; > sa...@li

Re: [Qemu-devel] [PATCH 1/2] qemu-img convert: ignore read errors

2019-04-10 Thread Vladimir Sementsov-Ogievskiy
09.04.2019 17:56, Andrey Shinkevich wrote: > The 'qemu-img convert' new command option 'force read' with the key '-R' > allows converting a damaged image to get all the available information > in case of the read errors. The program reports read errors and continue > the image conversion. The users

Re: [Qemu-devel] [PATCH 2/2] iotests: new test 253 check qemu-img convert force read

2019-04-10 Thread Vladimir Sementsov-Ogievskiy
09.04.2019 17:56, Andrey Shinkevich wrote: > A new test for the patch 'qemu-img convert: ignore read errors' > > Signed-off-by: Andrey Shinkevich > --- > tests/qemu-iotests/253 | 69 > ++ > tests/qemu-iotests/253.out | 4 +++ > tests/qemu-iote

Re: [Qemu-devel] [PATCH] migration: savevm: fix error code with migration blockers

2019-04-10 Thread Juan Quintela
Cole Robinson wrote: > The only caller that checks the error code is looking for != 0, > so returning false is incorrect. > > Fixes: 5aaac467938 "migration: savevm: consult migration blockers" > > Signed-off-by: Cole Robinson Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH for-4.1] q35: acpi: do not create dummy MCFG table

2019-04-10 Thread Igor Mammedov
On Wed, 10 Apr 2019 09:12:31 +0800 Wei Yang wrote: > On Tue, Apr 09, 2019 at 05:00:37PM +0200, Igor Mammedov wrote: > >Dummy table (with signature "QEMU") creation came from original SeaBIOS > >codebase. And QEMU would have to keep it around if there were Q35 machine > >that depended on keeping A

Re: [Qemu-devel] [PATCH for-4.1 v4 00/12] bundle edk2 platform firmware with QEMU

2019-04-10 Thread Igor Mammedov
On Wed, 10 Apr 2019 01:00:10 +0200 Laszlo Ersek wrote: > Repo: https://github.com/lersek/qemu.git > Branch: edk2_build_v4 > > Posting a v4 is necessary because patch #6 needs > - manual conflict resolution against some commits between v4.0.0-rc2 and > v4.0.0-rc3, > - and corresponding commit

[Qemu-devel] [PATCH] migration: Fix handling fd protocol

2019-04-10 Thread Yury Kotov
Currently such case is possible for incoming: QMP: add-fd (fdset = 0, fd of some file): adds fd to fdset 0 and returns QEMU's fd (e.g. 33) QMP: migrate-incoming (uri = "fd:33"): fd is stored in QIOChannel *ioc ... Incoming migration completes and unrefs ioc -> close(33) QMP: remove-fd (fdset =

Re: [Qemu-devel] [PULL 0/1] device-tree queue

2019-04-10 Thread Peter Maydell
On Wed, 10 Apr 2019 at 00:55, Alistair Francis wrote: > > The following changes since commit f151f8aca5cf5da24f6eb743a55a2233091ae532: > > migration/ram.c: Fix use-after-free in multifd_recv_unfill_packet() > (2019-04-09 20:46:34 +0100) > > are available in the Git repository at: > > g...@git

Re: [Qemu-devel] [PATCH] migration: Fix handling fd protocol

2019-04-10 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190410092652.22616-1-yury-ko...@yandex-team.ru/ Hi, This series failed the docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEG

Re: [Qemu-devel] [PATCH for-4.1 v4 11/12] Makefile: install the edk2 firmware images and their descriptors

2019-04-10 Thread Philippe Mathieu-Daudé
On 4/10/19 1:00 AM, Laszlo Ersek wrote: > Decompress and install the edk2 firmware blobs as part of "make install", > unless blob installation was disabled with configure's "--disable-blobs" > option. > > Additionally, decompress the blobs as a pre-requisite for building softmmu > binaries -- this

Re: [Qemu-devel] [PATCH for-4.1 v4 08/12] pc-bios: add edk2 firmware binaries and variable store templates

2019-04-10 Thread Philippe Mathieu-Daudé
On 4/10/19 1:00 AM, Laszlo Ersek wrote: > Add the files built by the last patch: (compressed) binaries, and the > cumulative license text that covers them. > > Signed-off-by: Laszlo Ersek > Reviewed-by: Michal Privoznik > Reviewed-by: Philippe Mathieu-Daudé > Reviewed-by: Michael S. Tsirkin T

Re: [Qemu-devel] [PATCH for-4.1 v4 10/12] tests: add missing dependency to build QTEST_QEMU_BINARY, round 2

2019-04-10 Thread Philippe Mathieu-Daudé
On 4/10/19 1:00 AM, Laszlo Ersek wrote: > In commit b94b330e2333 ("tests: add missing dependency to build > QTEST_QEMU_BINARY", 2017-07-31), Phil fixed the dependency list of make > target "check-qtest-%". Namely, the recipe would set QTEST_QEMU_BINARY to > the softmmu emulator for the emulation ta

[Qemu-devel] [Bug 1823831] Re: BSD bootloader halts with hypervisor.framework

2019-04-10 Thread Chen Zhang
In good versions (27e18b8952f8b7a1e26350846f8a0d5a9b33bfb8), calls to x86_cpu_has_work(), likely due to IRQ 0, returned interchanging true or false. In bad versions (92d5f1a4147c3722b5e9a8bcfb7dc261b7a8b855), all calls returned false. -- You received this bug notification because you are a membe

Re: [Qemu-devel] [PATCH 1/3] xen-bus: use a separate fd for each event channel

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:15PM +0100, Paul Durrant wrote: > To better support use of IOThread-s it will be necessary to be able to set > the AioContext for each XenEventChannel and hence it is necessary to open a > separate handle to libxenevtchan for each channel. > > This patch stops using N

Re: [Qemu-devel] [PATCH RESEND v2 1/2] hvf: declare hvf_handle_io if NEED_CPU_H is defined

2019-04-10 Thread Roman Bolshakov
On Sun, Apr 07, 2019 at 05:28:38PM +0530, Sukrit Bhatnagar wrote: > hvf_handle_io needs the poisoned type CPUArchState as its argument. > Declaring it if NEED_CPU_H is defined enables include/sysemu/hvf.h > to be included for common object compilation as well. > Reviewed-by: Roman Bolshakov Test

Re: [Qemu-devel] [PATCH RESEND v2 2/2] cpus: move hvf_cpu_synchronize* calls to cpu_synchronize* functions

2019-04-10 Thread Roman Bolshakov
On Sun, Apr 07, 2019 at 05:28:39PM +0530, Sukrit Bhatnagar wrote: > Keep the calls made to synchronize cpu by all hypervisors in one place > inside cpu_synchronize_* functions in include/sysemu/hw_accel.h > > Cc: Richard Henderson > Cc: Paolo Bonzini > Signed-off-by: Sukrit Bhatnagar > --- > c

Re: [Qemu-devel] [PATCH RESEND v2 2/2] cpus: move hvf_cpu_synchronize* calls to cpu_synchronize* functions

2019-04-10 Thread Sukrit Bhatnagar
On Wed, 10 Apr 2019 at 17:20, Roman Bolshakov wrote: > > On Sun, Apr 07, 2019 at 05:28:39PM +0530, Sukrit Bhatnagar wrote: > > Keep the calls made to synchronize cpu by all hypervisors in one place > > inside cpu_synchronize_* functions in include/sysemu/hw_accel.h > > > > Cc: Richard Henderson >

Re: [Qemu-devel] [PATCH 03/13] libqos: move common i2c code to libqos

2019-04-10 Thread Paolo Bonzini
On 09/04/19 11:42, Thomas Huth wrote: >> -static void tmp105_set16(I2CAdapter *i2c, uint8_t addr, uint8_t reg, >> - uint16_t value) >> -{ >> -uint8_t cmd[3]; >> -uint8_t resp[2]; >> - >> -cmd[0] = reg; >> -cmd[1] = value >> 8; >> -cmd[2] = value & 255; >>

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Cornelia Huck
On Wed, 10 Apr 2019 09:38:22 +0530 Pankaj Gupta wrote: > This patch adds virtio-pmem driver for KVM guest. > > Guest reads the persistent memory range information from > Qemu over VIRTIO and registers it on nvdimm_bus. It also > creates a nd_region object with the persistent memory > range infor

[Qemu-devel] [Bug 1572959] Re: bcm2835_property: inconsistent values when both setting and querying the framebuffer

2019-04-10 Thread Thomas Huth
The patch had been merged here: https://git.qemu.org/?p=qemu.git;a=commitdiff;h=27a5dc7be6a55b60039e3920 ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.ne

[Qemu-devel] [Bug 1525676] Re: qemu runas and sandbox option incompatible, process will hang in futex after setgid

2019-04-10 Thread Thomas Huth
I haven't tried, but I think this should be fixed now with the new elevateprivileges parameter of the -sandbox option. Have you tried to reproduce the problem with the latest version of QEMU already? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification becau

[Qemu-devel] [PATCH for-4.1] gitlab-ci.yml: Test the TCG interpreter in a CI pipeline

2019-04-10 Thread Thomas Huth
So far we do not have any test coverage for TCI (the TCG interpreter) yet. Thus let's add a CI pipeline that runs at least some basic TCG tests with a TCI build, to make sure that there are no further regressions. Signed-off-by: Thomas Huth --- NB: I did not include the sparc and sparc64 targets

Re: [Qemu-devel] [PATCH v5 0/6] virtio pmem driver

2019-04-10 Thread Arkadiusz Miśkiewicz
On 10/04/2019 06:08, Pankaj Gupta wrote: > This patch series has implementation for "virtio pmem". > "virtio pmem" is fake persistent memory(nvdimm) in guest > which allows to bypass the guest page cache. This also Will kernel pstore be able to use this persistent memory for storing crash dum

Re: [Qemu-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:16PM +0100, Paul Durrant wrote: > This patch adds an AioContext parameter to xen_device_bind_event_channel() > and then uses aio_set_fd_handler() to set the callback rather than > qemu_set_fd_handler(). > > Signed-off-by: Paul Durrant > --- > @@ -943,6 +944,7 @@ stat

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Michael S. Tsirkin
On Wed, Apr 10, 2019 at 09:38:22AM +0530, Pankaj Gupta wrote: > This patch adds virtio-pmem driver for KVM guest. > > Guest reads the persistent memory range information from > Qemu over VIRTIO and registers it on nvdimm_bus. It also > creates a nd_region object with the persistent memory > range

Re: [Qemu-devel] [PATCH] migration: Fix handling fd protocol

2019-04-10 Thread Dr. David Alan Gilbert
* Yury Kotov (yury-ko...@yandex-team.ru) wrote: > Currently such case is possible for incoming: > QMP: add-fd (fdset = 0, fd of some file): > adds fd to fdset 0 and returns QEMU's fd (e.g. 33) > QMP: migrate-incoming (uri = "fd:33"): fd is stored in QIOChannel *ioc > ... > Incoming migration co

Re: [Qemu-devel] [PATCH for-4.1] q35: acpi: do not create dummy MCFG table

2019-04-10 Thread Wei Yang
On Wed, Apr 10, 2019 at 11:08:33AM +0200, Igor Mammedov wrote: >On Wed, 10 Apr 2019 09:12:31 +0800 >Wei Yang wrote: > >> On Tue, Apr 09, 2019 at 05:00:37PM +0200, Igor Mammedov wrote: >> >Dummy table (with signature "QEMU") creation came from original SeaBIOS >> >codebase. And QEMU would have to k

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Pankaj Gupta
> > On Wed, Apr 10, 2019 at 09:38:22AM +0530, Pankaj Gupta wrote: > > This patch adds virtio-pmem driver for KVM guest. > > > > Guest reads the persistent memory range information from > > Qemu over VIRTIO and registers it on nvdimm_bus. It also > > creates a nd_region object with the persisten

Re: [Qemu-devel] [PATCH RESEND v2 2/2] cpus: move hvf_cpu_synchronize* calls to cpu_synchronize* functions

2019-04-10 Thread Roman Bolshakov
On Wed, Apr 10, 2019 at 05:35:23PM +0530, Sukrit Bhatnagar wrote: > On Wed, 10 Apr 2019 at 17:20, Roman Bolshakov wrote: > > > > On Sun, Apr 07, 2019 at 05:28:39PM +0530, Sukrit Bhatnagar wrote: > > > Keep the calls made to synchronize cpu by all hypervisors in one place > > > inside cpu_synchroni

Re: [Qemu-devel] [PATCH for-4.1] q35: acpi: do not create dummy MCFG table

2019-04-10 Thread Igor Mammedov
On Wed, 10 Apr 2019 22:01:53 +0800 Wei Yang wrote: > On Wed, Apr 10, 2019 at 11:08:33AM +0200, Igor Mammedov wrote: > >On Wed, 10 Apr 2019 09:12:31 +0800 > >Wei Yang wrote: > > > >> On Tue, Apr 09, 2019 at 05:00:37PM +0200, Igor Mammedov wrote: > >> >Dummy table (with signature "QEMU") creat

Re: [Qemu-devel] [PATCH] migration: Fix handling fd protocol

2019-04-10 Thread Yury Kotov
Hi, 10.04.2019, 16:58, "Dr. David Alan Gilbert" : > * Yury Kotov (yury-ko...@yandex-team.ru) wrote: >>  Currently such case is possible for incoming: >>  QMP: add-fd (fdset = 0, fd of some file): >>  adds fd to fdset 0 and returns QEMU's fd (e.g. 33) >>  QMP: migrate-incoming (uri = "fd:33"):

Re: [Qemu-devel] [PATCH for-4.1] q35: acpi: do not create dummy MCFG table

2019-04-10 Thread Wei Yang
On Tue, Apr 09, 2019 at 05:00:37PM +0200, Igor Mammedov wrote: >Dummy table (with signature "QEMU") creation came from original SeaBIOS >codebase. And QEMU would have to keep it around if there were Q35 machine >that depended on keeping ACPI tables blob constant size. Luckily there >were no version

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Cornelia Huck
On Wed, 10 Apr 2019 10:03:01 -0400 (EDT) Pankaj Gupta wrote: > > > > On Wed, Apr 10, 2019 at 09:38:22AM +0530, Pankaj Gupta wrote: > > > This patch adds virtio-pmem driver for KVM guest. > > > > > > Guest reads the persistent memory range information from > > > Qemu over VIRTIO and registers

[Qemu-devel] Questions about acpi interrupt link device's ‘_PRS' field

2019-04-10 Thread Li Qiang
Hi all, I see the link device ‘_PRS’ uses irq line 5, 10, 11 in ‘build_link_dev’ function. But I never see the 5 lines uses in the guest, just uses 10 and 11. Why this happen? Maybe related with the guest? Thanks, Li Qiang

Re: [Qemu-devel] [PATCH 01/10] block/pflash_cfi02: Add test for supported commands

2019-04-10 Thread Thomas Huth
On 09/04/2019 17.37, Stephen Checkoway wrote: > > > On Apr 9, 2019, at 02:13, Thomas Huth wrote: > >> We'd like to get rid of global_qtest in the long run (since it is >> causing trouble for tests that run multiple instances of QEMU in >> parallel, e.g. migration tests)... so if it is feasible,

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Yuval Shaia
On Wed, Apr 10, 2019 at 02:24:26PM +0200, Cornelia Huck wrote: > On Wed, 10 Apr 2019 09:38:22 +0530 > Pankaj Gupta wrote: > > > This patch adds virtio-pmem driver for KVM guest. > > > > Guest reads the persistent memory range information from > > Qemu over VIRTIO and registers it on nvdimm_bus.

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Yuval Shaia
> + > +static int virtio_pmem_probe(struct virtio_device *vdev) > +{ > + int err = 0; > + struct resource res; > + struct virtio_pmem *vpmem; > + struct nvdimm_bus *nvdimm_bus; > + struct nd_region_desc ndr_desc = {}; > + int nid = dev_to_node(&vdev->dev); > + struct nd_

Re: [Qemu-devel] [Qemu-block] [PATCH] block/io.c: fix for the allocation failure

2019-04-10 Thread Stefan Hajnoczi
On Mon, Apr 08, 2019 at 12:14:49PM +0200, Kevin Wolf wrote: > Am 08.04.2019 um 12:04 hat Kevin Wolf geschrieben: > > Am 08.04.2019 um 11:44 hat Andrey Shinkevich geschrieben: > > > > > > > > > On 06/04/2019 01:50, John Snow wrote: > > > > > > > > > > > > On 4/5/19 10:24 AM, Andrey Shinkevich wr

Re: [Qemu-devel] [PATCH for-4.0 v2 2/2] roms: Allow the EDK2_EFIROM variable to be overridden

2019-04-10 Thread Laszlo Ersek
On 04/10/19 08:25, Olaf Hering wrote: > Am Mon, 8 Apr 2019 13:05:07 +0200 > schrieb Laszlo Ersek : > >> Then build scripts could be updated to invoke: >> >> make -C roms \ >> EDK2_BASETOOLS_OPTFLAGS='...' \ >> EDK2_BASETOOLS_LDFLAGS='...' \ >> efirom > > The question remains: 'But why

Re: [Qemu-devel] [PATCH v2 for-4.0?] aio-posix: ensure poll mode is left when aio_notify is called

2019-04-10 Thread Stefan Hajnoczi
On Tue, Apr 09, 2019 at 02:28:23PM +0200, Paolo Bonzini wrote: Why is this 4.0 material? It's not a 4.0 regression and tweaking the event loop is risky. I suggest waiting for 4.1. > With aio=thread, adaptive polling makes latency worse rather than > better, because it delays the execution of th

Re: [Qemu-devel] [PATCH for-4.1] roms: Correct the EDK2_BASETOOLS_OPTFLAGS variable description

2019-04-10 Thread Laszlo Ersek
On 04/10/19 06:57, Philippe Mathieu-Daudé wrote: > In commit 1cab464136b4 we incorrectly described the > EDK2_BASETOOLS_OPTFLAGS can pass CPPFLAGS and CFLAGS > options to the EDK2 build tools, but it only expands > the CFLAGS (not to the CPPFLAGS). > Update the description to be more accurate. > >

Re: [Qemu-devel] [PATCH] hostmem-file: warn when memory-backend-file, share=on and in incoming migration

2019-04-10 Thread Dr. David Alan Gilbert
* Catherine Ho (catherine.h...@gmail.com) wrote: > Hi Igor > > > On Mon, 8 Apr 2019 at 18:35, Igor Mammedov wrote: > > > On Sun, 7 Apr 2019 22:19:05 -0400 > > Catherine Ho wrote: > > > > > Currently it is not forbidden to use "-object > > memory-backend-file,share=on" > > > and together with

Re: [Qemu-devel] [PATCH 3/3] xen-bus / xen-block: add support for event channel polling

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:17PM +0100, Paul Durrant wrote: > This patch introduces a poll callback for event channel fd-s and uses > this to invoke the channel callback function. > > To properly support polling, it is necessary for the event channel callback > function to return a boolean sayin

Re: [Qemu-devel] [PATCH for-4.1] q35: acpi: do not create dummy MCFG table

2019-04-10 Thread Igor Mammedov
On Wed, 10 Apr 2019 22:27:56 +0800 Wei Yang wrote: [...] > >@@ -2411,19 +2410,7 @@ build_mcfg_q35(GArray *table_data, BIOSLinker > >*linker, AcpiMcfgInfo *info) > > mcfg->allocation[0].start_bus_number = 0; > > mcfg->allocation[0].end_bus_number = PCIE_MMCFG_BUS(info->mcfg_size - > > 1)

Re: [Qemu-devel] [PATCH for-4.1] roms: Correct the EDK2_BASETOOLS_OPTFLAGS variable description

2019-04-10 Thread Philippe Mathieu-Daudé
On 4/10/19 4:58 PM, Laszlo Ersek wrote: > On 04/10/19 06:57, Philippe Mathieu-Daudé wrote: >> In commit 1cab464136b4 we incorrectly described the >> EDK2_BASETOOLS_OPTFLAGS can pass CPPFLAGS and CFLAGS >> options to the EDK2 build tools, but it only expands >> the CFLAGS (not to the CPPFLAGS). >> U

Re: [Qemu-devel] [PATCH v2 for-4.0?] aio-posix: ensure poll mode is left when aio_notify is called

2019-04-10 Thread Stefan Hajnoczi
On Tue, Apr 09, 2019 at 02:28:23PM +0200, Paolo Bonzini wrote: > With aio=thread, adaptive polling makes latency worse rather than > better, because it delays the execution of the ThreadPool's > completion bottom half. > > event_notifier_poll() does run while polling, detecting that > a bottom hal

Re: [Qemu-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Anthony PERARD
On Wed, Apr 10, 2019 at 04:20:05PM +0100, Paul Durrant wrote: > > -Original Message- > > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > > Sent: 10 April 2019 13:57 > > To: Paul Durrant > > Cc: qemu-devel@nongnu.org; qemu-bl...@nongnu.org; > > xen-de...@lists.xenproject.org; Ste

[Qemu-devel] [PATCH for-4.1] commit: Make base read-only if there is an early failure

2019-04-10 Thread Alberto Garcia
You can reproduce this by passing an invalid filter-node-name (like "1234") to block-commit. In this case the base image is put in read-write mode but is never reset back to read-only. Signed-off-by: Alberto Garcia --- block/commit.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/c

Re: [Qemu-devel] [PATCH 0/2] qemu-img convert: ignore read errors

2019-04-10 Thread Max Reitz
On 09.04.19 16:56, Andrey Shinkevich wrote: > The 'qemu-img convert' new command option 'force read' with the key '-R' > allows converting a damaged image to get all the available information > in case of the read errors. The program reports read errors and continue > the image conversion. The user

Re: [Qemu-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 13:57 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; qemu-bl...@nongnu.org; > xen-de...@lists.xenproject.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz > Subje

Re: [Qemu-devel] [PATCH for-4.1] commit: Make base read-only if there is an early failure

2019-04-10 Thread Eric Blake
On 4/10/19 10:24 AM, Alberto Garcia wrote: > You can reproduce this by passing an invalid filter-node-name (like > "1234") to block-commit. In this case the base image is put in > read-write mode but is never reset back to read-only. > Is it worth iotest coverage? > Signed-off-by: Alberto Garcia

Re: [Qemu-devel] [PATCH] hostmem-file: warn when memory-backend-file, share=on and in incoming migration

2019-04-10 Thread Catherine Ho
Hi Dr. David On Wed, 10 Apr 2019 at 22:59, Dr. David Alan Gilbert wrote: > * Catherine Ho (catherine.h...@gmail.com) wrote: > > Hi Igor > > > > > > On Mon, 8 Apr 2019 at 18:35, Igor Mammedov wrote: > > > > > On Sun, 7 Apr 2019 22:19:05 -0400 > > > Catherine Ho wrote: > > > > > > > Currently i

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Pankaj Gupta
> > > This patch adds virtio-pmem driver for KVM guest. > > > > Guest reads the persistent memory range information from > > Qemu over VIRTIO and registers it on nvdimm_bus. It also > > creates a nd_region object with the persistent memory > > range information so that existing 'nvdimm/pmem' dr

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Pankaj Gupta
> > > > > This patch adds virtio-pmem driver for KVM guest. > > > > > > Guest reads the persistent memory range information from > > > Qemu over VIRTIO and registers it on nvdimm_bus. It also > > > creates a nd_region object with the persistent memory > > > range information so that existing 'n

Re: [Qemu-devel] [RFC-PATCH] Introducing virtio-example device.

2019-04-10 Thread Yoni Bettan
On 4/9/19 4:17 PM, Stefan Hajnoczi wrote: On Mon, Apr 01, 2019 at 02:18:43PM +0300, Yoni Bettan wrote: The main goal is to add an example device to Qemu to be used as template or guideline for contributors when they wish to create a new virtio device. Another reason for this device is to docu

[Qemu-devel] [PATCH for-4.0-maybe] device_tree: Fix integer overflowing in load_device_tree()

2019-04-10 Thread Philippe Mathieu-Daudé
On 4/10/19 8:34 AM, Alistair Francis wrote: > On Tue, Apr 9, 2019 at 10:59 PM Markus Armbruster wrote: >> Philippe Mathieu-Daudé writes: >>> On 4/10/19 7:28 AM, Markus Armbruster wrote: Philippe Mathieu-Daudé writes: > On 4/9/19 7:40 PM, Markus Armbruster wrote: >> If the value of g

Re: [Qemu-devel] [PATCH] xen-block: support feature-large-sector-size

2019-04-10 Thread Anthony PERARD
On Tue, Apr 09, 2019 at 05:40:38PM +0100, Paul Durrant wrote: > A recent Xen commit [1] clarified the semantics of sector based quantities > used in the blkif protocol such that it is now safe to create a xen-block > device with a logical_block_size != 512, as long as the device only > connects to

Re: [Qemu-devel] [PATCH] xen-block: support feature-large-sector-size

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 16:52 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; xen-de...@lists.xenproject.org; > qemu-bl...@nongnu.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz > Subje

Re: [Qemu-devel] Questions about acpi interrupt link device's ‘_PRS' field

2019-04-10 Thread Paolo Bonzini
On 10/04/19 16:33, Li Qiang wrote: > Hi all, > >   > > I see the link device ‘_PRS’  uses irq line 5, 10, 11 in > ‘build_link_dev’ function. > > But I never see the 5 lines uses in the guest, just uses 10 and 11. > > Why this happen?  Maybe related with the guest? Because the MADT table tells

Re: [Qemu-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 16:23 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; qemu-bl...@nongnu.org; > xen-de...@lists.xenproject.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz > Subje

[Qemu-devel] [PATCH] iotests: Let 245 pass on tmpfs

2019-04-10 Thread Max Reitz
tmpfs does not support O_DIRECT. Detect this case, and skip flipping @direct if the filesystem does not support it. Fixes: bf3e50f6239090e63a8ffaaec971671e66d88e07 Signed-off-by: Max Reitz --- tests/qemu-iotests/245 | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/

[Qemu-devel] [PATCH] target/riscv: Remove unused include of riscv_htif.h for virt board

2019-04-10 Thread Jonathan Behrens
Unless I'm missing something, the virt board doesn't support HTIF and should not be including this header. Jonathan Signed-off-by: Jonathan Behrens --- hw/riscv/virt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c index fc4c6b306e..3526463034 100644 --- a/

Re: [Qemu-devel] [PATCH for-4.0?] iotests: Let 245 pass on tmpfs

2019-04-10 Thread Eric Blake
On 4/10/19 11:29 AM, Max Reitz wrote: > tmpfs does not support O_DIRECT. Detect this case, and skip flipping > @direct if the filesystem does not support it. > > Fixes: bf3e50f6239090e63a8ffaaec971671e66d88e07 > Signed-off-by: Max Reitz > --- > tests/qemu-iotests/245 | 10 -- > 1 file c

Re: [Qemu-devel] [PATCH for-4.0?] iotests: Let 245 pass on tmpfs

2019-04-10 Thread Max Reitz
On 10.04.19 18:39, Eric Blake wrote: > On 4/10/19 11:29 AM, Max Reitz wrote: >> tmpfs does not support O_DIRECT. Detect this case, and skip flipping >> @direct if the filesystem does not support it. >> >> Fixes: bf3e50f6239090e63a8ffaaec971671e66d88e07 >> Signed-off-by: Max Reitz >> --- >> tests

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Michael S. Tsirkin
On Wed, Apr 10, 2019 at 04:31:39PM +0200, Cornelia Huck wrote: > On Wed, 10 Apr 2019 10:03:01 -0400 (EDT) > Pankaj Gupta wrote: > > > > > > > On Wed, Apr 10, 2019 at 09:38:22AM +0530, Pankaj Gupta wrote: > > > > This patch adds virtio-pmem driver for KVM guest. > > > > > > > > Guest reads the

Re: [Qemu-devel] [Qemu-block] [RFC PATCH] aio: Add a knob to always poll if there are in-flight requests

2019-04-10 Thread Sergio Lopez
Stefan Hajnoczi writes: > On Tue, Apr 02, 2019 at 02:19:08PM +0200, Sergio Lopez wrote: >> The polling mode in aio_poll is able to trim down ~20us on the average >> request latency, but it needs manual fine tuning to adjust it to the >> characteristics of the storage. >> >> Here we add a new kn

Re: [Qemu-devel] [PATCH v5 2/5] virtio-pmem: Add virtio pmem driver

2019-04-10 Thread Cornelia Huck
On Wed, 10 Apr 2019 12:46:12 -0400 "Michael S. Tsirkin" wrote: > On Wed, Apr 10, 2019 at 04:31:39PM +0200, Cornelia Huck wrote: > > On Wed, 10 Apr 2019 10:03:01 -0400 (EDT) > > Pankaj Gupta wrote: > > > > > > > > > > On Wed, Apr 10, 2019 at 09:38:22AM +0530, Pankaj Gupta wrote: > > > > >

Re: [Qemu-devel] [PATCH for-4.0?] iotests: Let 245 pass on tmpfs

2019-04-10 Thread Peter Maydell
On Wed, 10 Apr 2019 at 17:40, Max Reitz wrote: > > On 10.04.19 18:39, Eric Blake wrote: > > On 4/10/19 11:29 AM, Max Reitz wrote: > >> tmpfs does not support O_DIRECT. Detect this case, and skip flipping > >> @direct if the filesystem does not support it. > >> > >> Fixes: bf3e50f6239090e63a8ffaae

Re: [Qemu-devel] [PATCH] hostmem-file: warn when memory-backend-file, share=on and in incoming migration

2019-04-10 Thread Dr. David Alan Gilbert
* Catherine Ho (catherine.h...@gmail.com) wrote: > Hi Dr. David > > On Wed, 10 Apr 2019 at 22:59, Dr. David Alan Gilbert > wrote: > > > * Catherine Ho (catherine.h...@gmail.com) wrote: > > > Hi Igor > > > > > > > > > On Mon, 8 Apr 2019 at 18:35, Igor Mammedov wrote: > > > > > > > On Sun, 7 Apr

Re: [Qemu-devel] [PATCH v2 13/13] qemu-iotests: Test the x-blockdev-reopen QMP command

2019-04-10 Thread Max Reitz
On 06.03.19 19:11, Alberto Garcia wrote: > This patch adds several tests for the x-blockdev-reopen QMP command. > > Signed-off-by: Alberto Garcia > --- > tests/qemu-iotests/243 | 991 > + > tests/qemu-iotests/243.out | 5 + > tests/qemu-iotests/

Re: [Qemu-devel] [PATCH RESEND v2 2/2] cpus: move hvf_cpu_synchronize* calls to cpu_synchronize* functions

2019-04-10 Thread Paolo Bonzini
On 10/04/19 16:02, Roman Bolshakov wrote: > I've applied, built and tested both sequentially. Applying and running > with patch 1/2 alone doesn't result in the behavior I mentioned. I also > tried to apply only the first hunk that moves hvf_cpu_synchronize_state > into cpu_synchronize_state and it

Re: [Qemu-devel] [PATCH] qemu-img: fix .hx and .texi disparity

2019-04-10 Thread John Snow
On 4/10/19 1:39 AM, Markus Armbruster wrote: > John Snow writes: > >> It turns out that having options listed in three places continues to be >> a bad idea. I'm still toying with the idea of an improved infrastructure >> here, but in the meantime, another bandaid. >> >> There are three locatio

Re: [Qemu-devel] [Qemu-block] [PULL 07/14] iotests: Add 241 to test NBD on unaligned images

2019-04-10 Thread Max Reitz
On 01.04.19 16:08, Eric Blake wrote: > Add a test for the NBD client workaround in the previous patch. It's > not really feasible for an iotest to assume a specific tracing engine, > so we can't really probe trace_nbd_parse_blockstatus_compliance to see > if the server was fixed vs. whether the cl

Re: [Qemu-devel] [PATCH] roms: List and describe the Makefile 'clean' rule

2019-04-10 Thread Laszlo Ersek
On 04/10/19 07:36, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé > --- > roms/Makefile | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/roms/Makefile b/roms/Makefile > index 1ff78b63bb3..f55c4a2d3bb 100644 > --- a/roms/Makefile > +++ b/r

Re: [Qemu-devel] [PATCH v2 6/8] migration: Add multifd-compress parameter

2019-04-10 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Signed-off-by: Juan Quintela > > --- > Rename it to NONE > --- > hmp.c| 17 + > hw/core/qdev-properties.c| 11 +++ > include/hw/qdev-properties.h | 1 + > migration/migration.c| 16 +++

Re: [Qemu-devel] [PATCH RFC 2/3] pxtool: Add new qemu-img command info generation tool

2019-04-10 Thread John Snow
On 4/10/19 1:54 AM, Markus Armbruster wrote: > John Snow writes: > >> Presently we use hxtool and a .hx format to generate a few things like >> the qemu_img subcommand dispatch table, the qemu_img help() display output, >> and a help output in qemu-img.texi. >> >> Unfortunately, this means tha

Re: [Qemu-devel] [Qemu-block] [PULL 07/14] iotests: Add 241 to test NBD on unaligned images

2019-04-10 Thread Eric Blake
On 4/10/19 12:45 PM, Max Reitz wrote: > On 01.04.19 16:08, Eric Blake wrote: >> Add a test for the NBD client workaround in the previous patch. It's >> not really feasible for an iotest to assume a specific tracing engine, >> so we can't really probe trace_nbd_parse_blockstatus_compliance to see >

[Qemu-devel] [PATCH] hmp: delvm: use hmp_handle_error

2019-04-10 Thread Cole Robinson
This gives us the consistent 'Error:' prefix added in 66363e9a43f, which helps users like libvirt who still need to scrape hmp error messages to detect failure. Signed-off-by: Cole Robinson --- hmp.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hmp.c b/hmp.c index 8

Re: [Qemu-devel] [PATCH for-4.1] roms: Correct the EDK2_BASETOOLS_OPTFLAGS variable description

2019-04-10 Thread Laszlo Ersek
On 04/10/19 17:10, Philippe Mathieu-Daudé wrote: > On 4/10/19 4:58 PM, Laszlo Ersek wrote: >> On 04/10/19 06:57, Philippe Mathieu-Daudé wrote: >>> In commit 1cab464136b4 we incorrectly described the >>> EDK2_BASETOOLS_OPTFLAGS can pass CPPFLAGS and CFLAGS >>> options to the EDK2 build tools, but it

Re: [Qemu-devel] [PATCH] qemu-img: fix .hx and .texi disparity

2019-04-10 Thread Eric Blake
On 4/10/19 12:37 PM, John Snow wrote: > > > On 4/10/19 1:39 AM, Markus Armbruster wrote: >> John Snow writes: >> >>> It turns out that having options listed in three places continues to be >>> a bad idea. I'm still toying with the idea of an improved infrastructure >>> here, but in the meantime,

Re: [Qemu-devel] [PATCH v2] target/i386: kvm: add VMX migration blocker

2019-04-10 Thread Cole Robinson
On 11/20/18 6:44 AM, Dr. David Alan Gilbert wrote: > * Paolo Bonzini (pbonz...@redhat.com) wrote: >> Nested VMX does not support live migration yet. Add a blocker >> until that is worked out. >> >> Nested SVM only does not support it, but unfortunately it is >> enabled by default for -cpu host so

Re: [Qemu-devel] [PATCH for-4.0?] hmp: delvm: use hmp_handle_error

2019-04-10 Thread Eric Blake
On 4/10/19 1:03 PM, Cole Robinson wrote: > This gives us the consistent 'Error:' prefix added in 66363e9a43f, > which helps users like libvirt who still need to scrape hmp error > messages to detect failure. > > Signed-off-by: Cole Robinson > --- > hmp.c | 7 --- > 1 file changed, 4 insertio

[Qemu-devel] QEMU Logo question

2019-04-10 Thread Barajas, Felipe
Hi, I am trying to understand who in the QEMU community can give me permission to use the QEMU logo in a whitepaper. I am an application engineer at Intel and I have prepared some benchmarks using QEMU as a solution accelerated by Intel hardware (CPUs, SSDs, etc) I would like to use the QEMU log

Re: [Qemu-devel] [PATCH for-4.0?] hmp: delvm: use hmp_handle_error

2019-04-10 Thread Cole Robinson
On 4/10/19 2:27 PM, Eric Blake wrote: > On 4/10/19 1:03 PM, Cole Robinson wrote: >> This gives us the consistent 'Error:' prefix added in 66363e9a43f, >> which helps users like libvirt who still need to scrape hmp error >> messages to detect failure. >> >> Signed-off-by: Cole Robinson >> --- >> h

Re: [Qemu-devel] [PATCH for-4.1 v2] qemu-img: Saner printing of large file sizes

2019-04-10 Thread Max Reitz
On 01.04.19 16:57, Eric Blake wrote: > Disk sizes close to INT64_MAX cause overflow, for some pretty > ridiculous output: > > $ ./nbdkit -U - memory size=$((2**63 - 512)) --run 'qemu-img info $nbd' > image: nbd+unix://?socket=/tmp/nbdkitHSAzNz/socket > file format: raw > virtual size: -838

Re: [Qemu-devel] [PATCH] configure: Automatically fall back to TCI on non-release architectures

2019-04-10 Thread Stefan Weil
On 10.04.19 10:22, Thomas Huth wrote: > Additionally, I think it should be possible to compile with the > x86_64-linux-user target and then to run "make check-tcg" ... however, > that currently crashes with: > > TODO qemu/tcg/tci.c:859: tcg_qemu_tb_exec() > qemu/tcg/tci.c:859: tcg fatal error > qe

  1   2   >