Make nfs_fha sysctls per vnet and get rid of IS_DEFAULT_VNET()
ClosedPublic
Actions

Authored by rmacklem on Feb 23 2023, 9:17 PM.

Tags

None

Referenced Files

	F170745572: D38748.id117838.diff
	Sun, Sep 6, 9:23 AM

	F170687564: D38748.id117838.diff
	Sun, Sep 6, 1:00 AM

	Unknown Object (File)
	Sat, Sep 5, 11:56 AM

	Unknown Object (File)
	Sat, Sep 5, 4:01 AM

	Unknown Object (File)
	Sat, Sep 5, 12:00 AM

	Unknown Object (File)
	Fri, Sep 4, 9:21 AM

	Unknown Object (File)
	Fri, Sep 4, 3:58 AM

	Unknown Object (File)
	Thu, Sep 3, 4:20 PM

View All Files

Subscribers

imp

jamie

Details

Reviewers

glebius

Commits

rGa90b47abcbdb: nfs_fha_new: Fix nfs_fha_new so that sysctls work in prisons

Summary

Instead of only doing the tuning sysctls for prison0
by using IS_DEFAULT_VNET(), make them per-vnet.
This gets rid of the IS_DEFAULT_VNET() calls in
the initialization function.

Test Plan

Tested on a system that is running nfsd(8) in
vnet prisons ad well as prison0.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rmacklem created this revision.Feb 23 2023, 9:17 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 23 2023, 9:17 PM

rmacklem requested review of this revision.Feb 23 2023, 9:17 PM

Looks correct and better, thanks! Disclaimer: didn't test.

This revision is now accepted and ready to land.Feb 28 2023, 6:20 AM

Closed by commit rGa90b47abcbdb: nfs_fha_new: Fix nfs_fha_new so that sysctls work in prisons (authored by rmacklem). · Explain WhyMar 1 2023, 11:27 PM

This revision was automatically updated to reflect the committed changes.

rmacklem added a commit: rGa90b47abcbdb: nfs_fha_new: Fix nfs_fha_new so that sysctls work in prisons.

Revision Contents
Changeset List

Path

Size

sys/

fs/

nfsserver/

nfs_fha_new.h

3 lines

nfs_fha_new.c

118 lines

Diff 118146

View Options

sys/fs/nfsserver/nfs_fha_new.h