From: Christophe JAILLET Date: Sat, 16 May 2026 09:17:47 +0000 (+0200) Subject: RDMA/cma: Constify struct configfs_item_operations and configfs_group_operations X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=67464f388d52ec172be62c99fc43697437ffa384;p=thirdparty%2Fkernel%2Flinux.git RDMA/cma: Constify struct configfs_item_operations and configfs_group_operations 'struct configfs_item_operations' and 'configfs_group_operations' 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: Before: ====== text data bss dec hex filename 6677 2776 64 9517 252d drivers/infiniband/core/cma_configfs.o After: ===== text data bss dec hex filename 6901 2552 64 9517 252d drivers/infiniband/core/cma_configfs.o Signed-off-by: Christophe JAILLET Link: https://patch.msgid.link/6acd9c8a79b868b5e541a7e080a6b4b145e4fd4f.1778923041.git.christophe.jaillet@wanadoo.fr Signed-off-by: Leon Romanovsky --- diff --git a/drivers/infiniband/core/cma_configfs.c b/drivers/infiniband/core/cma_configfs.c index 819927ce4f0ec..891e52afb8f4b 100644 --- a/drivers/infiniband/core/cma_configfs.c +++ b/drivers/infiniband/core/cma_configfs.c @@ -255,7 +255,7 @@ static void release_cma_ports_group(struct config_item *item) cma_dev_group->ports = NULL; }; -static struct configfs_item_operations cma_ports_item_ops = { +static const struct configfs_item_operations cma_ports_item_ops = { .release = release_cma_ports_group }; @@ -264,7 +264,7 @@ static const struct config_item_type cma_ports_group_type = { .ct_owner = THIS_MODULE }; -static struct configfs_item_operations cma_device_item_ops = { +static const struct configfs_item_operations cma_device_item_ops = { .release = release_cma_dev }; @@ -327,7 +327,7 @@ static void drop_cma_dev(struct config_group *cgroup, struct config_item *item) config_item_put(item); } -static struct configfs_group_operations cma_subsys_group_ops = { +static const struct configfs_group_operations cma_subsys_group_ops = { .make_group = make_cma_dev, .drop_item = drop_cma_dev, };