linux-kernel - Re: [PATCH v4 5/5] platform/x86: acer-wmi: Ignore AC events

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2f695131-b875-4856-9782-c3d4231adc8b@gmx.de>
Date: Tue, 10 Dec 2024 05:06:26 +0100
From: Armin Wolf <W_Armin@....de>
To: Kurt Borja <kuurtb@...il.com>
Cc: jlee@...e.com, farhan.anwar8@...il.com, rayanmargham4@...il.com,
 hdegoede@...hat.com, ilpo.jarvinen@...ux.intel.com,
 platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 5/5] platform/x86: acer-wmi: Ignore AC events

Am 10.12.24 um 03:16 schrieb Kurt Borja:

> On Tue, Dec 10, 2024 at 01:16:57AM +0100, Armin Wolf wrote:
>> On the Acer Swift SFG14-41, the events 8 - 1 and 8 - 0 are printed on
>> AC connect/disconnect. Ignore those events to avoid spamming the
>> kernel log with error messages.
> I noticed acer_thermal_profile_change() behavior changed whether the
> laptop was on AC or not. Maybe users expect some kind of behavior around
> thermal profiles with this event? Like switching to low-power when not
> on AC or something like that.
>
> ~ Kurt

Good question, i will ask the person with the Acer notebook to test if the platform
profile changes automatically when switching to AC.

The other however patches can be taken regardless of this patch.

Thanks,
Armin Wolf

>> Reported-by: Farhan Anwar <farhan.anwar8@...il.com>
>> Closes: https://lore.kernel.org/platform-driver-x86/2ffb529d-e7c8-4026-a3b8-120c8e7afec8@gmail.com
>> Tested-by: Rayan Margham <rayanmargham4@...il.com>
>> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
>> Signed-off-by: Armin Wolf <W_Armin@....de>
>> ---
>>   drivers/platform/x86/acer-wmi.c | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
>> index aad8eb0ddae5..ef26ec8d90ea 100644
>> --- a/drivers/platform/x86/acer-wmi.c
>> +++ b/drivers/platform/x86/acer-wmi.c
>> @@ -99,6 +99,7 @@ enum acer_wmi_event_ids {
>>   	WMID_HOTKEY_EVENT = 0x1,
>>   	WMID_ACCEL_OR_KBD_DOCK_EVENT = 0x5,
>>   	WMID_GAMING_TURBO_KEY_EVENT = 0x7,
>> +	WMID_AC_EVENT = 0x8,
>>   };
>>
>>   enum acer_wmi_predator_v4_sys_info_command {
>> @@ -2304,6 +2305,9 @@ static void acer_wmi_notify(union acpi_object *obj, void *context)
>>   		if (return_value.key_num == 0x5 && has_cap(ACER_CAP_PLATFORM_PROFILE))
>>   			acer_thermal_profile_change();
>>   		break;
>> +	case WMID_AC_EVENT:
>> +		/* We ignore AC events here */
>> +		break;
>>   	default:
>>   		pr_warn("Unknown function number - %d - %d\n",
>>   			return_value.function, return_value.key_num);
>> --
>> 2.39.5
>>