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

bugfix: load jl_n_threads in jl_gc_pool_live_bytes (#52034) #106

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

d-netto
Copy link
Member

@d-netto d-netto commented Nov 6, 2023

PR Description

Otherwise we may just observe gc_n_threads = 0 (jl_gc_collect sets it to 0 in the very end of its body) and this function becomes a no-op.

Checklist

Requirements for merging:

Otherwise we may just observe `gc_n_threads = 0` (`jl_gc_collect` sets
it to 0 in the very end of its body) and this function becomes a no-op.
@github-actions github-actions bot added port-to-v1.10 This change should apply to Julia v1.10 builds port-to-master This change should apply to all future Julia builds port-to-v1.9 This change should apply to Julia v1.9 builds labels Nov 6, 2023
@d-netto d-netto removed the port-to-master This change should apply to all future Julia builds label Nov 6, 2023
@d-netto d-netto requested a review from kpamnany November 6, 2023 11:03
@d-netto d-netto merged commit dffb365 into v1.9.2+RAI Nov 6, 2023
5 checks passed
@d-netto d-netto deleted the dcn-pool-live-bytes branch November 6, 2023 11:57
@kpamnany kpamnany removed the port-to-v1.10 This change should apply to Julia v1.10 builds label Nov 16, 2023
@Drvi Drvi added port-to-v1.9 This change should apply to Julia v1.9 builds and removed port-to-v1.9 This change should apply to Julia v1.9 builds labels Dec 7, 2023
Drvi pushed a commit that referenced this pull request Dec 7, 2023
…106)

Otherwise we may just observe `gc_n_threads = 0` (`jl_gc_collect` sets
it to 0 in the very end of its body) and this function becomes a no-op.
@d-netto d-netto removed the port-to-v1.9 This change should apply to Julia v1.9 builds label Jan 2, 2024
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.

3 participants