Bug 217406 - nfsuserd usertimeout 0 behavior
Summary: nfsuserd usertimeout 0 behavior
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: bin (show other bugs)
Version: 11.0-RELEASE
Hardware: amd64 Any
: --- Affects Only Me
Assignee: freebsd-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-27 21:03 UTC by Jason Mader
Modified: 2017-05-07 21:11 UTC (History)
2 users (show)

See Also:
rmacklem: mfc-stable11+
rmacklem: mfc-stable10+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Mader 2017-02-27 21:03:52 UTC
Started nfsuserd with -usertimeout 0, and according to the man page,

If the timeout is specified as 0, cache entries never time out.

9Feb17      0:29.12 nfsuserd: slave (nfsuserd)

but then noticed much more cpu use than when the timeout was around 10 minutes. Looking at the source to nfsuserd, I don't think 0 has the special meaning indicated in the doc.
Comment 1 Jason Mader 2017-03-06 17:17:55 UTC
Typical time after two weeks running with -usertimeout 1523
Mon21        0:00.18 nfsuserd: slave (nfsuserd)
Comment 2 Rick Macklem freebsd_committer freebsd_triage 2017-03-06 23:02:28 UTC
I haven't looked yet, but since I think specifying
"-usertimeout <BIG-NUMBER>" will achieve "never times out",
I'm tempted to just take the sentence out of the man page.
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-04-21 20:54:36 UTC
A commit references this bug:

Author: rmacklem
Date: Fri Apr 21 20:53:52 UTC 2017
New revision: 317270
URL: https://svnweb.freebsd.org/changeset/base/317270

Log:
  Get rid of bogus statement in the nfsuserd.8 man page.

  The nfsuserd.8 man page stated that a usertimeout of 0 would disable
  the cache timeout. This was simply not true, so this patch deletes
  the sentence.

  This is a content change.

  PR:		217406
  MFC after:	2 weeks

Changes:
  head/usr.sbin/nfsuserd/nfsuserd.8
Comment 4 Rick Macklem freebsd_committer freebsd_triage 2017-05-07 21:11:32 UTC
Sentence deleted from man page and MFC'd.