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