Page MenuHomeFreeBSD

Make the CPU width a macro
Needs ReviewPublic

Authored by andrew on Aug 25 2022, 10:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 6, 11:01 PM
Unknown Object (File)
Sun, Sep 6, 10:49 PM
Unknown Object (File)
Aug 5 2026, 5:04 AM
Unknown Object (File)
Jul 25 2026, 3:19 PM
Unknown Object (File)
Jul 25 2026, 1:34 PM
Unknown Object (File)
Jul 23 2026, 7:23 AM
Unknown Object (File)
Jul 18 2026, 8:32 AM
Unknown Object (File)
Jul 17 2026, 10:12 AM
Subscribers

Details

Reviewers
None
Group Reviewers
pmc
Summary

This allows us to increase it later when 1000 core machines are
available. For now make it 3 digits to support the current > 99 core
machines.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 47126
Build 44013: arc lint + arc unit

Event Timeline

bz added inline comments.
usr.sbin/pmcstat/pmcstat.c
123

This may sound weird, but how far are we off from 4 digits? I understand that in academia 1088 core RISC-V exists...

123

also PMC_CPU_STR_WIDTH as it CPU_WIDTH could easily be confused with 32 vs. 64 vs. 128 bit?

emaste added inline comments.
usr.sbin/pmcstat/pmcstat.c
123

Not too far; I expect to have 768 in my office shortly and will almost certainly have > 1000 during 15.x's lifetime. But committing this as a NFC first (with the macro staying at 3) makes sense to me.