RE: [PATCH] EDAC/igen6: Constify struct res_config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
> [...]
> Subject: [PATCH] EDAC/igen6: Constify struct res_config
> 
> 'struct res_config' are not modified in this driver.
> 
> Constifying these structures moves some data to a read-only section, so
> increase overall security, especially when the structure holds some function
> pointers.
> 
> On a x86_64, with allmodconfig, as an example:
> Before:
> ======
>    text	   data	    bss	    dec	    hex	filename
>   36777	   2479	   4304	  43560	   aa28	drivers/edac/igen6_edac.o
> 
> After:
> =====
>    text	   data	    bss	    dec	    hex	filename
>   37297	   1959	   4304	  43560	   aa28	drivers/edac/igen6_edac.o
> 
> Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>

LGTM, thanks!

    Reviewed-by: Qiuxu Zhuo <qiuxu.zhuo@xxxxxxxxx>





[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux