[Qemu-devel] [PULL 39/44] xen_disk: Account for failed and invalid operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: e0cbb96cb0e1f86c37c7ce332efdf02b57b9d365.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/block/xen_disk.c | 23 ++- 1 file

[Qemu-devel] [PATCH for 2.5 v7 0/10] dataplane snapshot fixes

2015-11-10 Thread Denis V. Lunev
with test while /bin/true ; do virsh snapshot-create rhel7 sleep 10 virsh snapshot-delete rhel7 --current done with enabled iothreads on a running VM leads to a lot of troubles: hangs, asserts, errors. Anyway, I think that the construction like

[Qemu-devel] [PULL 06/57] qemu_ram_block_by_name

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add a function to find a RAMBlock by name; use it in two of the places that already open code that loop; we've got another use later in postcopy. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

[Qemu-devel] [PULL 38/44] virtio-blk: Account for failed and invalid operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 4f623ce52c9d673d35a043fc2959526b41b685c6.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/block/virtio-blk.c | 4 +++- 1 file changed, 3 insertions(+),

[Qemu-devel] [PULL 11/57] ram_debug_dump_bitmap: Dump a migration bitmap as text

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Useful for debugging the migration bitmap and other bitmaps of the same format (including the sentmap in postcopy). The bitmap is printed to stderr. Lines that are all the expected value are excluded so the output can be quite compact for many

[Qemu-devel] [PULL 33/57] Add qemu_savevm_state_complete_postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add qemu_savevm_state_complete_postcopy to complement qemu_savevm_state_complete_precopy together with a new save_live_complete_postcopy method on devices. The save_live_complete_precopy method is called on all devices during a precopy

[Qemu-devel] [PULL 36/57] postcopy: Incoming initialisation

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert Reviewed-by: David Gibson Reviewed-by: Amit Shah Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela

[Qemu-devel] [PULL 27/57] Modify save_live_pending for postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Modify save_live_pending to return separate postcopiable and non-postcopiable counts. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela Reviewed-by: Amit Shah

Re: [Qemu-devel] [PATCH] qga: fix append file open modes for win32

2015-11-10 Thread Michael Roth
Quoting Kirk Allan (2015-11-09 15:49:05) > For append file open modes, use FILE_APPEND_DATA for the desired access for > writing at the end of the file. > > Signed-off-by: Kirk Allan > --- > qga/commands-win32.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) >

Re: [Qemu-devel] [PATCH 03/16] tlb_set_page_with_attrs: Take argument specifying AddressSpace to use

2015-11-10 Thread Peter Maydell
On 9 November 2015 at 10:49, Peter Maydell wrote: > On 9 November 2015 at 10:44, Paolo Bonzini wrote: >> >> >> On 05/11/2015 19:15, Peter Maydell wrote: >>> Add an argument to tlb_set_page_with_attrs which allows the target CPU code >>> to tell the

[Qemu-devel] [PATCH 0/2] Clean up 32bit compilation for Migration

2015-11-10 Thread Juan Quintela
Hi We had two warnings on 32bit targets. With this change I am able to compile with -Werror on: - linux 64bit - linux 32bit - windows 32bit (cross-compile with Fedora mingw64) - windows 64bit (cross-compile with Fedora mingw64) I am lying in this one, really my crosscompiler for windows 64 bit

[Qemu-devel] [PATCH 2/2] migration: Make 32bit linux compile with RDMA

2015-11-10 Thread Juan Quintela
Rest of the file already use that trick. 64bit offsets make no sense in 32bit archs, but that is ram_addr_t for you. Signed-off-by: Juan Quintela --- migration/rdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/migration/rdma.c b/migration/rdma.c

[Qemu-devel] [PATCH 1/2] migration: print ram_addr_t as RAM_ADDR_FMT not %zx

2015-11-10 Thread Juan Quintela
Not all the wold is 64bits (yet). Signed-off-by: Juan Quintela --- migration/ram.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index 62cf42b..d8d5a50 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -1184,7

Re: [Qemu-devel] [PATCH v3 2/3] target-i386: calculate vcpu's TSC rate to be migrated

2015-11-10 Thread Haozhong Zhang
On 11/09/15 14:01, Eduardo Habkost wrote: > On Mon, Nov 09, 2015 at 08:33:55AM +0800, haozhong.zh...@intel.com wrote: > > On 11/06/15 13:12, Eduardo Habkost wrote: > > > On Fri, Nov 06, 2015 at 10:32:24AM +0800, haozhong.zh...@intel.com wrote: > > > > On 11/05/15 14:05, Eduardo Habkost wrote: > >

[Qemu-devel] [PULL v2 32/40] qemu-io: Check for trailing chars

2015-11-10 Thread Kevin Wolf
From: John Snow Make sure there's not trailing garbage, e.g. "64k-whatever-i-want-here" Reported-by: Max Reitz Signed-off-by: John Snow Reviewed-by: Eric Blake Reviewed-by: Kevin Wolf Signed-off-by:

[Qemu-devel] [PULL 00/57] Migration pull (take 2)

2015-11-10 Thread Juan Quintela
-tracking branch 'remotes/armbru/tags/pull-qapi-2015-11-10' into staging (2015-11-10 09:39:24 +) are available in the git repository at: git://github.com/juanquintela/qemu.git tags/migration/20151110 for you to fetch changes up to 15b3b8eaae8dbcc903bb164311ea0066c77536a7: migration

[Qemu-devel] [PATCH 03/10] snapshot: create bdrv_all_delete_snapshot helper

2015-11-10 Thread Denis V. Lunev
to delete snapshots from all loaded block drivers. The patch also ensures proper locking. Signed-off-by: Denis V. Lunev CC: Juan Quintela CC: Stefan Hajnoczi CC: Kevin Wolf --- block/snapshot.c | 22

[Qemu-devel] [PULL 16/57] Return path: Open a return path on QEMUFile for sockets

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Postcopy needs a method to send messages from the destination back to the source, this is the 'return path'. Wire it up for 'socket' QEMUFile's. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

[Qemu-devel] What to do about QAPI naming convention violations (was: [PATCH RFC 3/5] qapi: Use common name mangling for enumeration constants)

2015-11-10 Thread Markus Armbruster
Markus Armbruster writes: > Markus Armbruster writes: > >> Eric Blake writes: >> >>> On 11/05/2015 09:01 AM, Daniel P. Berrange wrote: On Thu, Nov 05, 2015 at 04:30:00PM +0100, Markus Armbruster wrote: > QAPI names needn't be

[Qemu-devel] [PULL 43/57] postcopy_ram.c: place_page and helpers

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" postcopy_place_page (etc) provide a way for postcopy to place a page into guests memory atomically (using the copy ioctl on the ufd). Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Shah

[Qemu-devel] [PULL 30/57] migration_completion: Take current state

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Soon we'll be in either ACTIVE or POSTCOPY_ACTIVE when we complete migration, and we need to know which we expect to be in to change state safely. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH v3 0/2] Fix compilation of netmap backend

2015-11-10 Thread Markus Armbruster
Vincenzo Maffione writes: > This patch series adds some fixes to the netmap net backend. It contains > two changes: > (1) Fix compilation issue of netmap.c introduced by the reorganization > of struct NetClientOptions > (2) Address the FIXME comment that was

[Qemu-devel] [PATCH 3/3] disas/arm: avoid clang shifting negative signed warning

2015-11-10 Thread Stefan Hajnoczi
clang 3.7.0 on x86_64 warns about the following: disas/arm.c:1782:17: warning: shifting a negative signed value is undefined [-Wshift-negative-value] imm |= (-1 << 7); ~~ ^ Note that this patch preserves the tab indent in this source file because the surrounding code still

[Qemu-devel] [PULL 44/44] block: Update copyright of the accounting code

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 80a2278e3ec2dafd5daab20a7cb2d6a9b83371e4.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- block/accounting.c | 1 + include/block/accounting.h | 1

[Qemu-devel] [PULL 03/57] Move configuration section writing

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The vmstate_configuration is currently written in 'qemu_savevm_state_begin', move it to 'qemu_savevm_state_header' since it's got a hard requirement that it must be the 1st thing after the header. (In postcopy some 'command' sections get sent

[Qemu-devel] [PULL 05/57] qemu_ram_block_from_host

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Postcopy sends RAMBlock names and offsets over the wire (since it can't rely on the order of ramaddr being the same), and it starts out with HVA fault addresses from the kernel. qemu_ram_block_from_host translates a HVA into a RAMBlock, an

[Qemu-devel] [PULL 20/57] Return path: Send responses from destination to source

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add migrate_send_rp_message to send a message from destination to source along the return path. (It uses a mutex to let it be called from multiple threads) Add migrate_send_rp_shut to send a 'shut' message to indicate the destination is

[Qemu-devel] Contribute to abstracting device address allocation

2015-11-10 Thread priyanshu jain
Hello, I am undergraduate computer science student in india .I am new to opensource and i want to contribute to project abstracting device address allocation in QEMU . I have good knowledge of c . please suggest me how i contribute to this. Regards Priyanshu Jain

[Qemu-devel] [PULL 25/57] Add wrappers and handlers for sending/receiving the postcopy-ram migration messages.

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The state of the postcopy process is managed via a series of messages; * Add wrappers and handlers for sending/receiving these messages * Add state variable that track the current state of postcopy Signed-off-by: Dr. David Alan Gilbert

[Qemu-devel] [PULL 56/57] Inhibit ballooning during postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Postcopy detects accesses to pages that haven't been transferred yet using userfaultfd, and it causes exceptions on pages that are 'not present'. Ballooning also causes pages to be marked as 'not present' when the guest inflates the balloon.

[Qemu-devel] [PULL 51/57] Start up a postcopy/listener thread ready for incoming page data

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The loading of a device state (during postcopy) may access guest memory that's still on the source machine and thus might need a page fill; split off a separate thread that handles the incoming page data so that the original incoming migration

Re: [Qemu-devel] [PATCH v4 00/21] Extended I/O accounting

2015-11-10 Thread Stefan Hajnoczi
On Wed, Oct 28, 2015 at 05:32:57PM +0200, Alberto Garcia wrote: > Here's v4 of the series that implements extended I/O accounting for > block devices. > > Since part of Max's BlockBackend series has already been merged, this > series can now be applied cleanly on top of the master branch without

[Qemu-devel] [PULL 57/57] migration: qemu_savevm_state_cleanup becomes mandatory operation

2015-11-10 Thread Juan Quintela
From: "Denis V. Lunev" since commit commit 94f5a43704129ca4995aa3385303c5ae225bde42 Author: Liang Li Date: Mon Nov 2 15:37:00 2015 +0800 migration: defer migration_end & blk_mig_cleanup when actual .cleanup callbacks calling was removed

[Qemu-devel] [PATCH 1/3] monitor: avoid clang shifting negative signed warning

2015-11-10 Thread Stefan Hajnoczi
clang 3.7.0 on x86_64 warns about the following: target-i386/monitor.c:38:22: warning: shifting a negative signed value is undefined [-Wshift-negative-value] addr |= -1LL << 48; ^ Signed-off-by: Stefan Hajnoczi --- target-i386/monitor.c | 2

Re: [Qemu-devel] [PATCH] kvm-all: PAGE_SIZE should be real host page size

2015-11-10 Thread Peter Maydell
On 10 November 2015 at 16:59, Andrew Jones wrote: > On Tue, Nov 10, 2015 at 04:29:31PM +, Peter Maydell wrote: >> On 10 November 2015 at 00:23, Andrew Jones wrote: >> > -/* KVM uses PAGE_SIZE in its definition of COALESCED_MMIO_MAX */ >> > -#define

[Qemu-devel] [PATCH 01/10] snapshot: create helper to test that block drivers supports snapshots

2015-11-10 Thread Denis V. Lunev
The patch enforces proper locking for this operation. Signed-off-by: Denis V. Lunev Reviewed-by: Greg Kurz CC: Juan Quintela CC: Stefan Hajnoczi CC: Kevin Wolf --- block/snapshot.c

[Qemu-devel] [PULL 36/44] iotests: Add test for the block device statistics

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 0fb8501bbf3666b3d5d3f67fa899729c88f21baf.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/136 | 349

[Qemu-devel] [PULL 25/44] ide: Account for write operations correctly

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Reviewed-by: Stefan Hajnoczi Message-id: 2e71323c0875c2b66a8ae9545e0c013af8d4.1446044837.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/ide/core.c

[Qemu-devel] [PATCH 08/10] migration: reorder processing in hmp_savevm

2015-11-10 Thread Denis V. Lunev
State deletion can be performed on running VM which reduces VM downtime This approach looks a bit more natural. Signed-off-by: Denis V. Lunev CC: Juan Quintela --- migration/savevm.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-)

[Qemu-devel] [PULL 33/44] block: New option to define the intervals for collecting I/O statistics

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia The BlockAcctStats structure contains a list of BlockAcctTimedStats. Each one of these collects statistics about the minimum, maximum and average latencies of all I/O operations in a certain interval of time. This patch adds a new "stats-intervals" option

[Qemu-devel] [PULL 35/57] migration_completion: Take current state

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Soon we'll be in either ACTIVE or POSTCOPY_ACTIVE when we complete migration, and we need to know which we expect to be in to change state safely. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

[Qemu-devel] [PULL 31/57] MIGRATION_STATUS_POSTCOPY_ACTIVE: Add new migration state

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" 'MIGRATION_STATUS_POSTCOPY_ACTIVE' is entered after migrate_start_postcopy 'migration_in_postcopy' is provided for other sections to know if they're in postcopy. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: David

Re: [Qemu-devel] [Qemu-block] [PULL v2 39/40] iotests: Add tests for the x-blockdev-del command

2015-11-10 Thread Kevin Wolf
Am 10.11.2015 um 15:59 hat Stefan Hajnoczi geschrieben: > On Tue, Nov 10, 2015 at 2:09 PM, Kevin Wolf wrote: > > From: Alberto Garcia > > > > Signed-off-by: Alberto Garcia > > Message-id: > >

Re: [Qemu-devel] [PATCH] kvm-all: PAGE_SIZE should be real host page size

2015-11-10 Thread Andrew Jones
On Tue, Nov 10, 2015 at 04:41:16PM +0100, Paolo Bonzini wrote: > > > On 10/11/2015 01:23, Andrew Jones wrote: > > Just noticed this while grepping TARGET_PAGE_SIZE for an unrelated > > reason. I didn't use qemu_real_host_page_size as kvm_set_phys_mem() > > does, because we'd need to make sure

[Qemu-devel] [PATCH 0/3] fix clang negative signed bit shift warning

2015-11-10 Thread Stefan Hajnoczi
LLVM's clang 3.7.0 compile warns about bit shifting negative numbers because the result is undefined. This series includes 3 small fixes to appease clang. Stefan Hajnoczi (3): monitor: avoid clang shifting negative signed warning tpm: avoid clang shifting negative signed warning disas/arm:

Re: [Qemu-devel] [PATCH v13 3/3] block/gluster: add support for multiple gluster servers

2015-11-10 Thread Eric Blake
On 11/10/2015 02:09 AM, Prasanna Kumar Kalever wrote: > This patch adds a way to specify multiple volfile servers to the gluster > block backend of QEMU with tcp|rdma transport types and their port numbers. > [...] > 2. > 'json:{"driver":"qcow2","file":{"driver":"gluster","volume":"testvol", >

[Qemu-devel] [PULL 24/44] xen_disk: Account for flush operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Currently both BLKIF_OP_WRITE and BLKIF_OP_FLUSH_DISKCACHE are being accounted as write operations. Signed-off-by: Alberto Garcia Reviewed-by: Stefan Hajnoczi Message-id:

[Qemu-devel] [PATCH 05/10] snapshot: create bdrv_all_find_snapshot helper

2015-11-10 Thread Denis V. Lunev
to check that snapshot is available for all loaded block drivers. The ability to switch to snapshot is verified separately using bdrv_all_can_snapshot. The patch also ensures proper locking. Signed-off-by: Denis V. Lunev CC: Juan Quintela CC: Stefan

[Qemu-devel] [PULL 14/57] Rename save_live_complete to save_live_complete_precopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" In postcopy we're going to need to perform the complete phase for postcopiable devices at a different point, start out by renaming all of the 'complete's to make the difference obvious. Signed-off-by: Dr. David Alan Gilbert

[Qemu-devel] [PULL 37/57] postcopy: ram_enable_notify to switch on userfault

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Mark the area of RAM as 'userfault' Start up a fault-thread to handle any userfaults we might receive from it (to be filled in later) Signed-off-by: Dr. David Alan Gilbert Reviewed-by: David Gibson

[Qemu-devel] [PULL 44/57] Postcopy: Use helpers to map pages during migration

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" In postcopy, the destination guest is running at the same time as it's receiving pages; as we receive new pages we must put them into the guests address space atomically to avoid a running CPU accessing a partially written page. Use the

Re: [Qemu-devel] [PATCH for-2.5 v3 0/2] Fix compilation of netmap backend

2015-11-10 Thread Eric Blake
On 11/10/2015 02:47 AM, Vincenzo Maffione wrote: > This patch series adds some fixes to the netmap net backend. It contains > two changes: > (1) Fix compilation issue of netmap.c introduced by the reorganization > of struct NetClientOptions > (2) Address the FIXME comment that was

[Qemu-devel] [PULL 40/57] Page request: Add MIG_RP_MSG_REQ_PAGES reverse command

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add MIG_RP_MSG_REQ_PAGES command on Return path for the postcopy destination to request a page from the source. Two versions exist: MIG_RP_MSG_REQ_PAGES_ID that includes a RAMBlock name and start/len MIG_RP_MSG_REQ_PAGES that just has

[Qemu-devel] [PATCH] nand: fix address overflow

2015-11-10 Thread Rabin Vincent
The shifts of the address mask and value shift beyond 32 bits when there are 5 address cycles. Signed-off-by: Rabin Vincent --- hw/block/nand.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/block/nand.c b/hw/block/nand.c index

[Qemu-devel] [PULL 55/57] Disable mlock around incoming postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Userfault doesn't work with mlock; mlock is designed to nail down pages so they don't move, userfault is designed to tell you when they're not there. munlock the pages we userfault protect before postcopy. mlock everything again at the end if

[Qemu-devel] [PULL 42/57] Page request: Consume pages off the post-copy queue

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" When transmitting RAM pages, consume pages that have been queued by MIG_RPCOMM_REQPAGE commands and send them ahead of normal page scanning. Note: a) After a queued page the linear walk carries on from after the unqueued page; there is a

[Qemu-devel] [PULL 50/57] Postcopy; Handle userfault requests

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" userfaultfd is a Linux syscall that gives an fd that receives a stream of notifications of accesses to pages registered with it and allows the program to acknowledge those stalls and tell the accessing thread to carry on. We convert the

Re: [Qemu-devel] [PATCH v10 24/30] qapi: Factor out QAPISchemaObjectType.check_clash()

2015-11-10 Thread Markus Armbruster
Eric Blake writes: > On 11/10/2015 02:15 AM, Markus Armbruster wrote: > >>> On the other hand, we've been arguing that check() should populate >>> everything after construction prior to anything else being run; and not >>> running Variant.type.check() during Variants.check()

[Qemu-devel] [PULL v2 for-2.5] Block pull request

2015-11-10 Thread Stefan Hajnoczi
v2: * Rebase onto Kevin's block pull request The following changes since commit c400bddb916268394e352f82809eb4728424a5b1: Merge remote-tracking branch 'mreitz/tags/pull-block-for-kevin-2015-11-10' into queue-block (2015-11-10 14:59:26 +0100) are available in the git repository at:

Re: [Qemu-devel] [PATCH] kvm-all: PAGE_SIZE should be real host page size

2015-11-10 Thread Paolo Bonzini
On 10/11/2015 01:23, Andrew Jones wrote: > Just noticed this while grepping TARGET_PAGE_SIZE for an unrelated > reason. I didn't use qemu_real_host_page_size as kvm_set_phys_mem() > does, because we'd need to make sure page_size_init() has run first. > > Signed-off-by: Andrew Jones

Re: [Qemu-devel] [PATCH 0/3] fix clang negative signed bit shift warning

2015-11-10 Thread Peter Maydell
On 10 November 2015 at 15:57, Stefan Hajnoczi wrote: > LLVM's clang 3.7.0 compile warns about bit shifting negative numbers because > the result is undefined. This series includes 3 small fixes to appease clang. > > Stefan Hajnoczi (3): > monitor: avoid clang shifting

[Qemu-devel] [PATCH] iotests: Check for quorum support in test 139

2015-11-10 Thread Alberto Garcia
The quorum driver is always built in, but it is disabled during run-time if there's no SHA256 support available (see commit e94867e). This patch skips the quorum test in iotest 139 in that case. Signed-off-by: Alberto Garcia --- tests/qemu-iotests/139 | 2 ++ 1 file changed,

Re: [Qemu-devel] [PATCH] kvm-all: PAGE_SIZE should be real host page size

2015-11-10 Thread Peter Maydell
On 10 November 2015 at 00:23, Andrew Jones wrote: > Just noticed this while grepping TARGET_PAGE_SIZE for an unrelated > reason. I didn't use qemu_real_host_page_size as kvm_set_phys_mem() > does, because we'd need to make sure page_size_init() has run first. > >

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

2015-11-10 Thread Peter Maydell
0' > into staging (2015-11-10 09:39:24 +) > > are available in the git repository at: > > > git://git.linaro.org/people/pmaydell/qemu-arm.git > tags/pull-target-arm-20151110 > > for you to fetch changes up to 577bf808958d06497928c639efaa473bf8c5e099: > > target-arm

[Qemu-devel] [PULL 08/44] block: Introduce BlockDriver.bdrv_drain callback

2015-11-10 Thread Stefan Hajnoczi
From: Fam Zheng Drivers can have internal request sources that generate IO, like the need_check_timer in QED. Since we want quiesced periods that contain nested event loops in block layer, we need to have a way to disable such event sources. Block drivers must implement the

[Qemu-devel] [PULL 22/57] Return path: Source handling of return path

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Open a return path, and handle messages that are received upon it. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela ---

[Qemu-devel] [PULL 10/57] Add QEMU_MADV_NOHUGEPAGE

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add QEMU_MADV_NOHUGEPAGE as an OS-independent version of MADV_NOHUGEPAGE. We include sys/mman.h before making the test to ensure that we pick up the system defines. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan

[Qemu-devel] [PULL 19/57] Return path: Control commands

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add two src->dest commands: * OPEN_RETURN_PATH - To request that the destination open the return path * PING - Request an acknowledge from the destination Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Shah

[Qemu-devel] [PULL 46/57] Don't sync dirty bitmaps in postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Once we're in postcopy the source processors are stopped and memory shouldn't change any more, so there's no need to look at the dirty map. There are two notes to this: 1) If we do resync and a page had changed then the page would get

[Qemu-devel] [PULL 24/57] Add migration-capability boolean for postcopy-ram.

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The 'postcopy ram' capability allows postcopy migration of RAM; note that the migration starts off in precopy mode until postcopy mode is triggered (see the migrate_start_postcopy patch later in the series). Signed-off-by: Dr. David Alan

[Qemu-devel] [PULL 48/57] Host page!=target page: Cleanup bitmaps

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Prior to the start of postcopy, ensure that everything that will be transferred later is a whole host-page in size. This is accomplished by discarding partially transferred host pages and marking any that are partially dirty as fully dirty.

[Qemu-devel] [PULL 49/57] Round up RAMBlock sizes to host page sizes

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" RAMBlocks that are not a multiple of host pages in length cause problems for postcopy (I've seen an ACPI table on aarch64 be 5k in length - i.e. 5x target-page), so round RAMBlock sizes up to a host-page. This potentially breaks migration

[Qemu-devel] [PULL 53/57] Postcopy: Mark nohugepage before discard

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Prior to servicing userfault requests we must ensure we've not got huge pages in the area that might include non-transferred memory, since a hugepage could incorrectly mark the whole huge page as present. We mark the area as non-huge page

Re: [Qemu-devel] [PATCH RESEND v2 0/3] pc: Set hw_version on all machine classes

2015-11-10 Thread Paolo Bonzini
On 09/11/2015 21:46, Michael S. Tsirkin wrote: > On Mon, Nov 02, 2015 at 12:55:07PM +0100, Paolo Bonzini wrote: >> On 30/10/2015 20:36, Eduardo Habkost wrote: >>> In 2012, QEMU had a bug where it exposed QEMU version information >>> to the guest, meaning a QEMU upgrade would expose different >>>

Re: [Qemu-devel] [PATCH 2/2] migration: Make 32bit linux compile with RDMA

2015-11-10 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > Rest of the file already use that trick. 64bit offsets make no sense in > 32bit archs, but that is ram_addr_t for you. > > Signed-off-by: Juan Quintela > --- > migration/rdma.c | 2 +- > 1 file changed, 1 insertion(+), 1

[Qemu-devel] [PULL 37/44] nvme: Account for failed and invalid operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 678dc67da229759d404b44f7cc2bf5ed8bf8ad14.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/block/nvme.c | 11 --- 1 file changed, 8

[Qemu-devel] [PULL 43/44] scsi-disk: Account for failed operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 0ead7b0e59c22926e033ca12725e3a31985ec46b.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/scsi/scsi-disk.c | 46

[Qemu-devel] [PATCH 02/10] snapshot: return error code from bdrv_snapshot_delete_by_id_or_name

2015-11-10 Thread Denis V. Lunev
this will make code better in the next patch Signed-off-by: Denis V. Lunev CC: Juan Quintela CC: Stefan Hajnoczi CC: Kevin Wolf --- block/snapshot.c | 7 --- include/block/snapshot.h | 6 +++--- 2 files

[Qemu-devel] [PULL 01/57] Add postcopy documentation

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Shah Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela --- docs/migration.txt | 191

[Qemu-devel] [PULL 28/44] block: Add idle_time_ns to BlockDeviceStats

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia This patch adds the new field 'idle_time_ns' to the BlockDeviceStats structure, indicating the time that has passed since the previous I/O operation. It also adds the block_acct_idle_time_ns() call, to ensure that all references to the clock type used for

[Qemu-devel] [PATCH 10/10] migration: normalize locking in migration/savevm.c

2015-11-10 Thread Denis V. Lunev
basically all bdrv_* operations must be called under aio_context_acquire except ones with bdrv_all prefix. Signed-off-by: Denis V. Lunev CC: Juan Quintela CC: Stefan Hajnoczi CC: Kevin Wolf --- migration/savevm.c |

[Qemu-devel] [PULL 41/44] ide: Account for failed and invalid operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: bf4d6c9c563877e699b0bf42e7eaf8b096c4a35e.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/ide/core.c | 10 -- 1 file changed, 8 insertions(+),

[Qemu-devel] [PULL 13/57] migrate_init: Call from savevm

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Suspend to file is very much like a migrate, and it makes life easier if we have the Migration state available, so initialise it in the savevm.c code for suspending. Signed-off-by: Dr. David Alan Gilbert Reviewed-by:

[Qemu-devel] [PULL 04/57] Move page_size_init earlier

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The HOST_PAGE_ALIGN macros don't work until the page size variables have been set up; later in postcopy I use those macros in the RAM code, and it can be triggered using -object. Fix this by initialising page_size_init() earlier - it's

[Qemu-devel] [PULL 29/57] migrate_start_postcopy: Command to trigger transition to postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Once postcopy is enabled (with migrate_set_capability), the migration will still start on precopy mode. To cause a transition into postcopy the: migrate_start_postcopy command must be issued. Postcopy will start sometime after this (when

[Qemu-devel] [PULL 38/57] Postcopy: Postcopy startup in migration thread

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Rework the migration thread to setup and start postcopy. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit Shah Reviewed-by: Juan Quintela Signed-off-by: Juan Quintela

[Qemu-devel] [PULL 54/57] End of migration for postcopy

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Tweak the end of migration cleanup; we don't want to close stuff down at the end of the main stream, since the postcopy is still sending pages on the other thread. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan

Re: [Qemu-devel] [Qemu-block] [PULL v2 39/40] iotests: Add tests for the x-blockdev-del command

2015-11-10 Thread Stefan Hajnoczi
On Tue, Nov 10, 2015 at 2:09 PM, Kevin Wolf wrote: > From: Alberto Garcia > > Signed-off-by: Alberto Garcia > Message-id: > 57c3b0d4d0c73ddadd19e5bded9492c359cc4568.1446475331.git.be...@igalia.com > Reviewed-by: Max Reitz

[Qemu-devel] [PULL 34/57] Postcopy: Maintain unsentmap

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Maintain an 'unsentmap' of pages that have yet to be sent. This is used in the following patches to discard some set of the pages already sent as we enter postcopy mode. Signed-off-by: Dr. David Alan Gilbert Reviewed-by:

Re: [Qemu-devel] [PATCH 0/3] block/gluster: add support for multiple gluster servers

2015-11-10 Thread Eric Blake
On 11/10/2015 02:09 AM, Prasanna Kumar Kalever wrote: > This release is rebased on qemu master branch. > In this series of patches 1/3 and 2/3 are unchanged. It's still nice to send the _entire_ series with v13 in the subject line (cover letter included), rather than mixing and matching (no

[Qemu-devel] [PULL 09/44] qed: Implement .bdrv_drain

2015-11-10 Thread Stefan Hajnoczi
From: Fam Zheng The "need_check_timer" is used to clear the "NEED_CHECK" flag in the image header after a grace period once metadata update has finished. In compliance to the bdrv_drain semantics we should make sure it remains deleted once .bdrv_drain is called. We cannot reuse

[Qemu-devel] [PULL 40/44] atapi: Account for failed and invalid operations

2015-11-10 Thread Stefan Hajnoczi
From: Alberto Garcia Signed-off-by: Alberto Garcia Message-id: 59dee4e2921b0c79d41c49b67dfb93d32db9f7f9.1446044838.git.be...@igalia.com Signed-off-by: Stefan Hajnoczi --- hw/ide/atapi.c | 31 +++ 1 file

[Qemu-devel] [PATCH 06/10] migration: drop find_vmstate_bs check in hmp_delvm

2015-11-10 Thread Denis V. Lunev
There is no much sense to do the check and write warning. Signed-off-by: Denis V. Lunev CC: Juan Quintela --- migration/savevm.c | 5 - 1 file changed, 5 deletions(-) diff --git a/migration/savevm.c b/migration/savevm.c index 90aa565..4c652f3 100644

[Qemu-devel] [PULL 15/57] Add Linux userfaultfd.h header

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Postcopy uses the userfaultfd.h feature in the Linux kernel; include the header. (In early versions of the patch series we had this, and then we dropped this by only including it if the kernel headers defined the syscall number; however

[Qemu-devel] [PATCH 07/10] snapshot: create bdrv_all_create_snapshot helper

2015-11-10 Thread Denis V. Lunev
to create snapshot for all loaded block drivers. The patch also ensures proper locking. Signed-off-by: Denis V. Lunev CC: Juan Quintela CC: Stefan Hajnoczi CC: Kevin Wolf --- block/snapshot.c | 26

[Qemu-devel] [PULL 28/57] postcopy: OS support test

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Provide a check to see if the OS we're running on has all the bits needed for postcopy. Creates postcopy-ram.c which will get most of the other helpers we need. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Amit

[Qemu-devel] [PULL 09/57] Add wrapper for setting blocking status on a QEMUFile

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" Add a wrapper to change the blocking status on a QEMUFile rather than having to use qemu_set_block(qemu_get_fd(f)); it seems best to avoid exposing the fd since not all QEMUFile's really have one. With this wrapper we could move the

[Qemu-devel] [PULL 26/57] MIG_CMD_PACKAGED: Send a packaged chunk of migration stream

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" MIG_CMD_PACKAGED is a migration command that wraps a chunk of migration stream inside a package whose length can be determined purely by reading its header. The destination guarantees that the whole MIG_CMD_PACKAGED is read off the stream

[Qemu-devel] [PULL 02/57] Provide runtime Target page information

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The migration code generally is built target-independent, however there are a few places where knowing the target page size would avoid artificially moving stuff into migration/ram.c. Provide 'qemu_target_page_bits()' that returns

[Qemu-devel] [PULL 12/57] ram_load: Factor out host_from_stream_offset call and check

2015-11-10 Thread Juan Quintela
From: "Dr. David Alan Gilbert" The main RAM load loop has a call to host_from_stream_offset for each page type that actually loads data with the same test; factor it out before the switch. The host = NULL is to silence a bogus gcc warning of an unitialised in the

  1   2   3   4   5   6   >