Date: Fri, 2 Nov 2018 07:35:29 -0600 From: Warner Losh <imp@bsdimp.com> To: Gary Jennejohn <gljennjohn@gmail.com> Cc: "Conrad E. Meyer" <cem@freebsd.org>, Konstantin Belousov <kostikbel@gmail.com>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers <src-committers@freebsd.org> Subject: Re: svn commit: r340038 - in head: lib/libc/sys sys/kern Message-ID: <CANCZdfoNoc2hdyw9j-wZ7APZZS56vMspU08gJA5VXuBho_0q_g@mail.gmail.com> In-Reply-To: <20181102094617.7340e6f2@ernst.home> References: <201811012346.wA1NkNS5079845@repo.freebsd.org> <20181102004555.GJ5335@kib.kiev.ua> <CAG6CVpU5aLjOB-1N92KpEOFDBpZe4gu5=w6ADWeD=6MgXFPjsQ@mail.gmail.com> <20181102094617.7340e6f2@ernst.home>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Nov 2, 2018 at 2:46 AM Gary Jennejohn <gljennjohn@gmail.com> wrote: > On Thu, 1 Nov 2018 20:29:27 -0700 > Conrad Meyer <cem@freebsd.org> wrote: > > > On Thu, Nov 1, 2018 at 5:46 PM Konstantin Belousov <kostikbel@gmail.com> > wrote: > > > > > > On Thu, Nov 01, 2018 at 11:46:23PM +0000, Conrad Meyer wrote: > > > > ... > > > > @@ -232,7 +232,15 @@ points outside the process's allocated address > space. > > > > A signal was delivered before the time limit expired and > > > > before any of the selected events occurred. > > > > .It Bq Er EINVAL > > > > -The specified time limit is invalid. One of its components is > negative or too large. > > > > +The specified time limit is invalid. > > > > +One of its components is negative or too large. > > > > +.It Bq Er EINVAL > > > > +The number of pollfd structures specified by > > > > +.Fa nfds > > > > +exceeds the system tunable > > > > +.Va kern.maxfilesperproc > > > > +and > > > > +.Dv FD_SETSIZE . > > > > > > This is somewhat confusing. Looking only at the man page text, most > > > straight reading of it is that nfds cannot exceed min(maxfilesperproc, > > > FD_SETSIZE). But in fact nfds cannot exceed max of it, which is > > > significantly different, i.e. poll(2) does not suffer from the > (userspace) > > > FD_SETSIZE limitation of select(2). > > > > min(maxfilesperproc, FD_SETSIZE) would be written as "nfds exceeds ... > > maxfilesperproc OR FD_SETSIZE," rather than "and." I think the > > sentence is unambiguous. > > > > Based on Konstantin's comment I'd have to say that it is not > unambiguous. > > As a native English speaker I had to read it several times to > understand it. > > I think something like "exceeds both" might be clearer. > "The number of pollfd structures specified by nfds exceeds both the system tunable <mumble> and FD_SETSIZE" Or even "nfds is larger than both the system tunable <mumble> and FD_SETSIZE" since the 'The number of pollfd structures" is repetitive given one can go look up what it specifies. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoNoc2hdyw9j-wZ7APZZS56vMspU08gJA5VXuBho_0q_g>