[PATCH 3/4] staging/lustre: Remove unnecessary space after a cast

2016-06-18 Thread Oleg Drokin
This patch fixes all checkpatch occurences of "CHECK: No space is necessary after a cast" in Lustre code. Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- drivers/staging/lustre/include/linux/lnet/lib-dlc.h | 2 +- drivers/staging/lustre/include/linux/lnet/types.h | 4 ++-

[PATCH 1/4] staging/lustre: Fix blank line before EXPORT_SYMBOL()

2016-06-18 Thread Oleg Drokin
From: Emoly Liu This patch fixes one checkpatch warning in lustre: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 1 - 1 file changed, 1 deletion(-) dif

[PATCH 4/4] staging/lustre: Update FID documentation link.

2016-06-18 Thread Oleg Drokin
When OpenSFS took over lustre.org, there was some reshuffling. FIDs on ZFS document is now at http://wiki.old.lustre.org/index.php/Architecture_-_Interoperability_fids_zfs instead of the old location, so update comments accordingly. Signed-off-by: Oleg Drokin Reported-by: Xose Vazquez Perez ---

[PATCH 2/4] staging/lustre: Keep logical continuations on the previous line

2016-06-18 Thread Oleg Drokin
From: Emoly Liu This patch fixes all checkpatch occurences of "CHECK: Logical continuations should be on the previous line" in Lustre code. Signed-off-by: Emoly Liu Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 9 - drivers/staging/lustre/lustre/

[PATCH 0/4] Some Lustre style cleanups

2016-06-18 Thread Oleg Drokin
These are just some more cleanups in Lustre styel and comments Emoly Liu (2): staging/lustre: Fix blank line before EXPORT_SYMBOL() staging/lustre: Keep logical continuations on the previous line Oleg Drokin (2): staging/lustre: Remove unnecessary space after a cast staging/lustre: Update

Re: [PATCH 3/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread Greg Kroah-Hartman
On Sat, Jun 18, 2016 at 10:26:08PM +0100, Craig Inches wrote: > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' You can't send 3 different patches all with the same subject line :( ___ devel mailing list de...@linuxdriverproject.org http://driv

Re: [PATCH 1/3] staging: luster: cl_object.h Checkpatch Cleanup

2016-06-18 Thread Greg Kroah-Hartman
On Sat, Jun 18, 2016 at 10:25:42PM +0100, Craig Inches wrote: > Line length greater than 80 char. What does that mean? What does your subject mean? Please be more descriptive, again, look at other patches that have been merged for examples... greg k-h ___

Re: [PATCH 2/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread Greg Kroah-Hartman
On Sat, Jun 18, 2016 at 10:25:55PM +0100, Craig Inches wrote: > Macros with complex values should be enclosed in parenthesis > > Signed-off-by: Craig Inches > --- > drivers/staging/lustre/lustre/include/cl_object.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) ALWAYS test build yo

Re: [PATCH 2/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread kbuild test robot
Hi, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.7-rc3 next-20160617] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Craig-Inches/staging-luster-cl_object-

Re: [PATCH 2/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread kbuild test robot
Hi, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.7-rc3 next-20160617] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Craig-Inches/staging-luster-cl_object-

Re: [PATCH 2/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread kbuild test robot
Hi, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.7-rc3 next-20160617] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Craig-Inches/staging-luster-cl_object-

[PATCH 1/3] staging: luster: cl_object.h Checkpatch Cleanup

2016-06-18 Thread Craig Inches
Line length greater than 80 char. Signed-off-by: Craig Inches --- drivers/staging/lustre/lustre/include/cl_object.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/include/cl_object.h b/drivers/staging/lustre/lustre/include/cl_object.h index 99

[PATCH 2/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread Craig Inches
Macros with complex values should be enclosed in parenthesis Signed-off-by: Craig Inches --- drivers/staging/lustre/lustre/include/cl_object.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/cl_object.h b/drivers/staging/lustre/lustr

[PATCH 3/3] staging: luster: Checkpatch Cleanup

2016-06-18 Thread Craig Inches
WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Craig Inches --- drivers/staging/lustre/lustre/include/cl_object.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/cl_object.h b/drivers/staging/lustr

[PATCH 0/3] staging: luster: cl_object.h Checkpatch Cleanup

2016-06-18 Thread Craig Inches
Fix three different issues from checkpatch: Line over 80 Chars Unsigned int prefered over unsigned Macros with complex values should be enclosed in parenthesis Craig Inches (3): checkpatch style issue: Line over 80 char Checkpatch style issue: Checkpatch error cleanup drivers/staging/lustr

[PATCH] staging: dgnc: add __exit macro to dgnc_driver.c

2016-06-18 Thread Joshua Houghton
Add the __exit macro to the dgnc_cleanup_module(void) function in dgnc_driver.c Signed-off-by: Joshua Houghton --- drivers/staging/dgnc/dgnc_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/drivers/staging/dgnc/dgnc_driver.c index

[GIT PULL] Staging driver fixes for 4.7-rc4

2016-06-18 Thread Greg KH
The following changes since commit af8c34ce6ae32addda3788d54a7e340cad22516b: Linux 4.7-rc2 (2016-06-05 14:31:26 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/ tags/staging-4.7-rc4 for you to fetch changes up to bc8201e3e1885

Re: [PATCH v2 4/4] Staging: comedi: fix line over 80 characters issue in s626.c

2016-06-18 Thread Luis de Bethencourt
On 18/06/16 04:51, Greg KH wrote: > On Mon, May 30, 2016 at 03:21:27PM +0100, Luis de Bethencourt wrote: >> On 28/05/16 06:26, Ravishankar Karkala Mallikarjunayya wrote: >>> This is a patch to the s626.c file that fixes up a line over 80 >>> characters issues found by the checkpatch.pl tool. >>> >

Re: [PATCH] android: binder: fix dangling pointer comparison

2016-06-18 Thread Jann Horn
On Sat, Jun 18, 2016 at 11:19 AM, ZhaoJunmin Zhao(Junmin) wrote: > 在 2016/6/16 6:39, Jann Horn 写道: >> On Thu, Jun 16, 2016 at 12:31 AM, Arve Hjønnevåg wrote: >>> On Wed, Jun 15, 2016 at 3:09 PM, Jann Horn wrote: If /dev/binder is opened and the opener process then e.g. calls execve, pr

Re: [PATCH] android: binder: fix dangling pointer comparison

2016-06-18 Thread ZhaoJunmin Zhao(Junmin)
在 2016/6/16 6:39, Jann Horn 写道: On Thu, Jun 16, 2016 at 12:31 AM, Arve Hjønnevåg wrote: On Wed, Jun 15, 2016 at 3:09 PM, Jann Horn wrote: If /dev/binder is opened and the opener process then e.g. calls execve, proc->vma_vm_mm will still point to the location of the now-freed mm_struct. If t

[PATCH] Drivers: hv: connection: Remove create_workqueue

2016-06-18 Thread Bhaktipriya Shridhar
alloc_workqueue replaces deprecated create_workqueue(). A dedicated workqueue has been used since the workitem (viz &ctx->work, which maps to vmbus_onmessage_work), is engaged in normal device operation which involves invoking the handler for channel protocol messages. WQ_MEM_RECLAIM has been set t