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

client: fix segment fault in Client::_invalidate_kernel_dcache(). #11170

Merged
merged 1 commit into from Sep 23, 2016

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented Sep 21, 2016

when umounting, root can be NULL

Fixes: http://tracker.ceph.com/issues/17253
Signed-off-by: Yan, Zheng zyan@redhat.com

when umounting, root can be NULL

Fixes: http://tracker.ceph.com/issues/17253
Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel ukernel added bug-fix cephfs Ceph File System labels Sep 21, 2016
@jcsp
Copy link
Contributor

jcsp commented Sep 22, 2016

I'm seeing an instance of ceph-fuse failing to terminate when running with this patch:
http://pulpito.ceph.com/jspray-2016-09-21_18:37:24-fs-wip-jcsp-greenish-distro-basic-smithi/429512/

@ukernel
Copy link
Contributor Author

ukernel commented Sep 22, 2016

2016-09-21 22:20:38.765059 7fe7baf92f00  1 client.4139 dump_inode: DISCONNECTED inode 1000000dd1c #1000000dd1c ref 11000000dd1c.head(faked_ino=0 ref=1 ll_ref=0 cap_refs={1024=0} open={} mode=40755 size=0/0 mtime=2016-09-21 22:00:40.349543 caps=-(0=pAsLsXsFsx) 0x7fe7cb640000)

this hang was caused by inode refernece leak, it's unlikely caused by this RP

@jcsp
Copy link
Contributor

jcsp commented Sep 23, 2016

OK, now I've seen the same failure in a run without this patch: http://pulpito.ceph.com/jspray-2016-09-23_10:14:18-fs-wip-jcsp-greenish-distro-basic-mira/432687/

@jcsp jcsp merged commit 8ddb7fe into ceph:master Sep 23, 2016
@ukernel ukernel deleted the wip-17253 branch January 12, 2017 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
2 participants