Page MenuHomeFreeBSD
Differential D9242

lib/libc/x86/sys/__vdso_gettc.c: Conditionalize hyperv support based on MK_HYPERV
ClosedPublic
Actions

Authored by ngie on Jan 19 2017, 8:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 30, 3:32 PM
Unknown Object (File)
Sat, Nov 16, 3:40 AM
Unknown Object (File)
Nov 12 2024, 8:27 PM
Unknown Object (File)
Nov 12 2024, 5:34 PM
Unknown Object (File)
Nov 12 2024, 11:29 AM
Unknown Object (File)
Nov 11 2024, 7:31 PM
Unknown Object (File)
Nov 11 2024, 5:58 PM
Unknown Object (File)
Nov 11 2024, 5:35 PM
View All 67 Files
Subscribers
imp
kib

Details

Reviewers
sepherosa_gmail.com
Commits
rS312418: Conditionalize hyperv support in gettimeofday(2) based on MK_HYPERV
Summary

This conditionalizes hyperv support in gettimeofday based on MK_HYPERV, so systems where the administrator has made a conscious decision not to build hyperv support in the system will not be hit by the hyperv path.

MFC after: 3 weeks
Sponsored by: Dell EMC Isilon

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ngie updated this revision to Diff 24188.Jan 19 2017, 8:08 AM
ngie retitled this revision from to lib/libc/x86/sys/__vdso_gettc.c: Conditionalize hyperv support based on MK_HYPERV.
ngie updated this object.
ngie edited the test plan for this revision. (Show Details)
ngie added a reviewer: sepherosa_gmail.com.
ngie set the repository for this revision to rS FreeBSD src repository - subversion.
ngie added a subscriber: kib.
Herald added a subscriber: imp. · View Herald TranscriptJan 19 2017, 8:08 AM
ngie updated this revision to Diff 24189.Jan 19 2017, 8:09 AM

Tweak a comment marking the end of the WANT_HYPERV #endif .

sepherosa_gmail.com edited edge metadata.Jan 19 2017, 8:13 AM

I don't see why it is needed. If hyperv is not in kernel or the ref rdtsc is not the current timecounter, this path will never be hit. Or do you have other concerns?

ngie added a comment.Jan 19 2017, 8:20 AM
In D9242#191080, @sepherosa_gmail.com wrote:

I don't see why it is needed. If hyperv is not in kernel or the ref rdtsc is not the current timecounter, this path will never be hit. Or do you have other concerns?

I just saw that it only hits this case if the time counter is enabled, and after that only hits it if opening the character device succeeds.

It's not an issue today because MK_HYPERV != no doesn't control hyperv header installation, but if that ever changes (and I would kind of like that to be honest), then this would be an issue.

So, the reasoning I claimed here originally might be a bit misleading, but this unfortunately stands in the way of removing the hyperv headers from the system when MK_HYPERV == no.

sepherosa_gmail.com accepted this revision.Jan 19 2017, 8:26 AM
sepherosa_gmail.com edited edge metadata.

OK, I see. Please fix the commit message when committing then. BTW, in addition to include/dev/hyperv, there are usr.sbin/hyperv and several manpages that you probably want to manage w/ MK_HYPERV.

This revision is now accepted and ready to land.Jan 19 2017, 8:26 AM
sepherosa_gmail.com added inline comments.Jan 19 2017, 8:27 AM
lib/libc/x86/sys/__vdso_gettc.c
220 ↗(On Diff #24189)

WANT_HYPERV, i.e. no __ prefix.

ngie marked an inline comment as done.Jan 19 2017, 5:00 PM
ngie added inline comments.
lib/libc/x86/sys/__vdso_gettc.c
220 ↗(On Diff #24189)

Thanks :)!

Closed by commit rS312418: Conditionalize hyperv support in gettimeofday(2) based on MK_HYPERV (authored by ngie). · Explain WhyJan 19 2017, 5:03 PM
This revision was automatically updated to reflect the committed changes.
ngie marked an inline comment as done.

Revision Contents
Changeset List

PathSize
head/
lib/
libc/
x86/
sys/
4 lines
8 lines

Diff 24198

View Options

head/lib/libc/x86/sys/Makefile.inc

Loading...
View Options

head/lib/libc/x86/sys/__vdso_gettc.c

Loading...