Page MenuHomeFreeBSD
Differential D10206

Remove OLD_NFSV2 from loader and libstand
ClosedPublic
Actions

Authored by tsoome on Mar 30 2017, 5:04 PM.
Tags
None
Referenced Files
F153694375: D10206.diff
Wed, Apr 22, 11:56 PM
Unknown Object (File)
Sun, Apr 19, 11:40 AM
Unknown Object (File)
Sun, Apr 19, 10:11 AM
Unknown Object (File)
Thu, Apr 16, 1:00 AM
Unknown Object (File)
Wed, Apr 15, 2:52 PM
Unknown Object (File)
Wed, Apr 15, 2:33 PM
Unknown Object (File)
Wed, Apr 15, 11:31 AM
Unknown Object (File)
Wed, Apr 15, 11:30 AM
View All Files
Subscribers
None

Details

Reviewers
allanjude
imp
Commits
rS316287: Remove OLD_NFSV2 from loader and libstand
Summary

We have parallel NFSv2 and NFSv3 reader implementations, only configurable at
build time, defaulting to v3. Remove v2.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tsoome created this revision.Mar 30 2017, 5:04 PM
allanjude accepted this revision.Mar 30 2017, 6:51 PM

Looks like a good cleanup to me

Approved By: allanjude

This revision is now accepted and ready to land.Mar 30 2017, 6:51 PM
tsoome updated this revision to Diff 26840.Mar 30 2017, 7:18 PM
tsoome edited edge metadata.

I think the NQNFS comment is also not actual any more

This revision now requires review to proceed.Mar 30 2017, 7:18 PM
Harbormaster completed remote builds in B8403: Diff 26840.Mar 30 2017, 7:18 PM
allanjude accepted this revision.Mar 30 2017, 7:29 PM
This revision is now accepted and ready to land.Mar 30 2017, 7:29 PM
Closed by commit rS316287: Remove OLD_NFSV2 from loader and libstand (authored by tsoome). · Explain WhyMar 30 2017, 7:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
lib/
libstand/
675 lines
43 lines
sys/
boot/
i386/
libi386/
55 lines

Diff 26841

View Options

head/lib/libstand/nfs.c

Loading...
View Options

head/lib/libstand/nfsv2.h

Loading...
View Options

head/sys/boot/i386/libi386/pxe.c

Loading...