Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1679874

Re: [PATCH] zram: constify attribute_group structures.

From Arvind Yadav <arvind.yadav.cs@gmail.com>
Newsgroups linux.kernel
Subject Re: [PATCH] zram: constify attribute_group structures.
Date 2017-07-03 12:40 +0200
Message-ID <tZ6VS-5S-25@gated-at.bofh.it> (permalink)
References <tZ2Sf-5BT-11@gated-at.bofh.it> <tZ5wL-7r5-31@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

Hi,


On Monday 03 July 2017 02:30 PM, Sergey Senozhatsky wrote:
> On (07/03/17 11:43), Arvind Yadav wrote:
>> 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
>>     8293	    841	      4	   9138	   23b2	drivers/block/zram/zram_drv.o
>>
>> File size After adding 'const':
>>     text	   data	    bss	    dec	    hex	filename
>>     8357	    777	      4	   9138	   23b2	drivers/block/zram/zram_drv.o
> sorry, what exactly is the improvement here?
> we grew .text bigger, why should we be happy about it?
Yes, .text size is increased but .data size got decreased.
Overall size will be same. the text segment is often read-only,
to prevent a program from accidentally modifying its instructions.
Here, attribute_group should be const.
> 	-ss
Thanks,
~arvind

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] zram: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-07-03 08:20 +0200
  Re: [PATCH] zram: constify attribute_group structures. Minchan Kim <minchan@kernel.org> - 2017-07-03 08:30 +0200
    Re: [PATCH] zram: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-07-03 09:20 +0200
  Re: [PATCH] zram: constify attribute_group structures. Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-07-03 11:10 +0200
    Re: [PATCH] zram: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-07-03 12:40 +0200
      Re: [PATCH] zram: constify attribute_group structures. Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-07-03 14:10 +0200

csiph-web