Date: Sun, 23 Aug 2009 16:12:18 -0400 From: Nathan Lay <nslay@comcast.net> To: Ed Schouten <ed@80386.nl> Cc: freebsd-current@FreeBSD.org, Jeremie Le Hen <jeremie@le-hen.org> Subject: Re: truss(1) locked in kernel with 8.0-BETA2 Message-ID: <4A91A2A2.9000004@comcast.net> In-Reply-To: <20090823065058.GR1292@hoeg.nl> References: <20090822185812.GC61707@felucia.tataz.chchile.org> <20090822194009.GQ1292@hoeg.nl> <4A90C099.8030605@comcast.net> <20090823065058.GR1292@hoeg.nl>
next in thread | previous in thread | raw e-mail | index | archive | help
Ed Schouten wrote:
> * Nathan Lay <nslay@comcast.net> wrote:
>
>> Instead of zeroing and setting master and STDIN_FILENO on the fd_set
>> every iteration, <snip>
>>
>
> The current code doesn't zero the fdsets each iteration, only during
> startup:
>
> | FD_ZERO(&rfd);
> | for (;;) {
> | FD_SET(master, &rfd);
> | if (!nostdin)
> | FD_SET(STDIN_FILENO, &rfd);
>
>
Ah right, I misread. In any case, once 0 bytes are written to the
master descriptor is that continually needed and does select() really
need to continue to examine STDIN_FILENO?
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4A91A2A2.9000004>