lkml.org 
[lkml]   [2019]   [Jan]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH V6 3/5] perf/x86/intel: Clean up SNB pebs quirk
    Date
    From: Kan Liang <kan.liang@linux.intel.com>

    Clean up SNB pebs quirk to use the new facility to check for min
    microcode revisions.

    Only check the boot CPU, assuming models and features are consistent
    over all CPUs.

    Signed-off-by: Kan Liang <kan.liang@linux.intel.com>
    ---

    Changes since V5:
    - New patch to address Peter's comments.

    arch/x86/events/intel/core.c | 35 ++++++++++-------------------------
    1 file changed, 10 insertions(+), 25 deletions(-)

    diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
    index 13baff5..a3c7845 100644
    --- a/arch/x86/events/intel/core.c
    +++ b/arch/x86/events/intel/core.c
    @@ -3780,36 +3780,21 @@ static void intel_check_isolation(void)
    x86_pmu.pebs_no_isolation = !x86_cpu_has_min_microcode_rev(isolation_ucodes);
    }

    -static int intel_snb_pebs_broken(int cpu)
    -{
    - u32 rev = UINT_MAX; /* default to broken for unknown models */
    -
    - switch (cpu_data(cpu).x86_model) {
    - case INTEL_FAM6_SANDYBRIDGE:
    - rev = 0x28;
    - break;
    -
    - case INTEL_FAM6_SANDYBRIDGE_X:
    - switch (cpu_data(cpu).x86_stepping) {
    - case 6: rev = 0x618; break;
    - case 7: rev = 0x70c; break;
    - }
    - }
    +static const struct x86_cpu_desc pebs_ucodes[] = {
    + INTEL_CPU_DESC(INTEL_FAM6_SANDYBRIDGE, 7, 0x00000028),
    + INTEL_CPU_DESC(INTEL_FAM6_SANDYBRIDGE_X, 6, 0x00000618),
    + INTEL_CPU_DESC(INTEL_FAM6_SANDYBRIDGE_X, 7, 0x0000070c),
    + {}
    +};

    - return (cpu_data(cpu).microcode < rev);
    +static bool intel_snb_pebs_broken(void)
    +{
    + return !x86_cpu_has_min_microcode_rev(pebs_ucodes);
    }

    static void intel_snb_check_microcode(void)
    {
    - int pebs_broken = 0;
    - int cpu;
    -
    - for_each_online_cpu(cpu) {
    - if ((pebs_broken = intel_snb_pebs_broken(cpu)))
    - break;
    - }
    -
    - if (pebs_broken == x86_pmu.pebs_broken)
    + if (intel_snb_pebs_broken() == x86_pmu.pebs_broken)
    return;

    /*
    --
    2.7.4
    \
     
     \ /
      Last update: 2019-01-21 22:44    [W:2.579 / U:0.116 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site