| Message ID | 20190711080816.6405-1-richardw.yang@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | migration/postcopy: PostcopyState is already set in loadvm_postcopy_handle_advise() | expand |
On Thu, Jul 11, 2019 at 04:08:16PM +0800, Wei Yang wrote: >PostcopyState is already set to ADVISE at the beginning of >loadvm_postcopy_handle_advise(). > >Remove the redundant set. Ping~ > >Signed-off-by: Wei Yang <richardw.yang@linux.intel.com> >--- > migration/savevm.c | 2 -- > 1 file changed, 2 deletions(-) > >diff --git a/migration/savevm.c b/migration/savevm.c >index 8a2ada529e..2350e219fc 100644 >--- a/migration/savevm.c >+++ b/migration/savevm.c >@@ -1648,8 +1648,6 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis, > return -1; > } > >- postcopy_state_set(POSTCOPY_INCOMING_ADVISE); >- > return 0; > } > >-- >2.17.1
* Wei Yang (richardw.yang@linux.intel.com) wrote: > PostcopyState is already set to ADVISE at the beginning of > loadvm_postcopy_handle_advise(). > > Remove the redundant set. > > Signed-off-by: Wei Yang <richardw.yang@linux.intel.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > --- > migration/savevm.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/migration/savevm.c b/migration/savevm.c > index 8a2ada529e..2350e219fc 100644 > --- a/migration/savevm.c > +++ b/migration/savevm.c > @@ -1648,8 +1648,6 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis, > return -1; > } > > - postcopy_state_set(POSTCOPY_INCOMING_ADVISE); > - > return 0; > } > > -- > 2.17.1 > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
* Wei Yang (richardw.yang@linux.intel.com) wrote: > PostcopyState is already set to ADVISE at the beginning of > loadvm_postcopy_handle_advise(). > > Remove the redundant set. > > Signed-off-by: Wei Yang <richardw.yang@linux.intel.com> Queued > --- > migration/savevm.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/migration/savevm.c b/migration/savevm.c > index 8a2ada529e..2350e219fc 100644 > --- a/migration/savevm.c > +++ b/migration/savevm.c > @@ -1648,8 +1648,6 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis, > return -1; > } > > - postcopy_state_set(POSTCOPY_INCOMING_ADVISE); > - > return 0; > } > > -- > 2.17.1 > > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
diff --git a/migration/savevm.c b/migration/savevm.c index 8a2ada529e..2350e219fc 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -1648,8 +1648,6 @@ static int loadvm_postcopy_handle_advise(MigrationIncomingState *mis, return -1; } - postcopy_state_set(POSTCOPY_INCOMING_ADVISE); - return 0; }
PostcopyState is already set to ADVISE at the beginning of loadvm_postcopy_handle_advise(). Remove the redundant set. Signed-off-by: Wei Yang <richardw.yang@linux.intel.com> --- migration/savevm.c | 2 -- 1 file changed, 2 deletions(-)