-
Notifications
You must be signed in to change notification settings - Fork 304
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
DAOS-14824 dfuse: Allow parallel readdir calls. #13606
Conversation
Remove the inode lock preventing parallel readdir calls. Update readdir caching to be thread safe. Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Bug-tracker data: |
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13606/1/testReport/ |
Test stage Functional Hardware Medium completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13606/1/testReport/ |
Test stage Functional Hardware Large completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13606/2/execution/node/1475/log |
Test stage Functional Hardware Medium completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13606/2/testReport/ |
Required-githooks: true
Features: dfuse Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Test stage Functional Hardware Medium completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13606/3/testReport/ |
Test-tag: dfuse Required-githooks: true
Features: dfuse
Features: dfuse Required-githooks: true
Test stage Functional Hardware Medium completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13606/6/execution/node/1481/log |
This ran and passed all the testing but that's mostly pointless as due to a libfuse bug then the kernel feature isn't enabled in our testing. I'm separately working on a test for this which shows significant performance improvement locally but again that won't show any change in performance or exercise this new code until we can get a new libfuse in CI so it should not block this (although I can see that the libfuse update itself might). |
Features: dfuse Required-githooks: true
Remove the inode lock preventing parallel readdir calls.
Update readdir caching to be thread safe.
Required-githooks: true
Signed-off-by: Ashley Pittman [email protected]