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

From: Zhuo, Qiuxu
Date: Wed Feb 05 2025 - 06:56:06 EST


> 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>