Re: [PATCH] Constify struct acpi_dock_ops for v2.6.33-rc6

From: Jeff Garzik
Date: Thu Feb 04 2010 - 01:17:39 EST


On 01/31/2010 02:16 PM, Emese Revfy wrote:
From: Emese Revfy<re.emese@xxxxxxxxx>

Constify struct acpi_dock_ops.

This is part of the ops structure constification
effort started by Arjan van de Ven et al.

Benefits of this constification:

* prevents modification of data that is shared
(referenced) by many other structure instances
at runtime

* detects/prevents accidental (but not intentional)
modification attempts on archs that enforce
read-only kernel data at runtime

* potentially better optimized code as the compiler
can assume that the const data cannot be changed

* the compiler/linker move const data into .rodata
and therefore exclude them from false sharing

Signed-off-by: Emese Revfy<re.emese@xxxxxxxxx>
---

drivers/acpi/dock.c | 4 ++--
drivers/ata/libata-acpi.c | 4 ++--
drivers/pci/hotplug/acpiphp_glue.c | 2 +-
include/acpi/acpi_drivers.h | 4 ++--
4 files changed, 7 insertions(+), 7 deletions(-)

Acked-by: Jeff Garzik <jgarzik@xxxxxxxxxx>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/