- 15 Jul, 2015 3 commits
-
-
Abdul Hussain authored
This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by:
Abdul Hussain <habdul@visteon.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Abdul Hussain authored
This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by:
-
Luis de Bethencourt authored
fixes checkpatch.pl WARNING: quoted string split across lines +MODULE_AUTHOR("David A. Schleef <ds@schleef.org>, " + "Frank Mori Hess <fmhess@users.sourceforge.net>"); Modules with multiple authors have a MODULE_AUTHOR line for each one. Signed-off-by:Luis de Bethencourt <luis@debethencourt.com> Reviewed-by:
-
- 14 Jul, 2015 37 commits
-
-
Guillermo O. Freschi authored
Simple style fix. Signed-off-by:
Guillermo O. Freschi <kedrot@gmail.com> Signed-off-by:
-
Dmitry Kalinkin authored
Prevents false positive "missing empty line after a definition" checkpatch warning. Signed-off-by:
Dmitry Kalinkin <dmitry.kalinkin@gmail.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Markus Elfring authored
The following functions test whether their argument is NULL and then return immediately. * kfree * ll_file_data_put * ptlrpc_connection_put Thus the test around such calls is not needed. This issue was detected by using the Coccinelle software. See also a previous update suggestion: "remove unneeded null test before free" by Julia Lawall https://lkml.org/lkml/2015/5/1/498 https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg878600.htmlSigned-off-by:
Markus Elfring <elfring@users.sourceforge.net> Signed-off-by:
-
Lukasz Janyst authored
This is a patch to the libcfs/module.c file fixing whitespace warnings found by checkpatch.pl. Signed-off-by:
Lukasz Janyst <lj@buggybrain.net> Signed-off-by:
-
Oleg Drokin authored
Apparently kstrtouint could return not just -EINVAL, but also -ERANGE, so make sure we just check the return value for something negative. Noticed by Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Dmitry Eremin authored
Remove portal_enter_debugger because it's not used any more. Signed-off-by:
Dmitry Eremin <dmitry.eremin@intel.com> Signed-off-by:
-
Dmitry Eremin authored
They are just fancy module parameters wrappers, so just the same functionality now would be accessible via /sys/module/libcfs/parameters/libcfs_console_{min,max}_delay Also install compatibility symlinks Signed-off-by: -
Oleg Drokin authored
It's just a fancy libcfs_debug_mb module parameter wrapper, so just add debug buffer size check and resizing and the same functionality now would be accessible via /sys/module/libcfs/parameters/libcfs_debug_mb Also add a symlink for backwards compatibility. Signed-off-by:
-
Perry Hooker authored
Found by checkpatch.pl Signed-off-by:
Perry Hooker <perry.hooker@gmail.com> Signed-off-by:
-
Ronit Halder authored
Fixed a blank line after declarations coding style issue Signed-off-by:
Ronit Halder <ronit.crj@gmail.com> Signed-off-by:
-
Kris Baumann authored
Fixed several brace coding style issues. Signed-off-by:
Kris Baumann <krisbaumann@gmail.com> Signed-off-by:
-
Anders Fridlund authored
This is a patch to the cl_page.c file that fixes a "space required after that ','" error found by the checkpatch.pl tool. Signed-off-by:
Anders Fridlund <anders.fridlund@gmail.com> Signed-off-by:
-
Matt Mooney authored
fix sparse warning in lustre/ptlrpc/ptlrpc_module.c:54:12: warning: symbol 'ptlrpc_init' was not declared. Should it be static? The __init macro is moved before the function name to match standard usage. Signed-off-by:Matt Mooney <mfm@muteddisk.com> Signed-off-by:
-
Hari Prasath Gujulan Elango authored
This patch removes commented code. Signed-off-by:
Hari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by:
-
Dmitry Eremin authored
It is no longer used anywhere. Signed-off-by:
-
Dmitry Eremin authored
/proc/sys/lnet/lnet_memused Remove memory tracking for LNet. Remove redundant enums definition. Signed-off-by:
-
Oleg Drokin authored
module parameter libcfs_console_backoff accessible through /sys/module/libcfs/parameters/libcfs_console_backoff would do the same thing, just add a special "uintpos" parameter type to disallow 0 values too. Also add a symlink to the module parameter variable for backwards compatibility Signed-off-by:
-
Oleg Drokin authored
/proc/sys/lnet/console_ratelimit, debug_path and panic_on_lbug are module parameters with no special magic accessible via /sys/module/libcfs/parameters/libcfs_console_ratelimit, /sys/module/libcfs/parameters/libcfs_debug_file_path and /sys/module/libcfs/parameters/libcfs_panic_on_lbug respectively. As such just replace them with corresponding symlinks Signed-off-by:
-
Oleg Drokin authored
Parameters in lnet sysctl are of debug quantity, so let's move them to debugfs instead. Signed-off-by:
-
Oleg Drokin authored
Since all of the variables from /proc/sys/lustre were moved to /sys/fs/lustre, get rid of the remaining infrastructure. Signed-off-by:
-
Oleg Drokin authored
Adaptive Timeouts controls are being moved from /proc/sys/lustre to /sys/fs/lustre Signed-off-by:
-
Oleg Drokin authored
debug_peer_on_timeout, dump_on_timeout and dump_on_eviction controls from /proc/sys/lustre to /sys/fs/lustre Signed-off-by:
-
Oleg Drokin authored
max_dirty_mb is now a parameter in /sys/fs/lustre/ Signed-off-by:
-
Oleg Drokin authored
This is the first step of moving lustre sysctls from /proc/sys/lustre to /sys/fs/lustre Signed-off-by:
-
Oleg Drokin authored
ldlm_timeout is used server-side to determine AST timeouts, so it makes no sense on the client, esp. since it's not really used anywhere. Remove all traces of it except from the config where make it a noop. Signed-off-by:
-
Oleg Drokin authored
In the past when Lustre did its own allocation amounts tracking the results were shown in sysctl as current and overall max number of bytes and pages allocated. Now that we don't track these, remove the sysctls. Signed-off-by:
-
Oleg Drokin authored
It was used to control allocation failure rate, but there is in-kernel way of doing that that's more versatile too. This is going to remove just the sysctl, the underlying variable will be removed once all OBD_ALLOC* macros removal patchseries land. Signed-off-by:
-
Oleg Drokin authored
Dan Carpenter noticed that procfs conversion patches introduced a bug where should kobject_create_and_add, an error is not returned from class_procfs_init. Reported-by:
-
Oleg Drokin authored
It turns out if you call ldlm_pools_fini without completing the ldlm_pools_init, then attempt to unregister not yet registered shrinkers makes the kernel very unhappy. So make sure we have them registered first. Signed-off-by:
-
Oleg Drokin authored
ldebugfs_remove is usually called on directories with files passed in as attributes, so simple debugfs_remove failes on them as not empty Switch to debugfs_remove_recursive. This fixes a number of problems where a new filesystem is mounted after being unmounted first. Signed-off-by:
-
Oleg Drokin authored
This helps to avoid use after free on unmount. Signed-off-by:
-
Oleg Drokin authored
Apparently m86k cannot build if you include asm/irq.h before linux/* includes and fixing it there is hard. So just move asm/irq.h include to where it does not cause any problems. Thanks to Geert Uytterhoeven for getting to the root of it. Signed-off-by:
Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by:
-
Julia Lawall authored
Remove initialization of a variable that is immediately reassigned. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Julia Lawall authored
Remove initialization of a variable that is immediately reassigned. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
Julia Lawall authored
Remove initialization of a variable that is immediately reassigned. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
Julia Lawall authored
Remove initialization of a variable that is immediately reassigned. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
Julia Lawall authored
Remove initialization of a variable that is immediately reassigned. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
