Re: [PATCH] rtl8xxxu: Stop log spam from each successful interrupt

2016-09-18 Thread Kalle Valo
Jes Sorensen writes: > Joe Perches writes: >> On Sat, 2016-09-17 at 12:09 -0500, Larry Finger wrote: >>> As soon as debugging is turned on, the logs are filled with messages >>> reporting the interrupt status. As this quantity is usually zero, this >>> output is not needed. In fact, there will b

[PATCH v2] stagging:iio:ad9834: add devicetree property support

2016-09-18 Thread Gwenhael Goavec-Merou
ad9834 driver needs some default properties. Currently these parameters are provided through platform_data. This patch adds a function to create this pdata based on device-tree node. Signed-off-by: Gwenhael Goavec-Merou --- Changes v1 -> v2: - use clock bindings for input clock (Lars-Pete

Re: [PATCH] stagging:iio:ad9834: add devicetree property support

2016-09-18 Thread Gwenhael Goavec-Merou
Hi, Thanks for the review. I will send a new patch. Gwenhael On 12/09/2016 14:02, Lars-Peter Clausen wrote: Hi, Thanks for the patch. On 09/11/2016 12:52 PM, Gwenhael Goavec-Merou wrote: +static struct ad9834_platform_data *ad9834_parse_dt(struct spi_device *spi) +{ + struct ad9834_plat

[PATCH] staging: ks7010: clean function declaration in ks_hostif.c up

2016-09-18 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/staging/ks7010/ks_wlan_net.c:3392:6: warning: no previous prototype for 'send_packet_complete' [-Wmissing-prototypes] In fact, this function is declared in drivers/staging/ks7010/ks_hostif.c, but should be declared in a header file. thus can

[PATCH] staging: ks7010: clean function declarations in ks_hostif.c up

2016-09-18 Thread Baoyou Xie
We get 2 warnings when building kernel with W=1: drivers/staging/ks7010/ks7010_sdio.c:152:6: warning: no previous prototype for 'ks_wlan_hw_wakeup_request' [-Wmissing-prototypes] drivers/staging/ks7010/ks7010_sdio.c:255:5: warning: no previous prototype for 'ks_wlan_hw_power_save' [-Wmissing-prot

[PATCH] staging: ks7010: move ks_wlan_hw_tx() declaration to header file

2016-09-18 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/staging/ks7010/ks7010_sdio.c:363:5: warning: no previous prototype for 'ks_wlan_hw_tx' [-Wmissing-prototypes] In fact, this function is declared in drivers/staging/ks7010/ks_wlan_net.c and drivers/staging/ks7010/ks_hostif.c, but should be de

[PATCH] staging: lustre: lmv: add missing function declaration

2016-09-18 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/staging/lustre/lustre/lmv/lmv_obd.c:2774:5: warning: no previous prototype for 'lmv_pack_md' [-Wmissing-prototypes] In fact, this function is not declared in any file,but should be declared in a header file, thus can be recognized in other f

[PATCH] vme: mark symbols static where possible

2016-09-18 Thread Baoyou Xie
We get 4 warnings when building kernel with W=1: drivers/vme/bridges/vme_fake.c:374:6: warning: no previous prototype for 'fake_lm_check' [-Wmissing-prototypes] drivers/vme/bridges/vme_fake.c:609:6: warning: no previous prototype for 'fake_vmewrite8' [-Wmissing-prototypes] drivers/vme/bridges/vme

[PATCH] staging: ks7010: fixes typo in ks_hostif.c

2016-09-18 Thread Hariharan R
Fixes typo, FAILUER -> FAILURE Recieve -> Receive Signed-off-by: Hariharan R --- drivers/staging/ks7010/ks_hostif.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index a8822fe..8a18c85 100644 ---

Re: [PATCH] rtl8xxxu: Stop log spam from each successful interrupt

2016-09-18 Thread Jes Sorensen
Larry Finger writes: > On 09/17/2016 03:59 PM, Jes Sorensen wrote: >> Larry Finger writes: >>> As soon as debugging is turned on, the logs are filled with messages >>> reporting the interrupt status. As this quantity is usually zero, this >>> output is not needed. In fact, there will be a report

Re: [PATCH] rtl8xxxu: Stop log spam from each successful interrupt

2016-09-18 Thread Jes Sorensen
Kalle Valo writes: > Jes Sorensen writes: > >> Joe Perches writes: >>> I think it'd be nicer to use dev_dbg for all these cases >>> and as well use some new macro that includes the test >>> >>> Something like: >>> >>> #define rtl8xxxu_dbg(type, fmt, ...)\ >>> do {

[PATCH 01/11] staging: r8188eu: change last argument type of the usb_write_port function

2016-09-18 Thread Ivan Safonov
usb_write_port writes only xmit_buf object data to device. In addition, an appropriate name for this argument is used. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c| 4 ++-- drivers/staging/rtl8188eu/include/usb_ops_linux.h | 2 +- drivers/staging/rtl8188eu

[PATCH 03/11] staging: r8188eu: remove pkt_hdrlen member of pkt_attrib structure

2016-09-18 Thread Ivan Safonov
pkt_hdrlen has a constant value. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_xmit.c| 4 +--- drivers/staging/rtl8188eu/include/rtw_xmit.h | 1 - 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c b/drivers/staging/r

[PATCH 04/11] staging: r8188eu: delete rtw_usb_bulk_size_boundary function

2016-09-18 Thread Ivan Safonov
This function does not used. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/usb_ops_linux.h | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/usb_ops_linux.h b/drivers/staging/rtl8188eu/include/usb_ops_linux.h index fde7753.

[PATCH 02/11] staging: r8188eu: remove xmitframe_direct function

2016-09-18 Thread Ivan Safonov
xmitframe_direct is a simple wrapper around rtw_xmitframe_coalesce and rtw_dump_xframe functions. Many wrappers complicates code reading. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) dif

[PATCH 08/11] staging: r8188eu: remove HW_VAR_DM_FLAG member of hw_variables enumeration

2016-09-18 Thread Ivan Safonov
rtw_hal_set_hwreg and rtw_hal_get_hwreg does not used with HW_VAR_DM_FLAG parameter. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 7 --- drivers/staging/rtl8188eu/include/hal_intf.h | 1 - 2 files changed, 8 deletions(-) diff --git a/drivers/staging/rtl8188

[PATCH 07/11] staging: r8188eu: remove usb_hal.h

2016-09-18 Thread Ivan Safonov
usb_hal.h is empty. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/hal_intf.c| 1 - drivers/staging/rtl8188eu/hal/usb_halinit.c | 1 - drivers/staging/rtl8188eu/include/usb_hal.h | 18 -- drivers/staging/rtl8188eu/os_dep/os_intfs.c | 2 -- drivers/staging/rt

[PATCH 06/11] staging: r8188eu: remove rtl8188eu_set_hal_ops function

2016-09-18 Thread Ivan Safonov
rtl8188eu_set_hal_ops only allocates HalData member of adapter structure. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 7 --- drivers/staging/rtl8188eu/include/usb_hal.h | 2 -- drivers/staging/rtl8188eu/os_dep/usb_intf.c | 7 +-- 3 files changed, 5 inser

[PATCH 10/11] staging: r8188eu: remove GET_RF_TYPE macro

2016-09-18 Thread Ivan Safonov
This macro does not used. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/rtl8188e_hal.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_hal.h b/drivers/staging/rtl8188eu/include/rtl8188e_hal.h index 418bdb9..5362218 100644 --- a/d

[PATCH 11/11] staging: r8188eu: remove GET_HAL_DATA macro

2016-09-18 Thread Ivan Safonov
GET_HAL_DATA replaced by its definition. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_sreset.c | 9 +-- drivers/staging/rtl8188eu/hal/bb_cfg.c| 13 ++-- drivers/staging/rtl8188eu/hal/odm.c | 17 ++--- drivers/staging/rtl8188eu/hal/phy.c

[PATCH 05/11] staging: r8188eu: replace N_BYTE_ALIGMENT macro with PTR_ALIGN

2016-09-18 Thread Ivan Safonov
PTR_ALIGN is a bit shorter than N_BYTE_ALIGMENT. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_recv.c | 2 +- drivers/staging/rtl8188eu/core/rtw_xmit.c | 6 +++--- drivers/staging/rtl8188eu/include/basic_types.h | 4 drivers/staging/rtl8188eu/os_dep/xmit_lin

[PATCH 09/11] staging: r8188eu: set correct type for HalData member of adapter structure

2016-09-18 Thread Ivan Safonov
To avoid unnecessary typecast. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/include/drv_types.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/drv_types.h b/drivers/staging/rtl8188eu/include/drv_types.h index 7af690e..5c275fb 1

[PATCH] Staging: rts5208: rtsx_card.c: Fixed brace style issues

2016-09-18 Thread Cathal Mullaney
Fixed several minor brace coding style issues. Signed-off-by: Cathal Mullaney --- drivers/staging/rts5208/rtsx_card.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/rts5208/rtsx_card.c b/drivers/staging/rts5208/rtsx_card.c index

Re: [PATCH] staging: lustre: lmv: add missing function declaration

2016-09-18 Thread James Simmons
> We get 1 warning when building kernel with W=1: > drivers/staging/lustre/lustre/lmv/lmv_obd.c:2774:5: warning: no previous > prototype for 'lmv_pack_md' [-Wmissing-prototypes] > > In fact, this function is not declared in any file,but should be > declared in a header file, thus can be recogniz

Re: [PATCH] staging: lustre: lustre/ldlm: Fixed sparse warnings

2016-09-18 Thread nayeem
On Friday 16 September 2016 01:30 PM, Dilger, Andreas wrote: On Sep 15, 2016, at 12:33, nayeem wrote: On Wednesday 14 September 2016 10:44 AM, Dilger, Andreas wrote: On Sep 12, 2016, at 04:27, Greg KH wrote: On Fri, Sep 09, 2016 at 08:50:35PM +0530, Nayeemahmed Badebade wrote: Added __ac

Re: [PATCH] staging: lustre: lustre/ldlm: Fixed sparse warnings

2016-09-18 Thread Dilger, Andreas
On Sep 18, 2016, at 14:21, nayeem wrote: > On Friday 16 September 2016 01:30 PM, Dilger, Andreas wrote: >> On Sep 15, 2016, at 12:33, nayeem wrote: >>> On Wednesday 14 September 2016 10:44 AM, Dilger, Andreas wrote: On Sep 12, 2016, at 04:27, Greg KH wrote: > > On Fri, Sep 09, 2016

Re: [PATCH] staging: lustre: lmv: mark symbols static where possible

2016-09-18 Thread Dilger, Andreas
On Sep 17, 2016, at 06:04, Baoyou Xie wrote: > > We get a few warnings when building kernel with W=1: > drivers/staging/lustre/lustre/lmv/lmv_obd.c:1640:1: warning: no previous > prototype for 'lmv_locate_target_for_name' [-Wmissing-prototypes] > drivers/staging/lustre/lustre/lmv/lmv_obd.c:2421:

[PATCH 013/124] staging: lustre: osc: update kms in brw_interpret() properly

2016-09-18 Thread James Simmons
From: Niu Yawei In brw_interpret(), we forgot page offset when calculating write offset, that leads to wrong kms for sync write. Signed-off-by: Niu Yawei Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5463 Reviewed-on: http://review.whamcloud.com/11374 Reviewed-by: Bobi Jam Reviewed-by: J

[PATCH 009/124] staging: lustre: obdclass: serialize lu_site purge

2016-09-18 Thread James Simmons
From: Niu Yawei Umount process relies on lu_site_purge(-1) to purge all objects before umount, however, if there happen to have a cache shrinker which calls lu_site_purge(nr) in parallel, some objects may still being freed by cache shrinker even after the lu_site_purge(-1) called by umount done.

[PATCH 015/124] staging: lustre: clio: lu_ref_del() mismatch ref add scope

2016-09-18 Thread James Simmons
From: Bobi Jam 'commit 77605e41a26f ("staging/lustre/clio: add pages into writeback cache in batches")' adds a page to a list aggregate issuing them to writeback cache; A page add is referenced in llite/vvp io scope, while writeback cache commit de-refers it under osc sub io scope, and enabling -

[PATCH 003/124] staging: lustre: ldlm: fix a use after free in ldlm_resource_get()

2016-09-18 Thread James Simmons
From: John L. Hammond If lvbo initialization has failed then save the return status (from lr_lvb_len) before putting the resource. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5305 Reviewed-on: http://review.whamcloud.com/11017 Reviewed-by: Andreas Dilger

[PATCH 004/124] staging: lustre: lmv: honor MDT index when creating volatile file

2016-09-18 Thread James Simmons
From: wang di LMV should honor MDT index embedded in the name of volatile file, then during hsm restore, the file under striped dir can be restored to the right MDT. Signed-off-by: wang di Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4691 Reviewed-on: http://review.whamcloud.com/10866 Re

[PATCH 019/124] staging: lustre: ptlrpc: fix magic return value of ptlrpc_init_portals

2016-09-18 Thread James Simmons
From: Wang Shilong Previously, when running 'modprobe lustre', it hit the following error message which is becaue of network initialisation failure: modprobe: ERROR: could not insert 'lustre': Input/output error However, error code is there, just let it return to caller, after this patch, error

[PATCH 011/124] staging: lustre: llite: update ras stride offset

2016-09-18 Thread James Simmons
From: Bobi Jam When a read ahead does not reach the end of the region reserved from ras, we'd set ras::ras_next_readahead back to where we left off; For stride read ahead, it needs to make sure that the offset is no less than ras_stride_offset, so that the stride read ahead can work correctly. S

[PATCH 021/124] staging: lustre: build: bump build version warnings to x.y.53

2016-09-18 Thread James Simmons
From: Andreas Dilger Move the LUSTRE_VERSION_CODE checks to trigger on x.y.53 instead of x.y.50, so that it is into the development cycle that they are hit instead of right at the start. In many cases, the #warning has been removed (to prevent build errors) and instead the code is just disabled

[PATCH 006/124] staging: lustre: lmv: Do not ignore ENOENT in lmv_unlink

2016-09-18 Thread James Simmons
From: wang di Return correct value (rc) in lmv_unlink. In lmv_unlink, -ENOENT might be ingored for local directory unlink. Signed-off-by: wang di Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5371 Reviewed-on: http://review.whamcloud.com/11170 Reviewed-by: John L. Hammond Reviewed-by: An

[PATCH 053/124] staging: lustre: obd: restore linkea support

2016-09-18 Thread James Simmons
Original linkea was only used for the lustre server code so it was removed from the upstream client. Now it needs to be restored for client work that uses this infrastructure. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/include/lu_object.h |7 + .../staging/lustre/lustre/

[PATCH 028/124] staging: lustre: ldlm: resend AST callbacks

2016-09-18 Thread James Simmons
From: Vitaly Fertman While clients will resend client->server RPCs, servers would not resend server->client RPCs such as LDLM callbacks (blocking or completion callbacks/ASTs). This could result in clients being evicted from the server if blocking callbacks were dropped by the network (a failed

[PATCH 037/124] staging: lustre: cleanup lustre_lib.h

2016-09-18 Thread James Simmons
From: John L. Hammond Remove some unused declarations from lustre_lib.h and move some others to more natural headers. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/11500 Reviewed-by: Andreas Dilger Reviewed-by:

[PATCH 036/124] staging: lustre: llite: no need to check dentry is NULL

2016-09-18 Thread James Simmons
From: John L. Hammond We are already touching dentry in CDEBUG macros so it will crash long before these checks. Since this is the case no need to do an additional check. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud

[PATCH 012/124] staging: lustre: lmv: fix some byte order issues

2016-09-18 Thread James Simmons
From: John L. Hammond In the handler for LL_IOC_LMV_GETSTRIPE convert stripe FIDs from little to CPU endian when unpacking lmv_user_md. In lmv_unpack_md_v1() fix a double conversion of the stripe count. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5342 Revi

[PATCH 027/124] staging: lustre: ldlm: reconstruct proper flags on enqueue resend

2016-09-18 Thread James Simmons
From: Vitaly Fertman otherwise, waiting lock may get granted as no BLOCKED_GRANTED flag is returned Signed-off-by: Vitaly Fertman Xyratex-bug-id: MRP-1944 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5496 Reviewed-on: http://review.whamcloud.com/11644 Reviewed-by: James Simmons Reviewed

[PATCH 007/124] staging: lustre: obd: add lnb_ prefix to members of struct niobuf_local

2016-09-18 Thread James Simmons
From: John L. Hammond Add the prefix lnb_ to the members of struct niobuf_local that do not already have it. Change the struct dentry *lnb_dentry member to void *lnb_data as it is not used to hold a pointer to struct dentry. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.c

[PATCH 029/124] staging: lustre: ldlm: restore some of the interval functionality

2016-09-18 Thread James Simmons
Earlier a bunch of interval handling got removed since it wasn't used by the upstream client. Now some of it is needed again for the client code so this patch restores what is needed. Signed-off-by: James Simmons --- .../staging/lustre/lustre/include/interval_tree.h | 26 + drivers/stagin

[PATCH 002/124] staging: lustre: ptlrpc: enlarge OST_MAXREQSIZE for 4MB RPC

2016-09-18 Thread James Simmons
From: Li Xi This patch enlarges OST_MAXREQSIZE so as to make the request size large enough for 4MB RPC. Signed-off-by: Li Xi Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4755 Reviewed-on: http://review.whamcloud.com/9599 Reviewed-by: Andreas Dilger Reviewed-by: Jinshan Xiong Reviewed-b

[PATCH 024/124] staging: lustre: remove RCU2HANDLE macro

2016-09-18 Thread James Simmons
From: John L. Hammond Remove RCU2HANDLE macro from lustre_handles.h. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/11498 Reviewed-by: Andreas Dilger Reviewed-by: Bob Glossman Signed-off-by: James Simmons ---

[PATCH 025/124] staging: lustre: llite: Compare of unsigned value against 0 is always true

2016-09-18 Thread James Simmons
From: Dmitry Eremin Comparison of unsigned value against 0 is always true. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5200 Reviewed-on: http://review.whamcloud.com/11217 Reviewed-by: John L. Hammond Reviewed-by: Fan Yong Reviewed-by: Oleg Drokin Signed-o

[PATCH 038/124] staging: lustre: osc: debug to match extent to brw RPC

2016-09-18 Thread James Simmons
From: Patrick Farrell Currently, it's difficult to match brw RPCs to objects and extents from client logs. This patch adds a D_RPCTRACE debug message giving the necessary information. Signed-off-by: Patrick Farrell Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5531 Reviewed-on: http://re

[PATCH 057/124] staging: lustre: statahead: ll_intent_drop_lock() called in spinlock

2016-09-18 Thread James Simmons
From: Lai Siyao ll_intent_drop_lock() may sleep, which should not be called inside spinlock. Signed-off-by: Lai Siyao Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2272 Reviewed-on: http://review.whamcloud.com/9665 Reviewed-by: Fan Yong Reviewed-by: James Simmons Reviewed-by: Oleg Droki

[PATCH 071/124] staging: lustre: obd: remove unused obd methods

2016-09-18 Thread James Simmons
From: John L. Hammond Remove no longer used osc_packmd() and osc_getstripe(). Several ioctls cases that are no longer used are removed. Remove no longer used adjust_kms() infrastructure. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2785 Reviewed-on: http://

[PATCH 065/124] staging: lustre: obdclass: change lu_site->ls_purge_start to unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change the type accordant usage. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12384 Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simm

[PATCH 042/124] staging: lustre: obd: change type of lmv_tgt_desc->ltd_idx to u32

2016-09-18 Thread James Simmons
From: Dmitry Eremin ltd_idx is used as unsigned. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/11879 Reviewed-by: John L. Hammond Reviewed-by: Fan Yong Reviewed-by: Oleg Drokin Signed-off-by: James Simmons ---

[PATCH 016/124] staging: lustre: fix comparison between signed and unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Cleanup in general headers. * use size_t in cfs_size_round*() * make unsigned index and len in lustre_cfg_*() * make iteration variable the same type as comparing value * make unsigned pages counters Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/brow

[PATCH 064/124] staging: lustre: lu_dirent_calc_size() return type to size_t

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change the type accordant usage. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12383 Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Signed-off-by: James Simm

[PATCH 063/124] staging: lustre: ldlm: count of pools is unsigned long

2016-09-18 Thread James Simmons
From: Dmitry Eremin Function ldlm_pools_count() return unsigned long but counter is int. Use ldlm_pool_granted() everywhere. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12304 Reviewed-by: James Simmons Reviewed-

[PATCH 062/124] staging: lustre: replace direct HZ access with kernel APIs

2016-09-18 Thread James Simmons
From: Jian Yu On some customer's systems, kernel was compiled with HZ defined to 100, instead of 1000. This improves performance for HPC applications. However, to use these systems with Lustre, customers have to re-build Lustre for the kernel because Lustre directly uses the defined constant HZ.

[PATCH 069/124] staging: lustre: fix messages with missing newlines

2016-09-18 Thread James Simmons
From: John L. Hammond Restore the trailing newline in the definition of OSC_DUMP_GRANT(). Remove an unnecessary CDEBUG() from ldlm_pool_recalc(). Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5551 Reviewed-on: http://review.whamcloud.com/11996 Reviewed-by: J

[PATCH 050/124] staging: lustre: ldlm: per-export lock callback timeout

2016-09-18 Thread James Simmons
From: Vitaly Fertman The lock callback timeout is calculated as an average per namespace. This does not reflect individual client behavior. Instead, we should calculate it on a per-export basis. This is the client side changes for upstream client. Signed-off-by: Vitaly Fertman Intel-bug-id: ht

[PATCH 055/124] staging: lustre: llite: allow setting stripes to specify OSTs

2016-09-18 Thread James Simmons
From: Jinshan Xiong Extend the llite layer to support specifying individual target OSTs. Support specifying OSTs for regular files only. Directory support will be implemented later in a separate project. With this a file could have for example a OST index layout of 2,4,5,9,11. In addition, duplic

[PATCH 056/124] staging: lustre: statahead: use dcache-like interface for sa entry

2016-09-18 Thread James Simmons
From: Lai Siyao Rename ll_sa_entry to sa_entry, and manage sa_entry cache with dcache-like interfaces. sa_entry is not needed to be refcounted, because only scanner can free it, so after it's put in stat list, statahead thread shouldn't access it any longer. ll_statahead_interpret() doesn't nee

[PATCH 018/124] staging: lustre: obdclass: fix comparison between signed and unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Make lu_buf->lb_len unsigned. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5417 Reviewed-on: http://review.whamcloud.com/11281 Reviewed-by: Fan Yong Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons ---

[PATCH 052/124] staging: lustre: lmv: add testing for bad name hash

2016-09-18 Thread James Simmons
From: Fan Yong Enable testing of the lfsck recovery feature in the client code for the case when name hash for some entry becomes corrupt. Signed-off-by: Fan Yong Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5519 Reviewed-on: http://review.whamcloud.com/11846 Reviewed-by: Alex Zhuravlev

[PATCH 000/124] missing patches from Lustre 2.7 release

2016-09-18 Thread James Simmons
This batch of patches includes various cleanups and fixes from the Lustre 2.7 release. Most of these patches are removal of cruft and unsigned to signed comparsion fixes. Also added in the latest LNet fixes as well. Alexander Boyko (3): staging: lustre: at: net AT after connect staging: lustre

[PATCH 070/124] staging: lustre: statahead: small fixes and cleanup

2016-09-18 Thread James Simmons
From: Lai Siyao small fixes: * when 'unplug' is set for ll_statahead(), sa_put() shouldn't kill the entry found, because its inflight RPC may not finish yet. * remove 'sai_generation', add 'lli_sa_generation' because the former one is not safe to access without lock. * revalidate_statahe

[PATCH 060/124] staging: lustre: mdc: fix comparison between signed and unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change type of client_obd->*_mds_*size from int to __u32 and argumanets of related create/rename/setattr functions. Change type of op_data->op_namelen to size_t. Change type of argument size for all mdc_*_pack() to size_t. Signed-off-by: Dmitry Eremin Intel-bug-id: https://j

[PATCH 034/124] staging: lustre: llite: turn mode to umode_t for ll_new_inode()

2016-09-18 Thread James Simmons
From: John L. Hammond Change int mode to umode_t. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/10769 Reviewed-by: Yang Sheng Reviewed-by: Lai Siyao Reviewed-by: Oleg Drokin Signed-off-by: James Simmons ---

[PATCH 066/124] staging: lustre: lov: remove LL_IOC_RECREATE_{FID, OBJ}

2016-09-18 Thread James Simmons
From: John L. Hammond Remove the obsolete ioctls LL_IOC_RECREATE_FID and LL_IOC_RECREATE_OBJ along with their handlers in llite. Remove the then unused OBD method lov_create(). Remove OBD_FL_RECREATE_OBJS handling from osc_create(). Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd

[PATCH 067/124] staging: lustre: changelog: fix comparison between signed and unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change type of changelog_*{namelen,size}() to size_t. Fixed string specifier for unsigned types. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12474 Reviewed-by: James Simmons Reviewed-by: Andr

[PATCH 032/124] staging: lustre: llite: remove lookup_flags from ll_lookup_it()

2016-09-18 Thread James Simmons
From: John L. Hammond Remove the effectively unused lookup_flags parameter from ll_lookup_it(). Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/10769 Reviewed-by: Yang Sheng Reviewed-by: Lai Siyao Reviewed-by: Ol

[PATCH 041/124] staging: lustre: llite: don't call make_bad_inode() on an old inode

2016-09-18 Thread James Simmons
From: John L. Hammond In ll_iget() if ll_update_inode() fails then do not call make_bad_inode() on the inode since it may still be in use. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5468 Reviewed-on: http://review.whamcloud.com/11609 Reviewed-by: Lai Siya

[PATCH 033/124] staging: lustre: llite: remove mode from ll_create_it()

2016-09-18 Thread James Simmons
From: John L. Hammond Remove the unused mode parameter from ll_create_it(). Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/10769 Reviewed-by: Yang Sheng Reviewed-by: Lai Siyao Reviewed-by: Oleg Drokin Signed-of

[PATCH 068/124] staging: lustre: lov: remove unused {get, set}_info handlers

2016-09-18 Thread James Simmons
From: John L. Hammond In LOV and OSC remove handlers for the obsolete get and set info keys: KEY_CAPA_KEY, KEY_CONNECT_FLAG, KEY_EVICT_BY_NID, KEY_LAST_ID, KEY_LOCK_TO_STRIPE, KEY_MDS_CONN, KEY_NEXT_ID. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5814 Revi

[PATCH 080/124] staging: lustre: llite: fix dup flags names

2016-09-18 Thread James Simmons
From: Bob Glossman The name 'xattr' is used for two different ll_flags bits. Change the names to be distinct and different, reflecting the names of the bits used in LL_SBI_xbitnamex #defines. Signed-off-by: Bob Glossman Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5586 Reviewed-on: http:

[PATCH 079/124] staging: lustre: mgc: add nid iteration

2016-09-18 Thread James Simmons
From: Alexander Boyko mgc_apply_recover_logs use only first nid from entry, this could be the problem for a cluster with several network address for a one node. Signed-off-by: Alexander Boyko Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5950 Xyratex-bug-id: MRP-2255 Reviewed-on: http://r

[PATCH 077/124] staging: lustre: ptlrpc: quiet errors on initial connection

2016-09-18 Thread James Simmons
From: Andreas Dilger It may be that a client or MDS is trying to connect to a target (OST or peer MDT) before that target is finished setup. Rather than spamming the console logs during initial connection, only print a console error message if there are repeated failures trying to connect to the

[PATCH 046/124] staging: lustre: llite: handle concurrent use of cob_transient_pages

2016-09-18 Thread James Simmons
From: Stephen Champion With the lockless __generic_file_aio_write introduced in LU-1669, ll_direct_IO_26 is no longer protected by the inode i_isem. This renders obsoltete checks that all transient pages have been handled before and after entry, and requires atomic access to their counter. Sign

[PATCH 058/124] staging: lustre: statahead: race in start/stop statahead

2016-09-18 Thread James Simmons
From: Lai Siyao When starting statahead thread, it should check whether current lli_opendir_key was deauthorized in the mean time by another process. Signed-off-by: Lai Siyao Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3270 Reviewed-on: http://review.whamcloud.com/9666 Reviewed-by: Fan

[PATCH 049/124] staging: lustre: lmv: move some inline functions to lustre_lmv.h

2016-09-18 Thread James Simmons
From: Fan Yong Move some inline code out of lmv core into lustre_lmv.h. This is to prepare for use outside of the lmv layer in the future of these functions. Change from passing in struct lmv_stripe_md to just int for lmv_is_known_hash_type. Signed-off-by: Fan Yong Intel-bug-id: https://jira.hp

[PATCH 054/124] staging: lustre: llite: Add ioctl to get parent fids from link EA.

2016-09-18 Thread James Simmons
From: Henri Doreau Added LL_IOC_GETPARENT to retrieve the /name(s) of a given entry, based on its link EA. This saves multiple calls to path2fid/fid2path. Merged with second later patch that does various cleanups. Avoid unneeded allocation. Get read-only attributes from the user getparent struct

[PATCH 073/124] staging: lustre: llite: remove ll_objects_destroy()

2016-09-18 Thread James Simmons
From: John L. Hammond Remove ll_objects_destroy(). This function is not needed for interoperability with servers of version 2.4 or higher. Remove the then unused function lov_destroy() and its supporting functions. Remove the lsm_destroy method of struct lsm_operations. Remove the unused struct

[PATCH 051/124] staging: lustre: llite: ensure all data flush out when umount

2016-09-18 Thread James Simmons
From: Yang Sheng Write out all extents when clear inode. Otherwise we may lose data while umount. Signed-off-by: Yang Sheng Signed-off-by: Jinshan Xiong Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5584 Reviewed-on: http://review.whamcloud.com/12103 Reviewed-by: Bobi Jam Reviewed-by: O

[PATCH 040/124] staging: lustre: obd: rename LUSTRE_STRIPE_MAXBYTES

2016-09-18 Thread James Simmons
From: John L. Hammond Rename LUSTRE_STRIPE_MAXBYTES to LUSTRE_EXT3_STRIPE_MAXBYTES and correct the comment describing its use. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on: http://review.whamcloud.com/11800 Reviewed-by: Andreas Dilger Revi

[PATCH 074/124] staging: lustre: changelog: Proper record remapping

2016-09-18 Thread James Simmons
From: Henri Doreau Fixed changelog_remap_rec() to correctly remap records emitted with jobid_var=disabled, i.e. delivered by new servers but with no jobid field. Signed-off-by: Henri Doreau Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5862 Reviewed-on: http://review.whamcloud.com/12574 R

[PATCH 082/124] staging: lustre: mdc: Proper accessing struct lov_user_md

2016-09-18 Thread James Simmons
From: Yoshifumi Uemura In mdc_setattr_pack() access the members of struct lov_user_md by little endian byte order. Signed-off-by: Yoshifumi Uemura Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5889 Reviewed-on: http://review.whamcloud.com/12683 Reviewed-by: Dmitry Eremin Reviewed-by: Jam

[PATCH 078/124] staging: lustre: llog: prevent out-of-bound index

2016-09-18 Thread James Simmons
From: frank zago llog_process_thread() can be called from llog_cat_process_cb with an index already out of bound, leading to the following crash: LustreError: 3773:0:(llog.c:310:llog_process_thread()) ASSERTION(index <= last_index + 1 ) failed: LustreError: 3773:0:(llog.c:310:llog_process_thre

[PATCH 083/124] staging: lustre: ldlm: evict clients returning errors on ASTs

2016-09-18 Thread James Simmons
From: Alexey Lyashkov To test proper behavior of clients returning errors on ASTs we can induce a failure with setting OBD_FAIL_LDLM_BL_CALLBACK_NET. Handle the new additonal case of cfs_fail_err being set as well so that the cfs_fail_err can be sent back in a reply. Signed-off-by: Alexey Lyashk

[PATCH 075/124] staging: lustre: recovery: don't replay closed open

2016-09-18 Thread James Simmons
From: Niu Yawei To avoid scanning the replay open list every time in the ptlrpc_free_committed(), the fix of LU-2613 (4322e0f9) changed the ptlrpc_free_committed() to skip the open list unless the import generation is changed. That introduced a race which could make a closed open being replayed:

[PATCH 030/124] staging: lustre: llite: Replace write mutex with range lock

2016-09-18 Thread James Simmons
From: Prakash Surya Testing has shown the ll_inode_inode's lli_write_mutex to be a limiting factor with single shared file write performance, when using many writing threads on a single machine. Even if each thread is writing to a unique portion of the file, the lli_write_mutex will prevent no mo

[PATCH 086/124] staging: lustre: obdclass: eliminate NULL error return

2016-09-18 Thread James Simmons
From: Bob Glossman Always return an ERR_PTR() on errors, never return a NULL, in lu_object_find_slice(). Also clean up callers who no longer need special case handling of NULL returns. Signed-off-by: Bob Glossman Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5858 Reviewed-on: http://revi

[PATCH 089/124] staging: lustre: obd: change brw_page->count to unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Pages count is unsigned. So, change the type accordant usage. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12378 Reviewed-by: John L. Hammond Reviewed-by: James Simmons Reviewed-by: Oleg Drok

[PATCH 059/124] staging: lustre: at: net AT after connect

2016-09-18 Thread James Simmons
From: Alexander Boyko Once connected, the previously gathered AT statistics is not valid anymore because may reflect other routing, etc. The connect by itself could take a long time due to different reasons (e.g. server was not ready) and net latency got very high (see import_select_connection())

[PATCH 044/124] staging: lustre: misc: Reduce exposure to overflow on page counters.

2016-09-18 Thread James Simmons
From: Stephen Champion When the number of an object in use or circulation is tied to memory size of the system, very large memory systems can overflow 32 bit counters. This patch addresses overflow on page counters in the osc LRU and obd accounting. Signed-off-by: Stephen Champion Intel-bug-id

[PATCH 084/124] staging: lustre: fiemap: set FIEMAP_EXTENT_LAST correctly

2016-09-18 Thread James Simmons
From: Bobi Jam When we've collected enough extents as user requested, we'd check one further to decide whether we've reached the last extent of the file. Signed-off-by: Bobi Jam Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5933 Reviewed-on: http://review.whamcloud.com/12781 Reviewed-by:

[PATCH 085/124] staging: lustre: obdclass: change loop indexes to unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Cleanup warnings about comparison between signed and unsigned. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12387 Reviewed-by: Bob Glossman Reviewed-by: John L. Hammond Reviewed-by: Jinshan X

[PATCH 093/124] staging: lustre: ldlm: move LDLM_GID_ANY to lustre_dlm.h

2016-09-18 Thread James Simmons
From: Jinshan Xiong lustre_idl.h only includes wire data; lustre_dlm.h is the right place for LDLM_GID_ANY. Signed-off-by: Jinshan Xiong Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6028 Reviewed-on: http://review.whamcloud.com/13074 Reviewed-by: Bobi Jam Reviewed-by: James Simmons Rev

[PATCH 047/124] staging: lustre: llite: enforce pool name length limit

2016-09-18 Thread James Simmons
From: Li Xi The pool related codes have some inconsistency about the length of pool name. Creating and setting a pool name of length 16 to a directory will succeed. However, creating a file under that directory will fail. This patch disables any pool name which is longer or equal to 16. And it c

[PATCH 092/124] staging: lustre: ptlrpc: fix comparison between signed and unsigned

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change return type and size argiments of lustre_msg_hdr_size(), lustre_msg_buf{len,count}() and req_capsule_*_size() to __u32. Change type of req_format->rf_idx and req_format->rf_fields.nr to size_t. Also return zero for incorrect message magic instead of -EINVAL. This will b

[PATCH 031/124] staging: lustre: vvp: Use lockless __generic_file_aio_write

2016-09-18 Thread James Simmons
From: Prakash Surya Testing multi-threaded single shard file write performance has shown the inode mutex to be a limiting factor when using the generic_file_write_iter function. To work around this bottle neck, this change replaces the locked version of that call with the lock less version, speci

[PATCH 081/124] staging: lustre: obdclass: lu_htable_order() return type to long

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change the type accordant usage. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/12385 Reviewed-by: Bob Glossman Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmo

[PATCH 088/124] staging: lustre: ldlm: Recalculate interval in ldlm_pool_recalc()

2016-09-18 Thread James Simmons
From: Nathaniel Clark Instead of rechecking a static value, recalculate to see if pool stats need to be updated. Add newline so message will print instead of warning about missing newline. Signed-off-by: Nathaniel Clark Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4536 Reviewed-on: http:

  1   2   >