[PATCH 15/34] staging: lustre: lnet: change lnet_peer_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_peer_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 15/34] staging: lustre: lnet: change lnet_peer_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_peer_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- .../staging/lustre

[PATCH 13/34] staging: lustre: lnet: change lnet_ni_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_ni_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 13/34] staging: lustre: lnet: change lnet_ni_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_ni_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- .../staging/lustre/include

[PATCH 12/34] staging: lustre: lnet: change lnd_t to proper structure

2017-02-26 Thread James Simmons
Change lnd_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 12/34] staging: lustre: lnet: change lnd_t to proper structure

2017-02-26 Thread James Simmons
Change lnd_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/include

[PATCH 10/34] staging: lustre: lnet: change lnet_libmd_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_libmd_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 11/34] staging: lustre: lnet: change lnet_test_peer_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_test_peer_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry

[PATCH 10/34] staging: lustre: lnet: change lnet_libmd_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_libmd_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- .../staging/lustre

[PATCH 11/34] staging: lustre: lnet: change lnet_test_peer_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_test_peer_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging

[PATCH 08/34] staging: lustre: lnet: change lnet_eq_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_eq_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 09/34] staging: lustre: lnet: change lnet_me_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_me_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 07/34] staging: lustre: lnet: change lnet_libhandle_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_libhandle_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry

[PATCH 08/34] staging: lustre: lnet: change lnet_eq_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_eq_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre

[PATCH 09/34] staging: lustre: lnet: change lnet_me_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_me_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre

[PATCH 07/34] staging: lustre: lnet: change lnet_libhandle_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_libhandle_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging

[PATCH 05/34] staging: lustre: lnet: remove generic lnet_handle_any_t

2017-02-26 Thread James Simmons
With the creation of unique handlers we can remove the barely used generic handlers. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> R

[PATCH 06/34] staging: lustre: lnet: change lnet_msg_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_msg_t from typedef to proper structure. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber <o...@sgi.com> Reviewed-by: Doug Oucharek <doug.s.oucha...@intel.com> Reviewed-by: Dmitry Eremin <dm

[PATCH 05/34] staging: lustre: lnet: remove generic lnet_handle_any_t

2017-02-26 Thread James Simmons
With the creation of unique handlers we can remove the barely used generic handlers. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James

[PATCH 06/34] staging: lustre: lnet: change lnet_msg_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_msg_t from typedef to proper structure. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed-by: Olaf Weber Reviewed-by: Doug Oucharek Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- .../staging/lustre

[PATCH 00/34] staging: lustre: lnet: remove most of typedefs from LNet headers

2017-02-26 Thread James Simmons
Remove the majority of typedefs from the LNet headers. Change them into structures or enums instead. Currently only lnet_nid_t are lnet_pid_t are left. Removed the rest of the typedefs in socklnd.h as well James Simmons (34): staging: lustre: socklnd: change UAPI typedefs to proper structure

[PATCH 00/34] staging: lustre: lnet: remove most of typedefs from LNet headers

2017-02-26 Thread James Simmons
Remove the majority of typedefs from the LNet headers. Change them into structures or enums instead. Currently only lnet_nid_t are lnet_pid_t are left. Removed the rest of the typedefs in socklnd.h as well James Simmons (34): staging: lustre: socklnd: change UAPI typedefs to proper structure

[PATCH 01/34] staging: lustre: socklnd: change UAPI typedefs to proper structure

2017-02-26 Thread James Simmons
The upstream kernel requires proper structures so convert all the UAPI typedefs in socklnd.h. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/18506 Reviewed-by: Dmitry Eremin <d

[PATCH 01/34] staging: lustre: socklnd: change UAPI typedefs to proper structure

2017-02-26 Thread James Simmons
The upstream kernel requires proper structures so convert all the UAPI typedefs in socklnd.h. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142 Reviewed-on: https://review.whamcloud.com/18506 Reviewed-by: Dmitry Eremin Reviewed-by: Doug Oucharek Reviewed

Re: [PATCH] staging: lustre: ko2iblnd: Adapt to the removal of ib_get_dma_mr()

2017-02-19 Thread James Simmons
n o2iblnd.c which caused > compiling to fail. > > Signed-off-by: Doug Oucharek <doug.s.oucha...@intel.com> > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9026 > Reviewed-on: https://review.whamcloud.com/#/c/24931/ > Reviewed-by: James Simmons <uja.o...@yahoo.com>

Re: [PATCH] staging: lustre: ko2iblnd: Adapt to the removal of ib_get_dma_mr()

2017-02-19 Thread James Simmons
n o2iblnd.c which caused > compiling to fail. > > Signed-off-by: Doug Oucharek > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9026 > Reviewed-on: https://review.whamcloud.com/#/c/24931/ > Reviewed-by: James Simmons Reviewed-by: James Simmons > Changelog: > v1) I

[PATCH 00/14] staging: lustre: missing fixes from lustre 2.8

2017-02-18 Thread James Simmons
export parameter Bobi Jam (2): staging: lustre: llite: lower message level for ll_setattr_raw() staging: lustre: llite: omit to update wire data James Simmons (1): staging: lustre: lprocfs: move lprocfs_stats_[un]lock to a source file Jinshan Xiong (4): staging: lustre: osc: remove

[PATCH 00/14] staging: lustre: missing fixes from lustre 2.8

2017-02-18 Thread James Simmons
export parameter Bobi Jam (2): staging: lustre: llite: lower message level for ll_setattr_raw() staging: lustre: llite: omit to update wire data James Simmons (1): staging: lustre: lprocfs: move lprocfs_stats_[un]lock to a source file Jinshan Xiong (4): staging: lustre: osc: remove

[PATCH 01/14] staging: lustre: llite: lower message level for ll_setattr_raw()

2017-02-18 Thread James Simmons
.@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/llite/llite_lib.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_li

[PATCH 01/14] staging: lustre: llite: lower message level for ll_setattr_raw()

2017-02-18 Thread James Simmons
in this case. Signed-off-by: Bobi Jam Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6817 Reviewed-on: http://review.whamcloud.com/15541 Reviewed-by: Andreas Dilger Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/llite_lib.c

[PATCH 03/14] staging: lustre: osc: remove obsolete asserts

2017-02-18 Thread James Simmons
viewed-on: http://review.whamcloud.com/16456 Reviewed-on: http://review.whamcloud.com/16727 Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com&

[PATCH 02/14] staging: lustre: llite: omit to update wire data

2017-02-18 Thread James Simmons
Reviewed-on: http://review.whamcloud.com/16462 Reviewed-by: Jinshan Xiong <jinshan.xi...@intel.com> Reviewed-by: Niu Yawei <yawei@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre

[PATCH 02/14] staging: lustre: llite: omit to update wire data

2017-02-18 Thread James Simmons
viewed-by: Jinshan Xiong Reviewed-by: Niu Yawei Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/llite_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_lib.c b/drivers/staging/lustre/

[PATCH 03/14] staging: lustre: osc: remove obsolete asserts

2017-02-18 Thread James Simmons
ttp://review.whamcloud.com/16727 Reviewed-by: Bobi Jam Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/osc/osc_cache.c | 1 - drivers/staging/lustre/lustre/osc/osc_object.c | 4 2 files changed, 5 deletions(-) d

[PATCH 07/14] staging: lustre: lov: trying smaller memory allocations

2017-02-18 Thread James Simmons
Bob Glossman <bob.gloss...@intel.com> Reviewed-by: Jian Yu <jian...@intel.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lust

[PATCH 07/14] staging: lustre: lov: trying smaller memory allocations

2017-02-18 Thread James Simmons
Dilger Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/cl_object.h | 6 ++-- drivers/staging/lustre/lustre/llite/file.c | 16 - drivers/staging/lustre/lustre/llite/glimpse.c | 4 +-- drivers/staging/lustre/lustre/llite

[PATCH 10/14] staging: lustre: ldlm: disconnect speedup

2017-02-18 Thread James Simmons
y: Alexey Lyashkov <alexey_lyash...@xyratex.com> Reviewed-on: http://review.whamcloud.com/5843 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <

[PATCH 10/14] staging: lustre: ldlm: disconnect speedup

2017-02-18 Thread James Simmons
. remove it from export list. Signed-off-by: Vitaly Fertman Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3031 Xyratex-bug-id: MRP-395 MRP-1366 MRP-1366 Reviewed-by: Andriy Skulysh Reviewed-by: Alexey Lyashkov Reviewed-on: http://review.whamcloud.com/5843 Reviewed-by: James Simmons Reviewed

[PATCH 13/14] staging: lustre: llog: limit file size of plain logs

2017-02-18 Thread James Simmons
/18028 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: wangdi <di.w...@intel.com> Reviewed-by: Mike Pershin <mike.pers...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drive

[PATCH 12/14] staging: lustre: llog: change lgh_hdr_lock to mutex

2017-02-18 Thread James Simmons
wed-on: http://review.whamcloud.com/15274 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Lai Siyao <lai.si...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/include/lustre_

[PATCH 13/14] staging: lustre: llog: limit file size of plain logs

2017-02-18 Thread James Simmons
wed-by: Mike Pershin Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/obdclass/llog.c | 16 1 file changed, 16 insertions(+) diff --git a/drivers/staging/lustre/lustre/obdclass/llog.c b/drivers/staging/lustre/lustre/obdclass/llog.c index 83c5

[PATCH 12/14] staging: lustre: llog: change lgh_hdr_lock to mutex

2017-02-18 Thread James Simmons
Simmons Reviewed-by: Lai Siyao Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/lustre_log.h | 2 +- drivers/staging/lustre/lustre/obdclass/llog.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 08/14] staging: lustre: llite: remove extraneous export parameter

2017-02-18 Thread James Simmons
viewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/llite/file.c | 33 +- 1 file changed, 14 insertions(+), 19 deletions(-)

[PATCH 08/14] staging: lustre: llite: remove extraneous export parameter

2017-02-18 Thread James Simmons
extracted from "inode" directly as needed. Signed-off-by: Andreas Dilger Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6627 Reviewed-on: http://review.whamcloud.com/14953 Reviewed-by: Frank Zago Reviewed-by: Bobi Jam Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- dri

[PATCH 09/14] staging: lustre: ldlm: reduce ldlm pool recalc window

2017-02-18 Thread James Simmons
MRP-395 MRP-1366 MRP-1366 Reviewed-by: Andriy Skulysh <andriy_skul...@xyratex.com> Reviewed-by: Alexey Lyashkov <alexey_lyash...@xyratex.com> Reviewed-on: http://review.whamcloud.com/5843 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Andreas Dilger <andreas.di

[PATCH 09/14] staging: lustre: ldlm: reduce ldlm pool recalc window

2017-02-18 Thread James Simmons
-by: Alexey Lyashkov Reviewed-on: http://review.whamcloud.com/5843 Reviewed-by: James Simmons Reviewed-by: Andreas Dilger Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 11/14] staging: lustre: ldlm: fix race of starting bl threads

2017-02-18 Thread James Simmons
<yawei@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7330 Reviewed-on: http://review.whamcloud.com/17026 Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org&g

[PATCH 11/14] staging: lustre: ldlm: fix race of starting bl threads

2017-02-18 Thread James Simmons
://jira.hpdd.intel.com/browse/LU-7330 Reviewed-on: http://review.whamcloud.com/17026 Reviewed-by: Bobi Jam Reviewed-by: Andreas Dilger Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 49 ++--- 1 file changed, 28 insertions(+), 21 deletions(-) diff

[PATCH 14/14] staging: lustre: lprocfs: move lprocfs_stats_[un]lock to a source file

2017-02-18 Thread James Simmons
to the functions lprocfs_stats_[un]lock being inline functions in a header file. Lets untangle this mess and turn those inline functions into real functions in a source file. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8836 Re

[PATCH 05/14] staging: lustre: ldlm: handle ldlm lock cancel race when evicting client.

2017-02-18 Thread James Simmons
ed-off-by: Jinshan Xiong <jinshan.xi...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6271 Reviewed-on: http://review.whamcloud.com/16456 Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: James Simmons <u

[PATCH 06/14] staging: lustre: osc: further LRU OSC cleanup after eviction

2017-02-18 Thread James Simmons
271 Reviewed-on: http://review.whamcloud.com/16456 Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: J

[PATCH 14/14] staging: lustre: lprocfs: move lprocfs_stats_[un]lock to a source file

2017-02-18 Thread James Simmons
to the functions lprocfs_stats_[un]lock being inline functions in a header file. Lets untangle this mess and turn those inline functions into real functions in a source file. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8836 Reviewed-on: https

[PATCH 05/14] staging: lustre: ldlm: handle ldlm lock cancel race when evicting client.

2017-02-18 Thread James Simmons
-id: https://jira.hpdd.intel.com/browse/LU-6271 Reviewed-on: http://review.whamcloud.com/16456 Reviewed-by: Bobi Jam Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/cl_object.h | 7

[PATCH 06/14] staging: lustre: osc: further LRU OSC cleanup after eviction

2017-02-18 Thread James Simmons
-by: Bobi Jam Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- .../staging/lustre/lustre/osc/osc_cl_internal.h| 4 +- drivers/staging/lustre/lustre/osc/osc_internal.h | 3 +- drivers/staging/lustre/lustre/osc/osc_io.c

[PATCH 04/14] staging: lustre: lov: cleanup when cl_io_iter_init() fails

2017-02-18 Thread James Simmons
owse/LU-6271 Reviewed-on: http://review.whamcloud.com/16456 Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: J

[PATCH 04/14] staging: lustre: lov: cleanup when cl_io_iter_init() fails

2017-02-18 Thread James Simmons
-by: Bobi Jam Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/lov/lov_io.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/staging/lustre/lustre

Re: [PATCH v4 0/2] staging: lustre: align else if statement to coding standard

2017-02-11 Thread James Simmons
> Align else if statement to normal kernel style > > Maksymilian Piechota (2): > staging: lustre: move logical continuations at the end of line > staging: lustre: move else if statement to a single line Its not lustre :-) > drivers/staging/wlan-ng/prism2mgmt.c | 6 ++ > 1 file

Re: [PATCH v4 0/2] staging: lustre: align else if statement to coding standard

2017-02-11 Thread James Simmons
> Align else if statement to normal kernel style > > Maksymilian Piechota (2): > staging: lustre: move logical continuations at the end of line > staging: lustre: move else if statement to a single line Its not lustre :-) > drivers/staging/wlan-ng/prism2mgmt.c | 6 ++ > 1 file

[PATCH 2/3] staging: lustre: llite: root inode checking for migration

2017-02-11 Thread James Simmons
Hammond <john.hamm...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/llite/file.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a

[PATCH 3/3] staging: lustre: llite: check reply status in ll_migrate()

2017-02-11 Thread James Simmons
ed-on: https://review.whamcloud.com/23666 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Lai Siyao <lai.si...@intel.com> Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm

[PATCH 2/3] staging: lustre: llite: root inode checking for migration

2017-02-11 Thread James Simmons
From: wang di Do not migrate root inode. Signed-off-by: wang di Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7577 Reviewed-on: http://review.whamcloud.com/17669 Reviewed-by: Lai Siyao Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers

[PATCH 3/3] staging: lustre: llite: check reply status in ll_migrate()

2017-02-11 Thread James Simmons
Dilger Reviewed-by: Lai Siyao Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/file.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/file.c b

[PATCH 1/3] staging: lustre: llite: check request != NULL in ll_migrate

2017-02-11 Thread James Simmons
d <john.hamm...@intel.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/llite/file.c | 41 +- 1 file changed, 23 insertions(+), 18 deletions(-) diff

[PATCH 1/3] staging: lustre: llite: check request != NULL in ll_migrate

2017-02-11 Thread James Simmons
-by: James Simmons --- drivers/staging/lustre/lustre/llite/file.c | 41 +- 1 file changed, 23 insertions(+), 18 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/file.c b/drivers/staging/lustre/lustre/llite/file.c index 9870901..0c83bd7 100644 --- a/drivers/staging

[PATCH 0/3] staging: lustre: llite: fixups for ll_migrate()

2017-02-11 Thread James Simmons
This patch series covers all the current fixes for the function ll_migrate(). Originally only the first patch was submitted but it was pointed out other flaws existed in the code. Two more patches that cover those flaws are included in this patch set. These patches are order dependent. Niu Yawei

[PATCH 0/3] staging: lustre: llite: fixups for ll_migrate()

2017-02-11 Thread James Simmons
This patch series covers all the current fixes for the function ll_migrate(). Originally only the first patch was submitted but it was pointed out other flaws existed in the code. Two more patches that cover those flaws are included in this patch set. These patches are order dependent. Niu Yawei

Re: [PATCH 05/60] staging: lustre: llite: check request != NULL in ll_migrate

2017-02-11 Thread James Simmons
> On Sat, Jan 28, 2017 at 07:04:33PM -0500, James Simmons wrote: > > From: wang di <di.w...@intel.com> > > > > Check if the request is NULL, before retrieve reply body > > from the request. > > > > Signed-off-by: wang di <di.w...@intel.com> &

Re: [PATCH 05/60] staging: lustre: llite: check request != NULL in ll_migrate

2017-02-11 Thread James Simmons
> On Sat, Jan 28, 2017 at 07:04:33PM -0500, James Simmons wrote: > > From: wang di > > > > Check if the request is NULL, before retrieve reply body > > from the request. > > > > Signed-off-by: wang di > > Intel-bug-id: https://jira.hpdd.i

Re: [PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-30 Thread James Simmons
> It looks like what happened is there were two patches applied out of > sync. Let's add a fixes tag and CC the original author. So the only problem here is the commit message. I will update it then. > Fixes: ed2f549dc0f6 ("staging: lustre: libcfs: test if userland data is to > small") > >

Re: [PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-30 Thread James Simmons
> It looks like what happened is there were two patches applied out of > sync. Let's add a fixes tag and CC the original author. So the only problem here is the commit message. I will update it then. > Fixes: ed2f549dc0f6 ("staging: lustre: libcfs: test if userland data is to > small") > >

Re: [PATCH 13/60] staging: lustre: obdclass: health_check to report unhealthy upon LBUG

2017-01-30 Thread James Simmons
> Wat? > > I'm sorry but this patch makes no sense at all. > > On Sat, Jan 28, 2017 at 07:04:41PM -0500, James Simmons wrote: > > From: Bruno Faccini <bruno.facc...@intel.com> > > > > When a LBUG has occurred, without panic_on_lbug being set,

Re: [PATCH 13/60] staging: lustre: obdclass: health_check to report unhealthy upon LBUG

2017-01-30 Thread James Simmons
> Wat? > > I'm sorry but this patch makes no sense at all. > > On Sat, Jan 28, 2017 at 07:04:41PM -0500, James Simmons wrote: > > From: Bruno Faccini > > > > When a LBUG has occurred, without panic_on_lbug being set, > > health_check sysfs file must re

Re: [PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-30 Thread James Simmons
> On Mon, Jan 30, 2017 at 01:51:56PM +0300, Dan Carpenter wrote: > > The lstcon_ioctl_entry() function doesn't have enough size checking. > > Actually, the lstcon_ioctl_entry() would have been fine before we apply > this [patch 60/60]... As near as I can tell, no in kernel code is > negatively

Re: [PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-30 Thread James Simmons
> On Mon, Jan 30, 2017 at 01:51:56PM +0300, Dan Carpenter wrote: > > The lstcon_ioctl_entry() function doesn't have enough size checking. > > Actually, the lstcon_ioctl_entry() would have been fine before we apply > this [patch 60/60]... As near as I can tell, no in kernel code is > negatively

Re: [PATCH] staging: lustre: headers: potential UAPI headers

2017-01-29 Thread James Simmons
> > > > > On Mon, Dec 19, 2016 at 12:06:47PM -0500, James Simmons wrote: > > > > > > Not for landing. This is the purposed UAPI headers > > > > > > with the removal of unlikely and debugging macros. > > > > > > This is just f

Re: [PATCH] staging: lustre: headers: potential UAPI headers

2017-01-29 Thread James Simmons
> > > > > On Mon, Dec 19, 2016 at 12:06:47PM -0500, James Simmons wrote: > > > > > > Not for landing. This is the purposed UAPI headers > > > > > > with the removal of unlikely and debugging macros. > > > > > > This is just f

Re: [PATCH 0/8] staging: lustre: lnet: change wire protocol typedefs to proper structure

2017-01-29 Thread James Simmons
> On Sat, 2017-01-21 at 19:40 -0500, James Simmons wrote: > > The upstream kernel requires proper structures so > > convert nearly all the LNet wire protocols typedefs in > > the LNet core. > > Thanks. > > Perhaps s/\bWIRE_ATTR\b/__packed/g one day too I

Re: [PATCH 0/8] staging: lustre: lnet: change wire protocol typedefs to proper structure

2017-01-29 Thread James Simmons
> On Sat, 2017-01-21 at 19:40 -0500, James Simmons wrote: > > The upstream kernel requires proper structures so > > convert nearly all the LNet wire protocols typedefs in > > the LNet core. > > Thanks. > > Perhaps s/\bWIRE_ATTR\b/__packed/g one day too I

[PATCH 40/60] staging: ptlrpc: leaked rs on difficult reply

2017-01-28 Thread James Simmons
w.whamcloud.com/22696 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Lai Siyao <lai.si...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/ptlrpc/events.

[PATCH 40/60] staging: ptlrpc: leaked rs on difficult reply

2017-01-28 Thread James Simmons
ewed-by: Lai Siyao Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/ptlrpc/events.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc/events.c b/drivers/staging/lustre/lustre/ptlrpc/events.c index ae165

[PATCH 47/60] staging: lustre: mdc: avoid returning freed request

2017-01-28 Thread James Simmons
tel.com/browse/LU-8811 Reviewed-on: https://review.whamcloud.com/23843 Reviewed-by: Patrick Farrell <p...@cray.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/mdc/mdc_request.c | 1 + 1

[PATCH 47/60] staging: lustre: mdc: avoid returning freed request

2017-01-28 Thread James Simmons
.com/23843 Reviewed-by: Patrick Farrell Reviewed-by: Andreas Dilger Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/mdc/mdc_request.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/lustre/lustre/mdc/mdc_request.c b/drivers/staging/lustre/lustre/mdc/mdc_reque

[PATCH 46/60] staging: lustre: mdc: Make IT_OPEN take lookup bits lock

2017-01-28 Thread James Simmons
Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/mdc/mdc_locks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/mdc/mdc_locks.c b/drivers/staging/lustre

[PATCH 46/60] staging: lustre: mdc: Make IT_OPEN take lookup bits lock

2017-01-28 Thread James Simmons
Reviewed-on: https://review.whamcloud.com/23797 Fixes: 70a251f68dea ("staging: lustre: obd: decruft md_enqueue() and md_intent_lock()" Reviewed-by: John L. Hammond Reviewed-by: Lai Siyao Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/mdc/mdc_l

[PATCH 38/60] staging: lustre: llite: Adding timed wait in ll_umount_begin

2017-01-28 Thread James Simmons
e/LU-1882 Seagate-bug-id: MRP-1192 Reviewed-on: http://review.whamcloud.com/20061 Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Lai Siyao <lai.si...@intel.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.

[PATCH 38/60] staging: lustre: llite: Adding timed wait in ll_umount_begin

2017-01-28 Thread James Simmons
Siyao Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/llite_internal.h | 1 + drivers/staging/lustre/lustre/llite/llite_lib.c | 12 ++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH 44/60] staging: lustre: libcfs: fix error messages

2017-01-28 Thread James Simmons
hamcloud.com/23307 Reviewed-by: Patrick Farrell <p...@cray.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> --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd

[PATCH 44/60] staging: lustre: libcfs: fix error messages

2017-01-28 Thread James Simmons
-by: Doug Oucharek Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 2 +- drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 4 ++-- drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c | 5 +++-- drivers/staging/lustre/lnet

[PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-28 Thread James Simmons
The check for the smallest ioctl data in libcfs_ioctl_getdata() is incorrect. Instead of checking against struct libcfs_ioctl_data compare the size to struct libcfs_ioctl_hdr. Reported-by: Doug Oucharek <doug.s.oucha...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org>

[PATCH 01/60] staging: lustre: llite: Remove access of stripe in ll_setattr_raw

2017-01-28 Thread James Simmons
solves the problem in lov_io_init_released(). Signed-off-by: Jinshan Xiong <jinshan.xi...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5823 Reviewed-on: http://review.whamcloud.com/13514 Reviewed-by: James Simmons <uja.o...@yahoo.com> Reviewed-by: Henri Doreau <henri.do

[PATCH 60/60] staging: lustre: libcfs: fix minimum size check for libcfs ioctl

2017-01-28 Thread James Simmons
The check for the smallest ioctl data in libcfs_ioctl_getdata() is incorrect. Instead of checking against struct libcfs_ioctl_data compare the size to struct libcfs_ioctl_hdr. Reported-by: Doug Oucharek Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/libcfs/linux/linux-module.c

[PATCH 01/60] staging: lustre: llite: Remove access of stripe in ll_setattr_raw

2017-01-28 Thread James Simmons
in lov_io_init_released(). Signed-off-by: Jinshan Xiong Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5823 Reviewed-on: http://review.whamcloud.com/13514 Reviewed-by: James Simmons Reviewed-by: Henri Doreau Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers

[PATCH 58/60] staging: lustre: osc: avoid 64 divide in osc_cache_too_much

2017-01-28 Thread James Simmons
The use of 64 bit time introduces an expensive 64 bit division operation. Since the time lapse being calculated in osc_cache_too_much will never be more than seventy years we can cast the time lapse to an long and perform a normal 32 bit divison operation instead. Signed-off-by: James Simmons

[PATCH 59/60] staging: lustre: ptlrpc : remove userland usage from ptlrpc

2017-01-28 Thread James Simmons
. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8945 Reviewed-on: https://review.whamcloud.com/24396 Reviewed-by: Dmitry Eremin <dmitry.ere...@intel.com> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> Reviewed-by: Ole

[PATCH 58/60] staging: lustre: osc: avoid 64 divide in osc_cache_too_much

2017-01-28 Thread James Simmons
The use of 64 bit time introduces an expensive 64 bit division operation. Since the time lapse being calculated in osc_cache_too_much will never be more than seventy years we can cast the time lapse to an long and perform a normal 32 bit divison operation instead. Signed-off-by: James Simmons

[PATCH 59/60] staging: lustre: ptlrpc : remove userland usage from ptlrpc

2017-01-28 Thread James Simmons
. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8945 Reviewed-on: https://review.whamcloud.com/24396 Reviewed-by: Dmitry Eremin Reviewed-by: Andreas Dilger Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include

[PATCH 57/60] staging: lustre: lmv: remove nlink check in lmv_revalidate_slaves

2017-01-28 Thread James Simmons
broadcast back from the server will be reported as one. The return value of 1 is not invalid so lets remove the check. Signed-off-by: wang di <di.w...@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6984 Reviewed-on: http://review.whamcloud.com/16490 Reviewed-by: James

[PATCH 57/60] staging: lustre: lmv: remove nlink check in lmv_revalidate_slaves

2017-01-28 Thread James Simmons
rom the server will be reported as one. The return value of 1 is not invalid so lets remove the check. Signed-off-by: wang di Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6984 Reviewed-on: http://review.whamcloud.com/16490 Reviewed-by: James Simmons Reviewed-by: Jian Yu Reviewed-by: Oleg Dro

[PATCH 56/60] staging: lustre: llite: don't invoke direct_IO for the EOF case

2017-01-28 Thread James Simmons
ed-on: https://review.whamcloud.com/24552 Reviewed-by: Bob Glossman <bob.gloss...@intel.com> Reviewed-by: Bobi Jam <bobi...@hotmail.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/llit

[PATCH 56/60] staging: lustre: llite: don't invoke direct_IO for the EOF case

2017-01-28 Thread James Simmons
Glossman Reviewed-by: Bobi Jam Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/rw26.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/lustre/lustre/llite/rw26.c b/drivers/staging/lustre/lustre/llite/rw26.c index 21e06e5..d89e795

<    4   5   6   7   8   9   10   11   12   13   >