Skip to content

hwpmc: Promote hwpmc EXTERROR diagnostics#2180

Open
afscoelho wants to merge 1 commit into
freebsd:mainfrom
AMDESE:pr/pmc/hwpmc-exterr
Open

hwpmc: Promote hwpmc EXTERROR diagnostics#2180
afscoelho wants to merge 1 commit into
freebsd:mainfrom
AMDESE:pr/pmc/hwpmc-exterr

Conversation

@afscoelho
Copy link
Copy Markdown
Contributor

Hi - this promotes the hwpmc extended-error work into upstream. Highlights:

  • give the AMD core and IBS PMCs better extended-error reporting so users actually see the misconfig cases that come up
  • add a root-only Kyua check (hwpmc_exterr_test) that walks through the typical negative scenarios

Testing

  • make -j${NCPU} buildworld
  • make -j${NCPU} buildkernel KERNCONF=${KERNCONF}
  • kyua test -k /usr/tests/Kyuafile hwpmc_exterr_test

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Thank you for taking the time to contribute to FreeBSD!

Some of files have special handling:

Important

@ngie-eign wants to review changes to tests

@afscoelho afscoelho changed the title Promote hwpmc EXTERROR diagnostics hwpmc: Promote hwpmc EXTERROR diagnostics May 6, 2026
@afscoelho afscoelho force-pushed the pr/pmc/hwpmc-exterr branch 3 times, most recently from d81b7c5 to 34314c4 Compare May 6, 2026 01:41
@afscoelho
Copy link
Copy Markdown
Contributor Author

Hi @mashtizadeh / @mhorne, just a continuation of the first PR that introduced exterror. Thanks for your time

@afscoelho
Copy link
Copy Markdown
Contributor Author

@mashtizadeh / @mhorne

Copy link
Copy Markdown
Contributor

@mashtizadeh mashtizadeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you check the output for a DF counter with a reserved bit set? If it shows all those errors multiple times we have a problem then.

Comment thread sys/dev/hwpmc/hwpmc_ibs.c
Comment thread sys/dev/hwpmc/hwpmc_ibs.c Outdated
Comment thread sys/dev/hwpmc/hwpmc_amd.c Outdated
Add EXTERROR coverage for hwpmc_amd, hwpmc_ibs, hwpmc_mod
user-visible validation paths. Add root-only ATF regression
test for representative negative cases.

Sponsored by:  AMD
Signed-off-by: Andre Silva <andasilv@amd.com>
@afscoelho afscoelho force-pushed the pr/pmc/hwpmc-exterr branch from 34314c4 to 16b89cb Compare May 16, 2026 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants