staging: lustre: constify attribute_group structures.
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Fri, 21 Jul 2017 05:59:51 +0000 (11:29 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 30 Jul 2017 15:01:54 +0000 (08:01 -0700)
attribute_groups are not supposed to change at runtime. All functions
working with attribute_groups provided by <linux/sysfs.h> work
with const attribute_group. So mark the non-const structs as const.

File size before:
   text    data     bss     dec     hex filename
   9489     992      40   10521    2919 lustre/lustre/osc/lproc_osc.o
   1289     288       0    1577     629 lustre/lustre/lmv/lproc_lmv.o
   3794     928      40    4762    129a lustre/lustre/lov/lproc_lov.o
   3802     576      40    4418    1142 lustre/lustre/mdc/lproc_mdc.o

File size After adding 'const':
   text    data     bss     dec     hex filename
   9553     928      40   10521    2919 lustre/lustre/osc/lproc_osc.o
   1353     224       0    1577     629 lustre/lustre/lmv/lproc_lmv.o
   3858     864      40    4762    129a lustre/lustre/lov/lproc_lov.o
   3866     512      40    4418    1142 lustre/lustre/mdc/lproc_mdc.o

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/include/lprocfs_status.h
drivers/staging/lustre/lustre/lmv/lproc_lmv.c
drivers/staging/lustre/lustre/lov/lproc_lov.c
drivers/staging/lustre/lustre/mdc/lproc_mdc.c
drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
drivers/staging/lustre/lustre/osc/lproc_osc.c

index 915283c04094290c810d0f3f4098b96af000d5bd..9054d3745785685d84758d8dc3f03fc5dad439dd 100644 (file)
@@ -59,7 +59,7 @@ struct lprocfs_vars {
 
 struct lprocfs_static_vars {
        struct lprocfs_vars *obd_vars;
-       struct attribute_group *sysfs_vars;
+       const struct attribute_group *sysfs_vars;
 };
 
 /* if we find more consumers this could be generalized */
@@ -468,7 +468,7 @@ struct dentry *ldebugfs_register(const char *name,
 void ldebugfs_remove(struct dentry **entryp);
 
 int lprocfs_obd_setup(struct obd_device *obd, struct lprocfs_vars *list,
-                     struct attribute_group *attrs);
+                     const struct attribute_group *attrs);
 int lprocfs_obd_cleanup(struct obd_device *obd);
 
 int ldebugfs_seq_create(struct dentry *parent,
index bf25f887062d779cbcbe6f708af0a366f3e38b89..4c13e3926898402f24c4f1c11ed81ee8e3c9957e 100644 (file)
@@ -161,7 +161,7 @@ static struct attribute *lmv_attrs[] = {
        NULL,
 };
 
-static struct attribute_group lmv_attr_group = {
+static const struct attribute_group lmv_attr_group = {
        .attrs = lmv_attrs,
 };
 
index eb6d30d34e3a7f71adc6ac15816dd08728d84fe0..ce4682120ffbe4fc480a06d5995ad46fc244f8fa 100644 (file)
@@ -279,7 +279,7 @@ static struct attribute *lov_attrs[] = {
        NULL,
 };
 
-static struct attribute_group lov_attr_group = {
+static const struct attribute_group lov_attr_group = {
        .attrs = lov_attrs,
 };
 
index 9021c465c04498d07e4252975fd2baf9e088f0bc..9fea535d6fc6280b5ae38085cb61878816814d3f 100644 (file)
@@ -219,7 +219,7 @@ static struct attribute *mdc_attrs[] = {
        NULL,
 };
 
-static struct attribute_group mdc_attr_group = {
+static const struct attribute_group mdc_attr_group = {
        .attrs = mdc_attrs,
 };
 
index bc19f19d38d9a2e08bf7687fe156998e44e7e35c..ba41983e85a93472372d236c8681872c6c6e0126 100644 (file)
@@ -1031,7 +1031,7 @@ static struct kobj_type obd_ktype = {
 };
 
 int lprocfs_obd_setup(struct obd_device *obd, struct lprocfs_vars *list,
-                     struct attribute_group *attrs)
+                     const struct attribute_group *attrs)
 {
        int rc = 0;
 
index 86f252d6adbdc2affe9d6ed213d906d89491e49c..6e0fd155e6b8aa6d37588ab9bafebb87440f80f1 100644 (file)
@@ -831,7 +831,7 @@ static struct attribute *osc_attrs[] = {
        NULL,
 };
 
-static struct attribute_group osc_attr_group = {
+static const struct attribute_group osc_attr_group = {
        .attrs = osc_attrs,
 };