On 2016-09-30 14:01, Mateusz Guzik wrote:
On Fri, Sep 30, 2016 at 01:58:16PM -0400, Larry Rosenman wrote:
On 2016-09-30 13:55, Mateusz Guzik wrote:
>On Fri, Sep 30, 2016 at 01:36:26PM -0400, Larry Rosenman wrote:
>>On 2016-09-30 13:28, Mateusz Guzik wrote:
>>>On Fri, Sep 30, 2016 at 01:18:45PM -0400, Shawn Webb wrote:
>>>>On Fri, Sep 30, 2016 at 05:11:03PM +0000, Mateusz Guzik wrote:
>>>>> Author: mjg
>>>>> Date: Fri Sep 30 17:11:03 2016
>>>>> New Revision: 306509
>>>>> URL: https://svnweb.freebsd.org/changeset/base/306509
>>>>>
>>>>> Log:
>>>>>   vfs: remove the __bo_vnode field from struct vnode
>>>>>
>>>>>   The pointer can be obtained using __containerof instead.
>>>>>
>>>>>   Reviewed by:     kib
>>>>
>>>>Should __FreeBSD_Version be bumped?
>>>>
>>>
>>>Unlikely. It can be in an odd case it turns out there is a module which
>>>is using the field.
>>
>>Can someone do me a favor and make sure sysutils/lsof still
>>compiles/works?
>>
>>I'm OOT at the moment, and have a test IPv6 patch on my 12 system.
>
>I just ran lsof and it worked fine at least for the basic case.
>
>Apparently it indeed still goes through kernel memory. Did anyone ask
>the author what kind of interfaces would they want to NOT have to do
>this?
>
>Also, it looks like it constains some kind of a loop to imitate
>closefrom.
>
>On truss I see over 3.7 milion (yes, MILION) calls to close. Like this:
>close(3773297)                                   ERR#9 'Bad file
>descriptor'
>close(3773298)                                   ERR#9 'Bad file
>descriptor'
>close(3773299)                                   ERR#9 'Bad file
>descriptor'
>close(3773300)                                   ERR#9 'Bad file
>descriptor'
>close(3773301)                                   ERR#9 'Bad file
>descriptor'
>close(3773302)                                   ERR#9 'Bad file
>descriptor'
>close(3773303)                                   ERR#9 'Bad file
>descriptor'
>[snip]
>
>At least for freebsd this can be patched with the use of closefrom() on
>all supported releases.
>
>Arguably this may be a side effect of limits set too high, perhaps
>linux
>sets them sigifnicanly lower and the problem is not that terrible.

We've had this discussion before (re: interfaces). I thought the
current versions of lsof only close
the first 1k FD's.  (what version of lsof is this? on 12-CURRENT?)


This is 4.90.g,8 "formally" built for 11.0. The typical limit for fd is
1024 on linux, while it is higher on fbsd. I presume it goes up to the
limit. (as in, ulimit -n)

So what's up with the discussion on interfaces?

The current state seems detrimental to everyone.

Vic Abell (Author, Retired from Purdue, 70+ yrs old) has to maintain portability to
other OS's, and every time we go through this discussion it drops off.

You are welcome to try again, but I wouldn't bet on it.

Vic Abell <vab...@lsof.comcastbiz.net>


--
Larry Rosenman                     http://www.lerctr.org/~ler
Phone: +1 214-642-9640                 E-Mail: l...@lerctr.org
US Mail: 17716 Limpia Crk, Round Rock, TX 78664-7281
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to