-
Notifications
You must be signed in to change notification settings - Fork 193
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
Optimize blobfs implementation #1295
Conversation
Use rwlock instead of mutex for blobfs, to avoid serializaiton. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Cache blobfs inode information to avoid opening file on every dax window operations. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Enhance blobfs to support read() operation, in addition to DAX. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
@jiangliu , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/74466 |
Codecov Report
@@ Coverage Diff @@
## master #1295 +/- ##
==========================================
+ Coverage 45.70% 45.71% +0.01%
==========================================
Files 124 124
Lines 36884 36884
Branches 36884 36884
==========================================
+ Hits 16857 16862 +5
+ Misses 19138 19131 -7
- Partials 889 891 +2 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
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.
Looks good.
Relevant Issue (if applicable)
If there are Issues related to this PullRequest, please list it.
Details
Optimize blobfs implementation by:
Types of changes
What types of changes does your PullRequest introduce? Put an
x
in all the boxes that apply:Checklist
Go over all the following points, and put an
x
in all the boxes that apply.