On 6/26/19 4:23 PM, Eric Blake wrote: > On 6/26/19 4:18 PM, John Snow wrote: >> >> >> On 6/5/19 12:18 PM, Vladimir Sementsov-Ogievskiy wrote: >>> Hi all! >>> >>> Here is a suggestion to enable keepalive option to track server availablity. >>> We suggest to enable it by default. If we need, we'll be able to add option >>> to specify timeout by hand later. >>> >>> v2: 01 - Fix io channel returned errors to be -1 [Daniel] >>> 02 - Fix typo in commit message [Eric] >>> >>> Vladimir Sementsov-Ogievskiy (2): >>> io/channel: add qio_channel_set_keepalive >>> nbd-client: enable TCP keepalive >>> >>> include/io/channel.h | 15 +++++++++++++++ >>> block/nbd-client.c | 1 + >>> io/channel-socket.c | 20 ++++++++++++++++++++ >>> io/channel.c | 14 ++++++++++++++ >>> 4 files changed, 50 insertions(+) >>> >> >> Ping -- I think this was good to go with Dan's ACK, based on the >> discussion from v1. > > Actually, I thought that we changed tactics, and that the latest version is: > > https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg01989.html > > [PATCH v2] qapi: InitSocketAddress: add keepalive option > > to make the setting conditional based on blockdev parameters rather than > unconditional.
Or even v3, which still had review comments pending: https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg05508.html [PATCH v3] qapi: Add InetSocketAddress member keep-alive -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature