Page MenuHomeFreeBSD

pmcannotate: Add a '-m' mode option to control per-line annotations.
ClosedPublic

Authored by jhb on Jun 3 2022, 10:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 14, 7:54 PM
Unknown Object (File)
Nov 28 2024, 12:12 PM
Unknown Object (File)
Oct 3 2024, 7:54 AM
Unknown Object (File)
Sep 9 2024, 1:47 AM
Unknown Object (File)
Sep 8 2024, 6:04 PM
Unknown Object (File)
Sep 7 2024, 11:57 AM
Unknown Object (File)
Sep 4 2024, 10:49 PM
Unknown Object (File)
Aug 27 2024, 9:21 AM

Details

Summary
  • "block" (the default mode) displays per-block percentages in front of each annotated line.
  • "global" displays a global percentage in front of each line.
  • "raw" displays the raw count of sample hits in front of each line.

Sponsored by: University of Cambridge, Google, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Jun 3 2022, 10:04 PM

I have no objections, but I am fine with the current format. Global percents from the whole kernel would be a pretty small number. Raw number may be pretty big for a hot part of a small program. But whatever...

This revision is now accepted and ready to land.Jun 4 2022, 1:11 AM
pauamma_gundo.com added inline comments.
usr.sbin/pmcannotate/pmcannotate.8
88–98

What I think this means, after staring at it for a bit, is (using example numbers of 4 samples in a given line, 20 samples in the block - however defined - that line is in, and 80 samples altogether:

  • raw would display: 4
  • block would display; 20%
  • global would display: 1.25%

Is that what you mean?

usr.sbin/pmcannotate/pmcannotate.8
88–98
  • global would display: 1.25%

5%, not 1.25%.

usr.sbin/pmcannotate/pmcannotate.8
88–98

Yes, that is what it means.

pauamma_gundo.com added inline comments.
usr.sbin/pmcannotate/pmcannotate.8
88–98

Thanks for confirming.

93–96

Suggestion for clarity.

This revision now requires changes to proceed.Jun 5 2022, 1:11 AM
jhb marked an inline comment as done.Jun 6 2022, 11:55 PM
bcr added a subscriber: bcr.

OK from manpages.

This revision is now accepted and ready to land.Oct 13 2022, 4:45 PM