Re: [PATCH v2 03/31] qapi/qom: Add ObjectOptions for iothread

On 2/24/21 7:52 AM, Kevin Wolf wrote: > Add an ObjectOptions union that will eventually describe the options of > all user creatable object types. As unions can't exist without any > branches, also add the first object type. > > This adds a QAPI schema for the properties of the iothread object. > > Signed-off-by: Kevin Wolf <kwolf(a)redhat.com&gt; > --- > qapi/qom.json | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 53 insertions(+) > > diff --git a/qapi/qom.json b/qapi/qom.json > index 96c91c1faf..bf2ecb34be 100644 > --- a/qapi/qom.json > +++ b/qapi/qom.json > @@ -202,6 +202,59 @@ > 'returns': [ 'ObjectPropertyInfo' ], > 'allow-preconfig': true } > > +## > +# @IothreadProperties: > +# > +# Properties for iothread objects. > +# > +# @poll-max-ns: the maximum number of nanoseconds to busy wait for events. > +# 0 means polling is disabled (default: 32768 on POSIX hosts, > +# 0 otherwise) > +# > +# @poll-grow: the multiplier used to increase the polling time when the > +# algorithm detects it is missing events due to not polling long > +# enough. 0 selects a default behaviour (default: 0) > +# > +# @poll-shrink: the divisor used to decrease the polling time when the > +# algorithm detects it is spending too long polling without > +# encountering events. 0 selects a default behaviour (default: 0) Matches PollParamInfo declarations in iothread.c. > +# > +# Since: 2.0 How did you determine this value? (I'm not questioning it being something other than 6.0, because we have indeed supported QMP configuration of these values via the untyped magic previously present in add-object).

> +## > +{ 'struct': 'IothreadProperties', > + 'data': { '*poll-max-ns': 'int', > + '*poll-grow': 'int', > + '*poll-shrink': 'int' } } These are correctly typed per the code in iothread.c, but it does raise the question of whether a signed 64-bit value is the best choice, or if we might later want to revisit things to pick more constrained types. I don't think such an audit should hold up this series, though.