On Wed, 2025-06-11 at 18:30 -0300, Kurt Borja wrote:
This reverts commit 5ff79cabb23a2f14d2ed29e9596aec908905a0e6.
Although the Alienware m16 R1 AMD model supports G-Mode, it actually has a lower power ceiling than plain "performance" profile, which results in lower performance.
Reported-by: Cihan Ozakca cozakca@outlook.com Cc: stable@vger.kernel.org # 6.15.x Signed-off-by: Kurt Borja kuurtb@gmail.com
Hi all,
Contrary to (my) intuition, imitating Windows behavior actually results in LOWER performance.
I was having second thoughts about this revert because users will notice that "performance" not longer turns on the G-Mode key found in this laptop. Some users may think this is actually a regression, but IMO lower performance is worse.
drivers/platform/x86/dell/alienware-wmi-wmax.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/dell/alienware-wmi-wmax.c b/drivers/platform/x86/dell/alienware-wmi-wmax.c index c42f9228b0b255fe962b735ac96486824e83945f..20ec122a9fe0571a1ecd2ccf630 615564ab30481 100644 --- a/drivers/platform/x86/dell/alienware-wmi-wmax.c +++ b/drivers/platform/x86/dell/alienware-wmi-wmax.c @@ -119,7 +119,7 @@ static const struct dmi_system_id awcc_dmi_table[] __initconst = { DMI_MATCH(DMI_SYS_VENDOR, "Alienware"), DMI_MATCH(DMI_PRODUCT_NAME, "Alienware m16 R1 AMD"), },
.driver_data = &g_series_quirks,
.driver_data = &generic_quirks,}, { .ident = "Alienware m16 R2",
I think, at least a code comment explaining the situation is warranted (or maybe even a print on keypress or both), because otherwise sometime in the future other people may try to add the functional back, and then yet other people may again revert it, regressing things back and forth.