[PATCH 33/40] staging: lustre: remove messages from lazy portal on NI shutdown

2015-11-20 Thread James Simmons
From: Amir Shehata When shutting down an NI in a busy system, some messages received on this NI, might be on the lazy portal. They would have grabbed a ref count on the NI. Therefore NI will not be removed until messages are processed. In order to avoid this scenario,

Re: [PATCH 1/2] staging: lustre: lloop_device: Replace semaphore lo_sem with completion

2016-06-08 Thread James Simmons
> The semaphore 'lo_sem' in lloop_device is used as completion, so it > should be written as one. Semaphores are going away in the future. > > Signed-off-by: Binoy Jayan NAK. The lloop_device is about to get deleted. > --- > drivers/staging/lustre/lustre/llite/lloop.c

Re: [PATCH 0/2] *** staging: lustre: Replace semaphore lock with mutex ***

2016-06-08 Thread James Simmons
> Hi, > > These are a set of patches which removes semaphores from: > > drivers/staging/lustre (lnet) > > These are part of a bigger effort to eliminate all semaphores > from the linux kernel. > > They build correctly (individually and as a whole). > > Thanks, > Binoy I just finishing

Re: [PATCH 2/2] staging: lustre: lnet: Replace semaphore ln_rc_signal with completion

2016-06-08 Thread James Simmons
> The semaphore ln_rc_signal is used as completion, so convert it to > struct completion. Semaphores are going away in the future. > > Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> No problems in testing. Acked-by: James Simmons <jsimm...@infradead.org> > --

[PATCH] staging: lustre: llite: remove lloop device

2016-06-08 Thread James Simmons
supports direct I/O. Since this is the case we can delete this device. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/Kconfig|6 - drivers/staging/lustre/lustre/llite/Makefile |3 - drivers/staging/lustre/lustre/llite/lloop.c

[PATCH] staging: lustre: o2iblnd: remove typedefs

2016-06-10 Thread James Simmons
Remove all remaining typedefs in o2iblnd driver. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 339 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h| 299 .../staging/lustre/lnet

[PATCH] staging: lustre: socklnd: remove typedefs

2016-06-10 Thread James Simmons
Remove all remaining typedefs in socklnd driver. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/socklnd/socklnd.c| 302 ++-- .../staging/lustre/lnet/klnds/socklnd/socklnd.h| 209 +++--- .../staging/lustre/lnet

[PATCH 2/3] staging: lustre: lnet: Allocate MEs and small MDs in own kmem_caches

2016-06-09 Thread James Simmons
wed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/include/linux/lnet/lib-lnet.h | 36 ++-- drivers/staging/lustre/lnet/lnet/api-ni.c | 4

[PATCH 0/3] staging: lustre: lnet: bug fixs for 4.7-rc2

2016-06-09 Thread James Simmons
Here are the latest fixes for LNet. One fix covers a bug for the infiniband driver that happens when the o2iblnd intereface is pinged before it is finishing setting up. The next patch set from Bruno adds kmem_caches for the LNet layer. Bruno Faccini (2): staging: lustre: lnet: Allocate MEs and

[PATCH 1/3] staging: lustre: lnet: Don't access NULL NI on failure path

2016-06-09 Thread James Simmons
ensures we don't dereference a NULL NI on that failure path. Signed-off-by: Doug Oucharek <doug.s.oucha...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8022 Reviewed-on: http://review.whamcloud.com/19614 Reviewed-by: Dmitry Eremin <dmitry.ere...@intel.com> Reviewed-by: James

[PATCH 3/3] staging: lustre: lnet: optimize memory foot print for lnet_libmd

2016-06-09 Thread James Simmons
ud.com/18586 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/include/linux/lnet/lib-ty

Re: [PATCH 2/3] staging: lustre: lnet: Allocate MEs and small MDs in own kmem_caches

2016-06-14 Thread James Simmons
> > This may also possibly help to save cycles due to high usage and > > contention when using a generic kmem_cache (when they stay separate > > from others, thanks for the precision!). > > Have you measured this? > > This isn't applicable for 4.7-rc at this time, _unless_ it fixes a bug, >

[PATCH] staging: lustre: lnet: Remove old commented out code

2016-06-22 Thread James Simmons
-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/socklnd/socklnd.c|5 - .

[PATCH] staging: lustre: remove remote client support

2016-06-19 Thread James Simmons
ug-id: https://jira.hpdd.intel.com/browse/LU-6971 Reviewed-on: http://review.whamcloud.com/19789 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Lai Siyao <lai.si...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.

[PATCH 00/11] Massive style cleanup for LNet layer

2016-02-12 Thread James Simmons
is required for the linux kernel. James Simmons (11): staging: lustre: drop *_t from end of struct lnet_text_buf staging: lustre: format properly all comment blocks for LNet core staging: lustre: align all code properly for LNet core staging: lustre: remove unnecessary parentheses around LNet

[PATCH 03/11] staging: lustre: align all code properly for LNet core

2016-02-12 Thread James Simmons
In several places in the LNet core the code doesn't align up properly. This resolves those checkpath issues. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 49 +-- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h

[PATCH 04/11] staging: lustre: remove unnecessary parentheses around LNet function pointer

2016-02-12 Thread James Simmons
No need for the parentheses around any function pointer. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |2 +- drivers/staging/lustre/lnet/lnet/api-ni.c |4 ++-- drivers/staging/lustre/lnet/lnet/lib-

[PATCH 08/11] staging: lustre: remove space in LNet function declarations

2016-02-12 Thread James Simmons
Several function declarations have spacing in them. Lets remove all those instances reported by checkpatch.pl. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h|2 +- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c

[PATCH 09/11] staging: lustre: balance braces properly in LNet layer

2016-02-12 Thread James Simmons
Properly balance the braces done wrong as reported by checkpatch.pl. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c |3 ++- .../lustre/lnet/klnds/socklnd/socklnd_lib.c|3 ++- .../lustre/lnet/klnds/s

[PATCH 05/11] staging: lustre: remove unnecessary blank lines reported by checkpatch.pl

2016-02-12 Thread James Simmons
Remove any useless blank lines reported by checkpatch.pl for LNet layer. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/include/linux/lnet/nidstr.h |3 +++ .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c|6 -- .../staging/lustre/lnet

[PATCH 06/11] staging: lustre: add missing spaces for LNet layer reported by checkpatch.pl

2016-02-12 Thread James Simmons
Add missing spaces in the code reported by checkpatch.pl. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/include/linux/lnet/lib-types.h |2 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c|2 +- .../staging/lustre/lnet/klnds/o2iblnd/o2i

[PATCH 02/11] staging: lustre: format properly all comment blocks for LNet core

2016-02-12 Thread James Simmons
In several places in the LNet core comment blocks don't follow the linux kernel style. This patch cleans those problems up. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/include/linux/lnet/api.h| 22 ++- .../staging/lustre/include/linux/lnet/lib-

[PATCH 01/11] staging: lustre: drop *_t from end of struct lnet_text_buf

2016-02-12 Thread James Simmons
When lnet_text_buf data structure was transform from typedef to struct the *_t which is typical of typedef was not drop. This patch removes the *_t to be consistent. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/lnet/config.c

[PATCH 07/11] staging: lustre: don't set more than one variable per line in LNet layer

2016-02-12 Thread James Simmons
Cleanup all occurances of more than one variable being set per line as reported by checkpatch.pl. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/include/linux/lnet/lib-lnet.h |3 ++- .../staging/lustre/include/linux/lnet/socklnd.h|3 ++- .../s

[PATCH 5/6] staging: lustre: make module_init/exit naming consistent

2016-02-23 Thread James Simmons
From: Andreas Dilger Make the name of the module_init()/_exit() functions consistently {module_name}_init and {module_name}_exit. Signed-off-by: Andreas Dilger Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on:

[PATCH 2/6] staging: lustre: add missing MODULE_AUTHOR for LNet selftest module

2016-02-23 Thread James Simmons
From: Andreas Dilger For LNet selftest module the MODULE_AUTHOR was missing. Add proper OpenSFS authorship. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger Intel-bug-id:

[PATCH 1/6] staging: lustre: move module info to end of libcfs module.c file

2016-02-23 Thread James Simmons
Move the MODULE_* field in module.c that belongs to libcfs to the end of the file like it is done for other kernel drivers. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/libcfs/module.c |7 +++ 1 files changed, 3 insertions(+), 4 del

[PATCH 3/6] staging: lustre: update the MODULE_VERSION for all lustre modules

2016-02-23 Thread James Simmons
For several lustre modules the MODULE_VERSION has the wrong value, located in the wrong place in the source code, or completely missing. This patch brings it up to date. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on

[PATCH 0/6] staging: lustre: update modinfo data

2016-02-23 Thread James Simmons
: update the MODULE_DESCRIPTION for all lustre modules staging: lustre: make module_init/exit naming consistent staging: lustre: update format for lnet_lib_init/exit comment James Simmons (2): staging: lustre: move module info to end of libcfs module.c file staging: lustre: update

[PATCH 4/6] staging: lustre: update the MODULE_DESCRIPTION for all lustre modules

2016-02-23 Thread James Simmons
From: Andreas Dilger Fixup the MODULE_DESCRIPTION for several lustre modules. Some wrongly place the version in the string or they are not descriptive enough. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger

[PATCH 6/6] staging: lustre: update format for lnet_lib_init/exit comment

2016-02-23 Thread James Simmons
From: Andreas Dilger The documentation about the return values for lnet_lib_init and lnet_lib_exit was in the old style format. Bring it in sync with the rest of the LNet core. Broken out of patch 16787. Signed-off-by: Andreas Dilger

[PATCH 00/24] Second batch of LNet updates

2016-02-22 Thread James Simmons
Zago (4): staging: lustre: make local functions static for LNet ni staging: lustre: make some lnet functions static staging: lustre: missed a few cases of using NULL instead of 0 staging: lustre: remove unnecessary EXPORT_SYMBOL from lnet layer James Simmons (1): staging: lustre: use

[PATCH 01/24] staging: lustre: Dynamic LNet Configuration (DLC) IOCTL changes

2016-02-22 Thread James Simmons
wed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- .../lustre/include/linux/libcfs/libcfs_ioctl.h | 23 --

[PATCH 06/24] staging: lustre: remove LUSTRE_{,SRV_}LNET_PID

2016-02-22 Thread James Simmons
ug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/11985 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Bob Glossman <bob.gloss...@intel.com> Reviewed-by: Dmitry Eremin <dmitry.ere...@intel.com> Reviewed-by: Andreas Di

[PATCH 08/24] staging: lustre: return appropriate errno when adding route

2016-02-22 Thread James Simmons
m> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6045 Reviewed-on: http://review.whamcloud.com/13116 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Isaac Huang <he.hu...@intel.com> Reviewed-by: Oleg Drokin <oleg

[PATCH 03/24] staging: lustre: fix crash due to NULL networks string

2016-02-22 Thread James Simmons
From: Amir Shehata If there is an invalid networks or ip2nets lnet_parse_networks() gets called with a NULL 'network' string parameter lnet_parse_networks() needs to sanitize its input string now that it's being called from multiple places. Instead, check for a NULL

[PATCH 02/24] staging: lustre: Dynamic LNet Configuration (DLC) show command

2016-02-22 Thread James Simmons
Signed-off-by: Amir Shehata <amir.sheh...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2456 Reviewed-on: http://review.whamcloud.com/8022 Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-

[PATCH 16/24] staging: lustre: assume a kernel build

2016-02-22 Thread James Simmons
John L. Hammond <john.hamm...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/13121 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Amir Shehata <amir.sheh...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@

[PATCH 19/24] staging: lustre: remove unnecessary EXPORT_SYMBOL from lnet layer

2016-02-22 Thread James Simmons
320 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Isaac Huang <he.hu...@intel.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- drivers/staging/lustre/lnet/lnet/lib-socket.c |3 --- 1 files c

[PATCH 24/24] staging: lustre: Remove LASSERTS from router checker

2016-02-22 Thread James Simmons
om> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7362 Reviewed-on: http://review.whamcloud.com/17003 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Chris Horn <ho...@cray.com> Reviewed-by: Matt Ezell <ezel...@ornl.gov> Reviewed-by: Oleg Drokin <oleg.dro..

[PATCH v3 0/6] staging: lustre: update modinfo data

2016-02-26 Thread James Simmons
module_init/exit naming consistent staging: lustre: update comment for lnet_lib_init/exit James Simmons (2): staging: lustre: move module info to end of libcfs module.c file staging: lustre: update the MODULE_VERSION for all lustre modules .../staging/lustre/include/linux/libcfs/libcfs.h

[PATCH v3 6/6] staging: lustre: update comment for lnet_lib_init/exit

2016-02-26 Thread James Simmons
From: Andreas Dilger The documentation about the return values for lnet_lib_init and lnet_lib_exit was in the old style format. Bring it in sync with the rest of the LNet core. Broken out of patch 16787. Signed-off-by: Andreas Dilger

[PATCH v3 1/6] staging: lustre: move module info to end of libcfs module.c file

2016-02-26 Thread James Simmons
Move the MODULE_* field in module.c that belongs to libcfs to the end of the file like it is done for other kernel drivers. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/libcfs/module.c |7 +++ 1 files changed, 3 insertions(+), 4 del

[PATCH v3 5/6] staging: lustre: make module_init/exit naming consistent

2016-02-26 Thread James Simmons
From: Andreas Dilger Make the name of the module_init()/_exit() functions consistently {module_name}_init and {module_name}_exit. Signed-off-by: Andreas Dilger Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on:

[PATCH v3 4/6] staging: lustre: update the MODULE_DESCRIPTION for all lustre modules

2016-02-26 Thread James Simmons
From: Andreas Dilger Fixup the MODULE_DESCRIPTION for several lustre modules. Some wrongly place the version in the string or they are not descriptive enough. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger

[PATCH v3 2/6] staging: lustre: add missing MODULE_AUTHOR for LNet selftest module

2016-02-26 Thread James Simmons
For several lustre modules the MODULE_VERSION has the wrong value, located in the wrong place in the source code, or completely missing. This patch brings it up to date. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on

[PATCH v3 2/6] staging: lustre: add missing MODULE_AUTHOR for LNet selftest module

2016-02-26 Thread James Simmons
From: Andreas Dilger For LNet selftest module the MODULE_AUTHOR was missing. Add proper OpenSFS authorship. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger Intel-bug-id:

[PATCH v2 3/6] staging: lustre: update the MODULE_VERSION for all lustre modules

2016-02-25 Thread James Simmons
For several lustre modules the MODULE_VERSION has the wrong value, located in the wrong place in the source code, or completely missing. This patch brings it up to date. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on

[PATCH v2 1/6] staging: lustre: move module info to end of libcfs module.c file

2016-02-25 Thread James Simmons
Move the MODULE_* field in module.c that belongs to libcfs to the end of the file like it is done for other kernel drivers. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/libcfs/module.c |7 +++ 1 files changed, 3 insertions(+), 4 del

[PATCH v2 2/6] staging: lustre: add missing MODULE_AUTHOR for LNet selftest module

2016-02-25 Thread James Simmons
From: Andreas Dilger For LNet selftest module the MODULE_AUTHOR was missing. Add proper OpenSFS authorship. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger Intel-bug-id:

[PATCH v2 6/6] staging: lustre: update comment for lnet_lib_init/exit

2016-02-25 Thread James Simmons
From: Andreas Dilger The documentation about the return values for lnet_lib_init and lnet_lib_exit was in the old style format. Bring it in sync with the rest of the LNet core. Broken out of patch 16787. Signed-off-by: Andreas Dilger

[PATCH v2 5/6] staging: lustre: make module_init/exit naming consistent

2016-02-25 Thread James Simmons
From: Andreas Dilger Make the name of the module_init()/_exit() functions consistently {module_name}_init and {module_name}_exit. Signed-off-by: Andreas Dilger Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6204 Reviewed-on:

[PATCH v2 0/6] staging: lustre: update modinfo data

2016-02-25 Thread James Simmons
Dilger (4): staging: lustre: add missing MODULE_AUTHOR for LNet selftest module staging: lustre: update the MODULE_DESCRIPTION for all lustre modules staging: lustre: make module_init/exit naming consistent staging: lustre: update comment for lnet_lib_init/exit James Simmons (2): staging

[PATCH v2 4/6] staging: lustre: update the MODULE_DESCRIPTION for all lustre modules

2016-02-25 Thread James Simmons
From: Andreas Dilger Fixup the MODULE_DESCRIPTION for several lustre modules. Some wrongly place the version in the string or they are not descriptive enough. Broken out of patch http://review.whamcloud.com/16787. Signed-off-by: Andreas Dilger

[PATCH 11/24] staging: lustre: startup lnet acceptor thread dynamically

2016-02-22 Thread James Simmons
From: Amir Shehata With DLC it's possible to start up a system with no NIs that require the acceptor thread, and thus it won't start. Later on the user can add an NI that requires the acceptor thread to start, it is then necessary to start it up. If the user removes a

[PATCH 14/24] staging: lustre: handle lnet_check_routes() errors

2016-02-22 Thread James Simmons
From: Amir Shehata After adding a route, lnet_check_routes() is called to ensure that the route added doesn't invalidate the routing configuration. If lnet_check_routes() fails then the route just added, which caused the current configuration to be invalidated is

[PATCH 21/24] staging: lustre: use sock.h in only acceptor.c

2016-02-22 Thread James Simmons
On some platforms having sock.h in lib-types.h would collide with other included header files being used in the LNet layer. Looking at what was needed from sock.h only acceptor.c is dependent on it. To avoid these issues we just use sock.h only in acceptor.c. Signed-off-by: James Simmons <js

[PATCH 09/24] staging: lustre: make some lnet functions static

2016-02-22 Thread James Simmons
outers' was not declared. Should it be static? Some prototypes were removed from C files and added to the proper header. Signed-off-by: Frank Zago <fz...@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5396 Reviewed-on: http://review.whamcloud.com/12206 Reviewed-by: J

[PATCH 15/24] staging: lustre: improvement to router checker

2016-02-22 Thread James Simmons
.s.oucha...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- .../staging/lustre/include/linux/lnet/lib-types.h |7 +++ drivers/staging/lustre/lnet/lnet/api-ni.c |1 + drivers/staging/lustr

[PATCH 17/24] staging: lustre: prevent assert on LNet module unload

2016-02-22 Thread James Simmons
count. Signed-off-by: Amir Shehata <amir.sheh...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6010 Reviewed-on: http://review.whamcloud.com/13110 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-b

[PATCH 04/24] staging: lustre: DLC user/kernel space glue code

2016-02-22 Thread James Simmons
ed-off-by: Amir Shehata <amir.sheh...@intel.com> ntel-bug-id: https://jira.hpdd.intel.com/browse/LU-2456 Reviewed-on: http://review.whamcloud.com/8023 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: John L. Hammond

[PATCH 07/24] staging: lustre: improve LNet clean up code and API

2016-02-22 Thread James Simmons
hehata <amir.sheh...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5734 Reviewed-on: http://review.whamcloud.com/12658 Reviewed-by: Liang Zhen <liang.z...@intel.com> Reviewed-by: Isaac Huang <he.hu...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Revi

[PATCH 22/24] staging: lustre: Allocate the correct number of rtr buffers

2016-02-22 Thread James Simmons
From: Amir Shehata This patch ensures that the correct number of router buffers are allocated. It keeps a count that keeps track of the number of buffers allocated. Another count keeps the number of buffers requested. The number of buffers allocated is set when creating

[PATCH 13/24] staging: lustre: return -EEXIST if NI is not unique

2016-02-22 Thread James Simmons
wed-on: http://review.whamcloud.com/13056 Reviewed-by: Isaac Huang <he.hu...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- drivers/staging/lustre/lnet/lnet/api-ni.c | 20 +--- 1 files changed,

[PATCH 05/24] staging: lustre: make local functions static for LNet ni

2016-02-22 Thread James Simmons
From: Frank Zago <fz...@cray.com> The function lnet_unprepare can be made static. Signed-off-by: Frank Zago <fz...@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5396 Reviewed-on: http://review.whamcloud.com/11306 Reviewed-by: James Simmons <uja.o...@gmai

[PATCH 10/24] staging: lustre: missed a few cases of using NULL instead of 0

2016-02-22 Thread James Simmons
eviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Bob Glossman <bob.gloss...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- drivers/staging/lustre/lnet/lnet/api-ni.c |2 +- drive

[PATCH 18/24] staging: lustre: remove messages from lazy portal on NI shutdown

2016-02-22 Thread James Simmons
From: Amir Shehata When shutting down an NI in a busy system, some messages received on this NI, might be on the lazy portal. They would have grabbed a ref count on the NI. Therefore NI will not be removed until messages are processed. In order to avoid this scenario,

[PATCH 23/24] staging: lustre: Use lnet_is_route_alive for router aliveness

2016-02-22 Thread James Simmons
ed-off-by: Chris Horn <ho...@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5733 Reviewed-on: http://review.whamcloud.com/14055 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Amir Shehata <amir.sheh...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro

[PATCH 12/24] staging: lustre: reject invalid net configuration for lnet

2016-02-22 Thread James Simmons
ud.com/12912 Reviewed-by: Isaac Huang <he.hu...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- drivers/staging/lustre/lnet/lnet/api-ni.c | 18 +- 1 files changed, 17 insertions(+), 1 deletions(-) diff

[PATCH 20/24] staging: lustre: avoid race during lnet acceptor thread termination

2016-02-22 Thread James Simmons
From: Bruno Faccini This patch will avoid potential race, around socket sleepers wait list, during acceptor thread termination and using sk_callback_lock RW-Lock protection. Signed-off-by: Bruno Faccini Intel-bug-id:

[PATCH 04/10] staging: lustre: eliminate obsolete Cray SeaStar support

2016-02-15 Thread James Simmons
Remove the bulk of code for the no longer supported SeaStar interconnect found on older Cray systems. Signed-off-by: James Simmons <uja.o...@gmail.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-1422 Reviewed-on: http://review.whamcloud.com/7469 Reviewed-by: Liang Zhen &l

[PATCH 05/10] staging: lustre: remove uses of IS_ERR_VALUE()

2016-02-15 Thread James Simmons
From: John L. Hammond Remove most uses of IS_ERR_VALUE(). This macro was often given an int argument coming from PTR_ERR(). This invokes implementation defined behavior since the long value gotten by applying PTR_ERR() to a kernel pointer will usually not be representable

[PATCH 06/10] staging: lustre: return +ve for blocked lnet message

2016-02-15 Thread James Simmons
EAGAIN Signed-off-by: Liang Zhen <liang.z...@intel.com> Signed-off-by: James Simmons <uja.o...@gmail.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5151 Reviewed-on: http://review.whamcloud.com/10625 Reviewed-by: Chris Horn <ho...@cray.com> Reviewed-by: Andreas Dilger &

[PATCH 03/10] staging: lustre: fix failure handle of create reply

2016-02-15 Thread James Simmons
From: Liang Zhen error handler of lnet_create_reply_msg() didn't release lnet_res_lock if lnet_msg_alloc() failed. It can be fixed by moving validation check of msg out from lock. Signed-off-by: Liang Zhen Intel-bug-id:

[PATCH 00/10] First 10 patches for LNet layer bug fixes

2016-02-15 Thread James Simmons
staging: lustre: DLC Feature dynamic net config Chris Horn (2): staging: lustre: rename variables in lnet_find_route_locked staging: lustre: reflect down routes in /proc/sys/lnet/routes Frank Zago (1): staging: lustre: do not memset after LIBCFS_ALLOC James Simmons (1): staging: lustre

[PATCH 09/10] staging: lustre: Dynamic LNet Configuration (DLC) dynamic routing

2016-02-15 Thread James Simmons
: Amir Shehata <amir.sheh...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2456 Change-Id: Id07d4ad424d8f5ba72475d4149380afe2ac54e77 Reviewed-on: http://review.whamcloud.com/9831 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Doug Oucharek <doug.s.oucha..

[PATCH 07/10] staging: lustre: do not memset after LIBCFS_ALLOC

2016-02-15 Thread James Simmons
012 Reviewed-by: Patrick Farrell <p...@cray.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Dmitry Eremin <dmitry.ere...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> --- .../staging/lus

[PATCH 02/10] staging: lustre: reflect down routes in /proc/sys/lnet/routes

2016-02-15 Thread James Simmons
From: Chris Horn We consider routes "down" if the router is down or the router NI for the target network is down. This should be reflected in the output of /proc/sys/lnet/routes Signed-off-by: Chris Horn Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3679

[PATCH 08/10] staging: lustre: Dynamic LNet Configuration (DLC)

2016-02-15 Thread James Simmons
om> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2456 Change-Id: I8c8bbf3b55acf4d76f22a8be587b553a70d31889 Reviewed-on: http://review.whamcloud.com/9830 Reviewed-by: Liang Zhen <liang.z...@intel.com> Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Oleg Droki

[PATCH 10/10] staging: lustre: DLC Feature dynamic net config

2016-02-15 Thread James Simmons
om> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2456 Reviewed-on: http://review.whamcloud.com/9832 Reviewed-by: James Simmons <uja.o...@gmail.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Liang Zhen <liang.z...@intel.com> Reviewed-by: Oleg

[PATCH 01/10] staging: lustre: rename variables in lnet_find_route_locked

2016-02-15 Thread James Simmons
From: Chris Horn Rename several variables in lnet_find_route_locked to make the code easier to understand. Broken out of patch 7857. Signed-off-by: Chris Horn Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3679 Reviewed-on: http://review.whamcloud.com/7857

[PATCH 07/10] staging: lustre: cleanup comment style for lnet selftest

2016-03-11 Thread James Simmons
Apply a consistent style for comments in the lnet selftest code. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c |8 ++-- drivers/staging/lustre/lnet/selftest/conctl.c| 50 +++--- drivers/staging/lustr

[PATCH 03/10] staging: lustre: fix spacing issues checkpatch reported in lnet selftest

2016-03-11 Thread James Simmons
Remove any extra spacing as reported by checkpatch. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c |6 +++--- drivers/staging/lustre/lnet/selftest/console.c |6 +++--- drivers/staging/lustre/lnet/selftest/selftest.h

[PATCH 01/10] staging: lustre: fix bogus lst errors for lnet selftest

2016-03-11 Thread James Simmons
From: Isaac Huang It should not be counted as errors if a test RPC has been stopped due to administrative actions, e.g. lst end_session from the remote test console. Signed-off-by: Isaac Huang Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4181

[PATCH 08/10] staging: lustre: test for proper errno code in lstcon_rpc_trans_abort

2016-03-11 Thread James Simmons
The error value returned will be -ETIMEDOUT not ETIMEDOUT. This fixes a typo that prevents us from handling the error case. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/conrpc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH 10/10] staging: lustre: fix aligments in lnet selftest

2016-03-11 Thread James Simmons
Some aligment issues were not caught by checkpatch. We address them here. Some of the alignment issues caused greater than 80 character checkpatch issues. Some changes were done to just make the code more readable and to match our production code. Signed-off-by: James Simmons <js

[PATCH 02/10] staging: lustre: remove returns in void function for lnet selftest

2016-03-11 Thread James Simmons
No reason to have returns at end of void function. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c |9 +++-- drivers/staging/lustre/lnet/selftest/conrpc.c|6 -- drivers/staging/lustre/lnet/selftest/framework.c

[PATCH 04/10] staging: lustre: remove extra spacing of variable declartions for lnet selftest

2016-03-11 Thread James Simmons
Remove any extra spacing such as "int rc" to "int rc" to match the proper kernel style Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c | 28 drivers/staging/lustre/lnet/s

[PATCH 00/10] Cleanup and audit of lnet selftest code

2016-03-11 Thread James Simmons
checkpatch issues were addressed. Isaac Huang (1): staging: lustre: fix bogus lst errors for lnet selftest James Simmons (9): staging: lustre: remove returns in void function for lnet selftest staging: lustre: fix spacing issues checkpatch reported in lnet selftest staging: lustre: remove extra

[PATCH 09/10] staging: lustre: report minimum of two buffers for LNet selftest load test

2016-03-11 Thread James Simmons
The minimum number reserve buffer for lnet selftest load test is two not one. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/framework.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/lustre/lnet/se

[PATCH 05/10] staging: lustre: remove extra spacing when setting variable for lnet selftest

2016-03-11 Thread James Simmons
Remove any extra spacing for the lines of code setting variables to some value. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c | 40 ++-- drivers/staging/lustre/lnet/selftest/conrpc.c| 68 ++-- d

[PATCH 06/10] staging: lustre: filter remaining extra spacing for lnet selftest

2016-03-11 Thread James Simmons
This patch is a result of a filter applied to the lnet selftest code to remove the last bits of hidden white spaces. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/brw_test.c |6 +- drivers/staging/lustre/lnet/selftest/conctl.c

[PATCH 0/4] Support lustre routers with stand alone LNet stack

2016-03-08 Thread James Simmons
the LNet stack for the routers which is all that is needed anyways. This patch series allows that option for users of the upstream client. James Simmons (4): staging: lustre: move libcfs to lnet layer staging: lustre: fixup kernel Kconfig option LNET_MAX_PAYLOAD staging: lustre: add help section

[PATCH 3/4] staging: lustre: add help section of Kconfig config LNET

2016-03-08 Thread James Simmons
Include a help section for Kconfig LNET. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/Kconfig |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/drivers/staging/lustre/lnet/Kconfig b/drivers/staging/lustre/lnet/Kconfig

[PATCH 1/4] staging: lustre: move libcfs to lnet layer

2016-03-08 Thread James Simmons
into LNet. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/Makefile |2 +- .../lustre/{lustre => lnet}/libcfs/Makefile|2 +- .../staging/lustre/{lustre => lnet}/libcfs/debug.c |0 .../staging/lustre/{lustre => lnet

[PATCH 4/4] staging: lustre: make lustre dependent on LNet

2016-03-08 Thread James Simmons
In the case of lustre routers you only need a functioning LNet stack. Especially since often the routers are very light weight and want to avoid any addition software that would create additional pressures on the system. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/s

[PATCH 2/4] staging: lustre: fixup kernel Kconfig option LNET_MAX_PAYLOAD

2016-03-08 Thread James Simmons
A few errors exist for the Kconfig option LNET_MAX_PAYLOAD. First mistake is the default size is 1MB not 2MB as it is shown to the person configuring the kernel. Second the LNET_MAX_PAYLOAD option is more closely related to LNET than the LUSTRE_FS option. Signed-off-by: James Simmons <js

[PATCH 1/6] staging: lustre: Correct missing newline for CERROR call in sfw_handle_server_rpc

2016-03-12 Thread James Simmons
From: James Nunez This is one of the fixes broken out of patch 1 that was missed in the merger. With this fix the CERROR called in sfw_handle_server_rpc will print out correctly. Signed-off-by: James Nunez Intel-bug-id:

[PATCH 3/6] staging: lustre: handle error returned from wait_event_timeout seltest timer

2016-03-12 Thread James Simmons
The function wait_event_timeout can fail and return an error. Handle this case in stt_timer_main(). Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lnet/selftest/timer.c | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/d

[PATCH 0/6] Last batch of lnet selftest cleanup

2016-03-12 Thread James Simmons
): staging: lustre: Correct missing newline for CERROR call in sfw_handle_server_rpc James Simmons (4): staging: lustre: handle error returned from wait_event_timeout seltest timer staging: lustre: remove excess blank lines in lnet selftest code staging: lustre: realign some code in lnet selftest so

<    1   2   3   4   5   6   7   8   9   10   >