Whamcloud - gitweb
LU-13783 procfs: fix improper prop_ops fields 80/43880/3
authorJames Simmons <jsimmons@infradead.org>
Mon, 31 May 2021 17:05:50 +0000 (13:05 -0400)
committerOleg Drokin <green@whamcloud.com>
Tue, 8 Jun 2021 21:59:44 +0000 (21:59 +0000)
The lod pool and nodemap proc_ops missed renaming the fields to
start with .proc_*. On newer distros like Ubuntu 20.04 HWE you
get the following compile error:

lustre-release/lustre/ptlrpc/nodemap_lproc.c:686:3: error: ‘const struct proc_ops’ has no member named ‘open’
  686 |  .open   = nodemap_ranges_open,

Test-Parameters: trivial
Fixes: 13cd0f9f667 ("LU-13344 libcfs: Abstract proc_fs with proc_ops")
Signed-off-by: James Simmons <jsimmons@infradead.org>
Change-Id: I5fff7519a801f585690d468255f7ca6c73adcc90
Reviewed-on: https://review.whamcloud.com/43880
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/lod/lod_pool.c
lustre/ptlrpc/nodemap_lproc.c

index 0a3fce1..17b625d 100644 (file)
@@ -318,10 +318,10 @@ static int pool_proc_open(struct inode *inode, struct file *file)
 }
 
 const static struct proc_ops pool_proc_operations = {
-       .open           = pool_proc_open,
-       .read           = seq_read,
-       .llseek         = seq_lseek,
-       .release        = seq_release,
+       .proc_open      = pool_proc_open,
+       .proc_read      = seq_read,
+       .proc_lseek     = seq_lseek,
+       .proc_release   = seq_release,
 };
 
 /**
index 408afee..e6f109c 100644 (file)
@@ -683,24 +683,24 @@ LPROC_SEQ_FOPS_RO(nodemap_audit_mode);
 LPROC_SEQ_FOPS_RO(nodemap_forbid_encryption);
 
 static const struct proc_ops nodemap_ranges_fops = {
-       .open                   = nodemap_ranges_open,
-       .read                   = seq_read,
-       .llseek                 = seq_lseek,
-       .release                = single_release
+       .proc_open              = nodemap_ranges_open,
+       .proc_read              = seq_read,
+       .proc_lseek             = seq_lseek,
+       .proc_release           = single_release
 };
 
 static const struct proc_ops nodemap_idmap_fops = {
-       .open                   = nodemap_idmap_open,
-       .read                   = seq_read,
-       .llseek                 = seq_lseek,
-       .release                = single_release
+       .proc_open              = nodemap_idmap_open,
+       .proc_read              = seq_read,
+       .proc_lseek             = seq_lseek,
+       .proc_release           = single_release
 };
 
 static const struct proc_ops nodemap_exports_fops = {
-       .open                   = nodemap_exports_open,
-       .read                   = seq_read,
-       .llseek                 = seq_lseek,
-       .release                = single_release
+       .proc_open              = nodemap_exports_open,
+       .proc_read              = seq_read,
+       .proc_lseek             = seq_lseek,
+       .proc_release           = single_release
 };
 
 static struct lprocfs_vars lprocfs_nodemap_vars[] = {