tree bd40fa64f4a6a5b4a9bcd9f37fafcb32e0efc9da
parent c45198eda2794bb72601c9f96266d8b95db66dd5
author Jeff Layton <jlayton@primarydata.com> 1409596063 -0400
committer Jeff Layton <jlayton@primarydata.com> 1412705173 -0400

locks: remove i_have_this_lease check from __break_lease

I think that the intent of this code was to ensure that a process won't
deadlock if it has one fd open with a lease on it and then breaks that
lease by opening another fd. In that case it'll treat the __break_lease
call as if it were non-blocking.

This seems wrong -- the process could (for instance) be multithreaded
and managing different fds via different threads. I also don't see any
mention of this limitation in the (somewhat sketchy) documentation.

Remove the check and the non-blocking behavior when i_have_this_lease
is true.

Signed-off-by: Jeff Layton <jlayton@primarydata.com>