[PATCH 119/124] staging: lustre: lnet: potential deadlock in lnet

2016-09-18 Thread James Simmons
From: Quentin Bouget Fixes potential deadlock in LNetMDAttach Signed-off-by: Quentin Bouget Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8249 Reviewed-on: http://review.whamcloud.com/20676 Reviewed-by: Doug Oucharek Reviewed-by: James Simmons Reviewed-by: Henri Doreau Reviewed-by: Ole

[PATCH 103/124] staging: lustre: llite: pack suppgid to MDS correctly

2016-09-18 Thread James Simmons
From: Fan Yong The ll_lookup_it() may trigger IT_OPEN RPC to open a file by name. But at that time, the client does not know the target file's GID, so it cannot pack the necessary supplementary group ID in the RPC. Because of missing the supplementary group ID, the RPC maybe fail for open permiss

[PATCH 097/124] staging: lustre: osc: change cl_extent_tax and *grants to unsigned

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

[PATCH 104/124] staging: lustre: clio: rename coo_attr_set to coo_attr_update

2016-09-18 Thread James Simmons
From: Bobi Jam coo_attr_set() is used to update object's attribute but its name makes confusion that people intuitively think that it is used to pass object's attribute down to server sides. Signed-off-by: Bobi Jam Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-1154 Reviewed-on: http://rev

[PATCH 120/124] staging: lustre: lnet: check if ni is in current net namespace

2016-09-18 Thread James Simmons
From: Sebastien Buisson Add new 'ni_net_ns' field to struct lnet_ni to hold a reference to original net namespace in which ni is created. In LNetDist(), check if ni was created in same net namespace as current's one. If not, assign order above 0x, to make this ni not a priority. Signed-o

[PATCH 117/124] staging: lustre: mdc: cl_default_mds_easize not refreshed

2016-09-18 Thread James Simmons
From: Ned Bass The client_obd::cl_default_mds_easize field should track the largest observed EA size advertised by the MDT, subject to a reasonable upper bound. The MDC uses cl_default_mds_easize to calculate the initial size of request buffers. The default value should be small enough to avoid

[PATCH 094/124] staging: lustre: lov: flatten struct lov_stripe_md

2016-09-18 Thread James Simmons
From: John L. Hammond Flatten out the lsm_wire struct from the middle of struct lov_stripe_md and remove the member name macros. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5814 Reviewed-on: http://review.whamcloud.com/12581 Reviewed-by: Bobi Jam Reviewed

[PATCH 100/124] staging: lustre: obd: change type of cl_conn_count to size_t

2016-09-18 Thread James Simmons
From: Dmitry Eremin Change type of cl_conn_count to size_t. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577 Reviewed-on: http://review.whamcloud.com/13125 Reviewed-by: James Simmons Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: Jam

[PATCH 087/124] staging: lustre: ptlrpc: Suppress error message when imp_sec is freed

2016-09-18 Thread James Simmons
From: Amir Shehata There is a race condition on client reconnect when the import is being destroyed. Some outstanding client bound requests are being processed when the imp_sec has alread been freed. Ensure to suppress the error message in import_sec_validate_get() in that case Signed-off-by: A

[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 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 098/124] staging: lustre: lprocfs: cleanup stats locking code

2016-09-18 Thread James Simmons
From: Andreas Dilger Add comment blocks on lprocfs_stats_lock() and lprocfs_stats_unlock(). Move common NOPERCPU code out of the switch() statements to reduce code size and complexity, since it doesn't depend on the opc at all. Replace switch() in lprocfs_stats_unlock() with a simple if/else, si

[PATCH 102/124] staging: lustre: remove lustre/include/linux/

2016-09-18 Thread James Simmons
From: John L. Hammond Merge the contents of lustre/include/linux/lvfs.h into lustre/include/lvfs.h. Merge lustre/include/linux/lustre_user.h into lustre/include/lustre/lustre_user.h. Move lustre_compat25.h and lustre_patchless_compat.h from lustre/include/linux/ to lustre/include/ and rename lust

[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

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

2016-09-18 Thread James Simmons
> On Sunday, September 18, 2016 8:43:18 PM CEST James Simmons wrote: > > > > Nak. I'm not seeing this error with W=1. Also if you look > > lmv_pack_md() is only used in lmv_obd.c and the function > > appears early in the file before it is used, so no prototype > > missing errors should happen. >

[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 076/124] staging: lustre: ldlm: revert the changes for lock canceling policy

2016-09-18 Thread James Simmons
From: Jinshan Xiong The changes for LRU lock policy was introduced by commit bfae5a4e, where I was trying to revise the policy to pick locks for canceling. However, this caused two problems as mentioned in LU-5727. The first problem is that the lock can only be picked for canceling only if the n

[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 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 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 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 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 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 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 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 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 072/124] staging: lustre: echo: replace lov_stripe_md with lov_oinfo

2016-09-18 Thread James Simmons
From: John L. Hammond In echo_client replace uses of struct lov_stripe_md with struct lov_oinfo (since the instances of the former really only contained a single instance of the latter). Remove the then unneccessary functions echo_alloc_memmd(), echo_free_memmd(), osc_unpackmd(), and obd_alloc_me

[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 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 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 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 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 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 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 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 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 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 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 061/124] staging: lustre: obd: cleanup struct md_op_data and uses

2016-09-18 Thread James Simmons
From: John L. Hammond Make the following changes in or around struct md_op_data: * rename enum op_cli_flags to enum md_cli_flags. * Change to type of the op_flags member from __u32 to enum md_op_flags. * Remove the used but never set member op_npages. * Remove the set but never used me

[PATCH 045/124] staging: lustre: lmv: remove dead code

2016-09-18 Thread James Simmons
From: Dmitry Eremin The member lmv_obd->server_timeout and function lmv_set_timeouts() are not used. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-991 Reviewed-on: http://review.whamcloud.com/11880 Reviewed-by: John L. Hammond Reviewed-by: James Simmons Revi

[PATCH 048/124] staging: lustre: Flexible changelog format.

2016-09-18 Thread James Simmons
From: Henri Doreau Added jobid fields to Changelog records (and extended records). The CLF_JOBID flags allows to check if the field is present or not (old format) when reading an entry. Jobids are expressed as 32 chars long, zero-terminated strings. Updated test_205 in sanity.sh. Signed-off-by:

[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 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 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 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 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

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

2016-09-18 Thread Arnd Bergmann
On Sunday, September 18, 2016 8:43:18 PM CEST James Simmons wrote: > > Nak. I'm not seeing this error with W=1. Also if you look > lmv_pack_md() is only used in lmv_obd.c and the function > appears early in the file before it is used, so no prototype > missing errors should happen. So it should

[PATCH 043/124] staging: lustre: lmv: change type of lmv_obd->tgts_size to u32

2016-09-18 Thread James Simmons
From: Dmitry Eremin tgts_size 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/11881 Reviewed-by: Fan Yong Reviewed-by: John L. Hammond Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --

[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 039/124] staging: lustre: remove lustre_lite.h

2016-09-18 Thread James Simmons
From: John L. Hammond Move several definition only used in lustre/llite/ to lustre/llite/llite_internal.h. Remove lustre/include/{,linux/}lustre_lite.h and fixup the missing includes in other headers that this exposes. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/bro

[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 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 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 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 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 035/124] staging: lustre: llite: style cleanup for ll_mkdir

2016-09-18 Thread James Simmons
From: John L. Hammond Style cleanup to make the code readable. 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 S

[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 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 022/124] staging: lustre: llog: add newly opened llog at tail of handle list

2016-09-18 Thread James Simmons
From: Li Xi Add newly opened llog handle at the tail of handle list to increase lookup speed, especially for cancel operation, because the canceled log will be removed from the list, and lookup is from the beginning. Signed-off-by: Lai Siyao Signed-off-by: Li Xi Intel-bug-id: https://jira.hpdd

[PATCH 020/124] staging: lustre: lmv: release request in lmv_revalidate_slaves()

2016-09-18 Thread James Simmons
From: John L. Hammond In lmv_revalidate_slaves() ensure that the request returned by md_intent_lock() is properly released on all paths. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5452 Reviewed-on: http://review.whamcloud.com/11326 Reviewed-by: wang di R

[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 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 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 005/124] staging: lustre: obdclass: optimize busy loop wait

2016-09-18 Thread James Simmons
From: Alexander Zarochentsev two optimizations in the busy loop wait in class_cleanup(): (1) remove unnecessary spinlocks protecting obd_conn_inprogress access (2) replace cond_resched by more suitable yield to unconditionally yield the CPU if the condition check fails. Signed-off-by: Al

[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 010/124] staging: lustre: llite: add LL_LEASE_{RD,WR,UN}LCK

2016-09-18 Thread James Simmons
From: John L. Hammond Define new constants LL_LEASE_{RD,WR,UN}LCK for use as the argument to and return value from the LL_IOC_{GET,SET}_LEASE ioctls. As arguments, these contants replace the use of F_{RD,WR,UN}LCK from fcntl.h. As return values they replace the use of FMODE_{READ,WRITE} which are

[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 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 008/124] staging: lustre: obd: add rnb_ prefix to struct niobuf_remote members

2016-09-18 Thread James Simmons
From: John L. Hammond Add the prefix rnb_ to the members of struct niobuf_remote. Delete the relevant compat macros from ofd_internal.h. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5061 Reviewed-on: http://review.whamcloud.com/10452 Reviewed-by: Andreas Di

[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

Re: [PATCH v3) posix-timers: make it configurable

2016-09-18 Thread Richard Cochran
On Sun, Sep 18, 2016 at 04:30:21PM -0400, Nicolas Pitre wrote: > I don't think the driver changes are significant enough to warrant a ACK > from individual driver maintainers. > > Yours would be welcome though. ;-) Acked-by: Richard Cochran > Then I'll resend the whole thing to John as a seri

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

2016-09-18 Thread Nayeemahmed Badebade
Added __acquires / __releases sparse locking annotations to lock_res_and_lock() and unlock_res_and_lock() functions in l_lock.c, to fix below sparse warnings: l_lock.c:47:22: warning: context imbalance in 'lock_res_and_lock' - wrong count at exit l_lock.c:61:6: warning: context imbalance in 'unlo

Re: [PATHC v2 0/9] ima: carry the measurement list across kexec

2016-09-18 Thread Thiago Jung Bauermann
Am Samstag, 17 September 2016, 00:17:37 schrieb Eric W. Biederman: > Thiago Jung Bauermann writes: > > Hello Eric, > > > > Am Freitag, 16 September 2016, 14:47:13 schrieb Eric W. Biederman: > >> I can see tracking to see if the list has changed at some > >> point and causing a reboot(LINUX_REBOOT

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

2016-09-18 Thread Dilger, Andreas
> On Sep 18, 2016, at 23:18, Nayeemahmed Badebade > wrote: > > Added __acquires / __releases sparse locking annotations > to lock_res_and_lock() and unlock_res_and_lock() functions > in l_lock.c, to fix below sparse warnings: > > l_lock.c:47:22: warning: context imbalance in 'lock_res_and_lock

Re: [PATCH] phy: meson-usb2: Fix build dependency

2016-09-18 Thread Martin Blumenstingl
Hi Axel, On Fri, Sep 16, 2016 at 4:07 AM, Axel Lin wrote: > of_usb_get_dr_mode_by_phy will not be compiled if !USB_COMMON, fix below > build error: > > drivers/built-in.o: In function `phy_meson_usb2_probe': > debugfs.c:(.text+0x76b4): undefined reference to `of_usb_get_dr_mode_by_phy' > Makefile

[PATCH] dcache: fix implicit wait.h include in

2016-09-18 Thread Paul Gortmaker
To fix: In file included from fs/notify/fsnotify.c:19:0: ./include/linux/dcache.h:103:3: error: unknown type name ‘wait_queue_head_t’ wait_queue_head_t *d_wait; /* in-lookup ones only */ ^ ./include/linux/dcache.h:233:6: error: unknown type name ‘wait_queue_head_t’ wait_que

[PATCH] rbtree_augmented: fix implicit rcu assign pointer dependency

2016-09-18 Thread Paul Gortmaker
To fix: In file included from include/linux/interval_tree_generic.h:22:0, from lib/interval_tree.c:2: include/linux/rbtree_augmented.h: In function ‘__rb_change_child_rcu’: include/linux/rbtree_augmented.h:139:4: error: implicit declaration of function ‘rcu_assign_pointer’ [-Werr

[PATCH v3] console: Add persistent scrollback buffers for all VGA consoles

2016-09-18 Thread Manuel Schölling
Add a scrollback buffers for each VGA console. The benefit is that the scrollback history is not flushed when switching between consoles but is persistent. The buffers are allocated on demand when a new console is opened. It always annoys me when I switch back to a console and I can just read half

[PATCH 1/3] pcmcia: m32r_pcc: check return from request_irq

2016-09-18 Thread Sudip Mukherjee
While building m32r allmodconfig we were getting warning: drivers/pcmcia/m32r_pcc.c:331:2: warning: ignoring return value of 'request_irq', declared with attribute warn_unused_result request_irq() can fail and we should always be checking the result from it. Check the result and return it to th

[PATCH 3/3] pcmcia: m32r_pcc: check return from add_pcc_socket

2016-09-18 Thread Sudip Mukherjee
If request_irq() fails it passes the error to the caller. The caller now checks it and jumps to the common error path on failure. Signed-off-by: Sudip Mukherjee --- drivers/pcmcia/m32r_pcc.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/pcmcia/m32r_pcc.c

[PATCH 2/3] pcmcia: m32r_pcc: use common error path

2016-09-18 Thread Sudip Mukherjee
Use a common error path for the failure. Signed-off-by: Sudip Mukherjee --- drivers/pcmcia/m32r_pcc.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/pcmcia/m32r_pcc.c b/drivers/pcmcia/m32r_pcc.c index fad4455..56bf388 100644 --- a/drivers/pcmcia/m3

Re: [m32r] undefined reference to `printk'

2016-09-18 Thread Sudip Mukherjee
Hi Fengguang, On Sunday 11 September 2016 02:16 AM, Fengguang Wu wrote: Sorry m32r looks orphaned and there is no maintainer to CC such arch related error report.. Yes, its a pity that it has no maintainer. I have tried to contact the previous maintainer (Hirokazu Takata) but without any succe

Re: bcachefs: Encryption (Posting for review)

2016-09-18 Thread Eric Wheeler
On Thu, 1 Sep 2016, Kent Overstreet wrote: > Encryption in bcachefs is done and working and I just finished documenting the > design - so now, it needs more eyeballs and vetting before letting users play > with it. > > ### Algorithms > > By virtue of working within a copy on write filesystem wit

RE: [PATCH] thermal: cpu_cooling: Fix wrong comment call function name

2016-09-18 Thread
You're welcome. Will be merged this patch? Thank you Hugh Kang No One ever is defeated until defeat has been accepted as a reality. 강인혁 (Hugh Kang) Chief Engineer SW Platform(연) Advanced OS Technology TEAM (AOT) Mobile: +82-10-5513-7957 -Original Message- From: Javi Merino [mailto:javi.m

Re: [PATCH v4] m68k: let clk_disable() return immediately if clk is NULL

2016-09-18 Thread Greg Ungerer
Hi Masahiro, On 19/09/16 04:01, Masahiro Yamada wrote: > In many of clk_disable() implementations, it is a no-op for a NULL > pointer input, but this is one of the exceptions. > > Making it treewide consistent will allow clock consumers to call > clk_disable() without NULL pointer check. > > Sig

Re: [PATCH net-next 2/3] r8152: support ECM mode

2016-09-18 Thread kbuild test robot
Hi Hayes, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Hayes-Wang/r8152-configuration-setting/20160907-192351 config: x86_64-randconfig-s0-09190146 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the a

Linux 4.8-rc7

2016-09-18 Thread Linus Torvalds
Another week, another rc. Normally rc7 is the last in the series before the final release, but by now I'm pretty sure that this is going to be one of those releases that come with an rc8. Things did't calm down as much as I would have liked, there are still a few discussions going on, and it's jus

Re: [PATCH] asus-laptop: get rid of parse_arg()

2016-09-18 Thread Darren Hart
On Sat, Sep 03, 2016 at 08:51:26AM +0300, Giedrius Statkevičius wrote: > On Wed, Aug 17, 2016 at 11:23:15AM -0700, Darren Hart wrote: > > On Tue, Aug 16, 2016 at 12:49:50PM +0300, Giedrius Statkevičius wrote: > > > On Fri, Aug 12, 2016 at 02:40:02PM -0700, Darren Hart wrote: > > > > On Sat, Aug 06,

Re: [PATCH 3/4] autofs - make mountpoint checks namespace aware

2016-09-18 Thread Ian Kent
On Fri, 2016-09-16 at 10:58 +0800, Ian Kent wrote: > On Thu, 2016-09-15 at 19:47 -0500, Eric W. Biederman wrote: > > Ian Kent writes: > > > > > On Wed, 2016-09-14 at 21:08 -0500, Eric W. Biederman wrote: > > > > Ian Kent writes: > > > > > > > > > On Wed, 2016-09-14 at 12:28 -0500, Eric W. Biede

Re: [PATCH v8 9/9] drm/mediatek: add support for Mediatek SoC MT2701

2016-09-18 Thread CK Hu
Hi, YT: On Mon, 2016-09-12 at 20:01 +0800, YT Shen wrote: > This patch add support for the Mediatek MT2701 DISP subsystem. > There is only one OVL engine in MT2701. > > Signed-off-by: YT Shen > --- > drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 6 ++ > drivers/gpu/drm/mediatek/mtk_disp_rd

Re: [PATCH] ASoC: samsung: make audio interface/controller explicitly

2016-09-18 Thread Ayaka
從我的 iPad 傳送 > Krzysztof Kozlowski 於 2016年9月19日 上午2:09 寫道: > >> On Sun, Sep 18, 2016 at 11:12:34PM +0800, ayaka wrote: >> >> >>> On 09/18/2016 10:42 PM, Krzysztof Kozlowski wrote: On Sun, Sep 18, 2016 at 10:09:11PM +0800, Randy Li wrote: It is simple sound card time, we could assign

linux-next: manual merge of the f2fs tree with the ext4 tree

2016-09-18 Thread Stephen Rothwell
Hi Jaegeuk, Today's linux-next merge of the f2fs tree got a conflict in: fs/f2fs/dir.c between commit: 59aa5a3aeead ("fscrypto: make filename crypto functions return 0 on success") from the ext4 tree and commit: e06f86e61d7a ("f2fs crypto: avoid unneeded memory allocation in ->readdir")

[PATCH] of: thermal: Fixed governor at each thermal zone

2016-09-18 Thread Inhyuk Kang
It is necessary to be added governor at each thermal_zone. Because some governors should be operated in the during the kernel booting in order to avoid heating problem. Default governor cannot be covered all thermal zones policy because some thermal zones want to apply different one. For example,

Re: [PATCH 3/4] autofs - make mountpoint checks namespace aware

2016-09-18 Thread Ian Kent
On Sat, 2016-09-17 at 22:10 +0200, Mateusz Guzik wrote: > On Wed, Sep 14, 2016 at 02:14:45PM +0800, Ian Kent wrote: > > If an automount mount is clone(2)ed into a file system that is > > propagation private, when it later expires in the originating > > namespace subsequent calls to autofs ->d_autom

[PATCH 00/19] ARM: imx legacy: move peripheral initializations to .init_late

2016-09-18 Thread Vladimir Zapolskiy
The series moves registrations and initializations of all peripherals which are GPIO line consumers for all legacy (prior to DT) i.MX powered boards from .init_machine to .init_late init level. This is needed to proactively prevent boot time issues on the legacy boards due to the deprioritized init

Re: [PATCH] ASoC: samsung: make audio interface/controller explicitly

2016-09-18 Thread kbuild test robot
ord what (public, well-known) commit your patch series was built on] [Check https://git-scm.com/docs/git-format-patch for more information] url: https://github.com/0day-ci/linux/commits/Randy-Li/ASoC-samsung-make-audio-interface-controller-explicitly/20160918-222618 base: https://git.kernel.

Re: [GIT PULL] tpmdd updates for Linux 4.9

2016-09-18 Thread James Morris
On Thu, 15 Sep 2016, Jarkko Sakkinen wrote: > Hi James, > > This pull request contains tpmdd updates for Linux 4.9. There are no > major new features. This is purely a bug fix and cleanup release. > > /Jarkko > > The following changes since commit a39f44faa8c7d84dff7d1fc0bc851c6499f94d7b: > >

RE: [PATCH] ASoC: rt5663: fix platform_no_drv_owner.cocci warnings

2016-09-18 Thread Bard Liao
> -Original Message- > From: kbuild test robot [mailto:l...@intel.com] > Sent: Saturday, September 17, 2016 10:45 AM > To: Wei Yongjun > Cc: kbuild-...@01.org; Bard Liao; Oder Chiou; Liam Girdwood; Mark Brown; > Jaroslav Kysela; Takashi Iwai; Wei Yongjun; alsa-de...@alsa-project.org; > linu

Re: [PATCH v2 2/4] docs-rst: Add compressed video formats used on MT8173 codec driver

2016-09-18 Thread Tiffany Lin
Hi Hans, On Wed, 2016-09-14 at 13:51 +0200, Hans Verkuil wrote: > Hi Tiffany, > > On 09/09/2016 05:48 PM, Tiffany Lin wrote: > > Add V4L2_PIX_FMT_MT21C documentation > > > > Signed-off-by: Tiffany Lin > > --- > > Documentation/media/uapi/v4l/pixfmt-reserved.rst | 10 ++ > > 1 file ch

Re: [PATCH][V2] net: r6040: add in missing white space in error message text

2016-09-18 Thread David Miller
From: Colin King Date: Fri, 16 Sep 2016 10:43:38 +0100 > From: Colin Ian King > > A couple of dev_err messages span two lines and the literal > string is missing a white space between words. Add the white > space and join the two lines into one. > > Signed-off-by: Colin Ian King Applied to n

<    1   2   3   4   5   >