[PATCH 64/64 v2] staging: lustre: lnet: cleanup paths for all LNet headers

2017-08-19 Thread James Simmons
Rationalize include paths in all the lnet header files. Signed-off-by: James Simmons --- drivers/staging/lustre/include/linux/lnet/api.h | 2 +- drivers/staging/lustre/include/linux/lnet/lib-lnet.h | 14 +++--- drivers/staging/lustre/include/linux/lnet/lib-types.h |

[PATCH 63/64 v2] staging: lustre: libcfs: cleanup paths for libcfs headers

2017-08-19 Thread James Simmons
Rationalize include paths in all the libcfs header files. Signed-off-by: James Simmons --- .../staging/lustre/include/linux/libcfs/libcfs.h | 24 +++--- .../lustre/include/linux/libcfs/libcfs_debug.h | 2 +- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/

[PATCH 59/64 v2] staging: lustre: lnet: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the lnet core source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/lnet/Makefile | 3 +++ drivers/staging/lustre/lnet/lnet/acceptor.c| 2 +- drivers/staging/lustre/lnet/lnet/api-ni.c | 4 ++-- drivers/staging/lustre/lnet/lnet/

[PATCH 57/64 v2] staging: lustre: lustre: cleanup paths for lustre UAPI headers

2017-08-19 Thread James Simmons
Rationalize include paths for the lustre uapi headers Signed-off-by: James Simmons --- drivers/staging/lustre/include/uapi/linux/lustre/lustre_cfg.h | 2 +- drivers/staging/lustre/include/uapi/linux/lustre/lustre_fid.h | 2 +- drivers/staging/lustre/include/uapi/linux/lustre/lustre_idl.h |

[PATCH 60/64 v2] staging: lustre: ko2iblnd: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the ko2iblnd source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/klnds/o2iblnd/Makefile | 3 +++ drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

[PATCH 58/64 v2] staging: lustre: lnet: selftest: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the lnet selftest source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/selftest/Makefile | 3 +++ drivers/staging/lustre/lnet/selftest/conctl.c | 6 +++--- drivers/staging/lustre/lnet/selftest/conrpc.c | 4 ++-- drivers/staging/lustre

[PATCH 61/64 v2] staging: lustre: ksocklnd: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the ksocklnd source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/klnds/socklnd/Makefile | 3 +++ drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h | 6 +++--- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/stagin

[PATCH 56/64 v2] staging: lustre: lustre: cleanup paths for lustre internal headers

2017-08-19 Thread James Simmons
Rationalize include paths for the lustre internal headers Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/cl_object.h | 4 +-- drivers/staging/lustre/lustre/include/llog_swab.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 6 ++--- drivers/staging/lustr

[PATCH 62/64 v2] staging: lustre: libcfs: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the libcfs source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/libcfs/Makefile| 3 +++ drivers/staging/lustre/lnet/libcfs/debug.c | 2 +- drivers/staging/lustre/lnet/libcfs/fail.c | 2 +- dr

[PATCH 51/64 v2] staging: lustre: mdc: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the mdc source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/mdc/Makefile | 3 +++ drivers/staging/lustre/lustre/mdc/lproc_mdc.c| 4 ++-- drivers/staging/lustre/lustre/mdc/mdc_internal.h | 2 +- drivers/staging/lustre/lustre/

[PATCH 52/64 v2] staging: lustre: mgc: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the mgc source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/mgc/Makefile | 3 +++ drivers/staging/lustre/lustre/mgc/lproc_mgc.c| 4 ++-- drivers/staging/lustre/lustre/mgc/mgc_internal.h | 10 +- drivers/staging/lustre

[PATCH 54/64 v2] staging: lustre: obdecho: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the obdecho source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/obdecho/Makefile | 3 +++ .../staging/lustre/lustre/obdecho/echo_client.c| 24 +++--- 2 files changed, 15 insertions(+), 12 deletions(-) diff --git

[PATCH 50/64 v2] staging: lustre: lov: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the lov source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/lov/Makefile | 3 ++ .../staging/lustre/lustre/lov/lov_cl_internal.h| 6 ++-- drivers/staging/lustre/lustre/lov/lov_dev.c| 2 +- drivers/staging/lustre/lu

[PATCH 55/64 v2] staging: lustre: osc: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the osc source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/osc/Makefile | 3 ++ drivers/staging/lustre/lustre/osc/lproc_osc.c | 6 ++-- .../staging/lustre/lustre/osc/osc_cl_internal.h| 6 ++-- drivers/staging/lustre/

[PATCH 53/64 v2] staging: lustre: obdclass: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the obdclass source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/obdclass/Makefile | 3 +++ drivers/staging/lustre/lustre/obdclass/cl_io.c | 8 drivers/staging/lustre/lustre/obdclass/cl_lock.c | 8 dri

[PATCH 49/64 v2] staging: lustre: lmv: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the lmv source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/lmv/Makefile | 3 +++ drivers/staging/lustre/lustre/lmv/lmv_fld.c | 14 +++--- drivers/staging/lustre/lustre/lmv/lmv_intent.c | 16 driver

[PATCH 48/64 v2] staging: lustre: llite: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the llite source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/Makefile | 3 ++ drivers/staging/lustre/lustre/llite/dcache.c | 6 ++-- drivers/staging/lustre/lustre/llite/dir.c | 16 +- drivers/staging

[PATCH 46/64 v2] staging: lustre: fld: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the fld source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/fld/Makefile | 3 +++ drivers/staging/lustre/lustre/fld/fld_cache.c| 16 drivers/staging/lustre/lustre/fld/fld_internal.h | 8 drivers/stag

[PATCH 45/64 v2] staging: lustre: fid: add include path to Makefile

2017-08-19 Thread James Simmons
Start to rationalize include paths in the fid source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/fid/Makefile | 3 +++ drivers/staging/lustre/lustre/fid/fid_internal.h | 4 ++-- drivers/staging/lustre/lustre/fid/fid_lib.c | 4 ++-- drivers/staging/lus

[PATCH 47/64 v2] staging: lustre: ptlrpc: add include path to Makefile

2017-08-19 Thread James Simmons
Rationalize include paths in the ptlrpc/ldlm source code files. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/ldlm/interval_tree.c | 6 +++--- drivers/staging/lustre/lustre/ldlm/l_lock.c | 6 +++--- drivers/staging/lustre/lustre/ldlm/ldlm_extent.c | 12 ++---

[PATCH 44/64 v2] staging: lustre: uapi: remove BIT macro from UAPI headers

2017-08-19 Thread James Simmons
The BIT macro is not available for UAPI headers so remove it from the lustre UAPI headers. Signed-off-by: James Simmons --- .../lustre/include/uapi/linux/lustre/lustre_idl.h | 38 +++--- .../lustre/include/uapi/linux/lustre/lustre_user.h | 12 +++ 2 files changed, 25 inserti

[PATCH 42/64 v2] staging: lustre: uapi: remove CONFIG_LUSTRE_OBD_MAX_IOCTL

2017-08-19 Thread James Simmons
Now that lustre_ioctl.h is a UAPI header the kernel configuration option CONFIG_LUSTRE_OBD_MAX_IOCTL needs to be remove. The user land utilites will no longer be able to see this option and actually they never used this option before. Since this is the case setting the kernel configuration to somet

[PATCH 40/64 v2] staging: lustre: uapi: remove libcfs.h from lustre_id.h/lustre_user.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https:/

[PATCH 41/64 v2] staging: lustre: uapi: migrate remaining uapi headers to uapi directory

2017-08-19 Thread James Simmons
Move all the remaining lustre headers shared between user land and kernel space to the uapi directory. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/25246 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans Reviewe

[PATCH 43/64 v2] staging: lustre: uapi: use proper byteorder functions in lustre_idl.h

2017-08-19 Thread James Simmons
In order for lustre_idl.h to be usable for both user land and kernel space it has to use the proper byteorder functions. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/16916 Reviewed-by: Frank Zago Reviewed-by: Dmitr

Re: [PATCH v2 01/14] staging: lustre: llite: Remove filtering of seclabel xattr

2017-08-19 Thread James Simmons
> On Mon, Aug 14, 2017 at 12:20:51PM -0400, James Simmons wrote: > > From: Robin Humble > > > > The security.capability xattr is used to implement File > > Capabilities in recent Linux versions. Capabilities are a > > fine grained approach to granting executables elevated > > privileges. eg. /bi

Re: [PATCH] staging: lustre: fix structure size for ARM OABI

2017-08-19 Thread James Simmons
> On Aug 17, 2017, at 10:26, Greg KH wrote: > > > > On Wed, Aug 16, 2017 at 05:44:15PM +0300, Cihangir Akturk wrote: > >> When building the kernel for the ARM architecture without setting > >> CONFIG_AEABI, size of struct lov_user_md_v3 and struct lov_mds_md_v3 > >> differs, due to different ali

[PATCH RFC] staging: lustre: libcfs: tracepoint implementation

2017-08-19 Thread James Simmons
The other eye sore for lustre after the UAPI mess is the loony debugging system. This debugging system was written long before trace point existed. Well it shows and todays trace point has surpassed lustre debugging system. Some of the things implemented are: 1) trace events have a specific namin

[PATCH 27/64 v2] staging: lustre: uapi: change variable type to match

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans R

[PATCH 28/64 v2] staging: lustre: uapi: check if argument for lustre_cfg_buf() is NULL

2017-08-19 Thread James Simmons
Check if lcfg passed in is NULL and if it is just return NULL. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/22138 Reviewed-by: Dmitry Eremin Reviewed-by: Jinshan Xiong Reviewed-by: Fan Yong Reviewed-by: Oleg Dro

[PATCH 23/64 v2] staging: lustre: uapi: move lustre_cfg_string() to obd_config.c

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.

[PATCH 38/64 v2] staging: lustre: lnet: remove userland function prototype in lnetctl.h

2017-08-19 Thread James Simmons
Several function prototypes of the form jt_ptl_* are only needed by userland so they can be removed. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/17643 Reviewed-by: Bob Glossman Reviewed-by: John L. Hammond Review

[PATCH 20/64 v2] staging: lustre: uapi: style cleanups for lustre_param.h

2017-08-19 Thread James Simmons
Remove the white space in the lustre_param.h header and align the code so its easier to read. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24325 Reviewed-by: Ben Evans Reviewed-by: John L. Hammond Reviewed-by: Ol

[PATCH 31/64 v2] staging: lustre: libcfs: remove htonl hack in libcfs.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/22138 Reviewed-by: Dmitry Eremin Reviewed-by: Jinshan Xiong

[PATCH 30/64 v2] staging: lustre: uapi: label lustre_cfg.h as an uapi header

2017-08-19 Thread James Simmons
Change LUSTRE_CFG_H to _UAPI_LUSTRE_CFG_H_ now that it is a proper UAPI header. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Sign

[PATCH 26/64 v2] staging: lustre: uapi: remove need for libcfs.h from lustre_cfg.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/L

[PATCH 33/64 v2] staging: lustre: libcfs: create libcfs_debug.h UAPI header

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/28089 Reviewed-by: Dmitry Eremin Reviewed-by: Olaf Weber

[PATCH 29/64 v2] staging: lustre: uapi: style cleanup of lustre_cfg.h

2017-08-19 Thread James Simmons
Some style and white space cleanups to make lustre_cfg.h easy to read. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Signed-off-by

[PATCH 35/64 v2] staging: lustre: lnet: delete lnet.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/28089 Reviewed-by: Dmitry Eremin Rev

[PATCH 24/64 v2] staging: lustre: obdclass: no need to check for kfree

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Re

[PATCH 37/64 v2] staging: lustre: libcfs: sort headers in libcfs.h

2017-08-19 Thread James Simmons
Move all the included headers in libcfs.h to the top of the file. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/28089 Reviewed-by: Dmitry Eremin Reviewed-by: Olaf Weber Reviewed-by: Oleg Drokin Signed-off-by: Jam

[PATCH 32/64 v2] staging: lustre: libcfs: remove LOGL and LOGU macros

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/22138 Reviewed-by: Dmitry Eremin Reviewed-by: Jinshan Xiong Re

[PATCH 36/64 v2] staging: lustre: lnet: migrate headers to lnet uapi directory

2017-08-19 Thread James Simmons
Migrate the headers used by user land and kernel space to the libcfs/lnet uapi directory. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/28089 Reviewed-by: Dmitry Eremin Reviewed-by: Olaf Weber Reviewed-by: Oleg Dr

[PATCH 34/64 v2] staging: lustre: socklnd: create socklnd.h UAPI header

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: https://review.whamcloud.com/28089 Reviewed-by: Dmitry Eremin Reviewed-by: Olaf Weber

[PATCH 39/64 v2] staging: lustre: lnet: remove BIT macro from lnetctl.h

2017-08-19 Thread James Simmons
Now that lnetctl.h is a UAPI header the BIT macro has to be removed. Signed-off-by: James Simmons --- drivers/staging/lustre/include/uapi/linux/lnet/lnetctl.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/include/uapi/linux/lnet/lnetctl.h b/d

[PATCH 25/64 v2] staging: lustre: uapi: move lustre_cfg.h to uapi directory

2017-08-19 Thread James Simmons
Move lustre_cfg.h to its proper place. Adjust additonal headers included in lustre_cfg.h. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans Reviewed-by: Oleg Dr

[PATCH 07/64 v2] staging: lustre: uapi: remove userland version of obd_ioctl_*()

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamc

[PATCH 11/64 v2] staging: lustre: uapi: move obd_ioctl_is_invalid() to linux-module.c

2017-08-19 Thread James Simmons
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 Intel-bug-id: http

[PATCH 15/64 v2] staging: lustre: uapi: label lustre_ioctl.h as a UAPI header

2017-08-19 Thread James Simmons
Change LUSTRE_IOCTL_H_ to _UAPI_LUSTRE_IOCTL_H_ now that is a proper UAPI header Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Sig

Re: [staging:staging-testing 306/323] drivers/staging/lustre/lustre/ptlrpc/errno.c:29:44: fatal error: ../include/lustre/lustre_errno.h: No such file or directory

2017-08-19 Thread James Simmons
> On Sat, Aug 19, 2017 at 02:50:48AM +0100, James Simmons wrote: > > > > > On Fri, Aug 18, 2017 at 03:53:56PM -0700, Greg Kroah-Hartman wrote: > > > > On Sat, Aug 19, 2017 at 02:49:02AM +0800, kbuild test robot wrote: > > > > > tree: > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/gr

[PATCH 17/64 v2] staging: lustre: uapi: remove included headers out of lustre_param.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24325 Reviewed-by: Ben Evans Reviewed-by:

[PATCH 12/64 v2] staging: lustre: uapi: move lustre_ioctl.h to uapi directory

2017-08-19 Thread James Simmons
Move the header lustre_ioctl.h to proper uapi directory. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Signed-off-by: James Simmon

[PATCH 13/64 v2] staging: lustre: uapi: use __ALIGN_KERNEL for lustre_ioctl.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewe

[PATCH 08/64 v2] staging: lustre: uapi: remove obd_ioctl_freedata() wrapper

2017-08-19 Thread James Simmons
Replace obd_ioctl_freedata() with direct kvfree() call. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Signed-off-by: James Simmons

[PATCH 05/64 v2] staging: lustre: uapi: move lu_fid, ost_id funcs out of lustre_idl.h

2017-08-19 Thread James Simmons
From: Ben Evans 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 a

[PATCH 03/64 v2] staging: lustre: uapi: return error code for ostid_set_id

2017-08-19 Thread James Simmons
From: Ben Evans 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

[PATCH 06/64 v2] staging: lustre: uapi: remove unused function in lustre_disk.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/bro

[PATCH 09/64 v2] staging: lustre: uapi: remove obd_ioctl_popdata() wrapper

2017-08-19 Thread James Simmons
Replace obd_ioctl_popdata() with direct copy_to_user() call. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Signed-off-by: James Si

[PATCH 04/64 v2] staging: lustre: uapi: update URL doc link in lustre_fid.h

2017-08-19 Thread James Simmons
From: Ben Evans The wiki has been revived so update the URL about the FID design in lustre_fid.h. Signed-off-by: Ben Evans Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/22712 Reviewed-on: https://review.whamcloud

[PATCH 22/64 v2] staging: lustre: uapi: don't memory allocate in UAPI header

2017-08-19 Thread James Simmons
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-of

[PATCH 21/64 v2] staging: lustre: uapi: remove lustre_cfg_free wrapper

2017-08-19 Thread James Simmons
Replace all lustre_cfg_free() calls with direct kfree() call. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/26966 Reviewed-by: Quentin Bouget Reviewed-by: Ben Evans Reviewed-by: Oleg Drokin Signed-off-by: James S

[PATCH 16/64 v2] staging: lustre: uapi: move kernel only prototypes out of lustre_param.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24325 Reviewed-by: Ben Evans Reviewed-by: John L.

[PATCH 18/64 v2] staging: lustre: uapi: move lustre_param.h to uapi directory

2017-08-19 Thread James Simmons
Move the header lustre_param.h to proper uapi directory. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24325 Reviewed-by: Ben Evans Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmo

[PATCH 01/64 v2] staging: lustre: uapi: Move functions out of lustre_idl.h

2017-08-19 Thread James Simmons
From: Ben Evans 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_stri

[PATCH 10/64 v2] staging: lustre: uapi: move obd_ioctl_getdata() declaration

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans

[PATCH 00/64 v2] staging: lustre: uapi: normalize the lustre headers

2017-08-19 Thread James Simmons
The headers for lustre/LNet for a long time lacked a clean separation in its internal headers which resulted in kernel specific data structures being exposed in user land code. This work unravels this mess and creates a clear separation between lustre kernel space and lustre user land. With this wo

[PATCH 14/64 v2] staging: lustre: uapi: cleanup headers for lustre_ioctl.h

2017-08-19 Thread James Simmons
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 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24568 Reviewed-by: Andreas Dilger Reviewed-by: Ben Evans R

[PATCH 02/64 v2] staging: lustre: uapi: remove unused functions for lustre_fid.h

2017-08-19 Thread James Simmons
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 --- drivers/staging/lustre/lustre/include/lustre_fid.h | 29 -- 1 file changed, 29 deletions(-) diff --git a/drivers/staging/lu

[PATCH 19/64 v2] staging: lustre: uapi: label lustre_param.h as an uapi header

2017-08-19 Thread James Simmons
Change LUSTRE_PARAM_H_ to _UAPI_LUSTRE_PARAM_H_ now that is a proper UAPI header Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: https://review.whamcloud.com/24325 Reviewed-by: Ben Evans Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Si

[PATCH] Staging: fsl-dpaa2: ethernet: dpni.c: Fixed alignment to match open parenthesis.

2017-08-19 Thread Srishti Sharma
Fixed alignment so that it matched open parenthesis Signed-off-by: Srishti Sharma --- drivers/staging/fsl-dpaa2/ethernet/dpni.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpni.c b/drivers/staging/fsl-dpaa2/ethernet/dpni.c index 5b9d442

Re: [PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided.

2017-08-19 Thread Srishti Sharma
On Sun, Aug 20, 2017 at 5:37 AM, Greg KH wrote: > On Sun, Aug 20, 2017 at 04:01:25AM +0530, Srishti Sharma wrote: >> Fixed a check reported by checkpatch.pl to avoid multiple assignments in a >> single statement. > > Always wrap your changelog text at 72 columns please. > >> >> Signed-off-by: Sri

Re: [PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided.

2017-08-19 Thread Greg KH
On Sun, Aug 20, 2017 at 04:01:25AM +0530, Srishti Sharma wrote: > Fixed a check reported by checkpatch.pl to avoid multiple assignments in a > single statement. Always wrap your changelog text at 72 columns please. > > Signed-off-by: Srishti Sharma > --- > drivers/staging/fsl-dpaa2/ethernet/d

[PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided.

2017-08-19 Thread Srishti Sharma
Fixed a check reported by checkpatch.pl to avoid multiple assignments in a single statement. Signed-off-by: Srishti Sharma --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c b/d

Re: [PATCH 7/8] staging: r8822be: Add Makefiles and Kconfig for new driver

2017-08-19 Thread kbuild test robot
Hi Larry, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.13-rc5 next-20170817] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Larry-Finger/staging-r8822be-Ad

[PATCH] staging: fsl-mc: move bus driver out of staging

2017-08-19 Thread laurentiu.tudor
From: Stuart Yoder Move the source files out of staging into their final locations: -include files in drivers/staging/fsl-mc/include go to include/linux/fsl -irq-gic-v3-its-fsl-mc-msi.c goes to drivers/irqchip -source in drivers/staging/fsl-mc/bus goes to drivers/bus/fsl-mc -README.txt, p

RE: [PATCH 05/15] mei: make device_type const

2017-08-19 Thread Winkler, Tomas
> Subject: [PATCH 05/15] mei: make device_type const > > Make this const as it is only stored in the type field of a device structure, > which is const. > Done using Coccinelle. > > Signed-off-by: Bhumika Goyal > --- > drivers/misc/mei/bus.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion

[PATCH] Staging: greybus: vibrator.c: Fixed alignment to match open parenthesis.

2017-08-19 Thread Srishti Sharma
Fixed alignment so that it matched open parenthesis Signed-off-by: Srishti Sharma --- drivers/staging/greybus/vibrator.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/greybus/vibrator.c b/drivers/staging/greybus/vibrator.c index 77a2365..5cd8a50 100

[PATCH 14/15] staging: greybus: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/staging/greybus/gbphy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/gbphy.c b/drivers/stag

[PATCH 15/15] usb: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/usb/common/ulpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/common/ulpi.c b/drivers/usb/common/ulpi.

[PATCH 13/15] scsi: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/scsi/fcoe/fcoe_sysfs.c | 4 ++-- drivers/scsi/scsi_transport_iscsi.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-)

[PATCH 11/15] remoteproc: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/remoteproc/remoteproc_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/remoteproc/remoteproc_core.c b/d

[PATCH 12/15] s390/zcrypt: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/s390/crypto/ap_card.c | 2 +- drivers/s390/crypto/ap_queue.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/d

[PATCH 10/15] platform/x86: wmi: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/platform/x86/wmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/platform/x86/wmi.c b/drivers/plat

[PATCH 08/15] PCI: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/pci/endpoint/pci-epf-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/endpoint/pci-epf-core.c b

[PATCH 07/15] mux: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/mux/mux-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mux/mux-core.c b/drivers/mux/mux-core.c index 2

[PATCH 09/15] phy: tegra: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/phy/tegra/xusb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/

[PATCH 05/15] mei: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/misc/mei/bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index 4

[PATCH 04/15] [media] rc: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/rc/rc-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-ma

[PATCH 06/15] mtd: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/mtd/mtdcore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c index f872

[PATCH 03/15] [media] i2c: make device_type const

2017-08-19 Thread Bhumika Goyal
Make this const as it is only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/media/i2c/soc_camera/mt9t031.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/soc_camera/mt9t031.c

[PATCH 00/15] drivers: make device_type const

2017-08-19 Thread Bhumika Goyal
Make device_type const. Done using Coccinelle. Bhumika Goyal (15): EDAC: make device_type const drm: make device_type const [media] i2c: make device_type const [media] rc: make device_type const mei: make device_type const mtd: make device_type const mux: make device_type const PC

[PATCH 02/15] drm: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. --- drivers/gpu/drm/drm_sysfs.c | 2 +- drivers/gpu/drm/ttm/ttm_module.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_sysfs.c

[PATCH 01/15] EDAC: make device_type const

2017-08-19 Thread Bhumika Goyal
Make these const as they are only stored in the type field of a device structure, which is const. Done using Coccinelle. Signed-off-by: Bhumika Goyal --- drivers/edac/edac_mc_sysfs.c | 8 drivers/edac/i7core_edac.c | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git