orangefs: make struct orangefs_file_vm_ops static

Colin Ian King · hubcapsc · commit ec62e95ae3f1 · 2018-06-01T14:49:00.000-04:00
The struct orangefs_file_vm_ops is local to the source and does not need
to be in global scope, so make it static.

Cleans up sparse warning:
fs/orangefs/file.c:547:35: warning: symbol 'orangefs_file_vm_ops' was not
declared. Should it be static?

Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Mike Marshall &lt;hubcap@omnibond.com&gt;
diff --git a/fs/orangefs/file.c b/fs/orangefs/file.c
@@ -544,7 +544,7 @@ static int orangefs_fault(struct vm_fault *vmf)
 	return filemap_fault(vmf);
 }
 
-const struct vm_operations_struct orangefs_file_vm_ops = {
+static const struct vm_operations_struct orangefs_file_vm_ops = {
 	.fault = orangefs_fault,
 	.map_pages = filemap_map_pages,
 	.page_mkwrite = filemap_page_mkwrite,

Original file line number	Diff line number	Diff line change
`@@ -544,7 +544,7 @@ static int orangefs_fault(struct vm_fault *vmf)`
`544`	`544`	`return filemap_fault(vmf);`
`545`	`545`	`}`
`546`	`546`
`547`		`-const struct vm_operations_struct orangefs_file_vm_ops = {`
	`547`	`+static const struct vm_operations_struct orangefs_file_vm_ops = {`
`548`	`548`	`.fault = orangefs_fault,`
`549`	`549`	`.map_pages = filemap_map_pages,`
`550`	`550`	`.page_mkwrite = filemap_page_mkwrite,`