* Wei Yang (richardw.y...@linux.intel.com) wrote: > Since we break the loop when there is no more page to discard, we are > sure the following process would find some page to discard. > > It is not necessary to check it again. > > Signed-off-by: Wei Yang <richardw.y...@linux.intel.com>
Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com> > --- > migration/ram.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/migration/ram.c b/migration/ram.c > index b41b58ee54..246efe6939 100644 > --- a/migration/ram.c > +++ b/migration/ram.c > @@ -2789,9 +2789,7 @@ static int postcopy_send_discard_bm_ram(MigrationState > *ms, > } else { > discard_length = zero - one; > } > - if (discard_length) { > - postcopy_discard_send_range(ms, pds, one, discard_length); > - } > + postcopy_discard_send_range(ms, pds, one, discard_length); > current = one + discard_length; > } > > -- > 2.19.1 > -- Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK