padata: Constify padata_sysfs_entry structs

These structs are never modified.

To prevent malicious or accidental modifications due to bugs,
mark them as const.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/kernel/padata.c b/kernel/padata.c
index aa66d91e20f9c17e978cfc330d5e4b873fb842d0..db7c75787a2b3974875014a0ae1c9158176e6591 100644 (file)
--- a/kernel/padata.c
+++ b/kernel/padata.c
@@ -819,7 +819,7 @@ static void __padata_free(struct padata_instance *pinst)
 #define kobj2pinst(_kobj)                                      \
        container_of(_kobj, struct padata_instance, kobj)
 #define attr2pentry(_attr)                                     \
-       container_of(_attr, struct padata_sysfs_entry, attr)
+       container_of_const(_attr, struct padata_sysfs_entry, attr)
 
 static void padata_sysfs_release(struct kobject *kobj)
 {
@@ -829,13 +829,13 @@ static void padata_sysfs_release(struct kobject *kobj)
 
 struct padata_sysfs_entry {
        struct attribute attr;
-       ssize_t (*show)(struct padata_instance *, struct attribute *, char *);
-       ssize_t (*store)(struct padata_instance *, struct attribute *,
+       ssize_t (*show)(struct padata_instance *, const struct attribute *, char *);
+       ssize_t (*store)(struct padata_instance *, const struct attribute *,
                         const char *, size_t);
 };
 
 static ssize_t show_cpumask(struct padata_instance *pinst,
-                           struct attribute *attr,  char *buf)
+                           const struct attribute *attr,  char *buf)
 {
        struct cpumask *cpumask;
        ssize_t len;
@@ -853,7 +853,7 @@ static ssize_t show_cpumask(struct padata_instance *pinst,
 }
 
 static ssize_t store_cpumask(struct padata_instance *pinst,
-                            struct attribute *attr,
+                            const struct attribute *attr,
                             const char *buf, size_t count)
 {
        cpumask_var_t new_cpumask;
@@ -880,10 +880,10 @@ out:
 }
 
 #define PADATA_ATTR_RW(_name, _show_name, _store_name)         \
-       static struct padata_sysfs_entry _name##_attr =         \
+       static const struct padata_sysfs_entry _name##_attr =   \
                __ATTR(_name, 0644, _show_name, _store_name)
-#define PADATA_ATTR_RO(_name, _show_name)              \
-       static struct padata_sysfs_entry _name##_attr = \
+#define PADATA_ATTR_RO(_name, _show_name)                      \
+       static const struct padata_sysfs_entry _name##_attr =   \
                __ATTR(_name, 0400, _show_name, NULL)
 
 PADATA_ATTR_RW(serial_cpumask, show_cpumask, store_cpumask);
@@ -894,7 +894,7 @@ PADATA_ATTR_RW(parallel_cpumask, show_cpumask, store_cpumask);
  * serial_cpumask   [RW] - cpumask for serial workers
  * parallel_cpumask [RW] - cpumask for parallel workers
  */
-static struct attribute *padata_default_attrs[] = {
+static const struct attribute *const padata_default_attrs[] = {
        &serial_cpumask_attr.attr,
        &parallel_cpumask_attr.attr,
        NULL,
@@ -904,8 +904,8 @@ ATTRIBUTE_GROUPS(padata_default);
 static ssize_t padata_sysfs_show(struct kobject *kobj,
                                 struct attribute *attr, char *buf)
 {
+       const struct padata_sysfs_entry *pentry;
        struct padata_instance *pinst;
-       struct padata_sysfs_entry *pentry;
        ssize_t ret = -EIO;
 
        pinst = kobj2pinst(kobj);
@@ -919,8 +919,8 @@ static ssize_t padata_sysfs_show(struct kobject *kobj,
 static ssize_t padata_sysfs_store(struct kobject *kobj, struct attribute *attr,
                                  const char *buf, size_t count)
 {
+       const struct padata_sysfs_entry *pentry;
        struct padata_instance *pinst;
-       struct padata_sysfs_entry *pentry;
        ssize_t ret = -EIO;
 
        pinst = kobj2pinst(kobj);