RE: svn commit: r322076 - in head/sys: amd64/amd64 i386/i386 x86/include x86/x86

2017-08-09 Thread Dexuan Cui via svn-src-head
> Cc: Yanmin Qiao <ya...@microsoft.com>; Hongjiang Zhang > <honz...@microsoft.com> > Subject: Re: svn commit: r322076 - in head/sys: amd64/amd64 i386/i386 > x86/include x86/x86 > > On 08/09/2017 00:09, Dexuan Cui wrote: > > Hi jkim, > > The patch breaks Fre

Re: svn commit: r322076 - in head/sys: amd64/amd64 i386/i386 x86/include x86/x86

2017-08-09 Thread Jung-uk Kim
On 08/09/2017 00:09, Dexuan Cui wrote: > Hi jkim, > The patch breaks FreeBSD VM on Hyper-V. > > identify_hypervisor() is moved from identify_cpu() to an earlier place, but > the global > variable cpu_feature2 used by the function is still initialized in > identify_cpu(). > > I'm not sure

RE: svn commit: r322076 - in head/sys: amd64/amd64 i386/i386 x86/include x86/x86

2017-08-08 Thread Dexuan Cui via svn-src-head
Hi jkim, The patch breaks FreeBSD VM on Hyper-V. identify_hypervisor() is moved from identify_cpu() to an earlier place, but the global variable cpu_feature2 used by the function is still initialized in identify_cpu(). I'm not sure about the background of your patch. Can you please have a