Skip to content
Commit f1c6bb2c authored by Jeff Layton's avatar Jeff Layton
Browse files

locks: allow __break_lease to sleep even when break_time is 0



A fl->fl_break_time of 0 has a special meaning to the lease break code
that basically means "never break the lease". knfsd uses this to ensure
that leases don't disappear out from under it.

Unfortunately, the code in __break_lease can end up passing this value
to wait_event_interruptible as a timeout, which prevents it from going
to sleep at all. This makes __break_lease to spin in a tight loop and
causes soft lockups.

Fix this by ensuring that we pass a minimum value of 1 as a timeout
instead.

Cc: <stable@vger.kernel.org>
Cc: J. Bruce Fields <bfields@fieldses.org>
Reported-by: default avatarTerry Barnaby <terry1@beam.ltd.uk>
Signed-off-by: default avatarJeff Layton <jlayton@redhat.com>
parent 55101e2d
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment