-
Notifications
You must be signed in to change notification settings - Fork 218
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
Reduce duplicated code by using utils/filemap #840
Conversation
@jiangliu , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31134 |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31135 |
4edddc4
to
63f78d6
Compare
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31136 |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31137 |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31138 |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31139 |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31186 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
Still have more pending patch for this, hold on:) |
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31225 |
Use the generic filemap from nydus-utils crate to avoid duplicated code. Also reduce unsafe code, now there's only two unsafe left in direct v5. Fix a lifetime related by xattr related code. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Implement chunk map with FileMap to reduce duplicated code. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Get rid of unused digest validation code from RAFS v6. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Fix a bug in impl Clone for DirectSuperBlockV6, also reduce deep memory copy. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
Use filemap to reduce duplicated for direct v6. Signed-off-by: Jiang Liu <gerry@linux.alibaba.com>
@jiangliu , the code has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/31226 |
@jiangliu , The CI test is completed, please check result:
Congratulations, your test job passed! |
Use the generic filemap from nydus-utils crate to avoid duplicated code.
Also reduce unsafe code, now there's only two unsafe left in direct v5.
Fix a lifetime related by xattr related code.
Signed-off-by: Jiang Liu gerry@linux.alibaba.com