On Mon, Sep 21, 2020 at 03:30:24PM +0100, Alexandru Elisei wrote: > Hi Andrew, > > Thank you for having a look at this. > > On 9/21/20 3:17 PM, Andrew Jones wrote: > > > On Mon, Sep 21, 2020 at 02:08:38PM +0100, Alexandru Elisei wrote: > >> Update the description of the PMU KVM_{GET, SET}_DEVICE_ATTR error codes > >> to be a better match for the code that returns them. > >> > >> Signed-off-by: Alexandru Elisei <alexandru.elisei@xxxxxxx> > >> --- > >> Documentation/virt/kvm/devices/vcpu.rst | 8 ++++---- > >> 1 file changed, 4 insertions(+), 4 deletions(-) > >> > >> diff --git a/Documentation/virt/kvm/devices/vcpu.rst b/Documentation/virt/kvm/devices/vcpu.rst > >> index 96d6cf77cb1e..5b8db30c61db 100644 > >> --- a/Documentation/virt/kvm/devices/vcpu.rst > >> +++ b/Documentation/virt/kvm/devices/vcpu.rst > >> @@ -26,8 +26,8 @@ Returns: > >> ======= ======================================================== > >> -EBUSY The PMU overflow interrupt is already set > >> -EFAULT Error reading interrupt number > >> - -ENXIO The overflow interrupt not set when attempting to get it > >> - -ENODEV PMUv3 not supported > >> + -ENXIO PMUv3 not supported > > I think -ENXIO matched the code better before the change. > > KVM_SET_DEVICE_ATTR for KVM_ARM_VCPU_PMU_V3_IRQ returns -ENXIO if > CONFIG_KVM_ARM_PMU is not set (in include/kvm/arm_pmu.h). > > But I think I understand what you mean, you're referring to > kvm_arm_pmu_v3_get_attr(), which returns -ENXIO if the irq number was not set. I > can change the description to "PMUv3 not supported or the overflow interrupt not > set when attempting to get it", what do you think? > Works for me. Thanks, drew _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm