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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html