- 19 Nov, 2016 19 commits
-
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: No space is necessary after a cast Signed-off-by:
Walt Feasel <waltfeasel@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: Alignment should match open parenthesis Signed-off-by:
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: Logical continuations should be on the previous line Signed-off-by:
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: No space is necessary after a cast Signed-off-by:
-
Walt Feasel authored
Make suggested checkpatch modificationfor WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Walt Feasel authored
Make suggested checkpatch modificationfor CHECK: spaces preferred around that '+' Signed-off-by:
-
Walt Feasel authored
Make comment style modifications. Signed-off-by:
-
Walt Feasel authored
Make spelling correction for 'vertical'. Signed-off-by:
-
James Simmons authored
The types __[u|s]XX is only used for UAPI headers and userspace. Only keep these types for the libcfs headers that are UAPI headers. The rest convert to the standard uXX types. Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
James Simmons authored
Remove the zero comparisions in the libcfs headers. Signed-off-by:
-
James Simmons authored
Remove the NULL comparisions in the libcfs headers. Signed-off-by:
-
James Simmons authored
One last white space is still left in libcfs_fail.h. Lets remove it. Signed-off-by:
-
James Simmons authored
Use the BIT macros instead of (1 << ***) in libcfs headers. Signed-off-by:
-
James Simmons authored
Spell the word destroy correctly. Signed-off-by:
-
James Simmons authored
The header libcfs_fail.h has a extra blank line that is not needed. Signed-off-by:
-
James Simmons authored
Give the parameters names for the function prototypes in the libcfs headers. Signed-off-by:
-
James Simmons authored
Fixup the libcfs headers to use the proper unsigned int instead of raw unsigned. Signed-off-by:
-
James Simmons authored
Properly format the incorrect comments sections that were reported by checkpatch. Signed-off-by:
-
Mikhail Pershin authored
- llh_cat_idx may become bigger than llog bitmap size in llog_cat_set_first_idx() function - it is wrong to use previous cur_offset as new buffer offset, new offset should be calculated from value returned by llog_next_block(). - optimize llog_skip_over() to find llog entry offset by index for llog with fixed-size records. Signed-off-by:
Mikhail Pershin <mike.pershin@intel.com> Signed-off-by:
Bob Glossman <bob.glossman@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6714 Reviewed-on: http://review.whamcloud.com/15316 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6163 Reviewed-on: http://review.whamcloud.com/18819Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
James Simmons <uja.ornl@yahoo.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
- 18 Nov, 2016 19 commits
-
-
James Simmons authored
Cleanup some of the checkpatch over 80 character lines reported. Signed-off-by:
-
James Simmons authored
Some of the defines for lnet selftest can use the BIT macros. Signed-off-by:
-
James Simmons authored
The __[u|s]* types are for UAPI headers or user space. They shouldn't be used in core kernel code. Signed-off-by:
-
James Simmons authored
Checkpatch recommended that we use pr_*() instead of printk directly. Signed-off-by:
-
James Simmons authored
The envp char array can be made static constant. Signed-off-by:
-
James Simmons authored
Fix typo for correct spelling. Signed-off-by:
-
James Simmons authored
Remove white space present for variable declarations or initialization. Cleanup structs was strange alignments due to white spacing. Signed-off-by:
-
James Simmons authored
Testing == 0 is not kernel style so remove this type of testing from libcfs. Signed-off-by:
-
James Simmons authored
Testing != 0 is not kernel style so remove this type of testing from libcfs. Signed-off-by:
-
James Simmons authored
Checkpatch for some reason doesn't like the way libcfs_str2net_internal. Lets just replace it with nf being set to NULL instead Signed-off-by:
-
James Simmons authored
While checkpatch reported an alignment issue its just ugly to fill in a data structure being passed to a function. Instead fill in a lnet_md_t on the stack and pass that to LNetMDBind. Signed-off-by:
-
James Simmons authored
Add in missing blank line in lnet_copy_iov2iter() after variable declarations. Signed-off-by:
-
James Simmons authored
Remove unused lib_me_dump. Signed-off-by:
-
James Simmons authored
Fix misspelling of destroy in LNet core. Signed-off-by:
-
James Simmons authored
Make alignment match open parenthesis for parameters to wait_event_interruptible_exclusive() call. Signed-off-by:
-
James Simmons authored
Remove != NULL which is not needed to test key existence. Signed-off-by:
-
James Simmons authored
Change the static const array libcfs_debug_subsystems to use static const char const * as pointed out by checkpatch. Signed-off-by:
-
James Simmons authored
This patch resolves the checkpatch error: Block comments use a trailing */ on a separate line for the LNet/libcfs layer Signed-off-by:
-
James Simmons authored
Turn all bare unsigned to unsigned int that were detected by checkpatch in the LNet/libcfs layer. Signed-off-by:
-
- 17 Nov, 2016 2 commits
-
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: Alignment should match open parenthesis Signed-off-by:
-
Walt Feasel authored
Make suggested checkpatch modification for CHECK: Blank lines aren't necessary before a close brace '}' Signed-off-by:
-
