diff options
author | Hans de Goede <hdegoede@redhat.com> | 2021-11-25 11:36:16 +0100 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2021-12-01 20:19:30 +0100 |
commit | b66f86849414807745b5c2129e2de5f27a788c9f (patch) | |
tree | a170d69bffcb423b1dc624b054a9849ea28ecf61 | |
parent | befd9b5b0c621af33a363596c65a8fc0176e2795 (diff) |
ACPI: EC: Mark the ec_sys write_support param as module_param_hw()
Using write_support=1 with the ec_sys module changes the mode of the
"io" debugfs file to 0600. This will cause any attempts to access it under
a kernel in lockdown mode to return -EPERM, which makes the entire ec_sys
module unusable.
Use the special module_param_hw() macro for module parameters which
may not be used while in lockdown mode, to avoid this.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/acpi/ec_sys.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/ec_sys.c b/drivers/acpi/ec_sys.c index fd39c14493ab..c074a0fae059 100644 --- a/drivers/acpi/ec_sys.c +++ b/drivers/acpi/ec_sys.c @@ -19,7 +19,7 @@ MODULE_DESCRIPTION("ACPI EC sysfs access driver"); MODULE_LICENSE("GPL"); static bool write_support; -module_param(write_support, bool, 0644); +module_param_hw(write_support, bool, other, 0644); MODULE_PARM_DESC(write_support, "Dangerous, reboot and removal of battery may " "be needed."); |