RE: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread KY Srinivasan
> -Original Message- > From: Greg KH > Sent: Tuesday, October 16, 2018 10:04 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Stephen > Hemminger ; Michael Kelley (EOSG) > ; vkuznets ; >

[v5 3/4] mpt3sas:Fix driver modifying persistent data.

2018-10-16 Thread Suganath Prabu
* If EEDPTagMode field in manufacturing page11 is set, unset it. This is needed to fix a hardware bug in SAS3/SAS2 cards, So, skipping EEDPTagMode changes in Manufacturing page11 for SAS35 controllers. * Fix driver modifying NVRAM/persistent data in Manufacturing page11 along with current copy.

RE: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread KY Srinivasan
> -Original Message- > From: Greg KH > Sent: Tuesday, October 16, 2018 10:04 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Stephen > Hemminger ; Michael Kelley (EOSG) > ; vkuznets ; >

[v5 3/4] mpt3sas:Fix driver modifying persistent data.

2018-10-16 Thread Suganath Prabu
* If EEDPTagMode field in manufacturing page11 is set, unset it. This is needed to fix a hardware bug in SAS3/SAS2 cards, So, skipping EEDPTagMode changes in Manufacturing page11 for SAS35 controllers. * Fix driver modifying NVRAM/persistent data in Manufacturing page11 along with current copy.

Fate of ihex2fw tool

2018-10-16 Thread Andrey Smirnov
Everyone: Since commit 5620a0d1aacd554ebebcff373e31107bb1ef7769 ("firmware: delete in-kernel firmware"): https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.19-rc7=5620a0d1aacd554ebebcff373e31107bb1ef7769 firmware/ihex2fw.c is gone from the source tree and with it

Fate of ihex2fw tool

2018-10-16 Thread Andrey Smirnov
Everyone: Since commit 5620a0d1aacd554ebebcff373e31107bb1ef7769 ("firmware: delete in-kernel firmware"): https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.19-rc7=5620a0d1aacd554ebebcff373e31107bb1ef7769 firmware/ihex2fw.c is gone from the source tree and with it

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Gustavo A. R. Silva
On 10/17/18 7:07 AM, Greg KH wrote: > On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: >> From: Dexuan Cui >> >> In kvp_send_key(), we do need call process_ib_ipinfo() if >> message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out >> the userland

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Gustavo A. R. Silva
On 10/17/18 7:07 AM, Greg KH wrote: > On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: >> From: Dexuan Cui >> >> In kvp_send_key(), we do need call process_ib_ipinfo() if >> message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out >> the userland

Re: [PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:06AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > The patch fixes: > > hv_kvp_daemon.c: In function 'kvp_set_ip_info': > hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes > into a destination of size 4096 > > Signed-off-by:

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > In kvp_send_key(), we do need call process_ib_ipinfo() if > message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out > the userland hv_kvp_daemon needs the info of operation, adapter_id

Re: [PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:06AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > The patch fixes: > > hv_kvp_daemon.c: In function 'kvp_set_ip_info': > hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes > into a destination of size 4096 > > Signed-off-by:

Re: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:04AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > In kvp_send_key(), we do need call process_ib_ipinfo() if > message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out > the userland hv_kvp_daemon needs the info of operation, adapter_id

Re: [PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:02AM +, k...@linuxonhyperv.com wrote: > From: "K. Y. Srinivasan" > > Currently we are replicating state in struct hv_context that is unnecessary - > this state can be retrieved from the hypervisor. Furthermore, this is a > per-cpu > state that is being

Re: [PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:02AM +, k...@linuxonhyperv.com wrote: > From: "K. Y. Srinivasan" > > Currently we are replicating state in struct hv_context that is unnecessary - > this state can be retrieved from the hypervisor. Furthermore, this is a > per-cpu > state that is being

Re: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:05AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > I didn't find a real issue. Let's just make it consistent with the > next "case REG_U64:" where %llu is used. > > Signed-off-by: Dexuan Cui > Cc: K. Y. Srinivasan > Cc: Haiyang Zhang > Cc: Stephen

Re: [PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread Greg KH
On Wed, Oct 17, 2018 at 03:14:05AM +, k...@linuxonhyperv.com wrote: > From: Dexuan Cui > > I didn't find a real issue. Let's just make it consistent with the > next "case REG_U64:" where %llu is used. > > Signed-off-by: Dexuan Cui > Cc: K. Y. Srinivasan > Cc: Haiyang Zhang > Cc: Stephen

Re: [PATCH v2 6/6] arm64: platform: Add enable Exynos Multi-Core Timer driver

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > On Exynos SoCs enabling MCT driver is required even if ARM Architected > Timer driver is used to for managing timer hardware and clock source > events. > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/Kconfig.platforms | 1 + >

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Nathan Chancellor
Hi Masahiro, On Wed, Oct 17, 2018 at 01:48:46PM +0900, Masahiro Yamada wrote: > Hi Nathan, > > > On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor > wrote: > > > > There are only a few instances of this warning in an arm64 allyesconfig > > build but none of them appear useful. I believe the

Re: [PATCH v2 6/6] arm64: platform: Add enable Exynos Multi-Core Timer driver

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > On Exynos SoCs enabling MCT driver is required even if ARM Architected > Timer driver is used to for managing timer hardware and clock source > events. > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/Kconfig.platforms | 1 + >

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Nathan Chancellor
Hi Masahiro, On Wed, Oct 17, 2018 at 01:48:46PM +0900, Masahiro Yamada wrote: > Hi Nathan, > > > On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor > wrote: > > > > There are only a few instances of this warning in an arm64 allyesconfig > > build but none of them appear useful. I believe the

Re: [PATCH v2 5/6] arm64: dts: exynos: Move arch-timer node to right place

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > Move ARM architected timer device-tree node to the beginning of 'soc' > node, to group it together with other ARM CPU core devices (like PMU). > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 23

Re: [PATCH v2 5/6] arm64: dts: exynos: Move arch-timer node to right place

2018-10-16 Thread Chanwoo Choi
Hi Marek, On 2018년 10월 15일 21:31, Marek Szyprowski wrote: > Move ARM architected timer device-tree node to the beginning of 'soc' > node, to group it together with other ARM CPU core devices (like PMU). > > Signed-off-by: Marek Szyprowski > --- > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 23

Re: [PATCH v2 3/6] clocksource: exynos_mct: Add arch_timer cooperation mode for ARM64

2018-10-16 Thread Chanwoo Choi
Hi Marek, I tested it about kernel booting and CPU hotplug through sysfs path on ARM64 Exynos5433-based TM2 board. It is well working. Reviewed-by: Chanwoo Choi Tested-by: Chanwoo Choi But, I have a minor comment for code clean-up. On exynos4_mct_starting_cpu() function, the initialization

Re: [PATCH v2 3/6] clocksource: exynos_mct: Add arch_timer cooperation mode for ARM64

2018-10-16 Thread Chanwoo Choi
Hi Marek, I tested it about kernel booting and CPU hotplug through sysfs path on ARM64 Exynos5433-based TM2 board. It is well working. Reviewed-by: Chanwoo Choi Tested-by: Chanwoo Choi But, I have a minor comment for code clean-up. On exynos4_mct_starting_cpu() function, the initialization

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Masahiro Yamada
Hi Nathan, On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor wrote: > > There are only a few instances of this warning in an arm64 allyesconfig > build but none of them appear useful. I believe the intention of the > warning is to avoid situations like this: > > if (condition); >

Re: [PATCH] Kbuild: Hide Clang's -Wempty-body behind W=1

2018-10-16 Thread Masahiro Yamada
Hi Nathan, On Tue, Oct 16, 2018 at 11:15 AM Nathan Chancellor wrote: > > There are only a few instances of this warning in an arm64 allyesconfig > build but none of them appear useful. I believe the intention of the > warning is to avoid situations like this: > > if (condition); >

Re: [RFC][PATCHv2 1/4] panic: avoid deadlocks in re-entrant console drivers

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:04), Sergey Senozhatsky wrote: > > Fix this by setting oops_in_progress before console_flush_on_panic(), > so re-entrant console drivers will trylock the port->lock instead of > spinning on it forever. > Just a small note: Regardless of what's going to happen to the series,

Re: [RFC][PATCHv2 1/4] panic: avoid deadlocks in re-entrant console drivers

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:04), Sergey Senozhatsky wrote: > > Fix this by setting oops_in_progress before console_flush_on_panic(), > so re-entrant console drivers will trylock the port->lock instead of > spinning on it forever. > Just a small note: Regardless of what's going to happen to the series,

Re: [Lkcamp] [PATCH 0/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patches. On 10/16/18 9:07 PM, Leonardo Brás wrote: > This patchset add -Wshadow=local on KBUILD_HOSTCFLAGS and fixes > all code that show this warning. > > The third patch was already submitted, but was not merged yet. > I like to think it's part of this patchset,

Re: [Lkcamp] [PATCH 0/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patches. On 10/16/18 9:07 PM, Leonardo Brás wrote: > This patchset add -Wshadow=local on KBUILD_HOSTCFLAGS and fixes > all code that show this warning. > > The third patch was already submitted, but was not merged yet. > I like to think it's part of this patchset,

Re: [RFC][PATCHv2 2/4] printk: move printk_safe macros to printk header

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:54), Peter Zijlstra wrote: > On Tue, Oct 16, 2018 at 09:27:34PM +0900, Sergey Senozhatsky wrote: > > per-CPU printk_safe _semi-magic_ makes some things simple to handle. > > We can't just remove per-CPU buffers and add a wake_up_process() at > > the bottom of vprintk_emit().

Re: [RFC][PATCHv2 2/4] printk: move printk_safe macros to printk header

2018-10-16 Thread Sergey Senozhatsky
On (10/16/18 14:54), Peter Zijlstra wrote: > On Tue, Oct 16, 2018 at 09:27:34PM +0900, Sergey Senozhatsky wrote: > > per-CPU printk_safe _semi-magic_ makes some things simple to handle. > > We can't just remove per-CPU buffers and add a wake_up_process() at > > the bottom of vprintk_emit().

Re: [Lkcamp] [PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Adds -Wshadow=local on KBUILD_HOSTCFLAGS to show shadow warnings > on tools built for HOST. > > Signed-off-by: Leonardo Brás > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Lkcamp] [PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Adds -Wshadow=local on KBUILD_HOSTCFLAGS to show shadow warnings > on tools built for HOST. > > Signed-off-by: Leonardo Brás > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Lkcamp] [PATCH 4/4] Changes macro usage to avoid shadowing a variable.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. On 10/16/18 9:09 PM, Leonardo Brás wrote: > Changes the usage of DEF_FIELD_ADDR in this function to create a > reference and operate over it using an aux variable. > It also changes the loop logic used to find duplicates, to

Re: [Lkcamp] [PATCH 4/4] Changes macro usage to avoid shadowing a variable.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. On 10/16/18 9:09 PM, Leonardo Brás wrote: > Changes the usage of DEF_FIELD_ADDR in this function to create a > reference and operate over it using an aux variable. > It also changes the loop logic used to find duplicates, to

[PATCH] perf: Add Sparc jitdump support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/Makefile | 2 ++ tools/perf/util/genelf.h | 6 ++ 2 files changed, 8 insertions(+) diff --git a/tools/perf/arch/sparc/Makefile b/tools/perf/arch/sparc/Makefile index 7fbca175099e..275dea7ff59a 100644 ---

[PATCH] perf: Add Sparc jitdump support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/Makefile | 2 ++ tools/perf/util/genelf.h | 6 ++ 2 files changed, 8 insertions(+) diff --git a/tools/perf/arch/sparc/Makefile b/tools/perf/arch/sparc/Makefile index 7fbca175099e..275dea7ff59a 100644 ---

[[PATCH] 4/9] Documentation-DTbindings-DMA-controller-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- .../devicetree/bindings/dma/ast-uart-sdma.txt | 23 ++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma/ast-uart-sdma.txt diff --git a/Documentation/devicetree/bindings/dma/ast-uart-sdma.txt

[[PATCH] 2/9] Defconfig-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/configs/aspeed_g5_defconfig | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm/configs/aspeed_g5_defconfig b/arch/arm/configs/aspeed_g5_defconfig index b7f8fa1..25813b5 100644 --- a/arch/arm/configs/aspeed_g5_defconfig +++

[[PATCH] 8/9] DMA-UART-Driver-for-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- drivers/tty/serial/8250/8250_aspeed_uart_dma.c | 1594 1 file changed, 1594 insertions(+) create mode 100644 drivers/tty/serial/8250/8250_aspeed_uart_dma.c diff --git a/drivers/tty/serial/8250/8250_aspeed_uart_dma.c

[[PATCH] 2/9] Defconfig-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/configs/aspeed_g5_defconfig | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm/configs/aspeed_g5_defconfig b/arch/arm/configs/aspeed_g5_defconfig index b7f8fa1..25813b5 100644 --- a/arch/arm/configs/aspeed_g5_defconfig +++

[[PATCH] 8/9] DMA-UART-Driver-for-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- drivers/tty/serial/8250/8250_aspeed_uart_dma.c | 1594 1 file changed, 1594 insertions(+) create mode 100644 drivers/tty/serial/8250/8250_aspeed_uart_dma.c diff --git a/drivers/tty/serial/8250/8250_aspeed_uart_dma.c

[[PATCH] 4/9] Documentation-DTbindings-DMA-controller-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- .../devicetree/bindings/dma/ast-uart-sdma.txt | 23 ++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/dma/ast-uart-sdma.txt diff --git a/Documentation/devicetree/bindings/dma/ast-uart-sdma.txt

[[PATCH] 0/9] *** DMA support for UART in ASPEED's AST2500 ***

2018-10-16 Thread sudheer.v
DMA controller driver and UART dma client driver for aspeed's AST2500 sudheer.v (9): DT-changes-for-DMA-UART-of-AST2500 Defconfig-changes-for-DMA-UART-of-AST2500 configuration-for-DMA-of-AST2500 Documentation-DTbindings-DMA-controller-of-AST2500 DMA-driver-for-AST2500

[[PATCH] 0/9] *** DMA support for UART in ASPEED's AST2500 ***

2018-10-16 Thread sudheer.v
DMA controller driver and UART dma client driver for aspeed's AST2500 sudheer.v (9): DT-changes-for-DMA-UART-of-AST2500 Defconfig-changes-for-DMA-UART-of-AST2500 configuration-for-DMA-of-AST2500 Documentation-DTbindings-DMA-controller-of-AST2500 DMA-driver-for-AST2500

[[PATCH] 1/9] DT-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/boot/dts/aspeed-ast2500-evb.dts | 20 arch/arm/boot/dts/aspeed-g5.dtsi | 85 2 files changed, 105 insertions(+) diff --git a/arch/arm/boot/dts/aspeed-ast2500-evb.dts

[[PATCH] 1/9] DT-changes-for-DMA-UART-of-AST2500

2018-10-16 Thread sudheer.v
Signed-off-by: sudheer.v --- arch/arm/boot/dts/aspeed-ast2500-evb.dts | 20 arch/arm/boot/dts/aspeed-g5.dtsi | 85 2 files changed, 105 insertions(+) diff --git a/arch/arm/boot/dts/aspeed-ast2500-evb.dts

[PATCH] perf: Add Sparc annotate support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/annotate/instructions.c | 169 ++ tools/perf/util/annotate.c| 8 + 2 files changed, 177 insertions(+) create mode 100644 tools/perf/arch/sparc/annotate/instructions.c diff --git

Re: [PATCH 4.14 000/109] 4.14.77-stable review

2018-10-16 Thread Dan Rue
On Tue, Oct 16, 2018 at 07:04:28PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.77 release. > There are 109 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me

Re: [PATCH 4.14 000/109] 4.14.77-stable review

2018-10-16 Thread Dan Rue
On Tue, Oct 16, 2018 at 07:04:28PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.77 release. > There are 109 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me

[PATCH] perf: Add Sparc annotate support.

2018-10-16 Thread David Miller
Signed-off-by: David S. Miller --- tools/perf/arch/sparc/annotate/instructions.c | 169 ++ tools/perf/util/annotate.c| 8 + 2 files changed, 177 insertions(+) create mode 100644 tools/perf/arch/sparc/annotate/instructions.c diff --git

RE: [PATCH v3 1/2] mtd: spi-nor: add support to non-uniform SFDP SPI NOR flash memories

2018-10-16 Thread Yogesh Narayan Gaur
Hi Tudor, > -Original Message- > From: Yogesh Narayan Gaur > Sent: Wednesday, October 17, 2018 7:38 AM > To: 'Cyrille Pitchen' ; Tudor Ambarus > ; marek.va...@gmail.com; > dw...@infradead.org; computersforpe...@gmail.com; > boris.brezil...@bootlin.com; rich...@nod.at > Cc:

RE: [PATCH v3 1/2] mtd: spi-nor: add support to non-uniform SFDP SPI NOR flash memories

2018-10-16 Thread Yogesh Narayan Gaur
Hi Tudor, > -Original Message- > From: Yogesh Narayan Gaur > Sent: Wednesday, October 17, 2018 7:38 AM > To: 'Cyrille Pitchen' ; Tudor Ambarus > ; marek.va...@gmail.com; > dw...@infradead.org; computersforpe...@gmail.com; > boris.brezil...@bootlin.com; rich...@nod.at > Cc:

Re: [PATCH v4 3/4] perf/smmuv3: Add MSI irq support

2018-10-16 Thread kbuild test robot
Hi Shameer, Thank you for the patch! Yet something to improve: [auto build test ERROR on linux-sof-driver/master] [also build test ERROR on v4.19-rc8 next-20181016] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com

Re: [PATCH v4 3/4] perf/smmuv3: Add MSI irq support

2018-10-16 Thread kbuild test robot
Hi Shameer, Thank you for the patch! Yet something to improve: [auto build test ERROR on linux-sof-driver/master] [also build test ERROR on v4.19-rc8 next-20181016] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com

[PATCH] selftests/ftrace: Strip escape sequences for log file

2018-10-16 Thread Masami Hiramatsu
Strip escape sequences from the stream to the ftracetest summary log file. Note that all test-case results are dumped raw in each file. Signed-off-by: Masami Hiramatsu --- tools/testing/selftests/ftrace/ftracetest | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git

[PATCH] selftests/ftrace: Strip escape sequences for log file

2018-10-16 Thread Masami Hiramatsu
Strip escape sequences from the stream to the ftracetest summary log file. Note that all test-case results are dumped raw in each file. Signed-off-by: Masami Hiramatsu --- tools/testing/selftests/ftrace/ftracetest | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 0/4] Add clock support for Hi3670 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Thu, Sep 20, 2018 at 11:00:59PM -0700, Manivannan Sadhasivam wrote: > This patchset adds clock support for Hi3670 SoC from HiSilicon utilizing > the HiSi common clock code. While adding clock support, let's remove the > fixed clock for UART and source SoC clock on HiKey970 board. > > This

Re: [PATCH 0/4] Add clock support for Hi3670 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Thu, Sep 20, 2018 at 11:00:59PM -0700, Manivannan Sadhasivam wrote: > This patchset adds clock support for Hi3670 SoC from HiSilicon utilizing > the HiSi common clock code. While adding clock support, let's remove the > fixed clock for UART and source SoC clock on HiKey970 board. > > This

Re: [PATCH 11/19] PCI: keystone: Cleanup PHY handling

2018-10-16 Thread Kishon Vijay Abraham I
Lorenzo, On Tuesday 16 October 2018 10:36 PM, Lorenzo Pieralisi wrote: > On Mon, Oct 15, 2018 at 06:37:13PM +0530, Kishon Vijay Abraham I wrote: >> Cleanup PHY handling by using devm_phy_optional_get to get PHYs if >> the PHYs are optional, creating a device link between the PHY device >> and the

Re: [PATCH 11/19] PCI: keystone: Cleanup PHY handling

2018-10-16 Thread Kishon Vijay Abraham I
Lorenzo, On Tuesday 16 October 2018 10:36 PM, Lorenzo Pieralisi wrote: > On Mon, Oct 15, 2018 at 06:37:13PM +0530, Kishon Vijay Abraham I wrote: >> Cleanup PHY handling by using devm_phy_optional_get to get PHYs if >> the PHYs are optional, creating a device link between the PHY device >> and the

[PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread kys
From: Dexuan Cui In kvp_send_key(), we do need call process_ib_ipinfo() if message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out the userland hv_kvp_daemon needs the info of operation, adapter_id and addr_family. With the incorrect fc62c3b1977d, the host can't get the VM's IP

[PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Currently we are replicating state in struct hv_context that is unnecessary - this state can be retrieved from the hypervisor. Furthermore, this is a per-cpu state that is being maintained as a global state in struct hv_context. Get rid of this state in struct

[PATCH 2/5] hv_utils: update name in struct hv_driver util_drv

2018-10-16 Thread kys
From: Haiyang Zhang The correct module name is hv_utils. This patch corrects the name in struct hv_driver util_drv. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hv/hv_util.c

[PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-10-16 Thread kys
From: Dexuan Cui In kvp_send_key(), we do need call process_ib_ipinfo() if message->kvp_hdr.operation is KVP_OP_GET_IP_INFO, because it turns out the userland hv_kvp_daemon needs the info of operation, adapter_id and addr_family. With the incorrect fc62c3b1977d, the host can't get the VM's IP

[PATCH 1/5] Drivers: hv: vmbus: Get rid of unnecessary state in hv_context

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Currently we are replicating state in struct hv_context that is unnecessary - this state can be retrieved from the hypervisor. Furthermore, this is a per-cpu state that is being maintained as a global state in struct hv_context. Get rid of this state in struct

[PATCH 2/5] hv_utils: update name in struct hv_driver util_drv

2018-10-16 Thread kys
From: Haiyang Zhang The correct module name is hv_utils. This patch corrects the name in struct hv_driver util_drv. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hv/hv_util.c

[PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread kys
From: Dexuan Cui The patch fixes: hv_kvp_daemon.c: In function 'kvp_set_ip_info': hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes into a destination of size 4096 Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc:

[PATCH 5/5] Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1

2018-10-16 Thread kys
From: Dexuan Cui The patch fixes: hv_kvp_daemon.c: In function 'kvp_set_ip_info': hv_kvp_daemon.c:1305:2: note: 'snprintf' output between 41 and 4136 bytes into a destination of size 4096 Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc:

[PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread kys
From: Dexuan Cui I didn't find a real issue. Let's just make it consistent with the next "case REG_U64:" where %llu is used. Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc: Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_kvp.c | 2 +- 1 file

[PATCH 4/5] Drivers: hv: kvp: Use %u to print U32

2018-10-16 Thread kys
From: Dexuan Cui I didn't find a real issue. Let's just make it consistent with the next "case REG_U64:" where %llu is used. Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Cc: Haiyang Zhang Cc: Stephen Hemminger Cc: Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_kvp.c | 2 +- 1 file

[PATCH 0/5] Drivers: hv: Miscellaneous fixes

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Miscellaneous fixes. Dexuan Cui (3): Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up Drivers: hv: kvp: Use %u to print U32 Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1 Haiyang Zhang (1): hv_utils: update name in

[PATCH 0/5] Drivers: hv: Miscellaneous fixes

2018-10-16 Thread kys
From: "K. Y. Srinivasan" Miscellaneous fixes. Dexuan Cui (3): Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up Drivers: hv: kvp: Use %u to print U32 Tools: hv: kvp: Fix a warning of buffer overflow with gcc 8.0.1 Haiyang Zhang (1): hv_utils: update name in

Re: [Lkcamp] [PATCH 3/4] kbuild: Removes unnecessary shadowed local variable and optimize testing.

2018-10-16 Thread Helen Koike
Hi Leonardo, On 10/16/18 9:09 PM, Leonardo Brás wrote: > Removes an unnecessary shadowed local variable (start). > Optimize test of isdigit: > - If isalpha returns true, isdigit will return false, so no need to test. This patch does two different things, it should be in two separated

Re: [Lkcamp] [PATCH 3/4] kbuild: Removes unnecessary shadowed local variable and optimize testing.

2018-10-16 Thread Helen Koike
Hi Leonardo, On 10/16/18 9:09 PM, Leonardo Brás wrote: > Removes an unnecessary shadowed local variable (start). > Optimize test of isdigit: > - If isalpha returns true, isdigit will return false, so no need to test. This patch does two different things, it should be in two separated

Re: [PATCH v3 6/9] arm64: dts: actions: Add Reset Controller support for S900 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Tue, Oct 16, 2018 at 02:42:19PM -0700, Stephen Boyd wrote: > Quoting Manivannan Sadhasivam (2018-08-10 02:51:10) > > Add reset controller property and bindings header for the > > Actions Semi S900 SoC DTS. > > > > Signed-off-by: Manivannan Sadhasivam > > --- > > Are these DT patches

Re: [PATCH v3 6/9] arm64: dts: actions: Add Reset Controller support for S900 SoC

2018-10-16 Thread Manivannan Sadhasivam
On Tue, Oct 16, 2018 at 02:42:19PM -0700, Stephen Boyd wrote: > Quoting Manivannan Sadhasivam (2018-08-10 02:51:10) > > Add reset controller property and bindings header for the > > Actions Semi S900 SoC DTS. > > > > Signed-off-by: Manivannan Sadhasivam > > --- > > Are these DT patches

Re: [Lkcamp] [PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. Please, check previous log messages with git log arch/x86/entry/vdso/vdso2c.h, you will see that most patches had the prefix "x86/vdso:" in the commit message. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Renames the char

Re: [Lkcamp] [PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Helen Koike
Hi Leonardo, Thanks for the patch, just some small comments below. Please, check previous log messages with git log arch/x86/entry/vdso/vdso2c.h, you will see that most patches had the prefix "x86/vdso:" in the commit message. On 10/16/18 9:08 PM, Leonardo Brás wrote: > Renames the char

Re: [PATCH v9 5/5] lib/dlock-list: Scale dlock_lists_empty()

2018-10-16 Thread Davidlohr Bueso
On Thu, 04 Oct 2018, Waiman Long wrote: On 10/04/2018 03:16 AM, Jan Kara wrote: On Wed 12-09-18 15:28:52, Waiman Long wrote: From: Davidlohr Bueso Instead of the current O(N) implementation, at the cost of adding an atomic counter, we can convert the call to an atomic_read(). The counter

Re: [PATCH v9 5/5] lib/dlock-list: Scale dlock_lists_empty()

2018-10-16 Thread Davidlohr Bueso
On Thu, 04 Oct 2018, Waiman Long wrote: On 10/04/2018 03:16 AM, Jan Kara wrote: On Wed 12-09-18 15:28:52, Waiman Long wrote: From: Davidlohr Bueso Instead of the current O(N) implementation, at the cost of adding an atomic counter, we can convert the call to an atomic_read(). The counter

Re: [Ksummit-discuss] [PATCH v3 1/3] code-of-conduct: Fix the ambiguity about collecting email addresses

2018-10-16 Thread James Bottomley
On Tue, 2018-10-16 at 19:10 -0700, Frank Rowand wrote: > On 10/16/18 07:58, James Bottomley wrote: > > The current code of conduct has an ambiguity in the it considers > > publishing > > private information such as email addresses unacceptable > > behaviour. Since > > the Linux kernel collects

Re: [Ksummit-discuss] [PATCH v3 1/3] code-of-conduct: Fix the ambiguity about collecting email addresses

2018-10-16 Thread James Bottomley
On Tue, 2018-10-16 at 19:10 -0700, Frank Rowand wrote: > On 10/16/18 07:58, James Bottomley wrote: > > The current code of conduct has an ambiguity in the it considers > > publishing > > private information such as email addresses unacceptable > > behaviour. Since > > the Linux kernel collects

[PATCH] scripts/gdb: fix lx-version for gdb 7.3-

2018-10-16 Thread Du Changbin
For gdb version less than 7.3, lx-version only one character. (gdb) lx-version L(gdb) This can be fixed by casting 'linux_banner' as (char *). (gdb) lx-version Linux version 4.19.0-rc1+ (changbin@acer) (gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3)) #21 SMP Sat Sep 1 21:43:30 CST 2018 gdb 7.4

[PATCH] scripts/gdb: fix lx-version for gdb 7.3-

2018-10-16 Thread Du Changbin
For gdb version less than 7.3, lx-version only one character. (gdb) lx-version L(gdb) This can be fixed by casting 'linux_banner' as (char *). (gdb) lx-version Linux version 4.19.0-rc1+ (changbin@acer) (gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3)) #21 SMP Sat Sep 1 21:43:30 CST 2018 gdb 7.4

Re: [PATCH] mmc: sdhci: use goto rather than return directly

2018-10-16 Thread Chunyan Zhang
On Tue, 16 Oct 2018 at 16:30, Adrian Hunter wrote: > > On 16/10/18 11:19 AM, Chunyan Zhang wrote: > > The driver should clean resources requested in the function before > > unnormal return. > > > > CC: Linus Walleij > > Signed-off-by: Chunyan Zhang > > Fixes: bd9b902798ab ("mmc: sdhci:

Re: [PATCH] mmc: sdhci: use goto rather than return directly

2018-10-16 Thread Chunyan Zhang
On Tue, 16 Oct 2018 at 16:30, Adrian Hunter wrote: > > On 16/10/18 11:19 AM, Chunyan Zhang wrote: > > The driver should clean resources requested in the function before > > unnormal return. > > > > CC: Linus Walleij > > Signed-off-by: Chunyan Zhang > > Fixes: bd9b902798ab ("mmc: sdhci:

Re: [PATCH v3 1/2] sysctl: handle overflow in proc_get_long

2018-10-16 Thread Kees Cook
On Tue, Oct 16, 2018 at 5:24 PM, Christian Brauner wrote: > Right, but if you write a value that exceeds the buffer of 22 chars that is > used > to parse you already get EINVAL back on current kernels. > So it didn't feel like returning EOVERFLOW or ERANGE might make sense. > I saw a change in

Re: [PATCH v3 1/2] sysctl: handle overflow in proc_get_long

2018-10-16 Thread Kees Cook
On Tue, Oct 16, 2018 at 5:24 PM, Christian Brauner wrote: > Right, but if you write a value that exceeds the buffer of 22 chars that is > used > to parse you already get EINVAL back on current kernels. > So it didn't feel like returning EOVERFLOW or ERANGE might make sense. > I saw a change in

Re: [Ksummit-discuss] [PATCH v3 1/3] code-of-conduct: Fix the ambiguity about collecting email addresses

2018-10-16 Thread Frank Rowand
On 10/16/18 07:58, James Bottomley wrote: > The current code of conduct has an ambiguity in the it considers publishing > private information such as email addresses unacceptable behaviour. Since > the Linux kernel collects and publishes email addresses as part of the patch > process, add an

Re: [Ksummit-discuss] [PATCH v3 1/3] code-of-conduct: Fix the ambiguity about collecting email addresses

2018-10-16 Thread Frank Rowand
On 10/16/18 07:58, James Bottomley wrote: > The current code of conduct has an ambiguity in the it considers publishing > private information such as email addresses unacceptable behaviour. Since > the Linux kernel collects and publishes email addresses as part of the patch > process, add an

Re: [PATCH] mm/thp: Correctly differentiate between mapped THP and PMD migration entry

2018-10-16 Thread Andrea Arcangeli
Hello Zi, On Sun, Oct 14, 2018 at 08:53:55PM -0400, Zi Yan wrote: > Hi Andrea, what is the purpose/benefit of making x86’s pmd_present() returns > true > for a THP under splitting? Does it cause problems when ARM64’s pmd_present() > returns false in the same situation? !pmd_present means it's a

Re: [PATCH] mm/thp: Correctly differentiate between mapped THP and PMD migration entry

2018-10-16 Thread Andrea Arcangeli
Hello Zi, On Sun, Oct 14, 2018 at 08:53:55PM -0400, Zi Yan wrote: > Hi Andrea, what is the purpose/benefit of making x86’s pmd_present() returns > true > for a THP under splitting? Does it cause problems when ARM64’s pmd_present() > returns false in the same situation? !pmd_present means it's a

Re: [RFC 00/60] Coscheduling for Linux

2018-10-16 Thread Frederic Weisbecker
On Fri, Sep 07, 2018 at 11:39:47PM +0200, Jan H. Schönherr wrote: > C) How does it work? > > > This patch series introduces hierarchical runqueues, that represent larger > and larger fractions of the system. By default, there is one runqueue per > scheduling domain. These

Re: [RFC 00/60] Coscheduling for Linux

2018-10-16 Thread Frederic Weisbecker
On Fri, Sep 07, 2018 at 11:39:47PM +0200, Jan H. Schönherr wrote: > C) How does it work? > > > This patch series introduces hierarchical runqueues, that represent larger > and larger fractions of the system. By default, there is one runqueue per > scheduling domain. These

[PATCH v6 0/2] spi: Add Macronix controller driver

2018-10-16 Thread masonccyang
From: Mason Yang Hi Mark & Boris, I patched v6 spi-mxic.c based on Boris's comments of v5. Thanks for your review. best regards, Mason Mason Yang (2): spi: Add MXIC controller driver dt-binding: spi: Document Macronix controller bindings

[PATCH v6 1/2] spi: Add MXIC controller driver

2018-10-16 Thread masonccyang
From: Mason Yang Add a driver for Macronix SPI controller IP. Signed-off-by: Mason Yang --- drivers/spi/Kconfig| 6 + drivers/spi/Makefile | 1 + drivers/spi/spi-mxic.c | 619 + 3 files changed, 626 insertions(+) create mode 100644

[PATCH v6 2/2] dt-binding: spi: Document Macronix controller bindings

2018-10-16 Thread masonccyang
From: Mason Yang Document the bindings used by the Macronix controller. Signed-off-by: Mason Yang --- Documentation/devicetree/bindings/spi/spi-mxic.txt | 34 ++ 1 file changed, 34 insertions(+) create mode 100644 Documentation/devicetree/bindings/spi/spi-mxic.txt diff

[PATCH v6 0/2] spi: Add Macronix controller driver

2018-10-16 Thread masonccyang
From: Mason Yang Hi Mark & Boris, I patched v6 spi-mxic.c based on Boris's comments of v5. Thanks for your review. best regards, Mason Mason Yang (2): spi: Add MXIC controller driver dt-binding: spi: Document Macronix controller bindings

  1   2   3   4   5   6   7   8   9   10   >