acpi shows wrong battery state

[Andriy Gapon]

on 03/09/2010 14:01 kuba guzik said the following:
> 2010/9/3 Andriy Gapon <avg at icyb.net.ua>:
>> on 31/08/2010 04:40 kuba guzik said the following:
>>>>  It would be good too see full verbose dmesg and acpidump -dt output.
>>>>  (Preferably uploaded somewhere)
>>>
>>> My configs from 8.0 and 8.1
>>> http://kubag.pl/BSD
>>
>> Oh, hey, you know what's confusing?
>> Your dmesg dumps having information from two boots each.
>> And in 8.1 link the first part is actually for 8.0, you have to scroll to the
>> middle to see start of 8.1 boot actually.
>> Thanks :)
> 
> I don't think it's possible, now I have 2 Freebsd systems and they
> have different slices. It was late, so maybe on my account on sever
> I've mixed configs, or, I don't know...

Well, just check for yourself, FreeBSD version is printed at the beginning of boot.

>> Can you try the following patch with 8.1?
>> diff --git a/sys/dev/acpica/acpi_ec.c b/sys/dev/acpica/acpi_ec.c
>> index 2c18972..cc8a305 100644
>> --- a/sys/dev/acpica/acpi_ec.c
>> +++ b/sys/dev/acpica/acpi_ec.c
>> @@ -828,6 +828,7 @@ EcWaitEvent
>>      * the status checking loop, hopefully to allow the EC to go to work
>>      * and produce a non-stale status.
>>      */
>> +#if 0
>>     if (need_poll) {
>>        static int      once;
>>
>> @@ -840,7 +841,7 @@ EcWaitEvent
>>            AcpiOsStall(10);
>>        }
>>     }
>> -
>> +#endif
>>     /* Wait for event by polling or GPE (interrupt). */
>>     if (need_poll) {
>>        count = (ec_timeout * 1000) / EC_POLL_DELAY;
>>
> Sorry if it's stupid question, but could you tell me how to use this patch?

cd /usr/src
patch -p1 < /path/to/patch
make buildkernel
make installkernel
shutdown -r <when>

> I've noticed that always after installation on "clean" system acpi
> shows proper battery state( 8.0 and 8.1), but when I install xorg,
> dbus, kde and reboot I will be dealing with acpi errors... again.

Look for other things that you can correlate - like first reboot after booting to
other OS, etc.

-- 
Andriy Gapon