Skip to content

Bugfix: data race issue exists in the message processing process#3154

Closed
chenBright wants to merge 1 commit intoapache:masterfrom
chenBright:fix_rdma_iobuf_memory
Closed

Bugfix: data race issue exists in the message processing process#3154
chenBright wants to merge 1 commit intoapache:masterfrom
chenBright:fix_rdma_iobuf_memory

Conversation

@chenBright
Copy link
Copy Markdown
Contributor

@chenBright chenBright commented Nov 17, 2025

What problem does this PR solve?

Issue Number: resolve #2265 , resolve #2814

Problem Summary:

Messages should not be processed after no events have occurred. Otherwise, multi-threaded operations on _read_buf may cause a data race problem.

What is changed and the side effects?

Changed:

Side effects:

  • Performance effects:

  • Breaking backward compatibility:


Check List:

@yanglimingcn
Copy link
Copy Markdown
Contributor

yanglimingcn commented Nov 17, 2025

I checked and all these break points are inside the while loop. Using break will also exit the function, so there is no logical difference in making this change, right?

@chenBright
Copy link
Copy Markdown
Contributor Author

I think I've made a mistake.

@chenBright chenBright closed this Nov 17, 2025
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.

[RDMA] 启用RDMA后client侧解析响应失败 RDMA failure

2 participants