ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

migration/multifd: Drop pages->num check in sender thread 

Now with a split SYNC handler, we always have pages->num set for
pending_job==true.  Assert it instead.

Reviewed-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/r/20240202102857.110210-9-peterx@redhat.com
Signed-off-by: Peter Xu <peterx@redhat.com>
This commit is contained in:
Peter Xu 2024-02-02 18:28:42 +08:00
parent e3cce9af10
commit 83c560fb42
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
migration/multifd.c
View File

@ -710,13 +710,14 @@ static void *multifd_send_thread(void *opaque)
p->iovs_num = 1; p->iovs_num = 1;
} }
if (pages->num) { assert(pages->num);
ret = multifd_send_state->ops->send_prepare(p, &local_err);
if (ret != 0) { ret = multifd_send_state->ops->send_prepare(p, &local_err);
qemu_mutex_unlock(&p->mutex); if (ret != 0) {
break; qemu_mutex_unlock(&p->mutex);
} break;
} }
multifd_send_fill_packet(p); multifd_send_fill_packet(p);
p->num_packets++; p->num_packets++;
p->total_normal_pages += pages->num; p->total_normal_pages += pages->num;