On Tue, 8 Dec 2009, Linus Torvalds wrote: > Please just use the lock. Don't make up your own locking crap. Really. > > Your patch is horrible. Exactly because your locking is horribly > mis-designed. You can't say things are complete from an interrupt, for > example, since you made it some random bitfield, which has unknown > characteristics (ie non-atomic read-modify-write etc). > > The fact is, any time anybody makes up a new locking mechanism, THEY > ALWAYS GET IT WRONG. Don't do it. > > I suggested using the rwsem locking for a good reason. It made sense. It > was simpler. Just do it that way, stop making up crap. The semantics needed for this kind of lock aren't really the same as for an rwsem (although obviously an rwsem will do the job). Basically it needs to have the capability for multiple users to lock it (no blocking when acquiring a lock) and the capability for a user to wait until it is totally unlocked. It could be implemented trivially using an atomic_t counter and a waitqueue head. Is this a standard sort of lock? It's a lot simpler than most others. I don't recall seeing anything quite like it anywhere; the closest thing might be some kind of barrier. Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
