Messages in this thread Patch in this message | | | From | Arvind Yadav <> | Subject | [PATCH 3/7] perf: x86: intel: uncore_snb: constify attribute_group structures. | Date | Thu, 20 Jul 2017 12:51:41 +0530 |
| |
attribute_group are not supposed to change at runtime. All functions working with attribute_group provided by 'uncore.h' work with const attribute_group. So mark the non-const structs as const.
File size before: text data bss dec hex filename 4158 4792 0 8950 22f6 arch/x86/events/intel/uncore_snb.o
File size After adding 'const': text data bss dec hex filename 4350 4600 0 8950 22f6 arch/x86/events/intel/uncore_snb.o
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com> --- arch/x86/events/intel/uncore_snb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/x86/events/intel/uncore_snb.c b/arch/x86/events/intel/uncore_snb.c index a3dcc12..db1127c 100644 --- a/arch/x86/events/intel/uncore_snb.c +++ b/arch/x86/events/intel/uncore_snb.c @@ -130,7 +130,7 @@ static void snb_uncore_msr_exit_box(struct intel_uncore_box *box) NULL, }; -static struct attribute_group snb_uncore_format_group = { +static const struct attribute_group snb_uncore_format_group = { .name = "format", .attrs = snb_uncore_formats_attr, }; @@ -289,7 +289,7 @@ enum { NULL, }; -static struct attribute_group snb_uncore_imc_format_group = { +static const struct attribute_group snb_uncore_imc_format_group = { .name = "format", .attrs = snb_uncore_imc_formats_attr, }; @@ -769,7 +769,7 @@ static void nhm_uncore_msr_enable_event(struct intel_uncore_box *box, struct per NULL, }; -static struct attribute_group nhm_uncore_format_group = { +static const struct attribute_group nhm_uncore_format_group = { .name = "format", .attrs = nhm_uncore_formats_attr, }; -- 1.9.1
| |