Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Jing Zhang <jingzhangos@xxxxxxxxxx>, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
From: Paolo Bonzini <pbonzini@xxxxxxxxxx>
Date: Thu, 17 Jun 2021 17:29:12 +0200
Cc: KVM <kvm@xxxxxxxxxxxxxxx>, KVMARM <kvmarm@xxxxxxxxxxxxxxxxxxxxx>, LinuxMIPS <linux-mips@xxxxxxxxxxxxxxx>, KVMPPC <kvm-ppc@xxxxxxxxxxxxxxx>, LinuxS390 <linux-s390@xxxxxxxxxxxxxxx>, Linuxkselftest <linux-kselftest@xxxxxxxxxxxxxxx>, Marc Zyngier <maz@xxxxxxxxxx>, James Morse <james.morse@xxxxxxx>, Julien Thierry <julien.thierry.kdev@xxxxxxxxx>, Suzuki K Poulose <suzuki.poulose@xxxxxxx>, Will Deacon <will@xxxxxxxxxx>, Huacai Chen <chenhuacai@xxxxxxxxxx>, Aleksandar Markovic <aleksandar.qemu.devel@xxxxxxxxx>, Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxx>, Janosch Frank <frankja@xxxxxxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Cornelia Huck <cohuck@xxxxxxxxxx>, Claudio Imbrenda <imbrenda@xxxxxxxxxxxxx>, Sean Christopherson <seanjc@xxxxxxxxxx>, Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>, Jim Mattson <jmattson@xxxxxxxxxx>, Peter Shier <pshier@xxxxxxxxxx>, Oliver Upton <oupton@xxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>, Emanuele Giuseppe Esposito <eesposit@xxxxxxxxxx>, David Matlack <dmatlack@xxxxxxxxxx>, Ricardo Koller <ricarkol@xxxxxxxxxx>, Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx>, Fuad Tabba <tabba@xxxxxxxxxx>
In-reply-to: <CAAdAUtiiQ0304vWR3Zm2XUKz374W4LY3=qdrFZCsQ27VkqMn9A@mail.gmail.com>
References: <20210617044146.2667540-1-jingzhangos@google.com> <20210617044146.2667540-3-jingzhangos@google.com> <YMr4rArKvj3obDEM@kroah.com> <CAAdAUtiiQ0304vWR3Zm2XUKz374W4LY3=qdrFZCsQ27VkqMn9A@mail.gmail.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1

On 17/06/21 16:56, Jing Zhang wrote:

Actually, it is really not easy to separate this change into two patches even by
following Paolo's suggestion. And it would be a surprise to userspace to see
only VM stats, no VCPU stats.

That does not matter. Having two or three patches is useful because itmakes review easier; they will never appear separately in a release.


Paolo

Follow-Ups:
- Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
  - From: Jing Zhang

References:
- [PATCH v10 0/5] KVM statistics data fd-based binary interface
  - From: Jing Zhang
- [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
  - From: Jing Zhang
- Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
  - From: Greg KH
- Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
  - From: Jing Zhang

Prev by Date: Re: [PATCH v10 3/5] KVM: stats: Add documentation for binary statistics interface
Next by Date: Re: [PATCH v10 3/5] KVM: stats: Add documentation for binary statistics interface
Previous by thread: Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
Next by thread: Re: [PATCH v10 2/5] KVM: stats: Add fd-based API to read binary stats data
Index(es):
- Date
- Thread

[Index of Archives] [LKML Archive] [Linux ARM Kernel] [Linux ARM] [Git] [Yosemite News] [Linux SCSI] [Linux Hams]