http://bugzilla.kernel.org/show_bug.cgi?id=13109





--- Comment #29 from Tiago <[email protected]>  2009-05-29 18:58:43 ---
Created an attachment (id=21627)
 --> (http://bugzilla.kernel.org/attachment.cgi?id=21627)
make the callback save the previous state

No go!

I applied your patch on a vanilla kernel, and recompiled with each of the
supplied patches in order, I tried all but the patch sbshc patch.

After all the patches are applied, I still can not use the touchpad, and I get
no notification on ac removal.

I did however try a little tweak on top of your patch that did give me the
notifications back, but not the touchpad. The patch I attached applies on top
of yours.

If I read the  suggested patch correctly, the mutex_trylock() may miss some
events, and given that I am seeing a lot of events per second, I tried to
convert saved* to static, and initialize it with random values. this may cause
extra events on bootup, but at that time, I do not think any userland program
will act upon them.

I am planning to further test the patch I attached, but I welcome your opinion,
as I do not like the way I initialized the variables, and I am not sure how
static variables are seen by kernel programmers...

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

------------------------------------------------------------------------------
Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT 
is a gathering of tech-side developers & brand creativity professionals. Meet
the minds behind Google Creative Lab, Visual Complexity, Processing, & 
iPhoneDevCamp as they present alongside digital heavyweights like Barbarian 
Group, R/GA, & Big Spaceship. http://p.sf.net/sfu/creativitycat-com 
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to