[PATCH v2 0/2] filelock: fix conflict detection with userland file delegations

Jeff Layton posted 2 patches 2 months ago
Documentation/filesystems/locking.rst |   1 +
fs/locks.c                            | 119 +++++++++++++++++-----------------
fs/nfsd/nfs4layouts.c                 |  23 ++++++-
fs/nfsd/nfs4state.c                   |  19 ++++++
include/linux/filelock.h              |   1 +
5 files changed, 103 insertions(+), 60 deletions(-)
[PATCH v2 0/2] filelock: fix conflict detection with userland file delegations
Posted by Jeff Layton 2 months ago
This patchset fixes the way that conflicts are detected when userland
requests file delegations. The problem is due to a hack that was added
long ago which worked up until userland could request a file delegation.

This fixes the bug and makes things a bit less hacky. Please consider
for v6.19.

Signed-off-by: Jeff Layton <jlayton@kernel.org>
---
Changes in v2:
- add kerneldoc headers over nfsd's lm_open_conflict operations
- revise changelog on lease_free_list() patch
- whitespace fixes in locking.rst
- Link to v1: https://lore.kernel.org/r/20251201-dir-deleg-ro-v1-0-2e32cf2df9b7@kernel.org

---
Jeff Layton (2):
      filelock: add lease_dispose_list() helper
      filelock: allow lease_managers to dictate what qualifies as a conflict

 Documentation/filesystems/locking.rst |   1 +
 fs/locks.c                            | 119 +++++++++++++++++-----------------
 fs/nfsd/nfs4layouts.c                 |  23 ++++++-
 fs/nfsd/nfs4state.c                   |  19 ++++++
 include/linux/filelock.h              |   1 +
 5 files changed, 103 insertions(+), 60 deletions(-)
---
base-commit: 3f9f0252130e7dd60d41be0802bf58f6471c691d
change-id: 20251201-dir-deleg-ro-41a16bc22838

Best regards,
-- 
Jeff Layton <jlayton@kernel.org>
Re: [PATCH v2 0/2] filelock: fix conflict detection with userland file delegations
Posted by Christian Brauner 1 month, 3 weeks ago
On Thu, 04 Dec 2025 08:48:31 -0500, Jeff Layton wrote:
> This patchset fixes the way that conflicts are detected when userland
> requests file delegations. The problem is due to a hack that was added
> long ago which worked up until userland could request a file delegation.
> 
> This fixes the bug and makes things a bit less hacky. Please consider
> for v6.19.
> 
> [...]

Applied to the vfs.fixes branch of the vfs/vfs.git tree.
Patches in the vfs.fixes branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.fixes

[1/2] filelock: add lease_dispose_list() helper
      https://git.kernel.org/vfs/vfs/c/392e317a20c3
[2/2] filelock: allow lease_managers to dictate what qualifies as a conflict
      https://git.kernel.org/vfs/vfs/c/12965a190eae