From: Ian Jackson Date: Sun, 21 Sep 2014 14:02:40 +0000 (+0100) Subject: poll: Document reentrancy restriction on before() X-Git-Tag: proposed.polypath.v1~24 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ian/git?a=commitdiff_plain;ds=sidebyside;h=e9c39cc8b7fef143123ea632a3d43b886c03750f;p=secnet.git poll: Document reentrancy restriction on before() If the before() callback might modify the wanted fds or timeouts of other poll users, the loop over poll users in run() might produce wrong answers. Therefore, document that this is not permitted. (All of the existing before() implementations are indeed fine.) Signed-off-by: Ian Jackson --- diff --git a/secnet.h b/secnet.h index 40ad1d3..72e3e7c 100644 --- a/secnet.h +++ b/secnet.h @@ -198,7 +198,14 @@ int32_t calculate_max_start_pad(void); /* If nfds_io is insufficient for your needs, set it to the required number and return ERANGE. timeout is in milliseconds; if it is too - high then lower it. It starts at -1 (==infinite) */ + high then lower it. It starts at -1 (==infinite). */ +/* Note that beforepoll_fn may NOT do anything which might change the + fds or timeouts wanted by other registered poll loop loopers. + Callers should make sure of this by not making any calls into other + modules from the beforepoll_fn; the easiest way to ensure this is + for beforepoll_fn to only retreive information and not take any + action. + */ typedef int beforepoll_fn(void *st, struct pollfd *fds, int *nfds_io, int *timeout_io); typedef void afterpoll_fn(void *st, struct pollfd *fds, int nfds);