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