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)
	Wed, May 27, 7:30 AM

	Unknown Object (File)
	May 1 2026, 12:05 PM

	Unknown Object (File)
	Apr 28 2026, 4:14 PM

	Unknown Object (File)
	Apr 28 2026, 6:27 AM

	Unknown Object (File)
	Apr 27 2026, 2:55 PM

	Unknown Object (File)
	Apr 27 2026, 6:43 AM

	Unknown Object (File)
	Apr 27 2026, 6:28 AM

	Unknown Object (File)
	Apr 22 2026, 10:32 PM

View All 33 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