Skip to content

report: add cpu info to report output

Rodrigo Muino Tomonari requested to merge github/fork/boneskull/process-report-cpus into master

The report shows CPU consumption %, but without the number of CPU cores, a consumer cannot tell if the percent (given across all cores) is actually problematic. E.g., 100% on one CPU is a problem, but 100% on four CPUs is not necessarily (EDIT: the max theoretical usage is 400% in this case)

This change adds CPU information (similar to os.cpus()) to the report output. Extra info besides the count is also provided as to avoid future breaking changes in the eventuality that someone needs it; changing the datatype of header.cpus would be breaking.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports