Re: [Linaro-mm-sig] [PATCH v3] staging: android: ion: Zero CMA allocated memory

2018-01-26 Thread Laura Abbott
On 01/26/2018 06:04 PM, Chen Feng wrote: On 2018/1/27 1:48, Liam Mark wrote: Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly") the CMA API is now used directly and therefore the allocated memory is no longer automatically zeroed. Explicitly zero CMA allocated memory

Re: [Linaro-mm-sig] [PATCH v3] staging: android: ion: Zero CMA allocated memory

2018-01-26 Thread Chen Feng
On 2018/1/27 1:48, Liam Mark wrote: > Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly") > the CMA API is now used directly and therefore the allocated memory is no > longer automatically zeroed. > > Explicitly zero CMA allocated memory to ensure that no data is exposed

[PATCH] hv_vmbus: Correct the stale comments regarding cpu affinity

2018-01-26 Thread Haiyang Zhang
The comments doesn't match what the current code does, also have a typo. This patch corrects them. Signed-off-by: Haiyang Zhang --- drivers/hv/channel_mgmt.c | 6 ++ include/linux/hyperv.h| 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git

Re: [PATCH v2 char-misc 1/1] Drivers: hv: vmbus: Implement Direct Mode for stimer0

2018-01-26 Thread Greg KH
On Fri, Jan 26, 2018 at 05:34:28PM +, Michael Kelley (EOSG) wrote: > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Thursday, January 25, 2018 2:00 AM > > To: Michael Kelley (EOSG) > > Cc:

Re: [RFC PATCH 0/2] hv_netvsc: Fix shutdown regression on Win2012 hosts

2018-01-26 Thread Stephen Hemminger
On Tue, 23 Jan 2018 10:34:03 +0100 Mohammed Gamal wrote: > Commit 0cf737808ae7 ("hv_netvsc: netvsc_teardown_gpadl() split") introduced > a regression that caused VMs not to shutdown after netvsc_device_remove() is > called. This is caused by GPADL teardown sequence change, and

[PATCH v3] staging: android: ion: Zero CMA allocated memory

2018-01-26 Thread Liam Mark
Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly") the CMA API is now used directly and therefore the allocated memory is no longer automatically zeroed. Explicitly zero CMA allocated memory to ensure that no data is exposed to userspace. Fixes: 204f672255c2 ("staging:

RE: [PATCH v2 char-misc 1/1] Drivers: hv: vmbus: Implement Direct Mode for stimer0

2018-01-26 Thread Michael Kelley (EOSG)
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, January 25, 2018 2:00 AM > To: Michael Kelley (EOSG) > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; > a...@canonical.com;

Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig

2018-01-26 Thread Andy Shevchenko
On Fri, Jan 26, 2018 at 2:49 PM, LABBE Corentin wrote: > On Tue, Jan 23, 2018 at 07:20:12PM +0100, Greg Kroah-Hartman wrote: >> On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: >> > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe >> >

[PATCH] media: staging: atomisp: remove redundant assignments to various variables

2018-01-26 Thread Colin King
From: Colin Ian King There are various assignments that are being made to variables that are not read and the variables are being updated later on, hence the redundant assignments can be removed. Cleans up clang warnings:

RE: [PATCH v5 1/2] staging: fsl-mc: Move core bus out of staging

2018-01-26 Thread Bogdan Purcareata
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Friday, January 26, 2018 4:06 PM > To: Bogdan Purcareata > Cc: Laurentiu Tudor ; Ruxandra Ioana Ciocoi Radulescu > ;

Re: [PATCH v5 1/2] staging: fsl-mc: Move core bus out of staging

2018-01-26 Thread Greg KH
On Fri, Jan 26, 2018 at 06:51:26AM -0600, Bogdan Purcareata wrote: > Move the source files out of staging into their final locations: > -mc.h include file in drivers/staging/fsl-mc/include go to include/linux/fsl > -source files in drivers/staging/fsl-mc/bus go to drivers/bus/fsl-mc >

[PATCH] staging: rtlwifi: remove redundant initialization of 'cfg_cmd'

2018-01-26 Thread Colin King
From: Colin Ian King The initialization of cfg_cmd is redundant as the value is never read and it is being re-assigned to cfg_cmd = pwrcfgcmd[ary_idx] inside a loop, hence it can be removed. Cleans up clang warning: drivers/staging/rtlwifi/core.c:1819:22: warning:

Re: [PATCH v5 2/2] staging: fsl-mc: Move irqchip code out of staging

2018-01-26 Thread Marc Zyngier
On 26/01/18 12:51, Bogdan Purcareata wrote: > Now that the fsl-mc bus core infrastructure is out of staging, the > remaining irqchip glue code used (irq-gic-v3-its-fsl-mc-msi.c) goes > to drivers/irqchip. > > Signed-off-by: Stuart Yoder > [rebased, add dpaa2_eth and dpio

[PATCH v5 0/2] staging: fsl-mc: Move bus driver out of staging

2018-01-26 Thread Bogdan Purcareata
Previous submission: https://patchwork.kernel.org/patch/10081731/ Most of the feedback was addressed by a separate series of cleanup / refactoring patches [1] which were applied. The thing left to do is to split the original patch into moving out the core bus infrastructure separate from the

[PATCH v5 2/2] staging: fsl-mc: Move irqchip code out of staging

2018-01-26 Thread Bogdan Purcareata
Now that the fsl-mc bus core infrastructure is out of staging, the remaining irqchip glue code used (irq-gic-v3-its-fsl-mc-msi.c) goes to drivers/irqchip. Signed-off-by: Stuart Yoder [rebased, add dpaa2_eth and dpio #include updates] Signed-off-by: Laurentiu Tudor

Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig

2018-01-26 Thread LABBE Corentin
On Tue, Jan 23, 2018 at 07:20:12PM +0100, Greg Kroah-Hartman wrote: > On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe wrote: > > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 > > >

[PATCH] staging: rtl8723bs: remove a couple of redundant initializations

2018-01-26 Thread Colin King
From: Colin Ian King The initialization of PwrCfgCmd is redundant as the value is never read and it is being re-assigned to PwrSeqCmd[AryIdx] inside a loop, hence it can be removed. Also, the initialization of ie_ptr is redundant as the value is never read and it is