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

WIP-8797: fix python thread hanging bug #3053

Closed
wants to merge 1 commit into from

Conversation

joejulian
Copy link

sys.exit does not guarantee garbage collection in threads, possibly
preventing Rados.shutdown() from being called causing python to hang due
to http://bugs.python.org/issue21963 .

Signed-off-by: Joe Julian me@joejulian.name

sys.exit does not guarantee garbage collection in threads, possibly
preventing Rados.shutdown() from being called causing python to hang due
to http://bugs.python.org/issue21963 .

Signed-off-by: Joe Julian <me@joejulian.name>
@joejulian
Copy link
Author

This patch depends on 92615ea and cf2104d being cherry-picked to cure the SIGILL on calls to librados.shutdown.

@dmick

@ghost ghost assigned dmick Dec 2, 2014
@ghost ghost added bug-fix core labels Dec 2, 2014
@ghost ghost modified the milestones: dumpling, firefly Dec 2, 2014
@dmick
Copy link
Member

dmick commented Dec 2, 2014

I assert that this does not depend on the two commits referenced above, and can serve as a useful hardening against these Python versions that have this behavior (and Python3 probably does as well).

@danderson
Copy link
Contributor

Note that 0.87 is also affected by this bug. The proposed patch applies cleanly to the 0.87 source tree, and fixes the hang-on-exit for Arch Linux's Ceph 0.87 package.

Agreed with @dmick that this patch doesn't depend on the SIGILL fix, though both sets should be backported to 0.87.

@joejulian joejulian closed this Dec 8, 2014
@dmick
Copy link
Member

dmick commented Dec 9, 2014

See #3119

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants