fs: dlm: fix F_CANCELLK to cancel pending request
authorAlexander Aring <aahringo@redhat.com>
Thu, 20 Jul 2023 12:22:41 +0000 (08:22 -0400)
committerDavid Teigland <teigland@redhat.com>
Thu, 20 Jul 2023 22:25:04 +0000 (17:25 -0500)
commitdc52cd2eff4ac924a795efcef27f8fd58a5260bb
tree9d78febd4a8b4c4b97051c3e69edf473b4af109f
parent568f915655b3b4c40032104e4d0014e5e2c474b9
fs: dlm: fix F_CANCELLK to cancel pending request

This patch fixes the current handling of F_CANCELLK by not just doing a
unlock as we need to try to cancel a lock at first. A unlock makes sense
on a non-blocking lock request but if it's a blocking lock request we
need to cancel the request until it's not granted yet. This patch is fixing
this behaviour by first try to cancel a lock request and if it's failed
it's unlocking the lock which seems to be granted.

Note: currently the nfs locking handling was disabled by commit
40595cdc93ed ("nfs: block notification on fs with its own ->lock").
However DLM was never being updated regarding to this change. Future
patches will try to fix lockd lock requests for DLM. This patch is
currently assuming the upstream DLM lockd handling is correct.

Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
fs/dlm/plock.c
fs/gfs2/file.c
fs/ocfs2/stack_user.c
include/linux/dlm_plock.h