Branch: refs/heads/main
Revision: 616277d
Author: ronawho
Link: Fix ugni MR extent check when remote cache is on by ronawho · Pull Request #20642 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #20642 from ronawho/fix-ugni-mr-extent-check
Fix ugni MR extent check when remote cache is on
[discussed with @mppf]
In #19516 we updated the ugni MR extent check to consider that the cache can read to the end of a CACHEPAGE (1K), but it turns out the cache can read to the end of a system page (usually 4K). Adjust the extent check to allow the larger system page size check here.
We've seen the MR extent check failures in nightly recently. I haven't been able to come up with a reliable reproducer yet, but I'm confident about this change just from code inspection. I believe we should be able to come up with a reproducer, but I want to get this into the release and I'm timing out coming up with the test case now.
Resolves Cray/chapel-private#3341
Modified Files:
M runtime/src/comm/ugni/comm-ugni.c
Compare: https://github.com/chapel-lang/chapel/compare/ba7dcfbe1fdb...616277d0794c