On Fri, 2022-01-28 at 11:39 -0800, Dai Ngo wrote:
> Modify leases_conflict to call lm_breaker_owns_lease only if
> there is real conflict. This is to allow the lock manager to
> resolve the conflict if possible.
>
> Signed-off-by: Dai Ngo <dai.ngo@xxxxxxxxxx>
> ---
> fs/locks.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/fs/locks.c b/fs/locks.c
> index 052b42cc7f25..456717873cff 100644
> --- a/fs/locks.c
> +++ b/fs/locks.c
> @@ -1357,9 +1357,6 @@ static bool leases_conflict(struct file_lock *lease, struct file_lock *breaker)
> {
> bool rc;
>
> - if (lease->fl_lmops->lm_breaker_owns_lease
> - && lease->fl_lmops->lm_breaker_owns_lease(lease))
> - return false;
> if ((breaker->fl_flags & FL_LAYOUT) != (lease->fl_flags & FL_LAYOUT)) {
> rc = false;
> goto trace;
> @@ -1370,6 +1367,9 @@ static bool leases_conflict(struct file_lock *lease, struct file_lock *breaker)
> }
>
> rc = locks_conflict(breaker, lease);
> + if (rc && lease->fl_lmops->lm_breaker_owns_lease &&
> + lease->fl_lmops->lm_breaker_owns_lease(lease))
> + rc = false;
> trace:
> trace_leases_conflict(rc, lease, breaker);
> return rc;
Acked-by: Jeff Layton <jlayton@xxxxxxxxxx>
