Re: [PATCH 0/3] migration/postcopy: replace have_listen_thread check with PostcopyState check

2019-10-11 Thread Dr. David Alan Gilbert
* Wei Yang (richardw.y...@linux.intel.com) wrote:
> have_listen_thread is used to be a indication of whether postcopy thread is
> running. Since we use PostcopyState to record state and the postcopy thread
> only runs when postcopy_is_running(), we can leverage the PostcopyState to
> replace the meaning of have_listen_thread.
> 
> To do so, two preparation cleanup is included.

1,2 queued (2 with fixup for original postcopy_state_set)

> 
> Wei Yang (3):
>   migration/postcopy: mis->have_listen_thread check will never be
> touched
>   migration/postcopy: postpone setting PostcopyState to END
>   migration/postcopy: replace have_listen_thread check with
> PostcopyState check
> 
>  migration/migration.h|  1 -
>  migration/postcopy-ram.c |  2 --
>  migration/ram.c  |  2 +-
>  migration/ram.h  |  1 +
>  migration/savevm.c   | 11 +++
>  5 files changed, 5 insertions(+), 12 deletions(-)
> 
> -- 
> 2.17.1
> 
> 
--
Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK



[PATCH 0/3] migration/postcopy: replace have_listen_thread check with PostcopyState check

2019-10-05 Thread Wei Yang
have_listen_thread is used to be a indication of whether postcopy thread is
running. Since we use PostcopyState to record state and the postcopy thread
only runs when postcopy_is_running(), we can leverage the PostcopyState to
replace the meaning of have_listen_thread.

To do so, two preparation cleanup is included.

Wei Yang (3):
  migration/postcopy: mis->have_listen_thread check will never be
touched
  migration/postcopy: postpone setting PostcopyState to END
  migration/postcopy: replace have_listen_thread check with
PostcopyState check

 migration/migration.h|  1 -
 migration/postcopy-ram.c |  2 --
 migration/ram.c  |  2 +-
 migration/ram.h  |  1 +
 migration/savevm.c   | 11 +++
 5 files changed, 5 insertions(+), 12 deletions(-)

-- 
2.17.1