Re: [PULL 12/30] Makefile: Remove generated files when doing 'distclean'

2019-10-07 Thread Aleksandar Markovic
On Monday, October 7, 2019, Thomas Huth wrote: > - Original Message - > > From: "Paolo Bonzini" > > To: "Peter Maydell" > > Cc: "QEMU Developers" , "Thomas Huth" < > th...@redhat.com> > > Sent: Friday, October 4, 2019 6:48:47 PM > > Subject: Re: [PULL 12/30] Makefile: Remove generated f

Re: [PATCH v5 4/9] target/arm/cpu64: max cpu: Introduce sve properties

2019-10-07 Thread Auger Eric
Hi Drew, On 10/1/19 2:58 PM, Andrew Jones wrote: > Introduce cpu properties to give fine control over SVE vector lengths. > We introduce a property for each valid length up to the current > maximum supported, which is 2048-bits. The properties are named, e.g. > sve128, sve256, sve384, sve512, ...,

Re: [PATCH v5 5/9] target/arm/kvm64: Add kvm_arch_get/put_sve

2019-10-07 Thread Auger Eric
Hi Drew, On 10/1/19 2:58 PM, Andrew Jones wrote: > These are the SVE equivalents to kvm_arch_get/put_fpsimd. Note, the > swabbing is different than it is for fpsmid because the vector format > is a little-endian stream of words. > > Signed-off-by: Andrew Jones Reviewed-by: Eric Auger Eric > --

Re: [PULL 18/19] virtio: add vhost-user-fs base device

2019-10-07 Thread Michael S. Tsirkin
On Mon, Oct 07, 2019 at 07:57:04AM +0200, Igor Mammedov wrote: > On Sat, 5 Oct 2019 18:00:09 -0400 > "Michael S. Tsirkin" wrote: > > > From: "Dr. David Alan Gilbert" > > > > The virtio-fs virtio device provides shared file system access using > > the FUSE protocol carried over virtio. > > The a

[Bug 1846816] Re: Booting error on AIX 6.1 "Illegal Trap Instruction Interrupt in Kernel""

2019-10-07 Thread Thomas Huth
Did this used to work for you in older versions of QEMU, or is it a new bug (i.e. a regression)? AFAIK running AIX in QEMU has never worked so far... ** Tags added: ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.lau

[PATCH] target/mips: add gdb xml files

2019-10-07 Thread Mikhail Abakumov
From: Mikhail Abakumov This patch add xml files with gdb registers for mips. Signed-off-by: Mikhail Abakumov --- configure | 3 ++ gdb-xml/mips-core.xml | 84 + gdb-xml/mips64-core.xml | 84 + tar

Re: [Qemu-devel] [PATCH v2 02/11] qcrypto-luks: extend the create options for upcoming encryption key management

2019-10-07 Thread Markus Armbruster
Quick QAPI schema review only. Maxim Levitsky writes: > Now you can specify which slot to put the encryption key to > Plus add 'active' option which will let user erase the key secret > instead of adding it. > Check that active=true it when creating. > > Signed-off-by: Maxim Levitsky [...] > d

Re: [Qemu-devel] [PATCH v2 07/11] block: add x-blockdev-amend qmp command

2019-10-07 Thread Markus Armbruster
Maxim Levitsky writes: > Signed-off-by: Maxim Levitsky > --- [...] > diff --git a/qapi/block-core.json b/qapi/block-core.json > index e6edd641f1..7900914506 100644 > --- a/qapi/block-core.json > +++ b/qapi/block-core.json > @@ -4650,6 +4650,32 @@ >'data': { 'job-id': 'str', > 'o

[Bug 1623998] Re: pulseaudio Invalid argument error

2019-10-07 Thread Thomas Huth
What output do you get when you run "qemu-system-x86_64 -audio-help" ? Could you provide your full command line, please? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1623998 Title: pulseaudio Inva

Re: [Qemu-devel] [PATCH v2 08/11] block/crypto: implement blockdev-amend

2019-10-07 Thread Markus Armbruster
Maxim Levitsky writes: > Signed-off-by: Maxim Levitsky > Reviewed-by: Daniel P. Berrangé > --- [...] > diff --git a/qapi/block-core.json b/qapi/block-core.json > index 7900914506..4a6db98938 100644 > --- a/qapi/block-core.json > +++ b/qapi/block-core.json > @@ -4211,8 +4211,11 @@ > # Driver sp

Re: [PATCH v2 09/11] block/qcow2: implement blockdev-amend

2019-10-07 Thread Markus Armbruster
Max Reitz writes: > On 13.09.19 00:30, Maxim Levitsky wrote: >> Currently only for changing crypto parameters > > Yep, that elegantly avoids most of the problems we’d have otherwise. :-) > >> Signed-off-by: Maxim Levitsky [...] >> diff --git a/qapi/block-core.json b/qapi/block-core.json >> index

Re: [Qemu-devel] [PATCH v2 00/11] RFC crypto/luks: encryption key managment using amend interface

2019-10-07 Thread Markus Armbruster
Maxim Levitsky writes: > On Fri, 2019-09-20 at 17:14 -0400, John Snow wrote: >> >> On 9/12/19 6:30 PM, Maxim Levitsky wrote: >> > This patch series is continuation of my work to add encryption >> > key managment to luks/qcow2 with luks. >> > >> > This is second version of this patch set. >> > T

Re: [PATCH v4 15/24] audio: add mixing-engine option (documentation)

2019-10-07 Thread Markus Armbruster
"Zoltán Kővágó" writes: > On 2019-10-01 08:23, Markus Armbruster wrote: >> "Zoltán Kővágó" writes: >> >>> On 2019-09-25 11:49, Markus Armbruster wrote: "Zoltán Kővágó" writes: > On 2019-09-23 15:08, Markus Armbruster wrote: >> "Kővágó, Zoltán" writes: >> >>> This will

Re: [PULL 25/30] Fix wrong behavior of cpu_memory_rw_debug() function in SMM

2019-10-07 Thread Laszlo Ersek
Hi Paolo, (+Peter) On 10/02/19 18:51, Paolo Bonzini wrote: > From: Dmitry Poletaev > > There is a problem, that you don't have access to the data using > cpu_memory_rw_debug() function when in SMM. You can't remotely debug SMM mode > program because of that for example. > Likely attrs version

Re: [PATCH v5 4/9] target/arm/cpu64: max cpu: Introduce sve properties

2019-10-07 Thread Alex Bennée
Andrew Jones writes: > Introduce cpu properties to give fine control over SVE vector lengths. > We introduce a property for each valid length up to the current > maximum supported, which is 2048-bits. The properties are named, e.g. > sve128, sve256, sve384, sve512, ..., where the number is the

[PATCH 4/9] ppc/xive: Introduce helpers for the NVT id

2019-10-07 Thread Cédric Le Goater
The NVT space is 19 bits wide, giving a maximum of 512K per chip. When a vPCU is dispatched on a HW thread, the NVT identifier is pushed in the CAM line (QW1W2). This identifier is used in the presenter subengine to fetch a NVT structure which might contain pending interrupts that need a resend. S

[PATCH 7/9] ppc/pnv: Quiesce some XIVE errors

2019-10-07 Thread Cédric Le Goater
When dumping the END and NVT tables, the error logging is too noisy. Signed-off-by: Cédric Le Goater --- hw/intc/pnv_xive.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/hw/intc/pnv_xive.c b/hw/intc/pnv_xive.c index 9f7e0e1662d6..6e0bb6dbe44f 100644 --- a/hw/intc/pnv_x

[PATCH 0/9] ppc/pnv: XIVE cleanup and fixes

2019-10-07 Thread Cédric Le Goater
Hello, Here is a short series adding the cleanups and fixes of the bigger series "ppc/pnv: add XIVE support for KVM guests". There is still some rework to be done on the XivePresenter before it can be resent. These are valuable changes which can come first and which should not impact the work yet

[PATCH 8/9] ppc/xive: Introduce OS CAM line helpers

2019-10-07 Thread Cédric Le Goater
The OS CAM line has a special encoding exploited by the HW. Provide helper routines to hide the details to the TIMA command handlers. This also clarifies the endianness of different variables : 'qw1w2' is big-endian and 'cam' is native. Signed-off-by: Cédric Le Goater --- hw/intc/xive.c | 41 +++

[PATCH 3/9] ppc/xive: Record the IPB in the associated NVT

2019-10-07 Thread Cédric Le Goater
When an interrupt can not be presented to a vCPU, the XIVE presenter updates the Interrupt Pending Buffer of the XIVE NVT if backlog is activated in the END. Signed-off-by: Cédric Le Goater --- include/hw/ppc/xive_regs.h | 1 + hw/intc/xive.c | 11 +-- 2 files changed, 10 in

[PATCH 1/9] ppc/pnv: Improve trigger data definition

2019-10-07 Thread Cédric Le Goater
The trigger data is used for both triggers of a HW source interrupts, PHB, PSI, and triggers for rerouting interrupts between interrupt controllers. When an interrupt is rerouted, the trigger data follows an "END trigger" format. In that case, the remote IC needs EAS containing an END index to per

[PATCH 9/9] ppc/xive: Check V bit in TM_PULL_POOL_CTX

2019-10-07 Thread Cédric Le Goater
A context should be 'valid' when pulled from the thread interrupt context registers. Signed-off-by: Cédric Le Goater --- hw/intc/xive.c | 5 + 1 file changed, 5 insertions(+) diff --git a/hw/intc/xive.c b/hw/intc/xive.c index 143418c232a2..91eb7789f329 100644 --- a/hw/intc/xive.c +++ b/hw/i

[PATCH 2/9] ppc/pnv: Use address_space_stq_be() when triggering an interrupt from PSI

2019-10-07 Thread Cédric Le Goater
Include the XIVE_TRIGGER_PQ bit in the trigger data which is how hardware signals to the IC that the PQ bits of the interrupt source have been checked. Signed-off-by: Cédric Le Goater --- hw/ppc/pnv_psi.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/hw/ppc

[PATCH 6/9] ppc/pnv: Dump the XIVE NVT table

2019-10-07 Thread Cédric Le Goater
This is useful to dump the contexts of the KVM vCPUs : configuration of the base END index of the vCPU and the Interrupt Pending Buffer, which is updated when an interrupt can not be presented. When dumping the NVT table, we skip empty indirect pages which are not necessarily allocated. Signed-of

[PATCH 5/9] ppc/pnv: Remove pnv_xive_vst_size() routine

2019-10-07 Thread Cédric Le Goater
pnv_xive_vst_size() tries to compute the size of a VSD table from the information given by FW. The number of entries of the table are deduced from the result and the MMIO regions of the ESBs and the END ESBs are then resized accordingly with the computed value. This reduces the number of elements t

Re: [PATCH 3/5] hw/sd/sdhci: Add dummy Samsung SDHCI controller

2019-10-07 Thread Krzysztof Kozlowski
On Sat, Oct 05, 2019 at 05:47:46PM +0200, Philippe Mathieu-Daudé wrote: > The Linux kernel access few S3C-specific registers [1] to set some > clock. We don't care about this part for device emulation [2]. Add > a dummy device to properly ignore these accesses, so we can focus > on the important re

Re: [PATCH 4/5] hw/arm/exynos4210: Use the Samsung s3c SDHCI controller

2019-10-07 Thread Krzysztof Kozlowski
On Sat, Oct 05, 2019 at 05:47:47PM +0200, Philippe Mathieu-Daudé wrote: > The Exynos SoC has specific SDHCI registers. Use the s3c SDHCI > model which handle these specific registers. > > This silents the following "SDHC ... not implemented" warnings so > we can focus on the important registers mi

Re: [PATCH] xive: Make some device types not user creatable

2019-10-07 Thread Markus Armbruster
Greg Kurz writes: > Some device types of the XIVE model are exposed to the QEMU command > line: > > $ ppc64-softmmu/qemu-system-ppc64 -device help | grep xive > name "xive-end-source", desc "XIVE END Source" > name "xive-source", desc "XIVE Interrupt Source" > name "xive-tctx", desc "XIVE Interru

[PATCH] migration/postcopy: not necessary to discard all RAM at the beginning

2019-10-07 Thread Wei Yang
ram_discard_range() unmap page for specific range. To be specific, this clears related page table entries so that userfault would be triggered. But this step is not necessary at the very beginning. ram_postcopy_incoming_init() is called when destination gets ADVISE command. ADVISE command is sent

Re: [PATCH 0/5] hw/arm/exynos4210: Add acceptance tests to the SMDKC210 board

2019-10-07 Thread Krzysztof Kozlowski
On Sat, Oct 05, 2019 at 05:47:43PM +0200, Philippe Mathieu-Daudé wrote: > Hi all, > > Yesterday Peter Maydell asked on IRC if I had any working Exynos4 > image. I looked at some old backuped notes and could boot Guenter > initrd with BusyBox. > I'll use this cover letter to share my notes, they mi

Re: [PATCH v5 01/10] hw/virtio: Factorize virtio-mmio headers

2019-10-07 Thread Markus Armbruster
Eric Blake writes: > On 10/3/19 6:26 AM, Sergio Lopez wrote: >> >> Philippe Mathieu-Daudé writes: >> >>> On 10/2/19 1:30 PM, Sergio Lopez wrote: Put QOM and main struct definition in a separate header file, so it can be accessed from other components. Signed-off-by: Sergio Lo

Re: [edk2-devel] [Qemu-devel] [PATCH 1/2] q35: implement 128K SMRAM at default SMBASE address

2019-10-07 Thread Laszlo Ersek
On 10/04/19 13:31, Igor Mammedov wrote: > On Tue, 1 Oct 2019 20:03:20 +0200 > "Laszlo Ersek" wrote: >> (1) What values to use. > SeaBIOS writes 0x00 into command port, but it seems that's taken by > EFI_SMM_COMMUNICATION_PROTOCOL. So we can use the next unused value > (lets say 0x4). We probably

Re: [PATCH 1/1] MAINTAINERS: Add Vladimir as a reviewer for bitmaps

2019-10-07 Thread Vladimir Sementsov-Ogievskiy
05.10.2019 22:44, John Snow wrote: > I already try to make sure all bitmaps patches have been reviewed by both > Red Hat and Virtuozzo anyway, so this formalizes the arrangement. > > Fam meanwhile is no longer as active, so I am removing him as a co-maintainer > simply to reflect the current pract

Re: [PATCH 1/1] IDE: deprecate ide-drive

2019-10-07 Thread Markus Armbruster
John Snow writes: > It's an old compatibility shim that just delegates to ide-cd or ide-hd. > I'd like to refactor these some day, and getting rid of the super-object > will make that easier. Device "scsi-disk" is similar. However, it's still used by the scsi_bus_legacy_add_drive() magic. Not

Re: [PATCH] iotests: Do not run the iotests during "make check" anymore

2019-10-07 Thread Max Reitz
On 02.10.19 16:21, Thomas Huth wrote: > Running the iotests during "make check" is causing more headaches than > benefits for the block layer maintainers, so let's disable the iotests > during "make check" again. > > Signed-off-by: Thomas Huth > --- > tests/Makefile.include | 2 +- > tests/qem

Re: [PATCH 00/11] hw: Convert various reset() handler to DeviceReset

2019-10-07 Thread Philippe Mathieu-Daudé
Hi Michael, On 10/5/19 11:46 PM, Michael S. Tsirkin wrote: On Thu, Sep 26, 2019 at 05:17:22PM +0200, Philippe Mathieu-Daudé wrote: Hi. Following the thread discussion between Peter/Markus/Damien about reset handlers: https://www.mail-archive.com/qemu-devel@nongnu.org/msg617103.html I started t

Re: [PATCH 4/4] migration: report SaveStateEntry id and name on failure

2019-10-07 Thread Philippe Mathieu-Daudé
On 10/6/19 12:05 AM, Wei Yang wrote: This provides helpful information on which entry failed. Signed-off-by: Wei Yang --- migration/savevm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/migration/savevm.c b/migration/savevm.c index 9f0122583d..feb757de79 100644 --- a/migration/savev

Re: [PATCH 2/4] migration/postcopy: fix typo in mark_postcopy_blocktime_begin's comment

2019-10-07 Thread Philippe Mathieu-Daudé
On 10/6/19 12:05 AM, Wei Yang wrote: Signed-off-by: Wei Yang --- migration/postcopy-ram.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c index d2bdd21ae3..a394c7c3a6 100644 --- a/migration/postcopy-ram.c +++ b/mi

Re: [PULL v3 00/29] Misc patches for 2019-10-02

2019-10-07 Thread Peter Maydell
On Fri, 4 Oct 2019 at 18:02, Paolo Bonzini wrote: > > The following changes since commit 7f21573c822805a8e6be379d9bcf3ad9effef3dc: > > Merge remote-tracking branch > 'remotes/huth-gitlab/tags/pull-request-2019-10-01' into staging (2019-10-01 > 13:13:38 +0100) > > are available in the git repos

Re: [PATCH] target/i386: log guest name and memory error type AO, AR for MCEs

2019-10-07 Thread Philippe Mathieu-Daudé
Hi Mario, On 10/5/19 1:53 AM, Mario Smarduch wrote: In a large VPC environment we want to log memory error occurrences and log them with guest name and type - there are few use cases - if VM crashes on AR mce inform the user about the reason and resolve the case - if VM hangs notify the use

Re: [PATCH] virtio-net: prevent offloads reset on migration

2019-10-07 Thread Mikhail Sennikovsky
Guys, any update on this? Note that I have also sent a second version of the patch here, which has the coding style issue fixed. Thanks, Mikhail Am Mi., 2. Okt. 2019 um 11:55 Uhr schrieb Dr. David Alan Gilbert : > > Copying in Stefan, Jason and Michael who know the virtio details > > Dave > > * M

[PATCH] migration: Don't try and recover return path in non-postcopy

2019-10-07 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" In normal precopy we can't do reconnection recovery - but we also don't need to, since you can just rerun migration. At the moment if the 'return-path' capability is on, we use the return path in precopy to give a postiive 'OK' to the end of migration; however if mi

Re: [PATCH] migration: Don't try and recover return path in non-postcopy

2019-10-07 Thread Greg Kurz
On Mon, 7 Oct 2019 11:35:07 +0100 "Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > In normal precopy we can't do reconnection recovery - but we also > don't need to, since you can just rerun migration. > At the moment if the 'return-path' capability is on, we use > th

Re: [Bug 1623998] Re: pulseaudio Invalid argument error

2019-10-07 Thread John Arbuckle
Here is the output. I assumed you meant qemu-system-ppc. $ ./ppc-softmmu/qemu-system-ppc -audio-help Environment variable based configuration deprecated. Please use the new -audiodev option. Equivalent -audiodev to your current environment variables: (Since you didn't specify QEMU_AUDIO_DRV, I'll

Re: [PATCH v9 3/3] iotests: test nbd reconnect

2019-10-07 Thread Vladimir Sementsov-Ogievskiy
04.10.2019 21:05, Eric Blake wrote: > On 9/24/19 3:31 AM, Vladimir Sementsov-Ogievskiy wrote: > +def qemu_nbd_popen(*args): +    '''Run qemu-nbd in daemon mode and return the parent's exit code''' +    return subprocess.Popen(qemu_nbd_args + ['--persistent'] + list(args)) + >>>

Re: [PATCH 1/1] nbd: add empty .bdrv_reopen_prepare

2019-10-07 Thread Max Reitz
On 30.09.19 23:38, Maxim Levitsky wrote: > Fixes commit job / qemu-img commit, when > commiting qcow2 file which is based on nbd export. > > > Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1718727 > > Signed-off-by: Maxim Levitsky > --- > block/nbd.c | 15 +++ > 1 file chan

Re: [PATCH] netmap: support git-submodule build otption

2019-10-07 Thread Markus Armbruster
Peter Maydell writes: D> On Fri, 4 Oct 2019 at 14:03, Giuseppe Lettieri wrote: >> >> From: Giuseppe Lettieri >> >> With this patch, netmap support can be enabled with >> the following options to the configure script: >> >> --enable-netmap[=system] >> >> Use the host system netmap ins

Re: [Qemu-devel] qemu_futex_wait() lockups in ARM64: 2 possible issues

2019-10-07 Thread Paolo Bonzini
On 02/10/19 11:23, Jan Glauber wrote: > I've looked into this on ThunderX2. The arm64 code generated for the > atomic_[add|sub] accesses of ctx->notify_me doesn't contain any > memory barriers. It is just plain ldaxr/stlxr. > > From my understanding this is not sufficient for SMP sync. > > If I r

Re: [PATCH] tests: fix I/O test for hosts defaulting to LUKSv2

2019-10-07 Thread Max Reitz
On 27.09.19 12:11, Daniel P. Berrangé wrote: > Some distros are now defaulting to LUKS version 2 which QEMU cannot > process. For our I/O test that validates interoperability between the > kernel/cryptsetup and QEMU, we need to explicitly ask for version 1 > of the LUKS format. > > Signed-off-by:

[Bug 1846816] Re: Booting error on AIX 6.1 "Illegal Trap Instruction Interrupt in Kernel""

2019-10-07 Thread Leonardo
I only tried this with the last version of QEMU using an AIX image generated from a running AIX server using mksysb. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1846816 Title: Booting error on AI

Re: [PATCH 0/2] enhance iotest 223 coverage

2019-10-07 Thread Max Reitz
On 24.09.19 21:51, Eric Blake wrote: > On 9/24/19 2:26 PM, Vladimir Sementsov-Ogievskiy wrote: >> 24.09.2019 17:35, Eric Blake wrote: >>> Commit 506902c6 dropped non-iothread coverage in order to test iothread, >>> better is to run things twice. In doing this, I found it easier to >>> edit the tes

Re: [PATCH v2 3/6] qemu-nbd: add compression flag support

2019-10-07 Thread Vladimir Sementsov-Ogievskiy
04.10.2019 13:19, Roman Kagan wrote: > On Wed, Oct 02, 2019 at 05:22:43PM +0300, Andrey Shinkevich wrote: >> Added possibility to write compressed data by using the >> blk_write_compressed. This action has the limitations of the format >> driver. For example we can't write compressed data over othe

Re: [PATCH] netmap: support git-submodule build otption

2019-10-07 Thread Peter Maydell
On Mon, 7 Oct 2019 at 11:50, Markus Armbruster wrote: > Peter Maydell writes: > > Basically new submodules are a pain so we seek to minimize > > the use of them. > > I suggested making it a submodule upthread[*]. Let me try to distill > the conversation into a rationale. Giuseppe, please correc

Re: [PATCH 1/2] tests: Make iotest 223 easier to edit

2019-10-07 Thread Max Reitz
On 24.09.19 16:35, Eric Blake wrote: > Log the QMP input to qemu, not just the QMP output. Why not just add this functionality to _send_qemu_cmd directly? (Like silent already does for replies, although it’s inverted.) (Although I’m not quite sold on the indentation for commands, because (1) we

[PATCH 00/16] io_uring: add Linux io_uring AIO engine

2019-10-07 Thread Stefan Hajnoczi
v10: * Dropped kernel submission queue polling, it requires root and has additional limitations. It should be benchmarked and considered for inclusion later, maybe even together with kernel side changes. * Add io_uring_register_files() return value to trace_luring_fd_register() * Fix inde

[PATCH 02/16] qapi/block-core: add option for io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Only enumerates option for devices that support it. Since QAPI schema supports io_uring, which is the actual name of the Linux API, it is preferred over io-uring. Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- qapi/block-core.json | 4 +++- 1 file changed,

[PATCH 01/16] configure: permit use of io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Reviewed-by: Maxim Levitsky Signed-off-by: Stefan Hajnoczi --- configure | 27 +++ 1 file changed, 27 insertions(+) diff --git a/configure b/configure index 8f8446f52b..3ac597c547 100755 --- a/configure +++ b/configure

[PATCH 03/16] block/block: add BDRV flag for io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Reviewed-by: Maxim Levitsky Signed-off-by: Stefan Hajnoczi --- include/block/block.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/block/block.h b/include/block/block.h index 37c9de7446..88a9673734 100644 --- a/include/block/block

[PATCH 04/16] block/io_uring: implements interfaces for io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Aborts when sqe fails to be set as sqes cannot be returned to the ring. Adds slow path for short reads for older kernels Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- v10: * Update MAINTAINERS file [Julia] * Rename MAX_EVENTS to MAX_ENTRIES [Julia] * De

[PATCH 07/16] blockdev: adds bdrv_parse_aio to use io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- include/block/block.h | 1 + block.c | 22 ++ blockdev.c| 12 3 files changed, 27 insertions(+), 8 deletions(-) diff --git a/include/block/block.h b/

[PATCH 05/16] stubs: add stubs for io_uring interface

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- MAINTAINERS | 1 + stubs/Makefile.objs | 1 + stubs/io_uring.c| 32 3 files changed, 34 insertions(+) create mode 100644 stubs/io_uring.c diff --git a/MAINTAINER

Re: [PATCH 2/2] tests: More iotest 223 improvements

2019-10-07 Thread Max Reitz
On 24.09.19 16:35, Eric Blake wrote: > Run the test twice, once without iothreads, and again with, for more > coverage of both setups. > > Suggested-by: Nir Soffer > Signed-off-by: Eric Blake > --- > tests/qemu-iotests/223 | 66 + > tests/qemu-iotests/223.out | 98 ++

[PATCH 12/16] qemu-img: adds option to use aio engine for benchmarking

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- v10: * Add missing space to qemu-img command-line documentation --- qemu-img.c | 11 ++- qemu-img-cmds.hx | 4 ++-- qemu-img.texi| 5 - 3 files changed, 16 insertions(+), 4 deletions(-)

[PATCH 06/16] util/async: add aio interfaces for io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- util/async.c | 36 1 file changed, 36 insertions(+) diff --git a/util/async.c b/util/async.c index 4e4c7af51e..f8502a9310 100644 --- a/util/async.c +++ b/util/async.c @@ -2

[PATCH 09/16] block: add trace events for io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- block/io_uring.c | 21 ++--- block/trace-events | 12 2 files changed, 30 insertions(+), 3 deletions(-) diff --git a/block/io_uring.c b/block/io_uring.c index 307c4c5823..a5c0d16

[PATCH 08/16] block/file-posix.c: extend to use io_uring

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Reviewed-by: Maxim Levitsky Signed-off-by: Stefan Hajnoczi --- block/file-posix.c | 99 -- 1 file changed, 79 insertions(+), 20 deletions(-) diff --git a/block/file-posix.c b/block/file-posix.c index

[PATCH 13/16] qemu-nbd: adds option for aio engines

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Acked-by: Eric Blake Signed-off-by: Stefan Hajnoczi --- qemu-nbd.c| 12 qemu-nbd.texi | 4 ++-- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/qemu-nbd.c b/qemu-nbd.c index 9032b6de2a..afa969c785 100644 --- a/

[PATCH 11/16] qemu-io: adds option to use aio engine

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- qemu-io.c | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/qemu-io.c b/qemu-io.c index f64eca6940..0abb4af134 100644 --- a/qemu-io.c +++ b/qemu-io.c @@ -130,7 +130,

[PATCH 15/16] tests/qemu-iotests: use AIOMODE with various tests

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/028 | 3 ++- tests/qemu-iotests/058 | 2 +- tests/qemu-iotests/089 | 4 ++-- tests/qemu-iotests/091 | 7 --- tests/qemu-iotests/109 | 3 ++- tests/qemu-iotests/147 | 5 +++-- tests/

[PATCH 10/16] block/io_uring: adds userspace completion polling

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- block/io_uring.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/block/io_uring.c b/block/io_uring.c index a5c0d16220..56892fd1ab 100644 --- a/block/io_uring.c +++ b/block/

[PATCH 14/16] tests/qemu-iotests: enable testing with aio options

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta Signed-off-by: Aarushi Mehta Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/check | 15 ++- tests/qemu-iotests/common.rc | 14 ++ tests/qemu-iotests/iotests.py | 12 ++-- 3 files changed, 38 insertions(+), 3 deletions(-) diff --

[PATCH 16/16] block/io_uring: adds fd registration

2019-10-07 Thread Stefan Hajnoczi
From: Aarushi Mehta File descriptor registration eliminates the need to get file descriptors for each request on the host kernel side. The host kernel keeps references to a set of files that were registered by the application. These files can be quickly accessed by index instead of a traditional

Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms

2019-10-07 Thread Thomas Huth
On 04/10/2019 14.44, Max Reitz wrote: > On 04.10.19 12:19, Kevin Wolf wrote: >> Am 02.10.2019 um 19:47 hat Max Reitz geschrieben: >>> On 02.10.19 18:44, Kevin Wolf wrote: Am 02.10.2019 um 13:57 hat Max Reitz geschrieben: > It usually worked fine for me because it’s rather rare that non-blo

[Bug 1846816] Re: Booting error on AIX 6.1 "Illegal Trap Instruction Interrupt in Kernel""

2019-10-07 Thread Greg Kurz
There are some limitations but an AIX guest can run in QEMU: https://www.ibm.com/support/knowledgecenter/ssw_aix_72/aixnutanix/nutanix_concepts.html The command line in the bug description uses an old syntax: # qemu-system-ppc64 -cpu POWER8,compat=power7 -machine pseries ... This should be: #

Re: [PULL v2 0/8] Ide patches

2019-10-07 Thread Peter Maydell
On Thu, 3 Oct 2019 at 20:33, John Snow wrote: > > The following changes since commit 7f21573c822805a8e6be379d9bcf3ad9effef3dc: > > Merge remote-tracking branch > 'remotes/huth-gitlab/tags/pull-request-2019-10-01' into staging (2019-10-01 > 13:13:38 +0100) > > are available in the Git repositor

Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms

2019-10-07 Thread Peter Maydell
On Mon, 7 Oct 2019 at 13:32, Thomas Huth wrote: > On 04/10/2019 14.44, Max Reitz wrote: > > Whenever make check fails, it’s urgent. Without iotests running in make > > check, we had some time to sort the situation out. That’s no longer the > > case. > > > > That means we need to take care of eve

Re: [PATCH] netmap: support git-submodule build otption

2019-10-07 Thread Markus Armbruster
Peter Maydell writes: > On Mon, 7 Oct 2019 at 11:50, Markus Armbruster wrote: >> Peter Maydell writes: >> > Basically new submodules are a pain so we seek to minimize >> > the use of them. >> >> I suggested making it a submodule upthread[*]. Let me try to distill >> the conversation into a rat

Re: [PATCH] netmap: support git-submodule build otption

2019-10-07 Thread Peter Maydell
On Mon, 7 Oct 2019 at 13:36, Markus Armbruster wrote: > If CI of QEMU code isn't useful, then I suspect the QEMU code isn't > useful, period. Giuseppe assures us the netmap QEMU code *is* useful. > It followe we better make sure our CI covers it. It would be an interesting idea to have a require

Re: [PATCH v5 4/9] target/arm/cpu64: max cpu: Introduce sve properties

2019-10-07 Thread Andrew Jones
On Mon, Oct 07, 2019 at 09:35:35AM +0100, Alex Bennée wrote: > > Andrew Jones writes: > > > Introduce cpu properties to give fine control over SVE vector lengths. > > We introduce a property for each valid length up to the current > > maximum supported, which is 2048-bits. The properties are nam

Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms

2019-10-07 Thread Max Reitz
On 07.10.19 14:16, Thomas Huth wrote: > On 04/10/2019 14.44, Max Reitz wrote: >> On 04.10.19 12:19, Kevin Wolf wrote: >>> Am 02.10.2019 um 19:47 hat Max Reitz geschrieben: On 02.10.19 18:44, Kevin Wolf wrote: > Am 02.10.2019 um 13:57 hat Max Reitz geschrieben: >> It usually worked fine

[Bug 1846816] Re: Booting error on AIX 6.1 "Illegal Trap Instruction Interrupt in Kernel""

2019-10-07 Thread Philippe Mathieu-Daudé
If you want to debug this, it'd be helpful to know few instructions/registers before the TRAP: Illegal Trap Instruction Interrupt in Kernel 04151A74 tweqi r0,0 r0=0 KDB(0)> A quicker way might be running QEMU with '-d unimp' to display missing devices/SPAPR hcalls. -- You received this bu

Re: [PATCH] target/riscv: PMP violation due to wrong size parameter

2019-10-07 Thread Richard Henderson
On 10/6/19 10:28 PM, Dayeol Lee wrote: > riscv_cpu_tlb_fill() uses the `size` parameter to check PMP violation > using pmp_hart_has_privs(). > However, the size passed from tlb_fill(), which is called by > get_page_addr_code(), is always a hard-coded value 0. > This causes a false PMP violation if

Re: [PATCH] iotests: Do not run the iotests during "make check" anymore

2019-10-07 Thread Max Reitz
On 03.10.19 01:51, John Snow wrote: > > > On 10/2/19 11:50 AM, Max Reitz wrote: >> On 02.10.19 17:10, Thomas Huth wrote: >>> On 02/10/2019 17.03, Daniel P. Berrangé wrote: On Wed, Oct 02, 2019 at 04:21:46PM +0200, Thomas Huth wrote: > Running the iotests during "make check" is causing mo

[Bug 1623998] Re: pulseaudio Invalid argument error

2019-10-07 Thread Thomas Huth
Looks like your current QEMU now only contains the audio backend for OSS, but not for pulseaudio anymore. Please make sure that the right pulse-audio development package (e.g. "pulseaudio-libs-devel") is installed before running the "configure" script of QEMU. -- You received this bug notificatio

Re: [PATCH 0/4] iotests: trivial cleanups

2019-10-07 Thread Max Reitz
On 27.09.19 16:17, Cleber Rosa wrote: > The most trivial set of cleanups to iotests common libraries and the > 044 test. > > Cleber Rosa (4): > qemu-iotests: remove bash shebang from library files > qemu-iotests: remove forceful execution success from library files > qemu-iotests: 044: pass

Re: [PATCH v5 1/5] iotests: remove 'linux' from default supported platforms

2019-10-07 Thread Thomas Huth
On 07/10/2019 14.52, Max Reitz wrote: > On 07.10.19 14:16, Thomas Huth wrote: >> On 04/10/2019 14.44, Max Reitz wrote: >>> On 04.10.19 12:19, Kevin Wolf wrote: Am 02.10.2019 um 19:47 hat Max Reitz geschrieben: > On 02.10.19 18:44, Kevin Wolf wrote: >> Am 02.10.2019 um 13:57 hat Max Rei

Re: [RFC 1/4] hw/timer/arm_timer: Add trace events

2019-10-07 Thread Richard Henderson
On 10/4/19 4:48 AM, Peter Maydell wrote: > Add some basic trace events to the arm_timer device. > > Signed-off-by: Peter Maydell > --- > hw/timer/arm_timer.c | 27 +-- > hw/timer/trace-events | 7 +++ > 2 files changed, 28 insertions(+), 6 deletions(-) Reviewed-by:

Re: [RFC 2/4] ptimer: Rename ptimer_init() to ptimer_init_with_bh()

2019-10-07 Thread Richard Henderson
On 10/4/19 4:48 AM, Peter Maydell wrote: > Currently the ptimer design uses a QEMU bottom-half as its > mechanism for calling back into the device model using the > ptimer when the timer has expired. Unfortunately this design > is fatally flawed, because it means that there is a lag > between the p

Re: [RFC 3/4] ptimer: Provide new transaction-based API

2019-10-07 Thread Richard Henderson
On 10/4/19 4:48 AM, Peter Maydell wrote: > Provide the new transaction-based API. If a ptimer is created > using ptimer_init() rather than ptimer_init_with_bh(), then > instead of providing a QEMUBH, it provides a pointer to the > callback function directly, and has opted into the transaction > API

Re: [PATCH 1/6] block/block-copy: allocate buffer in block_copy_with_bounce_buffer

2019-10-07 Thread Max Reitz
On 03.10.19 19:15, Vladimir Sementsov-Ogievskiy wrote: > Move bounce_buffer allocation block_copy_with_bounce_buffer. This > commit simplifies further work on implementing copying by larger chunks > (of different size) and further asynchronous handling of block_copy > iterations (with help of block

Re: [RFC 4/4] hw/timer/arm_timer.c: Switch to transaction-based ptimer API

2019-10-07 Thread Richard Henderson
On 10/4/19 4:48 AM, Peter Maydell wrote: > Switch the arm_timer.c code away from bottom-half based ptimers > to the new transaction-based ptimer API. This just requires > adding begin/commit calls around the various arms of > arm_timer_write() that modify the ptimer state, and using the > new ptime

[PULL 0/5] edk2-next patches for 2019-10-07

2019-10-07 Thread Philippe Mathieu-Daudé
The following changes since commit 9e5319ca52a5b9e84d55ad9c36e2c0b317a122bb: Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2019-10-04 18:32:34 +0100) are available in the Git repository at: https://gitlab.com/philmd/qemu.git tags/edk2-next-2019100

[PULL 1/5] make-release: pull in edk2 submodules so we can build it from tarballs

2019-10-07 Thread Philippe Mathieu-Daudé
From: Michael Roth The `make efi` target added by 536d2173 is built from the roms/edk2 submodule, which in turn relies on additional submodules nested under roms/edk2. The make-release script currently only pulls in top-level submodules, so these nested submodules are missing in the resulting ta

Re: [Qemu-devel] [PATCH v2 01/15] target/arm/arm-semi: Capture errno in softmmu version of set_swi_errno()

2019-10-07 Thread Richard Henderson
On 9/16/19 7:15 AM, Peter Maydell wrote: > The set_swi_errno() function is called to capture the errno > from a host system call, so that we can return -1 from the > semihosting function and later allow the guest to get a more > specific error code with the SYS_ERRNO function. It comes in > two ver

[PULL 3/5] roms: Add a 'make help' target alias

2019-10-07 Thread Philippe Mathieu-Daudé
Various C projects provide a 'make help' target. Our root directory does so. The roms/ directory lacks a such rule, but already displays a help output when the default target is called. Add a 'help' target aliased to the default one, to avoid: $ make -C roms help make: *** No rule to make targ

[PULL 2/5] roms/Makefile.edk2: don't pull in submodules when building from tarball

2019-10-07 Thread Philippe Mathieu-Daudé
From: Michael Roth Currently the `make efi` target pulls submodules nested under the roms/edk2 submodule as dependencies. However, when we attempt to build from a tarball this fails since we are no longer in a git tree. A preceding patch will pre-populate these submodules in the tarball, so assu

[PULL 4/5] edk2 build scripts: honor external BaseTools flags with uefi-test-tools

2019-10-07 Thread Philippe Mathieu-Daudé
From: Laszlo Ersek Unify the recipe for "build-edk2-tools" in "tests/uefi-test-tools/Makefile" with the recipe for "edk2-basetools" in "roms/Makefile". Cc: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Reviewed-by: John Snow Signed-off-by: Laszl

Re: [Qemu-devel] [PATCH v2 02/15] target/arm/arm-semi: Always set some kind of errno for failed calls

2019-10-07 Thread Richard Henderson
On 9/16/19 7:15 AM, Peter Maydell wrote: > If we fail a semihosting call we should always set the > semihosting errno to something; we were failing to do > this for some of the "check inputs for sanity" cases. > > Signed-off-by: Peter Maydell > Reviewed-by: Alex Bennée > --- > target/arm/arm-se

[PULL 5/5] edk2 build scripts: work around TianoCore#1607 without forcing Python 2

2019-10-07 Thread Philippe Mathieu-Daudé
From: Laszlo Ersek It turns out that forcing python2 for running the edk2 "build" utility is neither necessary nor sufficient. Forcing python2 is not sufficient for two reasons: - QEMU is moving away from python2, with python2 nearing EOL, - according to my most recent testing, the lacking dep

[Bug 1846816] Re: Booting error on AIX 6.1 "Illegal Trap Instruction Interrupt in Kernel""

2019-10-07 Thread Leonardo
# qemu-system-ppc64 -cpu POWER8 -machine pseries,max-cpu-compat=power7 -m 8G -d unimp -serial stdio \ -prom-env boot-command='boot cdrom: -s verbose'> -drive file=/qemu/AIX61.img,if=none,id=drive-virtio-disk0 \ > -device virtio-scsi-pci,id=scsi -device scsi-hd,drive=drive-virtio-disk0 \ > -cdrom

  1   2   3   >