- 16 Nov, 2015 40 commits
-
-
Shivani Bhardwaj authored
Remove the function cfs_node_to_cpumask() and replace all its calls with the function it wrapped. Signed-off-by:
Shivani Bhardwaj <shivanib134@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shivani Bhardwaj authored
Remove the function cfs_cpu_ht_siblings() and replace all its calls with the function it wrapped. Siigned-off-by:
-
Shivani Bhardwaj authored
Remove the function cfs_cpu_core_siblings() and replace its calls with the function it wrapped. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function __adler32() and replace its calls with the function it wrapped. Signed-off-by:
-
Shraddha Barke authored
The functions cfs_hash_bd_findadd_locked and cfs_hash_bd_finddel_locked are not used anywhere. Thus remove these functions. Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
Function cl_lock_mutex_try has been used only in this particular file. Thus declare the function as static. Signed-off-by:
-
Shraddha Barke authored
Functions ptlrpc_handle_failed_import and ptlrpc_lprocfs_do_request_stat have been declared but not used. Thus drop the declarations. Signed-off-by:
-
Shraddha Barke authored
The Linux kernel coding style guidelines suggest not using typedefs for structure types. This patch gets rid of the typedef for srpc_server_rpc_t. Signed-off-by:
-
James Simmons authored
Always place linux headers first in libcfs header files. This avoid can potential build issues if any changes to a libcfs header land that starts using a linux header definition. Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
James Simmons authored
Cleanup all the unneeded white space in hash.c. Signed-off-by:
-
James Simmons authored
Minor style cleanup to put hash.h header to the top of the libcfs_hash.h file. Signed-off-by:
-
James Simmons authored
Remove comment hash_long which was removed long ago. Signed-off-by:
-
James Simmons authored
Cleanup all the unneeded white space in libcfs_hash.h. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the functions node_equal() and node_compare() and replace their calls with appropriate functions. Signed-off-by:
-
Shraddha Barke authored
The functions mdc_pack_req and mdc_getxattr_pack have been declared in header file but not used. Thus remove the declarations. Signed-off-by:
-
Shraddha Barke authored
Declare functions lov_set_add_req, lov_set_finished, lov_update_set, lov_check_and_wait_active and lov_update_statfs as static since they are used only in this particular file. Also remove corresponding declarations from header file. Signed-off-by:
-
Shraddha Barke authored
These functions have been declared in header but not used anywhere. Thus drop the declarations. Signed-off-by:
-
Shraddha Barke authored
Declare functions lmv_intent_open and lmv_intent_lookup as static since they are used only in this particular file. Also remove corresponding declarations from header file. Signed-off-by:
-
Shraddha Barke authored
The function lmv_blocking_ast is declared in header file but not used. Hence remove the declaration. Signed-off-by:
-
Shraddha Barke authored
The functions cl_isize_lock and cl_isize_unlock can be replaced with direct calls to ll_inode_size_lock and ll_inode_size_unlock. Thus drop the wrapper functions. Signed-off-by:
-
Shraddha Barke authored
Declare osc_real_create, osc_create and osc_cleanup as static since they are used only in this particular file. Also remove the corresponding declarations from header file. Signed-off-by:
Arnd Bergmann <arnd@arndb.de> Signed-off-by:
-
Shraddha Barke authored
Function fld_dump_cache_entries is declared in header file but not used. Hence remove the declaration. Signed-off-by:
-
Shraddha Barke authored
Remove functions fld_cache_delete and fld_cache_delete_nolock since they are defined but not used. Signed-off-by:
-
Shraddha Barke authored
Declare fld_cache_entry_delete and fld_cache_insert_nolock as static since they are used only in this particular file. Also remove corresponding declarations from header file. Signed-off-by:
-
Shraddha Barke authored
Remove function seq_client_alloc_super since it is defined but not used. Also remove corresponding declaration from header file. Signed-off-by:
-
Shraddha Barke authored
Function mdc_open is declared in header file but it is not used anywhere. Hence drop the declaration. Signed-off-by:
-
Shraddha Barke authored
Declare mdc_get_lustre_md, mdc_free_lustre_md and mdc_clear_open_replay_data as static since they are used only in this particular file. Also remove corresponding declarations from header files. Signed-off-by:
-
Dan Carpenter authored
My static checker says that "group" is a user controlled number that can be negative leading to an array underflow. I have looked at it, and I'm not an expert enough in lustre to say for sure if it is really a bug. Anyway, it's simple enough to make the variable unsigned which pleases the static checker and makes it easier to audit. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Dan Carpenter authored
All the callers call these function pointers without the "m_" prefix and use macro magic to add it later behind the scenes. It means that you can't grep for the call sites. I considered modifying the call sites but in the end I decided that the "m_" prefix doesn't add anything so we can just get rid of it. Signed-off-by:
-
Dan Carpenter authored
We mostly refer to these function pointers using macros that use macro magic to add the "o_" prefix to the front. It means that you can't use cscope to find the caller. Heck, you can't even grep for it. I looked at preserving the "o_" prefix by removing the macro magic and adding "o_" to all the call sites but then I realized that, really, the prefix doesn't add any value. Let's just remove it. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function srpc_post_active_rqtbuf() and replace its only call with the function srpc_post_active_rdma() by adding appropriate parameters. Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function lnet_ipaddr_free_enumeration() and replace all its calls with LIBCFS_FREE(). Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function lnet_create_interface_cookie() and replace its call with the function ktime_get_ns(). Signed-off-by:
-
Shivani Bhardwaj authored
Remove return statement from the function lstcon_group_ndlink_move() as its return type is void. Fix checkpatch WARNING: void function return statements are not generally useful Signed-off-by:
-
Shivani Bhardwaj authored
Remove the function lstcon_group_put() and replace all its calls with the function lstcon_group_decref() because the former function just performs the job of calling the latter. Signed-off-by:
-
Ksenija Stanojevic authored
In all call sites type of argument that macro LPROCFS_CLIMP_EXIT use is the same, so replace it with up_read function. Also remove the macro since it's no longer used. Signed-off-by:
Ksenija Stanojevic <ksenija.stanojevic@gmail.com> Signed-off-by:
-
H Hartley Sweeten authored
The PCI-1710 series boards are multifunction data acquisition boards with analog inputs and outputs, digital inputs and outputs, and counter/timer functions. The PCI-1720 is a simple 4 channel analog output board. It also uses a unique register map. Separate out the PCI-1720 support as a new driver, adv_pci1720, to ease maintainability. Fix some issues with the PCI-1720 support in the new driver: 1) the registers are all 8-bit 2) remove the analog output "reset" when the driver attaches/detaches 3) disable "synchronized output" to simplify the analog outputs 4) remove the need for the private data 5) add support for the BoardID register to allow multiple cards Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Ksenija Stanojevic authored
Fix following sparse warning: warning: cast to restricted __be16 data is pointer of type void and can be used to store any type of data. barray and array have the same 16 bit offset. Signed-off-by:
-
Ksenija Stanojevic authored
Fix following sparse warnings: warning: cast to restricted __le32 warning: cast to restricted __le16 warning: incorrect type in assignment (different base types) expected unsigned short [unsigned] [assigned] val got restricted __le16 [usertype] <noident> Data is pointer of type void and can be used to store any type of data. In function ni_ai_munge: barray and array have the same 16 bit offset. blarray and larray have the same 32 bit offset. Signed-off-by:
-
Ksenija Stanojevic authored
Fix following sparse warning: warning: cast to restricted __le16 This change is safe because array is pointer of type void and can be used to store any type of data, also offset of the array would be the same since unsigned short and __le16 are both 16 bits in size. Signed-off-by:
-
