Re: kern/39114: sys/arch/x86/x86/patch.c needs a minor #ifdef for PARALLELS (VT extensions problem)

["Jared D. McNeill" <jmcneill%invisible.ca@localhost>]

ggm%pobox.com@localhost wrote:
> > Number:         39114
> > Category:       kern
> > Synopsis:       sys/arch/x86/x86/patch.c needs a minor #ifdef for PARALLELS (VT 
> > extensions problem)
> > Confidential:   no
> > Severity:       non-critical
> > Priority:       low
> > Responsible:    kern-bug-people
> > State:          open
> > Class:          sw-bug
> > Submitter-Id:   net
> > Arrival-Date:   Tue Jul 08 02:05:00 +0000 2008
> > Originator:     George Michaelson
> > Release:        current
> > Organization:
> > Environment:
> NetBSD snoid 4.99.69 NetBSD 4.99.69 (GENERIC) #3: Mon Jul  7 20:34:51 EST 2008  
> ggm@snoid:/usr/obj/sys/arch/i386/compile/GENERIC i386
> > Description:
> Parallels on Mac has support for 'VT extensions' but when they hit a NetBSD 
> current kernel, some code in patch.c causes a hang during boot.
>
> You can disable the VT extensions, but thats a PITA. There is an existing #if 
> !defined(GPROF) on the code which causes the problem, but having to enable 
> GPROF to get a working kernel also has downsides.
>
> If this patch is acceptable, or something close to it, then a single new 
> Option/define in any kernel Makefile which sets -DPARALLELS will fix the 
> problem.

Hi George --

Newer Parallels versions should give reasonable data in DMI tables. Can 
you look into modifying your patch to automatically detect and disable 
this section of code based on that information rather than adding 
another compile-time option? See the pmf(4) man page for information on 
retrieving DMI information from the driver.

Cheers,
Jared