Re: [Qemu-devel] openbios.git mirror on git.qemu.org

2015-06-11 Thread Mark Cave-Ayland
On 09/06/15 11:22, Stefan Hajnoczi wrote: > On Tue, May 19, 2015 at 09:55:12PM +0100, Peter Maydell wrote: >> On 19 May 2015 at 21:47, Mark Cave-Ayland >> wrote: >>> On 19/05/15 13:55, Andreas Färber wrote: >>> Am 19.05.2015 um 12:42 schrieb Stefan Hajnoczi: > Ping. Should we stick with

Re: [Qemu-devel] [PATCH] MIPS: exceptions handling in icount mode

2015-06-11 Thread Aurelien Jarno
On 2015-06-10 11:33, Pavel Dovgalyuk wrote: > This patch fixes exception handling in MIPS. > MIPS instructions generate several types of exceptions. > When exception is generated, it breaks the execution of the current > translation > block. Implementation of the exceptions handling in MIPS does n

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-11 Thread John Snow
On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: > From: Vladimir Sementsov-Ogievskiy > > Adds dirty-bitmaps feature to qcow2 format as specified in > docs/specs/qcow2.txt > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/

Re: [Qemu-devel] [PATCH] dma/rc4030: do multiple calls to address_space_rw when doing DMA transfers

2015-06-11 Thread Aurelien Jarno
On 2015-06-11 22:30, Hervé Poussineau wrote: > This workarounds a bug in memory management. > > To reproduce the problem, try to start the Windows NT 4.0/MIPS installer. > After loading some files, you should see a screen saying > "To set up Windows NT now, press ENTER." > However, you're welcomed

Re: [Qemu-devel] [PATCH target-arm v1 8/9] arm: xlnx-zynqmp: Preface CPU variables with "A"

2015-06-11 Thread Alistair Francis
On Thu, Jun 11, 2015 at 9:58 AM, Peter Crosthwaite wrote: > On Tue, Jun 2, 2015 at 4:57 PM, Alistair Francis > wrote: >> On Tue, Jun 2, 2015 at 4:04 AM, Peter Crosthwaite >> wrote: >>> The CPUs currently supported by zynqmp are the APU (application >>> processing unit) CPUs. There are other CPUs

Re: [Qemu-devel] [PULL 20/22] hw/arm/boot: arm_load_kernel implemented as a machine init done notifier

2015-06-11 Thread Peter Crosthwaite
On Tue, Jun 2, 2015 at 9:33 AM, Peter Maydell wrote: > From: Eric Auger > > Device tree nodes for the platform bus and its children dynamic sysbus > devices are added in a machine init done notifier. To load the dtb once, > after those latter nodes are built and before ROM freeze, the actual > ar

[Qemu-devel] [PULL v2 00/21] migration pull request

2015-06-11 Thread Juan Quintela
Hi [v2] Just rebased. [v1] Here are the pull request, it includes: - generic patches form postcopy that are reviewed (dave) - generic patches form RDMA fixes that are reviewed (dave) - patches form optional section reviewed (me) - patches for migration events reviewed (me) - fix RDMA and ipv6

[Qemu-devel] [PULL 02/21] migration: move savevm.c inside migration/

2015-06-11 Thread Juan Quintela
Now, everything is in place. Signed-off-by: Juan Quintela Reviewed-by: Eric Blake --- MAINTAINERS| 1 - Makefile.target| 4 ++-- savevm.c => migration/savevm.c | 0 trace-events | 2 +- 4 files changed, 3 insertions(+), 4 deletions(-) renam

[Qemu-devel] [PULL 08/21] migration: create savevm_state

2015-06-11 Thread Juan Quintela
This way, we will put savevm global state here, instead of lots of variables. Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert --- migration/savevm.c | 51 --- 1 file changed, 28 insertions(+), 23 deletions(-) diff --git a/migrati

[Qemu-devel] [PULL 05/21] arch_init: Clean up the duplicate variable 'len' defining in ram_load()

2015-06-11 Thread Juan Quintela
From: zhanghailiang There are two places that define 'len' variable, It's OK for compiling, but makes it difficult for reading. Remove the local one which defined in the inside 'while' loop. Signed-off-by: zhanghailiang Signed-off-by: Juan Quintela --- migration/ram.c | 1 - 1 file changed,

[Qemu-devel] [PULL 03/21] migration: Add myself to the copyright list of both files

2015-06-11 Thread Juan Quintela
If anyone feels like adding himself to the list, just sent me a patch. Signed-off-by: Juan Quintela Reviewed-by: Eric Blake --- migration/ram.c| 4 migration/savevm.c | 4 2 files changed, 8 insertions(+) diff --git a/migration/ram.c b/migration/ram.c index ff889ba..9db72a4 10064

[Qemu-devel] [PULL 04/21] migration: reduce include files

2015-06-11 Thread Juan Quintela
To make changes easier, with the copy, I maintained almost all include files. Now I remove the unnecessary ones on this patch. This compiles on linux x64 with all architectures configured, and cross-compiles for windows 32 and 64 bits. Signed-off-by: Juan Quintela Reviewed-by: Eric Blake ---

[Qemu-devel] [PULL 06/21] rdma: Fix qemu crash when IPv6 address is used for migration

2015-06-11 Thread Juan Quintela
From: Padmanabh Ratnakar Qemu crashes when IPv6 address is specified for migration and access to any RDMA uverbs device available on the system is blocked using cgroups. Fix the crash by checking the return value of ibv_open_device routine. Signed-off-by: Meghana Cheripady Signed-off-by: Padman

[Qemu-devel] [PULL 18/21] Add a protective section footer

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Badly formatted migration streams can go undetected or produce misleading errors due to a lock of checking at the end of sections. In particular a section that adds an extra 0x00 at the end causes what looks like a normal end of stream and thus doesn't produce any e

[Qemu-devel] [PULL 07/21] migration: Remove duplicated assignment of SETUP status

2015-06-11 Thread Juan Quintela
We assign the MIGRATION_STATUS_SETUP status in two places. Just in succession. Just remove the second one. Signed-off-by: Juan Quintela Reviewed-by: Eric Blake --- migration/migration.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/migration/migration.c b/migration/migration.c index 7

[Qemu-devel] [PULL 09/21] migration: Use normal VMStateDescriptions for Subsections

2015-06-11 Thread Juan Quintela
We create optional sections with this patch. But we already have optional subsections. Instead of having two mechanism that do the same, we can just generalize it. For subsections we just change: - Add a needed function to VMStateDescription - Remove VMStateSubsection (after removal of the need

[Qemu-devel] [PULL 01/21] migration: move ram stuff to migration/ram

2015-06-11 Thread Juan Quintela
For historic reasons, ram migration have been on arch_init.c. Just split it into migration/ram.c, the same that happened with block.c. There is only code movement, no changes altogether. Signed-off-by: Juan Quintela Reviewed-by: Eric Blake --- MAINTAINERS |1 - Makefile.

[Qemu-devel] [PULL 20/21] Rename RDMA structures to make destination clear

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" RDMA has two data types that are named confusingly; RDMALocalBlock (pointed to indirectly by local_ram_blocks) RDMARemoteBlock (pointed to by block in RDMAContext) RDMALocalBlocks, as the name suggests is a data strucuture that represents the RDMAable RAM Blo

[Qemu-devel] [PULL 10/21] Add qemu_get_counted_string to read a string prefixed by a count byte

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" and use it in loadvm_state and ram_load. Where ever it's used, check the return and error if it failed. Minor: ram_load was using a 257 byte array for its string, the maximum length is 255 bytes + 0 terminator, so fix to 256 Signed-off-by: Dr. David Alan G

[Qemu-devel] [PULL 13/21] Create MigrationIncomingState

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" There are currently lots of pieces of incoming migration state scattered around, and postcopy is adding more, and it seems better to try and keep it together. allocate MIS in process_incoming_migration_co Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Sh

[Qemu-devel] [PULL 17/21] Disable section footers on older machine types

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The next patch adds section footers; but we don't want to break migration compatibility so disable them on older machine types Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela --- hw/i386/pc_piix.c | 2 ++

[Qemu-devel] [PULL 11/21] Split header writing out of qemu_savevm_state_begin

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Split qemu_savevm_state_begin to: qemu_savevm_state_header That writes the initial file header. qemu_savevm_state_beginThat sets up devices and does the first device pass. Used later in postcopy. Signed-off-by: Dr. David Alan

[Qemu-devel] [PULL 14/21] Move copy out of qemu_peek_buffer

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" qemu_peek_buffer currently copies the data it reads into a buffer, however a future patch wants access to the buffer without the copy, hence rework to remove the copy to the layer above. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Shah Reviewed-by: Ju

[Qemu-devel] [PULL 21/21] Remove unneeded memset

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Michael R. Hines Signed-off-by: Juan Quintela --- migration/rdma.c | 1 - 1 file changed, 1 deletion(-) diff --git a/migration/rdma.c b/migration/rdma.c index 6c1e73f..48b3e64 100644 --- a/migration/rdma.c +++

[Qemu-devel] [PULL 12/21] qemu_ram_foreach_block: pass up error value, and down the ramblock name

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" check the return value of the function it calls and error if it's non-0 Fixup qemu_rdma_init_one_block that is the only current caller, and rdma_add_block the only function it calls using it. Pass the name of the ramblock to the function; helps in debugging. Sig

[Qemu-devel] [PULL 15/21] Move loadvm_handlers into MigrationIncomingState

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" In postcopy we need the loadvm_handlers to be used in a couple of different instances of the loadvm loop/routine, and thus it can't be local any more. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: David Gibson Reviewed-by: Juan Quintela Signed-off-by: Juan

[Qemu-devel] [PULL 16/21] Merge section header writing

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The header writing for device sections is open coded in a few places, merge it into one. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela --- migration/savevm.c | 72 +-

[Qemu-devel] [PULL 19/21] Teach analyze-migration.py about section footers

2015-06-11 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert Signed-off-by: Juan Quintela --- scripts/analyze-migration.py | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts/analyze-migration.py b/scripts/analyze-migration.py index 0c8b22f..f6894be 100755 --- a/scripts/an

Re: [Qemu-devel] [PATCH 1/2] qobject: Use 'bool' for qbool

2015-06-11 Thread Markus Armbruster
Patch looks good to me, but it made me wonder about something. Please find the question inline. Eric Blake writes: > We require a C99 compiler, so let's use 'bool' instead of 'int' > when dealing with boolean values. There are few enough clients > to fix them all in one pass. > > Signed-off-by

Re: [Qemu-devel] [PATCH V2] pci: fixes to allow booting from extra root pci buses.

2015-06-11 Thread Gerd Hoffmann
Hi, > On each boot, coreboot might decide to assign a different bus id to > the extra roots (for example, if a device with a PCI bridge is > inserted and it's bus allocation causes bus ids to shift). > Technically, coreboot could even change the order extra buses are > assigned bus ids, but does

Re: [Qemu-devel] [PATCH v5 2/4] monitor: cleanup parsing of cmd name and cmd arguments

2015-06-11 Thread Markus Armbruster
Bandan Das writes: > There's too much going on in monitor_parse_command(). > Split up the arguments parsing bits into a separate function > monitor_parse_arguments(). Let the original function check for > command validity and sub-commands if any and return data (*cmd) > that the newly introduced

<    1   2   3