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)
Wed, May 6, 4:25 AM
Unknown Object (File)
Tue, May 5, 11:41 AM
Unknown Object (File)
Wed, Apr 29, 12:50 AM
Unknown Object (File)
Tue, Apr 28, 11:49 AM
Unknown Object (File)
Mon, Apr 27, 6:09 AM
Unknown Object (File)
Apr 23 2026, 3:07 AM
Unknown Object (File)
Apr 20 2026, 3:19 AM
Unknown Object (File)
Apr 18 2026, 10:33 PM
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.