Re: [PATCH 01/13] PM: Add wake lock api.

Matthew Garrett
2009-02-13T17:05:18Z

On Fri, Feb 13, 2009 at 05:46:42PM +0100, Uli Luckas wrote:

> That's racy. By the time the daemon notices that a process crashed, the kernel
> might already have triggered suspend. userspace might then be frozen before
> it can accuire the 'process restarting' lock.
> I also wonder, if it is immanently racy to use userspcae communication
> (client/daemon) for suspend locks. What if the daemon is already frozen when
> a client sends a lock request request?

The daemon holds the lock in the first place. There's no race. As for
issues with the freezer, I think my position on that is fairly well
known...

--
Matthew Garrett | mjg59@srcf.ucam.org