-
Notifications
You must be signed in to change notification settings - Fork 76
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
Fix #7314: Use mtime instead of ctime when tier2 FS is enabled. #7319
Conversation
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.
@Utkarsh-pro See comments, I don't have a concrete suggestion for renaming the use_mtime_check option, but I think it's a bit misleading as it is right now...
f05863c
to
518db32
Compare
518db32
to
a1da9ba
Compare
a1da9ba
to
d25e03c
Compare
a14e70d
to
b20ded9
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.
!!
b20ded9
to
01d9d3f
Compare
cleanup Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com> restructure Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com> remove use_mtime_check flag Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com> reformat ifs to conform to the style guide Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com> fix errenous mtime call Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com> Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
01d9d3f
to
2a0f07b
Compare
Explain the changes
This PR adds a simple flag in
fs_napi
which forces usingmtime
instead ofctime
wherever appropriate. At some places use of mtime is not appropriate so in that place the check is completely skipped.Fixes: #7314