nfs client: switch nfs_advlock() to use exclusive vnode lock
ClosedPublic
Actions

Authored by kib on Aug 27 2025, 9:11 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, May 18, 6:28 PM

	Unknown Object (File)
	Mon, May 18, 6:28 PM

	Unknown Object (File)
	Sun, May 17, 1:53 AM

	Unknown Object (File)
	Sun, May 17, 1:53 AM

	Unknown Object (File)
	Sun, May 17, 1:43 AM

	Unknown Object (File)
	Apr 23 2026, 8:58 AM

	Unknown Object (File)
	Apr 23 2026, 4:10 AM

	Unknown Object (File)
	Apr 22 2026, 6:57 PM

View All 50 Files

Subscribers

imp

Details

Reviewers

rmacklem

Commits

rGfe53e046cd9c: nfs client: switch nfs_advlock() to use exclusive vnode lock

Summary

It eliminates the need to upgrade the lock in the function.
More importantly, the calls to nfs_advlock_p()/nlm_advlock() sometimes
flush buffers, which requires exclusive locking.

Reported by:    bz

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.Aug 27 2025, 9:11 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 27 2025, 9:11 PM

kib requested review of this revision.Aug 27 2025, 9:11 PM

Looks fine to me.

This revision is now accepted and ready to land.Aug 27 2025, 10:32 PM

Closed by commit rGfe53e046cd9c: nfs client: switch nfs_advlock() to use exclusive vnode lock (authored by kib). · Explain WhyAug 27 2025, 10:37 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rGfe53e046cd9c: nfs client: switch nfs_advlock() to use exclusive vnode lock.

Revision Contents
Changeset List

Path

Size

sys/

fs/

nfsclient/

nfs_clvnops.c

7 lines

Diff 161093

View Options

sys/fs/nfsclient/nfs_clvnops.c

nfs client: switch nfs_advlock() to use exclusive vnode lockClosedPublicActions