Page MenuHomeFreeBSD

powerpc: Fix regression introduced in r342771
ClosedPublic

Authored by cem on Jan 7 2019, 6:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 25, 3:55 AM
Unknown Object (File)
Mon, Dec 16, 9:45 AM
Unknown Object (File)
Thu, Dec 5, 3:30 AM
Unknown Object (File)
Wed, Dec 4, 12:09 PM
Unknown Object (File)
Wed, Dec 4, 12:09 PM
Unknown Object (File)
Wed, Dec 4, 7:27 AM
Unknown Object (File)
Wed, Dec 4, 7:21 AM
Unknown Object (File)
Sun, Dec 1, 11:34 PM
Subscribers

Details

Summary

In r342771, I introduced a regression in Power by abusing the platform
smp_topo() method as a shortcut for providing the MI information needed for
the stated sysctls. The smp_topo() method was already called later by
sched_ule (under the name cpu_topo()), and initializes a static array of
scheduler topology information. I had skimmed the smp_topo_foo() functions
and assumed they were idempotent; empirically, they are not (or at least,
detect re-initialization and panic).

Do the cleaner thing I should have done in the first place and add a
platform method specifically for core- and thread-count probing.

Reported by: luporl via jhibbits
X-MFC-With: r342771

Test Plan

Tinderbox ppc kernel compiled, but I do not have PPC equipment to test with.
Hoping for help from luporl or jhibbits or anyone :-).

(luporl@, I'd encourage you to change your Phabricator identity to 'luporl' to
match your committer address to make it easier to tag you in reviews :-))

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Tested on my POWER8 machine.

(cem@, I started working on changing my Phabricator identity to 'luporl', thanks for the tip :))

This revision is now accepted and ready to land.Jan 7 2019, 7:30 PM

Thanks for the quick review and test!

This revision was automatically updated to reflect the committed changes.