Patchwork [PULL,00/13] thread queue

login
register
mail settings
Submitter Juan Quintela
Date Jan. 17, 2013, 1:39 p.m.
Message ID <1358429973-9321-1-git-send-email-quintela@redhat.com>
Download mbox
Permalink /patch/213264/
State New
Headers show

Pull-request

git://repo.or.cz/qemu/quintela.git thread.next

Comments

Juan Quintela - Jan. 17, 2013, 1:39 p.m.
This is the intersect of the paolo & me patches for migration thread,

Changes from 2 days ago:
- spelling check from Eric
- put the commit that patch refered from (Eric)
- drop the buffered rename at paolo request.

Please, pull.

Thanks, Juan.

The following changes since commit 47f4dac3fde809e3da4e60d9eb699f1d4b378249:

  Merge remote-tracking branch 'kraxel/chardev.1' into staging (2013-01-16 15:20:05 -0600)

are available in the git repository at:


  git://repo.or.cz/qemu/quintela.git thread.next

for you to fetch changes up to 6522773f88a2e37800f0bf7dc3632a14649f53c6:

  migration: remove argument to qemu_savevm_state_cancel (2013-01-17 13:54:52 +0100)

----------------------------------------------------------------
Juan Quintela (7):
      qemu-file: Only set last_error if it is not already set
      migration: move beginning stage to the migration thread
      migration: Add buffered_flush error handling
      migration: move exit condition to migration thread
      migration: unfold rest of migrate_fd_put_ready() into thread
      migration: Only go to the iterate stage if there is anything to send
      migration: remove argument to qemu_savevm_state_cancel

Paolo Bonzini (6):
      Unlock ramlist lock also in error case
      Protect migration_bitmap_sync() with the ramlist lock
      use XFER_LIMIT_RATIO consistently
      migration: make function static
      migration: remove double call to migrate_fd_close
      migration: fix off-by-one in buffered_rate_limit

 arch_init.c                   |   6 +-
 include/migration/migration.h |   3 -
 include/sysemu/sysemu.h       |   2 +-
 migration.c                   | 149 ++++++++++++++++++++----------------------
 savevm.c                      |  12 ++--
 5 files changed, 81 insertions(+), 91 deletions(-)
Anthony Liguori - Jan. 17, 2013, 3:22 p.m.
Juan Quintela <quintela@redhat.com> writes:

> This is the intersect of the paolo & me patches for migration thread,
>
> Changes from 2 days ago:
> - spelling check from Eric
> - put the commit that patch refered from (Eric)
> - drop the buffered rename at paolo request.
>
> Please, pull.

If these were just posted two days ago, it's too early to do a pull
request.  Give people time to review the series.

Regards,

Anthony Liguori

>
> Thanks, Juan.
>
> The following changes since commit 47f4dac3fde809e3da4e60d9eb699f1d4b378249:
>
>   Merge remote-tracking branch 'kraxel/chardev.1' into staging (2013-01-16 15:20:05 -0600)
>
> are available in the git repository at:
>
>
>   git://repo.or.cz/qemu/quintela.git thread.next
>
> for you to fetch changes up to 6522773f88a2e37800f0bf7dc3632a14649f53c6:
>
>   migration: remove argument to qemu_savevm_state_cancel (2013-01-17 13:54:52 +0100)
>
> ----------------------------------------------------------------
> Juan Quintela (7):
>       qemu-file: Only set last_error if it is not already set
>       migration: move beginning stage to the migration thread
>       migration: Add buffered_flush error handling
>       migration: move exit condition to migration thread
>       migration: unfold rest of migrate_fd_put_ready() into thread
>       migration: Only go to the iterate stage if there is anything to send
>       migration: remove argument to qemu_savevm_state_cancel
>
> Paolo Bonzini (6):
>       Unlock ramlist lock also in error case
>       Protect migration_bitmap_sync() with the ramlist lock
>       use XFER_LIMIT_RATIO consistently
>       migration: make function static
>       migration: remove double call to migrate_fd_close
>       migration: fix off-by-one in buffered_rate_limit
>
>  arch_init.c                   |   6 +-
>  include/migration/migration.h |   3 -
>  include/sysemu/sysemu.h       |   2 +-
>  migration.c                   | 149 ++++++++++++++++++++----------------------
>  savevm.c                      |  12 ++--
>  5 files changed, 81 insertions(+), 91 deletions(-)
Orit Wasserman - Jan. 23, 2013, 8:29 a.m.
On 01/17/2013 03:39 PM, Juan Quintela wrote:
> This is the intersect of the paolo & me patches for migration thread,
> 
> Changes from 2 days ago:
> - spelling check from Eric
> - put the commit that patch refered from (Eric)
> - drop the buffered rename at paolo request.
> 
> Please, pull.
> 
> Thanks, Juan.
> 
> The following changes since commit 47f4dac3fde809e3da4e60d9eb699f1d4b378249:
> 
>   Merge remote-tracking branch 'kraxel/chardev.1' into staging (2013-01-16 15:20:05 -0600)
> 
> are available in the git repository at:
> 
> 
>   git://repo.or.cz/qemu/quintela.git thread.next
> 
> for you to fetch changes up to 6522773f88a2e37800f0bf7dc3632a14649f53c6:
> 
>   migration: remove argument to qemu_savevm_state_cancel (2013-01-17 13:54:52 +0100)
> 
> ----------------------------------------------------------------
> Juan Quintela (7):
>       qemu-file: Only set last_error if it is not already set
>       migration: move beginning stage to the migration thread
>       migration: Add buffered_flush error handling
>       migration: move exit condition to migration thread
>       migration: unfold rest of migrate_fd_put_ready() into thread
>       migration: Only go to the iterate stage if there is anything to send
>       migration: remove argument to qemu_savevm_state_cancel
> 
> Paolo Bonzini (6):
>       Unlock ramlist lock also in error case
>       Protect migration_bitmap_sync() with the ramlist lock
>       use XFER_LIMIT_RATIO consistently
>       migration: make function static
>       migration: remove double call to migrate_fd_close
>       migration: fix off-by-one in buffered_rate_limit
> 
>  arch_init.c                   |   6 +-
>  include/migration/migration.h |   3 -
>  include/sysemu/sysemu.h       |   2 +-
>  migration.c                   | 149 ++++++++++++++++++++----------------------
>  savevm.c                      |  12 ++--
>  5 files changed, 81 insertions(+), 91 deletions(-)
> 
Looks good to me,
Series: Reviewed-by: Orit Wasserman <owasserm@redhat.com>