diff options
| author | Mario Limonciello <mario.limonciello@amd.com> | 2024-02-16 20:23:11 -0600 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2024-02-20 14:35:36 +0100 |
| commit | 427c70dec738318b7f71e1b9d829ff0e9771d493 (patch) | |
| tree | d7eea25f0e45f4c977a9e35d1542fb03f7d2c8ce /net/unix/garbage.c | |
| parent | 84c16d01ff219bc0a5dca5219db6b8b86a6854fb (diff) | |
platform/x86: thinkpad_acpi: Only update profile if successfully converted
Randomly a Lenovo Z13 will trigger a kernel warning traceback from this
condition:
```
if (WARN_ON((profile < 0) || (profile >= ARRAY_SIZE(profile_names))))
```
This happens because thinkpad-acpi always assumes that
convert_dytc_to_profile() successfully updated the profile. On the
contrary a condition can occur that when dytc_profile_refresh() is called
the profile doesn't get updated as there is a -EOPNOTSUPP branch.
Catch this situation and avoid updating the profile. Also log this into
dynamic debugging in case any other modes should be added in the future.
Fixes: c3bfcd4c6762 ("platform/x86: thinkpad_acpi: Add platform profile support")
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/r/20240217022311.113879-1-mario.limonciello@amd.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'net/unix/garbage.c')
0 files changed, 0 insertions, 0 deletions