diff options
author | Tomas Mraz <tomas@openssl.org> | 2023-07-24 18:05:47 +0200 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2023-08-08 15:58:59 +0200 |
commit | 44cb36d04adb737be1aee32908232003deeb67dd (patch) | |
tree | f1fe8280f92f99a4316db58b3c3b6de628d16110 /ssl/quic/quic_reactor.c | |
parent | a2ca189e273584a7af3fcb90d893df9439e96659 (diff) |
Resolve some of the TODO(QUIC) items
For some of the items we add FUTURE/SERVER/TESTING/MULTIPATH
designation to indicate these do not need to be resolved
in QUIC MVP release.
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21539)
Diffstat (limited to 'ssl/quic/quic_reactor.c')
-rw-r--r-- | ssl/quic/quic_reactor.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/ssl/quic/quic_reactor.c b/ssl/quic/quic_reactor.c index ef447b78c3..f89337b38e 100644 --- a/ssl/quic/quic_reactor.c +++ b/ssl/quic/quic_reactor.c @@ -342,14 +342,14 @@ int ossl_quic_reactor_block_until_pred(QUIC_REACTOR *rtor, * things again. If poll_two_fds returns 0, this is some other * non-timeout failure and we should stop here. * - * TODO(QUIC): In the future we could avoid unnecessary syscalls by - * not retrying network I/O that isn't ready based on the result of - * the poll call. However this might be difficult because it - * requires we do the call to poll(2) or equivalent syscall - * ourselves, whereas in the general case the application does the - * polling and just calls SSL_handle_events(). Implementing this - * optimisation in the future will probably therefore require API - * changes. + * TODO(QUIC FUTURE): In the future we could avoid unnecessary + * syscalls by not retrying network I/O that isn't ready based + * on the result of the poll call. However this might be difficult + * because it requires we do the call to poll(2) or equivalent + * syscall ourselves, whereas in the general case the application + * does the polling and just calls SSL_handle_events(). + * Implementing this optimisation in the future will probably + * therefore require API changes. */ return 0; } |