[Qemu-devel] [PULL 18/18] mps2-an511: Fix wiring of UART overflow interrupt lines

2017-09-14 Thread Peter Maydell
Fix an error that meant we were wiring every UART's overflow interrupts into the same inputs 0 and 1 of the OR gate, rather than giving each its own input. Cc: qemu-sta...@nongnu.org Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé

[Qemu-devel] [PULL 14/18] target/arm: Avoid an extra temporary for store_exclusive

2017-09-14 Thread Peter Maydell
From: Richard Henderson Instead of copying addr to a local temp, reuse the value (which we have just compared as equal) already saved in cpu_exclusive_addr. Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis

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

2017-09-14 Thread Peter Maydell
ARM queue: nothing particularly exciting, but 18 patches is enough to send out. thanks -- PMM The following changes since commit 3dabde1128b671f36ac6cb36b97b273139964420: Merge remote-tracking branch 'remotes/dgilbert/tags/pull-hmp-20170914' into staging (2017-09-14 16:33:02 +0100

[Qemu-devel] [PULL 12/18] xlnx-zcu102: Mark the EP108 machine as deprecated

2017-09-14 Thread Peter Maydell
From: Alistair Francis The EP108 is the same as the ZCU102, mark it as deprecated as we don't need two machines. Signed-off-by: Alistair Francis Reviewed-by: Edgar E. Iglesias Signed-off-by: Peter Maydell

[Qemu-devel] [PULL 04/18] nvic: Don't apply group priority mask to negative priorities

2017-09-14 Thread Peter Maydell
In several places we were unconditionally applying the nvic_gprio_mask() to a priority value. This is incorrect if the priority is one of the fixed negative priority values (for NMI and HardFault), so don't do it. This bug would have caused both NMI and HardFault to be considered as the same

[Qemu-devel] [PULL 15/18] hw/pci-host/gpex: Set INTx index/gsi mapping

2017-09-14 Thread Peter Maydell
From: Pranavkumar Sawargaonkar To implement INTx to gsi routing we need to pass the gpex host bridge the gsi associated to each INTx index. Let's introduce irq_num array and gpex_set_irq_num setter function. Signed-off-by: Pranavkumar Sawargaonkar

[Qemu-devel] [PULL 01/18] target/arm: Use M_REG_NUM_BANKS rather than hardcoding 2

2017-09-14 Thread Peter Maydell
Use a symbolic constant M_REG_NUM_BANKS for the array size for registers which are banked by M profile security state, rather than hardcoding lots of 2s. Suggested-by: Philippe Mathieu-Daudé Signed-off-by: Peter Maydell Reviewed-by: Philippe

[Qemu-devel] [PULL 11/18] xlnx-zcu102: Add a machine level virtualization property

2017-09-14 Thread Peter Maydell
From: Alistair Francis Add a machine level virtualization property. This defaults to false and can be set to true using this machine command line argument: -machine xlnx-zcu102,virtualization=on This follows what the ARM virt machine does. This property only

[Qemu-devel] [PULL 17/18] hw/pci-host/gpex: Implement PCI INTx routing

2017-09-14 Thread Peter Maydell
From: Pranavkumar Sawargaonkar Now we are able to retrieve the gsi from the INTx pin, let's enable intx_to_irq routing. From that point on, irqfd becomes usable along with INTx when assigning a PCIe device. Signed-off-by: Pranavkumar Sawargaonkar

[Qemu-devel] [PULL 13/18] AArch64: Fix single stepping of ERET instruction

2017-09-14 Thread Peter Maydell
From: Jaroslaw Pelczar Previously when single stepping through ERET instruction via GDB would result in debugger entering the "next" PC after ERET instruction. When debugging in kernel mode, this will also cause unintended behavior, because debugger will try to access

Re: [Qemu-devel] [Qemu devel v8 PATCH 5/5] msf2: Add Emcraft's Smartfusion2 SOM kit

2017-09-14 Thread Philippe Mathieu-Daudé
On 09/14/2017 02:14 PM, Peter Maydell wrote: On 7 September 2017 at 20:24, Subbaraya Sundeep wrote: Emulated Emcraft's Smartfusion2 System On Module starter kit. +static void emcraft_sf2_machine_init(MachineClass *mc) +{ +mc->desc = "SmartFusion2 SOM kit from

[Qemu-devel] [Bug 932490] Re: Qemu fails on -fda /dev/fd0 when no medium is present

2017-09-14 Thread John Snow
Likely the bug as reported still exists, because this attempts to use the disk image, not the floppy drive as a whole. If there's no floppy inserted, there's no disk image to use. Later versions of QEMU even explicitly remove support for pass-through floppy disks. Basically, what you want to do

Re: [Qemu-devel] [Qemu devel v8 PATCH 0/5] Add support for Smartfusion2 SoC

2017-09-14 Thread Peter Maydell
On 14 September 2017 at 18:06, Alistair Francis wrote: > Once you have all the devices there you can enable bus access errors > as well. That's a future patch set though. Ah, I hadn't noticed this code was opting out of bus access errors; I don't want to permit new boards

Re: [Qemu-devel] [Qemu devel v8 PATCH 5/5] msf2: Add Emcraft's Smartfusion2 SOM kit

2017-09-14 Thread Peter Maydell
On 7 September 2017 at 20:24, Subbaraya Sundeep wrote: > Emulated Emcraft's Smartfusion2 System On Module starter > kit. > +static void emcraft_sf2_machine_init(MachineClass *mc) > +{ > +mc->desc = "SmartFusion2 SOM kit from Emcraft (M2S010)"; > +mc->init =

Re: [Qemu-devel] [Qemu devel v8 PATCH 0/5] Add support for Smartfusion2 SoC

2017-09-14 Thread Alistair Francis
On Wed, Sep 13, 2017 at 9:10 PM, Philippe Mathieu-Daudé wrote: > On 09/08/2017 04:24 AM, sundeep subbaraya wrote: >> >> Hi Phillipe, >> >> On Fri, Sep 8, 2017 at 3:14 AM, Philippe Mathieu-Daudé > > wrote: >> >> Hi Subbaraya, >> >>

Re: [Qemu-devel] [PULL 0/7] hmp queue

2017-09-14 Thread Peter Maydell
edef clash (2017-09-14 15:00:41 +0100) > > are available in the git repository at: > > git://github.com/dagrh/qemu.git tags/pull-hmp-20170914 > > for you to fetch changes up to d0f63c1e291a0c27cafc0e6faec5a84130b012e0: > > hmp: introduce 'in

Re: [Qemu-devel] [PATCH] xen_disk: avoid use of g_malloc0_n()

2017-09-14 Thread Philippe Mathieu-Daudé
Marc-André can you queue this patch in your gnew series? On 09/12/2017 11:24 AM, Jan Beulich wrote: Prefer g_new() / g_new0() to be farther backwards compatible with older glib versions. As there's no point in zeroing the allocation here (the loop right afterwards fully initializes the memory),

Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device

2017-09-14 Thread Halil Pasic
On 09/14/2017 04:26 PM, Cornelia Huck wrote: > On Wed, 13 Sep 2017 15:27:51 +0200 > Halil Pasic wrote: > >> Add a fake device meant for testing the correctness of our css emulation. >> >> What we currently have is writing a Fibonacci sequence of uint32_t to the >>

[Qemu-devel] [PATCH v4 0/1] target/m68k, linux-user: manage FP registers in ucontext

2017-09-14 Thread Laurent Vivier
v3: rebase and add R-b from rth should also fix patchew errors as the missing commits are now in the master v2: add missing cpu_m68k_set_fpcr() to update FPU internal state The v1 was in the v5 of the series "target-m68k: implement 680x0 FPU" Laurent Vivier (1):

[Qemu-devel] [PATCH v4 1/1] target/m68k, linux-user: manage FP registers in ucontext

2017-09-14 Thread Laurent Vivier
Signed-off-by: Laurent Vivier Reviewed-by: Richard Henderson --- linux-user/signal.c | 43 +++ 1 file changed, 43 insertions(+) diff --git a/linux-user/signal.c b/linux-user/signal.c index cc0c3fcee9..7a238aaea1

Re: [Qemu-devel] MIPS 'move' insn emulation

2017-09-14 Thread Sergey Smolov
On 14.09.2017 17:23, Yongbok Kim wrote: On 14/09/2017 15:16, Sergey Smolov wrote: On 14.09.2017 16:58, Peter Maydell wrote: At translate time it is generating some extra code which at runtime will call the helper_trace_reg_access() function, passing it the values in the registers at this

[Qemu-devel] [PATCH] 9pfs: check the size of transport buffer before marshaling

2017-09-14 Thread Jan Dakinevich
v9fs_do_readdir_with_stat() and v9fs_do_readdir() stores as much data in the buffer as can fit unless marshaling erorr occurs. However, after commit 23a006d the behavior pdu_marshal was changed, and on error the routine assumes that buffers are misconfigured and breaks communication.

Re: [Qemu-devel] [PATCH] xen_disk: avoid use of g_malloc0_n()

2017-09-14 Thread Anthony PERARD
On Tue, Sep 12, 2017 at 08:24:21AM -0600, Jan Beulich wrote: > Prefer g_new() / g_new0() to be farther backwards compatible with older > glib versions. As there's no point in zeroing the allocation here (the > loop right afterwards fully initializes the memory), use the former. > > Signed-off-by:

Re: [Qemu-devel] [PATCH 6/7] trace: Add event "guest_inst_after"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:35 AM, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- >> accel/tcg/translator.c | 23 ++- >> trace-events |8 >> 2 files changed, 26 insertions(+), 5 deletions(-) >> >>

Re: [Qemu-devel] [PATCH v2] osdep: Fix ROUND_UP(64-bit, 32-bit)

2017-09-14 Thread Laszlo Ersek
On 09/14/17 15:49, Eric Blake wrote: > When using bit-wise operations that exploit the power-of-two > nature of the second argument of ROUND_UP(), we still need to > ensure that the mask is as wide as the first argument (done > by using a ternary to force proper arithmetic promotion). > Unpatched,

Re: [Qemu-devel] [PATCH v2 09/16] tcg/i386: Add vector operations

2017-09-14 Thread Alex Bennée
Richard Henderson writes: > Signed-off-by: Richard Henderson > --- > tcg/i386/tcg-target.h | 46 - > tcg/i386/tcg-target.inc.c | 438 > +- > 2 files changed, 438 insertions(+), 46

Re: [Qemu-devel] [PATCH 5/7] trace: Add event "guest_bbl_after"

2017-09-14 Thread Richard Henderson
On 09/14/2017 08:20 AM, Lluís Vilanova wrote: >> I don't understand why you wouldn't just modify tcg_gen_goto_tb and >> tcg_gen_exit_tb instead. > > I prefer to keep all generic translation-related tracing on a single file, > where > it is easier to reason about. My point here was more about

Re: [Qemu-devel] [PATCH 4/7] tcg: Add support for "inlining" regions of code

2017-09-14 Thread Richard Henderson
On 09/14/2017 08:20 AM, Lluís Vilanova wrote: > Richard Henderson writes: > >> On 09/10/2017 09:27 AM, Lluís Vilanova wrote: >>> TCG BBLs and instructions have multiple exit points from where to raise >>> tracing events, but some of the necessary information in the generic >>> disassembly

Re: [Qemu-devel] [PATCH 3/7] trace: Add event "guest_inst_info_before"

2017-09-14 Thread Richard Henderson
On 09/14/2017 07:59 AM, Lluís Vilanova wrote: > Since physical address information is not gonna be added in this series (if at > all), what do you say about hoisting instruction length info into > guest_insn_before/after and dropping the *_info variants? That should be fine. r~

Re: [Qemu-devel] [libvirt] QEMU -M nvdimm=on and hotplug

2017-09-14 Thread Stefan Hajnoczi
On Wed, Sep 13, 2017 at 05:28:56PM +0200, Michal Privoznik wrote: > On 09/13/2017 03:54 PM, Stefan Hajnoczi wrote: > > 2. Only allow NVDIMM hotplug if the domain was started with -M > >nvdimm=on. > > > > I think QEMU will not add -M nvdimm=on to the "pc" machine type by > > default since it

Re: [Qemu-devel] [PATCH v7 11/20] dirty-bitmap: Change bdrv_dirty_iter_next() to report byte offset

2017-09-14 Thread John Snow
On 09/14/2017 07:58 AM, Eric Blake wrote: > On 09/13/2017 07:15 PM, John Snow wrote: >> >> >> On 09/12/2017 04:31 PM, Eric Blake wrote: >>> Thanks to recent cleanups, most callers were scaling a return value >>> of sectors into bytes (the exception, in qcow2-bitmap, will be >>> converted to

Re: [Qemu-devel] [PATCH 15/18] block/mirror: Add active mirroring

2017-09-14 Thread Stefan Hajnoczi
On Wed, Sep 13, 2017 at 08:19:07PM +0200, Max Reitz wrote: > This patch implements active synchronous mirroring. In active mode, the > passive mechanism will still be in place and is used to copy all > initially dirty clusters off the source disk; but every write request > will write data both to

[Qemu-devel] [PATCH v2 2/2] virtio-gpu-ccw: Create a virtio gpu device for the ccw bus

2017-09-14 Thread Farhan Ali
Wire up the virtio-gpu device for the CCW bus. The virtio-gpu is a virtio-1 device, so disable revision 0. Signed-off-by: Farhan Ali Acked-by: Christian Borntraeger Reviewed-by: Halil Pasic --- hw/s390x/virtio-ccw.c |

[Qemu-devel] [PATCH v2 0/2] Virtio GPU for S390

2017-09-14 Thread Farhan Ali
These patches wire up the virtio-gpu device for CCW bus for S390. For the S390 architecture which does not natively support any graphics device, virtio gpu in 2D mode could be used to emulate a simple graphics card and use VNC as the display. eg: qemu-system-s390x ... -device

[Qemu-devel] [PATCH v2 1/2] virtio-gpu: Handle endian conversion

2017-09-14 Thread Farhan Ali
Virtio GPU code currently only supports litte endian format, and so using the Virtio GPU device on a big endian machine does not work. Let's fix it by supporting the correct host cpu byte order. Signed-off-by: Farhan Ali --- hw/display/virtio-gpu.c | 75

Re: [Qemu-devel] [PATCHv6 5/5] seccomp: add resourcecontrol argument to command line

2017-09-14 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 01:44:07PM +0200, Eduardo Otubo wrote: > This patch adds [,resourcecontrol=deny] to `-sandbox on' option. It > blacklists all process affinity and scheduler priority system calls to > avoid any bigger of the process. > > Signed-off-by: Eduardo Otubo >

Re: [Qemu-devel] [PATCHv6 0/6] seccomp: feature refactoring

2017-09-14 Thread Daniel P. Berrange
On Thu, Sep 14, 2017 at 05:32:15PM +0200, Eduardo Otubo wrote: > On Fri, Sep 08, 2017 at 01:44:02PM +0200, Eduardo Otubo wrote: > > v6: > > * remove switch-case > > * invert obsolete option logic at vl.c > > * remove debug info > > v5: > > * replaced strcmp by g_str_equal > > *

Re: [Qemu-devel] [PATCHv6 4/5] seccomp: add spawn argument to command line

2017-09-14 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 01:44:06PM +0200, Eduardo Otubo wrote: > This patch adds [,spawn=deny] argument to `-sandbox on' option. It > blacklists fork and execve system calls, avoiding Qemu to spawn new > threads or processes. > > Signed-off-by: Eduardo Otubo > --- >

Re: [Qemu-devel] [PATCHv6 3/5] seccomp: add elevateprivileges argument to command line

2017-09-14 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 01:44:05PM +0200, Eduardo Otubo wrote: > This patch introduces the new argument > [,elevateprivileges=allow|deny|children] to the `-sandbox on'. It allows > or denies Qemu process to elevate its privileges by blacklisting all > set*uid|gid system calls. The 'children'

Re: [Qemu-devel] [PATCHv6 2/5] seccomp: add obsolete argument to command line

2017-09-14 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 01:44:04PM +0200, Eduardo Otubo wrote: > This patch introduces the argument [,obsolete=allow] to the `-sandbox on' > option. It allows Qemu to run safely on old system that still relies on > old system calls. > > Signed-off-by: Eduardo Otubo > --- >

Re: [Qemu-devel] [PATCHv6 1/5] seccomp: changing from whitelist to blacklist

2017-09-14 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 01:44:03PM +0200, Eduardo Otubo wrote: > This patch changes the default behavior of the seccomp filter from > whitelist to blacklist. By default now all system calls are allowed and > a small black list of definitely forbidden ones was created. > > Signed-off-by: Eduardo

Re: [Qemu-devel] [PATCH 00/18] block/mirror: Add active-sync mirroring

2017-09-14 Thread Stefan Hajnoczi
On Wed, Sep 13, 2017 at 08:18:52PM +0200, Max Reitz wrote: > There may be a couple of things to do on top of this series: > - Allow switching between active and passive mode at runtime: This > should not be too difficult to implement, the main question is how to > expose it to the user. > (I

Re: [Qemu-devel] [Qemu devel v8 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block

2017-09-14 Thread sundeep subbaraya
Hi Philippe, On Thu, Sep 14, 2017 at 10:06 AM, Philippe Mathieu-Daudé wrote: > Hi Sundeep, > > > On 09/07/2017 04:24 PM, Subbaraya Sundeep wrote: > >> Added Sytem register block of Smartfusion2. >> This block has PLL registers which are accessed by guest. >> >> Signed-off-by:

Re: [Qemu-devel] [RFC 14/15] qmp: support out-of-band (oob) execution

2017-09-14 Thread Stefan Hajnoczi
On Thu, Sep 14, 2017 at 03:50:35PM +0800, Peter Xu wrote: > diff --git a/docs/devel/qapi-code-gen.txt b/docs/devel/qapi-code-gen.txt > index 61fa167..47d16bb 100644 > --- a/docs/devel/qapi-code-gen.txt > +++ b/docs/devel/qapi-code-gen.txt > @@ -665,6 +665,16 @@ allowed to run out-of-band can also

Re: [Qemu-devel] [PULL 0/4] Usb 20170913 patches

2017-09-14 Thread Peter Maydell
On 13 September 2017 at 10:32, Gerd Hoffmann wrote: > The following changes since commit 619c200f6ce2f44fbc8a5732174530c3b4782aab: > > Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed' > into staging (2017-09-11 11:44:30 +0100) > > are available in

Re: [Qemu-devel] [PATCHv6 0/6] seccomp: feature refactoring

2017-09-14 Thread Eduardo Otubo
On Fri, Sep 08, 2017 at 01:44:02PM +0200, Eduardo Otubo wrote: > v6: > * remove switch-case > * invert obsolete option logic at vl.c > * remove debug info > v5: > * replaced strcmp by g_str_equal > * removed useless goto > * fixed style problems > > v4: > * include

Re: [Qemu-devel] [Qemu devel v8 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block

2017-09-14 Thread sundeep subbaraya
Hi Philippe, On Thu, Sep 14, 2017 at 6:43 PM, Peter Maydell wrote: > On 14 September 2017 at 05:36, Philippe Mathieu-Daudé > wrote: > > On 09/07/2017 04:24 PM, Subbaraya Sundeep wrote: > >> +static uint64_t msf2_sysreg_read(void *opaque, hwaddr

Re: [Qemu-devel] [PATCH 5/7] trace: Add event "guest_bbl_after"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:31 AM, Lluís Vilanova wrote: >> +void translator__gen_goto_tb(TCGContext *ctx) >> +{ >> +if (ctx->disas.in_guest_code && >> +(TRACE_GUEST_BBL_AFTER_ENABLED)) { >> +if (ctx->disas.inline_label == NULL) { >> +

Re: [Qemu-devel] [PATCH 4/7] tcg: Add support for "inlining" regions of code

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:27 AM, Lluís Vilanova wrote: >> TCG BBLs and instructions have multiple exit points from where to raise >> tracing events, but some of the necessary information in the generic >> disassembly infrastructure is not available until after generating these

Re: [Qemu-devel] [RFC 00/15] QMP: out-of-band (OOB) execution support

2017-09-14 Thread Stefan Hajnoczi
On Thu, Sep 14, 2017 at 01:15:09PM +0200, Marc-André Lureau wrote: > There should be a limit in the number of requests the thread can > queue. Before the patch, the limit was enforced by system socket > buffering I think. Now, should oob commands still be processed even if > the queue is full? If

Re: [Qemu-devel] [PATCH v4 0/3] Generic PCIe host bridge INTx determination for INTx routing

2017-09-14 Thread Peter Maydell
On 13 September 2017 at 10:46, Eric Auger wrote: > This series implements INTx to gsi routing for ARM VIRT/Gpex. This is > a respin of [1] which was lost in limbo. > > ARM virt uses GPEX PCIe bridge. This latter does not implement INTx > to GSI routing. PCIe/INTx assignment

[Qemu-devel] [PULL 6/7] qmp: introduce query-memory-size-summary command

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Vadim Galitsyn Add a new query-memory-size-summary command which provides the following memory information in bytes: * base-memory - size of "base" memory specified with command line option -m. * plugged-memory - amount of memory that was hot-plugged.

[Qemu-devel] [PULL 3/7] dump: do not dump non-existent guest memory

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Cornelia Huck It does not really make sense to dump memory that is not there. Moreover, that fixes a segmentation fault when calling dump-guest-memory with no filter for a machine with no memory defined. New behaviour is: (qemu) dump-guest-memory /dev/null dump: no

[Qemu-devel] [PULL 4/7] tests/hmp: test "none" machine with memory

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Laurent Vivier and add a test case of dump-guest-memory without "[begin length]" parameters. Signed-off-by: Laurent Vivier Reviewed-by: Thomas Huth Reviewed-by: Cornelia Huck Message-Id:

Re: [Qemu-devel] [PATCH 3/7] trace: Add event "guest_inst_info_before"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:23 AM, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- >> accel/tcg/translator.c | 18 ++ >> trace-events |9 + >> 2 files changed, 27 insertions(+) >> >> diff --git

[Qemu-devel] [PULL 7/7] hmp: introduce 'info memory_size_summary' command

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Vadim Galitsyn Add 'info memory_size_summary' command which is a sibling of QMP command query-memory-size-summary. It provides the following memory information in bytes: * base-memory - size of "base" memory specified with command line option -m. *

[Qemu-devel] [PULL 1/7] hmp: fix "dump-quest-memory" segfault (ppc)

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Laurent Vivier Running QEMU with qemu-system-ppc64 -M none -nographic -m 256 and executing dump-guest-memory /dev/null 0 8192 results in segfault Fix by checking if we have CPU, and exit with error if there is no CPU: (qemu) dump-guest-memory /dev/null

Re: [Qemu-devel] [Qemu-arm] [PATCH v7 13/20] hw/arm/smmuv3: Implement IOMMU memory region replay callback

2017-09-14 Thread Tomasz Nowicki
On 14.09.2017 16:31, Tomasz Nowicki wrote: Hi Eric, On 14.09.2017 11:27, Linu Cherian wrote: Hi Eric, On Fri Sep 01, 2017 at 07:21:16PM +0200, Eric Auger wrote: memory_region_iommu_replay() is used for VFIO integration. However its default implementation is not adapted to SMMUv3 IOMMU

[Qemu-devel] [PULL 5/7] hmp: extend "info numa" with hotplugged memory information

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Vadim Galitsyn Report amount of hotplugged memory in addition to total amount per NUMA node. Signed-off-by: Vadim Galitsyn Cc: Eduardo Habkost Cc: David Hildenbrand Cc:

[Qemu-devel] [PULL 0/7] hmp queue

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" <dgilb...@redhat.com> The following changes since commit 9d81b2d2000f41be55a0624a26873f993fb6e928: sparc: Fix typedef clash (2017-09-14 15:00:41 +0100) are available in the git repository at: git://github.com/dagrh/qemu.git tags/pull-hmp-2

[Qemu-devel] [PULL 2/7] hmp: fix "dump-quest-memory" segfault (arm)

2017-09-14 Thread Dr. David Alan Gilbert (git)
From: Laurent Vivier Running QEMU with qemu-system-aarch64 -M none -nographic -m 256 and executing dump-guest-memory /dev/null 0 8192 results in segfault Fix by checking if we have CPU, and exit with error if there is no CPU: (qemu) dump-guest-memory /dev/null

Re: [Qemu-devel] [PATCH v6 01/22] instrument: Add documentation

2017-09-14 Thread Peter Maydell
On 13 September 2017 at 10:57, Lluís Vilanova wrote: > Signed-off-by: Lluís Vilanova > --- > MAINTAINERS |6 ++ > docs/instrument.txt | 173 > +++ > 2 files changed, 179 insertions(+) >

[Qemu-devel] [PATCH v3 18/20] vpc: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vpc driver accordingly. Drop the now-unused get_sector_offset(). Signed-off-by: Eric Blake --- v3: rebase to master v2: drop get_sector_offset() [Kevin], rebase to mapping flag ---

[Qemu-devel] [PATCH v3 11/20] qcow2: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qcow2 driver accordingly. For now, we are ignoring the 'mapping' hint. However, it should be relatively straightforward to honor the hint as a way to return larger *pnum values when we have consecutive

[Qemu-devel] [PATCH v3 08/20] null: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the null driver accordingly. Signed-off-by: Eric Blake --- v2: rebase to mapping parameter --- block/null.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-)

Re: [Qemu-devel] [PATCH] configure: Allow --enable-seccomp on s390x, too

2017-09-14 Thread Cornelia Huck
On Thu, 14 Sep 2017 12:36:03 +0200 Thomas Huth wrote: > libseccomp supports s390x since version 2.3.0, and I was able to start > a VM with "-sandbox on" without any obvious problems by using this patch, > so it should be safe to allow --enable-seccomp on s390x nowadays, too. >

Re: [Qemu-devel] [PATCH v5 00/22] instrument: Add basic event instrumentation

2017-09-14 Thread Peter Maydell
On 12 September 2017 at 22:01, Lluís Vilanova wrote: > This series adds an API to add instrumentation events. > > It also provides additional APIs for: > * Controlling tracing events. > * Peek/poke guest memory. > Future APIs (for later series): > * Peek/poke guest

[Qemu-devel] [PATCH v3 09/20] parallels: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the parallels driver accordingly. Note that the internal function block_status() is still sector-based, because it is still in use by other sector-based functions; but that's okay because request_alignment is

[Qemu-devel] [PATCH v3 20/20] block: Drop unused .bdrv_co_get_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Now that all drivers have been updated to provide the byte-based .bdrv_co_block_status(), we can delete the sector-based interface. Signed-off-by: Eric Blake --- v2: rebase to earlier changes ---

Re: [Qemu-devel] [PULL 0/4] Ui 20170913 patches

2017-09-14 Thread Peter Maydell
On 13 September 2017 at 09:40, Gerd Hoffmann wrote: > The following changes since commit 619c200f6ce2f44fbc8a5732174530c3b4782aab: > > Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed' > into staging (2017-09-11 11:44:30 +0100) > > are available in

[Qemu-devel] [PATCH v3 06/20] iscsi: Switch iscsi_allocmap_update() to byte-based

2017-09-14 Thread Eric Blake
We are gradually converting to byte-based interfaces, as they are easier to reason about than sector-based. Convert all uses of the allocmap (no semantic change). Callers that already had bytes available are simpler, and callers that now scale to bytes will be easier to switch to byte-based in

[Qemu-devel] [PATCH v3 16/20] vdi: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vdi driver accordingly. Note that the TODO is already covered (the block layer guarantees bounds of its requests), and that we can remove the now-unused s->block_sectors. Signed-off-by: Eric Blake

[Qemu-devel] [PATCH v3 17/20] vmdk: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vmdk driver accordingly. Signed-off-by: Eric Blake --- v2: rebase to mapping flag --- block/vmdk.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-)

[Qemu-devel] [PATCH v3 04/20] gluster: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the gluster driver accordingly. In mapping mode, note that the entire file is reported as allocated, so we can take a shortcut and skip find_allocation(). Signed-off-by: Eric Blake --- v2:

[Qemu-devel] [PATCH v3 19/20] vvfat: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the vvfat driver accordingly. Note that we can rely on the block driver having already clamped limits to our block size, and simplify accordingly. Signed-off-by: Eric Blake --- v2: rebase

[Qemu-devel] [PATCH v3 15/20] vdi: Avoid bitrot of debugging code

2017-09-14 Thread Eric Blake
Rework the debug define so that we always get -Wformat checking, even when debugging is disabled. Signed-off-by: Eric Blake Reviewed-by: Stefan Weil Reviewed-by: Philippe Mathieu-Daudé --- v2: no change --- block/vdi.c | 12 +---

[Qemu-devel] [PATCH v3 02/20] block: Switch passthrough drivers to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the generic helpers, and all passthrough clients (blkdebug, commit, mirror, throttle) accordingly. Signed-off-by: Eric Blake --- v3: rebase to addition of throttle driver v2: rebase to

Re: [Qemu-devel] [PATCH] sparc: Fix typedef clash

2017-09-14 Thread Peter Maydell
On 14 September 2017 at 13:36, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Older compilers (rhel6) don't like redefinition of typedefs > > Fixes: 12a6c15ef31c98ecefa63e91ac36955383038384 > > Signed-off-by: Dr. David Alan

[Qemu-devel] [PATCH v3 12/20] qed: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qed driver accordingly. There is no intent to optimize based on the mapping flag for this format. Signed-off-by: Eric Blake --- v2: rebase to mapping flag, fix mask in

[Qemu-devel] [PATCH v3 14/20] sheepdog: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the sheepdog driver accordingly. Signed-off-by: Eric Blake --- v2: rebase to mapping flag --- block/sheepdog.c | 23 +++ 1 file changed, 11 insertions(+), 12

[Qemu-devel] [PATCH v3 01/20] block: Add .bdrv_co_block_status() callback

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Now that the block layer exposes byte-based allocation, it's time to tackle the drivers. Add a new callback that operates on as small as byte boundaries. Subsequent patches will then update individual drivers, then

[Qemu-devel] [PATCH v3 13/20] raw: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the raw driver accordingly. Signed-off-by: Eric Blake --- v2: rebase to mapping --- block/raw-format.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git

[Qemu-devel] [PATCH v3 10/20] qcow: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the qcow driver accordingly. There is no intent to optimize based on the mapping flag for this format. Signed-off-by: Eric Blake --- v3: rebase to master v2: rebase to mapping flag ---

[Qemu-devel] [PATCH v3 03/20] file-posix: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the file protocol driver accordingly. In mapping mode, note that the entire file is reported as allocated, so we can take a shortcut and skip lseek(). Signed-off-by: Eric Blake --- v2:

Re: [Qemu-devel] [PATCH 2/7] trace: Add event "guest_inst_before"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:19 AM, Lluís Vilanova wrote: >> while (true) { >> +target_ulong pc_insn = db->pc_next; > Why not just "pc"? >> + db-> num_insns++; ops-> insn_start(db, cpu); >> tcg_debug_assert(db->is_jmp == DISAS_NEXT); /* no early exit */ >> @@ -96,6

[Qemu-devel] [PATCH v3 00/20] add byte-based block_status driver callbacks

2017-09-14 Thread Eric Blake
There are patches floating around to add NBD_CMD_BLOCK_STATUS, but NBD wants to report status on byte granularity (even if the reporting will probably be naturally aligned to sectors or even much higher levels). I've therefore started the task of converting our block status code to report at a

[Qemu-devel] [PATCH v3 07/20] iscsi: Switch to .bdrv_co_block_status()

2017-09-14 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the iscsi driver accordingly. In this case, it is handy to teach iscsi_co_block_status() to handle a NULL file parameter, even though the block layer passes a non-NULL value, because we also call the function

[Qemu-devel] [PATCH v3 05/20] iscsi: Switch cluster_sectors to byte-based

2017-09-14 Thread Eric Blake
We are gradually converting to byte-based interfaces, as they are easier to reason about than sector-based. Convert all uses of the cluster size in sectors, along with adding assertions that we are not dividing by zero. Signed-off-by: Eric Blake --- v2: no change ---

Re: [Qemu-devel] [Qemu-ppc] [PATCH] ppc/kvm: use kvm_vm_check_extension() in kvmppc_is_pr()

2017-09-14 Thread Greg Kurz
On Thu, 14 Sep 2017 13:27:49 +0200 Greg Kurz wrote: > On Thu, 14 Sep 2017 13:17:48 +0200 > Thomas Huth wrote: > > > On 14.09.2017 12:48, Greg Kurz wrote: > > > If the host has both KVM PR and KVM HV loaded and we pass: > > > > > > -machine

Re: [Qemu-devel] [RFC 0/6] initial plugin support

2017-09-14 Thread Peter Maydell
On 6 September 2017 at 21:28, Emilio G. Cota wrote: > Related threads: > [PATCH 00/13] instrument: Add basic event instrumentation > Date: Mon, 24 Jul 2017 20:02:24 +0300 > https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg07419.html > and > [PATCH v4 00/20]

Re: [Qemu-devel] [Qemu-arm] [PATCH v7 13/20] hw/arm/smmuv3: Implement IOMMU memory region replay callback

2017-09-14 Thread Tomasz Nowicki
Hi Eric, On 14.09.2017 11:27, Linu Cherian wrote: Hi Eric, On Fri Sep 01, 2017 at 07:21:16PM +0200, Eric Auger wrote: memory_region_iommu_replay() is used for VFIO integration. However its default implementation is not adapted to SMMUv3 IOMMU memory region. Indeed the input address range is

Re: [Qemu-devel] MIPS 'move' insn emulation

2017-09-14 Thread Peter Maydell
On 14 September 2017 at 15:23, Yongbok Kim wrote: > The reason why your modification is failed is because you passed wrong > argument. Remember that you are not just calling the helper function from > translate.c but you are generating some code to let call the helper >

Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device

2017-09-14 Thread Cornelia Huck
On Wed, 13 Sep 2017 15:27:51 +0200 Halil Pasic wrote: > Add a fake device meant for testing the correctness of our css emulation. > > What we currently have is writing a Fibonacci sequence of uint32_t to the > device via ccw write. The write is going to fail if it

Re: [Qemu-devel] MIPS 'move' insn emulation

2017-09-14 Thread Yongbok Kim
On 14/09/2017 15:16, Sergey Smolov wrote: > > On 14.09.2017 16:58, Peter Maydell wrote: >> At translate time it is generating some extra code which at runtime >> will call the helper_trace_reg_access() function, passing it the >> values in the registers at this point. This will result in poor

Re: [Qemu-devel] [PATCH 1/7] trace: Add event "guest_bbl_before"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:15 AM, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- >> accel/tcg/translator.c |6 ++ >> trace-events | 11 +++ >> 2 files changed, 17 insertions(+) >> >> diff --git

Re: [Qemu-devel] [PATCH v2 0/4] add CCW indirect data access support

2017-09-14 Thread Cornelia Huck
On Wed, 13 Sep 2017 13:50:25 +0200 Halil Pasic wrote: > Abstract > > > The objective of this series is introducing CCW IDA (indirect data > access) support to our virtual channel subsystem implementation. Briefly > CCW IDA can be thought of as a kind of a

[Qemu-devel] [PATCH v2 1/2] checkpatch: add hwaddr to @typeList

2017-09-14 Thread Greg Kurz
The script doesn't know about all possible types and learn them as it parses the code. If it reaches a line with a type cast but the type isn't known yet, it is misinterpreted as an identifier. For example the following line: foo = (hwaddr) -1; results in the following false-positive to be

Re: [Qemu-devel] MIPS 'move' insn emulation

2017-09-14 Thread Sergey Smolov
On 14.09.2017 16:58, Peter Maydell wrote: At translate time it is generating some extra code which at runtime will call the helper_trace_reg_access() function, passing it the values in the registers at this point. This will result in poor performance if you do it for frequently executed

[Qemu-devel] [PATCH v2 2/2] spapr_pci: make index property mandatory

2017-09-14 Thread Greg Kurz
Creating several PHBs without index property confuses the DRC code and causes issues: - only the first index-less PHB is functional, the other ones will silently ignore hotplugging of PCI devices - QEMU will even terminate if these PHBs have cold-plugged devices qemu-system-ppc64: -device

[Qemu-devel] [PATCH v2 0/2] spapr_pci: make index property mandatory

2017-09-14 Thread Greg Kurz
Patch 1 is a proposal to silence patchew when it parses patch 2 :) -- Greg --- Greg Kurz (2): checkpatch: add hwaddr to @typeList spapr_pci: make index property mandatory hw/ppc/spapr_pci.c| 53 ++--- scripts/checkpatch.pl |1

<    1   2   3   4   >