Details
Details
- Reviewers
rmacklem - Commits
- rS281691: Remove some oldnfs remnants.
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Thanks for doing this.
Everything looks good.
I don't think it will be possible to load the old nfsserver in
FreeBSD11. Given that, you could delete the entire
"else" block in readstats() in nfsstat.c. (The else
that matches "if (deadkernel)".) It is tempting to get
rid of readstats() completely, but it might be useful to
get stats from an old kernel dump, so I guess it should
stay?
Btw, however you choose to change this, it doesn't need
another review by me. I'm happy with what you have.
(And I'll admit I find Phabricator's habit of asking for
re-reviews after every change irritating;-).