[intel-tdx:kvm-upstream 121/267] arch/x86/events/core.c:2114 init_hw_perf_events() warn: missing error code 'err'

From: Dan Carpenter
Date: Tue Jun 28 2022 - 03:35:57 EST


tree: https://github.com/intel/tdx.git kvm-upstream
head: 7af4efe32638544aecb58ed7365d0ef2ea6f85ea
commit: ac0739f7f73f45210f810e6e389416b5b62bf9d3 [121/267] x86: events: Do not return bogus capabilities if PMU is broken
config: x86_64-randconfig-m001-20220627 (https://download.01.org/0day-ci/archive/20220628/202206280900.c8PW41Dn-lkp@xxxxxxxxx/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>
Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

smatch warnings:
arch/x86/events/core.c:2114 init_hw_perf_events() warn: missing error code 'err'

vim +/err +2114 arch/x86/events/core.c

dda99116969142c arch/x86/kernel/cpu/perf_event.c Yinghai Lu 2011-01-21 2079 static int __init init_hw_perf_events(void)
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2080 {
c1d6f42f1a42c72 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2011-12-06 2081 struct x86_pmu_quirk *quirk;
72eae04d3a3075c arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2082 int err;
72eae04d3a3075c arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2083
cdd6c482c9ff9c5 arch/x86/kernel/cpu/perf_event.c Ingo Molnar 2009-09-21 2084 pr_info("Performance Events: ");
1123e3ad73697d6 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2009-05-29 2085
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2086 switch (boot_cpu_data.x86_vendor) {
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2087 case X86_VENDOR_INTEL:
72eae04d3a3075c arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2088 err = intel_pmu_init();
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2089 break;
f87ad35d37fa543 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2090 case X86_VENDOR_AMD:
72eae04d3a3075c arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2091 err = amd_pmu_init();
f87ad35d37fa543 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2092 break;
6d0ef316b9f8ea0 arch/x86/events/core.c Pu Wen 2018-09-23 2093 case X86_VENDOR_HYGON:
6d0ef316b9f8ea0 arch/x86/events/core.c Pu Wen 2018-09-23 2094 err = amd_pmu_init();
6d0ef316b9f8ea0 arch/x86/events/core.c Pu Wen 2018-09-23 2095 x86_pmu.name = "HYGON";
6d0ef316b9f8ea0 arch/x86/events/core.c Pu Wen 2018-09-23 2096 break;
3a4ac121c2cacbf arch/x86/events/core.c CodyYao-oc 2020-04-13 2097 case X86_VENDOR_ZHAOXIN:
3a4ac121c2cacbf arch/x86/events/core.c CodyYao-oc 2020-04-13 2098 case X86_VENDOR_CENTAUR:
3a4ac121c2cacbf arch/x86/events/core.c CodyYao-oc 2020-04-13 2099 err = zhaoxin_pmu_init();
3a4ac121c2cacbf arch/x86/events/core.c CodyYao-oc 2020-04-13 2100 break;
4138960a9251a26 arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2101 default:
8a3da6c7d0031fc arch/x86/kernel/cpu/perf_event.c Ingo Molnar 2013-09-28 2102 err = -ENOTSUPP;
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2103 }
1123e3ad73697d6 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2009-05-29 2104 if (err != 0) {
cdd6c482c9ff9c5 arch/x86/kernel/cpu/perf_event.c Ingo Molnar 2009-09-21 2105 pr_cont("no PMU driver, software events only.\n");
ac0739f7f73f452 arch/x86/events/core.c Paolo Bonzini 2022-06-01 2106 err = 0;
ac0739f7f73f452 arch/x86/events/core.c Paolo Bonzini 2022-06-01 2107 goto out_bad_pmu;
1123e3ad73697d6 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2009-05-29 2108 }
b56a3802dc6df29 arch/x86/kernel/cpu/perf_counter.c Jaswinder Singh Rajput 2009-02-27 2109
125580380f41800 arch/x86/kernel/cpu/perf_event.c Cyrill Gorcunov 2009-12-10 2110 pmu_check_apic();
125580380f41800 arch/x86/kernel/cpu/perf_event.c Cyrill Gorcunov 2009-12-10 2111
33c6d6a7ad0ffab arch/x86/kernel/cpu/perf_event.c Don Zickus 2010-11-22 2112 /* sanity check that the hardware exists or is emulated */
fc4b8fca2d8fc8a arch/x86/events/core.c Kan Liang 2021-04-12 2113 if (!check_hw_exists(&pmu, x86_pmu.num_counters, x86_pmu.num_counters_fixed))
ac0739f7f73f452 arch/x86/events/core.c Paolo Bonzini 2022-06-01 @2114 goto out_bad_pmu;

Does this need an error code? It's unclear why this needs a goto
out_bad_pmu when

33c6d6a7ad0ffab arch/x86/kernel/cpu/perf_event.c Don Zickus 2010-11-22 2115
1123e3ad73697d6 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2009-05-29 2116 pr_cont("%s PMU driver.\n", x86_pmu.name);
faa28ae018ed004 arch/x86/kernel/cpu/perf_counter.c Robert Richter 2009-04-29 2117
e97df76377b8b3b arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2014-02-05 2118 x86_pmu.attr_rdpmc = 1; /* enable userspace RDPMC usage by default */
e97df76377b8b3b arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2014-02-05 2119
c1d6f42f1a42c72 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2011-12-06 2120 for (quirk = x86_pmu.quirks; quirk; quirk = quirk->next)
c1d6f42f1a42c72 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2011-12-06 2121 quirk->func();
3c44780b220e876 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-03-04 2122
a1eac7ac903ea9a arch/x86/kernel/cpu/perf_event.c Robert Richter 2012-06-20 2123 if (!x86_pmu.intel_ctrl)
948b1bb89a44561 arch/x86/kernel/cpu/perf_event.c Robert Richter 2010-03-29 2124 x86_pmu.intel_ctrl = (1 << x86_pmu.num_counters) - 1;
241771ef016b5c0 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2008-12-03 2125
cdd6c482c9ff9c5 arch/x86/kernel/cpu/perf_event.c Ingo Molnar 2009-09-21 2126 perf_events_lapic_init();
9c48f1c629ecfa1 arch/x86/kernel/cpu/perf_event.c Don Zickus 2011-09-30 2127 register_nmi_handler(NMI_LOCAL, perf_event_nmi_handler, 0, "PMI");
1123e3ad73697d6 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2009-05-29 2128
63b146490befc02 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-01-22 2129 unconstrained = (struct event_constraint)
948b1bb89a44561 arch/x86/kernel/cpu/perf_event.c Robert Richter 2010-03-29 2130 __EVENT_CONSTRAINT(0, (1ULL << x86_pmu.num_counters) - 1,
9fac2cf316b070a arch/x86/kernel/cpu/perf_event.c Stephane Eranian 2013-01-24 2131 0, x86_pmu.num_counters, 0, 0);
63b146490befc02 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-01-22 2132
641cc938815dfd0 arch/x86/kernel/cpu/perf_event.c Jiri Olsa 2012-03-15 2133 x86_pmu_format_group.attrs = x86_pmu.format_attrs;
0c9d42ed4cee2aa arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2011-11-20 2134
a47473939db20e3 arch/x86/kernel/cpu/perf_event.c Jiri Olsa 2012-10-10 2135 if (!x86_pmu.events_sysfs_show)
a47473939db20e3 arch/x86/kernel/cpu/perf_event.c Jiri Olsa 2012-10-10 2136 x86_pmu_events_group.attrs = &empty_attrs;
a47473939db20e3 arch/x86/kernel/cpu/perf_event.c Jiri Olsa 2012-10-10 2137
baa0c83363c7aaf arch/x86/events/core.c Jiri Olsa 2019-05-12 2138 pmu.attr_update = x86_pmu.attr_update;
6089327f5424f22 arch/x86/events/core.c Kan Liang 2017-05-12 2139
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2140 if (!is_hybrid()) {
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2141 x86_pmu_show_pmu_cap(x86_pmu.num_counters,
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2142 x86_pmu.num_counters_fixed,
e11c1a7eb302ac8 arch/x86/events/core.c Kan Liang 2021-04-12 2143 x86_pmu.intel_ctrl);
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2144 }
3f6da3905398826 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-03-05 2145
7c9903c9bf716d8 arch/x86/events/core.c Peter Zijlstra 2020-08-18 2146 if (!x86_pmu.read)
7c9903c9bf716d8 arch/x86/events/core.c Peter Zijlstra 2020-08-18 2147 x86_pmu.read = _x86_pmu_read;
7c9903c9bf716d8 arch/x86/events/core.c Peter Zijlstra 2020-08-18 2148
abd562df94d19d0 arch/x86/events/core.c Like Xu 2021-01-25 2149 if (!x86_pmu.guest_get_msrs)
c8e2fe13d1d1f3a arch/x86/events/core.c Sean Christopherson 2021-03-09 2150 x86_pmu.guest_get_msrs = (void *)&__static_call_return0;
abd562df94d19d0 arch/x86/events/core.c Like Xu 2021-01-25 2151
7c9903c9bf716d8 arch/x86/events/core.c Peter Zijlstra 2020-08-18 2152 x86_pmu_static_call_update();
7c9903c9bf716d8 arch/x86/events/core.c Peter Zijlstra 2020-08-18 2153
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2154 /*
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2155 * Install callbacks. Core will call them for each online
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2156 * cpu.
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2157 */
73c1b41e63f040e arch/x86/events/core.c Thomas Gleixner 2016-12-21 2158 err = cpuhp_setup_state(CPUHP_PERF_X86_PREPARE, "perf/x86:prepare",
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2159 x86_pmu_prepare_cpu, x86_pmu_dead_cpu);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2160 if (err)
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2161 return err;

This error path does a direct return

95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2162
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2163 err = cpuhp_setup_state(CPUHP_AP_PERF_X86_STARTING,
73c1b41e63f040e arch/x86/events/core.c Thomas Gleixner 2016-12-21 2164 "perf/x86:starting", x86_pmu_starting_cpu,
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2165 x86_pmu_dying_cpu);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2166 if (err)
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2167 goto out;
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2168
73c1b41e63f040e arch/x86/events/core.c Thomas Gleixner 2016-12-21 2169 err = cpuhp_setup_state(CPUHP_AP_PERF_X86_ONLINE, "perf/x86:online",
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2170 x86_pmu_online_cpu, NULL);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2171 if (err)
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2172 goto out1;
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2173
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2174 if (!is_hybrid()) {
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2175 err = perf_pmu_register(&pmu, "cpu", PERF_TYPE_RAW);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2176 if (err)
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2177 goto out2;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2178 } else {
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2179 u8 cpu_type = get_this_hybrid_cpu_type();
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2180 struct x86_hybrid_pmu *hybrid_pmu;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2181 int i, j;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2182
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2183 if (!cpu_type && x86_pmu.get_hybrid_cpu_type)
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2184 cpu_type = x86_pmu.get_hybrid_cpu_type();
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2185
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2186 for (i = 0; i < x86_pmu.num_hybrid_pmus; i++) {
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2187 hybrid_pmu = &x86_pmu.hybrid_pmu[i];
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2188
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2189 hybrid_pmu->pmu = pmu;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2190 hybrid_pmu->pmu.type = -1;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2191 hybrid_pmu->pmu.attr_update = x86_pmu.attr_update;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2192 hybrid_pmu->pmu.capabilities |= PERF_PMU_CAP_HETEROGENEOUS_CPUS;
55bcf6ef314ae8b arch/x86/events/core.c Kan Liang 2021-04-12 2193 hybrid_pmu->pmu.capabilities |= PERF_PMU_CAP_EXTENDED_HW_TYPE;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2194
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2195 err = perf_pmu_register(&hybrid_pmu->pmu, hybrid_pmu->name,
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2196 (hybrid_pmu->cpu_type == hybrid_big) ? PERF_TYPE_RAW : -1);
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2197 if (err)
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2198 break;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2199
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2200 if (cpu_type == hybrid_pmu->cpu_type)
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2201 x86_pmu_update_cpu_context(&hybrid_pmu->pmu, raw_smp_processor_id());
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2202 }
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2203
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2204 if (i < x86_pmu.num_hybrid_pmus) {
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2205 for (j = 0; j < i; j++)
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2206 perf_pmu_unregister(&x86_pmu.hybrid_pmu[j].pmu);
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2207 pr_warn("Failed to register hybrid PMUs\n");
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2208 kfree(x86_pmu.hybrid_pmu);
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2209 x86_pmu.hybrid_pmu = NULL;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2210 x86_pmu.num_hybrid_pmus = 0;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2211 goto out2;
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2212 }
d9977c43bff895e arch/x86/events/core.c Kan Liang 2021-04-12 2213 }
004417a6d468e24 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-11-25 2214
004417a6d468e24 arch/x86/kernel/cpu/perf_event.c Peter Zijlstra 2010-11-25 2215 return 0;
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2216
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2217 out2:
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2218 cpuhp_remove_state(CPUHP_AP_PERF_X86_ONLINE);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2219 out1:
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2220 cpuhp_remove_state(CPUHP_AP_PERF_X86_STARTING);
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2221 out:
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2222 cpuhp_remove_state(CPUHP_PERF_X86_PREPARE);
ac0739f7f73f452 arch/x86/events/core.c Paolo Bonzini 2022-06-01 2223 out_bad_pmu:
ac0739f7f73f452 arch/x86/events/core.c Paolo Bonzini 2022-06-01 2224 memset(&x86_pmu, 0, sizeof(x86_pmu));
95ca792c7582fde arch/x86/events/core.c Thomas Gleixner 2016-07-13 2225 return err;
241771ef016b5c0 arch/x86/kernel/cpu/perf_counter.c Ingo Molnar 2008-12-03 2226 }

--
0-DAY CI Kernel Test Service
https://01.org/lkp