mbox

[migration,PULL,0/9] Migration queue - for Juan

Message ID cover.1443508205.git.amit.shah@redhat.com
State New
Headers show

Pull-request

https://git.kernel.org/pub/scm/virt/qemu/amit/migration.git tags/for-juan-201509

Message

Amit Shah Sept. 29, 2015, 6:32 a.m. UTC
Hi Juan,

Here are a few patches from the list that have been reviewed by at
least one person.

Would be great to have your review and then a pull.

Thanks,

The following changes since commit 9e071429e649346c14b2dc76902f84f8352d2333:

  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2015-09-25 21:52:30 +0100)

are available in the git repository at:

  https://git.kernel.org/pub/scm/virt/qemu/amit/migration.git tags/for-juan-201509

for you to fetch changes up to b9e6092814735853cc1149e2e68245b09f621306:

  ram_find_and_save_block: Split out the finding (2015-09-29 11:38:29 +0530)

----------------------------------------------------------------
Migration queue

----------------------------------------------------------------



Dr. David Alan Gilbert (7):
  migration/ram.c: Use RAMBlock rather than MemoryRegion
  Split out end of migration code from migration_thread
  Init page sizes in qtest
  migration: size_t'ify some of qemu-file
  migration: qemu-file more size_t'ifying
  Move dirty page search state into separate structure
  ram_find_and_save_block: Split out the finding

Markus Armbruster (1):
  migration: Use g_new() & friends where that makes obvious sense

Soren Brinkmann (1):
  vmstate: Remove redefinition of VMSTATE_UINT32_ARRAY

 include/migration/qemu-file.h |  18 +++---
 include/migration/vmstate.h   |   3 -
 migration/migration.c         |  77 ++++++++++++++---------
 migration/qemu-file-buf.c     |   9 +--
 migration/qemu-file-stdio.c   |  15 ++---
 migration/qemu-file-unix.c    |  10 +--
 migration/qemu-file.c         |  24 +++----
 migration/ram.c               | 143 +++++++++++++++++++++++++++---------------
 migration/rdma.c              |  30 ++++-----
 migration/savevm.c            |  19 +++---
 qtest.c                       |   1 +
 trace-events                  |   4 +-
 12 files changed, 211 insertions(+), 142 deletions(-)

Comments

Peter Maydell Sept. 29, 2015, 12:50 p.m. UTC | #1
On 29 September 2015 at 07:32, Amit Shah <amit.shah@redhat.com> wrote:
> Hi Juan,
>
> Here are a few patches from the list that have been reviewed by at
> least one person.
>
> Would be great to have your review and then a pull.

Whoops, I just applied this to master because I didn't pay enough attention
to the cover letter.

Juan -- can you let me know if you want me to revert that?

thanks
-- PMM
Amit Shah Sept. 29, 2015, 1:05 p.m. UTC | #2
On (Tue) 29 Sep 2015 [13:50:10], Peter Maydell wrote:
> On 29 September 2015 at 07:32, Amit Shah <amit.shah@redhat.com> wrote:
> > Hi Juan,
> >
> > Here are a few patches from the list that have been reviewed by at
> > least one person.
> >
> > Would be great to have your review and then a pull.
> 
> Whoops, I just applied this to master because I didn't pay enough attention
> to the cover letter.

ouch; sorry for the confusion.  The patches are small and
uncontroversial, so I don't mind you merging them.

> Juan -- can you let me know if you want me to revert that?

Yes, if there's a problem, we can work out a new pull.


		Amit
Peter Maydell Sept. 29, 2015, 1:23 p.m. UTC | #3
On 29 September 2015 at 14:05, Amit Shah <amit.shah@redhat.com> wrote:
> On (Tue) 29 Sep 2015 [13:50:10], Peter Maydell wrote:
>> On 29 September 2015 at 07:32, Amit Shah <amit.shah@redhat.com> wrote:
>> > Hi Juan,
>> >
>> > Here are a few patches from the list that have been reviewed by at
>> > least one person.
>> >
>> > Would be great to have your review and then a pull.
>>
>> Whoops, I just applied this to master because I didn't pay enough attention
>> to the cover letter.
>
> ouch; sorry for the confusion.

Your mail was clear enough -- entirely my fault for not
reading it or looking at the subject tag. I've just got
used to all the pulls on the list being destined for master...

-- PMM
Juan Quintela Sept. 30, 2015, 4:23 p.m. UTC | #4
Peter Maydell <peter.maydell@linaro.org> wrote:
> On 29 September 2015 at 07:32, Amit Shah <amit.shah@redhat.com> wrote:
>> Hi Juan,
>>
>> Here are a few patches from the list that have been reviewed by at
>> least one person.
>>
>> Would be great to have your review and then a pull.
>
> Whoops, I just applied this to master because I didn't pay enough attention
> to the cover letter.
>
> Juan -- can you let me know if you want me to revert that?

It is ok, thanks for the pull, Amit.

Reviewed-by: Juan Quintela <quintela@redhat.com>

For what is worth so late.


>
> thanks
> -- PMM