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,
> 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
> 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
> 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>
> --
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
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
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
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
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
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
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
> > 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,
>
-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 -
.
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.
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
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
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-
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
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
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
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
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-
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
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
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:
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:
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
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
: 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
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
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
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
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 --
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
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
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
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-
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...@
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
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..
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
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
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
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:
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
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
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:
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
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
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:
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
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:
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
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
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
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
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
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
.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
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
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
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
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
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,
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
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
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,
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
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
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:
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
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
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 &
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:
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
: 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..
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
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
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
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
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
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
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
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
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(-)
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
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
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
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
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
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
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
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
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
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
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
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
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:
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
):
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
201 - 300 of 1637 matches
Mail list logo