Re: [PATCH v2] chardev: introduce 'reconnect-ms' and deprecate 'reconnect'

Hi Mark-Andre! Could you please take a look? We have collected acks for QAPI changes, could this be queued?

> The 'reconnect' option only allows to specify the time in seconds, > which is way too long for certain workflows. > > We have a lightweight disk backend server, which takes about 20ms to > live update, but due to this limitation in QEMU, previously the guest > disk controller would hang for one second because it would take this > long for QEMU to reinitialize the socket connection. > > Introduce a new option called 'reconnect-ms', which is the same as > 'reconnect', except the value is treated as milliseconds. These are > mutually exclusive and specifying both results in an error. > > 'reconnect' is also deprecated by this commit to make it possible to > remove it in the future as to not keep two options that control the > same thing. > > Reviewed-by: Vladimir Sementsov-Ogievskiy<vsementsov(a)yandex-team.ru> > Acked-by: Peter Krempa<pkrempa(a)redhat.com> > Signed-off-by: Daniil Tatianin<d-tatianin(a)yandex-team.ru> -- Best regards, Vladimir