[tip: x86/cpu] x86/cpu: Inline generic_identify() into identify_cpu()

From: tip-bot2 for Ihor Solodrai

Date: Wed Sep 16 2026 - 22:05:20 EST


The following commit has been merged into the x86/cpu branch of tip:

Commit-ID: ad86fe2134ccdc97a967693d2723a731c99fb7bc
Gitweb: https://git.kernel.org/tip/ad86fe2134ccdc97a967693d2723a731c99fb7bc
Author: Ihor Solodrai <ihor.solodrai@xxxxxxxxx>
AuthorDate: Wed, 16 Sep 2026 12:52:01 -07:00
Committer: Borislav Petkov (AMD) <bp@xxxxxxxxx>
CommitterDate: Thu, 17 Sep 2026 02:00:25 +02:00

x86/cpu: Inline generic_identify() into identify_cpu()

generic_identify() has exactly one call site: at the top of
identify_cpu(). Fold it into identify_cpu() so that a single function
does the job for both the boot CPU and the secondary CPUs.

While at it, fix up both copies of the Cyrix comment.

No functional changes.

Signed-off-by: Ihor Solodrai <ihor.solodrai@xxxxxxxxx>
Signed-off-by: Borislav Petkov (AMD) <bp@xxxxxxxxx>
Link: https://patch.msgid.link/20260916195203.1099646-4-ihor.solodrai@xxxxxxxxx
---
arch/x86/kernel/cpu/common.c | 32 ++++++++++++++------------------
1 file changed, 14 insertions(+), 18 deletions(-)

diff --git a/arch/x86/kernel/cpu/common.c b/arch/x86/kernel/cpu/common.c
index d1a949a..79a1260 100644
--- a/arch/x86/kernel/cpu/common.c
+++ b/arch/x86/kernel/cpu/common.c
@@ -1827,7 +1827,7 @@ static void __init early_identify_cpu(struct cpuinfo_x86 *c)
if (!cpuid_feature())
identify_cpu_without_cpuid(c);

- /* cyrix could have cpuid enabled via c_identify()*/
+ /* Cyrix could have CPUID enabled via c_identify(). */
if (cpuid_feature()) {
cpuid_scan_cpu(c);
cpu_detect(c);
@@ -1991,14 +1991,23 @@ void check_null_seg_clears_base(struct cpuinfo_x86 *c)
set_cpu_bug(c, X86_BUG_NULL_SEG);
}

-static void generic_identify(struct cpuinfo_x86 *c)
+/*
+ * This does the hard work of actually picking apart the CPU stuff...
+ */
+static void identify_cpu(struct cpuinfo_x86 *c)
{
+ int i;
+
+ c->loops_per_jiffy = loops_per_jiffy;
+
+ init_cpu_info(c);
+
if (!cpuid_feature())
identify_cpu_without_cpuid(c);

- /* cyrix could have cpuid enabled via c_identify()*/
+ /* Cyrix could have CPUID enabled via c_identify(). */
if (!cpuid_feature())
- return;
+ goto no_cpuid;

cpuid_scan_cpu(c);
cpu_detect(c);
@@ -2026,21 +2035,8 @@ static void generic_identify(struct cpuinfo_x86 *c)
#ifdef CONFIG_X86_32
set_cpu_bug(c, X86_BUG_ESPFIX);
#endif
-}
-
-/*
- * This does the hard work of actually picking apart the CPU stuff...
- */
-static void identify_cpu(struct cpuinfo_x86 *c)
-{
- int i;
-
- c->loops_per_jiffy = loops_per_jiffy;
-
- init_cpu_info(c);
-
- generic_identify(c);

+no_cpuid:
cpu_parse_topology(c);

if (this_cpu->c_identify)