Re: [PATCH] i386: add command line option "local_apic_timer_c2_ok"

Grzegorz Chwesewicz
2007-03-31T07:47:20Z

On Fri, 30 Mar 2007 00:05:39 +0200, Andi Kleen wrote
> On Thursday 29 March 2007 23:16, Linus Torvalds wrote:
> >
> > On Thu, 29 Mar 2007, Andi Kleen wrote:
> > >
> > > Here's a patch. I don't have a system with C1E, so i only tested that
> > > the apic timer still works on a older AMD box.
> >
> > I think this looks better than what we have now, but it would look even
> > better if the core CPUID stuff was in arch/i386/kernel/cpu/amd.c, and we
> > simply had X86_FEATURE_BROKEN_C1_LAPIC etc..
> >
> > And then the apic.c code would just check
> >
> > if (boot_cpu_has(X86_FEATURE_BROKEN_C1_LAPIC))
> > return -1;
> >
> > or similar.
>
> Ok fair point. Here's an updated patch.

I've tested this patch little bit more on my nx6325 and I've found scenario in
which my box works slow. When I boot HP with connected AC (it boots fast), and
then after boot I unplug AC and try to power HP off it's working very slow
(powering off process take few minutes). On battery it's always booting and
powering off fast. Can enybody with nx6325 confirm this ?

--
Greetings - CeHo - Grzegorz Chwesewicz