Skip to content

rdma runtime extend block restriction#3076

Merged
yanglimingcn merged 2 commits intoapache:masterfrom
yanglimingcn:bugfix/rdma_block_pool_runtime_extend
Sep 8, 2025
Merged

rdma runtime extend block restriction#3076
yanglimingcn merged 2 commits intoapache:masterfrom
yanglimingcn:bugfix/rdma_block_pool_runtime_extend

Conversation

@yanglimingcn
Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Issue Number:

Problem Summary:

When both rdma_memory_pool_max_regions and rdma_memory_pool_buckets are greater than 1, dynamic memory expansion may cause concurrent modification issues in the memory linked list due to lock contention problems. To address this, we increase the region_num count for each block_type. Dynamic memory expansion is only permitted when both of the following conditions are met:

rdma_memory_pool_buckets equals 1
g_info->region_num[block_type] is less than 1

What is changed and the side effects?

Changed:

Side effects:

  • Performance effects:

  • Breaking backward compatibility:


Check List:

@yanglimingcn yanglimingcn force-pushed the bugfix/rdma_block_pool_runtime_extend branch from 307e66a to 1231b58 Compare August 27, 2025 09:37
@wwbmmm
Copy link
Copy Markdown
Contributor

wwbmmm commented Aug 28, 2025

LGTM

@yanglimingcn yanglimingcn force-pushed the bugfix/rdma_block_pool_runtime_extend branch from 1231b58 to 43f48c8 Compare August 28, 2025 08:53
@yanglimingcn yanglimingcn force-pushed the bugfix/rdma_block_pool_runtime_extend branch from 43f48c8 to a83ae4a Compare September 5, 2025 01:20
@yanglimingcn yanglimingcn force-pushed the bugfix/rdma_block_pool_runtime_extend branch from a83ae4a to 4533167 Compare September 5, 2025 07:41
@yanglimingcn yanglimingcn changed the title rdma rumtime extend block restriction rdma runtime extend block restriction Sep 5, 2025
@yanglimingcn yanglimingcn merged commit 7781cd8 into apache:master Sep 8, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants