-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MR descriptor list #1117
MR descriptor list #1117
Conversation
f67706d
to
4317631
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wrrobin, do you happen to know how the MR descriptors are traversed by the provider? For example, if there is a list of 3 descriptors and the target is in the external heap, are all 3 MR descriptors read to lookup which applies? Could there be a performance cost associated with that?
Great question. And, I do not know what order it is traversed. Let me ask this question and add you to that email thread. |
External heap mrfd for fetch atomic Removed external mrfd from fetch_atomic Using atomic_nbi for fetch_nbi Clean up Added mr-descs for both atomic operations Trial check for backtrace Trial check removing mrfd for error case Trial check#2 removing mrfd for error case Trial check#3 removing mrfd for error case Trial check#4 removing mrfd for error case Trial check#5 removing mrfd for error case Trial with sockets provider
@rithwiktom - Could you get a chance to test this out on your end? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works on ORPRE.
No description provided.