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

FAQ: add dlclose() essay from IRC #62

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

FAQ: add dlclose() essay from IRC #62

wants to merge 12 commits into from

Conversation

rofl0r
Copy link
Collaborator

@rofl0r rofl0r commented Oct 7, 2021

No description provided.

Copy link
Collaborator

@ericonr ericonr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think just dumping the IRC thread isn't ideal, it would be nice to turn it into a proper text. (my comments are a stab at that process)

faq.md Outdated Show resolved Hide resolved
Further, managing thread-local-storage lifetime is a lot more problematic when slots can be freed and reused. (actually a lot of ppl think this is why musl doesn't unload libraries, which isn't the case. it's possible to do right, just more work).
On glibc this actually makes for some weird race conditions where a library that 'could be' unloaded doesn't get unloaded, and unloading it either never happens or gets deferred to the next dlclose operation - which breaks anything assuming that dlclose will actually unload.
So programs making this assumption are already broken on glibc too, just with random rare failures rather than failure every time.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally I think it would be nice to keep the formatting/wrapping like the other answers.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the others are formatted to 80 chars line length, tho md rendering disregards newlines anyway - and as this faq is not meant to be read in a terminal i opted for the quick way...

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you use vim, just select the whole block and gq it. It will auto format the whole thing.

faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
faq.md Outdated Show resolved Hide resolved
rofl0r and others added 11 commits October 9, 2021 23:00
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Co-authored-by: Érico Nogueira Rolim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants