drm/nouveau/ttm: constify static vm_operations_struct

rikardfalkeborn · ChristianKoenigAMD · commit 19bafac4f56f · 2021-02-25T14:40:25.000+01:00
The only usage of nouveau_ttm_vm_ops is to assign its address to the vm_ops field in the vm_area_struct struct. Make it const to allow the compiler to put it in read-only memory Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com> Reviewed-by: Christian König <christian.koenig@amd.com> Link: https://patchwork.freedesktop.org/patch/msgid/20210209234817.55112-4-rikard.falkeborn@gmail.com Signed-off-by: Christian König <christian.koenig@amd.com>
diff --git a/drivers/gpu/drm/nouveau/nouveau_ttm.c b/drivers/gpu/drm/nouveau/nouveau_ttm.c
@@ -154,7 +154,7 @@ static vm_fault_t nouveau_ttm_fault(struct vm_fault *vmf)
 	return ret;
 }
 
-static struct vm_operations_struct nouveau_ttm_vm_ops = {
+static const struct vm_operations_struct nouveau_ttm_vm_ops = {
 	.fault = nouveau_ttm_fault,
 	.open = ttm_bo_vm_open,
 	.close = ttm_bo_vm_close,

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ static vm_fault_t nouveau_ttm_fault(struct vm_fault *vmf)`
`154`	`154`	`return ret;`
`155`	`155`	`}`
`156`	`156`
`157`		`-static struct vm_operations_struct nouveau_ttm_vm_ops = {`
	`157`	`+static const struct vm_operations_struct nouveau_ttm_vm_ops = {`
`158`	`158`	`.fault = nouveau_ttm_fault,`
`159`	`159`	`.open = ttm_bo_vm_open,`
`160`	`160`	`.close = ttm_bo_vm_close,`