nfsclient: use new routing KPI when adding a new route
Needs ReviewPublic
Actions

Authored by melifaro on Aug 12 2022, 10:23 AM.

Tags

None

Referenced Files

	F174714489: D36171.id109239.diff
	Mon, Oct 5, 10:35 AM

	F174663858: D36171.id.diff
	Mon, Oct 5, 1:20 AM

	F174654142: D36171.id109239.diff
	Sun, Oct 4, 11:53 PM

	Unknown Object (File)
	Thu, Oct 1, 4:55 AM

	Unknown Object (File)
	Sun, Sep 13, 5:36 PM

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

	Unknown Object (File)
	Tue, Sep 8, 10:56 PM

	Unknown Object (File)
	Sun, Sep 6, 10:03 AM

Subscribers

Details

Reviewers

rmacklem

Group Reviewers

network

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 46897
Build 43786: arc lint + arc unit

Event Timeline

melifaro created this revision.Aug 12 2022, 10:23 AM

Herald added a subscriber: imp. · View Herald TranscriptAug 12 2022, 10:23 AM

melifaro requested review of this revision.Aug 12 2022, 10:23 AM

Harbormaster completed remote builds in B46897: Diff 109239.Aug 12 2022, 10:23 AM

melifaro added a parent revision: D36167: routing: add rib_add_default_route() wrapper.Aug 12 2022, 10:23 AM

lwhsu added a reviewer: rmacklem.Aug 13 2022, 2:27 PM

I know nothing about routing stuff and I have no test setup for
an NFS root fs. As such, I can't review this.

emaste added a subscriber: emaste.May 6 2024, 5:11 PM

Ref d8b2693414ae5
It would be great if we had a section 9 manpage for rib_add_default_route

zlei added a subscriber: zlei.May 7 2024, 9:42 AM

The change is simple and looks good, IMO manual test is required at least.

Revision Contents
Changeset List

Path

Size

sys/

fs/

nfsclient/

nfs_clvfsops.c

22 lines

Diff 109239

View Options

sys/fs/nfsclient/nfs_clvfsops.c

nfsclient: use new routing KPI when adding a new routeNeeds ReviewPublicActions