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.

Sign up for GitHub

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

Jump to bottom

Improve information displayed on threads by versioninfo #52404

Closed
nilshg opened this issue Dec 5, 2023 · 0 comments · Fixed by #52423
Closed

Improve information displayed on threads by versioninfo #52404

nilshg opened this issue Dec 5, 2023 · 0 comments · Fixed by #52423

Comments

@nilshg
Copy link
Contributor

nilshg commented Dec 5, 2023

Current 1.10-rc2 shows:

C:\>julia -q --threads=auto --startup-file=no -e "using InteractiveUtils; versioninfo()"

Julia Version 1.10.0-rc2
Commit dbb9c46795 (2023-12-03 15:25 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: 8 × 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-15.0.7 (ORCJIT, tigerlake)
  Threads: 11 on 8 virtual cores

I've learned on Slack that the surprisingly high number of threads is due to additional GC threads and that therefore the CPU isn't actually oversubscribed.

A related issue is #48580 which complained about nthreads() giving unexpected results since the introduction of interactive threads. To my mind ideally versioninfo() (and related places/functions which give threads info) should say something like:

Threads:
    - Default: 8
    - Interactive: 1
    - GC: 3
Number of virtual cores: 8
@IanButterworth IanButterworth mentioned this issue Dec 6, 2023
Merged
IanButterworth added a commit that referenced this issue Dec 28, 2023
@IanButterworth
update nthreads info in versioninfo (#52423)
e96c13a 
Fixes #52404
@nilshg I opted to make it one line as it fits.
```
julia> versioninfo()
Julia Version 1.11.0-DEV.1011
Commit bb7091c* (2023-12-04 14:58 UTC)
Platform Info:
  OS: macOS (arm64-apple-darwin23.0.0)
  CPU: 10 × Apple M2 Pro
  WORD_SIZE: 64
  LLVM: libLLVM-15.0.7 (ORCJIT, apple-m1)
  Threads: 1 default, 0 interactive, 1 GC (on 6 virtual cores)
Environment:
  JULIA_EDITOR = code
```
KristofferC pushed a commit that referenced this issue Jan 5, 2024
@IanButterworth
update nthreads info in versioninfo (#52423)
a2ffa91 
Fixes #52404
@nilshg I opted to make it one line as it fits.
```
julia> versioninfo()
Julia Version 1.11.0-DEV.1011
Commit bb7091c* (2023-12-04 14:58 UTC)
Platform Info:
  OS: macOS (arm64-apple-darwin23.0.0)
  CPU: 10 × Apple M2 Pro
  WORD_SIZE: 64
  LLVM: libLLVM-15.0.7 (ORCJIT, apple-m1)
  Threads: 1 default, 0 interactive, 1 GC (on 6 virtual cores)
Environment:
  JULIA_EDITOR = code
```

(cherry picked from commit e96c13a)
KristofferC pushed a commit that referenced this issue Jan 5, 2024
@IanButterworth
update nthreads info in versioninfo (#52423)
79684ad 
Fixes #52404
@nilshg I opted to make it one line as it fits.
```
julia> versioninfo()
Julia Version 1.11.0-DEV.1011
Commit bb7091c* (2023-12-04 14:58 UTC)
Platform Info:
  OS: macOS (arm64-apple-darwin23.0.0)
  CPU: 10 × Apple M2 Pro
  WORD_SIZE: 64
  LLVM: libLLVM-15.0.7 (ORCJIT, apple-m1)
  Threads: 1 default, 0 interactive, 1 GC (on 6 virtual cores)
Environment:
  JULIA_EDITOR = code
```

(cherry picked from commit e96c13a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update nthreads info in versioninfo IanButterworth/julia
1 participant
@nilshg