hwpstate_intel: don't unconditionally print the error message
ClosedPublic
Actions

Authored by yuripv on May 7 2020, 3:43 AM.

Tags

None

Referenced Files

	F160318638: D24744.id80106.diff
	Tue, Jun 23, 5:38 AM

	Unknown Object (File)
	Sun, Jun 21, 7:47 PM

	Unknown Object (File)
	Mon, Jun 15, 4:23 PM

	Unknown Object (File)
	Mon, Jun 15, 4:22 PM

	Unknown Object (File)
	Sat, Jun 13, 1:29 PM

	Unknown Object (File)
	Sat, Jun 13, 6:26 AM

	Unknown Object (File)
	Fri, Jun 12, 9:15 PM

	Unknown Object (File)
	Wed, Jun 10, 2:26 PM

View All Files

Subscribers

imp

Details

Reviewers

cem

Commits

rS368140: hwpstate_intel: don't unconditionally print the error message

Summary

If CPU supports PKG-wide control MSR, hwpstate_intel unconditionally prints the "Failed to set autonomous HWP for package" error message, forgetting to check the actual wrmsr_safe() return value.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

yuripv created this revision.May 7 2020, 3:43 AM

Herald added a subscriber: imp. · View Herald TranscriptMay 7 2020, 3:43 AM

yuripv requested review of this revision.May 7 2020, 3:43 AM

This revision was not accepted when it landed; it landed in state Needs Review.Nov 29 2020, 1:43 AM

Closed by commit rS368140: hwpstate_intel: don't unconditionally print the error message (authored by yuripv). · Explain Why

This revision was automatically updated to reflect the committed changes.

yuripv added a commit: rS368140: hwpstate_intel: don't unconditionally print the error message.

Revision Contents
Changeset List

Path

Size

head/

sys/

x86/

cpufreq/

hwpstate_intel.c

6 lines

Diff 80106

View Options

head/sys/x86/cpufreq/hwpstate_intel.c

hwpstate_intel: don't unconditionally print the error messageClosedPublicActions