Skip to content

migration/channel errors and cancelling

Dr. David Alan Gilbert (git) qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 8 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2migration: Allow migrate_fd_connect to take an Error *migration/channel.c migration/migration.c migration/migration.h +1 more+9 -41 reply
2/2migration: Route errors down through migration_channel_connectmigration/channel.c migration/channel.h migration/exec.c +4 more+23 -251 reply

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by.

PatchJuan Quintela
1/2R-b
2/2R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/2] migration/channel errors and cancelling

From: "Dr. David Alan Gilbert" <[email protected]>

Hi,
Where a channel fails asynchronously during connect, call
back through the migration code so it can clean up.
In particular this causes the transition of a 'cancelling' state
to 'cancelled' in the case of:

migrate -d tcp:deadhost:port
<host tries to connect>
migrate_cancel

previously the status would get stuck in cancelling because the final cleanup didn't happen.

This is the second part of the fix for:
https://bugzilla.redhat.com/show_bug.cgi?id=1525899

Dave

Dr. David Alan Gilbert (2):
migration: Allow migrate_fd_connect to take an Error *
migration: Route errors down through migration_channel_connect

 migration/channel.c    | 32 ++++++++++++++++----------------
 migration/channel.h    |  3 ++-
 migration/exec.c       |  2 +-
 migration/fd.c         |  2 +-
 migration/migration.c  |  7 ++++++-
 migration/migration.h  |  2 +-
 migration/rdma.c       |  2 +-
 migration/socket.c     |  4 +---
 migration/tls.c        |  3 +--
 migration/trace-events |  2 +-
 10 files changed, 31 insertions(+), 28 deletions(-)
Signature · 1 line
2.14.3