Hi,

"Michael S. Zick":
> Was diddling with the code to give the above a try and the eyes noticed this:
> (in inotify_read function above the patched one)
        :::
> Which skips the unlock if the list is not empty; continuing in the code;
> where there are various conditional paths that look as if it can be unlocked 
> twice.
> 
> Warning: My "C" skills are at a low ebb - someone else please take a look.

The mutex lock in inotify_read in linux-2.6.27 are balanced. We don't
need to care.

----------------------------------------------------------------------

> This is not on a auFS patched kernel - but - -
> 
> Makes a very noticeable difference!
> Even with Firefox trying to display the black hole of the simplynetbook.com 
> home page, KDE/X11 continues to respond.
> There is still a visible delay to cleaning up a closed window, but only 
> measured in seconds or less - they no longer
> act as if they had gone out to lunch. ;)
> 
> That one needs to go upstream.

Actually I have posted to LKML and inotify maintainers, but there is no
reply (yet). See <http://marc.info/?t=122468297600004&r=1&w=2>.


Junjiro R. Okajima


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/

Reply via email to