On Tue, 16 Feb 2021 at 19:10, Daniel P. Berrangé <berra...@redhat.com> wrote: > > The preferred syntax is to use "foo=on|off", rather than a bare > "foo" or "nofoo". > > Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> > - "-chardev > socket,id=id[,host=host],port=port[,to=to][,ipv4][,ipv6][,nodelay][,reconnect=seconds]\n" > - " > [,server][,nowait][,telnet][,websocket][,reconnect=seconds][,mux=on|off]\n" > + "-chardev > socket,id=id[,host=host],port=port[,to=to][,ipv4=on|off][,ipv6=on|off][,delay=on|off][,reconnect=seconds]\n"
Watch out, 'nodelay' is a special case, where the option name itself starts with 'no': { 'struct': 'ChardevSocket', 'data': { 'addr': 'SocketAddressLegacy', '*tls-creds': 'str', '*tls-authz' : 'str', '*server': 'bool', '*wait': 'bool', '*nodelay': 'bool', '*telnet': 'bool', '*tn3270': 'bool', '*websocket': 'bool', '*reconnect': 'int' }, 'base': 'ChardevCommon' } (because it's setting the TCP_NODELAY option). This probably applies elsewhere, eg I suspect the 'nodelay' in patch 1 also should remain. -- PMM