Re: [PATCH 7/7] x86: microcode: constify attribute_group structures.

From: Arvind Yadav
Date: Thu Jul 20 2017 - 05:01:53 EST


Hi,


On Thursday 20 July 2017 01:58 PM, Borislav Petkov wrote:
On Thu, Jul 20, 2017 at 01:48:17PM +0530, Arvind Yadav wrote:
It's just show, how much bytes move from data segment to text segment.
~arvind
3614 545 1560 5719 1657 x86/kernel/cpu/microcode/core.o

3614 + 545 = 4159

3742 417 1560 5719 1657 x86/kernel/cpu/microcode/core.o

3742 + 417 = 4159

3742 - 3614 = 128

So 128 bytes moved to text.
Yes,

I fail to see how this is relevant. Your commit message contains not
really relevant info for such a simple change. Rather, it should just be
a one-liner:
I have mentioned all this in commit message. I have added below
message in patch. Please check my patch.

"attribute_group are not supposed to change at runtime. All functions
working with attribute_group provided by <linux/sysfs.h> work with
const attribute_group. So mark the non-const structs as const."

"Constify attribute_group structs because they don't change at runtime."
I can add this as subject. But we should avoid lengthy subject.
Even checkpatch.pl will through warning.

That's it.

~arvind