Quoting Ian McDonald:
|  >  a) You helped to uncover at least one real bug here:
|  >
|  >     long delay;
|  >
|  >     delay = msecs_to_jiffies(rc);
|  >     if (delay > *timeo || delay < 0)
|  >
|  
|  I was thinking of changing the type from long to unsigned long to
|  match and removing the test too. I agree totally.
For this case I have prepared a patch. With regard to the EAGAIN case I was
half waiting for Arnaldo to say something, for the moment I have put it here:
http://www.erg.abdn.ac.uk/users/gerrit/dccp/experimental_patches/15a_handle_EAGAIN.diff

I think we should fix the EAGAIN case. Maybe also something for EINTR (err=-4).
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to