Re: [Qemu-devel] [PATCH v9 23/27] gdbstub: Implement qemu physical memory mode

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Add a new query/set which changes the memory GDB sees to physical memory > only. > > gdb> maint packet qqemu.PhyMemMode > will reply the current phy_mem_mode state (1 for enabled, 0 for disabled) > gdb> maint packet Qqemu.PhyMemMode:1 > Will make GDB read/write only to

Re: [Qemu-devel] [PATCH v12 12/12] MAINTAINERS: Add RX

2019-05-15 Thread Richard Henderson
On 5/13/19 11:14 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > MAINTAINERS | 19 +++ > 1 file changed, 19 insertions(+) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v12 00/12] Add RX archtecture support

2019-05-15 Thread Richard Henderson
On 5/13/19 11:14 PM, Yoshinori Sato wrote: > This patch series is added Renesas RX target emulation. > > I fixed the ROM address because v11 was incorrect. > > My git repository is bellow. > git://git.pf.osdn.net/gitroot/y/ys/ysato/qemu.git tags/rx-20190514 > > Testing binaries bellow. > u-boot

Re: [Qemu-devel] [PATCH v9 20/27] gdbstub: Implement target halted (? pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 36 ++-- > 1 file changed, 26 insertions(+), 10 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 2fd0d66f4d..d678191705 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -2239,13

Re: [Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 07:13:56PM +0200, Stefan Priebe - Profihost AG wrote: > Hello list, > > i've updated my host to kernel 4.19.43 and applied the following patch > to my qemu 2.12.1: > https://bugzilla.suse.com/attachment.cgi?id=798722 > > But my guest running 4.19.43 still says: >

Re: [Qemu-devel] [PATCH] configure: Disable slirp if --disable-system

2019-05-15 Thread Aleksandar Markovic
On May 15, 2019 12:07 PM, "Peter Maydell" wrote: > > On Tue, 14 May 2019 at 20:16, Aleksandar Markovic > wrote: > > > > On May 13, 2019 11:14 PM, "Richard Henderson" < richard.hender...@linaro.org> > > wrote: > > > > > > On 5/11/19 5:47 AM, Aleksandar Markovic wrote: > > > > If no, the patch

Re: [Qemu-devel] [PATCH v3] chardev/char-i2c: Implement Linux I2C character device

2019-05-15 Thread Stefan Hajnoczi
On Fri, May 10, 2019 at 07:04:10PM +0100, Ernest Esene wrote: > Add support for Linux I2C character device for I2C device passthrough > For example: > -chardev i2c,address=0x46,path=/dev/i2c-N,id=i2c-chardev > > QEMU supports emulation of I2C devices in software but currently can't > passthrough

Re: [Qemu-devel] Question about the vfio device interrupt

2019-05-15 Thread Maxim Levitsky
Hi! Let me add my 0.2 cents to this discussion by sharing my summary of INTx emulation in VFIO: COMMON: * real interrupt is signaled in the kernel * vfio-pci kernel interrupt hanlder (vfio_intx_handler), masks the INTx flag * vfio-pci kernel interrupt handler triggers an eventfd, if the

Re: [Qemu-devel] Question about the vfio device interrupt

2019-05-15 Thread Li Qiang
Alex Williamson 于2019年5月15日周三 下午10:54写道: > On Wed, 15 May 2019 17:27:52 +0300 > Maxim Levitsky wrote: > > > Hi! > > > > Let me add my 0.2 cents to this discussion by sharing my summary of INTx > emulation in VFIO: > > > > COMMON: > > * real interrupt is signaled in the kernel > > * vfio-pci

Re: [Qemu-devel] [PATCH v2] net/slirp: fix the IPv6 prefix length error message

2019-05-15 Thread Philippe Mathieu-Daudé
On 5/15/19 3:01 PM, Stefano Garzarella wrote: > On Wed, May 15, 2019 at 02:33:03PM +0200, Markus Armbruster wrote: >> Stefano Garzarella writes: >> >>> Reword and add a missing parentheses at the end of the >>> error message. >>> >>> Signed-off-by: Stefano Garzarella >>> Reviewed-by: Marc-André

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 10:22:08AM -0700, Richard Henderson wrote: > On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > > On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: > >> For user-only, we require only the random number bits of the > >> crypto subsystem. > >> > >> We need to

Re: [Qemu-devel] [RISU PATCH 0/5] Fix RISU build for i386

2019-05-15 Thread Jan Bobek
Hi Alex, I'm very sorry for the late reply, your emails got mixed up with everything else in qemu-devel; I didn't setup my mail filters very well (my bad). On 4/25/19 9:45 AM, Alex Bennée wrote: > > Jan Bobek writes: > >> Hi all, >> > >> Thanks, >> -Jan Bobek >> >> P.S. This is my first time

Re: [Qemu-devel] Question about the vfio device interrupt

2019-05-15 Thread Alex Williamson
On Wed, 15 May 2019 17:27:52 +0300 Maxim Levitsky wrote: > Hi! > > Let me add my 0.2 cents to this discussion by sharing my summary of INTx > emulation in VFIO: > > COMMON: > * real interrupt is signaled in the kernel > * vfio-pci kernel interrupt hanlder (vfio_intx_handler), masks the INTx

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Wed, May 15, 2019 at 08:34:17AM +0200, Markus Armbruster wrote: >> Jason Wang writes: >> >> > On 2019/5/14 下午8:18, Markus Armbruster wrote: >> >> Peter Maydell writes: >> >> >> >>> On Mon, 13 May 2019 at 14:21, Markus Armbruster >> >>> wrote: >> Perhaps

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 04:54:04PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Wed, May 15, 2019 at 08:34:17AM +0200, Markus Armbruster wrote: > >> Jason Wang writes: > >> > >> > On 2019/5/14 下午8:18, Markus Armbruster wrote: > >> >> Peter Maydell writes: > >> >> > >>

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Laurent Vivier
On 14/05/2019 21:16, Richard Henderson wrote: > For user-only, we require only the random number bits of the > crypto subsystem. > > We need to preserve --static linking, which for many recent Linux > distributions precludes using GnuTLS or GCrypt. Instead, use our > random-platform module

[Qemu-devel] [PATCH] spapr/xive: Sanity checks of OV5 during CAS

2019-05-15 Thread Greg Kurz
If a machine is started with ic-mode=xive but the guest only knows about XICS, eg. an RHEL 7.6 guest, the kernel panics. This is expected but a bit unfortunate since the crash doesn't provide much information for the end user to guess what's happening. Detect that during CAS and exit QEMU with a

[Qemu-devel] [PATCH 1/6] tests/libqos: Get rid of global_qtest dependency in qvring_init()

2019-05-15 Thread Thomas Huth
Library functions should not depend on global_qtest functions like writew() and writeq(), so that they can also be used in tests that deal with multiple QTestStates at the same time (like migration tests). Signed-off-by: Thomas Huth --- tests/libqos/virtio-mmio.c | 2 +-

[Qemu-devel] [PATCH 0/6] Get rid of global_qtest in q35-, qom-, numa- and more tests

2019-05-15 Thread Thomas Huth
Here are some more patches that get rid of global_qtest and related functions in some of the qtests (hacked along the way while waiting for other compilation and test processes to finish). A global variable like global_qtest is very problematic in tests that track multiple test states (like

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Richard Henderson
On 5/15/19 7:54 AM, Markus Armbruster wrote: > We don't want to deprecate qemu-bridge-helper while libvirt has a > sensible use for it. > > We can still deprecate -netdev tap parameter "helper" and -netdev bridge > entirely. > > Once they're gone, qemu-bridge-helper wull have no user within

[Qemu-devel] [Bug 1829079] Re: Can't build static on ARM (Raspbian)

2019-05-15 Thread Peter Maydell
No, --disable-tools won't change the qemu-* binaries that are built. It just stops us trying to build some binaries like the 'ivshmem-client' one that was causing a problem for you. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [PATCH 0/2] x86/cpu: add "md-clear" feature for MDS security flaws

2019-05-15 Thread Daniel P . Berrangé
This patch series provides the new "md-clear" feature that is used for mitigation with CVE-2018-12126, CVE-2018-12127, CVE-2018-12130, CVE-2019-11091. Assuming you have the updated microcode and kernel to support the md-clear feature, then using "-cpu host" will expose the new feature to guests.

Re: [Qemu-devel] [PATCH v9 12/27] gdbstub: Implement read memory (m pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 48 > 1 file changed, 32 insertions(+), 16 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 8dc2e1d507..daa602edc3 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@

Re: [Qemu-devel] [PATCH v9 17/27] gdbstub: Implement v commands with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 170 +++--- > 1 file changed, 110 insertions(+), 60 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 9b0556f8be..d56d0fd235 100644 > --- a/gdbstub.c > +++ b/gdbstub.c >

Re: [Qemu-devel] [PATCH v9 24/27] gdbstub: Add another handler for setting qemu.sstep

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Follow GDB general query/set packet conventions, qemu.sstep can now > be set with the following command as well: > gdb> maint packet Qqemu.sstep:Value I;m not sure about exposing internal values to a protocol like this. Maybe text based flags would be better? > >

[Qemu-devel] [PATCH 3/6] tests/numa-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/numa-test.c | 53 +++ 1 file changed,

[Qemu-devel] [PATCH 4/6] tests/qom-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/qom-test.c | 28 +--- 1 file changed, 13 insertions(+), 15

Re: [Qemu-devel] Failing QEMU iotest 175

2019-05-15 Thread Stefan Hajnoczi
On Mon, May 06, 2019 at 12:44:56PM -0500, Eric Blake wrote: > On 5/4/19 1:51 AM, Thomas Huth wrote: > > >> Hmm, it's passing for me on ext4, but that probably means we have > >> different configuration parameters. I'm not sure how to easily show what > >> parameters a particular ext4 partition

Re: [Qemu-devel] [PATCH v9 11/27] gdbstub: Implement write memory (M pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron Reviewed-by: Alex Bennée > --- > gdbstub.c | 51 +-- > 1 file changed, 33 insertions(+), 18 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index e9a3d0c2bc..8dc2e1d507 100644 > ---

[Qemu-devel] [Bug 1829079] Re: Can't build static on ARM (Raspbian)

2019-05-15 Thread Dariusz Zyzański
Well, the symlinks didn't resolve the issue. I just tried them to see if this will solve the issue. And I installed a lot of packages, blindly trying to solve this issue. Using full Raspbian instead of Raspbian Lite was also an attempt to do so. I'm just an advanced Linux user, not a developer! I

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Markus Armbruster
Markus Armbruster writes: > Daniel P. Berrangé writes: > >> On Wed, May 15, 2019 at 08:34:17AM +0200, Markus Armbruster wrote: >>> Jason Wang writes: >>> >>> > On 2019/5/14 下午8:18, Markus Armbruster wrote: >>> >> Peter Maydell writes: >>> >> >>> >>> On Mon, 13 May 2019 at 14:21, Markus

[Qemu-devel] [PATCH 2/2] docs: recommend use of md-clear feature on all Intel CPUs

2019-05-15 Thread Daniel P . Berrangé
Update x86 CPU model guidance to recommend that the md-clear feature is manually enabled with all Intel CPU models, when supported by the host microcode. Signed-off-by: Daniel P. Berrangé --- docs/qemu-cpu-models.texi | 12 1 file changed, 12 insertions(+) diff --git

Re: [Qemu-devel] [PATCH v3 5/8] migration: Add multifd-compress parameter

2019-05-15 Thread Markus Armbruster
Juan Quintela writes: > Signed-off-by: Juan Quintela > > --- > Rename it to NONE > Fix typos (dave) > --- > hmp.c| 17 + > hw/core/qdev-properties.c| 13 + > include/hw/qdev-properties.h | 1 + > migration/migration.c| 16

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Richard Henderson
On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: >> For user-only, we require only the random number bits of the >> crypto subsystem. >> >> We need to preserve --static linking, which for many recent Linux >> distributions precludes

Re: [Qemu-devel] [PATCH v9 22/27] gdbstub: Implement generic query qemu.Supported

2019-05-15 Thread Alex Bennée
Jon Doron writes: > qemu.Supported query reply back with the supported qemu query/set > commands (commands are seperated with a semicolon from each other). > > gdb> maint packet qqemu.Supported > > Signed-off-by: Jon Doron > --- > gdbstub.c | 9 + > 1 file changed, 9 insertions(+) >

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Richard Henderson
On 5/15/19 9:55 AM, Markus Armbruster wrote: > Proposal: > > 1. Add qemu-bridge-helper.c to Jason's "Network device backends" > > 2. Deprecate -netdev tap parameter "helper" > > 3. Improve documentation of -netdev bridge > > 4. Create a manual page for qemu-bridge-helper that also covers >

[Qemu-devel] [PATCH 6/6] tests/hd-geo-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/hd-geo-test.c | 76 - 1 file changed,

Re: [Qemu-devel] [PATCH v9 26/27] gdbstub: Add support to read a MSR for KVM target

2019-05-15 Thread Alex Bennée
Jon Doron writes: > gdb> maint packet qqemu.kvm.Rdmsr:MsrIndex gdbserver already has a mechanism for exposing system registers see: commit 200bf5b7ffea635079cc05fdfb363372b9544ce7 Author: Abdallah Bouassida Date: Fri May 18 17:48:07 2018 +0100 for an example. As MSR's are very

[Qemu-devel] [PATCH 1/2] target/i386: define md-clear bit

2019-05-15 Thread Daniel P . Berrangé
From: Paolo Bonzini md-clear is a new CPUID bit which is set when microcode provides the mechanism to invoke a flush of various exploitable CPU buffers by invoking the VERW instruction. Signed-off-by: Paolo Bonzini --- target/i386/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [Qemu-devel] [PATCH v12 11/12] qemu/bitops.h: Add extract8 and extract16

2019-05-15 Thread Richard Henderson
On 5/13/19 11:14 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > include/qemu/bitops.h | 38 ++ > 1 file changed, 38 insertions(+) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v9 15/27] gdbstub: Implement file io (F pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: There is a bit more going on here than a simple conversion. I think we need some additional commentary about the format of the data coming back. > Signed-off-by: Jon Doron > --- > gdbstub.c | 62 +++ > 1 file changed, 40

Re: [Qemu-devel] [PATCH v9 16/27] gdbstub: Implement step (s pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron Reviewed-by: Alex Bennée > --- > gdbstub.c | 25 +++-- > 1 file changed, 19 insertions(+), 6 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 9fe130f30d..9b0556f8be 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@

Re: [Qemu-devel] [PATCH v9 13/27] gdbstub: Implement write all registers (G pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 41 +++-- > 1 file changed, 31 insertions(+), 10 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index daa602edc3..adfe39b3a3 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -1734,6

Re: [Qemu-devel] [PATCH v9 14/27] gdbstub: Implement read all registers (g pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 31 +++ > 1 file changed, 23 insertions(+), 8 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index adfe39b3a3..3478ac778d 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -1757,6 +1757,21 @@

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: > For user-only, we require only the random number bits of the > crypto subsystem. > > We need to preserve --static linking, which for many recent Linux > distributions precludes using GnuTLS or GCrypt. Instead, use our >

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Richard Henderson
On 5/15/19 9:42 AM, Laurent Vivier wrote: > On 14/05/2019 21:16, Richard Henderson wrote: >> For user-only, we require only the random number bits of the >> crypto subsystem. >> >> We need to preserve --static linking, which for many recent Linux >> distributions precludes using GnuTLS or GCrypt.

Re: [Qemu-devel] [PATCH v9 18/27] gdbstub: Implement generic query (q pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: A bit more for the commit message here as there seems to be a fair amount going on. > Signed-off-by: Jon Doron > --- > gdbstub.c | 327 ++ > 1 file changed, 327 insertions(+) > > diff --git a/gdbstub.c b/gdbstub.c > index

[Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Stefan Priebe - Profihost AG
Hello list, i've updated my host to kernel 4.19.43 and applied the following patch to my qemu 2.12.1: https://bugzilla.suse.com/attachment.cgi?id=798722 But my guest running 4.19.43 still says: Vulnerable: Clear CPU buffers attempted, no microcode; SMT Host state unknown while the host says:

Re: [Qemu-devel] [PATCH v9 21/27] gdbstub: Clear unused variables in gdb_handle_packet

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron Reviewed-by: Alex Bennée > --- > gdbstub.c | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index d678191705..8bdfae4b29 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -2259,17

[Qemu-devel] [PATCH 2/6] tests/q35-test: Make test independent of global_qtest

2019-05-15 Thread Thomas Huth
Use a local QTestState variable, so that we can finally get rid of the undesired global_qtest variable in this file, too. Signed-off-by: Thomas Huth --- tests/q35-test.c | 39 ++- 1 file changed, 18 insertions(+), 21 deletions(-) diff --git

[Qemu-devel] [PATCH 5/6] tests/device-introspect: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/device-introspect-test.c | 85 ++ 1 file changed,

Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler

2019-05-15 Thread Alex Bennée
Jon Doron writes: > This patch series refactors the old gdbstub command packets handler > with a new infrastructure which should ease extending and adding new > and missing gdb command packets. Jon, I've finished my review and things are looking pretty good. The code is a good clean-up and

[Qemu-devel] [PATCH v1 3/5] s390x/tcg: Implement VECTOR FIND ELEMENT NOT EQUAL

2019-05-15 Thread David Hildenbrand
Similar to VECTOR FIND ELEMENT EQUAL, however the search also stops on any inequality. A match for inequality seems to have precedence over a match for zero, because both elements have to be zero. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6

[Qemu-devel] [PULL 09/21] hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Move the TYPE_PIIX4_PM definition to the corresponding header, so other files can use it. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20190427144025.22880-2-phi...@redhat.com> Signed-off-by: Paolo Bonzini --- hw/acpi/piix4.c | 2 --

[Qemu-devel] [PULL 15/21] build: replace GENERATED_FILES by generated-files-y

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier When possible use generated-files-$(FLAG) to disable some targets (like KEYCODEMAP_FILES). Suggested-by: Paolo Bonzini Signed-off-by: Laurent Vivier Message-Id: <20190401141222.30034-3-lviv...@redhat.com> --- Makefile | 36

Re: [Qemu-devel] [PATCH v9 4/7] dm: enable synchronous dax

2019-05-15 Thread Dan Williams
[ add Mike and dm-devel ] Mike, any concerns with the below addition to the device-mapper-dax implementation? On Tue, May 14, 2019 at 7:58 AM Pankaj Gupta wrote: > > This patch sets dax device 'DAXDEV_SYNC' flag if all the target > devices of device mapper support synchrononous DAX. If device

[Qemu-devel] [PULL 14/21] trace: only include trace-event-subdirs when they are needed

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Some directories are built only for softmmu targets, and the related trace-event-subdirs must do the same Signed-off-by: Laurent Vivier Reviewed-by: Stefan Hajnoczi Message-Id: <20190401141222.30034-2-lviv...@redhat.com> --- Makefile.objs | 12 +++- 1 file

[Qemu-devel] Unhandled Fault when trying to access PCIE [AARCH64/VIRT]

2019-05-15 Thread Adam Parker
Hello all, I am attempting to write PCIe drivers for my OP-TEE project against the Cortex-A57 using the virt machine. I am using the default packaged qemu, 3.0.93v3.1.0-rc3-dirty and v4.0.0-rc4. I have added the PCIE device to the page table of the OS and when I try to write to PIO (0xCF8) I

Re: [Qemu-devel] [PATCH v9 1/7] libnvdimm: nd_region flush callback support

2019-05-15 Thread Dan Williams
On Tue, May 14, 2019 at 7:55 AM Pankaj Gupta wrote: > > This patch adds functionality to perform flush from guest > to host over VIRTIO. We are registering a callback based > on 'nd_region' type. virtio_pmem driver requires this special > flush function. For rest of the region types we are

[Qemu-devel] [PATCH 0/4] iotests: Make 245 faster and more reliable

2019-05-15 Thread Max Reitz
245 is a bit flakey for me, because it uses block jobs that copy 1 MB of data but have a buffer size of 512 kB, so they may be done before the test gets to do the things it wants to do while the check is running. (Rate limiting doesn’t change this.) The boring way to fix this would be to increase

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 09:38:00PM +0200, Laurent Vivier wrote: > On 15/05/2019 19:49, Daniel P. Berrangé wrote: > > On Wed, May 15, 2019 at 10:22:08AM -0700, Richard Henderson wrote: > >> On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > >>> On Tue, May 14, 2019 at 12:16:30PM -0700, Richard

[Qemu-devel] [PATCH 1/4] block: Improve "Block node is read-only" message

2019-05-15 Thread Max Reitz
This message does not make any sense when it appears as the response to making an R/W node read-only. We should detect that case and emit a different message, then. Signed-off-by: Max Reitz --- block.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH v1 4/5] s390x/tcg: Implement VECTOR ISOLATE STRING

2019-05-15 Thread David Hildenbrand
Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 + target/s390x/insn-data.def | 2 ++ target/s390x/translate_vx.inc.c | 34 ++ target/s390x/vec_string_helper.c | 41 4 files changed, 83 insertions(+)

[Qemu-devel] [PATCH v1 0/5] s390x/tcg: Vector Instruction Support Part 3

2019-05-15 Thread David Hildenbrand
This is the third part of vector instruction support for s390x. It is based on part 2, which is will send a pull-request for to Conny soon. Part 1: Vector Support Instructions Part 2: Vector Integer Instructions Part 3: Vector String Instructions Part 4: Vector Floating-Point Instructions The

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread Dan Williams
On Tue, May 14, 2019 at 8:25 AM Pankaj Gupta wrote: > > > > On 5/14/19 7:54 AM, Pankaj Gupta wrote: > > > diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig > > > index 35897649c24f..94bad084ebab 100644 > > > --- a/drivers/virtio/Kconfig > > > +++ b/drivers/virtio/Kconfig > > > @@ -42,6

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread David Hildenbrand
> + vpmem->vdev = vdev; > + vdev->priv = vpmem; > + err = init_vq(vpmem); > + if (err) { > + dev_err(>dev, "failed to initialize virtio pmem vq's\n"); > + goto out_err; > + } > + > + virtio_cread(vpmem->vdev, struct virtio_pmem_config, > +

[Qemu-devel] [PULL 13/21] sun4m: obey -vga none

2019-05-15 Thread Paolo Bonzini
Do not create a TCX if "-vga none" was passed on the command line. Remove some dead code along the way to avoid big reindentation. Signed-off-by: Paolo Bonzini --- hw/sparc/sun4m.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c

[Qemu-devel] [PULL 11/21] hw/i386/acpi: Assert a pointer is not null BEFORE using it

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Commit 72c194f7e75c added a non-null check on the 'obj' pointer. Later, commit 500b11ea5095 added code which uses the 'obj' pointer _before_ the assertion check. Move the assertion _before_ the pointer use. Signed-off-by: Philippe Mathieu-Daudé Message-Id:

[Qemu-devel] [PULL 21/21] hw/net/ne2000: Extract the PCI device from the chipset common code

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé The ne2000.c file contains functions common the the ISA and PCI devices. To allow to build with one or another, extract the PCI specific part into a new file. This fix an issue where the NE2000_ISA Kconfig had to pull the full PCI core objects. Signed-off-by:

Re: [Qemu-devel] [Qemu-arm] [PATCH v2 1/1] target/arm: Fix vector operation segfault

2019-05-15 Thread Alistair Francis
On Wed, May 15, 2019 at 6:11 AM Alex Bennée wrote: > > > Alistair Francis writes: > > > Commit 89e68b575 "target/arm: Use vector operations for saturation" > > causes this abort() when booting QEMU ARM with a Cortex-A15: > > You may want to check your email settings because when I tried to apply

[Qemu-devel] [PULL 10/21] hw/i386/acpi: Add object_resolve_type_unambiguous to improve modularity

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé When building with CONFIG_Q35=n, we get: LINKx86_64-softmmu/qemu-system-x86_64 /usr/bin/ld: hw/i386/acpi-build.o: in function `acpi_get_misc_info': /source/qemu/hw/i386/acpi-build.c:243: undefined reference to `ich9_lpc_find' collect2: error: ld

[Qemu-devel] [PULL 04/21] vl: Add missing descriptions to the VGA adapters list

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Some VGA adapters do not contain an helpful description, this can be confusing: $ qemu-system-arm -M virt -vga help none std standard VGA cirrus Cirrus VGA (default) vmware VMWare SVGA xenfb Add a description

[Qemu-devel] [PULL 17/21] build: chardev is only needed for softmmu targets

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Move the dependency from SUBDIR_RULES to SOFTMMU_SUBDIR_RULES Suggested-by: Paolo Bonzini Signed-off-by: Laurent Vivier Message-Id: <20190401141222.30034-5-lviv...@redhat.com> --- Makefile | 3 ++- Makefile.objs | 2 +- 2 files changed, 3 insertions(+), 2

Re: [Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Stefan Priebe - Profihost AG
Am 15.05.19 um 19:54 schrieb Daniel P. Berrangé: > On Wed, May 15, 2019 at 07:13:56PM +0200, Stefan Priebe - Profihost AG wrote: >> Hello list, >> >> i've updated my host to kernel 4.19.43 and applied the following patch >> to my qemu 2.12.1: >> https://bugzilla.suse.com/attachment.cgi?id=798722

[Qemu-devel] [PULL 07/21] vl: fix -sandbox parsing crash when seccomp support is disabled

2019-05-15 Thread Paolo Bonzini
From: Marc-André Lureau $ ./x86_64-softmmu/qemu-system-x86_64 -sandbox off qemu-system-x86_64: -sandbox off: There is no option group 'sandbox' Segmentation fault Commit 5780760f5e ("seccomp: check TSYNC host capability") wrapped one use of the sandbox option group to produce a sensible error,

[Qemu-devel] [PULL 01/21] hw/input: Add a CONFIG_PS2 switch for the ps2.c file

2019-05-15 Thread Paolo Bonzini
From: Thomas Huth ps2.c only needs to be compiled if we are building pckbd.c or pl050.c. Signed-off-by: Thomas Huth Message-Id: <20190411182240.5957-1-th...@redhat.com> Tested-by: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Paolo Bonzini --- hw/input/Kconfig

[Qemu-devel] [PULL 06/21] hvf: Add missing break statement

2019-05-15 Thread Paolo Bonzini
From: Chen Zhang via Qemu-devel In target/i386/hvf/hvf.c, a break statement was probably missing in `hvf_vcpu_exec()`, in handling EXIT_REASON_HLT. These lines seemed to be equivalent to `kvm_handle_halt()`. Signed-off-by: Chen Zhang Message-Id: <087f1d9c-109d-41d1-be2c-ce5d840c9...@me.com>

[Qemu-devel] [PULL 20/21] hw/char: Move multi-serial devices into separate file

2019-05-15 Thread Paolo Bonzini
From: Thomas Huth In our downstream distribution of QEMU, we'd like to ship the binary without the multi-serial PCI devices. To make this disablement easier, let's move the devices into a separate file and add a proper Kconfig- switch for these devices. Signed-off-by: Thomas Huth Message-Id:

[Qemu-devel] [PATCH 4/4] iotests: Make 245 faster and more reliable

2019-05-15 Thread Max Reitz
Sometimes, 245 fails for me because some stream job has already finished while the test expects it to still be active. (With -c none, it fails basically every time.) The most reliable way to fix this is to simply set auto_finalize=false so the job will remain in the block graph as long as we need

[Qemu-devel] [PATCH v1 5/5] s390x/tcg: Implement VECTOR STRING RANGE COMPARE

2019-05-15 Thread David Hildenbrand
Crazy stuff. Implement it similar to VECTOR FIND ANY ELEMENT EQUAL. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 ++ target/s390x/insn-data.def | 2 + target/s390x/translate_vx.inc.c | 37 +++ target/s390x/vec_string_helper.c | 108

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread David Hildenbrand
On 15.05.19 22:46, David Hildenbrand wrote: >> +vpmem->vdev = vdev; >> +vdev->priv = vpmem; >> +err = init_vq(vpmem); >> +if (err) { >> +dev_err(>dev, "failed to initialize virtio pmem vq's\n"); >> +goto out_err; >> +} >> + >> +

[Qemu-devel] [PULL 08/21] memory: correct the comment to DIRTY_MEMORY_MIGRATION

2019-05-15 Thread Paolo Bonzini
From: Wei Yang The dirty bit is DIRTY_MEMORY_MIGRATION. Correct the comment. Signed-off-by: Wei Yang Message-Id: <20190426020927.25470-1-richardw.y...@linux.intel.com> Signed-off-by: Paolo Bonzini --- memory.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/memory.c

[Qemu-devel] [PULL 12/21] mips-fulong2e: obey -vga none

2019-05-15 Thread Paolo Bonzini
Do not create an ATI VGA if "-vga none" was passed on the command line. Cc: BALATON Zoltan Signed-off-by: Paolo Bonzini --- hw/mips/mips_fulong2e.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/hw/mips/mips_fulong2e.c b/hw/mips/mips_fulong2e.c index

[Qemu-devel] [PATCH v1 1/5] s390x/tcg: Implement VECTOR FIND ANY ELEMENT EQUAL

2019-05-15 Thread David Hildenbrand
Complicated stuff. Provide two variants, one for the CC and one without the CC. The CC is returned via cpu_env. Signed-off-by: David Hildenbrand --- target/s390x/Makefile.objs | 2 +- target/s390x/helper.h| 8 +++ target/s390x/insn-data.def | 5 ++

[Qemu-devel] [PATCH v1 2/5] s390x/tcg: Implement VECTOR FIND ELEMENT EQUAL

2019-05-15 Thread David Hildenbrand
Implement it similar to VECTOR FIND ANY ELEMENT EQUAL. The zero-check seems to have precedence in case we have "data1 == data2 == 0". The description in the PoP is a little bi confusing. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 target/s390x/insn-data.def

[Qemu-devel] [PULL 16/21] configure: qemu-ga is only needed with softmmu targets

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Remove it from the list of tools if --disable-system and --disable-tools are used as we don't need it for linux-user targets. Suggested-by: Paolo Bonzini [lv: I also disable it with disable-tools, not only with disable-system] Signed-off-by: Laurent Vivier Message-Id:

Re: [Qemu-devel] [PATCH for-4.1] vfio/common: Introduce vfio_set_irq_signaling helper

2019-05-15 Thread Alex Williamson
On Tue, 9 Apr 2019 17:58:31 +0200 Eric Auger wrote: > The code used to assign an interrupt index/subindex to an > eventfd is duplicated many times. Let's introduce an helper that > allows to set/unset the signaling for an ACTION_TRIGGER or > ACTION_UNMASK action. > > Signed-off-by: Eric Auger

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

2019-05-15 Thread Yan Zhao
On Tue, May 14, 2019 at 11:01:42PM +0800, Alex Williamson wrote: > On Tue, 14 May 2019 09:43:44 +0200 > Erik Skultety wrote: > > > On Tue, May 14, 2019 at 03:32:19AM -0400, Yan Zhao wrote: > > > On Tue, May 14, 2019 at 03:20:40PM +0800, Erik Skultety wrote: > > > > On Tue, May 14, 2019 at

Re: [Qemu-devel] [PATCH] spapr/xive: Sanity checks of OV5 during CAS

2019-05-15 Thread David Gibson
On Wed, May 15, 2019 at 07:04:24PM +0200, Greg Kurz wrote: > If a machine is started with ic-mode=xive but the guest only knows > about XICS, eg. an RHEL 7.6 guest, the kernel panics. This is > expected but a bit unfortunate since the crash doesn't provide > much information for the end user to

Re: [Qemu-devel] [RFC v2 PATCH 2/3] spapr: Add NVDIMM device support

2019-05-15 Thread David Gibson
On Wed, May 15, 2019 at 12:30:07PM +0530, Shivaprasad G Bhat wrote: > Hi David, > > Thanks for the comments. Replies inline.. > > On 05/14/2019 07:52 AM, David Gibson wrote: > > On Mon, May 13, 2019 at 04:28:02AM -0500, Shivaprasad G Bhat wrote: > > > Add support for NVDIMM devices for sPAPR.

Re: [Qemu-devel] [PATCH for-4.0.1] q35: Revert to kernel irqchip

2019-05-15 Thread Peter Xu
On Tue, May 14, 2019 at 02:22:03PM -0600, Alex Williamson wrote: > On Tue, 14 May 2019 13:03:31 -0600 > Alex Williamson wrote: > > > Commit b2fc91db8447 ("q35: set split kernel irqchip as default") changed > > the default for the pc-q35-4.0 machine type to use split irqchip, which > > turned out

Re: [Qemu-devel] [Qemu-ppc] [PATCH v8 4/6] target/ppc: Build rtas error log upon an MCE

2019-05-15 Thread David Gibson
On Tue, May 14, 2019 at 10:36:17AM +0530, Aravinda Prasad wrote: > > > On Tuesday 14 May 2019 10:10 AM, David Gibson wrote: > > On Tue, May 14, 2019 at 09:56:41AM +0530, Aravinda Prasad wrote: > >> > >> > >> On Tuesday 14 May 2019 05:38 AM, David Gibson wrote: > >>> On Mon, May 13, 2019 at

Re: [Qemu-devel] [PATCH v6 24/25] target/ppc: Use gen_io_start/end around DARN

2019-05-15 Thread David Gibson
On Fri, May 10, 2019 at 10:30:48AM -0700, Richard Henderson wrote: > Generating a random number counts as I/O, as it cannot be > replayed and produce the same results. > > Cc: David Gibson > Suggested-by: Peter Maydell > Signed-off-by: Richard Henderson Acked-by: David Gibson > --- >

[Qemu-devel] [PATCH v13 10/12] hw/registerfields.h: Add 8bit and 16bit register macros.

2019-05-15 Thread Yoshinori Sato
Some RX peripheral using 8bit and 16bit registers. Added 8bit and 16bit APIs. Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- include/hw/registerfields.h | 32 +++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH v13 09/12] Add rx-softmmu

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Tested-by: Philippe Mathieu-Daudé Using only CONFIG_RX=y: Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Yoshinori Sato --- configure | 8 default-configs/rx-softmmu.mak | 3 +++ include/sysemu/arch_init.h | 1 +

[Qemu-devel] [PATCH v13 00/12] Add RX archtecture support

2019-05-15 Thread Yoshinori Sato
Hello. This patch series is added Renesas RX target emulation. Add "Reviewed-by" for all changes. My git repository is bellow. git://git.pf.osdn.net/gitroot/y/ys/ysato/qemu.git tags/rx-20190514 Testing binaries bellow. u-boot Download - https://osdn.net/users/ysato/pf/qemu/dl/u-boot.bin.gz

[Qemu-devel] [PATCH v13 12/12] MAINTAINERS: Add RX

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- MAINTAINERS | 19 +++ 1 file changed, 19 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index a73a61a546..ef6a02702e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -272,6 +272,13 @@ F: include/hw/riscv/ F:

[Qemu-devel] [PATCH] target/ppc: Set PSSCR_EC on cpu halt to prevent spurious wakeup

2019-05-15 Thread Suraj Jitindar Singh
The processor stop status and control register (PSSCR) is used to control the power saving facilities of the thread. The exit criterion bit (EC) is used to specify whether the thread should be woken by any interrupt (EC == 0) or only an interrupt enabled in the LPCR to wake the thread (EC == 1).

Re: [Qemu-devel] [PATCH v2 11/13] tests/vm: netbsd autoinstall, using serial console

2019-05-15 Thread Kamil Rytarowski
On 10.05.2019 12:46, Gerd Hoffmann wrote: > Instead of fetching the prebuilt image from patchew download the install > iso and prepare the image locally. Install to disk, using the serial > console. Create qemu user, configure ssh login. Install packages > needed for qemu builds. > >

[Qemu-devel] [PATCH v13 11/12] qemu/bitops.h: Add extract8 and extract16

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- include/qemu/bitops.h | 38 ++ 1 file changed, 38 insertions(+) diff --git a/include/qemu/bitops.h b/include/qemu/bitops.h index 3f0926cf40..764f9d1ea0 100644 --- a/include/qemu/bitops.h +++

  1   2   3   >