Re: sbs-linux acpi-ec-nospinlock patch for 2.6.11

[Rich Townsend <rhdt-OBnUx95tOyn10jlvfTC4gA@public.gmane.org>]

Rich Townsend wrote:
> 
> Yu, Luming wrote:
> 
>>I don't know what is acpi-ec-nospinlock patch.
>>But the patch at http://bugzilla.kernel.org/show_bug.cgi?id=3851#c52 is worthy testing, if you have EC related issue.
>>Thanks,
>>Luming
> 
> 
> It was a small patch I wrote, to lock EC access using a semaphore rather 
> than a spinlock. This was to stop the stuttering caused when the EC 
> driver was polling for completion of a transaction.
> 
> Eventually, of course, we want to switch over to event-based EC 
> management. I already tried one of the older EC/event patches on the 
> page you give above, with not much success; but I'll give one of the 
> newer ones a shot.
> 

OK, I've tried the newest patch (I needed to fix a small error in it, 
relating to a statement label, to get it to work on 2.6.11). It seems to 
work OK in all respects apart from speed. On bootup, it takes about 30 
seconds for the EC to get initialized. Battery status reads take about 4 
seconds. Events (eg, AC power change) take about 5 seconds before they 
actually reach /proc/acpi/events.

This clearly makes the patch of limited usefullness. What can be done to 
increase the speed?

cheers,

Rich


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click