amdtemp - Fix family 0x17 support, add family 0x19 support (2)

Operating Systems / DragonFlyBSD - Matthew Dillon [apollo.backplane.com] - 30 May 2021 06:50 UTC

- Get rid of debugging kprintfs and the family 0x18 sillyness I had added earlier.

- Unfortunately, there are still a multitude of problems with this driver. Not only is the ncores supposed to be per-unit (and isn't, its whole-system), but the amdtemp driver attaches four times on the 3990X (units 0, 1, 2, 3) because there are four root complexes. Each attachment lists 8 CCDs. And, insofar as I can tell, each of the four attachments are reading the same sensors even though they are hanging off of four different root complexes.

(This is in addition to the driver not knowing how to map temp sensors to CPUs).

206afc23f9 amdtemp - Fix family 0x17 support, add family 0x19 support (2)
sys/dev/powermng/amdtemp/amdtemp.c | 5 -----
1 file changed, 5 deletions(-)

Upstream: gitweb.dragonflybsd.org


  • Share