mbox series

[0/3] Standardize kvm exit reason field

Message ID 20191212024512.39930-1-gshan@redhat.com (mailing list archive)
Headers show
Series Standardize kvm exit reason field | expand

Message

Gavin Shan Dec. 12, 2019, 2:45 a.m. UTC
Some utilities (e.g. kvm_stat) have the assumption that same filter
field "exit_reason" for kvm exit tracepoint. However, they're varied
on different architects. It can be fixed for the utilities to pick
different filter names, or standardize it to have unified name "exit_reason",
The series takes the second approach, suggested by Vitaly Kuznetsov.

I'm not sure which git tree this will go, so I have one separate patch
for each architect in case they are merged to different git tree.

Gavin Shan (3):
  kvm/mips: Standardize kvm exit reason field
  kvm/powerpc: Standardize kvm exit reason field
  kvm/aarch64: Standardize kvm exit reason field

 arch/mips/kvm/trace.h          | 10 +++++-----
 arch/powerpc/kvm/trace_booke.h | 10 +++++-----
 arch/powerpc/kvm/trace_pr.h    | 10 +++++-----
 virt/kvm/arm/trace.h           | 14 ++++++++------
 4 files changed, 23 insertions(+), 21 deletions(-)

Comments

Paolo Bonzini Jan. 18, 2020, 7:16 p.m. UTC | #1
On 12/12/19 03:45, Gavin Shan wrote:
> Some utilities (e.g. kvm_stat) have the assumption that same filter
> field "exit_reason" for kvm exit tracepoint. However, they're varied
> on different architects. It can be fixed for the utilities to pick
> different filter names, or standardize it to have unified name "exit_reason",
> The series takes the second approach, suggested by Vitaly Kuznetsov.
> 
> I'm not sure which git tree this will go, so I have one separate patch
> for each architect in case they are merged to different git tree.
> 
> Gavin Shan (3):
>   kvm/mips: Standardize kvm exit reason field
>   kvm/powerpc: Standardize kvm exit reason field
>   kvm/aarch64: Standardize kvm exit reason field
> 
>  arch/mips/kvm/trace.h          | 10 +++++-----
>  arch/powerpc/kvm/trace_booke.h | 10 +++++-----
>  arch/powerpc/kvm/trace_pr.h    | 10 +++++-----
>  virt/kvm/arm/trace.h           | 14 ++++++++------
>  4 files changed, 23 insertions(+), 21 deletions(-)
> 

Queued, thanks (patches 1-2).

Paolo