Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Apr 2011 15:33:11 -0700
From:      Garrett Cooper <yanegomi@gmail.com>
To:        Jung-uk Kim <jkim@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Dimitry Andric <dim@freebsd.org>
Subject:   Re: svn commit: r220584 - in head/sys: amd64/amd64 i386/i386
Message-ID:  <BANLkTinOV60xH_iXMAnE-oQ9P=P67d4-%2BQ@mail.gmail.com>
In-Reply-To: <201104131827.39373.jkim@FreeBSD.org>
References:  <201104122349.p3CNn7kK039179@svn.freebsd.org> <4DA6189A.5040200@FreeBSD.org> <4DA61A70.8040609@FreeBSD.org> <201104131827.39373.jkim@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Apr 13, 2011 at 3:27 PM, Jung-uk Kim <jkim@freebsd.org> wrote:
> On Wednesday 13 April 2011 05:49 pm, Dimitry Andric wrote:
>> On 2011-04-13 23:41, Dimitry Andric wrote:
>> ...
>>
>> > But I don't really see why, yet. :) =A0With r220532, it worked
>> > fine.
>>
>> Ah, I failed to notice the commit that came before, r220583.
>> Apparently, it can happen (at least in a VM environment) that the
>> DELAY(1000) in this fragment from cpu_est_clockrate():
>>
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0wrmsr(MSR_MPERF, 0);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0wrmsr(MSR_APERF, 0);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0tsc1 =3D rdtsc();
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0DELAY(1000);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0mcnt =3D rdmsr(MSR_MPERF);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0acnt =3D rdmsr(MSR_APERF);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0tsc2 =3D rdtsc();
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0intr_restore(reg);
>> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0perf =3D 1000 * acnt / mcnt;
>>
>> will still read 0 from MSR_MPERF, leading to a division by zero.
>> Maybe just fallback to the second method in the 'else' branch then?
>
> That means your VM has broken CPUID support. =A0To get there, it has to
> meet two conditions, i.e., TSC is invariant and it has APERF/MPERF
> MSRs. =A0A simple workaround is setting "machdep.disable_tsc=3D1"
> tuanable from loader but your VM is the real culprit here.

    VMware ESXi lies on multiple fronts in this area I've discovered
lately at $WORK, in particular the SMBIOS / FreeBSD are calling CPUID
which is returning info based on the host processor instead of the
emulated guest processor, which in turn is causes minor problems in
our software. Example being that the host processor is a Intel
Woodcrest CPU (this is what an in-house tool and FreeBSD reports)
instead of a Pentium Pro emulated CPU (that's what dmidecode reports
and that's what ESXi emulates).
    Wasn't happy about this breakage and someone should probably talk
to VMware about fixing their emulation software so that it tells a
consistent story across the board.
Thanks,
-Garrett



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?BANLkTinOV60xH_iXMAnE-oQ9P=P67d4-%2BQ>