[PATCH] staging: sm750fb: replace roundedDiv with DIV_ROUND_CLOSEST

Replace local implementation of rounded division (roundedDiv macro) with the in-kernel implementation (DIV_ROUND_CLOSEST macro) in ddk750_chip.c Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 6 ++ 1 file changed, 2 insertions(+), 4 del

Re: [PATCH] staging: sm750fb: rename macro roundedDiv to rounded_div in ddk750_chip.c

On Sun, Oct 02, 2016 at 11:47:47AM +0200, Greg KH wrote: > On Sun, Oct 02, 2016 at 08:48:32AM +0300, Moshe Green wrote: > > Rename CamelCased macro roundedDiv to rounded_div. > > > > This issue was found by checkpatch.pl > > > > Signed-off-by

[PATCH] staging: sm750fb: rename macro roundedDiv to rounded_div in ddk750_chip.c

Rename CamelCased macro roundedDiv to rounded_div. This issue was found by checkpatch.pl Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_chi

[PATCH] staging: sm750fb: rename getChipType to sm750_get_chip_type

Rename CamelCased function getChipType to sm750_get_chip_type (prefex with sm750 in order to make the context of the function clear). This issue was found by checkpatch.pl Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 16 d

Re: [PATCH] staging: sm750fb: rename getChipType to get_chip_type

On Fri, Sep 23, 2016 at 02:13:52PM +0200, Greg KH wrote: > On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote: > > Rename CamelCased function getChipType to get_chip_type. > > This issue was found by checkpatch.pl > > As this is a global function, can you rename

[PATCH] staging: sm750fb: rename getChipType to get_chip_type

Rename CamelCased function getChipType to get_chip_type. This issue was found by checkpatch.pl Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 16 drivers/staging/sm750fb/ddk750_chip.h | 2 +- drivers/staging/sm750fb/ddk750_

[PATCH 2/2] staging: sm750fb: fix block comment style and spelling issues in ddk750_chip.c

Fix the following warning types: - line length - block comment line * prefix - trailing */ on a separate line found by the checkpatch.pl tool in multiple block comments. Fix a single spelling error in a comment. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/s

[PATCH 1/2] staging: sm750fb: fix line length coding style issues in ddk750_chip.c

Fix multiple line length warnings found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/drivers/staging/s

Re: [PATCH 2/2] staging: sm750fb: fix block comment style and spelling issues in ddk750_chip.c

On Mon, Sep 12, 2016 at 01:17:25PM +0200, Greg KH wrote: > On Sun, Sep 04, 2016 at 09:04:10PM +0300, Moshe Green wrote: > > Fix the following warning types: > > - line length > > - block comment line * prefix > > - trailing */ on a separate line > > found by t

Re: [PATCH 1/2] staging: sm750fb: fix line length coding style issues in ddk750_chip.c

On Mon, Sep 12, 2016 at 01:16:35PM +0200, Greg KH wrote: > On Sun, Sep 04, 2016 at 09:03:27PM +0300, Moshe Green wrote: > > Fix multiple line length warnings found by the checkpatch.pl tool > > in ddk750_chip.c. > > > > Signed-off-by: Moshe Green <mgmos...@gmail.com

[PATCH 2/2] staging: sm750fb: fix block comment style and spelling issues in ddk750_chip.c

Fix the following warning types: - line length - block comment line * prefix - trailing */ on a separate line found by the checkpatch.pl tool in multiple block comments. Fix a single spelling error in a comment. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/s

[PATCH 1/2] staging: sm750fb: fix line length coding style issues in ddk750_chip.c

Fix multiple line length warnings found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/drivers/staging/s

Re: [PATCH 1/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

lude quotations after my reply? > > > http://daringfireball.net/2007/07/on_top > > On Fri, Sep 02, 2016 at 11:28:51AM +0300, moshe green wrote: > > There were (mostly) two kinds of warnings that I've handled, "80 > > character line length" and block comments str

Re: [PATCH 1/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

at a time? Or should I fix both warnings in the same commit - and place the commit in the most appropriate patch? thanks Moshe Green On 1 September 2016 at 19:04, Greg KH <gre...@linuxfoundation.org> wrote: > On Tue, Aug 30, 2016 at 10:04:02PM +0300, Moshe Green wrote: >> Fix a

Re: [PATCH 7/9] staging: sm750fb: fix block comment style issues in ddk750_chip.c

Will do. Thanks On 30 August 2016 at 22:19, Joe Perches <j...@perches.com> wrote: > On Tue, 2016-08-30 at 22:05 +0300, Moshe Green wrote: >> Fix block comment * prefix and trailing */ warnings found >> by the checkpatch.pl tool in ddk750_chip.c. > [] >> diff -

[PATCH 9/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

Fix a line length warning found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/s

[PATCH 8/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

Fix a line length warning found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/s

[PATCH 7/9] staging: sm750fb: fix block comment style issues in ddk750_chip.c

Fix block comment * prefix and trailing */ warnings found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/s

[PATCH 6/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

Fix a line length warning found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/s

[PATCH 4/9] staging: sm750fb: fix coding style issues in ddk750_chip.c

Fix line length and block comment * prefix warnings found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/s

[PATCH 1/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

Fix a line length warning found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/s

[PATCH 3/9] staging: sm750fb: fix coding style issues in ddk750_chip.c

Fix line length and block comment * prefix warnings found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/s

[PATCH 2/9] staging: sm750fb: fix line length coding style issue in ddk750_chip.c

Fix a line length warning found by the checkpatch.pl tool in ddk750_chip.c. Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/s

[PATCH] Staging: sm750fb: fix block comment coding style issue in ddk750_chip.c

This is a patch to the ddk750_chip.c file that fixes up two block comment coding style warnings found by the checkpatch.pl tool Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff

[PATCH 0/4] Lustre fixes and cleanups

From: Oleg Drokin Here are a few more Lustre fixes and cleanups. Jinshan Xiong (2): staging/lustre/lov: calculate file offset correctly staging/lustre/llite: define per open file cache for ll_cl_context Oleg Drokin (2): staging/lustre/osc: Remove ops_temp from

[PATCH 3/4] staging/lustre/osc: Remove ops_temp from osc_page

From: Oleg Drokin It's no longer used and never set anywhere. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/osc/osc_cl_internal.h | 5 - drivers/staging/lustre/lustre/osc/osc_page.c| 2 +- 2 files changed, 1 insertion(+),

[PATCH 2/4] staging/lustre/llite: define per open file cache for ll_cl_context

From: Jinshan Xiong In ll_readpage and ll_write_begin, it needs to find out the cl_env and cl_io, a.k.a ll_cl_context, when the IO is initialized. It used to call cl_env_get() to figure it out but turned out to be contended if multiple threads are doing IO. In this

[PATCH 4/4] staging/lustre/osc: Get rid of osc_page_protected()

From: Oleg Drokin There was a proper debugging function by that name that's long gone. The currently remaining shadow that always returns true is not really useful so it could be dropped along with all the asserts it is part of. Signed-off-by: Oleg Drokin

[PATCH 1/4] staging/lustre/lov: calculate file offset correctly

From: Jinshan Xiong In lov_stripe_pgoff(), it calls lov_stripe_size() to calculate the file size by ost_size, which will be wrong if the stripe_index happens to be stripe aligned. Signed-off-by: Jinshan Xiong Reviewed-on:

[PATCH] Staging: sm750fb: fix line length coding style issue in ddk750_chip.c

This is a patch to the ddk750_chip.c file that fixes up a line length warning found by the checkpatch.pl tool Signed-off-by: Moshe Green <mgmos...@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/s

[PATCH 2/2] staging/lustre/llite: suppress non active IO error message

From: Bobi Jam Current CLIO does not support fadvise, suppress the error message. Signed-off-by: Bobi Jam Reviewed-on: http://review.whamcloud.com/9658 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4717 Reviewed-by: Jinshan Xiong

[PATCH 0/2] Couple of lustre cleanups

From: Oleg Drokin These are rebased cleanups from before - style fixes and suppressign a spurious message. Bobi Jam (1): staging/lustre/llite: suppress non active IO error message Oleg Drokin (1): staging/lustre: Fix blank line after/before {/} style

[PATCH 1/2] staging/lustre: Fix blank line after/before {/} style

From: Oleg Drokin This patch fixes all checkpatch occurences of CHECK: Blank lines aren't necessary after an open brace '{' CHECK: Blank lines aren't necessary before a close brace '}' in Lustre code. Signed-off-by: Oleg Drokin ---

[PATCH 1/3] staging/lustre: Fix braces {} style

From: Oleg Drokin This fixes all checkpatch form of this from the Lustre tree: CHECK: braces {} should be used on all arms of this statement Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fld/fld_cache.c| 3 ++-

[PATCH 3/3] staging/lustre: Get rid of ldlm_policy_res_t typedef

From: Oleg Drokin Directly use enum ldlm_policy_res everywhere. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 - drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 49 +++--- 2 files

[PATCH 2/3] staging/lustre: Fix blank line after/before {/} style

From: Oleg Drokin This patch fixes all checkpatch occurences of CHECK: Blank lines aren't necessary after an open brace '{' CHECK: Blank lines aren't necessary before a close brace '}' in Lustre code. Signed-off-by: Oleg Drokin ---

[PATCH 0/3] A few more Lustre styel cleanups

From: Oleg Drokin More of the checkpatch warning fixes. Oleg Drokin (3): staging/lustre: Fix braces {} style staging/lustre: Fix blank line after/before {/} style staging/lustre: Get rid of ldlm_policy_res_t typedef drivers/staging/lustre/lustre/fld/fld_cache.c

[PATCH v2 21/46] staging/lustre/llite: clip page correctly for vvp_io_commit_sync

From: Jinshan Xiong The original code was wrong which clipped page incorrectly for partial pages started with zero. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/8531 Intel-bug-id:

[PATCH v2 28/46] staging/lustre/llite: rename ccc_lock to vvp_lock

From: "John L. Hammond" Rename struct ccc_lock to struct vvp_lock and merge the CCC lock methods into the VVP lock methods. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13088 Intel-bug-id:

[PATCH v2 11/46] staging/lustre/clio: remove stackable cl_page completely

From: Jinshan Xiong >From now on, cl_page becomes one to one mapping of vmpage. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7895 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Bobi Jam

[PATCH v2 45/46] staging/lustre/ldlm: Solve a race for LRU lock cancel

From: Vitaly Fertman This patch solves a race condition that the lock may be used again after LRU cancellation policy check. In that case, the lock may have locked or dirty pages that makes the policy check totally useless. The problem is solved by checking

[PATCH v2 44/46] staging/lustre: Fix spacing style before open parenthesis

From: Oleg Drokin This fixes the remaining occurences of checkpatch warnings of the form of WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lu_object.h |

[PATCH v2 46/46] staging/lustre: lov_io_init() should return error code

From: Bobi Jam lov_io_init_empty/release() should returns error code instead of true on error case. Fault IO needs to handle restart in the case of accessing HSM released file Signed-off-by: Bobi Jam Reviewed-on: http://review.whamcloud.com/17240

[PATCH v2 42/46] staging/lustre/ldlm: revert changes to ldlm_cancel_aged_policy()

From: Niu Yawei The changes to ldlm_cancel_aged_policy() introduced from LU-4300 was incorrect. This patch revert this part of changes. Signed-off-by: Niu Yawei Reviewed-on: http://review.whamcloud.com/12448 Intel-bug-id:

[PATCH v2 43/46] staging/lustre/ldlm: restore the ELC for enqueue

From: Vitaly Fertman after LU-4300 enqueue does not ELC anymore, however if enqueue is agressive (ls -la of a large dir) we may exceed lru-resize limit quickly because LRUR shrinker and recalc are called not so often. ELC is to be restored in enqueue. ELC also should

[PATCH v2 39/46] staging/lustre/llite: Move several declarations to llite_internal.h

From: John Hammond Move several declarations between llite_internal.h and vvp_internal.h with the goal of reserving the latter header for functions that pertain to vvp_{device,object,page,...}. Signed-off-by: John L. Hammond Signed-off-by:

[PATCH v2 40/46] staging/lustre/llite: Remove unused vui_local_lock field

From: Oleg Drokin vvp_io_setattr_lock is the only user that sets it, but it's never checked anywhere, so could go away. Also get rid of enum ccc_setattr_lock_type that becomes unused. Signed-off-by: Oleg Drokin ---

[PATCH v2 41/46] staging/lustre/ldlm: ELC picks locks in a safer policy

From: Jinshan Xiong Change the policy of ELC to pick locks that have no dirty pages, no page in writeback state, and no locked pages. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/9175 Intel-bug-id:

[PATCH v2 09/46] staging/lustre/clio: add pages into writeback cache in batches

From: Jinshan Xiong in ll_write_end(), instead of adding the page into writeback cache directly, it will be held in a page list. After enough pages have been collected, issue them all with cio_commit_async(). Signed-off-by: Jinshan Xiong

[PATCH v2 38/46] staging/lustre/llite: Move ll_dirent_type_get and make it static

From: Oleg Drokin ll_dirent_type_get is only used in one place in llite/dir.c, so move it there. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/dir.c | 22 ++

[PATCH v2 37/46] staging/lustre/llite: Remove ccc_global_{init, fini}()

From: John Hammond Merge their contents into vvp_global_{init,fini}() and {init,exit}_lustre_lite(). Rename ccc_inode_fini_* to cl_inode_fini_*. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong Reviewed-on:

[PATCH v2 36/46] staging/lustre/llite: rename struct ccc_thread_info to vvp_thread_info

From: John Hammond struct ccc_thread_info is used in the VVP parts of llite so rename it struct vvp_thread_info. Rename supporting functions accordingly. Move init code from lcommon_cl.c to vvp_dev.c Signed-off-by: John L. Hammond Signed-off-by:

[PATCH v2 26/46] staging/lustre/llite: rename ccc_object to vvp_object

From: "John L. Hammond" Rename struct ccc_object to struct vvp_object and merge the CCC object methods into the VVP object methods. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13077 Intel-bug-id:

[PATCH v2 34/46] staging/lustre/llite: Rename struct ccc_grouplock to ll_grouplock

From: John Hammond And move the definition from vvp_internal.h to llite_internal.h. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/13714 Intel-bug-id:

[PATCH v2 29/46] staging/lustre:llite: remove struct ll_ra_read

From: "John L. Hammond" Ever since removal of the the unused function ll_ra_read_get(), the struct ll_ra_read members lrr_reader and lrr_linkage and the struct ll_readahead_state member ras_read_beads unnecessary so remove them. In struct vvp_io replace the struct

[PATCH v2 32/46] staging/lustre/llite: move vvp_io functions to vvp_io.c

From: "John L. Hammond" Move all vvp_io related functions from lustre/llite/lcommon_cl.c to the sole file where they are used lustre/llite/vvp_io.c. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13376 Intel-bug-id:

[PATCH v2 35/46] staging/lustre/llite: Rename struct vvp_thread_info to ll_thread_info

From: John Hammond struct vvp_thread_info is used in the non-VVP parts of llite so rename it struct ll_thread_info. Rename supporting functions accordingly. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong

[PATCH v2 30/46] staging/lustre/llite: merge ccc_io and vvp_io

From: "John L. Hammond" Move the contents of struct vvp_io into struct ccc_io, delete the former, and rename the latter to struct vvp_io. Rename various ccc_io related functions to use vvp rather than ccc. Signed-off-by: John L. Hammond

[PATCH v2 31/46] staging/lustre/llite: use vui prefix for struct vvp_io members

From: "John L. Hammond" Rename members of struct vvp_io to used to start with vui_ rather than cui_. Rename several instances of struct vvp_io * from cio to vio. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13363

[PATCH v2 33/46] staging/lustre/llite: rename ccc_req to vvp_req

From: "John L. Hammond" Rename struct ccc_req to struct vvp_req and move related functions from lustre/llite/lcommon_cl.c to the new file lustre/llite/vvp_req.c. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13377

[PATCH v2 27/46] staging/lustre/llite: rename ccc_page to vvp_page

From: "John L. Hammond" Rename struct ccc_page to struct vvp_page and remove obsolete CCC page methods. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13086 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5971

[PATCH v2 24/46] staging/lustre/llite: merge lclient.h into llite/vvp_internal.h

From: "John L. Hammond" Move the definition of struct cl_client_cache to lustre/include/cl_object.h and move the rest of lustre/include/lclient.h in to lustre/llite/vvp_internal.h. Signed-off-by: John L. Hammond Reviewed-on:

[PATCH v2 23/46] staging/lustre/llite: make sure we do cl_page_clip on the last page

From: Li Dongyang When we are doing a partial IO on both first and last page, the logic currently only call cl_page_clip on the first page, which will end up with a incorrect i_size. Signed-off-by: Li Dongyang Reviewed-on:

[PATCH v2 25/46] staging/lustre/llite: rename ccc_device to vvp_device

From: "John L. Hammond" Rename struct ccc_device to struct vvp_device and merge the CCC device methods into the VVP device methods. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13075 Intel-bug-id:

[PATCH v2 17/46] staging/lustre: Remove struct ll_iattr

From: Oleg Drokin This was a compat code from the time it had ia_attr_flags. Instead convert all the cryptic callers that did ((struct ll_iattr *)_data->op_attr)->ia_attr_flags into direct access to op_data->op_attr_flags This also makes lustre/include/linux/obd.h not

[PATCH v2 15/46] staging/lustre/obd: remove struct client_obd_lock

From: "John L. Hammond" Remove the definition of struct client_obd_lock and the functions client_obd_list_{init,lock,unlock,done}(). Use spinlock_t for the cl_{loi,lru}_list_lock members of struct client_obd and call spin_{lock,unlock}() directly. Signed-off-by: John L.

[PATCH v2 22/46] staging/lustre/llite: deadlock for page write

From: Jinshan Xiong Writing thread already locked page #1, and then wait for the Writeback bit of page #2; Ptlrpc thread is composing a write RPC, so it sets Writeback on page #2 and tries to lock page #1 to make it ready. Deadlocked. Signed-off-by: Jinshan Xiong

[PATCH v2 20/46] staging/lustre: update comments after cl_lock simplification

From: Bobi Jam Update comments to reflect current cl_lock situations. Signed-off-by: Bobi Jam Reviewed-on: http://review.whamcloud.com/13137 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6046 Reviewed-by: John L. Hammond

[PATCH v2 18/46] staging/lustre/clio: generalize cl_sync_io

From: Jinshan Xiong To make cl_sync_io interfaces not just wait for pages, but to be a generic synchronization mechanism. Also remove cl_io_cancel that became not used. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/8656

[PATCH v2 13/46] staging/lustre/llite: remove lli_lvb

From: "John L. Hammond" In struct ll_inode_info remove the struct ost_lvb lli_lvb member and replace it with obd_time lli_{a,m,c}time. Rename ll_merge_lvb() to ll_merge_attr(). Remove cl_merge_lvb() and replace calls to it with calls to ll_merge_attr(). Signed-off-by:

[PATCH v2 14/46] staging/lustre/lmv: remove lmv_init_{lock,unlock}()

From: "John L. Hammond" In struct lmv_obd rename the init_mutex member to lmv_init_mutex. Remove the compat macros lmv_init_{lock,unlock}() and use mutex_{lock,unlock}(>lmv_init_mutex) instead. Signed-off-by: John L. Hammond Reviewed-on:

[PATCH v2 16/46] staging/lustre/llite: remove some cl wrappers

From: "John L. Hammond" In llite remove the wrapper functions and macros: cl_i2info() cl_i2sbi() cl_iattr2fd() cl_inode_info cl_inode_mode() cl_inode_{a,m,c}time() cl_isize_{read,write,write_nolock}() Signed-off-by: John L. Hammond

[PATCH v2 10/46] staging/lustre/osc: add weight function for DLM lock

From: Jinshan Xiong Use weigh_ast to decide if a lock covers any pages. In recovery, weigh_ast will be used to decide if a DLM read lock covers any locked pages, or it will be canceled instead being recovered. The problem with the original implementation is that it

[PATCH v2 12/46] staging/lustre/clio: optimize read ahead code

From: Jinshan Xiong It used to check each page in the readahead window is covered by a lock underneath, now cpo_page_is_under_lock() provides @max_index to help decide the maximum ra window. @max_index can be modified by OSC to extend the maximum lock region, to align

[PATCH v2 03/46] staging/lustre: merge lclient/*.c into llite/

From: "John L. Hammond" Separate lclient was necessary to be shared between different client implementations, make no sense to have them separate in Linux kernel. Signed-off-by: John L. Hammond Based-on: http://review.whamcloud.com/10171

[PATCH v2 07/46] staging/lustre/clio: collapse layer of cl_page

From: Jinshan Xiong Move radix tree to osc layer to for performance improvement. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7892 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Lai Siyao

[PATCH v2 08/46] staging/lustre/obdclass: Add a preallocated percpu cl_env

From: Jinshan Xiong This change adds support for a single preallocated cl_env per CPU which can be used in circumstances where reschedule is not possible. Currently this interface is only used by the ll_releasepage function. Signed-off-by: Jinshan Xiong

[PATCH v2 02/46] staging/lustre: Get rid of CFS_PAGE_MASK

From: Oleg Drokin CFS_PAGE_MASK is the same as PAGE_MASK, so get rid of it. We are replacing it with PAGE_MASK instead of PAGE_CACHE_MASK because PAGE_CACHE_* stuff is apparently going away. Signed-off-by: Oleg Drokin ---

[PATCH v2 04/46] staging/lustre: Reintroduce global env list

From: Jinshan Xiong This reverts a patch that was merged before lustre client was introduced into the stagign tree, so it's not in the history. The performance dropped a lot when memory reclaim process kicked in as ll_releasepage() was called to destroy lustre pages. It

[PATCH v2 05/46] staging/lustre/osc: Adjustment on osc LRU for performance

From: Jinshan Xiong Add and discard pages from LRU in batch. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7890 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Niu Yawei

[PATCH v2 01/46] staging/lustre/obdclass: limit lu_site hash table size

From: Oleg Drokin Allocating a big hash table using the formula for osd does not really work for clients. We will create new hash table for each mount on a single client which is a lot of memory more than expected. This patch limits the hash table up to 8M which has 524288

[PATCH v2 06/46] staging/lustre/osc: to drop LRU pages with cl_lru_work

From: Jinshan Xiong This way we can drop it async. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7891 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Lai Siyao Reviewed-by:

[PATCH v2 00/46] Lustre IO stack simplifications and cleanups

From: Oleg Drokin v2 due to a compile error that was accidentally introduced in one of the patches. Also added a few more fixes and cleanups as three additional patches at the end of the series. This large body of patches mostly deals with Lustre IO stack simplifications

[PATCH 43/43] staging/lustre/ldlm: restore the ELC for enqueue

From: Vitaly Fertman after LU-4300 enqueue does not ELC anymore, however if enqueue is agressive (ls -la of a large dir) we may exceed lru-resize limit quickly because LRUR shrinker and recalc are called not so often. ELC is to be restored in enqueue. ELC also should

[PATCH 06/43] staging/lustre/osc: to drop LRU pages with cl_lru_work

From: Jinshan Xiong This way we can drop it async. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7891 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Lai Siyao Reviewed-by:

[PATCH 05/43] staging/lustre/osc: Adjustment on osc LRU for performance

From: Jinshan Xiong Add and discard pages from LRU in batch. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/7890 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3321 Reviewed-by: Niu Yawei

[PATCH 39/43] staging/lustre/llite: Move several declarations to llite_internal.h

From: John Hammond Move several declarations between llite_internal.h and vvp_internal.h with the goal of reserving the latter header for functions that pertain to vvp_{device,object,page,...}. Signed-off-by: John L. Hammond Signed-off-by:

[PATCH 33/43] staging/lustre/llite: rename ccc_req to vvp_req

From: "John L. Hammond" Rename struct ccc_req to struct vvp_req and move related functions from lustre/llite/lcommon_cl.c to the new file lustre/llite/vvp_req.c. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13377

[PATCH 42/43] staging/lustre/ldlm: revert changes to ldlm_cancel_aged_policy()

From: Niu Yawei The changes to ldlm_cancel_aged_policy() introduced from LU-4300 was incorrect. This patch revert this part of changes. Signed-off-by: Niu Yawei Reviewed-on: http://review.whamcloud.com/12448 Intel-bug-id:

[PATCH 32/43] staging/lustre/llite: move vvp_io functions to vvp_io.c

From: "John L. Hammond" Move all vvp_io related functions from lustre/llite/lcommon_cl.c to the sole file where they are used lustre/llite/vvp_io.c. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13376 Intel-bug-id:

[PATCH 34/43] staging/lustre/llite: Rename struct ccc_grouplock to ll_grouplock

From: John Hammond And move the definition from vvp_internal.h to llite_internal.h. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/13714 Intel-bug-id:

[PATCH 37/43] staging/lustre/llite: Remove ccc_global_{init,fini}()

From: John Hammond Merge their contents into vvp_global_{init,fini}() and {init,exit}_lustre_lite(). Rename ccc_inode_fini_* to cl_inode_fini_*. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong Reviewed-on:

[PATCH 30/43] staging/lustre/llite: merge ccc_io and vvp_io

From: "John L. Hammond" Move the contents of struct vvp_io into struct ccc_io, delete the former, and rename the latter to struct vvp_io. Rename various ccc_io related functions to use vvp rather than ccc. Signed-off-by: John L. Hammond

[PATCH 24/43] staging/lustre/llite: merge lclient.h into llite/vvp_internal.h

From: "John L. Hammond" Move the definition of struct cl_client_cache to lustre/include/cl_object.h and move the rest of lustre/include/lclient.h in to lustre/llite/vvp_internal.h. Signed-off-by: John L. Hammond Reviewed-on:

[PATCH 40/43] staging/lustre/llite: Remove unused vui_local_lock field

From: Oleg Drokin vvp_io_setattr_lock is the only user that sets it, but it's never checked anywhere, so could go away. Also get rid of enum ccc_setattr_lock_type that becomes unused. Signed-off-by: Oleg Drokin ---

[PATCH 31/43] staging/lustre/llite: use vui prefix for struct vvp_io members

From: "John L. Hammond" Rename members of struct vvp_io to used to start with vui_ rather than cui_. Rename several instances of struct vvp_io * from cio to vio. Signed-off-by: John L. Hammond Reviewed-on: http://review.whamcloud.com/13363

[PATCH 36/43] staging/lustre/llite: rename struct ccc_thread_info to vvp_thread_info

From: John Hammond struct ccc_thread_info is used in the VVP parts of llite so rename it struct vvp_thread_info. Rename supporting functions accordingly. Move init code from lcommon_cl.c to vvp_dev.c Signed-off-by: John L. Hammond Signed-off-by:

[PATCH 41/43] staging/lustre/ldlm: ELC picks locks in a safer policy

From: Jinshan Xiong Change the policy of ELC to pick locks that have no dirty pages, no page in writeback state, and no locked pages. Signed-off-by: Jinshan Xiong Reviewed-on: http://review.whamcloud.com/9175 Intel-bug-id:

[PATCH 35/43] staging/lustre/llite: Rename struct vvp_thread_info to ll_thread_info

From: John Hammond struct vvp_thread_info is used in the non-VVP parts of llite so rename it struct ll_thread_info. Rename supporting functions accordingly. Signed-off-by: John L. Hammond Signed-off-by: Jinshan Xiong

[PATCH 38/43] staging/lustre/llite: Move ll_dirent_type_get and make it static

From: Oleg Drokin ll_dirent_type_get is only used in one place in llite/dir.c, so move it there. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/dir.c | 22 ++

  1   2   3   4   5   6   7   8   >