From 0abfff9ea7c56c2f6ad3cee10120915ec723cb32 Mon Sep 17 00:00:00 2001 From: Wei Yang Date: Thu, 27 Jun 2019 10:08:20 +0800 Subject: migration/postcopy: the valid condition is one less then end If one equals end, it means we have gone through the whole bitmap. Use a more restrict check to skip a unnecessary condition. Signed-off-by: Wei Yang Message-Id: <20190627020822.15485-2-richardw.yang@linux.intel.com> Reviewed-by: Dr. David Alan Gilbert Signed-off-by: Dr. David Alan Gilbert --- migration/ram.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/migration/ram.c b/migration/ram.c index 889148d..68bc11c 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -2865,7 +2865,7 @@ static int postcopy_send_discard_bm_ram(MigrationState *ms, for (current = 0; current < end; ) { unsigned long one = find_next_bit(unsentmap, end, current); - if (one <= end) { + if (one < end) { unsigned long zero = find_next_zero_bit(unsentmap, end, one + 1); unsigned long discard_length; -- cgit v1.1