On 7/4/22 16:00, zhenwei pi wrote:


    +##
    +# @GuestOsType:
    +#
    +# An enumeration of OS type
    +#
    +# Since: 7.1
    +##
    +{ 'enum': 'GuestOsType',
    +  'data': [ 'linuxos', 'windowsos' ] }


I would rather keep this enum specific to GuestCpuStats, "GuestLinuxCpuStatsType"?


Hi,

'GuestOsType' may be re-used in the future, not only for the CPU statistics case.

I would also drop the "os" suffix

I'm afraid we can not drop "os" suffix, build this without "os" suffix:
qga/qga-qapi-types.h:948:28: error: expected member name or ';' after declaration specifiers
         GuestLinuxCpuStats linux;
         ~~~~~~~~~~~~~~~~~~ ^
<built-in>:336:15: note: expanded from here
#define linux 1


Hi, Marc

Could you please give any hint about this issue?

    +
    +



Looks good to me otherwise.
thanks

--
Marc-André Lureau


--
zhenwei pi

Reply via email to