fs: Convert show_fdinfo functions to void

seq_printf functions shouldn't really check the return value.
Checking seq_has_overflowed() occasionally is used instead.

Update vfs documentation.

Link: http://lkml.kernel.org/p/e37e6e7b76acbdcc3bb4ab2a57c8f8ca1ae11b9a.1412031505.git.joe@perches.com

Cc: David S. Miller <davem@davemloft.net>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Joe Perches <joe@perches.com>
[ did a few clean ups ]
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

1 files changed, 14 insertions, 13 deletions

diff --git a/fs/timerfd.c b/fs/timerfd.c
index b46ffa94372a..b94fa6c3c6eb 100644
--- a/fs/timerfd.c
+++ b/fs/timerfd.c
@@ -288,7 +288,7 @@ static ssize_t timerfd_read(struct file *file, char __user *buf, size_t count,
 }
 
 #ifdef CONFIG_PROC_FS
-static int timerfd_show(struct seq_file *m, struct file *file)
+static void timerfd_show(struct seq_file *m, struct file *file)
 {
         struct timerfd_ctx *ctx = file->private_data;
         struct itimerspec t;
@@ -298,18 +298,19 @@ static int timerfd_show(struct seq_file *m, struct file *file)
         t.it_interval = ktime_to_timespec(ctx->tintv);
         spin_unlock_irq(&ctx->wqh.lock);
 
-        return seq_printf(m,
-                          "clockid: %d\n"
-                          "ticks: %llu\n"
-                          "settime flags: 0%o\n"
-                          "it_value: (%llu, %llu)\n"
-                          "it_interval: (%llu, %llu)\n",
-                          ctx->clockid, (unsigned long long)ctx->ticks,
-                          ctx->settime_flags,
-                          (unsigned long long)t.it_value.tv_sec,
-                          (unsigned long long)t.it_value.tv_nsec,
-                          (unsigned long long)t.it_interval.tv_sec,
-                          (unsigned long long)t.it_interval.tv_nsec);
+        seq_printf(m,
+                   "clockid: %d\n"
+                   "ticks: %llu\n"
+                   "settime flags: 0%o\n"
+                   "it_value: (%llu, %llu)\n"
+                   "it_interval: (%llu, %llu)\n",
+                   ctx->clockid,
+                   (unsigned long long)ctx->ticks,
+                   ctx->settime_flags,
+                   (unsigned long long)t.it_value.tv_sec,
+                   (unsigned long long)t.it_value.tv_nsec,
+                   (unsigned long long)t.it_interval.tv_sec,
+                   (unsigned long long)t.it_interval.tv_nsec);
 }
 #else
 #define timerfd_show NULL