lkml.org 
[lkml]   [2022]   [Aug]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 100/136] perf/x86/lbr: Enable the branch type for the Arch LBR by default
    Date
    From: Kan Liang <kan.liang@linux.intel.com>

    commit 32ba156df1b1c8804a4e5be5339616945eafea22 upstream.

    On the platform with Arch LBR, the HW raw branch type encoding may leak
    to the perf tool when the SAVE_TYPE option is not set.

    In the intel_pmu_store_lbr(), the HW raw branch type is stored in
    lbr_entries[].type. If the SAVE_TYPE option is set, the
    lbr_entries[].type will be converted into the generic PERF_BR_* type
    in the intel_pmu_lbr_filter() and exposed to the user tools.
    But if the SAVE_TYPE option is NOT set by the user, the current perf
    kernel doesn't clear the field. The HW raw branch type leaks.

    There are two solutions to fix the issue for the Arch LBR.
    One is to clear the field if the SAVE_TYPE option is NOT set.
    The other solution is to unconditionally convert the branch type and
    expose the generic type to the user tools.

    The latter is implemented here, because
    - The branch type is valuable information. I don't see a case where
    you would not benefit from the branch type. (Stephane Eranian)
    - Not having the branch type DOES NOT save any space in the
    branch record (Stephane Eranian)
    - The Arch LBR HW can retrieve the common branch types from the
    LBR_INFO. It doesn't require the high overhead SW disassemble.

    Fixes: 47125db27e47 ("perf/x86/intel/lbr: Support Architectural LBR")
    Reported-by: Stephane Eranian <eranian@google.com>
    Signed-off-by: Kan Liang <kan.liang@linux.intel.com>
    Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
    Cc: stable@vger.kernel.org
    Link: https://lkml.kernel.org/r/20220816125612.2042397-1-kan.liang@linux.intel.com
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    arch/x86/events/intel/lbr.c | 8 ++++++++
    1 file changed, 8 insertions(+)

    --- a/arch/x86/events/intel/lbr.c
    +++ b/arch/x86/events/intel/lbr.c
    @@ -1114,6 +1114,14 @@ static int intel_pmu_setup_hw_lbr_filter

    if (static_cpu_has(X86_FEATURE_ARCH_LBR)) {
    reg->config = mask;
    +
    + /*
    + * The Arch LBR HW can retrieve the common branch types
    + * from the LBR_INFO. It doesn't require the high overhead
    + * SW disassemble.
    + * Enable the branch type by default for the Arch LBR.
    + */
    + reg->reg |= X86_BR_TYPE_SAVE;
    return 0;
    }


    \
     
     \ /
      Last update: 2022-08-29 13:31    [W:3.637 / U:0.748 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site