diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2024-09-07 22:30:42 +0200 |
|---|---|---|
| committer | Hans Verkuil <hverkuil@xs4all.nl> | 2025-02-21 10:33:14 +0100 |
| commit | d22d91fb1d7f04130d6b5371315fdaeddd6a2fc6 (patch) | |
| tree | 1153a8710bc443266f03c9af0666dba7e3bb8b00 /io_uring/sqpoll.c | |
| parent | fc91d334e045dab1b5f01bd88b4906ad1dfbd223 (diff) | |
media: dvb-usb-v2: Constify struct i2c_algorithm
'struct i2c_algorithm' are not modified in these drivers.
Constifying this structure moves some data to a read-only section, so
increase overall security, especially when the structure holds some
function pointers.
More over, dvb_usb_device_properties->i2c_algo seems to only be copied in
i2c_adapter->algo, which is already a "const struct i2c_algorithm".
This is done in dvb_usbv2_i2c_init()
On a x86_64, with allmodconfig, as an example:
Before:
======
text data bss dec hex filename
35366 5832 36 41234 a112 drivers/media/usb/dvb-usb-v2/af9015.o
After:
=====
text data bss dec hex filename
35430 5768 36 41234 a112 drivers/media/usb/dvb-usb-v2/af9015.o
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Diffstat (limited to 'io_uring/sqpoll.c')
0 files changed, 0 insertions, 0 deletions