diff options
| author |   Aya Mahfouz <mahfouz.saif.elyazal@gmail.com> | 2015-02-27 14:55:54 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-03-01 16:28:07 -0800 |
| commit | e6eae365ad2273571c19aa2a8b5460874cd507d6 (patch) | |
| tree | 548ab5513c454182ddf62b26036fd9890316ad4b | |
| parent | staging: lustre: obdecho: remove unused variable (diff) | |
staging: lustre: osc: remove unused variable
This patch removes a variable that was simply used to
store the return value of a function call before
returning it.
The issue was detected and resolved using the following
coccinelle script:
@@
identifier len,f;
@@
-int len;
... when != len
when strict
-len =
+return
f(...);
-return len;
Signed-off-by: Aya Mahfouz <mahfouz.saif.elyazal@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/lustre/lustre/osc/lproc_osc.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/lustre/lustre/osc/lproc_osc.c b/drivers/staging/lustre/lustre/osc/lproc_osc.c index 1795d3a7a029..3ef2b554b65c 100644 --- a/drivers/staging/lustre/lustre/osc/lproc_osc.c +++ b/drivers/staging/lustre/lustre/osc/lproc_osc.c @@ -164,16 +164,13 @@ static int osc_cached_mb_seq_show(struct seq_file *m, void *v) struct obd_device *dev = m->private; struct client_obd *cli = &dev->u.cli; int shift = 20 - PAGE_CACHE_SHIFT; - int rc; - rc = seq_printf(m, + return seq_printf(m, "used_mb: %d\n" "busy_cnt: %d\n", (atomic_read(&cli->cl_lru_in_list) + atomic_read(&cli->cl_lru_busy)) >> shift, atomic_read(&cli->cl_lru_busy)); - - return rc; } /* shrink the number of caching pages to a specific number */ |