- 03 Jan, 2017 39 commits
-
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Reviewed-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Piotr Gregor authored
Add names of parameters to function prototypes in comedi PCI. Checkpatch reports now no errors. Signed-off-by:
Piotr Gregor <piotrgregor@rsyncme.org> Reviewed-by:
-
Matthew Smith authored
Fixes "WARNING: Statements should start on a tabstop" from checkpatch.pl Signed-off-by:
Matthew Smith <matthew11235@outlook.com> Signed-off-by:
-
Jaewon Kim authored
Initial Commit 349c9e13 ("gpu: ion: add CMA heap") returns -1 in allocation failure. The returned value is passed up to userspace through ioctl. So user can misunderstand error reason as -EPERM(1) rather than -ENOMEM(12). This patch simply changed this to return -ENOMEM. Signed-off-by:
Jaewon Kim <jaewon31.kim@samsung.com> Acked-by:
Laura Abbott <labbott@redhat.com> Signed-off-by:
-
David Kershner authored
Add the proper error handling to my_device_destroy so it can be send back up the stack for further error reporting. Reported-by:
David Kershner <david.kershner@unisys.com> Reviewed-by:
David Binder <david.binder@unisys.com> Signed-off-by:
-
David Kershner authored
The function my_device_changestate was not returning errors up the stack. Reported-by:
-
David Kershner authored
The function save_crash_message returns an error, don't ignore it, respond appropriately and send it up. Reported-by:
-
David Kershner authored
Add proper error handling to the function my_device_create and propagate the error message up the stack. Reported-by:
-
Bryan Thompson authored
Remove the MAX_BUF define and associated comments. Signed-off-by:
Bryan Thompson <bryan.thompson@unisys.com> Reviewed-by:
Tim Sell <Timothy.Sell@unisys.com> Signed-off-by:
-
David Binder authored
Removes references to visorchipset in the /proc filesystem, and replaces them with references to the /sys filesystem. Also removes reference to the visorchipset driver, and replaces it with a reference to plain visorchipset (since it is no longer a standalone driver). Signed-off-by:
-
Erik Arfvidson authored
This patch removes unused pound defines in controlvmchannel.h. Signed-off-by:
Erik Arfvidson <erik.arfvidson@unisys.com> Signed-off-by:
-
David Binder authored
The 'retry' variable created in handle_command() is statically allocated, and its address is never set to NULL. Therefore conditionals to verify the validity of the retry variable's address are unnecessary. Reported-by:
-
David Binder authored
Control flow is now directed using a switch statement, triggered by the enum crash_obj_type function parameter, instead of a set of conditional statements. Reported-by:
-
Sameer Wadgaonkar authored
This patch shortens a few error message defines by removing ERROR from the define. Reported-by:
Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com> Signed-off-by:
-
Geliang Tang authored
Use rb_entry_safe() instead of container_of() to simplify the code. Signed-off-by:
Geliang Tang <geliangtang@gmail.com> Signed-off-by:
-
Tabrez khan authored
Remove unnecessary braces {} for single while statement. This warning is found using checkpatch.pl. Signed-off-by:Tabrez khan <khan.tabrez21@gmail.com> Signed-off-by:
-
Ben Evans authored
Change the style of lustre_get_wire_obdo and lustre_set_wire_obdo to conform to linux kernel standard. Signed-off-by:
Ben Evans <bevans@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: http://review.whamcloud.com/16917 Reviewed-on: http://review.whamcloud.com/19266Reviewed-by:
James Simmons <uja.ornl@yahoo.com> Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
Ben Evans authored
Remove all obdo related functions from lustre_idl.h Create lustre_odbo.h. Include where appropriate. Make the functions lustre_get_wire_obdo and lustre_set_wire_obdo to not be inlined functions. Signed-off-by:
-
Quentin Lambert authored
A recent clean-up declared och_flags as a int rather than fmode_t. This lead to the following sparse warning: drivers/staging/lustre/lustre/llite/file.c:106:30: warning: restricted fmode_t degrades to integer This patch fixes this issue. Fixes: 0a120099 ("staging: lustre: cleanup lustre_lib.h") Signed-off-by:
Quentin Lambert <lambert.quentin@gmail.com> Acked-by:
-
James Simmons authored
It's not used anywhere outside of brw_test.c file. Highlighted by sparse. Signed-off-by:
-
Jinshan Xiong authored
Cleanup everything if an OSC is being evicted. Group lock is not well supported yet. Signed-off-by:
Jinshan Xiong <jinshan.xiong@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6271 Reviewed-on: http://review.whamcloud.com/14989Reviewed-by:
Bobi Jam <bobijam@hotmail.com> Reviewed-by:
-
Oleg Drokin authored
This avoids having an extern definition in a C file which is bad, and also silences sparse complaint as well. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Reviewed-by:
-
Oleg Drokin authored
Move it to obd.h, so that it's included from both the users and the actual definition, making sure they never get out of sync. This also silences a sparse warning. Signed-off-by:
-
Oleg Drokin authored
Make the declaration in a header, not as an extern in a C file, that is frowned upon. This also makes sparse a little bit more happy. Signed-off-by:
-
Oleg Drokin authored
This avoids frowned upon extern in the C file, and also shuts down a sparse warning of drivers/staging/lustre/lustre/osc/osc_dev.c:55:22: warning: symbol 'osc_caches' was not declared. Should it be static? Signed-off-by:
-
Oleg Drokin authored
It's not used anywhere outside of this file. Highlighted by sparse. Signed-off-by:
-
Oleg Drokin authored
It's not used anywhere out of this file. Highlighted by sparse. Signed-off-by:
-
Oleg Drokin authored
As per interval_search() prototype, the callback should return enum, not int. This fixes correspondign sparse warning. Signed-off-by:
-
Oleg Drokin authored
root_squash control got accidentally moved to sysfs instead of debugfs, and the write side of it was also broken expecting a userspace buffer. It contains both uid and gid values in a single file, so debugfs is a clear place for it. Reported-by:
Al Viro <viro@ZenIV.linux.org.uk> Fixes: c948390f "fix inconsistencies of root squash feature" Signed-off-by:
-
Kees Cook authored
Prepare to mark sensitive kernel structures for randomization by making sure they're using designated initializers. These were identified during allyesconfig builds of x86, arm, and arm64, with most initializer fixes extracted from grsecurity. Signed-off-by:
Kees Cook <keescook@chromium.org> Signed-off-by:
-
- 01 Jan, 2017 1 commit
-
-
Linus Torvalds authored
-
