Re: [PATCH 01/10] staging: lustre: LNet drop rule implementation

2016-03-04 Thread kbuild test robot
Hi Liang, [auto build test ERROR on staging/staging-testing] [also build test ERROR on v4.5-rc6 next-20160304] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/James-Simmons/Last-batch-of-fixes

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread kbuild test robot
Hi Arnd, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.5-rc6 next-20160304] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/isdn-icn-remove

Re: [PATCH 07/10] staging: lustre: add last missing sparse annotation __user

2016-03-04 Thread Drokin, Oleg
On Mar 4, 2016, at 9:09 PM, James Simmons wrote: > From: Frank Zago > > One of the __user was missed in being applied to upstream > client. This is broken out of patch 11819. It was not, the bug was fixed in another way. > Signed-off-by: Frank Zago > Intel-bug-id: https://jira.hpdd.intel.com

Re: [PATCH v3 16/52] mtd: nand: use mtd_set_ecclayout() where appropriate

2016-03-04 Thread Brian Norris
On Fri, Feb 26, 2016 at 01:57:24AM +0100, Boris Brezillon wrote: > Use the mtd_set_ecclayout() helper instead of directly assigning the > mtd->ecclayout field. > > Signed-off-by: Boris Brezillon > --- > drivers/mtd/nand/nand_base.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > dif

[PATCH 08/10] staging: lustre: change test to asser in LNetGetId

2016-03-04 Thread James Simmons
The ln_refcount test was changed into an assert. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/lnet/api-ni.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/api-ni.c b/drivers/staging/lustre/lnet/lnet/api-ni.c index 739

[PATCH 10/10] staging: lustre: make LNet use lprocfs_call_handler

2016-03-04 Thread James Simmons
Sometime ago a patch was submitted to duplicate the proc_call_handler code in the LNet layer. This was due to the thinking libcfs was not used by the LNet layer. This was a wrong assumption so lets make LNet use the lprocfs_call_handler from the libcfs layer. Signed-off-by: James Simmons --- ...

[PATCH 09/10] staging: lustre: rename proc_call_handler to lprocfs_call_handler

2016-03-04 Thread James Simmons
Using proc_call_handler as a function name is way too generic. Rename to lprocfs_call_handler to avoid possible collisions. Signed-off-by: James Simmons --- drivers/staging/lustre/lustre/libcfs/module.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/dri

[PATCH 05/10] staging: lustre: fix 'data race condition' issue in framework.c

2016-03-04 Thread James Simmons
From: Sebastien Buisson Fix 'data race condition' defects found by Coverity version 6.5.0: Data race condition (MISSING_LOCK) Accessing variable without holding lock. Elsewhere, this variable is accessed with lock held. Signed-off-by: Sebastien Buisson Intel-bug-id: https://jira.hpdd.intel.com/

[PATCH 06/10] staging: lustre: Correct missing newline

2016-03-04 Thread James Simmons
From: James Nunez Several error messages are missing newline characters at the end of the message. Newlines are added where necessary and other minor corrections; no punctuation at the end of an error message, add a return code to the end of error messages, device name at the beginning, etc. The

[PATCH 07/10] staging: lustre: add last missing sparse annotation __user

2016-03-04 Thread James Simmons
From: Frank Zago One of the __user was missed in being applied to upstream client. This is broken out of patch 11819. Signed-off-by: Frank Zago Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5396 Reviewed-on: http://review.whamcloud.com/11819 Reviewed-by: James Simmons Reviewed-by: Dmitry

[PATCH 03/10] staging: lustre: fix 'data race condition' issue in conrpc.c

2016-03-04 Thread James Simmons
From: Sebastien Buisson Fix 'data race condition' defects found by Coverity version 6.5.0: Data race condition (MISSING_LOCK) Accessing variable without holding lock. Elsewhere, this variable is accessed with lock held. Signed-off-by: Sebastien Buisson Intel-bug-id: https://jira.hpdd.intel.com/

[PATCH 00/10] Last batch of fixes for LNet

2016-03-04 Thread James Simmons
This batch merges the remaining LNet patches from the OpenSFS branch for the upstream client. Once merged the LNet code will be up to date with the latest production code. Only style issues are remaining. Still future patches being developed for LNet will be landed to the upstream client as soon as

[PATCH 02/10] staging: lustre: LNet network latency simulation

2016-03-04 Thread James Simmons
From: Liang Zhen Incoming lnet message can be delayed for seconds if it can match any of LNet Delay Rules. User can add/remove/list Delay Rule by lctl commands: - lctl net_delay_add Add a new Delay Rule to LNet, options <-s | --source SRC_NID> <-d | --dest DST_NID> <<-r | --rate RATE_NUM

[PATCH 01/10] staging: lustre: LNet drop rule implementation

2016-03-04 Thread James Simmons
From: Liang Zhen This is implementation of LNet Drop Rule, which can randomly drop LNet messages at specified rate. LNet Drop Rule can only be applied to receive side of message. User can add drop_rule either on end point of cluster (client/server) or on LNet routers. Here are lctl command to c

[PATCH 04/10] staging: lustre: fix 'NULL pointer dereference' errors

2016-03-04 Thread James Simmons
From: Sebastien Buisson Fix 'NULL pointer dereference' defects found by Coverity version 6.5.3: Dereference after null check (FORWARD_NULL) For instance, Passing null pointer to a function which dereferences it. Dereference before null check (REVERSE_INULL) Null-checking variable suggests that it

[PATCH net-next] hv_netvsc: Move subchannel waiting to rndis_filter_device_remove()

2016-03-04 Thread Haiyang Zhang
During hot add, vmbus_device_register() is called from vmbus_onoffer(), on the same workqueue as the subchannel offer message work-queue, so subchannel offer won't be processed until the vmbus_device_register()/... /netvsc_probe() is done. Also, vmbus_device_register() is called with channel_mutex

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread Arnd Bergmann
On Friday 04 March 2016 19:18:49 i...@linux-pingi.de wrote: > Am 04.03.2016 um 16:24 schrieb Arnd Bergmann: > > On Thursday 03 March 2016 09:30:38 i...@linux-pingi.de wrote: > >> Hi Arnd, > >> I fully agree and ack. > >> Thanks for the work. > >> > > > > I actually did more patches that I ended up

[PATCH] Staging: netlogic: removes an unnecessary cast on a void pointer.

2016-03-04 Thread Ben Marsh
This patch removes an unnecessary cast on a void pointer in xlr_net.c Signed-off-by: Ben Marsh --- drivers/staging/netlogic/xlr_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/netlogic/xlr_net.c b/drivers/staging/netlogic/xlr_net.c index 0015847..ec5b8e

Re: [PATCH] drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Jes Sorensen
Cihangir Akturk writes: > null test on pnetwork removed, because the iterator variable > list_for_each_entry_safe cannot be null. > > This commit fixes the following error reported by coccinelle: > > drivers/staging/rtl8723au/core/rtw_mlme.c:1621:7-15: ERROR: iterator > variable bound on line 1620

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread isdn
Am 04.03.2016 um 16:24 schrieb Arnd Bergmann: > On Thursday 03 March 2016 09:30:38 i...@linux-pingi.de wrote: >> Hi Arnd, >> I fully agree and ack. >> Thanks for the work. >> > > I actually did more patches that I ended up not submitting: > > * move hisax to staging > * remove i4l support from gi

Re: [PATCH] staging/android: add flags member to sync ioctl structs

2016-03-04 Thread Daniel Vetter
On Thu, Mar 03, 2016 at 08:17:14AM -0800, Greg Kroah-Hartman wrote: > On Thu, Mar 03, 2016 at 11:37:17AM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Play safe and add flags member to all structs. So we don't need to > > break API or create new IOCTL in the future if new featur

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread Arnd Bergmann
On Friday 04 March 2016 17:18:23 Paul Bolle wrote: > [Added Tilman and Christoph.] > > On vr, 2016-03-04 at 16:24 +0100, Arnd Bergmann wrote: > > I actually did more patches that I ended up not submitting: > > > > * move hisax to staging > > * remove i4l support from gigaset > > For the record:

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread Paul Bolle
[Added Tilman and Christoph.] On vr, 2016-03-04 at 16:24 +0100, Arnd Bergmann wrote: > I actually did more patches that I ended up not submitting: > > * move hisax to staging > * remove i4l support from gigaset For the record: I have no reason to object a patch that does that. (I'm not aware any

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2016-03-04 Thread Arnd Bergmann
On Thursday 03 March 2016 09:30:38 i...@linux-pingi.de wrote: > Hi Arnd, > I fully agree and ack. > Thanks for the work. > I actually did more patches that I ended up not submitting: * move hisax to staging * remove i4l support from gigaset * move i4l core to staging while I initially thought t

[PATCH] drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
null test on pnetwork removed, because the iterator variable list_for_each_entry_safe cannot be null. This commit fixes the following error reported by coccinelle: drivers/staging/rtl8723au/core/rtw_mlme.c:1621:7-15: ERROR: iterator variable bound on line 1620 cannot be NULL Signed-off-by: Cihan

Re: drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
On Fri, Mar 04, 2016 at 03:08:19PM +0200, Cihangir Akturk wrote: > null test on pnetwork removed, because the iterator variable > list_for_each_entry_safe cannot be null. > > This commit fixes the following error reported by coccinelle: > > drivers/staging/rtl8723au/core/rtw_mlme.c:1621:7-15: ERR

drivers: staging: rtl8723au: remove unneeded null test

2016-03-04 Thread Cihangir Akturk
null test on pnetwork removed, because the iterator variable list_for_each_entry_safe cannot be null. This commit fixes the following error reported by coccinelle: drivers/staging/rtl8723au/core/rtw_mlme.c:1621:7-15: ERROR: iterator variable bound on line 1620 cannot be NULL Signed-off-by: Cihan

[PATCH V3] staging: rtl8192u: check return value of read_nic_word_E

2016-03-04 Thread Salah Triki
The call of read_nic_word_E may fail, therefore its return value must be checked. Signed-off-by: Salah Triki --- drivers/staging/rtl8192u/r8180_93cx6.c | 30 +-- drivers/staging/rtl8192u/r8180_93cx6.h | 2 +- drivers/staging/rtl8192u/r8192U_core.c | 140 ++---