Quiet coretemp probe
ClosedPublic
Actions

Authored by kbowling on Jun 10 2018, 1:35 AM.

Tags

None

Referenced Files

	F106174178: D15727.diff
	Thu, Dec 26, 2:59 PM

	F106159523: D15727.id43795.diff
	Thu, Dec 26, 9:08 AM

	Unknown Object (File)
	Sat, Dec 21, 7:52 AM

	Unknown Object (File)
	Thu, Nov 28, 4:49 AM

	Unknown Object (File)
	Nov 13 2024, 6:32 PM

	Unknown Object (File)
	Oct 9 2024, 7:00 AM

	Unknown Object (File)
	Oct 3 2024, 5:09 AM

	Unknown Object (File)
	Sep 18 2024, 5:13 AM

View All 47 Files

Subscribers

imp

Details

Reviewers

mmacy
sbruno
imp

Commits

rS335186: Quiet coretemp probe

Summary

Only the first device will print
coretemp0: <CPU On-Die Thermal Sensors> numa-domain 0 on cpu0
instead of all hyper threads

I thought about printing once per package but there is no simple way to do that and I'm not sure the complexity is worth it

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kbowling created this revision.Jun 10 2018, 1:35 AM

Herald added a subscriber: imp. · View Herald TranscriptJun 10 2018, 1:35 AM

I'm surprised we need this since I set all CPUs to quiet children if they aren't CPU0.

sys/dev/coretemp/coretemp.c
150 ↗	(On Diff #43532)	This is similar to a change that I made to the CPU code.... But the CPU code had device_set_quiet_children that I thought would have taken care of this...

This revision is now accepted and ready to land.Jun 10 2018, 1:51 AM

sbruno accepted this revision.Jun 12 2018, 7:46 PM

Closed by commit rS335186: Quiet coretemp probe (authored by mmacy). · Explain WhyJun 15 2018, 2:28 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

coretemp/

coretemp.c

3 lines

Diff 43795

View Options

head/sys/dev/coretemp/coretemp.c

Quiet coretemp probeClosedPublicActions