- 20 Feb, 2016 40 commits
-
-
Oleg Drokin authored
This is against kernel-code style. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Oleg Drokin authored
They make checkpatch unhappy, and I guess overall might confuse people too. Signed-off-by:
-
Oleg Drokin authored
Replace it with enum cfs_trace_buf_type Signed-off-by:
-
Oleg Drokin authored
We do really mean to fall through to that default case statement from all previous ones, so add a comment to unconfuse verious tools Signed-off-by:
-
Oleg Drokin authored
No need for an empty line after opening curvy brace and no need for an empty line before the closing one too. Remove them. Signed-off-by:
-
Oleg Drokin authored
Return at the end of a void function does not serve any particular purpose and makes checkpatch unhappy, so eliminate them. Signed-off-by:
-
Oleg Drokin authored
checkpatch complains that the trailing */ on the multiline comments should be on it's own line. Signed-off-by:
-
Oleg Drokin authored
the struct page_collection, struct cfs_trace_page and struct tracefiled_ctl are only used by tracefile.c, so move them there. Signed-off-by:
-
Oleg Drokin authored
Does not appear to be used anywhere. Signed-off-by:
-
Oleg Drokin authored
Lines that were too long for not good reason were shortened in this patch. Found with checkpatch. Signed-off-by:
-
Oleg Drokin authored
Problem highlighted by checkpatch. Signed-off-by:
-
Oleg Drokin authored
Avoid using leading spaces that make checkpatch unhappy. Signed-off-by:
-
Oleg Drokin authored
This fixes checkpatch messages about "spaces preferred around that '-' (ctx:VxV)" Signed-off-by:
-
Oleg Drokin authored
Found with checkpatch Signed-off-by:
-
Oleg Drokin authored
Adjust alignment of argments that were pushed to next lines to conform to kernel code style. Found with checkpatch. Signed-off-by:
-
Oleg Drokin authored
Checkpatch highlighted that some of our Lustre files carry this extra paragraph and indeed it does seem somewhat redundant, so remove it. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also remove some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" The only exception is ll_update_inode where dropping != NULL in the construction below would break the logic. I guess we could change lsm != NULL to !!lsm, but that's uglier. (lsm != NULL) == ((body->valid & OBD_MD_FLEASIZE) != 0) Also removed some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also removed some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
All instances of "x == NULL" are changed to "!x" and "x != NULL" to "x" Also removed some redundant assertions. Signed-off-by:
-
Oleg Drokin authored
The complicated version of osc_page_protected and osc_page_is_dlocked are unsafe and were commented out for ages, so probably no point in carrying them on. Signed-off-by:
-
Oleg Drokin authored
lock_lock_multi_match stayed commented out unused for ages now, so let's just remove it. Signed-off-by:
-
Oleg Drokin authored
class_get_type has some references to various server modules that could not exist on the server, so get rid of them. Signed-off-by:
-
Oleg Drokin authored
In client_obd_setup references to LUSTRE_OSP_NAME could only happen on metadata servers, so remove them as never true Signed-off-by:
-
Oleg Drokin authored
This only makes sense on metadata server, so get rid of it. Also remove now unused MDS_OSC_MAX_RIF_DEFAULT define Signed-off-by:
-
Oleg Drokin authored
All the users seems to have disappeared. Signed-off-by:
-
Oleg Drokin authored
Now that the sysfs conversion is complete, also convert all the remaining comments Signed-off-by:
-
Oleg Drokin authored
Now that the sysfs conversion is complete, also convert all the remaining comments Signed-off-by:
-
Oleg Drokin authored
Now that the sysfs conversion is complete, also convert all the remaining comments Signed-off-by:
-
