Re: epoll (was Re: [PATCH] async poll for 2.5)

John G. Myers (jgmyers@netscape.com)
Sat, 19 Oct 2002 11:52:31 -0700


On Friday, October 18, 2002, at 06:27 PM, Tervel Atanassov wrote:
> The code you have
> below seems a bit awkward -- the line while(do_io(fd) != EAGAIN)
> appears
> twice. I think the reason for that is that you're trying to do too
> many
> things at once, namely, you're trying to handle both the initial
> accept/setup of the socket and its steady state servicing. I don't see
> any benefit to that -- it definitely doesn't make for cleaner code.
> Why
> not do things separately.

If you carefully reread the message you replied to, you will see that
this is exactly what I am proposing. The redundant copy of the line
you consider awkward would be removed.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/