- 21 Dec, 2015 40 commits
-
-
H Hartley Sweeten authored
Reformat the bolck comment in the kernel CodingStyle. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
H Hartley Sweeten authored
These are not necessary and just add cruft. Remove them. Signed-off-by:
-
Ranjith Thangavel authored
BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by:
Ranjith Thangavel <ranjithece24@gmail.com> Reviewed-by:
-
Andrzej Pietrasiewicz authored
ni_release_ai_mite_channel(), ni_release_ao_mite_channel(), ni_release_gpct_mite_channel() and ni_release_cdo_mite_channel() call functions which interpret -1 as a special value meaning "no channel". This patch adds explicit "no_channel" versions instead. On the other hand, after "no_channel" versions are used, ni_set_ai_dma_channel(), ni_set_ao_dma_channel(), ni_set_gpct_dma_channel(), ni_set_cdo_dma_channel() are called with actual "channel" parameter being always unsigned, so their signatures are changed accordingly. A side benefit of the changes is suppressesing 4 sparse warnings: "warning: shift too big (4294967295) for type int". Signed-off-by:
Andrzej Pietrasiewicz <andrzejtp2010@gmail.com> Reviewed-by:
-
Daniel H. Hemmingsen authored
Fixed multiple comment blocks that didn't comply with the kernels coding style, and fixed a few spacing issues as well. Signed-off-by:
Daniel H. Hemmingsen <dhh.kernel@gmail.com> Reviewed-by:
-
Ranjith Thangavel authored
BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by:
-
Ranjith Thangavel authored
BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by:
-
Ranjith Thangavel authored
BIT macro is used for defining BIT location instead of shifting operator - coding style issue Signed-off-by:
-
Geliang Tang authored
Use kmalloc_array instead of kmalloc to allocate memory for an array. Signed-off-by:
Geliang Tang <geliangtang@163.com> Reviewed-by:
-
Jitendra Kumar Khasdev authored
This patch is to file dgnc_cls.c that fixes up udelay function by usleep_range. It is safe to use according to the following documentation Documentation/timers/timers-howto.txt. So that is why I have given an appropriate time range. Signed-off-by:
Jitendra Kumar Khasdev <jkhasdev@gmail.com> Signed-off-by:
-
Nizam Haider authored
Fix Typo errror in the comment section of dgnc_wmove Signed-off-by:
Nizam Haider <nijamh@cdac.in> Signed-off-by:
-
Nizam Haider authored
Fix Checlpatch warning HECK: braces {} should be used on all arms of this statement Signed-off-by: -
Bogicevic Sasa authored
This fixes all "spaces preferred around that ..." messages from checkpatch.pl Signed-off-by:
Bogicevic Sasa <brutallesale@gmail.com> Signed-off-by:
-
Markus Elfring authored
The script "checkpatch.pl" pointed out that labels should not be indented. Thus delete a horizontal tab before the jump label "out" in the function "osc_get_info". Signed-off-by:
Markus Elfring <elfring@users.sourceforge.net> Signed-off-by:
-
Markus Elfring authored
The variable "mne_swab" will eventually be set to an appropriate value from a call of the ptlrpc_rep_need_swab() function. Thus let us omit the explicit initialisation at the beginning. Signed-off-by:
-
Markus Elfring authored
Six goto statements referred to a source code position directly behind them. Thus omit such unnecessary jumps. This issue was detected by using the Coccinelle software. Signed-off-by:
-
Shraddha Barke authored
lu_site_hash_ops is used only in this particular file. Thus declare it as static. Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
osc_attr_set is used only in this particular file. Thus declare it as static. Signed-off-by:
-
Shraddha Barke authored
Declare osc_extent_find and osc_unreserve_grant as static since they are used only in this particular file. Signed-off-by:
-
Shraddha Barke authored
Function libcfs_debug_set_level is defined but not used. Thus remove it. Signed-off-by:
-
Shraddha Barke authored
Declare cl_env_peek as static since it is used only in this file. Also remove the EXPORT SYMBOL. Signed-off-by:
-
Shraddha Barke authored
Functions LNetEQWait and LNetEQGet are defined but not used. Thus remove it. Also remove corresponding declarations from header file. Signed-off-by:
-
Shraddha Barke authored
Function llog_backup is declared in header file but not used. Thus remove it. Signed-off-by:
-
Rasmus Villemoes authored
When char is signed and one of the bytes in lmm happens to have a byte value above 127, the result of printing that with %.2X will be 8 hex chars, the first 6 of which are 'F'. Worst case, we'll overrun our 'carefully' allocated buffer. I didn't have the tenacity to work through the gazillion and seven layers of macros behind CERROR, but I assume it'll all end at some function implemented in terms of the kernel's vsnprintf. Use %*phN for a hexdump. That'll cap the number of dumped bytes at 64. If that's a problem, the loop could be replaced by "bin2hex(buffer, lmm, lmm_bytes);". Signed-off-by:
Rasmus Villemoes <linux@rasmusvillemoes.dk> Signed-off-by:
-
Anjali Menon authored
Removed a blank line after the open brace to remove the check detected by the checkpatch.pl. CHECK: Blank lines aren't necessary after an open brace '{' Signed-off-by:Anjali Menon <cse.anjalimenon@gmail.com> Signed-off-by:
-
Aya Mahfouz authored
Removes IS_PO2 and __is_po2 since the uses of IS_PO2 have been replaced by is_power_of_2 Signed-off-by:
Aya Mahfouz <mahfouz.saif.elyazal@gmail.com> Signed-off-by:
-
Aya Mahfouz authored
Replaces IS_PO2 by is_power_of_2. It is more accurate to use is_power_of_2 since it returns 1 for numbers that are powers of 2 only whereas IS_PO2 returns 1 for 0 and numbers that are powers of 2. Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Julia Lawall authored
The inode_operations structures are never modified, so declare them as const, like all the other inode_operations structures. Done with the help of Coccinelle. Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Shivani Bhardwaj authored
Remove the wrapper function cl_2queue_add() and replace all its calls in different files with the function it wrapped. Also, comments are added wherever necessary to make the working of function clear. Prototype of the function is also removed from the header file as it is no longer needed. Signed-off-by:
Shivani Bhardwaj <shivanib134@gmail.com> Signed-off-by:
-
Shivani Bhardwaj authored
Remove unnecessary wrapper function cl_lock_descr_fid() and replace all its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the wrapper function lustre_posix_acl_xattr_free() and replace its call in the file xattr with the function kfree() that it wrapped. Also, its prototype from the header lustre_eacl is removed as it is no longer of any use. Signed-off-by:
-
John L. Hammond authored
Remove comment about tracefile handling for user land version of libcfs that no longer exist. Broken out of patch http://review.whamcloud.com/11983. Signed-off-by:
John L. Hammond <john.hammond@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/11983Reviewed-by:
Bob Glossman <bob.glossman@intel.com> Reviewed-by:
James Simmons <uja.ornl@gmail.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
John L. Hammond authored
Move the definition of the trace buffer type enum in libcfs/libcfs/tracefile.h. Remove the then unneeded headers libcfs/libcfs/linux/linux-tracefile.h and libcfs/libcfs/posix/posix-tracefile.h. Signed-off-by:
-
Andreas Dilger authored
Update copyright messages in files modified by Intel employees in 2015 by non-trivial patches. Exclude patches that are only deleting code, renaming functions, or adding or removing whitespace. Signed-off-by:
James Nunez <james.a.nunez@intel.com> Reviewed-by:
James Simmons <uja.ornl@yahoo.com> Reviewed-by:
-
frank zago authored
A lot of symbols don't need to be exported at all because they are only used in the module they belong to. Signed-off-by:
frank zago <fzago@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5829 Reviewed-on: http://review.whamcloud.com/13320Reviewed-by:
Isaac Huang <he.huang@intel.com> Reviewed-by:
-
John L. Hammond authored
Remove the file /proc/fs/lustre/mdc/*/hsm_nl which was introduced "for testing purposes." Signed-off-by:
jacques-Charles Lafoucriere <jacques-charles.lafoucriere@cea.fr> Reviewed-by:
Aurelien Degremont <aurelien.degremont@cea.fr> Reviewed-by:
-
Shivani Bhardwaj authored
Remove the wrapper function mdc_kuc_reregister() and replace its call with the function it wrapped. Also, comment has been added for clarity. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the wrapper function cfs_wi_sched_unlock() and replace all its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the wrapper function cfs_wi_sched_lock() and replace all its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function agl_list_empty() and replace its calls with the function it wrapped. Signed-off-by:
-
