James Simmons
e86a615146
staging: lustre: uapi: migrate remaining uapi headers to uapi directory
...
Move all the remaining lustre headers shared between user land
and kernel space to the uapi directory.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/25246
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
1c07bedf73
staging: lustre: uapi: remove libcfs.h from lustre_id.h/lustre_user.h
...
These are the last UAPI headers that contain libcfs.h which is
internal kernel header. Since it is not available to user land
remove libcfs.h and add the need headers that libcfs.h provided.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/22138
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com >
Reviewed-by: Fan Yong <fan.yong@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
fa6f479cf7
staging: lustre: lnet: remove BIT macro from lnetctl.h
...
Now that lnetctl.h is a UAPI header the BIT macro has to be
removed.
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
150414213e
staging: lustre: lnet: remove userland function prototype in lnetctl.h
...
Several function prototypes of the form jt_ptl_* are only needed
by userland so they can be removed.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: http://review.whamcloud.com/17643
Reviewed-by: Bob Glossman <bob.glossman@intel.com >
Reviewed-by: John L. Hammond <john.hammond@intel.com >
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
6c1c720130
staging: lustre: libcfs: sort headers in libcfs.h
...
Move all the included headers in libcfs.h to the top of the
file.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/28089
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Olaf Weber <olaf.weber@hpe.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
037697dc2a
staging: lustre: lnet: migrate headers to lnet uapi directory
...
Migrate the headers used by user land and kernel space to the
libcfs/lnet uapi directory.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/28089
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Olaf Weber <olaf.weber@hpe.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
28cb3e268f
staging: lustre: lnet: delete lnet.h
...
The header lnet.h is just a bunch of headers included in
a header. Just delete it and include the appropriate
headers where needed.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/28089
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Olaf Weber <olaf.weber@hpe.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
e4d64b3084
staging: lustre: socklnd: create socklnd.h UAPI header
...
Break out the parts from libcfs_debug.h that is used by both user
land and kernel space into a new UAPI header.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/28089
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Olaf Weber <olaf.weber@hpe.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:50 -07:00
James Simmons
ad6f42598b
staging: lustre: libcfs: create libcfs_debug.h UAPI header
...
Break out the parts from libcfs_debug.h that is used by both user
land and kernel space into a new UAPI header.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/28089
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Olaf Weber <olaf.weber@hpe.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
cef7b965ae
staging: lustre: libcfs: remove LOGL and LOGU macros
...
The macro LOGU is not used anymore and LOGL is used in
one place. No reason to keep LOGL around anymore.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/22138
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com >
Reviewed-by: Fan Yong <fan.yong@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
6efbda3d62
staging: lustre: libcfs: remove htonl hack in libcfs.h
...
This is really old hack for earlier gcc version that
had a hard time compiling byteorder.h. Lets remove it.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/22138
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com >
Reviewed-by: Fan Yong <fan.yong@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
d0982327a7
staging: lustre: uapi: label lustre_cfg.h as an uapi header
...
Change LUSTRE_CFG_H to _UAPI_LUSTRE_CFG_H_ now that it is
a proper UAPI header.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
6238fbd2f6
staging: lustre: uapi: style cleanup of lustre_cfg.h
...
Some style and white space cleanups to make lustre_cfg.h
easy to read.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
ef27faebba
staging: lustre: uapi: check if argument for lustre_cfg_buf() is NULL
...
Check if lcfg passed in is NULL and if it is just return NULL.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245
Reviewed-on: https://review.whamcloud.com/22138
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com >
Reviewed-by: Fan Yong <fan.yong@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
63d384bfa6
staging: lustre: uapi: change variable type to match
...
Change i from int to __u32 to match lcfg_bufcount field. Also
this matches what the other functions also do.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
955073c4fe
staging: lustre: uapi: remove need for libcfs.h from lustre_cfg.h
...
Several libcfs.h macros are present in lustre_cfg.h, but libcfs.h
is not available so lets replace those macros with real code.
The header libcfs.h also provides errno.h so with libcfs.h gone
include errno.h directly.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
f55b74110f
staging: lustre: uapi: move lustre_cfg.h to uapi directory
...
Move lustre_cfg.h to its proper place. Adjust additonal headers
included in lustre_cfg.h.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
4599c8ebb2
staging: lustre: obdclass: no need to check for kfree
...
In class_config_llog_handler() a flag, inst, was used to
determine if it was safe to call kfree. Its not needed so
remove the flag.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
90d1e58830
staging: lustre: uapi: move lustre_cfg_string() to obd_config.c
...
To perserve the truncate warning move lustre_cfg_string()
to obd_config.c. A identical function was created for
userland. This function was a bit big for a inline function.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:49 -07:00
James Simmons
402fba76c3
staging: lustre: uapi: don't memory allocate in UAPI header
...
The inline function lustre_cfg_new() calls kzalloc() but
this is a UAPI header. Remove kzalloc() and rename the
function to lustre_cfg_init(). The lustre kernel code
that was calling lustre_cfg_new() can doing the memory
allocation and pass the new buffer to lustre_cfg_init()
to fill in.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
18bd20841b
staging: lustre: uapi: remove lustre_cfg_free wrapper
...
Replace all lustre_cfg_free() calls with direct kfree()
call.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
e3acb49a30
staging: lustre: uapi: style cleanups for lustre_param.h
...
Remove the white space in the lustre_param.h header and
align the code so its easier to read.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24325
Reviewed-by: Ben Evans <bevans@cray.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: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
7326fbb915
staging: lustre: uapi: label lustre_param.h as an uapi header
...
Change LUSTRE_PARAM_H_ to _UAPI_LUSTRE_PARAM_H_ now that is
a proper UAPI header
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24325
Reviewed-by: Ben Evans <bevans@cray.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: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
d563149f06
staging: lustre: uapi: move lustre_param.h to uapi directory
...
Move the header lustre_param.h to proper uapi directory.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24325
Reviewed-by: Ben Evans <bevans@cray.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: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
bcf0ec951b
staging: lustre: uapi: remove included headers out of lustre_param.h
...
All the included headers in lustre_param.h are lustre kernel
internal headers and they are not even needed. Just remove them.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24325
Reviewed-by: Ben Evans <bevans@cray.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: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
64a35c1b71
staging: lustre: uapi: move kernel only prototypes out of lustre_param.h
...
Move all the kernel specific function prototypes from
lustre_param.h into obd_config.h which is a kernel only
header.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24325
Reviewed-by: Ben Evans <bevans@cray.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: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
6fb4eaa2e5
staging: lustre: uapi: label lustre_ioctl.h as a UAPI header
...
Change LUSTRE_IOCTL_H_ to _UAPI_LUSTRE_IOCTL_H_ now that is
a proper UAPI header
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
3f443e003e
staging: lustre: uapi: cleanup headers for lustre_ioctl.h
...
With the cleanups we no longer need to drag in tons of
user land headers. Lets simplify the include headers.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
30bfc47885
staging: lustre: uapi: use __ALIGN_KERNEL for lustre_ioctl.h
...
Replace cfs_size_round() standard __ALIGN_KERNEL macro. This
removes the dependency of libcfs.h which is a kernel only
header.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
325da9674b
staging: lustre: uapi: move lustre_ioctl.h to uapi directory
...
Move the header lustre_ioctl.h to proper uapi directory.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:48 -07:00
James Simmons
efd117be43
staging: lustre: uapi: move obd_ioctl_is_invalid() to linux-module.c
...
The inline function obd_ioctl_is_invalid() is no longer needed by
userland and also the function was pretty bug for a inline function.
Since this is the case we can move this kernel only code to the
linux-module.c which is the only place it is used.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
f344c9e46f
staging: lustre: uapi: move obd_ioctl_getdata() declaration
...
Move obd_ioctl_getdata() from lustre_ioctl.h to obd_class.h
since this function is only used by kernel space.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
b03679f6a4
staging: lustre: uapi: remove obd_ioctl_popdata() wrapper
...
Replace obd_ioctl_popdata() with direct copy_to_user() call.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
bb44b987c8
staging: lustre: uapi: remove obd_ioctl_freedata() wrapper
...
Replace obd_ioctl_freedata() with direct kvfree() call.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
d014786271
staging: lustre: uapi: remove userland version of obd_ioctl_*()
...
The header lustre_ioctl.h has both kernel and user space versions
of obd_ioctl_*() functions. Lets remove the user land version
which have been integrated into the lustre tools.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/24568
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: Ben Evans <bevans@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
11dc946720
staging: lustre: uapi: remove unused function in lustre_disk.h
...
Delete code that is used by only the server or just in user
land. Now this file is a normal kernel header. The UAPI
version of this header is only needed for server side so
no need to create a new header in uapi directory.
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/25194
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com >
Reviewed-by: John L. Hammond <john.hammond@intel.com >
Reviewed-by: Lai Siyao <lai.siyao@intel.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
Ben Evans
071277d9c5
staging: lustre: uapi: move lu_fid, ost_id funcs out of lustre_idl.h
...
Move lu_fid functions into lustre/lustre_fid.h
Move ost_id functions into lustre/lustre_ostid.h
Fix indenting, include new headers as needed.
Several inline functions in the header lustre_ostid.h
are using debug macros instead of returning proper errors.
Remove the debug macros and properly handle the returned
error codes. Place both UAPI headers lustre_fid.h and
lustre_ostid.h into the uapi directory.
Signed-off-by: Ben Evans <bevans@cray.com >
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/22712
Reviewed-on: https://review.whamcloud.com/24569
Reviewed-by: James Simmons <uja.ornl@yahoo.com >
Reviewed-by: Frank Zago <fzago@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
Ben Evans
8d2003883e
staging: lustre: uapi: update URL doc link in lustre_fid.h
...
The wiki has been revived so update the URL about the FID
design in lustre_fid.h.
Signed-off-by: Ben Evans <bevans@cray.com >
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/22712
Reviewed-on: https://review.whamcloud.com/24569
Reviewed-by: James Simmons <uja.ornl@yahoo.com >
Reviewed-by: Frank Zago <fzago@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
Ben Evans
7d2fa4cbf3
staging: lustre: uapi: return error code for ostid_set_id
...
Currently the function ostid_set_id() just logs an error
and just keeps going. This function is also used by user
land tools so having special lustre kernel debugging
code is not desired. So instead just return an error
code and have the calling kernel code print the error
instead.
Signed-off-by: Ben Evans <bevans@cray.com >
Signed-off-by: James Simmons <uja.ornl@yahoo.com >
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/22712
Reviewed-on: https://review.whamcloud.com/24569
Reviewed-by: James Simmons <uja.ornl@yahoo.com >
Reviewed-by: Frank Zago <fzago@cray.com >
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
James Simmons
3c42fb1234
staging: lustre: uapi: remove unused functions for lustre_fid.h
...
The inline function fid_is_namespace_visible and ost_fid_from_resid
are only used for server code so we can remove it.
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:47 -07:00
Ben Evans
22c147fdd9
staging: lustre: uapi: Move functions out of lustre_idl.h
...
Migrate functions set/get_mrc_cr_flags, ldlm_res_eq
ldlm_extent_overlap, ldlm_extent_contain,
ldlm_request_bufsize, and all the PTLRPC dump_*
functions out of lustre_idl.h which is a UAPI header
to the places in the kernel code they are actually used.
Delete unused lmv_mds_md_stripe_count and
agent_req_in_final_state.
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/21484
Reviewed-by: Frank Zago <fzago@cray.com >
Reviewed-by: James Simmons <uja.ornl@yahoo.com >
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com >
Reviewed-by: John L. Hammond <john.hammond@intel.com >
Signed-off-by: James Simmons <jsimmons@infradead.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:46 -07:00
Colin Ian King
f942efe73a
staging: r8822be: fix a couple of spelling mistakes
...
Trivial fixes to spelling mistake in pr_err error messages:
errorr -> error
mismactch -> mismatch
Signed-off-by: Colin Ian King <colin.king@canonical.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:14 -07:00
Colin Ian King
53b70e5c2e
staging: typec: tcpm: make function tcpm_get_pwr_opmode
...
The function pointer tcpm_get_pwr_opmode is local to the source and does
not need to be in global scope, so make it static.
Cleans up sparse warning:
symbol 'tcpm_get_pwr_opmode' was not declared. Should it be static?
Signed-off-by: Colin Ian King <colin.king@canonical.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:36:14 -07:00
Srishti Sharma
ac985b6cd9
Staging: fsl-dpaa2: ethernet: dpni.c: Fixed alignment to match open parenthesis.
...
Fixed alignment so that it matched open parenthesis
Signed-off-by: Srishti Sharma <srishtishar@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:33:25 -07:00
Arvind Yadav
ccc5d98ae0
staging: greybus: audio: constify snd_soc_dai_ops structures
...
snd_soc_dai_ops are not supposed to change at runtime. All functions
working with snd_soc_dai_ops provided by <sound/soc-dai.h> work with
const snd_soc_dai_ops. So mark the non-const structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com >
Acked-by: Mark Greer <mgreer@animalcreek.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:33:19 -07:00
Eames Trinh
4a27e3e09d
Staging: greybus: Fix spelling error in comment
...
Fixed a spelling error.
Signed-off-by: Eames Trinh <eamestrinh@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:33:19 -07:00
Randy Dunlap
0be5dec503
staging: rtlwifi: fix multiple build errors
...
Fix many build errors when NETDEVICES is not enabled.
Signed-off-by: Randy Dunlap <rdunlap@infradead.org >
Cc: Larry Finger <Larry.Finger@lwfinger.net >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 18:33:19 -07:00
David Kershner
aa5ebdeb5d
staging: unisys: use ATTRIBUTE_GROUPS instead of creating our own
...
Greg noticed that we were creating our own attribute groups, instead of
using the ATTRIBUTE_GROUPS macro, this patch fixes our mistake.
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Signed-off-by: David Kershner <david.kershner@unisys.com >
Reviewed-by: Tim Sell <timothy.sell@unisys.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 15:14:55 -07:00
Sameer Wadgaonkar
b74856b419
staging: unisys: visorbus: Get rid of passthrough function visorchipset_device_destroy
...
The function visorchipset_device_destroy just called remove_visor_device,
we can just combine it with remove_visor_device.
Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com >
Signed-off-by: David Kershner <david.kershner@unisys.com >
Reviewed-by: Tim Sell <timothy.sell@unisys.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 15:14:55 -07:00
Sameer Wadgaonkar
51c0f81cca
staging: unisys: visorbus: Get rid of passthrough function visorchipset_device_create
...
The function visorchipset_device_create just called create_visor_device,
we can just combine it with create_visor_device.
Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com >
Signed-off-by: David Kershner <david.kershner@unisys.com >
Reviewed-by: Tim Sell <timothy.sell@unisys.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2017-08-22 15:14:55 -07:00