[Qemu-devel] [PULL 0/4] migration queue

[Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

Hi

Anthony,  this inculed the iovec patches that help savevm performance by paolo.  Please pull.

Later,  Juan.


The following changes since commit 47b5264eb3e1cd2825e48d28fd0d1b239ed53974:

  Merge remote-tracking branch 'bonzini/hw-dirs' into staging (2013-04-08 13:12:33 -0500)

are available in the git repository at:


  git://github.com/juanquintela/qemu.git migration.next

for you to fetch changes up to 4d1172472cdf28a444321ca8b165ce7326eb919e:

  migration: simplify writev vs. non-writev logic (2013-04-09 15:00:13 +0200)

----------------------------------------------------------------
Paolo Bonzini (4):
      migration: set f->is_write and flush in add_to_iovec
      migration: use a single I/O operation when writev_buffer is not defined
      migration: drop is_write complications
      migration: simplify writev vs. non-writev logic

 savevm.c | 104 ++++++++++++++++++++++++++-------------------------------------
 1 file changed, 42 insertions(+), 62 deletions(-)

[Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

Hi

Please pull, this include:

- make rdma as stable (patch has been reviewed since long ago)(mrhines)
- less_equal means less_equal not great_or_eual: (david and brown paperbag for me)
- fix two XBZRLE corruptions
- fwrite(): we are back to v2 of the patch. (me)


The following changes since commit 0459650d94d18218808fcabc8c3227d2ee99af39:

  Merge remote-tracking branch 'remotes/mdroth/qga-pull-2014-02-24' into staging (2014-02-25 12:30:01 +0000)

are available in the git repository at:


  git://github.com/juanquintela/qemu.git tags/migration/20140225

for you to fetch changes up to 41310c68781d742fa9bbfd5fcb1df9b7f23f5759:

  rdma: rename 'x-rdma' => 'rdma' (2014-02-25 14:30:28 +0100)

----------------------------------------------------------------
migration/next for 20140225

----------------------------------------------------------------
Dr. David Alan Gilbert (2):
      Fix vmstate_info_int32_le comparison/assign
      Fix two XBZRLE corruption issues

Juan Quintela (1):
      qemu_file: use fwrite() correctly

Michael R. Hines (1):
      rdma: rename 'x-rdma' => 'rdma'

 arch_init.c                    | 64 ++++++++++++++++++++++++++++++++----------
 docs/rdma.txt                  | 24 +++++++---------
 include/migration/page_cache.h |  2 +-
 migration-rdma.c               |  2 +-
 migration.c                    |  6 ++--
 page_cache.c                   |  2 +-
 qapi-schema.json               |  7 ++---
 qemu-file.c                    |  9 +++++-
 vmstate.c                      | 15 +++++-----
 9 files changed, 84 insertions(+), 47 deletions(-)

Re: [Qemu-devel] [PULL 0/4] migration queue

[Peter Maydell]

On 25 February 2014 14:07, Juan Quintela <quintela@redhat.com> wrote:
> Hi
>
> Please pull, this include:
>
> - make rdma as stable (patch has been reviewed since long ago)(mrhines)
> - less_equal means less_equal not great_or_eual: (david and brown paperbag for me)
> - fix two XBZRLE corruptions
> - fwrite(): we are back to v2 of the patch. (me)
>
>
> The following changes since commit 0459650d94d18218808fcabc8c3227d2ee99af39:
>
>   Merge remote-tracking branch 'remotes/mdroth/qga-pull-2014-02-24' into staging (2014-02-25 12:30:01 +0000)
>
> are available in the git repository at:
>
>
>   git://github.com/juanquintela/qemu.git tags/migration/20140225
>
> for you to fetch changes up to 41310c68781d742fa9bbfd5fcb1df9b7f23f5759:
>
>   rdma: rename 'x-rdma' => 'rdma' (2014-02-25 14:30:28 +0100)
>

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

The following changes since commit f53f3d0a00b6df39ce8dfca942608e5b6a9a4f71:

  Merge remote-tracking branch 'remotes/kvaneesh/for-upstream' into staging (2014-03-08 12:38:43 +0000)

are available in the git repository at:


  git://github.com/juanquintela/qemu.git tags/migration/20140308-1

for you to fetch changes up to 464400f6a5583eafb466595add435a3a33ea980f:

  migration: extend section_start/end traces (2014-03-08 22:22:34 +0100)

----------------------------------------------------------------
migration/next for 20140308

----------------------------------------------------------------
Alexey Kardashevskiy (2):
      vl: add system_wakeup_request tracepoint
      migration: extend section_start/end traces

Gonglei (1):
      XBZRLE: Fix qemu crash when resize the xbzrle cache

Markus Armbruster (1):
      qemu_file: Fix mismerge of "use fwrite() correctly"

 arch_init.c  | 52 +++++++++++++++++++++++++++++++++++++++++++++++++---
 qemu-file.c  |  2 +-
 savevm.c     | 12 ++++++------
 trace-events |  5 +++--
 vl.c         |  2 ++
 5 files changed, 61 insertions(+), 12 deletions(-)

Re: [Qemu-devel] [PULL 0/4] migration queue

[Peter Maydell]

On 8 March 2014 21:32, Juan Quintela <quintela@redhat.com> wrote:
> The following changes since commit f53f3d0a00b6df39ce8dfca942608e5b6a9a4f71:
>
>   Merge remote-tracking branch 'remotes/kvaneesh/for-upstream' into staging (2014-03-08 12:38:43 +0000)
>
> are available in the git repository at:
>
>
>   git://github.com/juanquintela/qemu.git tags/migration/20140308-1
>
> for you to fetch changes up to 464400f6a5583eafb466595add435a3a33ea980f:
>
>   migration: extend section_start/end traces (2014-03-08 22:22:34 +0100)
>
> ----------------------------------------------------------------
> migration/next for 20140308

Applied, thanks.

-- PMM

[Qemu-devel] RDMA upstream moved to stable status - will proceed formally with libvirt patchset and more FT review

[Michael R. Hines]

[-- Attachment #1: Type: text/plain, Size: 2256 bytes --]

Hi,

RDMA has finally moved out of 'experimental' status,
which is a fundamental part of the FT implementation
that is currently in review.

I can finally proceed to pushing this through the libvirt community
itself, and then finally to the openstack community later.

There still is no storage implementation - but this means
that finally there is a clear path for the dependencies that
FT needs to have in place to perform well in the future.

- Michael


-------- Original Message --------
Subject: 	[Qemu-devel] [PULL 0/4] migration queue
Date: 	Tue, 25 Feb 2014 15:07:34 +0100
From: 	Juan Quintela <quintela@redhat.com>
To: 	qemu-devel@nongnu.org



Hi

Please pull, this include:

- make rdma as stable (patch has been reviewed since long ago)(mrhines)
- less_equal means less_equal not great_or_eual: (david and brown paperbag for me)
- fix two XBZRLE corruptions
- fwrite(): we are back to v2 of the patch. (me)


The following changes since commit 0459650d94d18218808fcabc8c3227d2ee99af39:

   Merge remote-tracking branch 'remotes/mdroth/qga-pull-2014-02-24' into staging (2014-02-25 12:30:01 +0000)

are available in the git repository at:


   git://github.com/juanquintela/qemu.git tags/migration/20140225

for you to fetch changes up to 41310c68781d742fa9bbfd5fcb1df9b7f23f5759:

   rdma: rename 'x-rdma' => 'rdma' (2014-02-25 14:30:28 +0100)

----------------------------------------------------------------
migration/next for 20140225

----------------------------------------------------------------
Dr. David Alan Gilbert (2):
       Fix vmstate_info_int32_le comparison/assign
       Fix two XBZRLE corruption issues

Juan Quintela (1):
       qemu_file: use fwrite() correctly

Michael R. Hines (1):
       rdma: rename 'x-rdma' => 'rdma'

  arch_init.c                    | 64 ++++++++++++++++++++++++++++++++----------
  docs/rdma.txt                  | 24 +++++++---------
  include/migration/page_cache.h |  2 +-
  migration-rdma.c               |  2 +-
  migration.c                    |  6 ++--
  page_cache.c                   |  2 +-
  qapi-schema.json               |  7 ++---
  qemu-file.c                    |  9 +++++-
  vmstate.c                      | 15 +++++-----
  9 files changed, 84 insertions(+), 47 deletions(-)




[-- Attachment #2: Type: text/html, Size: 3440 bytes --]

[Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

Hi

This includes:

- last two patches that remove version_minimum_id_old.

- refactor of opening of files was already tested (this is test-vmstate only)
- v4 of flags in ram_load from Peter.

Please, apply.

Juan.


The following changes since commit 3334e929ae2d01e47c3e64757ee7fcf975cede94:

  Merge remote-tracking branch 'remotes/kraxel/tags/pull-console-20140610-1' into staging (2014-06-10 12:06:17 +0100)

are available in the git repository at:


  git://github.com/juanquintela/qemu.git tags/migration/20140610

for you to fetch changes up to e59d8b1399705ea94d6e66040e37848c56cb4843:

  vmstate: Refactor opening of files (2014-06-10 16:52:14 +0200)

----------------------------------------------------------------
migration/next for 20140610

----------------------------------------------------------------
Juan Quintela (3):
      savevm: Remove all the unneeded version_minimum_id_old (ppc)
      savevm: Remove all the unneeded version_minimum_id_old (x86)
      vmstate: Refactor opening of files

Peter Lieven (1):
      migration: catch unknown flags in ram_load

 arch_init.c                        | 42 +++++++++++++++-------------
 hw/acpi/ich9.c                     |  1 -
 hw/acpi/pcihp.c                    |  3 +-
 hw/acpi/piix4.c                    |  8 ++----
 hw/audio/ac97.c                    |  6 ++--
 hw/audio/cs4231.c                  |  3 +-
 hw/audio/cs4231a.c                 |  3 +-
 hw/audio/es1370.c                  |  6 ++--
 hw/audio/gus.c                     |  3 +-
 hw/audio/hda-codec.c               |  4 +--
 hw/audio/intel-hda.c               |  4 +--
 hw/audio/sb16.c                    |  3 +-
 hw/block/fdc.c                     | 16 ++++-------
 hw/char/escc.c                     |  6 ++--
 hw/char/serial-pci.c               |  4 +--
 hw/char/serial.c                   |  2 +-
 hw/char/spapr_vty.c                |  3 +-
 hw/display/cirrus_vga.c            |  6 ++--
 hw/display/vga-pci.c               |  3 +-
 hw/display/vga.c                   |  3 +-
 hw/display/vmware_vga.c            |  6 ++--
 hw/dma/i8257.c                     |  6 ++--
 hw/i386/acpi-build.c               |  3 +-
 hw/i386/kvm/clock.c                |  1 -
 hw/i386/kvmvapic.c                 |  3 --
 hw/i386/pc.c                       |  3 +-
 hw/i386/xen/xen_platform.c         |  3 +-
 hw/ide/ahci.c                      |  6 ++--
 hw/ide/ich.c                       |  2 +-
 hw/ide/isa.c                       |  3 +-
 hw/ide/pci.c                       | 12 +++-----
 hw/input/pckbd.c                   |  6 ++--
 hw/input/ps2.c                     | 12 +++-----
 hw/input/vmmouse.c                 |  3 +-
 hw/intc/heathrow_pic.c             |  6 ++--
 hw/intc/i8259_common.c             |  1 -
 hw/intc/ioapic_common.c            |  1 -
 hw/intc/xics.c                     |  9 ++----
 hw/isa/apm.c                       |  1 -
 hw/isa/lpc_ich9.c                  |  1 -
 hw/isa/piix4.c                     |  3 +-
 hw/isa/vt82c686.c                  |  6 ++--
 hw/net/e1000.c                     |  6 ++--
 hw/net/eepro100.c                  |  3 +-
 hw/net/ne2000-isa.c                |  3 +-
 hw/net/ne2000.c                    |  6 ++--
 hw/net/pcnet-pci.c                 |  3 +-
 hw/net/pcnet.c                     |  3 +-
 hw/net/rtl8139.c                   |  9 ++----
 hw/net/spapr_llan.c                |  3 +-
 hw/net/vmxnet3.c                   |  4 +--
 hw/nvram/eeprom93xx.c              |  3 +-
 hw/nvram/fw_cfg.c                  |  3 +-
 hw/pci-bridge/ioh3420.c            |  1 -
 hw/pci-bridge/xio3130_downstream.c |  1 -
 hw/pci-bridge/xio3130_upstream.c   |  1 -
 hw/pci-host/piix.c                 |  7 ++---
 hw/pci-host/ppce500.c              |  9 ++----
 hw/pci-host/q35.c                  |  3 +-
 hw/pci/pci.c                       |  9 ++----
 hw/pci/pcie_aer.c                  |  6 ++--
 hw/ppc/ppc4xx_pci.c                |  9 ++----
 hw/ppc/spapr.c                     |  3 +-
 hw/ppc/spapr_iommu.c               |  3 +-
 hw/ppc/spapr_pci.c                 |  9 ++----
 hw/ppc/spapr_vio.c                 |  3 +-
 hw/scsi/lsi53c895a.c               |  3 +-
 hw/scsi/megasas.c                  |  3 +-
 hw/scsi/scsi-bus.c                 |  4 +--
 hw/scsi/scsi-disk.c                |  1 -
 hw/scsi/spapr_vscsi.c              |  6 ++--
 hw/scsi/vmw_pvscsi.c               |  3 +-
 hw/timer/hpet.c                    |  9 ++----
 hw/timer/i8254_common.c            |  1 -
 hw/timer/m48t59.c                  |  3 +-
 hw/timer/mc146818rtc.c             |  3 +-
 hw/watchdog/wdt_i6300esb.c         | 14 +++++-----
 hw/watchdog/wdt_ib700.c            |  3 +-
 migration.c                        |  2 +-
 target-i386/machine.c              | 57 +++++++++++++-------------------------
 target-ppc/machine.c               | 38 +++++++++----------------
 tests/test-vmstate.c               | 38 ++++++++++++-------------
 82 files changed, 193 insertions(+), 331 deletions(-)

Re: [Qemu-devel] [PULL 0/4] migration queue

[Paolo Bonzini]

Il 10/06/2014 18:31, Juan Quintela ha scritto:
> Hi
>
> This includes:
>
> - last two patches that remove version_minimum_id_old.
>
> - refactor of opening of files was already tested (this is test-vmstate only)
> - v4 of flags in ram_load from Peter.
>
> Please, apply.
>
> Juan.

You missed the patch to change the default minimum downtime.

Paolo

Re: [Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

Paolo Bonzini <pbonzini@redhat.com> wrote:
> Il 10/06/2014 18:31, Juan Quintela ha scritto:
>> Hi
>>
>> This includes:
>>
>> - last two patches that remove version_minimum_id_old.
>>
>> - refactor of opening of files was already tested (this is test-vmstate only)
>> - v4 of flags in ram_load from Peter.
>>
>> Please, apply.
>>
>> Juan.
>
> You missed the patch to change the default minimum downtime.

Thanks, v4 was still not ready when I did the pull request.
Added the downtime one & RDMA for next pull request (that expect to be
just after this one gets integrated.)

Thanks again, Juan.

Re: [Qemu-devel] [PULL 0/4] migration queue

[Peter Maydell]

On 10 June 2014 17:31, Juan Quintela <quintela@redhat.com> wrote:
> Hi
>
> This includes:
>
> - last two patches that remove version_minimum_id_old.
>
> - refactor of opening of files was already tested (this is test-vmstate only)
> - v4 of flags in ram_load from Peter.
>
> Please, apply.
>
> Juan.
>
>
> The following changes since commit 3334e929ae2d01e47c3e64757ee7fcf975cede94:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-console-20140610-1' into staging (2014-06-10 12:06:17 +0100)
>
> are available in the git repository at:
>
>
>   git://github.com/juanquintela/qemu.git tags/migration/20140610
>
> for you to fetch changes up to e59d8b1399705ea94d6e66040e37848c56cb4843:
>
>   vmstate: Refactor opening of files (2014-06-10 16:52:14 +0200)
>
> ----------------------------------------------------------------
> migration/next for 20140610
>
> ----------------------------------------------------------------
> Juan Quintela (3):
>       savevm: Remove all the unneeded version_minimum_id_old (ppc)
>       savevm: Remove all the unneeded version_minimum_id_old (x86)
>       vmstate: Refactor opening of files
>
> Peter Lieven (1):
>       migration: catch unknown flags in ram_load

I'm afraid this doesn't seem to build on 32 bit:

/root/qemu/arch_init.c: In function ‘ram_load’:
/root/qemu/arch_init.c:1101:17: error: format ‘%llx’ expects argument
of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
[-Werror=format]
/root/qemu/arch_init.c:1113:17: error: format ‘%llx’ expects argument
of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
[-Werror=format]
/root/qemu/arch_init.c:1122:17: error: format ‘%llx’ expects argument
of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
[-Werror=format]
/root/qemu/arch_init.c:1129:30: error: format ‘%llx’ expects argument
of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
[-Werror=format]
cc1: all warnings being treated as errors
make[1]: *** [arch_init.o] Error 1

thanks
-- PMM

Re: [Qemu-devel] [PULL 0/4] migration queue

[Juan Quintela]

Peter Maydell <peter.maydell@linaro.org> wrote:
> On 10 June 2014 17:31, Juan Quintela <quintela@redhat.com> wrote:
>> Hi
>>
>> This includes:
>>
>> - last two patches that remove version_minimum_id_old.
>>
>> - refactor of opening of files was already tested (this is test-vmstate only)
>> - v4 of flags in ram_load from Peter.
>>
>> Please, apply.
>>
>> Juan.
>>
>>
>> The following changes since commit 3334e929ae2d01e47c3e64757ee7fcf975cede94:
>>
>>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-console-20140610-1' into staging (2014-06-10 12:06:17 +0100)
>>
>> are available in the git repository at:
>>
>>
>>   git://github.com/juanquintela/qemu.git tags/migration/20140610
>>
>> for you to fetch changes up to e59d8b1399705ea94d6e66040e37848c56cb4843:
>>
>>   vmstate: Refactor opening of files (2014-06-10 16:52:14 +0200)
>>
>> ----------------------------------------------------------------
>> migration/next for 20140610
>>
>> ----------------------------------------------------------------
>> Juan Quintela (3):
>>       savevm: Remove all the unneeded version_minimum_id_old (ppc)
>>       savevm: Remove all the unneeded version_minimum_id_old (x86)
>>       vmstate: Refactor opening of files
>>
>> Peter Lieven (1):
>>       migration: catch unknown flags in ram_load
>
> I'm afraid this doesn't seem to build on 32 bit:

grrrrr

Guess who is installing a 32bit guest.
I already build for all architectures on linux and cross-compile for
windows, clearly it is not enough.

Thanks, and sorry for the inconveniences.

Later, Juan.

>
> /root/qemu/arch_init.c: In function ‘ram_load’:
> /root/qemu/arch_init.c:1101:17: error: format ‘%llx’ expects argument
> of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
> [-Werror=format]
> /root/qemu/arch_init.c:1113:17: error: format ‘%llx’ expects argument
> of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
> [-Werror=format]
> /root/qemu/arch_init.c:1122:17: error: format ‘%llx’ expects argument
> of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
> [-Werror=format]
> /root/qemu/arch_init.c:1129:30: error: format ‘%llx’ expects argument
> of type ‘long long unsigned int’, but argument 2 has type ‘ram_addr_t’
> [-Werror=format]
> cc1: all warnings being treated as errors
> make[1]: *** [arch_init.o] Error 1
>
> thanks
> -- PMM