diff options
| author | NeilBrown <neil@brown.name> | 2026-07-17 19:28:00 +1000 |
|---|---|---|
| committer | Chuck Lever <cel@kernel.org> | 2026-08-26 16:40:32 -0400 |
| commit | 5c453c40bcda0e54a1b1d63dcb690edc2cc9a5fd (patch) | |
| tree | d2d770ae97e3fec0b84221a06ccf767481187c39 /scripts | |
| parent | e70f0ee26e363c12afa9776b1dff5dd9a0057cfc (diff) | |
| download | linux-next-5c453c40bcda0e54a1b1d63dcb690edc2cc9a5fd.tar.gz linux-next-5c453c40bcda0e54a1b1d63dcb690edc2cc9a5fd.zip | |
nfsd: reduce range of directory lock in nfsd4_create_file()
We only need to hold the lock taken by start_creating() until the create
has been attempted. Holding for longer can serve no purpose.
The lock is currently held across the setattr call. This might be the
intent but it serves no purpose. Holding the lock prevents the name
from being removed or renamed, but it doesn't prevent a GETATTR or a
racing SETATTR or an OPEN.
Calling end_creating() puts the reference to 'child', but we can still
use the reference that was stored in open->op_filp.
Signed-off-by: NeilBrown <neil@brown.name>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Link: https://patch.msgid.link/20260717093001.1972119-13-neilb@ownmail.net
Signed-off-by: Chuck Lever <cel@kernel.org>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions
