diff options
| author | Li Zhijian <lizhijian@fujitsu.com> | 2025-03-05 14:28:24 +0800 |
|---|---|---|
| committer | Peter Xu <peterx@redhat.com> | 2025-05-02 11:09:36 -0400 |
| commit | 5e7ca4a7d79da6c9e584886879002f24917c8d27 (patch) | |
| tree | 384c55516fd56a2d4fc0a42e69ad5281470b2a29 /migration/rdma.c | |
| parent | 4ecd6beaf9ff4ad88a33eab78789335a6e929564 (diff) | |
| download | focaccia-qemu-5e7ca4a7d79da6c9e584886879002f24917c8d27.tar.gz focaccia-qemu-5e7ca4a7d79da6c9e584886879002f24917c8d27.zip | |
migration: Unfold control_save_page()
control_save_page() is for RDMA only, unfold it to make the code more clear. In addition: - Similar to other branches style in ram_save_target_page(), involve RDMA only if the condition 'migrate_rdma()' is true. - Further simplify the code by removing the RAM_SAVE_CONTROL_NOT_SUPP. Reviewed-by: Peter Xu <peterx@redhat.com> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> Message-ID: <20250305062825.772629-6-lizhijian@fujitsu.com> Signed-off-by: Fabiano Rosas <farosas@suse.de>
Diffstat (limited to 'migration/rdma.c')
| -rw-r--r-- | migration/rdma.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/migration/rdma.c b/migration/rdma.c index a3c3b432d1..4875ca1987 100644 --- a/migration/rdma.c +++ b/migration/rdma.c @@ -3284,14 +3284,11 @@ err: int rdma_control_save_page(QEMUFile *f, ram_addr_t block_offset, ram_addr_t offset, size_t size) { - if (!migrate_rdma()) { - return RAM_SAVE_CONTROL_NOT_SUPP; - } + assert(migrate_rdma()); int ret = qemu_rdma_save_page(f, block_offset, offset, size); - if (ret != RAM_SAVE_CONTROL_DELAYED && - ret != RAM_SAVE_CONTROL_NOT_SUPP) { + if (ret != RAM_SAVE_CONTROL_DELAYED) { if (ret < 0) { qemu_file_set_error(f, ret); } |