Skip to content
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

pacific: client: readdir_r_cb: get rstat for dir only if using rbytes for size #54179

Closed
wants to merge 2 commits into from

Conversation

rishabh-d-dave
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/62003


backport of #46960
parent tracker: https://tracker.ceph.com/issues/61999

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

When `client_dirsize_rbytes` is off, there should be no need for getting
rstat on readdir operations.

This fixes performance when `client_dirsize_rbytes` is off after ceph#38222.

Fixes: https://tracker.ceph.com/issues/61999
Signed-off-by: Pinghao Wu <xdavidwuph@gmail.com>
(cherry picked from commit 69decab)
kclient also has this option, and is unchangeable unless remounted.
Let's keep parity with kclient.

Signed-off-by: Pinghao Wu <xdavidwuph@gmail.com>
(cherry picked from commit 05a48a4)
@rishabh-d-dave rishabh-d-dave added this to the pacific milestone Oct 25, 2023
@rishabh-d-dave rishabh-d-dave added the cephfs Ceph File System label Oct 25, 2023
@vshankar
Copy link
Contributor

vshankar commented Nov 8, 2023

Not a blocker/urgent for last pacific relase - closing.

@vshankar vshankar closed this Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System common
Projects
None yet
3 participants