Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Fri, Nov 13, 2015 at 6:33 PM, Sudip Mukherjee wrote: > > On Wed, Nov 11, 2015 at 11:03:34PM +0530, Shivani Bhardwaj wrote: > > On Wed, Nov 11, 2015 at 11:01 PM, Greg KH > > wrote: > > > On Wed, Nov 11, 2015 at 10:51:36PM +0530, Shivani Bhardwaj wrote: > > &

Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Fri, Nov 13, 2015 at 6:33 PM, Sudip Mukherjee <sudipm.mukher...@gmail.com> wrote: > > On Wed, Nov 11, 2015 at 11:03:34PM +0530, Shivani Bhardwaj wrote: > > On Wed, Nov 11, 2015 at 11:01 PM, Greg KH <gre...@linuxfoundation.org> > > wrote: > > > On Wed, No

Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Wed, Nov 11, 2015 at 11:01 PM, Greg KH wrote: > On Wed, Nov 11, 2015 at 10:51:36PM +0530, Shivani Bhardwaj wrote: >> On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot wrote: >> > Hi Shivani, >> > >> > [auto build test ERROR on staging/staging-testing] >

Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot wrote: > Hi Shivani, > > [auto build test ERROR on staging/staging-testing] > [also build test ERROR on v4.3 next-2015] > > url: > https://github.com/0day-ci/linux/commits/Shivani-Bhardwaj/staging-

[PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

Remove the wrapper function cl_lock_lockdep_init() and replace its only call with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/obdclass/cl_lock.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/lustre/lustre

[PATCH] staging: lustre: Remove cl_2queue_add wrapper

. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/include/cl_object.h | 1 - drivers/staging/lustre/lustre/llite/rw26.c | 5 - drivers/staging/lustre/lustre/llite/vvp_io.c| 2 +- drivers/staging/lustre/lustre/obdclass/cl_io.c | 14 -- drivers

[PATCH] staging: lustre: cl_io: Remove cl_lock_descr_fid wrapper

Remove unnecessary wrapper function cl_lock_descr_fid() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/obdclass/cl_io.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/staging

[PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

Remove the wrapper function cl_lock_lockdep_init() and replace its only call with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/obdclass/cl_lock.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff

[PATCH] staging: lustre: cl_io: Remove cl_lock_descr_fid wrapper

Remove unnecessary wrapper function cl_lock_descr_fid() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/obdclass/cl_io.c | 12 1 file changed, 4 insertions(+), 8 deletions(-)

[PATCH] staging: lustre: Remove cl_2queue_add wrapper

. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/include/cl_object.h | 1 - drivers/staging/lustre/lustre/llite/rw26.c | 5 - drivers/staging/lustre/lustre/llite/vvp_io.c| 2 +- drivers/staging/lustre/lustre/obdclass/cl_io.c

Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Wed, Nov 11, 2015 at 11:01 PM, Greg KH <gre...@linuxfoundation.org> wrote: > On Wed, Nov 11, 2015 at 10:51:36PM +0530, Shivani Bhardwaj wrote: >> On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot <l...@intel.com> wrote: >> > Hi Shivani, >> > >>

Re: [PATCH] staging: lustre: cl_lock: Remove cl_lock_lockdep_init wrapper

On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot <l...@intel.com> wrote: > Hi Shivani, > > [auto build test ERROR on staging/staging-testing] > [also build test ERROR on v4.3 next-2015] > > url: > https://github.com/0day-ci/linux/commits/Shivani-Bhardwaj/st

[PATCH v2 8/8] Staging: lustre: llite: Remove ll_finish_md_op_data wrapper

Remove the wrapper function ll_finish_md_op_data() and replace all its calls with the standard function kfree() that it wraps. Also, prototype of the function is removed as it is no longer of any use. Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch

[PATCH v2 7/8] Staging: lustre: llite_close: Substitute function calls

Substitute standard function kfree() in place of the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/llite_close.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2 6/8] Staging: lustre: llite_nfs: Replace function calls with kfree

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v2 5/8] Staging: lustre: symlink: Substitute standard function

Substitute the standard function kfree() for the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/symlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2 4/8] Staging: lustre: xattr_cache: Change function calls to kfree

Change the calls of the function ll_finish_md_op_data() to the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/xattr_cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v2 3/8] Staging: lustre: namei: Replace calls with kfree

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/namei.c | 12 ++-- 1 file changed, 6 insertions(+), 6

[PATCH v2 2/8] Staging: lustre: file: Replace function calls with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/file.c | 24 1 file changed, 12 insertions

[PATCH v2 1/8] Staging: lustre: dir: Replace function calls

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/dir.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[PATCH v2 0/8] Remove wrapper function and clean up the code

This patchset removes an unnecessary wrapper function, its prototype and replaces all its calls in different files with the standard function that it wraps. After applying this patch, code becomes cleaner. Changes in v2: Fix commit message and merge a patch Shivani Bhardwaj (8

Re: [PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

On Tue, Nov 10, 2015 at 12:48 AM, Simmons, James A. wrote: >>On Mon, Nov 9, 2015 at 7:07 PM, Michał Kępień wrote: Remove the function ll_finish_md_op_data() and replace all its calls with the standrd function ll_finish_md_op_data(). >>> >>> I believe you meant to write "standard

[PATCH] staging: lustre: acl: Remove lustre_posix_acl_xattr_free wrapper

Remove the wrapper function lustre_posix_acl_xattr_free() and replace its call in the file xattr with the function kfree() that it wrapped. Also, its prototype from the header lustre_eacl is removed as it is no longer of any use. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre

Re: [PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

On Mon, Nov 9, 2015 at 7:07 PM, Michał Kępień wrote: >> Remove the function ll_finish_md_op_data() and replace all its calls >> with the standrd function ll_finish_md_op_data(). > > I believe you meant to write "standard function kfree()". > Yes. I am so sorry. Should I be sending the whole

[PATCH v2 0/8] Remove wrapper function and clean up the code

This patchset removes an unnecessary wrapper function, its prototype and replaces all its calls in different files with the standard function that it wraps. After applying this patch, code becomes cleaner. Changes in v2: Fix commit message and merge a patch Shivani Bhardwaj (8

[PATCH v2 3/8] Staging: lustre: namei: Replace calls with kfree

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/namei.c | 12 ++-- 1 file chan

[PATCH v2 2/8] Staging: lustre: file: Replace function calls with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/file.c | 24

[PATCH v2 1/8] Staging: lustre: dir: Replace function calls

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/dir.c | 14 +++--- 1 file chan

[PATCH v2 4/8] Staging: lustre: xattr_cache: Change function calls to kfree

Change the calls of the function ll_finish_md_op_data() to the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/xattr_cache.c | 2 +- 1 file changed, 1 ins

[PATCH v2 5/8] Staging: lustre: symlink: Substitute standard function

Substitute the standard function kfree() for the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/symlink.c | 2 +- 1 file changed, 1 insertion

[PATCH v2 6/8] Staging: lustre: llite_nfs: Replace function calls with kfree

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- 1 file chan

[PATCH v2 8/8] Staging: lustre: llite: Remove ll_finish_md_op_data wrapper

Remove the wrapper function ll_finish_md_op_data() and replace all its calls with the standard function kfree() that it wraps. Also, prototype of the function is removed as it is no longer of any use. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix

[PATCH v2 7/8] Staging: lustre: llite_close: Substitute function calls

Substitute standard function kfree() in place of the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- Changes in v2: Fix commit message and merge a patch drivers/staging/lustre/lustre/llite/llite_close.c | 2 +- 1 file changed, 1 ins

[PATCH] staging: lustre: acl: Remove lustre_posix_acl_xattr_free wrapper

Remove the wrapper function lustre_posix_acl_xattr_free() and replace its call in the file xattr with the function kfree() that it wrapped. Also, its prototype from the header lustre_eacl is removed as it is no longer of any use. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.

Re: [PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

On Mon, Nov 9, 2015 at 7:07 PM, Michał Kępień wrote: >> Remove the function ll_finish_md_op_data() and replace all its calls >> with the standrd function ll_finish_md_op_data(). > > I believe you meant to write "standard function kfree()". > Yes. I am so sorry. Should I be

Re: [PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

On Tue, Nov 10, 2015 at 12:48 AM, Simmons, James A. wrote: >>On Mon, Nov 9, 2015 at 7:07 PM, Michał Kępień wrote: Remove the function ll_finish_md_op_data() and replace all its calls with the standrd function ll_finish_md_op_data(). >>> >>> I

Re: [PATCH] staging: lustre: lproc_mgc: Remove unused function mgc_ir_state_seq_show

On Sun, Nov 8, 2015 at 10:33 PM, Shivani Bhardwaj wrote: > Remove the function mgc_ir_state_seq_show() as it is not > referenced/called anywhere. > > Signed-off-by: Shivani Bhardwaj > --- > drivers/staging/lustre/lustre/mgc/lproc_mgc.c | 5 - > 1 file changed, 5 deleti

[PATCH] staging: lustre: lproc_mgc: Remove unused function mgc_ir_state_seq_show

Remove the function mgc_ir_state_seq_show() as it is not referenced/called anywhere. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/mgc/lproc_mgc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/mgc/lproc_mgc.c b/drivers/staging/lustre

[PATCH] staging: lustre: mdc_request: Remove mdc_kuc_reregister wrapper

Remove the wrapper function mdc_kuc_reregister() and replace its call with the function it wrapped. Also, comment has been added for clarity. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/mdc/mdc_request.c | 16 1 file changed, 4 insertions(+), 12 deletions

[PATCH 2/2] staging: lustre: workitem: Remove cfs_wi_sched_unlock wrapper

Remove the wrapper function cfs_wi_sched_unlock() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/workitem.c | 26 ++--- 1 file changed, 10 insertions(+), 16 deletions(-) diff --git a/drivers

[PATCH 0/2] Remove wrapper functions from workitem

This patchset removes unnecessary wrapper functions from the file workitem and replace all their calls with the function that they wrap. After applying this patch, code becomes cleaner. Shivani Bhardwaj (2): staging: lustre: workitem: Remove cfs_wi_sched_lock wrapper staging: lustre: workitem

[PATCH 1/2] staging: lustre: workitem: Remove cfs_wi_sched_lock wrapper

Remove the wrapper function cfs_wi_sched_lock() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/workitem.c | 22 -- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/drivers/staging

Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

On Sun, Nov 8, 2015 at 2:17 AM, Dan Carpenter wrote: > Don't resend, it's fine. I don't like re-reviewing patches and you > don't like sending them. It's not like it introduces a bug or generate > a warning with our current tools. > > But it's still important to understand how the one thing per

[PATCH] staging: lustre: lproc_mgc: Remove unused function mgc_ir_state_seq_show

Remove the function mgc_ir_state_seq_show() as it is not referenced/called anywhere. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/mgc/lproc_mgc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/mgc/lproc_mg

[PATCH 0/2] Remove wrapper functions from workitem

This patchset removes unnecessary wrapper functions from the file workitem and replace all their calls with the function that they wrap. After applying this patch, code becomes cleaner. Shivani Bhardwaj (2): staging: lustre: workitem: Remove cfs_wi_sched_lock wrapper staging: lustre: workitem

[PATCH 1/2] staging: lustre: workitem: Remove cfs_wi_sched_lock wrapper

Remove the wrapper function cfs_wi_sched_lock() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/libcfs/workitem.c | 22 -- 1 file changed, 8 insertions(+), 14 deletions(-)

[PATCH 2/2] staging: lustre: workitem: Remove cfs_wi_sched_unlock wrapper

Remove the wrapper function cfs_wi_sched_unlock() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/libcfs/workitem.c | 26 ++--- 1 file changed, 10 insertions(+), 16 del

Re: [PATCH] staging: lustre: lproc_mgc: Remove unused function mgc_ir_state_seq_show

On Sun, Nov 8, 2015 at 10:33 PM, Shivani Bhardwaj <shivanib...@gmail.com> wrote: > Remove the function mgc_ir_state_seq_show() as it is not > referenced/called anywhere. > > Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> > --- > drivers/staging/lus

[PATCH] staging: lustre: mdc_request: Remove mdc_kuc_reregister wrapper

Remove the wrapper function mdc_kuc_reregister() and replace its call with the function it wrapped. Also, comment has been added for clarity. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/mdc/mdc_request.c | 16 1 file chan

Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

On Sun, Nov 8, 2015 at 2:17 AM, Dan Carpenter wrote: > Don't resend, it's fine. I don't like re-reviewing patches and you > don't like sending them. It's not like it introduces a bug or generate > a warning with our current tools. > > But it's still important to

Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

On Sat, Nov 7, 2015 at 4:15 PM, Dan Carpenter wrote: > This one should have been folded in with the last one. Really the whole > series could have been sent as one patch, but especially the last two > should have been folded together. > > regards, > dan carpenter > Thanks Dan! Could you please

[PATCH 5/5] Staging: lustre: statahead: Remove agl_list_empty wrapper

Remove the function agl_list_empty() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/statahead.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH 4/5] Staging: lustre: statahead: Remove sa_received_empty wrapper

Remove the wrapper sa_received_empty() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/statahead.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 3/5] Staging: lustre: statahead: Remove agl_first_entry wrapper

Remove the wrapper function agl_first_entry() and replace its calls with the function list_entry() that it wraps. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/statahead.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers

[PATCH 2/5] Staging: lustre: statahead: Remove sa_first_received_entry wrapper

Remove the function sa_first_received_entry() and replace all its calls with the function list_entry() that it wraps. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/statahead.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers

[PATCH 1/5] Staging: lustre: statahead: Remove ll_sa_entry_unhashed wrapper

Remove the function ll_sa_entry_unhashed() and replace all its calls with the function list_empty() that it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/statahead.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging

[PATCH 0/5] Remove wrappers from statahead

. Shivani Bhardwaj (5): Staging: lustre: statahead: Remove ll_sa_entry_unhashed wrapper Staging: lustre: statahead: Remove sa_first_received_entry wrapper Staging: lustre: statahead: Remove agl_first_entry wrapper Staging: lustre: statahead: Remove sa_received_empty wrapper Staging: lustre

[PATCH] Staging: lustre: rw: Remove wrapper stride_page_count

Remove the function stride_page_count() and replace its calls with the function stride_pg_count() that it wraps. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/rw.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre

Re: [PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

On Sat, Nov 7, 2015 at 4:15 PM, Dan Carpenter wrote: > This one should have been folded in with the last one. Really the whole > series could have been sent as one patch, but especially the last two > should have been folded together. > > regards, > dan carpenter >

[PATCH 2/5] Staging: lustre: statahead: Remove sa_first_received_entry wrapper

Remove the function sa_first_received_entry() and replace all its calls with the function list_entry() that it wraps. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/statahead.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-)

[PATCH 1/5] Staging: lustre: statahead: Remove ll_sa_entry_unhashed wrapper

Remove the function ll_sa_entry_unhashed() and replace all its calls with the function list_empty() that it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/statahead.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-)

[PATCH 3/5] Staging: lustre: statahead: Remove agl_first_entry wrapper

Remove the wrapper function agl_first_entry() and replace its calls with the function list_entry() that it wraps. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/statahead.c | 19 --- 1 file changed, 8 insertions(+), 11 del

[PATCH 0/5] Remove wrappers from statahead

. Shivani Bhardwaj (5): Staging: lustre: statahead: Remove ll_sa_entry_unhashed wrapper Staging: lustre: statahead: Remove sa_first_received_entry wrapper Staging: lustre: statahead: Remove agl_first_entry wrapper Staging: lustre: statahead: Remove sa_received_empty wrapper Staging: lustre

[PATCH] Staging: lustre: rw: Remove wrapper stride_page_count

Remove the function stride_page_count() and replace its calls with the function stride_pg_count() that it wraps. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/rw.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff

[PATCH 5/5] Staging: lustre: statahead: Remove agl_list_empty wrapper

Remove the function agl_list_empty() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/statahead.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/d

[PATCH 4/5] Staging: lustre: statahead: Remove sa_received_empty wrapper

Remove the wrapper sa_received_empty() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/statahead.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

On Sat, Nov 7, 2015 at 1:02 PM, Dilger, Andreas wrote: > On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: > >>Remove the prototype of function cfs_trace_free_string_buffer() as it is >>no longer needed. > > These patches would be a lot more useful

[PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

Remove the prototype of the function ll_finish_md_op_data() as it is no longer needed. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_internal.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_internal.h b/drivers

[PATCH 7/9] Staging: lustre: llite_close: Substitute function calls

Substitute standard function kfree() in place of the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_close.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_close.c b

[PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

Remove the function ll_finish_md_op_data() and replace all its calls with the standrd function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_lib.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers

[PATCH 6/9] Staging: lustre: llite_nfs: Replace with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_nfs.c b

[PATCH 5/9] Staging: lustre: symlink: Substitute standard function

Substitute the standard function kfree() for the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/symlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/symlink.c b/drivers

[PATCH 4/9] Staging: lustre: xattr_cache: Change function calls

Change the calls of the function ll_finish_md_op_data() to the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/xattr_cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/xattr_cache.c

[PATCH 3/9] Staging: lustre: namei: Replace calls with kfree

Replace the calls of the function ll_finish_md_op_data() with thr stndard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/namei.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH 2/9] Staging: lustre: file: Replace function calls with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/file.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 1/9] Staging: lustre: dir: Replace function calls

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/dir.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c

[PATCH 0/9] Remove wrapper function and tidy up the code

This patchset removes an unnecessary wrapper function and replaces all its calls in different files with the standard function that it wrapped. Also, one patch removes its prototype. After applying this patch, code becomes cleaner. Shivani Bhardwaj (9): Staging: lustre: dir: Replace function

[PATCH 3/3] Staging: lustre: ldlm_pool: Drop unneeded wrapper function

Remove the function ldlm_pool_set_limit() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm

[PATCH 2/3] Staging: lustre: ldlm_pool: Drop wrapper function

Remove the function ldlm_pool_get_limit() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm

[PATCH 1/3] Staging: lustre: ldlm_pool: Remove unneeded wrapper function

Remove the function ldlm_pl2ns() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 0/3] Remove wrapper functions from ldlm_pool

This patchset removes different wrapper functions from the file ldlm_pool and replaces their calls with the function they wrapped. After applying this patchset, code becomes cleaner. Shivani Bhardwaj (3): Staging: lustre: ldlm_pool: Remove unneeded wrapper function Staging: lustre: ldlm_pool

[PATCH 2/3] Staging: lustre: tracefile: Remove wrapper function

Remove the function cfs_trace_free_string_buffer() as it can be replaced with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/tracefile.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

Remove the prototype of function cfs_trace_free_string_buffer() as it is no longer needed. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/tracefile.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/libcfs/tracefile.h b/drivers/staging

[PATCH 1/3] Staging: lustre: module: Replace function calls

Replace the calls of function cfs_trace_free_string_buffer() with kfree() as the former function is not required. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 0/3] Remove wrapper function and clean up the code

This patchset removes a wrapper function, its prototype and replace its calls in different files with a standard function. After applying this patch, code becomes cleaner. Shivani Bhardwaj (3): Staging: lustre: module: Replace function calls Staging: lustre: tracefile: Remove wrapper function

[PATCH] Staging: lustre: dir: Remove wrapper function

Remove the function ll_check_page() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/dir.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

On Sat, Nov 7, 2015 at 1:02 PM, Dilger, Andreas <andreas.dil...@intel.com> wrote: > On 2015/11/06, 10:19, "Shivani Bhardwaj" <shivanib...@gmail.com> wrote: > >>Remove the prototype of function cfs_trace_free_string_buffer() as it is >>no longer needed. >

[PATCH 0/9] Remove wrapper function and tidy up the code

This patchset removes an unnecessary wrapper function and replaces all its calls in different files with the standard function that it wrapped. Also, one patch removes its prototype. After applying this patch, code becomes cleaner. Shivani Bhardwaj (9): Staging: lustre: dir: Replace function

[PATCH 1/9] Staging: lustre: dir: Replace function calls

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/dir.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/s

[PATCH 2/9] Staging: lustre: file: Replace function calls with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/file.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff

[PATCH 3/9] Staging: lustre: namei: Replace calls with kfree

Replace the calls of the function ll_finish_md_op_data() with thr stndard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/namei.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/s

[PATCH 4/9] Staging: lustre: xattr_cache: Change function calls

Change the calls of the function ll_finish_md_op_data() to the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/xattr_cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/

[PATCH 6/9] Staging: lustre: llite_nfs: Replace with standard function

Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/

[PATCH 5/9] Staging: lustre: symlink: Substitute standard function

Substitute the standard function kfree() for the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/symlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 7/9] Staging: lustre: llite_close: Substitute function calls

Substitute standard function kfree() in place of the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/llite_close.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/

[PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

Remove the function ll_finish_md_op_data() and replace all its calls with the standrd function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/llite_lib.c | 13 - 1 file changed, 4 insertions(+), 9 del

[PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

Remove the prototype of the function ll_finish_md_op_data() as it is no longer needed. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/llite_internal.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 2/3] Staging: lustre: tracefile: Remove wrapper function

Remove the function cfs_trace_free_string_buffer() as it can be replaced with the standard function kfree(). Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/libcfs/tracefile.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff

[PATCH] Staging: lustre: dir: Remove wrapper function

Remove the function ll_check_page() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/llite/dir.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/

[PATCH 1/3] Staging: lustre: module: Replace function calls

Replace the calls of function cfs_trace_free_string_buffer() with kfree() as the former function is not required. Signed-off-by: Shivani Bhardwaj <shivanib...@gmail.com> --- drivers/staging/lustre/lustre/libcfs/module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

  1   2   >