Page MenuHomeFreeBSD
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)
Sep 29 2024, 11:24 PM
Unknown Object (File)
Sep 25 2024, 9:55 AM
Unknown Object (File)
Sep 24 2024, 11:35 PM
Unknown Object (File)
Sep 24 2024, 4:15 PM
Unknown Object (File)
Sep 24 2024, 1:06 AM
Unknown Object (File)
Sep 23 2024, 7:26 PM
Unknown Object (File)
Sep 23 2024, 4:05 AM
Unknown Object (File)
Sep 22 2024, 7:15 PM
View All 67 Files
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

Lint
Lint Skipped
Unit
Tests Skipped

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.
decui_microsoft.com accepted this revision.Mar 23 2016, 2:56 AM
decui_microsoft.com edited edge metadata.

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

Might be, use more structural names, like hyperv.heartbeat etc ?

99

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

Not 100% sure about the practice, but looks like most users of resource_disabled() just use the driver's name

99

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

PathSize
sys/
dev/
hyperv/
utilities/
4 lines
4 lines
4 lines
4 lines

Diff 14536

View Options

sys/dev/hyperv/utilities/hv_heartbeat.c

Loading...
View Options

sys/dev/hyperv/utilities/hv_kvp.c

Loading...
View Options

sys/dev/hyperv/utilities/hv_shutdown.c

Loading...
View Options

sys/dev/hyperv/utilities/hv_timesync.c

Loading...