[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finish
From: |
Samuel Thibault |
Subject: |
Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished |
Date: |
Tue, 12 Feb 2013 22:55:28 +0100 |
User-agent: |
Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30) |
Svante Signell, le Tue 12 Feb 2013 09:09:25 +0100, a écrit :
> Attached is a patch wrt the latest patch (step2) on January 24 2013. The
> indentation is different, therefore the diff blocks are a little large
> (but easier to read). Hopefully this is better. Including also the
> resulting file hurdselect_step7x.c. Explanations later in a separate
> mail.
Explanations don't need to be very long. All we need is the rationale
and the way you handle it in the patch.
AIUI, there are three things:
- pass 1ms as timeout. This will be fixed by Richard's patch.
- in the poll() case, do not fail completely if an FD is bogus. Instead,
poll on the remaining FDs, and later on set POLLNVAL. It seems your
patch does this by using the i_index array indirection.
- in the poll() case, on error returned by io_select, instead of setting
readiness, set POLLHUP or POLLERR according to the error.
Is that right?
Samuel
- Re: [PATCH,HURD][RFC] hurdselect: Step7, almost complete rewrite finished, Svante Signell, 2013/02/11
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished,
Samuel Thibault <=
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Samuel Thibault, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Samuel Thibault, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Samuel Thibault, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Samuel Thibault, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Richard Braun, 2013/02/12
- Re: [PATCH,HURD][RFC] hurdselect: Step7x, almost complete rewrite finished, Svante Signell, 2013/02/13