Re: [PATCH v2 13/53] migration/rdma: Drop qemu_rdma_search_ram_block() error handling

2023-10-04 Thread Juan Quintela
Markus Armbruster  wrote:
> qemu_rdma_search_ram_block() can't fail.  Return void, and drop the
> unreachable error handling.
>
> Signed-off-by: Markus Armbruster 
> Reviewed-by: Li Zhijian 

Reviewed-by: Juan Quintela 




[PATCH v2 13/53] migration/rdma: Drop qemu_rdma_search_ram_block() error handling

2023-09-28 Thread Markus Armbruster
qemu_rdma_search_ram_block() can't fail.  Return void, and drop the
unreachable error handling.

Signed-off-by: Markus Armbruster 
Reviewed-by: Li Zhijian 
---
 migration/rdma.c | 24 
 1 file changed, 8 insertions(+), 16 deletions(-)

diff --git a/migration/rdma.c b/migration/rdma.c
index f2f811ace2..7bea4d3947 100644
--- a/migration/rdma.c
+++ b/migration/rdma.c
@@ -1231,15 +1231,13 @@ static int qemu_rdma_reg_whole_ram_blocks(RDMAContext 
*rdma)
  *
  * Once the block is found, also identify which 'chunk' within that
  * block that the page belongs to.
- *
- * This search cannot fail or the migration will fail.
  */
-static int qemu_rdma_search_ram_block(RDMAContext *rdma,
-  uintptr_t block_offset,
-  uint64_t offset,
-  uint64_t length,
-  uint64_t *block_index,
-  uint64_t *chunk_index)
+static void qemu_rdma_search_ram_block(RDMAContext *rdma,
+   uintptr_t block_offset,
+   uint64_t offset,
+   uint64_t length,
+   uint64_t *block_index,
+   uint64_t *chunk_index)
 {
 uint64_t current_addr = block_offset + offset;
 RDMALocalBlock *block = g_hash_table_lookup(rdma->blockmap,
@@ -1251,8 +1249,6 @@ static int qemu_rdma_search_ram_block(RDMAContext *rdma,
 *block_index = block->index;
 *chunk_index = ram_chunk_index(block->local_host_addr,
 block->local_host_addr + (current_addr - block->offset));
-
-return 0;
 }
 
 /*
@@ -2321,12 +2317,8 @@ static int qemu_rdma_write(QEMUFile *f, RDMAContext 
*rdma,
 rdma->current_length = 0;
 rdma->current_addr = current_addr;
 
-ret = qemu_rdma_search_ram_block(rdma, block_offset,
- offset, len, , );
-if (ret) {
-error_report("ram block search failed");
-return ret;
-}
+qemu_rdma_search_ram_block(rdma, block_offset,
+   offset, len, , );
 rdma->current_index = index;
 rdma->current_chunk = chunk;
 }
-- 
2.41.0