* Eric Blake ([email protected]) wrote:
> On 1/28/19 11:03 AM, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <[email protected]>
> >
> > The 'announce timer' will be used by migration, and explicit
> > requests for qemu to perform network announces.
> >
> > Based on the work by Germano Veit Michel <[email protected]>
> > and Vlad Yasevich <[email protected]>
> >
> > Signed-off-by: Dr. David Alan Gilbert <[email protected]>
> > ---
>
> > +++ b/qapi/net.json
> > @@ -684,3 +684,26 @@
> > ##
> > { 'event': 'NIC_RX_FILTER_CHANGED',
> > 'data': { '*name': 'str', 'path': 'str' } }
> > +
> > +##
> > +# @AnnounceParameters:
> > +#
> > +# Parameters for self-announce timers
> > +#
> > +# @initial: Initial delay (in ms) before sending the first GARP/RARP
> > +# announcement
> > +#
> > +# @max: Maximum delay (in ms) to between GARP/RARP announcement packets
>
> "to between" should be either "to use in between" or merely "between"
I've changed it to the plain 'between'.
> > +#
> > +# @rounds: Number of self-announcement attempts
> > +#
> > +# @step: Delay increase (in ms) after each self-announcement attempt
> > +#
> > +# Since: 4.0
> > +##
> > +
> > +{ 'struct': 'AnnounceParameters',
> > + 'data': { 'initial': 'int',
> > + 'max': 'int',
> > + 'rounds': 'int',
> > + 'step': 'int' } }
> >
>
> Can/should any of the parameters be optional with a sane (documented)
> default?
I think last time Dan was arguing for it to be explicit, the only
default we could pick are the values we currently use, which I think are
only defendable as 'seem to have worked well so far'.
Dave
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3226
> Virtualization: qemu.org | libvirt.org
>
--
Dr. David Alan Gilbert / [email protected] / Manchester, UK
