Re: [Qemu-devel] [RFC PATCH 5/5] vfio/quirks: Enable ioeventfd quirks to be handled by vfio directly

On Fri, Feb 09, 2018 at 03:09:33PM -0700, Alex Williamson wrote: > On Fri, 9 Feb 2018 15:11:45 +0800 > Peter Xu wrote: > > > On Tue, Feb 06, 2018 at 05:26:46PM -0700, Alex Williamson wrote: > > > With vfio ioeventfd support, we can program vfio-pci to perform a > > > specified

Re: [Qemu-devel] [PATCHv2 09/12] misc: introduce new mos6522 VIA device and enable it for ppc builds

On Fri, Feb 09, 2018 at 06:51:39PM +, Mark Cave-Ayland wrote: > The MOS6522 VIA forms the bridge part of several Mac devices, including the > Mac via-cuda and via-pmu devices. Introduce a standard mos6522 device that > can be shared amongst multiple implementations. > > This is effectively

Re: [Qemu-devel] [PATCHv2 08/12] cuda: factor out timebase-derived counter value and load time

On Fri, Feb 09, 2018 at 06:51:38PM +, Mark Cave-Ayland wrote: > Commit b981289c49 "PPC: Cuda: Use cuda timer to expose tbfreq to guest" > altered > the timer calculations from those based upon the hardware CUDA clock frequency > to those based upon the CPU timebase frequency. > > In fact we

Re: [Qemu-devel] [PATCHv2 07/12] cuda: set timer 1 frequency property to CUDA_TIMER_FREQ

On Fri, Feb 09, 2018 at 06:51:37PM +, Mark Cave-Ayland wrote: > Now that we have successfully decoupled the timebase frequency and the > hardware > timer frequency, set the timer 1 frequency property to CUDA_TIMER_FREQ and > alter > get_next_irq_time() to use it rather than the hard-coded

Re: [Qemu-devel] [PATCHv2 03/12] cuda: don't call cuda_update() when writing to ACR register

On Fri, Feb 09, 2018 at 06:51:33PM +, Mark Cave-Ayland wrote: > The wire protocol for reading data to/from the VIA is triggered by changing > inputs on port B rather than changing the timer configuration via the ACR. > > Signed-off-by: Mark Cave-Ayland

Re: [Qemu-devel] [PATCHv2 06/12] cuda: minor cosmetic tidy-ups to get_next_irq_time()

On Fri, Feb 09, 2018 at 06:51:36PM +, Mark Cave-Ayland wrote: > Signed-off-by: Mark Cave-Ayland > Reviewed-by: Philippe Mathieu-Daudé Applied, thanks. > --- > hw/misc/macio/cuda.c | 23 --- > 1 file changed, 12

Re: [Qemu-devel] [PATCHv2 04/12] cuda: introduce CUDAState parameter to get_counter()

On Fri, Feb 09, 2018 at 06:51:34PM +, Mark Cave-Ayland wrote: > This will be required shortly and also happens to match nicely with the > corresponding signature for set_counter(). > > Signed-off-by: Mark Cave-Ayland > Reviewed-by: Philippe Mathieu-Daudé

[Qemu-devel] [PATCH 3/5] target/arm: Suppress TB end for FPCR/FPSR

Nothing in either register affects the TB. Signed-off-by: Richard Henderson --- target/arm/helper.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/arm/helper.c b/target/arm/helper.c index d41fb8371f..e0184c7162 100644 ---

[Qemu-devel] [PATCH 4/5] target/arm: Enforce access to ZCR_EL at translation

This also makes sure that we get the correct ordering of SVE vs FP exceptions. Signed-off-by: Richard Henderson --- target/arm/cpu.h | 3 ++- target/arm/internals.h | 6 ++ target/arm/helper.c| 22 --

Re: [Qemu-devel] [PATCHv2 05/12] cuda: rename frequency property to tb_frequency

On Fri, Feb 09, 2018 at 06:51:35PM +, Mark Cave-Ayland wrote: > This allows us to more easily differentiate between the timebase frequency > used > to calibrate the MacOS timers and the actual frequency of the hardware clock > as > indicated by CUDA_TIMER_FREQ. > > Signed-off-by: Mark

Re: [Qemu-devel] [V9fs-developer] [RFC] we should solve create-unlink-getattr idiom

On 2018/2/9 19:33, Greg Kurz wrote: > On Fri, 9 Feb 2018 15:10:46 +0800 > jiangyiwen wrote: > >> Hi Eric and Greg, >> >> I encountered the similar problem with create-unlink-getattr idiom. >> I use the testcase that create-unlink-setattr idiom, and I see the >> bug is

Re: [Qemu-devel] [PATCH 1/3] qapi: Pass '-u' when doing non-silent diff

On Sat, Feb 10, 2018 at 9:40 PM, Eric Blake wrote: > Ed-script diffs are awful compared to context diffs. Fix another > 'diff -q' while in the area (if the files are different, being > noisy makes it easier to diagnose why). > > Fixes: 46ec4fce > Signed-off-by: Eric Blake

[Qemu-devel] [PATCH 1/5] target/arm: Remove ARM_CP_64BIT from ZCR_EL registers

Because they are ARM_CP_STATE_AA64, ARM_CP_64BIT is implied. Signed-off-by: Richard Henderson --- target/arm/helper.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/target/arm/helper.c b/target/arm/helper.c index 180ab75458..4b102ec356

[Qemu-devel] [PATCH 2/5] target/arm: Enforce FP access to FPCR/FPSR

Signed-off-by: Richard Henderson --- target/arm/cpu.h | 35 ++- target/arm/helper.c| 6 -- target/arm/translate-a64.c | 3 +++ 3 files changed, 25 insertions(+), 19 deletions(-) diff --git a/target/arm/cpu.h

[Qemu-devel] [PATCH 0/5] target/arm: More SVE prep work

First, we had noted that ARM_CP_64BIT needed to be removed from the ZCR_EL registers, but the patch set was applied without actually fixing that. Second, there's an existing bug by which the FPCR/FPSR registers are not properly trapped when FP is disabled. Fix that with a translation-time check.

Re: [Qemu-devel] [RFC PATCH] vfio/pci: Add ioeventfd support

On Fri, Feb 09, 2018 at 02:45:41PM -0700, Alex Williamson wrote: > On Fri, 9 Feb 2018 15:05:11 +0800 > Peter Xu wrote: > > > On Tue, Feb 06, 2018 at 05:08:14PM -0700, Alex Williamson wrote: > > > > [...] > > > > > +long vfio_pci_ioeventfd(struct vfio_pci_device *vdev, loff_t

[Qemu-devel] [PATCH 2/3] qapi: Rename QMP and QGA schema files

Having two files in the tree both named qapi-schema.json just adds confusion. Rename these files, and relocate them into the common qapi/ subdirectory. Update all build rules that refer to the file names, and adjust other documentation and comment references to either track the new name or be

Re: [Qemu-devel] [PATCHv2 05/12] cuda: rename frequency property to tb_frequency

On Sun, Feb 11, 2018 at 09:32:14AM +1100, David Gibson wrote: > On Fri, Feb 09, 2018 at 06:51:35PM +, Mark Cave-Ayland wrote: > > This allows us to more easily differentiate between the timebase frequency > > used > > to calibrate the MacOS timers and the actual frequency of the hardware > >

[Qemu-devel] [PATCH 5/5] target/arm: Handle SVE registers when using clear_vec_high

When storing to an AdvSIMD FP register, all of the high bits of the SVE register are zeroed. Therefore, call it more often with is_q as a parameter. Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 162 +

[Qemu-devel] [PATCH 0/3] QAPI file renames

This has been mentioned on list before as a possible improvement, so I went ahead and did some renames to figure out how it would look. Patch 1 is something I hit on the way while debugging patch 2 and 3; patch 2 is rather non-controversial, and patch 3 may or may not be worth doing (but if we do

[Qemu-devel] [PATCH 1/3] qapi: Pass '-u' when doing non-silent diff

Ed-script diffs are awful compared to context diffs. Fix another 'diff -q' while in the area (if the files are different, being noisy makes it easier to diagnose why). Fixes: 46ec4fce Signed-off-by: Eric Blake --- tests/Makefile.include | 8 1 file changed, 4

Re: [Qemu-devel] [PATCH v7 01/23] docs: update QMP documents for OOB commands

On Fri, Feb 09, 2018 at 08:10:53AM -0600, Eric Blake wrote: > On 01/23/2018 11:39 PM, Peter Xu wrote: > > Update both the developer and spec for the new QMP OOB (Out-Of-Band) > > command. > > > > Signed-off-by: Peter Xu > > --- > > docs/devel/qapi-code-gen.txt | 68 > >

[Qemu-devel] [PULL 11/17] oslib-posix: check for posix_memalign in configure script

From: Andreas Gustafsson Check for the presence of posix_memalign() in the configure script, not using "defined(_POSIX_C_SOURCE) && !defined(__sun__)". This lets qemu use posix_memalign() on NetBSD versions that have it, instead of falling back to valloc() which is wasteful when

[Qemu-devel] [PULL 03/17] machine: Polish -machine xxx,help

From: Alexey Kardashevskiy The "-machine xxx,help" prints kernel-irqchip possible values as "OnOffSplit", this adds separators to the printed line. Also, since only lower case letters are specified in qapi/common.json, this changes the letter cases too. Signed-off-by: Alexey

[Qemu-devel] [PULL 07/17] scripts/make-release: Don't archive .git files

From: Cole Robinson As was last done in 379e21c25, we don't want .git files for submodules here, which we aren't presently doing for capstone and keycodemapdb. Rather than delete the offending files before archiving, ask tar to --exclude=.git Signed-off-by: Cole Robinson

[Qemu-devel] [PULL 14/17] async: use ARRAY_SIZE macro

From: Philippe Mathieu-Daudé Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Michael Tokarev Reviewed-by: Marc-André Lureau ---

Re: [Qemu-devel] [PATCH v3] spapr: set vsmt to MAX(8, smp_threads)

On Fri, Feb 09, 2018 at 09:18:58AM +0100, Laurent Vivier wrote: > We ignore silently the value of smp_threads when we set > the default VSMT value, and if smp_threads is greater than VSMT > kernel is going into trouble later. > > Fixes: 8904e5a750 > ("spapr: Adjust default VSMT value for better

[Qemu-devel] [PULL 13/17] qga: use ARRAY_SIZE macro

From: Philippe Mathieu-Daudé Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Michael Tokarev Reviewed-by: Marc-André Lureau ---

[Qemu-devel] [PULL 15/17] tests/hbitmap: use ARRAY_SIZE macro

From: Philippe Mathieu-Daudé Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Michael Tokarev Reviewed-by: Marc-André Lureau

[Qemu-devel] [PULL 09/17] build: fix typo in error message

From: Laurent Vivier Signed-off-by: Laurent Vivier Signed-off-by: Michael Tokarev Reviewed-by: Philippe Mathieu-Daudé Fixes: f62bbee55d503f639ee9498878ebf42ff4f4299a --- scripts/git-submodule.sh | 2 +- 1 file changed,

[Qemu-devel] [PULL 08/17] configure: Allow capstone=git only if git update is not disabled

From: Alexey Kardashevskiy Even with --disable-git-update, ./configure tries updating the capstone submodule instead of marking it "no"; this disables capstone submodule if git update is disabled. Signed-off-by: Alexey Kardashevskiy Signed-off-by: Michael

[Qemu-devel] [PULL 12/17] MAINTAINERS: Add qemu-binfmt-conf.sh script

From: Thomas Huth qemu-binfmt-conf.sh is used for the Linux usermode emulation, so let's add this file to that section in the MAINTAINERS file. Signed-off-by: Thomas Huth Signed-off-by: Michael Tokarev Reviewed-by: Laurent Vivier

[Qemu-devel] [PULL 17/17] tests/qapi: use ARRAY_SIZE macro

From: Philippe Mathieu-Daudé Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Michael Tokarev Reviewed-by: Marc-André Lureau ---

[Qemu-devel] [PULL 16/17] tests/qapi: use QEMU_IS_ALIGNED macro

From: Philippe Mathieu-Daudé Applied using the Coccinelle semantic patch scripts/coccinelle/use_osdep.cocci Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Michael Tokarev Reviewed-by: Marc-André Lureau ---

[Qemu-devel] [PULL 10/17] maint: Mention web site maintenance in README

From: Eric Blake Now that we have a website that accepts patches on the list, the main project should make it easier to find information about that process. Signed-off-by: Eric Blake Signed-off-by: Michael Tokarev Reviewed-by: Fam Zheng

[Qemu-devel] [PULL 04/17] Drop unneeded system header includes

From: Eric Blake is a non-standard obsolete header that was long ago replaced by . is a non-standard header; it is not obsolete (we must use it for malloc_trim, for example), but generally should not be used in files that just need malloc() and friends, where is the

Re: [Qemu-devel] [PATCH qemu v7 1/4] linux-headers: update to f1517df8701c

On Fri, Feb 09, 2018 at 06:55:00PM +1100, Alexey Kardashevskiy wrote: > Update headers against f1517df8701c. > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f1517df8701c > > Signed-off-by: Alexey Kardashevskiy Might be worth mentioning that

Re: [Qemu-devel] [PATCH v3] spapr: set vsmt to MAX(8, smp_threads)

On Fri, Feb 09, 2018 at 03:06:49PM +0100, Greg Kurz wrote: > On Fri, 9 Feb 2018 09:18:58 +0100 > Laurent Vivier wrote: > > > We ignore silently the value of smp_threads when we set > > the default VSMT value, and if smp_threads is greater than VSMT > > kernel is going into

[Qemu-devel] Qemu aborted in ide_restart_bh after migration

Empty IDE CD-ROM configured on the VM: Make migration for this VM, then qemu aborted in ide_restart_bh. IDEState expect end_transfer_func equal to ide_atapi_cmd, but it refer to ide_dummy_transfer_stop. I have no idea about this, can anyone help me? qemu

[Qemu-devel] [Bug 1748612] [NEW] qemu-user option -strace -D doesn't work

Public bug reported: I have been trying to access qemu -strace output from a script The main problem was it was on stderr, the strace output was merged with my program's stderr output. Then I tried to use the -D option, to log the output to a file. This didn't work even if the log file was

Re: [Qemu-devel] [PATCH 1/1] nbd: increase maximum size of the PWRITE_ZERO request

> On 8 Feb 2018, at 16:28, Eric Blake wrote: > > On 02/08/2018 07:23 AM, Edgar Kaziakhmedov wrote: >> Upstream NBD protocol implementation supports an efficient zero out >> mechanism over the wire, along with the ability to check whether a >> client allows using a hole. >>

Re: [Qemu-devel] [PATCH 1/1] nbd: increase maximum size of the PWRITE_ZERO request

> On 10 Feb 2018, at 18:43, Alex Bligh wrote: > > So I think a reasonable logic for Qemu would be to try NBD_CMD_INFO and find > the maximum write size, and if that's unsupported use 0x (capping at > export size, or export size minus write offset). Ur actually

Re: [Qemu-devel] [PULL 00/26] virtio, vhost, pci, pc: features, fixes and cleanups

> -Original Message- > From: Qemu-devel > [mailto:qemu-devel-bounces+arei.gonglei=huawei@nongnu.org] On > Behalf Of Peter Maydell > Sent: Friday, February 09, 2018 6:07 PM > To: Michael S. Tsirkin > Cc: QEMU Developers > Subject: Re: [Qemu-devel] [PULL 00/26] virtio, vhost, pci, pc:

Re: [Qemu-devel] fix evsrwu and others for powerpcspe

On 02/09/2018 11:25 PM, Michael Tokarev wrote: > 26.12.2017 14:29, i...@mobile-stream.com wrote: >> GEN_SPEOP_ARITH2 declares non-local temporaries then calls >> gen_op_evsrwu/evsrws/evslw which generates branch insn. > > I'm Cc'ing qemu-devel@ for review, since I don't know much about how ppc