powerpc/kvm: Remove comment related to moving PMU code to perf subsystem
authorKajol Jain <kjain@linux.ibm.com>
Mon, 11 Jul 2022 03:49:27 +0000 (09:19 +0530)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 20 Jul 2022 12:28:31 +0000 (22:28 +1000)
Commit aabcaf6ae2a0 ("KVM: PPC: Book3S HV P9: Move host OS save/restore
functions to  built-in") added a comment in switch_pmu_to_guest
function, indicating possibility of moving PMU handling code
to perf subsystem. But perf subsystem code compilation depends upon
the enablement of CONFIG_PERF_EVENTS whereas, kvm code don't have
any dependency on this config.
Patch remove this comment as switch_pmu_to_guest functionality is
needed even if perf subsystem is disabled.

Signed-off-by: Kajol Jain <kjain@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20220711034927.213192-2-kjain@linux.ibm.com
arch/powerpc/kvm/book3s_hv_p9_perf.c

index da3135cab9eaf9a97397086b44337e26fa108608..44d24cca3df1d54d104f19aa20608b815660541d 100644 (file)
@@ -44,12 +44,6 @@ void switch_pmu_to_guest(struct kvm_vcpu *vcpu,
 
        /* Save host */
        if (ppc_get_pmu_inuse()) {
-               /*
-                * It might be better to put PMU handling (at least for the
-                * host) in the perf subsystem because it knows more about what
-                * is being used.
-                */
-
                /* POWER9, POWER10 do not implement HPMC or SPMC */
 
                host_os_sprs->mmcr0 = mfspr(SPRN_MMCR0);