Skip to content

Commit 6199d8d

Browse files
H. Peter Anvin1Naim
authored andcommitted
x86/fred: enable FRED by default
When FRED was added to the mainline kernel, it was set up as an explicit opt-in due to the risk of regressions before hardware was available publicly. Now, Panther Lake (Core Ultra 300 series) has been released, and benchmarking by Phoronix has shown that it provides a significant performance benefit on most workloads: https://www.phoronix.com/review/intel-fred-panther-lake Accordingly, enable FRED by default if the CPU supports it. FRED can of course still be disabled via the fred=off command line option. Signed-off-by: H. Peter Anvin (Intel) <hpa@zytor.com> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
1 parent 3a2a401 commit 6199d8d

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

Documentation/admin-guide/kernel-parameters.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1750,8 +1750,8 @@ Kernel parameters
17501750
fred= [X86-64]
17511751
Enable/disable Flexible Return and Event Delivery.
17521752
Format: { on | off }
1753-
on: enable FRED when it's present.
1754-
off: disable FRED, the default setting.
1753+
on: enable FRED when it's present, the default setting.
1754+
off: disable FRED.
17551755

17561756
ftrace=[tracer]
17571757
[FTRACE] will set and start the specified tracer

arch/x86/kernel/cpu/common.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1759,7 +1759,7 @@ static void __init cpu_parse_early_param(void)
17591759

17601760
/* Minimize the gap between FRED is available and available but disabled. */
17611761
arglen = cmdline_find_option(boot_command_line, "fred", arg, sizeof(arg));
1762-
if (arglen != 2 || strncmp(arg, "on", 2))
1762+
if (arglen > 0 && (arglen != 2 || strncmp(arg, "on", 2)))
17631763
setup_clear_cpu_cap(X86_FEATURE_FRED);
17641764

17651765
arglen = cmdline_find_option(boot_command_line, "clearcpuid", arg, sizeof(arg));

0 commit comments

Comments
 (0)