mbox series

[0/3] ACPI: Constify 'struct bin_attribute'

Message ID 20241202-sysfs-const-bin_attr-acpi-v1-0-78f3b38d350d@weissschuh.net (mailing list archive)
Headers show
Series ACPI: Constify 'struct bin_attribute' | expand

Message

Thomas Weißschuh Dec. 2, 2024, 7:04 p.m. UTC 
The sysfs core now allows instances of 'struct bin_attribute' to be
moved into read-only memory. Make use of that to protect them against
accidental or malicious modifications.

The usage of read_new/write_new/bin_attrs_new is a transition mechanism
and will be reverted after the transition is complete.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
Thomas Weißschuh (3):
      ACPI: BGRT: Mark bin_attribute as __ro_after_init
      ACPI: BGRT: Constify 'struct bin_attribute'
      ACPI: sysfs: Constify 'struct bin_attribute'

 drivers/acpi/bgrt.c  | 6 +++---
 drivers/acpi/sysfs.c | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)
---
base-commit: e70140ba0d2b1a30467d4af6bcfe761327b9ec95
change-id: 20241201-sysfs-const-bin_attr-acpi-be28e6d66a2a

Best regards,

Comments

Rafael J. Wysocki Dec. 10, 2024, 6:36 p.m. UTC | #1 
On Mon, Dec 2, 2024 at 8:04 PM Thomas Weißschuh <linux@weissschuh.net> wrote:
>
> The sysfs core now allows instances of 'struct bin_attribute' to be
> moved into read-only memory. Make use of that to protect them against
> accidental or malicious modifications.
>
> The usage of read_new/write_new/bin_attrs_new is a transition mechanism
> and will be reverted after the transition is complete.
>
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
> Thomas Weißschuh (3):
>       ACPI: BGRT: Mark bin_attribute as __ro_after_init
>       ACPI: BGRT: Constify 'struct bin_attribute'
>       ACPI: sysfs: Constify 'struct bin_attribute'
>
>  drivers/acpi/bgrt.c  | 6 +++---
>  drivers/acpi/sysfs.c | 8 ++++----
>  2 files changed, 7 insertions(+), 7 deletions(-)
> ---

All applied as 6.14 material, thanks!