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

mimic: cephfs: client: bump ll_ref from int32 to uint64_t #29187

Merged
merged 1 commit into from Jul 25, 2019

Conversation

@xiaoxichen
Copy link
Contributor

xiaoxichen commented Jul 23, 2019

ll_ref can surge up if application repeatly lookup same dentry
and finally cause overflow.

https://tracker.ceph.com/issues/40775 is an example.

in kernel fuse driver and libfuse, uint64_t is used, update
ceph-fuse side to match.

Fixes: https://tracker.ceph.com/issues/40775

Signed-off-by: Xiaoxi CHEN <xiaoxchen@ebay.com>
(cherry picked from commit bb46de8)

 Conflicts:
	src/client/Client.h
        use new code, skip all non-toched changes.
	src/client/Inode.h
        use new code.
@xiaoxichen xiaoxichen added this to the mimic milestone Jul 23, 2019
@xiaoxichen xiaoxichen requested a review from batrick Jul 23, 2019
@xiaoxichen xiaoxichen self-assigned this Jul 23, 2019
@xiaoxichen xiaoxichen requested a review from smithfarm Jul 23, 2019
@smithfarm smithfarm changed the title Mimic: Client: bump ll_ref from int32 to uint64_t mimic: cephfs: client: bump ll_ref from int32 to uint64_t Jul 23, 2019
@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Jul 23, 2019

jenkins test docs

@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Jul 23, 2019

@yuriw yuriw merged commit adfdd00 into ceph:mimic Jul 25, 2019
4 checks passed
4 checks passed
Docs: build check OK - docs built
Details
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.