Differential D5714

hyperv/utils: Allow hint to disable individual utility
ClosedPublic
Actions

Authored by sepherosa_gmail.com on Mar 23 2016, 2:12 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Aug 9, 6:36 AM

	Unknown Object (File)
	Tue, Aug 4, 6:22 AM

	Unknown Object (File)
	Tue, Aug 4, 3:02 AM

	Unknown Object (File)
	Mon, Aug 3, 3:26 AM

	Unknown Object (File)
	Thu, Jul 23, 3:03 AM

	Unknown Object (File)
	Sat, Jul 18, 7:12 PM

	Unknown Object (File)
	Jul 6 2026, 11:59 PM

	Unknown Object (File)
	Jul 4 2026, 11:13 AM

Subscribers

None

Details

Reviewers

kib
howard0su_gmail.com
honzhan_microsoft.com
decui_microsoft.com
delphij
royger
adrian

Commits

rS297220: hyperv/utils: Allow hint to disable individual utility

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

sepherosa_gmail.com updated this revision to Diff 14536.Mar 23 2016, 2:12 AM

sepherosa_gmail.com retitled this revision from to hyperv/utils: Allow hint to disable individual utility.

sepherosa_gmail.com updated this object.

sepherosa_gmail.com edited the test plan for this revision. (Show Details)

sepherosa_gmail.com added reviewers: adrian, delphij, royger, kib, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com.

LGTM

This revision is now accepted and ready to land.Mar 23 2016, 2:56 AM

kib added inline comments.Mar 23 2016, 10:19 AM

sys/dev/hyperv/utilities/hv_heartbeat.c
98 ↗	(On Diff #14536)	Might be, use more structural names, like hyperv.heartbeat etc ?
99 ↗	(On Diff #14536)	Note that style requires braces () around the return value.

sepherosa_gmail.com added inline comments.Mar 24 2016, 12:45 AM

sys/dev/hyperv/utilities/hv_heartbeat.c
98 ↗	(On Diff #14536)	Not 100% sure about the practice, but looks like most users of resource_disabled() just use the driver's name
99 ↗	(On Diff #14536)	Hmm, this tries to make the style consistent within this function :)

Closed by commit rS297220: hyperv/utils: Allow hint to disable individual utility (authored by sephe). · Explain WhyMar 24 2016, 1:12 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

hyperv/

utilities/

4 lines

4 lines

4 lines

4 lines

Diff 14564

head/sys/dev/hyperv/utilities/hv_heartbeat.c

Loading...

head/sys/dev/hyperv/utilities/hv_kvp.c

Loading...

head/sys/dev/hyperv/utilities/hv_shutdown.c

Loading...

head/sys/dev/hyperv/utilities/hv_timesync.c

Loading...