[Qemu-devel] [PATCH v5 1/6] qemu-io: Add missing option documentation

2016-05-07 Thread Eric Blake
The Usage: summary is missing several options, but rather than having to maintain it, it's simpler to just state [OPTIONS], since the options are spelled out below. Commit 499afa2 added --image-opts, but forgot to document it in --help. Likewise for commit 9e8f183 and -d/--discard. Commit e3aff4

Re: [Qemu-devel] [PATCH v5 0/6] qemu-io: UI enhancements

2016-05-07 Thread Eric Blake
On 05/07/2016 09:16 PM, Eric Blake wrote: > While working on NBD, I found myself cursing the qemu-io UI for > not letting me test various scenarios, particularly after fixing > NBD to serve at byte granularity [1]. And in the process of > writing these qemu-io enhancements, I also managed to flush

[Qemu-devel] [PATCH v5 5/6] qemu-io: Add 'write -f' to test FUA flag

2016-05-07 Thread Eric Blake
Make it easier to test block drivers with BDRV_REQ_FUA in .supported_write_flags, by adding the '-f' flag to qemu-io to conditionally pass the flag through to specific writes ('write', 'write -z', 'writev', 'aio_write', 'aio_write -z'). You'll want to use 'qemu-io -t none' to actually make -f usefu

[Qemu-devel] [PATCH v5 3/6] qemu-io: Use bool for command line flags

2016-05-07 Thread Eric Blake
We require a C99 compiler; let's use it to express what we really mean. (Yes, we now have an instance of 'if (bool + bool + bool > 1)', which, although semantically valid C, looks ugly; it gets cleaned up later.) Signed-off-by: Eric Blake Reviewed-by: Max Reitz --- v5: tweak commit message ---

[Qemu-devel] [PATCH v5 4/6] qemu-io: Allow unaligned access by default

2016-05-07 Thread Eric Blake
There's no reason to require the user to specify a flag just so they can pass in unaligned numbers. Keep 'read -p' and 'write -p' as no-ops so that I don't have to hunt down and update all users of qemu-io, but otherwise make their behavior default as 'read' and 'write'. Also fix 'write -z', 'rea

[Qemu-devel] [PATCH v5 0/6] qemu-io: UI enhancements

2016-05-07 Thread Eric Blake
While working on NBD, I found myself cursing the qemu-io UI for not letting me test various scenarios, particularly after fixing NBD to serve at byte granularity [1]. And in the process of writing these qemu-io enhancements, I also managed to flush out several other bugs in the block layer proper,

[Qemu-devel] [PATCH v5 2/6] qemu-io: Make 'open' subcommand more like command line

2016-05-07 Thread Eric Blake
The command line defaults to BDRV_O_UNMAP, but can use -d to reset it. Meanwhile, the 'open' subcommand was defaulting to no discards, with no way to set it. The command line has both -n and -tMODE to set a variety of cache modes, but the 'open' subcommand had only -n. The 'open' subcommand had

[Qemu-devel] [PATCH v5 6/6] qemu-io: Add 'write -z -u' to test MAY_UNMAP flag

2016-05-07 Thread Eric Blake
Make it easier to control whether the BDRV_REQ_MAY_UNMAP flag can be passed through a write_zeroes command, by adding the '-u' flag to qemu-io 'write -z' and 'aio_write -z'. To be useful, the device has to be opened with BDRV_O_UNMAP (done by default in qemu-io, but can be made explicit with '-d u

[Qemu-devel] help RAM access

2016-05-07 Thread Stéphane GONZALEZ
Hello, I would like to read at a particular RAM target address from the QEMU side, but I didn't find where I have to do. Is someone could give me an example ? Many thanks,

Re: [Qemu-devel] [PATCH 51/52] target-m68k: add cmpm

2016-05-07 Thread Peter Maydell
On 7 May 2016 at 20:01, Laurent Vivier wrote: > > > Le 07/05/2016 à 00:00, Richard Henderson a écrit : >> On 05/04/2016 11:21 AM, Laurent Vivier wrote: >>> +reg = AREG(insn, 0); >>> +src = gen_load(s, opsize, reg, 1); >>> +tcg_gen_addi_i32(reg, reg, opsize_bytes(opsize)); >>> + >>> +

Re: [Qemu-devel] [PATCH 51/52] target-m68k: add cmpm

2016-05-07 Thread Laurent Vivier
Le 07/05/2016 à 00:00, Richard Henderson a écrit : > On 05/04/2016 11:21 AM, Laurent Vivier wrote: >> +reg = AREG(insn, 0); >> +src = gen_load(s, opsize, reg, 1); >> +tcg_gen_addi_i32(reg, reg, opsize_bytes(opsize)); >> + >> +reg = AREG(insn, 9); >> +dest = gen_load(s, opsize,

Re: [Qemu-devel] [Bug 1579327] [NEW] emulation of vexpress-a9 vexpress-a15 in multicore mode seems to be broken

2016-05-07 Thread Peter Maydell
On 7 May 2016 at 09:08, embedded0n3 wrote: > Public bug reported: > > Report based on: > https://stackoverflow.com/questions/37068688/have-you-got-issues-with-running-u-boot-for-versatile-express-a9-platform-under > > I don't see any bug reports about vexpress boards emulation here, so I > decided

[Qemu-devel] [Bug 1579327] [NEW] emulation of vexpress-a9 vexpress-a15 in multicore mode seems to be broken

2016-05-07 Thread embedded0n3
Public bug reported: Report based on: https://stackoverflow.com/questions/37068688/have-you-got-issues-with-running-u-boot-for-versatile-express-a9-platform-under I don't see any bug reports about vexpress boards emulation here, so I decided to let you know: version tested: qemu 2.0.0 on 32 bit

Re: [Qemu-devel] [PATCH 4/4] kvm: support MSI_X2APIC capability

2016-05-07 Thread Jan Kiszka
On 2016-05-06 22:53, Radim Krčmář wrote: > The capability alows us to express x2APIC destinations. "allows" Will the possibility to create >254 CPUs be indirectly coupled to this capability, or should userland check for it explicitly then? Will the kernel handle the case gracefully that AMD CPUs

Re: [Qemu-devel] [PATCH 1/4] apic: add deliver_msi to APICCommonClass

2016-05-07 Thread Jan Kiszka
On 2016-05-06 22:53, Radim Krčmář wrote: > The only way to send interrupts from outside of APIC devices is to use > the MSI-inspired memory mapped interface. The interface is not > sufficient because interrupt remapping in extended mode can exceed 8 bit > destination ids. > > The proper way to de

Re: [Qemu-devel] [PATCH 6/8] migration, block: better select BDS for VM state loading

2016-05-07 Thread Denis V. Lunev
On 01/14/2016 02:28 PM, Denis V. Lunev wrote: This patch does 2 things: - it merges all snapshot validity checks for load_vmstate into one function - it now selects BDS to load VM state by availability of the state in the snapshot This commit is preparatory to allow to select BDS to save snap

Re: [Qemu-devel] [PATCH] Fix some typos found by codespell

2016-05-07 Thread Michael Tokarev
23.03.2016 17:59, Stefan Weil wrote: [...] FINALLY! Applied to -trivial. With the additional fix suggested by Peter, and without util/qemu-sockets.c change which has been applied meanwhile. Thank you! /mjt

Re: [Qemu-devel] [PATCH 11/52] target-m68k: allow to update flags with operation on words and bytes

2016-05-07 Thread Laurent Vivier
Le 06/05/2016 à 18:45, Richard Henderson a écrit : > On 05/04/2016 10:11 AM, Laurent Vivier wrote: >> Signed-off-by: Laurent Vivier >> --- >> target-m68k/cpu.h | 14 - >> target-m68k/helper.c| 139 >> ++-- >> target-m68k/translate.c |

Re: [Qemu-devel] [PATCH 07/52] target-m68k: add bkpt instruction

2016-05-07 Thread Laurent Vivier
Le 06/05/2016 à 18:42, Richard Henderson a écrit : > On 05/04/2016 10:11 AM, Laurent Vivier wrote: >> +INSN(bkpt, 4848, fff8, M68000); > > Do we care that this comes in with 68010 not 68000? Yes, you're right: I'm adding M68K_FEATURE_BKPT to enable it only on 68010+ > Otherwise, > >