[v4] qga: add command 'guest-get-cpustats'

[PATCH v4 0/2] qga: add command 'guest-get-cpustats'

zhenwei pi posted 2 patches 1 year, 10 months ago

Diff against v1 v2 v3
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220707005602.696557-1-pizhenwei@bytedance.com

Maintainers: Michael Roth <michael.roth@amd.com>, Konstantin Kostiuk <kkostiuk@redhat.com>, Markus Armbruster <armbru@redhat.com>

qga/commands-posix.c   | 89 ++++++++++++++++++++++++++++++++++++++++++
qga/commands-win32.c   |  6 +++
qga/qapi-schema.json   | 81 ++++++++++++++++++++++++++++++++++++++
scripts/qapi/common.py |  2 +-
4 files changed, 177 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH v4 0/2] qga: add command 'guest-get-cpustats'

Posted by zhenwei pi 1 year, 10 months ago

v3 -> v4:
- Separate qapi script change from qga.

v2 -> v3:
- Rename 'GuestOsType' to 'GuestCpuStatsType'.
- Add 'linux' into polluted_words, rename 'linuxos' to 'linux'. Remove
  'windows' from 'GuestCpuStatsType', because currently we don't use it.

v1 -> v2:
- Konstantin & Marc-André pointed out that the structure 'GuestCpuStats'
  is too *linux style*, so re-define it to 'GuestLinuxCpuStats', and use
  an union type of 'GuestCpuStats'.

- Modify comment info from 'man proc', also add linux version infomation.

- Test sscanf return value by '(i == EOF)' (To Marc-André: name is declared
  as 'char name[64];', so we can't test '!name').

- Suggested by Marc-André, use 'int clk_tck = sysconf(_SC_CLK_TCK);'
  instead of hard code.

v1:
- Implement guest agent command 'guest-get-cpustats'

Zhenwei Pi (2):
  qapi: Avoid generating C identifier 'linux'
  qga: add command 'guest-get-cpustats'

 qga/commands-posix.c   | 89 ++++++++++++++++++++++++++++++++++++++++++
 qga/commands-win32.c   |  6 +++
 qga/qapi-schema.json   | 81 ++++++++++++++++++++++++++++++++++++++
 scripts/qapi/common.py |  2 +-
 4 files changed, 177 insertions(+), 1 deletion(-)

-- 
2.20.1

Re: [PATCH v4 0/2] qga: add command 'guest-get-cpustats'

Posted by Konstantin Kostiuk 1 year, 10 months ago

Queued, thanks.

On Thu, Jul 7, 2022 at 4:01 AM zhenwei pi <pizhenwei@bytedance.com> wrote:

> v3 -> v4:
> - Separate qapi script change from qga.
>
> v2 -> v3:
> - Rename 'GuestOsType' to 'GuestCpuStatsType'.
> - Add 'linux' into polluted_words, rename 'linuxos' to 'linux'. Remove
>   'windows' from 'GuestCpuStatsType', because currently we don't use it.
>
> v1 -> v2:
> - Konstantin & Marc-André pointed out that the structure 'GuestCpuStats'
>   is too *linux style*, so re-define it to 'GuestLinuxCpuStats', and use
>   an union type of 'GuestCpuStats'.
>
> - Modify comment info from 'man proc', also add linux version infomation.
>
> - Test sscanf return value by '(i == EOF)' (To Marc-André: name is declared
>   as 'char name[64];', so we can't test '!name').
>
> - Suggested by Marc-André, use 'int clk_tck = sysconf(_SC_CLK_TCK);'
>   instead of hard code.
>
> v1:
> - Implement guest agent command 'guest-get-cpustats'
>
> Zhenwei Pi (2):
>   qapi: Avoid generating C identifier 'linux'
>   qga: add command 'guest-get-cpustats'
>
>  qga/commands-posix.c   | 89 ++++++++++++++++++++++++++++++++++++++++++
>  qga/commands-win32.c   |  6 +++
>  qga/qapi-schema.json   | 81 ++++++++++++++++++++++++++++++++++++++
>  scripts/qapi/common.py |  2 +-
>  4 files changed, 177 insertions(+), 1 deletion(-)
>
> --
> 2.20.1
>
>