Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 May 2009 01:22:12 +0200
From:      Lucius Windschuh <lwindschuh@googlemail.com>
To:        Alexander Motin <mav@freebsd.org>
Cc:        freebsd-current@freebsd.org, freebsd-mobile@freebsd.org
Subject:   Re: Fighting for the power.
Message-ID:  <90a5caac0905041622oaddd7cek52f28a9b018b3ea7@mail.gmail.com>
In-Reply-To: <49FF6C11.5030607@FreeBSD.org>
References:  <49FE1826.4060000@FreeBSD.org> <90a5caac0905041119h70101d12i56863e57b27d2e55@mail.gmail.com> <49FF6C11.5030607@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
2009/5/5 Alexander Motin <mav@freebsd.org>:
> Lucius Windschuh wrote:
>> [...]
>> panic: lapic1: zero divisor
> [...]
> --- local_apic.c.prev =A0 2009-05-01 23:53:37.000000000 +0300
> +++ local_apic.c =A0 =A0 =A0 =A02009-05-05 01:10:04.000000000 +0300
> @@ -319,7 +319,7 @@ lapic_setup(int boot)
> =A0 =A0 =A0 =A0}
>
> =A0 =A0 =A0 =A0/* We don't setup the timer during boot on the BSP until l=
ater. */
> - =A0 =A0 =A0 if (!(boot && PCPU_GET(cpuid) =3D=3D 0)) {
> + =A0 =A0 =A0 if (!(boot && PCPU_GET(cpuid) =3D=3D 0) && lapic_timer_hz !=
=3D 0) {
> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0KASSERT(lapic_timer_period !=3D 0, ("lapic=
%u: zero divisor",
> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0lapic_id()));
> =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0lapic_timer_set_divisor(lapic_timer_diviso=
r);

This patch solves the panic.
C3 instead of C2 saves between 0.5 and 1.5 Watt here with some quick
measurements.

Thanks.

Lucius



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?90a5caac0905041622oaddd7cek52f28a9b018b3ea7>