Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1345291

[PATCH 1/2] staging: lustre: Use const static file_operations struct

From "M. Vefa Bicakci" <m.v.b@runbox.com>
Newsgroups linux.kernel
Subject [PATCH 1/2] staging: lustre: Use const static file_operations struct
Date 2016-02-28 15:50 +0100
Message-ID <r7aPw-3VM-1@gated-at.bofh.it> (permalink)
References <r7aPw-3VM-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


This commit corrects the following sparse warning regarding a
file_operations structure being non-static and constifies the
structure in question as well.

  warning: symbol 'lprocfs_stats_seq_fops' was not declared.
    Should it be static?

Signed-off-by: M. Vefa Bicakci <m.v.b@runbox.com>
---
 drivers/staging/lustre/lustre/obdclass/lprocfs_status.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
index 28bb4e53adaa..1c39e90979df 100644
--- a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
+++ b/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
@@ -1196,7 +1196,7 @@ static int lprocfs_stats_seq_open(struct inode *inode, struct file *file)
 	return 0;
 }
 
-struct file_operations lprocfs_stats_seq_fops = {
+static const struct file_operations lprocfs_stats_seq_fops = {
 	.owner   = THIS_MODULE,
 	.open    = lprocfs_stats_seq_open,
 	.read    = seq_read,
-- 
2.5.0

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

[PATCH 1/2] staging: lustre: Use const static file_operations struct "M. Vefa Bicakci" <m.v.b@runbox.com> - 2016-02-28 15:50 +0100

csiph-web