[PATCH] staging: lustre: ldlm: use designated initializers

2016-12-16 Thread Kees Cook
Prepare to mark sensitive kernel structures for randomization by making sure they're using designated initializers. These were identified during allyesconfig builds of x86, arm, and arm64, with most initializer fixes extracted from grsecurity. Signed-off-by: Kees Cook ---

[PATCH] staging: greybus: arche-apb-ctrl: fix unused warnings on resume/suspend

2016-12-16 Thread Jérémy Lefaure
When CONFIG_PM_SLEEP is disabled, SIMPLE_DEV_PM_OPS does not use arche_apb_ctrl_resume and arche_apb_ctrl_suspend functions: drivers/staging/greybus/arche-apb-ctrl.c:478:12: warning: ‘arche_apb_ctrl_resume’ defined but not used [-Wunused-function] static int arche_apb_ctrl_resume(struct device

Re: [PATCH 1/3] staging: android: remove ION_IOC_SYNC

2016-12-16 Thread Matthew Smith
On Fri, Dec 16, 2016 at 03:05:54PM -0800, Laura Abbott wrote: > The 'remove' from that TODO is more than just removing the code. > There's also an implicit 'replace it with something else'. The > work to do this is still ongoing (see some of my latest patches). > NAK for all 3 of these patches

Re: [PATCH 1/3] staging: android: remove ION_IOC_SYNC

2016-12-16 Thread Laura Abbott
On 12/16/2016 02:42 PM, Matthew Smith wrote: > Remove definition of ION_IOC_CUSTOM from ion.h. > Remove usage from compat_ion.c and ion-ioctl.c. > Remove item from TODO file. The 'remove' from that TODO is more than just removing the code. There's also an implicit 'replace it with something

[PATCH] vmbus: expose numa_node via sysfs

2016-12-16 Thread Stephen Hemminger
From: Stephen Hemminger Provide numa node information via sysfs for devices on Hyper-V (same as PCI). This is for some NUMA aware applications (like DPDK). Signed-off-by: Stephen Hemminger --- drivers/hv/vmbus_drv.c | 16 1 file

[PATCH 2/3] staging: android: remove ION_IOC_CUSTOM

2016-12-16 Thread Matthew Smith
Remove definition of ION_IOC_CUSTOM from ion.h. Remove usage from compat_ion.c and ion-ioctl.c. Remove item from TOOD file. Signed-off-by: Matthew Smith --- drivers/staging/android/TODO | 3 --- drivers/staging/android/ion/compat_ion.c | 17 -

[PATCH 3/3] staging: android: remove compat_get_ion_custom_data

2016-12-16 Thread Matthew Smith
After removing the ION_IOC_CUSTOM case, compat_get_ion_custom_data is no longer required. Signed-off-by: Matthew Smith --- drivers/staging/android/ion/compat_ion.c | 16 1 file changed, 16 deletions(-) diff --git

[PATCH 1/3] staging: android: remove ION_IOC_SYNC

2016-12-16 Thread Matthew Smith
Remove definition of ION_IOC_CUSTOM from ion.h. Remove usage from compat_ion.c and ion-ioctl.c. Remove item from TODO file. Signed-off-by: Matthew Smith --- drivers/staging/android/TODO | 3 --- drivers/staging/android/ion/compat_ion.c | 1 -

Re: [PATCH 03/14 V2] rtlwifi: rtl8821ae: Remove all instances of DBG_EMERG

2016-12-16 Thread Joe Perches
On Thu, 2016-12-15 at 12:22 -0600, Larry Finger wrote: > This is a step toward eliminating the RT_TRACE macros. Those calls that > have DBG_EMERG as the level are always logged, and they represent error > conditions, thus they are replaced with pr_err(). OK, > diff --git

[PATCH] staging: comedi: ni_670x: using the BIT(x) macro

2016-12-16 Thread Saber Rezvani
Fix the checkpatch.pl issue: CHECK: Prefer using the BIT macro replacing bit shifting on 1 with the BIT(x) macro. Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/ni_670x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] staging: comedi: ni_at_ao: using the BIT(x) macro

2016-12-16 Thread Saber Rezvani
Fix the checkpatch.pl issue: CHECK: Prefer using the BIT macro replacing bit shifting on 1 with the BIT(x) macro. Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/ni_at_ao.c | 62 +++ 1 file changed, 31 insertions(+), 31 deletions(-)

Re: [PATCH 02/14 V2] rtlwifi: Remove RT_TRACE messages that use DBG_EMERG

2016-12-16 Thread Joe Perches
On Thu, 2016-12-15 at 12:22 -0600, Larry Finger wrote: > These messages are always logged and represent error conditions, thus > we can use pr_err(). OK and some trivialities: > diff --git a/drivers/net/wireless/realtek/rtlwifi/base.c > b/drivers/net/wireless/realtek/rtlwifi/base.c [] > @@

Re: [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Joe Perches
On Fri, 2016-12-16 at 21:53 +0300, Dan Carpenter wrote: > On Fri, Dec 16, 2016 at 10:43:24AM -0800, Joe Perches wrote: > > On Fri, 2016-12-16 at 23:29 +0530, Tabrez khan wrote: > > > Remove unnecessary braces {} from single line if statement. > > > This warning is found using checkpatch.pl. > > >

[PATCH 3/3] staging: comedi: cb_pcidas64: use preferred kernel type u32

2016-12-16 Thread Saber Rezvani
Fix the checkpatch.pl issue: CHECK: Prefer kernel type 'u32' over 'uint32_t' Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/cb_pcidas64.c | 38 ++-- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git

[PATCH 1/3] staging: comedi: cb_pcidas64: use preferred kernel type u8

2016-12-16 Thread Saber Rezvani
Fix the checkpatch.pl issue: CHECK: Prefer kernel type 'u8' over 'uint8_t' Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/cb_pcidas64.c | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git

[PATCH 2/3] staging: comedi: cb_pcidas64: use preferred kernel type u16

2016-12-16 Thread Saber Rezvani
Fix the checkpatch.pl issue: CHECK: Prefer kernel type 'u16' over 'uint16_t' Signed-off-by: Saber Rezvani --- drivers/staging/comedi/drivers/cb_pcidas64.c | 58 ++-- 1 file changed, 29 insertions(+), 29 deletions(-) diff --git

Re: [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Dan Carpenter
On Fri, Dec 16, 2016 at 10:43:24AM -0800, Joe Perches wrote: > On Fri, 2016-12-16 at 23:29 +0530, Tabrez khan wrote: > > Remove unnecessary braces {} from single line if statement. > > This warning is found using checkpatch.pl. > [] > > diff --git a/drivers/staging/lustre/lustre/ptlrpc/import.c >

Re: [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Joe Perches
On Fri, 2016-12-16 at 23:29 +0530, Tabrez khan wrote: > Remove unnecessary braces {} from single line if statement. > This warning is found using checkpatch.pl. [] > diff --git a/drivers/staging/lustre/lustre/ptlrpc/import.c > b/drivers/staging/lustre/lustre/ptlrpc/import.c [] > @@ -1134,9

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-16 Thread KY Srinivasan
> -Original Message- > From: Haiyang Zhang > Sent: Friday, December 16, 2016 7:21 AM > To: KY Srinivasan ; Stephen Hemminger > ; Greg KH > Cc: o...@aepfle.de; jasow...@redhat.com; linux-ker...@vger.kernel.org; >

[PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Tabrez khan
Remove unnecessary braces {} from single line if statement. This warning is found using checkpatch.pl. Signed-off-by: Tabrez khan --- drivers/staging/lustre/lustre/ptlrpc/import.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [lustre-devel] [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Xiong, Jinshan
> On Dec 16, 2016, at 8:05 AM, Joe Perches wrote: > > On Fri, 2016-12-16 at 19:59 +0530, Tabrez khan wrote: >> Remove unnecessary braces {} for single while statement. > > Your patch is fine Tabrez, but to the lustre folk: > >> diff --git

[PATCH v5 0/8] staging: fsl-mc: add dpio driver

2016-12-16 Thread Stuart Yoder
This patch series adds the driver for the DPIO object which is a step to addressing the final item in the staging TODO list-- adding a functional driver on top of the bus driver. The DPIO driver is a dependency for other functional drivers such as Ethernet. An overview of the DPIO object and

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-16 Thread Greg KH
On Wed, Dec 14, 2016 at 03:51:34PM -0800, Stephen Hemminger wrote: > On Wed, 14 Dec 2016 15:27:58 -0800 > Greg KH wrote: > > > On Wed, Dec 14, 2016 at 11:18:59PM +, Haiyang Zhang wrote: > > > > > > > > > > -Original Message- > > > > From: Greg KH

[PATCH v5 2/8] bus: fsl-mc: dpio: add APIs for DPIO objects

2016-12-16 Thread Stuart Yoder
From: Ioana Radulescu Add the command build/parse APIs for operating on DPIO objects through the DPAA2 Management Complex. Signed-off-by: Ioana Radulescu Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder

[PATCH v5 5/8] bus: fsl-mc: dpio: add QBMan portal APIs for DPAA2

2016-12-16 Thread Stuart Yoder
From: Roy Pledge Add QBman APIs for frame queue and buffer pool operations. Signed-off-by: Roy Pledge Signed-off-by: Haiying Wang Signed-off-by: Stuart Yoder --- Notes: -v5 -fixed typo that caused

[PATCH v5 7/8] bus: fsl-mc: dpio: add the DPAA2 DPIO object driver

2016-12-16 Thread Stuart Yoder
From: Roy Pledge The DPIO driver registers with the fsl-mc bus to handle bus-related events for DPIO objects. Key responsibility is mapping I/O regions, setting up interrupt handlers, and calling the DPIO service initialization during probe. Signed-off-by: Roy Pledge

[PATCH v5 8/8] bus: fsl-mc: dpio: add maintainer for DPIO

2016-12-16 Thread Stuart Yoder
From: Roy Pledge add Roy Pledge as maintainer of DPIO Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- Notes: -v4 -adjust file location to be in drivers/staging -v3 -no changes -v2

[PATCH v5 6/8] bus: fsl-mc: dpio: add the DPAA2 DPIO service interface

2016-12-16 Thread Stuart Yoder
From: Roy Pledge The DPIO service interface handles initialization of DPIO objects and exports APIs to be used by other DPAA2 object drivers to perform queuing and buffer management related operations. The service allows registration of callbacks when frames or notifications

[PATCH v5 3/8] bus: fsl-mc: dpio: add frame descriptor and scatter/gather APIs

2016-12-16 Thread Stuart Yoder
From: Roy Pledge Add global definitions for DPAA2 frame descriptors and scatter gather entries. Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- Notes: -v4 -updated copyright -adjust file location to be

[PATCH v5 4/8] bus: fsl-mc: dpio: add global dpaa2 definitions

2016-12-16 Thread Stuart Yoder
From: Roy Pledge Create header for global dpaa2 definitions. Add definitions for dequeue results. Signed-off-by: Roy Pledge Signed-off-by: Stuart Yoder --- Notes: -v4 -adjust file location to be in drivers/staging

[PATCH v5 1/8] bus: fsl-mc: dpio: add DPIO driver overview document

2016-12-16 Thread Stuart Yoder
add document describing the dpio driver and it's role, components and major interfaces Signed-off-by: Stuart Yoder --- Notes: -v4 -updated copyright drivers/staging/fsl-mc/bus/dpio/dpio-driver.txt | 135 1 file changed, 135

RE: [PATCH v4 5/8] bus: fsl-mc: dpio: add QBMan portal APIs for DPAA2

2016-12-16 Thread Stuart Yoder
> -Original Message- > From: kbuild test robot [mailto:l...@intel.com] > Sent: Friday, December 16, 2016 1:57 AM > To: Stuart Yoder > Cc: kbuild-...@01.org; gre...@linuxfoundation.org; > de...@driverdev.osuosl.org; ag...@suse.de; > a...@arndb.de; Haiying Wang

Re: [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Joe Perches
On Fri, 2016-12-16 at 19:59 +0530, Tabrez khan wrote: > Remove unnecessary braces {} for single while statement. Your patch is fine Tabrez, but to the lustre folk: > diff --git a/drivers/staging/lustre/lustre/obdclass/cl_io.c > b/drivers/staging/lustre/lustre/obdclass/cl_io.c [] > @@ -1371,9

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-16 Thread Haiyang Zhang
> -Original Message- > From: KY Srinivasan > Sent: Thursday, December 15, 2016 8:11 PM > To: Stephen Hemminger ; Greg KH > > Cc: o...@aepfle.de; jasow...@redhat.com; linux-ker...@vger.kernel.org; > bjorn.helg...@gmail.com;

[PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Tabrez khan
Remove unnecessary braces {} for single while statement. This warning is found using checkpatch.pl. Signed-off-by: Tabrez khan --- drivers/staging/lustre/lustre/obdclass/cl_io.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] staging: emxx_udc: remove incorrect __init annotations

2016-12-16 Thread Arnd Bergmann
The probe function is not marked __init, but some other functions are. This leads to a warning on older compilers (e.g. gcc-4.3), and can cause executing freed memory when built with those compilers: WARNING: drivers/staging/emxx_udc/emxx_udc.o(.text+0x2d78): Section mismatch in reference from