cvs commit: src/sys/i386/cpufreq est.c

[John Baldwin]

On Wednesday 26 March 2008 01:56:04 pm Abdullah Ibn Hamad Al-Marri wrote:
> ----- Original Message ----
> > From: Abdullah Ibn Hamad Al-Marri <wearabnet at yahoo.ca>
> > To: John Baldwin <jhb at FreeBSD.org>; src-committers at FreeBSD.org; 
cvs-src at FreeBSD.org
> > Sent: Friday, March 14, 2008 3:45:01 PM
> > Subject: Re: cvs commit: src/sys/i386/cpufreq est.c
> > 
> > ----- Original Message ----
> > > From: John Baldwin 
> > > To: src-committers at FreeBSD.org; cvs-src at FreeBSD.org; cvs-all at FreeBSD.org
> > > Sent: Tuesday, March 11, 2008 1:00:36 AM
> > > Subject: cvs commit: src/sys/i386/cpufreq est.c
> > > 
> > > jhb         2008-03-10 22:00:36 UTC
> > > 
> > >   FreeBSD src repository
> > > 
> > >   Modified files:
> > >     sys/i386/cpufreq     est.c 
> > >   Log:
> > >   - Don't execute cpuid to fetch the features.  We already have the 
features
> > >     present in cpu_feature2.  Also, use CPUID2_EST rather than a magic
> > >     number.
> > >   - Don't free the ACPI settings list in detach if we are going to fail 
the
> > >     request.  Otherwise an attempt to kldunload est would free the array
> > >     but the driver would keep trying to use it.
> > >   
> > >   MFC after:      1 week
> > >   
> > >   Revision  Changes    Path
> > >   1.14      +5 -5      src/sys/i386/cpufreq/est.c
> > 
> > Hello John
> > 
> > Could you please MFC it to RELENG_7 now? :)
> > 
> > 
> > -- 
> > Regards,
> > 
> > -Abdullah Ibn Hamad Al-Marri
> > Arab Portal
> > http://www.WeArab.Net/
> 
> John,
> 
> This is C2Q CPU Q6600.
> 
> Upgraded today with RELENG_7.

Your CPU needs the est_msr.patch, though it is odd ACPI isn't providing the 
table in your case.  You can find it at 
http://www.freebsd.org/~jhb/patches/est_msr.patch

-- 
John Baldwin