bug-hurd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH,HURD] hurdselect: Step1, code split preparations


From: Svante Signell
Subject: Re: [PATCH,HURD] hurdselect: Step1, code split preparations
Date: Tue, 22 Jan 2013 20:53:06 +0100

On Tue, 2013-01-22 at 20:37 +0100, Svante Signell wrote:
> On Tue, 2013-01-22 at 19:01 +0100, Samuel Thibault wrote:
> > Svante Signell, le Tue 22 Jan 2013 18:53:43 +0100, a écrit :
> > > Attached is the first patch for a 3-way split of hurdselect.c into three
> > > cases: DELAY, POLL, SELECT leading to a more POSIX conforming POLL.
> > 
> > It is way more readable that the previous versions :)
> > 
> > > +  if (nfds > _hurd_dtablesize)
> > > +    nfds = _hurd_dtablesize;
> > 
> > You can't afford moving that to the general case: in the poll case, nfds
> > is not the maximum of the fds, but the number of elements in the array,
> > which can be very big, if the application puts the same fd several time
> > in the array.
> 
> I did update the patch but it was not included in the attachment (same
> problem with Pinos comment on fd < _hurd_tablesize) sorry.

Updated patch attached (not reindented). Shall I continue sending
patches or not?
And what about submitting to debian-hurd instead of bug-hurd? 


Attachment: hurdselect_step1_1.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]