Skip to content

client/libcephfs: Don't let one unmount screw up another mount#38858

Merged
batrick merged 1 commit intoceph:masterfrom
adamemerson:wip-47294
Jan 14, 2021
Merged

client/libcephfs: Don't let one unmount screw up another mount#38858
batrick merged 1 commit intoceph:masterfrom
adamemerson:wip-47294

Conversation

@adamemerson
Copy link
Contributor

Share threadpools between mounts. Don't shut them down until the last
user goes away, and make sure to start it back up when needed.

Fixes: https://tracker.ceph.com/issues/47294
For real this time.
Maybe.

@adamemerson adamemerson requested a review from batrick January 12, 2021 02:03
@github-actions github-actions bot added the cephfs Ceph File System label Jan 12, 2021
@adamemerson
Copy link
Contributor Author

@adamemerson
Copy link
Contributor Author

https://pulpito.ceph.com/aemerson-2021-01-12_22:32:47-fs:volumes-wip-47294-distro-basic-smithi/

A run with the update. Though that the other one worked suggests this was the root cause.

Share threadpools between mounts. Don't shut them down until the last
user goes away, and make sure to start it back up when needed.

Fixes: https://tracker.ceph.com/issues/47294
For real this time.
Maybe.
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
@adamemerson
Copy link
Contributor Author

@batrick
Copy link
Member

batrick commented Jan 14, 2021

Thanks @adamemerson !

@batrick batrick merged commit 855c134 into ceph:master Jan 14, 2021
@adamemerson adamemerson deleted the wip-47294 branch January 14, 2021 17:52
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

Development

Successfully merging this pull request may close these issues.

2 participants