Re: [PATCH v2] kvm_host: unify VM_STAT and VCPU_STAT definitions in a single place

2020-04-15 Thread Cornelia Huck
On Tue, 14 Apr 2020 17:56:25 +0200 Emanuele Giuseppe Esposito wrote: > The macros VM_STAT and VCPU_STAT are redundantly implemented in multiple > files, each used by a different architecure to initialize the debugfs > entries for statistics. Since they all have the same purpose, they can be > uni

Re: [PATCH v2] kvm_host: unify VM_STAT and VCPU_STAT definitions in a single place

2020-04-15 Thread Claudio Imbrenda
On Wed, 15 Apr 2020 10:30:06 +0200 Emanuele Giuseppe Esposito wrote: > On 4/15/20 8:36 AM, Philippe Mathieu-Daudé wrote: > > On 4/14/20 5:56 PM, Emanuele Giuseppe Esposito wrote: > >> The macros VM_STAT and VCPU_STAT are redundantly implemented in > >> multiple files, each used by a different a

Re: [PATCH v2] kvm_host: unify VM_STAT and VCPU_STAT definitions in a single place

2020-04-15 Thread Emanuele Giuseppe Esposito
On 4/15/20 8:36 AM, Philippe Mathieu-Daudé wrote: On 4/14/20 5:56 PM, Emanuele Giuseppe Esposito wrote: The macros VM_STAT and VCPU_STAT are redundantly implemented in multiple files, each used by a different architecure to initialize the debugfs entries for statistics. Since they all have the

Re: [PATCH v2] kvm_host: unify VM_STAT and VCPU_STAT definitions in a single place

2020-04-15 Thread Philippe Mathieu-Daudé
On 4/14/20 5:56 PM, Emanuele Giuseppe Esposito wrote: > The macros VM_STAT and VCPU_STAT are redundantly implemented in multiple > files, each used by a different architecure to initialize the debugfs > entries for statistics. Since they all have the same purpose, they can be > unified in a single

[PATCH v2] kvm_host: unify VM_STAT and VCPU_STAT definitions in a single place

2020-04-14 Thread Emanuele Giuseppe Esposito
The macros VM_STAT and VCPU_STAT are redundantly implemented in multiple files, each used by a different architecure to initialize the debugfs entries for statistics. Since they all have the same purpose, they can be unified in a single common definition in include/linux/kvm_host.h Signed-off-by: