[PATCH 18/20] staging: lustre: llite: Remove filtering of seclabel xattr

2017-07-26 Thread James Simmons
hamcloud.com/27292 Reviewed-by: John L. Hammond <john.hamm...@intel.com> Reviewed-by: Sebastien Buisson <sbuis...@ddn.com> Reviewed-by: Oleg Drokin <oleg.dro...@intel.com> Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lus

[PATCH 18/20] staging: lustre: llite: Remove filtering of seclabel xattr

2017-07-26 Thread James Simmons
Buisson Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/file.c | 58 ++ .../staging/lustre/lustre/llite/llite_internal.h | 4 ++ drivers/staging/lustre/lustre/llite/namei.c| 6 +++ 3 files changed, 68

[PATCH 19/20] staging: lustre: llite: add xattr.h header to xattr.c

2017-07-26 Thread James Simmons
ug-id: https://jira.hpdd.intel.com/browse/LU-7244 Reviewed-on: http://review.whamcloud.com/16707 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: James Simmons <jsimm...@in

[PATCH 19/20] staging: lustre: llite: add xattr.h header to xattr.c

2017-07-26 Thread James Simmons
://review.whamcloud.com/16707 Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/llite/xattr.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/lustre/lustre/llite/xattr.c b/drivers/staging

[PATCH 16/20] staging: lustre: libcfs: fix test for libcfs_ioctl_hdr minimum size

2017-07-26 Thread James Simmons
The function libcfs_ioctl_getdata() test to see if libcfs_ioctl_hdr is smaller than struct libcfs_ioctl_data in size. This is wrong and it breaks the ioctl that is used to collect LNet stats. The correct size to compare against is struct libcfs_ioctl_hdr. Signed-off-by: James Simmons <uj

[PATCH 13/20] staging: lustre: ptlrpc: restore 64-bit time for struct ptlrpc_cli_req

2017-07-26 Thread James Simmons
During the reorganization of ptlrpc_request some of the time64_t fields were incorrectly turned into time_t. Restore those fields back to time_64_t. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamcloud.com/24977 Intel-bug-id: https://jira.hpdd.intel.com/

[PATCH 20/20] staging: lustre: llite: set security xattr using __vfs_setxattr

2017-07-26 Thread James Simmons
of when size is zero the xattr should be set to empty EA. Signed-off-by: James Simmons <uja.o...@yahoo.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9183 Reviewed-on: https://review.whamcloud.com/27240 Reviewed-by: Dmitry Eremin <dmitry.ere...@intel.com> Reviewed-by: Bob Glossma

[PATCH 16/20] staging: lustre: libcfs: fix test for libcfs_ioctl_hdr minimum size

2017-07-26 Thread James Simmons
The function libcfs_ioctl_getdata() test to see if libcfs_ioctl_hdr is smaller than struct libcfs_ioctl_data in size. This is wrong and it breaks the ioctl that is used to collect LNet stats. The correct size to compare against is struct libcfs_ioctl_hdr. Signed-off-by: James Simmons Intel-bug

[PATCH 13/20] staging: lustre: ptlrpc: restore 64-bit time for struct ptlrpc_cli_req

2017-07-26 Thread James Simmons
During the reorganization of ptlrpc_request some of the time64_t fields were incorrectly turned into time_t. Restore those fields back to time_64_t. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/24977 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4423 Fixes

[PATCH 20/20] staging: lustre: llite: set security xattr using __vfs_setxattr

2017-07-26 Thread James Simmons
of when size is zero the xattr should be set to empty EA. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9183 Reviewed-on: https://review.whamcloud.com/27240 Reviewed-by: Dmitry Eremin Reviewed-by: Bob Glossman Reviewed-by: Sebastien Buisson Reviewed-by: Oleg Drokin

[PATCH 01/20] staging: lustre: osc: soft lock - osc_makes_rpc()

2017-07-26 Thread James Simmons
t; Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8680 Reviewed-on: http://review.whamcloud.com/23326 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 S

[PATCH 01/20] staging: lustre: osc: soft lock - osc_makes_rpc()

2017-07-26 Thread James Simmons
-8680 Reviewed-on: http://review.whamcloud.com/23326 Reviewed-by: Jinshan Xiong Reviewed-by: Niu Yawei Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/osc/osc_cache.c | 24 +--- 1 file changed, 21 insertions(+), 3 deletions(-) diff --git

Re: [PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
> On Sat, 2017-07-15 at 11:39 -0400, James Simmons wrote: > > Cleanup all braces that was reported by checkpatch. The only > > issue not fixed up is in mdc_lock.c. Removing the braces in > > the case of mdc_lock.c will break the build. > > what checkp

Re: [PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
> On Sat, 2017-07-15 at 11:39 -0400, James Simmons wrote: > > Cleanup all braces that was reported by checkpatch. The only > > issue not fixed up is in mdc_lock.c. Removing the braces in > > the case of mdc_lock.c will break the build. > > what checkp

[PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
Cleanup all braces that was reported by checkpatch. The only issue not fixed up is in mdc_lock.c. Removing the braces in the case of mdc_lock.c will break the build. Signed-off-by: James Simmons <jsimm...@infradead.org> ---

[PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
Cleanup all braces that was reported by checkpatch. The only issue not fixed up is in mdc_lock.c. Removing the braces in the case of mdc_lock.c will break the build. Signed-off-by: James Simmons -- v1) Initial patch v2) Rebased

[PATCH] staging: lustre: ko2iblnd: check copy_from_iter/copy_to_iter return code

2017-07-15 Thread James Simmons
x now, returning -EFAULT as expected. Cc: sta...@vger.kernel.org Signed-off-by: Arnd Bergmann <a...@arndb.de> Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c| 19 +++ 1 file changed, 15 insertions(+)

[PATCH] staging: lustre: ko2iblnd: check copy_from_iter/copy_to_iter return code

2017-07-15 Thread James Simmons
ng -EFAULT as expected. Cc: sta...@vger.kernel.org Signed-off-by: Arnd Bergmann Signed-off-by: James Simmons --- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c| 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2ibl

Re: [PATCH] lustre: check copy_from_iter/copy_to_iter return code

2017-07-15 Thread James Simmons
On Fri, 14 Jul 2017, Al Viro wrote: > On Thu, Jul 13, 2017 at 10:57:59PM +0200, Arnd Bergmann wrote: > > > Thanks for testing it! > > > > That means we did not copy any data and the kernel continues with > > an uninitialized buffer, right? The problem may be the definition of > > > > struct

Re: [PATCH] lustre: check copy_from_iter/copy_to_iter return code

2017-07-15 Thread James Simmons
On Fri, 14 Jul 2017, Al Viro wrote: > On Thu, Jul 13, 2017 at 10:57:59PM +0200, Arnd Bergmann wrote: > > > Thanks for testing it! > > > > That means we did not copy any data and the kernel continues with > > an uninitialized buffer, right? The problem may be the definition of > > > > struct

Re: [PATCH] lustre: check copy_from_iter/copy_to_iter return code

2017-07-13 Thread James Simmons
> We now get a helpful warning for code that calls copy_{from,to}_iter > without checking the return value, introduced by commit aa28de275a24 > ("iov_iter/hardening: move object size checks to inlined part"). > > drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c: In function >

Re: [PATCH] lustre: check copy_from_iter/copy_to_iter return code

2017-07-13 Thread James Simmons
> We now get a helpful warning for code that calls copy_{from,to}_iter > without checking the return value, introduced by commit aa28de275a24 > ("iov_iter/hardening: move object size checks to inlined part"). > > drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c: In function >

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Thu, 2017-06-15 at 17:57 +0100, James Simmons wrote: > > > On Thu, 2017-06-15 at 17:03 +0100, James Simmons wrote: > > > > > On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > > > > > Due to the way the DFID was embedded in our debu

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Thu, 2017-06-15 at 17:57 +0100, James Simmons wrote: > > > On Thu, 2017-06-15 at 17:03 +0100, James Simmons wrote: > > > > > On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > > > > > Due to the way the DFID was embedded in our debu

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Thu, 2017-06-15 at 17:03 +0100, James Simmons wrote: > > > On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > > > Due to the way the DFID was embedded in our debug strings checkpatch > > > > would report the following error: > > > >

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Thu, 2017-06-15 at 17:03 +0100, James Simmons wrote: > > > On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > > > Due to the way the DFID was embedded in our debug strings checkpatch > > > > would report the following error: > > > >

Re: [PATCH 6/6] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Cleanup all braces that was reported by checkpatch. The only > > issue not fixed up is in mdc_lock.c. Removing the braces in > > the case of mdc_lock.c will break the build. > > [] > > > diff -

Re: [PATCH 6/6] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Cleanup all braces that was reported by checkpatch. The only > > issue not fixed up is in mdc_lock.c. Removing the braces in > > the case of mdc_lock.c will break the build. > > [] > > > diff -

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

2017-06-15 Thread James Simmons
> On Mon, Jun 12, 2017 at 08:20:15PM +, Dilger, Andreas wrote: > > On Jan 21, 2017, at 02:24, Greg Kroah-Hartman <gre...@linuxfoundation.org> > > wrote: > > > > > > On Fri, Jan 20, 2017 at 11:33:11PM +, James Simmons wrote: > > >>

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

2017-06-15 Thread James Simmons
> On Mon, Jun 12, 2017 at 08:20:15PM +, Dilger, Andreas wrote: > > On Jan 21, 2017, at 02:24, Greg Kroah-Hartman > > wrote: > > > > > > On Fri, Jan 20, 2017 at 11:33:11PM +, James Simmons wrote: > > >> > > >>>>

Re: [PATCH 5/6] staging: lustre: lustre: several over 80 characters cleanups

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Cleanup many of the over 80 characters reported by checkpatch > > Please don't let checkpatch get in the way of lustre > readability. > > lustre commonly uses very long identifiers. > Long identifiers an

Re: [PATCH 5/6] staging: lustre: lustre: several over 80 characters cleanups

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Cleanup many of the over 80 characters reported by checkpatch > > Please don't let checkpatch get in the way of lustre > readability. > > lustre commonly uses very long identifiers. > Long identifiers an

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Due to the way the DFID was embedded in our debug strings checkpatch > > would report the following error: > > unrelated trivia > > > diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre

Re: [PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-15 Thread James Simmons
> On Wed, 2017-06-14 at 11:01 -0400, James Simmons wrote: > > Due to the way the DFID was embedded in our debug strings checkpatch > > would report the following error: > > unrelated trivia > > > diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre

[PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-14 Thread James Simmons
Due to the way the DFID was embedded in our debug strings checkpatch would report the following error: CHECK: Concatenated strings should use spaces between elements This patch introduces proper space to resolve these reports. Signed-off-by: James Simmons <jsimm...@infradead.org> --- d

[PATCH 1/6] staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors

2017-06-14 Thread James Simmons
Due to the way the DFID was embedded in our debug strings checkpatch would report the following error: CHECK: Concatenated strings should use spaces between elements This patch introduces proper space to resolve these reports. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/fid

[PATCH 0/6] staging: lustre: lustre: assortment of checkpatch cleanups

2017-06-14 Thread James Simmons
This patch series covers a good chunk of the checkpatch issues in the current lustre client code. James Simmons (6): staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors staging: lustre: lustre: make all struct file_operations constant staging: lust

[PATCH 0/6] staging: lustre: lustre: assortment of checkpatch cleanups

2017-06-14 Thread James Simmons
This patch series covers a good chunk of the checkpatch issues in the current lustre client code. James Simmons (6): staging: lustre: lustre: resolve "use spaces between elements" checkpatch errors staging: lustre: lustre: make all struct file_operations constant staging: lust

[PATCH 2/6] staging: lustre: lustre: make all struct file_operations constant

2017-06-14 Thread James Simmons
Checkpatch reported several cases of struct file_operations not being const. This resolves those warnings. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 2 +- drivers/staging/lustre/lustre/include/lprocfs_status.

[PATCH 5/6] staging: lustre: lustre: several over 80 characters cleanups

2017-06-14 Thread James Simmons
Cleanup many of the over 80 characters reported by checkpatch Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/include/cl_object.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 3 +- drivers/staging/lustre/lustre/ldlm/ldlm_extent.c

[PATCH 5/6] staging: lustre: lustre: several over 80 characters cleanups

2017-06-14 Thread James Simmons
Cleanup many of the over 80 characters reported by checkpatch Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/cl_object.h | 2 +- .../staging/lustre/lustre/include/lprocfs_status.h | 3 +- drivers/staging/lustre/lustre/ldlm/ldlm_extent.c | 5 ++-- drivers/staging

[PATCH 2/6] staging: lustre: lustre: make all struct file_operations constant

2017-06-14 Thread James Simmons
Checkpatch reported several cases of struct file_operations not being const. This resolves those warnings. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 2 +- drivers/staging/lustre/lustre/include/lprocfs_status.h | 8 drivers/staging

[PATCH 3/6] staging: lustre: lustre: fix all bare unsigned usage

2017-06-14 Thread James Simmons
Turn all bare unsigned usage in the lustre code to proper unsigned int. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/include/cl_object.h | 4 ++-- drivers/staging/lustre/lustre/include/lu_object.h | 4 ++-- drivers/staging/lustre/lustre/llite/n

[PATCH 3/6] staging: lustre: lustre: fix all bare unsigned usage

2017-06-14 Thread James Simmons
Turn all bare unsigned usage in the lustre code to proper unsigned int. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/cl_object.h | 4 ++-- drivers/staging/lustre/lustre/include/lu_object.h | 4 ++-- drivers/staging/lustre/lustre/llite/namei.c | 2 +- drivers

[PATCH 6/6] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-06-14 Thread James Simmons
Cleanup all braces that was reported by checkpatch. The only issue not fixed up is in mdc_lock.c. Removing the braces in the case of mdc_lock.c will break the build. Signed-off-by: James Simmons <jsimm...@infradead.org> --- drivers/staging/lustre/lustre/fld/fld_cache.c | 3 ++- drivers/s

[PATCH 6/6] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-06-14 Thread James Simmons
Cleanup all braces that was reported by checkpatch. The only issue not fixed up is in mdc_lock.c. Removing the braces in the case of mdc_lock.c will break the build. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/fld/fld_cache.c | 3 ++- drivers/staging/lustre/lustre/ldlm

[PATCH 4/6] staging: lustre: lustre: add all missing indentifier names

2017-06-14 Thread James Simmons
Create identifier names missing from function prototypes as reported by checkpatch. Signed-off-by: James Simmons <jsimm...@infradead.org> --- .../staging/lustre/lustre/include/lprocfs_status.h | 4 ++-- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 27 -- .../s

[PATCH 4/6] staging: lustre: lustre: add all missing indentifier names

2017-06-14 Thread James Simmons
Create identifier names missing from function prototypes as reported by checkpatch. Signed-off-by: James Simmons --- .../staging/lustre/lustre/include/lprocfs_status.h | 4 ++-- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 27 -- .../staging/lustre/lustre/llite

Re: [PATCH 5/5] staging: lustre: osc_page.c: Use list_for_each_entry_safe

2017-03-06 Thread James Simmons
> Doubly linked lists which are iterated using list_empty > and list_entry macros have been replaced with list_for_each_entry_safe > macro. > This makes the iteration simpler and more readable. > > This patch replaces the while loop containing list_empty and list_entry > with

Re: [PATCH 5/5] staging: lustre: osc_page.c: Use list_for_each_entry_safe

2017-03-06 Thread James Simmons
> Doubly linked lists which are iterated using list_empty > and list_entry macros have been replaced with list_for_each_entry_safe > macro. > This makes the iteration simpler and more readable. > > This patch replaces the while loop containing list_empty and list_entry > with

Re: [PATCH v5 2/5] staging: lustre: Remove unnecessary cast on void pointer

2017-03-04 Thread James Simmons
> The following Coccinelle script was used to detect this: > @r@ > expression x; > void* e; > type T; > identifier f; > @@ > ( > *((T *)e) > | > ((T *)x)[...] > | > ((T*)x)->f > | > - (T*) > e > ) > > Signed-off-by: simran si

Re: [PATCH v5 2/5] staging: lustre: Remove unnecessary cast on void pointer

2017-03-04 Thread James Simmons
> The following Coccinelle script was used to detect this: > @r@ > expression x; > void* e; > type T; > identifier f; > @@ > ( > *((T *)e) > | > ((T *)x)[...] > | > ((T*)x)->f > | > - (T*) > e > ) > > Signed-off-by: simran sin

Re: [PATCH v5 3/5] staging: lustre: lustre: Remove unnecessary cast on void pointer

2017-03-04 Thread James Simmons
> The following Coccinelle script was used to detect this: > @r@ > expression x; > void* e; > type T; > identifier f; > @@ > ( > *((T *)e) > | > ((T *)x)[...] > | > ((T*)x)->f > | > - (T*) > e > ) > > Signed-off-by: simran si

Re: [PATCH v5 3/5] staging: lustre: lustre: Remove unnecessary cast on void pointer

2017-03-04 Thread James Simmons
> The following Coccinelle script was used to detect this: > @r@ > expression x; > void* e; > type T; > identifier f; > @@ > ( > *((T *)e) > | > ((T *)x)[...] > | > ((T*)x)->f > | > - (T*) > e > ) > > Signed-off-by:

[PATCH 25/34] staging: lustre: lnet: change lnet_event_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_event_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 25/34] staging: lustre: lnet: change lnet_event_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_event_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 22/34] staging: lustre: lnet: give LNET_MATCH* enum a name

2017-02-26 Thread James Simmons
The LNET_MATCH* flags are an enum without a name. Lets label that enum. 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

[PATCH 22/34] staging: lustre: lnet: give LNET_MATCH* enum a name

2017-02-26 Thread James Simmons
The LNET_MATCH* flags are an enum without a name. Lets label that enum. 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

[PATCH 30/34] staging: lustre: lnet: change lnet_event_kind_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_event_kind_t from typedef to proper enum. 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 30/34] staging: lustre: lnet: change lnet_event_kind_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_event_kind_t from typedef to proper enum. 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 29/34] staging: lustre: lnet: change lnet_ins_pos_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_ins_pos_t from typedef to proper enum. 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 29/34] staging: lustre: lnet: change lnet_ins_pos_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_ins_pos_t from typedef to proper enum. 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 24/34] staging: lustre: lnet: change lnet_md_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_md_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 24/34] staging: lustre: lnet: change lnet_md_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_md_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 20/34] staging: lustre: lnet: change lnet_portal_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_portal_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 20/34] staging: lustre: lnet: change lnet_portal_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_portal_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 02/34] staging: lustre: lnet: change lnet_handle_eq_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_eq_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateEQHandle and LNetEQHandleIsInvalid to handle this new piece of data. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamclo

[PATCH 02/34] staging: lustre: lnet: change lnet_handle_eq_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_eq_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateEQHandle and LNetEQHandleIsInvalid to handle this new piece of data. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed

[PATCH 14/34] staging: lustre: lnet: change lnet_rc_data_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rc_data_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 14/34] staging: lustre: lnet: change lnet_rc_data_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rc_data_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 04/34] staging: lustre: lnet: change lnet_handle_me_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_me_t from a typedef of another typedef into a proper stand alone 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...@inte

[PATCH 03/34] staging: lustre: lnet: change lnet_handle_md_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_md_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateMDHandle and LNetMDHandleIsInvalid to handle this new piece of data. Signed-off-by: James Simmons <uja.o...@yahoo.com> Reviewed-on: https://review.whamclo

[PATCH 04/34] staging: lustre: lnet: change lnet_handle_me_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_me_t from a typedef of another typedef into a proper stand alone 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

[PATCH 03/34] staging: lustre: lnet: change lnet_handle_md_t to proper struct

2017-02-26 Thread James Simmons
Change lnet_handle_md_t from a typedef of another typedef into a proper stand alone structure. Create the inline functions LNetInvalidateMDHandle and LNetMDHandleIsInvalid to handle this new piece of data. Signed-off-by: James Simmons Reviewed-on: https://review.whamcloud.com/20831 Reviewed

[PATCH 34/34] staging: lustre: lnet: use struct bio_vec instead of typedef

2017-02-26 Thread James Simmons
Kill off lnet_kiov_t and use struct bio_vec directly. Signed-off-by: James Simmons <uja.o...@yahoo.com> --- drivers/staging/lustre/include/linux/lnet/lib-lnet.h| 8 drivers/staging/lustre/include/linux/lnet/lib-types.h | 6 +++--- drivers/staging/lustre/include/linu

[PATCH 32/34] staging: lustre: lnet: remove LNET_SEQ_BASETYPE

2017-02-26 Thread James Simmons
The lnet_seq_t is a simple unsigned long so lets simplify it. 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 33/34] staging: lustre: lnet: remove unused lnet_md_iovec_t

2017-02-26 Thread James Simmons
The typedef lnet_md_iovec_t is never used so kill it off. 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 34/34] staging: lustre: lnet: use struct bio_vec instead of typedef

2017-02-26 Thread James Simmons
Kill off lnet_kiov_t and use struct bio_vec directly. Signed-off-by: James Simmons --- drivers/staging/lustre/include/linux/lnet/lib-lnet.h| 8 drivers/staging/lustre/include/linux/lnet/lib-types.h | 6 +++--- drivers/staging/lustre/include/linux/lnet/types.h | 7

[PATCH 32/34] staging: lustre: lnet: remove LNET_SEQ_BASETYPE

2017-02-26 Thread James Simmons
The lnet_seq_t is a simple unsigned long so lets simplify it. 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 33/34] staging: lustre: lnet: remove unused lnet_md_iovec_t

2017-02-26 Thread James Simmons
The typedef lnet_md_iovec_t is never used so kill it off. 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 31/34] staging: lustre: lnet: change lnet_ack_req_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_ack_req_t from typedef to proper enum. 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 31/34] staging: lustre: lnet: change lnet_ack_req_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_ack_req_t from typedef to proper enum. 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 28/34] staging: lustre: lnet: change lnet_unlink_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_unlink_t from typedef to proper enum. 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 28/34] staging: lustre: lnet: change lnet_unlink_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_unlink_t from typedef to proper enum. 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 27/34] staging: lustre: lnet: change lnet_msg_type_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_msg_type_t from typedef to proper enum. 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 27/34] staging: lustre: lnet: change lnet_msg_type_t to proper enum

2017-02-26 Thread James Simmons
Change lnet_msg_type_t from typedef to proper enum. 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 26/34] staging: lustre: lnet: change lnet_process_id_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_process_id_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 26/34] staging: lustre: lnet: change lnet_process_id_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_process_id_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 23/34] staging: lustre: lnet: change lnet_counter_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_counter_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 23/34] staging: lustre: lnet: change lnet_counter_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_counter_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 21/34] staging: lustre: lnet: change lnet_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_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 21/34] staging: lustre: lnet: change lnet_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_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 19/34] staging: lustre: lnet: change lnet_rtrbuf_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rtrbuf_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 19/34] staging: lustre: lnet: change lnet_rtrbuf_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rtrbuf_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 18/34] staging: lustre: lnet: change lnet_rtrbufpool_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rtrbufpool_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 18/34] staging: lustre: lnet: change lnet_rtrbufpool_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_rtrbufpool_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 17/34] staging: lustre: lnet: change lnet_remotenet_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_remotenet_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 17/34] staging: lustre: lnet: change lnet_remotenet_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_remotenet_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 16/34] staging: lustre: lnet: change lnet_route_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_route_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 16/34] staging: lustre: lnet: change lnet_route_t to proper structure

2017-02-26 Thread James Simmons
Change lnet_route_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

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