All,

Sorry this took so long.  I have attached the Kconfig changes
requested by Andi, appended to the original patch.

On Feb 4, 2008 11:22 AM, Russell Leidich <rml@google.com> wrote:
quoted text
> On Feb 3, 2008 11:20 PM, Andi Kleen <andi@firstfloor.org> wrote:
> > On Sun, Feb 03, 2008 at 10:13:53PM -0800, Russell Leidich wrote:
> > > All,
> > >
> > > You can imagine my dismay when I recently learned that, after all our
> > > collective effort, hardware thermal throttling does not work reliably
> > > on Barcelona, according to AMD.  Due to NDA restrictions, I am unable
> > > to provide further details.
> >
> > Too bad. Hopefully that code will work then on the hypothetic
> > future parts with fixed throttling. Perhaps it would be better
> > to just remove it now and readd later.
>
> I really think it's best to leave it in.  It's harmless at the moment,
> and readding it later will just create a massive amount of duplicate
> work.    Sooner or later, I'm have little doubt that AMD can make this
> feature work.
>
> >
> > All this reminds me you need to adjust the Kconfig dependencies
> > for K8_NB too to be always enabled with MCE_AMD.
> >
> > Ideally on top of this patch with a || MCE_AMD
>
> OK, if you'll take the patch, I'd be happy to do this :).  I'm
> assuming that you mean I should force K8_NB = "yes" if MCE_AMD = "yes"
> in .config.  Is that what you're asking for?
>
> >
> > -Andi

-- 
Russell Leidich