* Wei Yang ([email protected]) 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 <[email protected]>
Reviewed-by: Dr. David Alan Gilbert <[email protected]> > --- > 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 / [email protected] / Manchester, UK
