nfs_commonsubs.c: Lock vp for server calls to nfsv4_fillattr()
AbandonedPublic
Actions

Authored by rmacklem on Jul 18 2025, 12:38 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 7, 9:15 PM

	Unknown Object (File)
	Sat, Apr 4, 2:15 PM

	Unknown Object (File)
	Tue, Mar 24, 4:49 PM

	Unknown Object (File)
	Mon, Mar 23, 6:52 AM

	Unknown Object (File)
	Sun, Mar 22, 7:08 AM

	Unknown Object (File)
	Mar 6 2026, 2:54 PM

	Unknown Object (File)
	Mar 6 2026, 7:24 AM

	Unknown Object (File)
	Oct 20 2025, 5:37 PM

View All 25 Files

Subscribers

imp

Details

Reviewers

kib
des

Summary

des@ reported a panic in the NFSv4 server, where
nfsv4_fillattr() did a VOP_PATHCONF() without having
"vp" locked. kib@ suggested D51398 to fix it.

This patch is similar, but puts the NFSVOPLOCK()
call in one place near the top of the function.

des@, please test.

Test Plan

I will be building a kernel with DEBUG_VFS_LOCKS
so that I can find out if there are any more of these
bugs. (Yes, I have just been building GENERIC. Bad me.)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

rmacklem created this revision.Jul 18 2025, 12:38 AM

Herald added a subscriber: imp. · View Herald TranscriptJul 18 2025, 12:38 AM

rmacklem requested review of this revision.Jul 18 2025, 12:38 AM

Don't bother with this one, it is broken.

I'll put up another patch once I have one that works.

Revision Contents
Changeset List

Path

Size

sys/

fs/

nfs/

	nfs_commonsubs.c
	nfs_commonsubs.c.vvroot

41 lines

Diff 158719

View Options

sys/fs/nfs/nfs_commonsubs.c

nfs_commonsubs.c: Lock vp for server calls to nfsv4_fillattr()AbandonedPublicActions