On Sun, Jun 29, 2025 at 02:35:49PM +0200, Christophe JAILLET wrote: > 'struct devlink_region_ops' and 'struct mv88e6xxx_region' are not modified > in this driver. > > Constifying these structures moves some data to a read-only section, so > increases 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 > 18076 6496 64 24636 603c drivers/net/dsa/mv88e6xxx/devlink.o > > After: > ===== > text data bss dec hex filename > 18652 5920 64 24636 603c drivers/net/dsa/mv88e6xxx/devlink.o > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> Reviewed-by: Andrew Lunn <andrew@xxxxxxx> Andrew