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

client: call the lru_remove() twice,when trim cache #15662

Merged
merged 1 commit into from Jun 21, 2017

Conversation

Projects
None yet
4 participants
@renhwztetecs
Member

renhwztetecs commented Jun 13, 2017

call the lru_remove() twice,when trim cache,
first: trim_cache()---> lru_expire() ---> lru_remove();
second: trim_cache()---> trim_dentry() ---> unlink() ---> lru_remove().
fix as:
In access to LRU's Dentry information, the first does not remove the operation,
unified in the trim_dentry() to deal with more reasonable.

Signed-off-by: huanwen ren ren.huanwen@zte.com.cn

client: call the lru_remove() twice,when trim cache
call the lru_remove() twice,when trim cache, 
first:  trim_cache()---> lru_expire() ---> lru_remove();
second: trim_cache()---> trim_dentry() ---> unlink() ---> lru_remove().
fix as: 
In access to lru's dentry information, the first does not remove the operation, 
unified in the trim_dentry function to deal with more reasonable.

Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>

@renhwztetecs renhwztetecs requested review from jcsp and ukernel Jun 13, 2017

@liewegas liewegas added the cephfs label Jun 13, 2017

@jcsp jcsp merged commit 32ff1de into ceph:master Jun 21, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@renhwztetecs renhwztetecs deleted the renhwztetecs:renhw-wip-client-trim_cache branch Jun 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment