acpi_ec_space_handler flooding logs

acpi_ec_space_handler flooding logs

[Florian Schricker]

Hello everybody!


I have quite some trouble with ACPI / DSDT and with my hddisk - and
quite frankly I'm a bit lost. :-(

Well, looong story short: I had previously a self-repaired DSDT in my
initrd which I suspect of being the cause of quite some trouble. I am
unsure if this can be possible, but I thought it'd be best to at least
fall back to the "integrated" one.

Although all ACPI modules load fine (I'm on a 2.6.12 kernel), I have
trouble with my battery - I had this before, and to be honest, I'd very
much appreciate any professional help on fixing it (I suspect _me_ being
the cause of troubles first)!

For that I extracted my DSDT, disassembled it using iasl and, for
testing purposes, I recompiled it.

Both files are accessible:
 - raw data from /proc/acpi/dsdt
     http://www.uni-koblenz.de/~eyerq/dsdt-samsung.dat
 - dissambled DSDT source
     http://www.uni-koblenz.de/~eyerq/dsdt-samsung.dsl

Recompiling gives:

-------
# ./iasl dsdt-samsung.dsl

Intel ACPI Component Architecture
ASL Optimizing Compiler / AML Disassembler version 20030918 [Sep 18
2003]
Copyright (C) 2000 - 2003 Intel Corporation
Supports ACPI Specification Revision 2.0b

dsdt-samsung.dsl   312:     Method (_WAK, 1, NotSerialized)
Warning  2026 -                        ^ Reserved method must return a
value (_WAK)

dsdt-samsung.dsl  2577:                     Field (ECR, DWordAcc, Lock,
Preserve)
Error    1048 -                                      ^ Host Operation
Region requires ByteAcc access

dsdt-samsung.dsl  2623:                     Method (_GLK, 1,
NotSerialized)
Warning  2024 - Reserved method has too many arguments ^  ( _GLK
requires 0)

ASL Input:  dsdt-samsung.dsl - 3748 lines, 125559 bytes, 1798 keywords
Compilation complete. 1 Errors, 2 Warnings, 0 Remarks, 410 Optimizations
-------

This broken DSDT is currently active, fails with battery as mentioned -
and: floods dmesg / /var/log/messages with

  ACPI: acpi_ec_space_handler: bit_width should be 8

Is this something known?
What would I need to fix the DSDT? Anything I can provide?


thank you - and hopefully this the correct list
Florian Schricker



-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

Re: acpi_ec_space_handler flooding logs

[Yu, Luming]

> and: floods dmesg / /var/log/messages with
> 
>   ACPI: acpi_ec_space_handler: bit_width should be 8
This is just warning.  If you lower debug output level,
It will disappear.




-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf

Re: acpi_ec_space_handler flooding logs

[Thomas Renninger]

Florian Schricker wrote:
> Hello everybody!
> 
...
> Intel ACPI Component Architecture
> ASL Optimizing Compiler / AML Disassembler version 20030918 [Sep 18
> 2003]
> Copyright (C) 2000 - 2003 Intel Corporation
> Supports ACPI Specification Revision 2.0b
> 
> dsdt-samsung.dsl   312:     Method (_WAK, 1, NotSerialized)
> Warning  2026 -                        ^ Reserved method must return a
> value (_WAK)
> 
don't care about.

> dsdt-samsung.dsl  2577:                     Field (ECR, DWordAcc, Lock,
> Preserve)
> Error    1048 -                                      ^ Host Operation
> Region requires ByteAcc access
Replace DWordAcc with ByteAcc :)
> 
> dsdt-samsung.dsl  2623:                     Method (_GLK, 1,
> NotSerialized)
> Warning  2024 - Reserved method has too many arguments ^  ( _GLK
> requires 0)
You could replace 1 with 0 to avoid the compile warning.
> 
> ASL Input:  dsdt-samsung.dsl - 3748 lines, 125559 bytes, 1798 keywords
> Compilation complete. 1 Errors, 2 Warnings, 0 Remarks, 410 Optimizations
> -------
> 
> This broken DSDT is currently active, fails with battery as mentioned -
> and: floods dmesg / /var/log/messages with
> 
>   ACPI: acpi_ec_space_handler: bit_width should be 8

Could you try whether the above suggestions fix your battery?
I thought this one is workarounded?
At least it is tried: ACPI: acpi_ec_space_handler: bit_width should be 8
but maybe it does not work as expected?
> 
> Is this something known?
Yes.
> What would I need to fix the DSDT? Anything I can provide?
See above.
> thank you - and hopefully this the correct list
It is.

You do use the latest kernel?
If not please try your unfixed DSDT with the latest kernel, this
should be workarounded correctly since (about 2.6.11?), better use
the latest.

    Thomas


-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf

Re: acpi_ec_space_handler flooding logs

[Florian Schricker]

On Fri, 2005-08-12 at 10:21 +0200, Thomas Renninger wrote:

[DSDT fixes]

> > This broken DSDT is currently active, fails with battery as mentioned -
> > and: floods dmesg / /var/log/messages with
> > 
> >   ACPI: acpi_ec_space_handler: bit_width should be 8
> 
> Could you try whether the above suggestions fix your battery?

They did - everything seems to be just fine now.
Thanks a lot!


> I thought this one is workarounded?

Well, uhmm... */me is puzzled


> At least it is tried: ACPI: acpi_ec_space_handler: bit_width should be 8
> but maybe it does not work as expected?

Anything I could check, provide, test?


> You do use the latest kernel?

I use 2.6.12-suspend2-r3; a kernel patched with the common Gentoo
patchset incl. Suspend2 (which I use a lot).

> If not please try your unfixed DSDT with the latest kernel, this
> should be workarounded correctly since (about 2.6.11?), better use
> the latest.

As said above, I'm already running "something" > 2.6.11...


regards,
Florian Schricker

-- 
A most overlooked advantage to owning a computer is: if they foul up
there's no law against wacking them around a bit.



-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf

Re: acpi_ec_space_handler flooding logs

[Thomas Renninger]

Florian Schricker wrote:
> On Fri, 2005-08-12 at 10:21 +0200, Thomas Renninger wrote:
> 
> [DSDT fixes]
> 
>>>This broken DSDT is currently active, fails with battery as mentioned -
>>>and: floods dmesg / /var/log/messages with
>>>
>>>  ACPI: acpi_ec_space_handler: bit_width should be 8
>>Could you try whether the above suggestions fix your battery?
> 
> They did - everything seems to be just fine now.
> Thanks a lot!
> 
> 
>>I thought this one is workarounded?
> 
> Well, uhmm... */me is puzzled
> 
> 
>>At least it is tried: ACPI: acpi_ec_space_handler: bit_width should be 8
>>but maybe it does not work as expected?
> 
> Anything I could check, provide, test?

It would be worth to test whether latest EC burst mode implementations
still walks this code path. If, you probably still have battery problems.

You could try latest 2.6.13-rcX patches and enable ec_burst=1 (with the
unfixed DSDT).
If this works, you will have to boot with this option a few kernels.
But it shouldn't be worth to look at any further as this will get default
sooner or later.

If you still have battery problems, you should open a bug at:
http://bugzilla.kernel.org
assign it to the ACPI component and take me into CC.

....

Thanks for digging into this,

    Thomas


-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf