Re: [PATCH v2] x86/kexec: Exclude GART aperture from vmcore

2017-12-26 Thread Baoquan He
On 12/19/17 at 06:58pm, Jiri Bohac wrote: Sorry for late response. Please see the inline comments. > > On Tue, Dec 19, 2017 at 09:58:04AM +0800, Baoquan He wrote: > > Hmm, as I have said in the first replying mail, the v2 will introduce > > issues: > > > > 1) If 'iommu=off' is specified in 1st

Re: [PATCH v2] zram: better utilization of zram swap space

2017-12-26 Thread Sergey Senozhatsky
On (12/27/17 15:29), Minchan Kim wrote: > On Fri, Dec 22, 2017 at 04:00:06PM +0530, Gopi Sai Teja wrote: > > 75% of the PAGE_SIZE is not a correct threshold to store uncompressed > > Please describe it in detail that why current threshold is bad in that > memory efficiency point of view. > > > pa

Re: [PATCH v2 2/3] dt-bindings: PCI: dra7xx: Add properties to enable x2 lane in dra7

2017-12-26 Thread Kishon Vijay Abraham I
Hi, On Tuesday 26 December 2017 11:40 PM, Tony Lindgren wrote: > * Rob Herring [171226 17:49]: >> On Fri, Dec 22, 2017 at 12:24 PM, Tony Lindgren wrote: >>> * Kishon Vijay Abraham I [171222 06:06]: Hi Rob, On Thursday 21 December 2017 12:27 AM, Rob Herring wrote: > On Tue, De

Re: [PATCH v2] zram: better utilization of zram swap space

2017-12-26 Thread Minchan Kim
Hello, On Fri, Dec 22, 2017 at 04:00:06PM +0530, Gopi Sai Teja wrote: > 75% of the PAGE_SIZE is not a correct threshold to store uncompressed Please describe it in detail that why current threshold is bad in that memory efficiency point of view. > pages in zs_page as this must be changed if the

Re: [PATCH -next v3 1/2] mtd: nand: toshiba: Retrieve ECC requirements from extended ID

2017-12-26 Thread KOBAYASHI Yoshitake
On 2017/12/19 20:56, Boris Brezillon wrote: > On Tue, 19 Dec 2017 20:42:36 +0900 > KOBAYASHI Yoshitake wrote: > >> On 2017/12/07 0:08, Boris Brezillon wrote: >>> On Wed, 6 Dec 2017 23:04:57 +0900 >>> KOBAYASHI Yoshitake wrote: >>> This patch enables support to read the ECC strength and

Re: [RFC 2/2] pci: dwc: pci-exynos: add the codes to support the exynos5433

2017-12-26 Thread Jaehoon Chung
On 12/27/2017 06:11 AM, Rob Herring wrote: > On Thu, Dec 21, 2017 at 09:14:07PM +0900, Jaehoon Chung wrote: >> Exynos5433 has the PCIe for WiFi. >> Added the codes relevant to PCIe for supporting the exynos5433. >> Also changed the binding documentation name to >> 'samsung,exynos-pcie.txt'. >> (It'

Re: [RFC PATCH bpf-next v2 1/4] tracing/kprobe: bpf: Check error injectable event is on function entry

2017-12-26 Thread Masami Hiramatsu
On Tue, 26 Dec 2017 17:57:32 -0800 Alexei Starovoitov wrote: > On Tue, Dec 26, 2017 at 04:46:59PM +0900, Masami Hiramatsu wrote: > > Check whether error injectable event is on function entry or not. > > Currently it checks the event is ftrace-based kprobes or not, > > but that is wrong. It should

[PATCH 3/3] staging: irda: separate multiple assignments

2017-12-26 Thread JI-HUN KIM
Clean up checkpatch warning: CHECK: multiple assignments should be avoided Signed-off-by: JI-HUN KIM --- drivers/staging/irda/drivers/esi-sir.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/irda/drivers/esi-sir.c b/drivers/staging/irda/drivers/esi-sir.c i

[PATCH 2/3] staging: irda: add spaces around '|' operator

2017-12-26 Thread JI-HUN KIM
Clean up checkpatch warning: CHECK: spaces preferred around that '|' (ctx:VxV) Signed-off-by: JI-HUN KIM --- drivers/staging/irda/drivers/esi-sir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/irda/drivers/esi-sir.c b/drivers/staging/irda/drivers/esi-sir.c

[PATCH 1/3] staging: irda: fix type from "unsigned" to "unsigned int"

2017-12-26 Thread JI-HUN KIM
Clean up checkpatch warning: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: JI-HUN KIM --- drivers/staging/irda/drivers/esi-sir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/irda/drivers/esi-sir.c b/drivers/staging/irda/driver

RE: [PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Peng Fan
> -Original Message- > From: Dong Aisheng [mailto:donga...@gmail.com] > Sent: Wednesday, December 27, 2017 10:51 AM > To: A.s. Dong > Cc: Shawn Guo ; Peng Fan ; > linux-kernel@vger.kernel.org; Russell King ; Fabio > Estevam ; Sascha Hauer ; > van.free...@gmail.com; linux-arm-ker...@lists

Re: [PATCH net-next v5 1/6] net: tcp: Add trace events for TCP congestion window tracing

2017-12-26 Thread Masami Hiramatsu
On Tue, 26 Dec 2017 18:51:55 -0500 (EST) David Miller wrote: > From: Masami Hiramatsu > Date: Fri, 22 Dec 2017 11:05:33 +0900 > > > This adds an event to trace TCP stat variables with > > slightly intrusive trace-event. This uses ftrace/perf > > event log buffer to trace those state, no needs t

[PATCH] x86/cpu, x86/pti: Do not enable PTI on AMD processors

2017-12-26 Thread Tom Lendacky
AMD processors are not subject to the types of attacks that the kernel page table isolation feature protects against. The AMD microarchitecture does not allow memory references, including speculative references, that access higher privileged data when running in a lesser privileged mode when that

Re: [PATCH v4] x86/microcode/intel: Blacklist the specific BDW-EP for late loading

2017-12-26 Thread Luck, Tony
When all the patch issues are ironed out consider this Acked-by: Tony Luck I was planning to hunt down the erratum when I got back from vacation in the new year. Sent from my iPhone > On Dec 26, 2017, at 04:42, Jia Zhang wrote: > > Instead of blacklisting all types of Broadwell processor w

[PATCH] rpmsg: virtio_rpmsg_bus: fix rpmsg_probe() for virtio-mmio transport

2017-12-26 Thread Anup Patel
When virtio-rpmsg device is provided via virtio-mmio transport, the dma_alloc_coherent() (called by rpmsg_probe()) fails on ARM/ARM64 systems because "vdev->dev.parent->parent" device is used as parameter to dma_alloc_coherent(). The "vdev->dev.parent->parent" device represents underlying remotepr

Re: [RFC V7 2/2] OPP: Allow "opp-hz" and "opp-microvolt" to contain magic values

2017-12-26 Thread Viresh Kumar
On 26-12-17, 14:23, Rob Herring wrote: > > cpu_opp_table: cpu_opp_table { > > compatible = "operating-points-v2"; > > opp-shared; > > > > opp00 { > > opp-hz = /bits/ 64 <20800>; > > clock-lat

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread Alexandru Chirvasitu
Sounds like it's been pinned down then. Just confirming this: On Tue, Dec 26, 2017 at 06:16:37PM -0800, Linus Torvalds wrote: > On Tue, Dec 26, 2017 at 3:19 PM, Alexandru Chirvasitu > wrote: > > > > I went back to the initial problematic commit e802a51 and modified it as > > you suggest: > > Th

Re: BUG warnings in 4.14.9

2017-12-26 Thread alexander . levin
On Tue, Dec 26, 2017 at 10:54:37PM +0200, Ido Schimmel wrote: >On Tue, Dec 26, 2017 at 07:59:55PM +0100, Willy Tarreau wrote: >> Guys, >> >> Chris reported the bug below and confirmed that reverting commit >> 9704f81 (ipv6: grab rt->rt6i_ref before allocating pcpu rt) seems to >> have fixed the iss

Re: [PATCH v3 00/11] Allwinner H3/H5/A64(DE2) SimpleFB support

2017-12-26 Thread Chen-Yu Tsai
On Wed, Dec 27, 2017 at 12:10 PM, Icenowy Zheng wrote: > 在 2017年12月27日星期三 CST 下午12:09:41,Chen-Yu Tsai 写道: >> On Fri, Dec 22, 2017 at 8:22 PM, Icenowy Zheng wrote: >> > This patchset adds support for the SimpleFB on Allwinner SoCs with >> > "Display Engine 2.0". >> > >> > PATCH 1 to PATCH 3 are DE

Re: [PATCH] ARM: dts: ls1021a: fix incorrect clock references

2017-12-26 Thread Shawn Guo
On Thu, Dec 21, 2017 at 10:44:20PM +0100, Arnd Bergmann wrote: > dtc warns about two 'clocks' properties that have an extraneous '1' > at the end: > > arch/arm/boot/dts/ls1021a-qds.dtb: Warning (clocks_property): > arch/arm/boot/dts/ls1021a-twr.dtb: Warning (clocks_property): Property > 'clocks'

Re: [PATCH v3 00/11] Allwinner H3/H5/A64(DE2) SimpleFB support

2017-12-26 Thread Icenowy Zheng
在 2017年12月27日星期三 CST 下午12:09:41,Chen-Yu Tsai 写道: > On Fri, Dec 22, 2017 at 8:22 PM, Icenowy Zheng wrote: > > This patchset adds support for the SimpleFB on Allwinner SoCs with > > "Display Engine 2.0". > > > > PATCH 1 to PATCH 3 are DE2 CCU fixes for H3/H5 SoCs. > > > > PATCH 4 adds the pipeline

Re: [PATCH v3 00/11] Allwinner H3/H5/A64(DE2) SimpleFB support

2017-12-26 Thread Chen-Yu Tsai
On Fri, Dec 22, 2017 at 8:22 PM, Icenowy Zheng wrote: > This patchset adds support for the SimpleFB on Allwinner SoCs with > "Display Engine 2.0". > > PATCH 1 to PATCH 3 are DE2 CCU fixes for H3/H5 SoCs. > > PATCH 4 adds the pipeline strings for DE2 SimpleFB. > > PATCH 5 to 7 adds necessary device

Re: [PATCH] clk: mediatek: remove superfluous line including linux/clk-provider.h

2017-12-26 Thread Sean Wang
On Tue, 2017-12-26 at 17:15 -0800, Stephen Boyd wrote: > On 12/25, sean.w...@mediatek.com wrote: > > From: Sean Wang > > > > clk-mtk.h has already included linux/clk-provider.h inside, so remove > > the superfluous line including linux/clk-provider.h from these files which > > also included clk-m

Re: [PATCH v5 03/78] xarray: Add the xa_lock to the radix_tree_root

2017-12-26 Thread Matthew Wilcox
On Tue, Dec 26, 2017 at 07:43:40PM -0800, Matthew Wilcox wrote: > Also add the xa_lock() and xa_unlock() family of wrappers to make it > easier to use the lock. If we could rely on -fplan9-extensions in > the compiler, we could avoid all of this syntactic sugar, but that > wasn't a

[PATCH 1/2] ARM: dts: imx6: RDU2: disable internal watchdog

2017-12-26 Thread Andrey Smirnov
The system has an external watchdog in the environment processor so the internal watchdog is of no use. Cc: Sascha Hauer Cc: Fabio Estevam Cc: Rob Herring Cc: Mark Rutland Cc: linux-arm-ker...@lists.infradead.org Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Cc: cphe...@gmail

[PATCH 2/2] ARM: dts: imx6: RDU2: correct RTC compatible

2017-12-26 Thread Andrey Smirnov
The RTC is manufactured by Maxim. This is a cosmetic fix, as Linux doesn't match the vendor string for i2c devices. Cc: Sascha Hauer Cc: Fabio Estevam Cc: Rob Herring Cc: Mark Rutland Cc: linux-arm-ker...@lists.infradead.org Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Cc: c

Re: [PATCH v5 03/78] xarray: Add the xa_lock to the radix_tree_root

2017-12-26 Thread Matthew Wilcox
On Tue, Dec 26, 2017 at 07:54:40PM +0300, Kirill A. Shutemov wrote: > On Fri, Dec 15, 2017 at 02:03:35PM -0800, Matthew Wilcox wrote: > > From: Matthew Wilcox > > > > This results in no change in structure size on 64-bit x86 as it fits in > > the padding between the gfp_t and the void *. > > The

Re: [PATCH] clk: mediatek: remove unnecessary include header from reset.c

2017-12-26 Thread Sean Wang
On Tue, 2017-12-26 at 17:10 -0800, Stephen Boyd wrote: > On 12/25, sean.w...@mediatek.com wrote: > > From: Sean Wang > > > > In fact, the clk-mtk.h header is indeed not needed for reset.c and thus > > it's safe and good change to remove it from the file. > > > > Signed-off-by: Sean Wang > > Rev

Re: [PATCH v5 06/78] xarray: Change definition of sibling entries

2017-12-26 Thread Matthew Wilcox
On Tue, Dec 26, 2017 at 08:21:53PM +0300, Kirill A. Shutemov wrote: > > +/** > > + * xa_is_internal() - Is the entry an internal entry? > > + * @entry: Entry retrieved from the XArray > > + * > > + * Return: %true if the entry is an internal entry. > > + */ > > What does it mean "internal entry"?

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread hpa
On December 26, 2017 6:54:55 PM PST, Linus Torvalds wrote: >On Tue, Dec 26, 2017 at 6:25 PM, wrote: >> >> This is why I personally prefer to see these kinds of terminal stubs >written in assembly explicitly: the C compiler simply doesn't have all >the information needed to do the right thing. >

Re: [PATCH] mmc: sdhci: disable irq in sdhci host suspend ranther than free this irq

2017-12-26 Thread Peng Fan
Hi All, Sorry for bring back this old topic again. On Thu, Jan 28, 2016 at 05:27:46PM +0100, Thomas Gleixner wrote: >On Thu, 28 Jan 2016, Thomas Gleixner wrote: >> On Thu, 28 Jan 2016, Ulf Hansson wrote: >> > Therefore, the only way we currently can make sure to don't get the >> > IRQ is to free

[PATCH 09/11 v3] ARM: orion5x: constify gpio_led

2017-12-26 Thread Arvind Yadav
gpio_led are not supposed to change at runtime. struct gpio_led_platform_data working with const gpio_led provided by . So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- changes in v2: The GPIO LED driver can be built as a module, it can be loaded

Re: [GIT PULL] hwmon fixes for v4.15-rc6

2017-12-26 Thread Guenter Roeck
On 12/26/2017 06:25 PM, Linus Torvalds wrote: On Tue, Dec 26, 2017 at 5:41 PM, Guenter Roeck wrote: Please pull hwmon fixes for Linux v4.15-rc6 from signed tag: git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-for-linus-v4.15-rc6 Hmm. That was a tag, but no

Re: [lkp-robot] [bpf] 82abbf8d2f: kernel_selftests.bpf.test_align.fail

2017-12-26 Thread Alexei Starovoitov
On Wed, Dec 27, 2017 at 11:00:30AM +0800, kernel test robot wrote: > > FYI, we noticed the following commit (built with gcc-7): > > commit: 82abbf8d2fc46d79611ab58daa7c608df14bb3ee ("bpf: do not allow root to > mangle valid pointers") > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux

Re: [PATCH v5 05/78] xarray: Replace exceptional entries

2017-12-26 Thread Matthew Wilcox
On Tue, Dec 26, 2017 at 08:15:42PM +0300, Kirill A. Shutemov wrote: > > 28 files changed, 249 insertions(+), 240 deletions(-) > > Everything looks fine to me after quick scan, but hat's a lot of changes for > one patch... Yeah. It's pretty mechanical though. > > - if (radix_t

[RFC PATCH v2] workqueue: Introduce a way to set percpu worker_pool's scheduler

2017-12-26 Thread Wen Yang
From: Liu Wei When pinning RT threads to specific cores using CPU affinity, the kworkers on the same CPU would starve, which may lead to some kind of priority inversion. In that case, the RT threads would also suffer high performance impact. The priority inversion looks like, CPU 0: libvirtd ac

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread Linus Torvalds
On Tue, Dec 26, 2017 at 6:25 PM, wrote: > > This is why I personally prefer to see these kinds of terminal stubs written > in assembly explicitly: the C compiler simply doesn't have all the > information needed to do the right thing. > > I'm personally very sceptical to nuking the GDT unless we

RE: [PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Peng Fan
> -Original Message- > From: Dong Aisheng [mailto:donga...@gmail.com] > Sent: Wednesday, December 27, 2017 10:51 AM > To: A.s. Dong > Cc: Shawn Guo ; Peng Fan ; > linux-kernel@vger.kernel.org; Russell King ; Fabio > Estevam ; Sascha Hauer ; > van.free...@gmail.com; linux-arm-ker...@lists

Re: [PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Dong Aisheng
On Wed, Dec 27, 2017 at 02:33:57AM +, A.s. Dong wrote: > > -Original Message- > > From: Shawn Guo [mailto:shawn...@kernel.org] > > Sent: Wednesday, December 27, 2017 10:32 AM > > To: Peng Fan > > Cc: A.s. Dong ; linux-kernel@vger.kernel.org; Russell > > King ; Fabio Estevam ; > > Sasch

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread hpa
On December 26, 2017 6:16:37 PM PST, Linus Torvalds wrote: >On Tue, Dec 26, 2017 at 3:19 PM, Alexandru Chirvasitu > wrote: >> >> I went back to the initial problematic commit e802a51 and modified it >as you suggest: > >Thank you. > >> This did not work out for me, but now it fails differently. Bo

RE: [PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread A.s. Dong
> -Original Message- > From: Shawn Guo [mailto:shawn...@kernel.org] > Sent: Wednesday, December 27, 2017 10:32 AM > To: Peng Fan > Cc: A.s. Dong ; linux-kernel@vger.kernel.org; Russell > King ; Fabio Estevam ; > Sascha Hauer ; van.free...@gmail.com; linux-arm- > ker...@lists.infradead.org

Re: [PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Shawn Guo
On Wed, Dec 27, 2017 at 09:57:47AM +0800, Peng Fan wrote: > Use outer_disable/resume for suspend/resume. > With the two APIs used, code could be simplified and easy to extend > to introduce l2c_write_sec for i.MX platforms when moving Linux Kernel > runs in non-secure world. > > Signed-off-by: Pen

Re: [PATCH v2 bpf-next 2/2] tools/bpftool: fix bpftool build with bintutils >= 2.8

2017-12-26 Thread Alexei Starovoitov
On Fri, Dec 22, 2017 at 06:50:01PM +, Quentin Monnet wrote: > Hi Roman, > > 2017-12-22 16:11 UTC+ ~ Roman Gushchin > > Bpftool build is broken with binutils version 2.28 and later. > > Could you check the binutils version? I believe it changed in 2.29 > instead of 2.28. Could you update

Re: [PATCH] vfio: mdev: make a couple of functions and structure vfio_mdev_driver static

2017-12-26 Thread Kirti Wankhede
On 12/22/2017 4:42 AM, Xiongwei Song wrote: > The functions vfio_mdev_probe, vfio_mdev_remove and the structure > vfio_mdev_driver are only used in this file, so make them static. > > Clean up sparse warnings: > drivers/vfio/mdev/vfio_mdev.c:114:5: warning: no previous prototype > for 'vfio_mdev

rename arcs_cmdline to mips_cmdline

2017-12-26 Thread Jiaxun Yang
This patch should based on "Add YeeLoong support v6"

[PATCH] MIPS: rename arcs_cmdline to mips_cmdline

2017-12-26 Thread Jiaxun Yang
arcs_cmdline refers to boot cmdline for all machs, not only arc systems. This patch renamed all arcs_cmdline to mips_cmdline. Signed-off-by: Jiaxun Yang --- arch/mips/alchemy/common/prom.c | 6 +++--- arch/mips/ar7/prom.c | 8 arch/mips/cavium-octeon/set

Re: [PATCH]cpuidle: preventive check in cpuidle_select against crash

2017-12-26 Thread Rafael J. Wysocki
On Wed, Dec 27, 2017 at 2:57 AM, gaurav jindal wrote: > On Wed, Dec 27, 2017 at 01:42:58AM +0100, Rafael J. Wysocki wrote: >> On Tue, Dec 26, 2017 at 8:26 AM, gaurav jindal >> wrote: >> > When selecting the idle state using cpuidle_select, there is no >> > check on cpuidle_curr_governor. In cpuid

Re: [PATCH bpf-next 2/3] libbpf: add error reporting in XDP

2017-12-26 Thread Alexei Starovoitov
On Mon, Dec 25, 2017 at 11:13:24PM +0100, Eric Leblond wrote: > Parse netlink ext attribute to get the error message returned by > the card. > > Signed-off-by: Eric Leblond ... > diff --git a/tools/lib/bpf/nlattr.c b/tools/lib/bpf/nlattr.c > new file mode 100644 > index ..962de14f74e3

Re: [GIT PULL] hwmon fixes for v4.15-rc6

2017-12-26 Thread Linus Torvalds
On Tue, Dec 26, 2017 at 5:41 PM, Guenter Roeck wrote: > > Please pull hwmon fixes for Linux v4.15-rc6 from signed tag: > > git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git > hwmon-for-linus-v4.15-rc6 Hmm. That was a tag, but not a signed one. It's just a naked tag that

Re: [PATCH] clk: fix reentrancy of clk_enable() on UP systems

2017-12-26 Thread Stephen Boyd
On 12/26, David Lechner wrote: > Reentrant calls to clk_enable() are not working on UP systems. This is > caused by the fact spin_trylock_irqsave() always returns true when > CONFIG_SMP=n (and CONFIG_DEBUG_SPINLOCK=n) which causes the reference > counting to not work correctly when clk_enable_lock(

RE: [PATCH] arm: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Peng Fan
Hi Shawn, > > > > On Sun, Dec 10, 2017 at 08:07:18PM +0800, Peng Fan wrote: > > > Use outer_disable/resume for suspend/resume. > > > With the two APIs used, code could be simplified and easy to extend > > > to introduce l2c_write_sec for i.MX platforms when moving Linux > > > Kernel runs in non-se

Re: [RFC PATCH 3/3] mm: unclutter THP migration

2017-12-26 Thread Zi Yan
On 8 Dec 2017, at 11:15, Michal Hocko wrote: > From: Michal Hocko > > THP migration is hacked into the generic migration with rather > surprising semantic. The migration allocation callback is supposed to > check whether the THP can be migrated at once and if that is not the > case then it alloca

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread Linus Torvalds
On Tue, Dec 26, 2017 at 3:19 PM, Alexandru Chirvasitu wrote: > > I went back to the initial problematic commit e802a51 and modified it as you > suggest: Thank you. > This did not work out for me, but now it fails differently. Both > (kexec -l + kexec -e) and (kexec -p + echo c > /proc/sysrq-tri

Re: [PATCH] thermal: power_allocator: fix one race condition issue for thermal_instances list

2017-12-26 Thread Zhang Rui
On Tue, 2017-12-26 at 19:22 +0800, Yi Zeng wrote: > When invoking allow_maximum_power and traverse tz->thermal_instances, > we should grab thermal_zone_device->lock to avoid race condition. For > example, during the system reboot, if the mali GPU device implements > device shutdown callback and unr

[PATCH V2] ARM: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Peng Fan
Use outer_disable/resume for suspend/resume. With the two APIs used, code could be simplified and easy to extend to introduce l2c_write_sec for i.MX platforms when moving Linux Kernel runs in non-secure world. Signed-off-by: Peng Fan Cc: Shawn Guo Cc: Sascha Hauer Cc: Fabio Estevam Cc: Russell

Re: [RFC PATCH bpf-next v2 4/4] error-injection: Support fault injection framework

2017-12-26 Thread Alexei Starovoitov
On Tue, Dec 26, 2017 at 04:48:25PM +0900, Masami Hiramatsu wrote: > Support in-kernel fault-injection framework via debugfs. > This allows you to inject a conditional error to specified > function using debugfs interfaces. > > Signed-off-by: Masami Hiramatsu > --- > Documentation/fault-injection

Re: [RFC PATCH 2/3] mm, migrate: remove reason argument from new_page_t

2017-12-26 Thread Zi Yan
On 8 Dec 2017, at 11:15, Michal Hocko wrote: > From: Michal Hocko > > No allocation callback is using this argument anymore. new_page_node > used to use this parameter to convey node_id resp. migration error > up to move_pages code (do_move_page_to_node_array). The error status > never made it in

Re: [PATCH 20/45] drivers: thermal: remove duplicate includes

2017-12-26 Thread Zhang Rui
On Wed, 2017-12-06 at 22:10 +0530, Pravin Shedge wrote: > These duplicate includes have been found with > scripts/checkincludes.pl but > they have been removed manually to avoid removing false positives. > > Signed-off-by: Pravin Shedge applied. thanks, rui > --- >  drivers/thermal/of-thermal.c

Re: [PATCH] arm: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Shawn Guo
On Tue, Dec 26, 2017 at 06:36:29PM +0800, Dong Aisheng wrote: > > > > diff --git a/arch/arm/mach-imx/cpuidle-imx6sx.c b/arch/arm/mach- > > > > imx/cpuidle-imx6sx.c > > > > index c5a5c3a..edce5bd 100644 > > > > --- a/arch/arm/mach-imx/cpuidle-imx6sx.c > > > > +++ b/arch/arm/mach-imx/cpuidle-imx6sx.c

Re: [RFC PATCH bpf-next v2 3/4] error-injection: Separate error-injection from kprobe

2017-12-26 Thread Alexei Starovoitov
On Tue, Dec 26, 2017 at 04:47:55PM +0900, Masami Hiramatsu wrote: > Since error-injection framework is not limited to be used > by kprobes, nor bpf. Other kernel subsystems can use it > freely for checking safeness of error-injection, e.g. > livepatch, ftrace etc. > So this separate error-injection

Re: [PATCH] ARM: make ARCH_S3C24XX select USE_OF and clean-up boot/dts/Makefile

2017-12-26 Thread Masahiro Yamada
2017-12-26 2:39 GMT+09:00 Krzysztof Kozlowski : > On Sun, Dec 24, 2017 at 6:19 PM, Masahiro Yamada > wrote: >> 2017-12-22 21:41 GMT+09:00 Krzysztof Kozlowski : >>> On Mon, Nov 27, 2017 at 3:19 AM, Masahiro Yamada >>> wrote: ARCH_S3C24XX is a very exceptional platform that some DT files in >>

Re: [RFC PATCH bpf-next v2 2/4] tracing/kprobe: bpf: Compare instruction pointer with original one

2017-12-26 Thread Alexei Starovoitov
On Tue, Dec 26, 2017 at 04:47:26PM +0900, Masami Hiramatsu wrote: > Compare instruction pointer with original one on the > stack instead using per-cpu bpf_kprobe_override flag. > > This patch also consolidates reset_current_kprobe() and > preempt_enable_no_resched() blocks. Those can be done > in

Re: [RFC PATCH bpf-next v2 1/4] tracing/kprobe: bpf: Check error injectable event is on function entry

2017-12-26 Thread Alexei Starovoitov
On Tue, Dec 26, 2017 at 04:46:59PM +0900, Masami Hiramatsu wrote: > Check whether error injectable event is on function entry or not. > Currently it checks the event is ftrace-based kprobes or not, > but that is wrong. It should check if the event is on the entry > of target function. Since error i

Re: [PATCH]cpuidle: preventive check in cpuidle_select against crash

2017-12-26 Thread gaurav jindal
On Wed, Dec 27, 2017 at 01:42:58AM +0100, Rafael J. Wysocki wrote: > On Tue, Dec 26, 2017 at 8:26 AM, gaurav jindal > wrote: > > When selecting the idle state using cpuidle_select, there is no > > check on cpuidle_curr_governor. In cpuidle_switch_governor, > > cpuidle_currr_governor can be set to

Re: [PATCH net-next v8 1/2] dt-bindings: net: add DT bindings for Socionext UniPhier AVE

2017-12-26 Thread Masahiro Yamada
2017-12-25 10:10 GMT+09:00 Kunihiko Hayashi : > DT bindings for the AVE ethernet controller found on Socionext's > UniPhier platforms. > > Signed-off-by: Kunihiko Hayashi > Signed-off-by: Jassi Brar > Acked-by: Rob Herring > --- > .../bindings/net/socionext,uniphier-ave4.txt | 47 > +

[GIT PULL] hwmon fixes for v4.15-rc6

2017-12-26 Thread Guenter Roeck
Hi Linus, Please pull hwmon fixes for Linux v4.15-rc6 from signed tag: git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-for-linus-v4.15-rc6 Thanks, Guenter -- The following changes since commit 464e1d5f23cca236b930ef068c328a64cab78fb1: Linux 4.15-rc5 (201

Re: [RFC PATCH V1 1/2] clk: use atomic runtime pm api in clk_core_is_enabled

2017-12-26 Thread Stephen Boyd
On 12/22, Dong Aisheng wrote: > Current clk_pm_runtime_put is using pm_runtime_put_sync which > is not safe to be called in clk_core_is_enabled as it should > be able to run in atomic context. > > Thus use pm_runtime_put instead which is atomic safe. > > Cc: Stephen Boyd > Cc: Michael Turquette

Re: [RFC PATCH v12 1/5] dt-bindings: PCI: Add definition of PCIe WAKE# irq and PCI irq

2017-12-26 Thread JeffyChen
Hi Rafael and Rob, Thanks for your reply. On 12/27/2017 08:43 AM, Rafael J. Wysocki wrote: On Wednesday, December 27, 2017 12:35:52 AM CET Rob Herring wrote: >On Tue, Dec 26, 2017 at 10:36:42AM +0800, Jeffy Chen wrote: > >We are going to handle PCIe WAKE# pin for PCI devices in the pci core,

Re: [PATCH] arm: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Shawn Guo
On Tue, Dec 26, 2017 at 09:49:01AM +, Peng Fan wrote: > Hi Shawn > > > -Original Message- > > From: Shawn Guo [mailto:shawn...@kernel.org] > > Sent: Tuesday, December 26, 2017 5:13 PM > > To: Peng Fan > > Cc: A.s. Dong ; linux-kernel@vger.kernel.org; Russell > > King ; Fabio Estevam ;

Re: [RFC PATCH v12 4/5] PCI / PM: Add support for the PCIe WAKE# signal for OF

2017-12-26 Thread JeffyChen
Hi Rob, Thanks for your reply. On 12/27/2017 07:56 AM, Rob Herring wrote: > > drivers/of/of_pci_irq.c | 49 +++ Please move this to drivers/pci/of.c (or perhaps create pci/of_irq.c). > drivers/pci/Makefile | 1 + > drivers/pci/pci-driver.c | 10 +++ > d

Re: [PATCH v6 4/5] clk: aspeed: Register gated clocks

2017-12-26 Thread Stephen Boyd
On 12/22, Benjamin Herrenschmidt wrote: > On Fri, 2017-12-22 at 13:36 +1100, Benjamin Herrenschmidt wrote: > > > > > No you can't sleep here. It needs to delay because this is inside > > > spinlock_irqsave. > > > > Additionally you really don't want to delay for 10ms with interrupts > > off :-( >

Re: [RFC PATCH V1 2/2] clk: add lock for clk_core_is_enabled

2017-12-26 Thread Stephen Boyd
On 12/22, Dong Aisheng wrote: > According to design doc, .is_enabled should be protected by enable lock. > Then users don't have to protect it against enable/disable operation > in clock drivers. > > See: Documentation/clk.txt > "The enable lock is a spinlock and is held across calls to the .enabl

Re: [PATCH] clk: mediatek: adjust dependency of reset.c to avoid unexpectedly being built

2017-12-26 Thread Stephen Boyd
On 12/26, sean.w...@mediatek.com wrote: > From: Sean Wang > > commit 74cb0d6dde8 ("clk: mediatek: fixup test-building of MediaTek clock > drivers") can let the build system looking into the directory where the > clock drivers resides and then allow test-building the drivers. > > But the change a

Re: [PATCH v2] clk: mediatek: Fix all warnings for missing struct clk_onecell_data

2017-12-26 Thread Stephen Boyd
On 12/25, sean.w...@mediatek.com wrote: > From: Sean Wang > > Even though the header file linux/clk-provider.h is already being properly > included in clk-mtk.h, the definition of struct clk_onecell_data still > must depend on CONFIG_COMMON_CLK defined and thus it's possible that > below build wa

Re: [PATCH] clk: mediatek: remove superfluous line including linux/clk-provider.h

2017-12-26 Thread Stephen Boyd
On 12/25, sean.w...@mediatek.com wrote: > From: Sean Wang > > clk-mtk.h has already included linux/clk-provider.h inside, so remove > the superfluous line including linux/clk-provider.h from these files which > also included clk-mtk.h. > > Signed-off-by: Sean Wang > Cc: Jean Delvare > --- Ple

Re: [PATCH] clk: mediatek: remove unnecessary include header from reset.c

2017-12-26 Thread Stephen Boyd
On 12/25, sean.w...@mediatek.com wrote: > From: Sean Wang > > In fact, the clk-mtk.h header is indeed not needed for reset.c and thus > it's safe and good change to remove it from the file. > > Signed-off-by: Sean Wang > Reviewed-by: Jean Delvare > Cc: Jean Delvare > --- > drivers/clk/mediat

Re: [PATCH 1/2] clk: rename clk_core_get_boundaries() to clk_hw_get_boundaries() and expose

2017-12-26 Thread Stephen Boyd
On 12/25, Alexander Kochetkov wrote: > > > 21 дек. 2017 г., в 23:07, Stephen Boyd написал(а): > > > > Can you convert to the determine_rate op instead of round_rate? > > That function should tell you the min/max limits so that you > > don't need to query that information from the core. > > I co

[PATCH v5 2/2] PCI: mediatek: Set up class type and vendor ID for MT7622

2017-12-26 Thread honghui.zhang
From: Honghui Zhang The hardware default value of IDs and class type is not correct, fix that by setup the correct values before start up. Signed-off-by: Honghui Zhang --- drivers/pci/host/pcie-mediatek.c | 12 include/linux/pci_ids.h | 2 ++ 2 files changed, 14 insertio

[PATCH v5 1/2] PCI: mediatek: Clear IRQ status after IRQ dispatched to avoid reentry

2017-12-26 Thread honghui.zhang
From: Honghui Zhang There maybe a same IRQ reentry scenario after IRQ received in current IRQ handle flow: EP device PCIe host driverEP driver 1. issue an IRQ 2. received IRQ 3. clear IRQ status

[PATCH v5 0/2] PCI: mediatek: Fixups for the IRQ handle routine and MT7622's class code

2017-12-26 Thread honghui.zhang
From: Honghui Zhang Two fixups for mediatek's host bridge: The first patch fixup the IRQ handle routine to avoid IRQ reentry which may exist for both MT2712 and MT7622. The second patch fixup class type for MT7622. Change since v4: - Only setup vendor ID for MT7622, igorning the device ID since

Re: [RFC PATCH v11 4/5] PCI / PM: Add support for the PCIe WAKE# signal for OF

2017-12-26 Thread Rafael J. Wysocki
On Tuesday, December 26, 2017 2:06:47 AM CET JeffyChen wrote: > Hi Rafael, > > Thanks for your reply :) > > On 12/26/2017 08:11 AM, Rafael J. Wysocki wrote: > >> >+ > >> >+ dn = pci_device_to_OF_node(ppdev); > >> >+ if (!dn) > >> >+ return 0; > >> >+ > >> >+ irq = of_irq_get_byname(dn, "w

RE: [PATCH] arm: imx: suspend/resume: use outer_disable/resume

2017-12-26 Thread Peng Fan
> -Original Message- > From: Dong Aisheng [mailto:donga...@gmail.com] > Sent: Tuesday, December 26, 2017 6:21 PM > To: Peng Fan > Cc: Shawn Guo ; A.s. Dong ; > linux-kernel@vger.kernel.org; Russell King ; Fabio > Estevam ; Sascha Hauer ; > van.free...@gmail.com; linux-arm-ker...@lists.in

Re: [RFC PATCH v12 0/5] PCI: rockchip: Move PCIe WAKE# handling into pci core

2017-12-26 Thread Rafael J. Wysocki
On Tuesday, December 26, 2017 3:36:41 AM CET Jeffy Chen wrote: > > Currently we are handling wake irq in mrvl wifi driver. Move it into > pci core. > > Tested on my chromebook bob(with cros 4.4 kernel and mrvl wifi). > > > Changes in v13: > Fix compiler error reported by kbuild test robot > >

Re: [RFC PATCH v12 1/5] dt-bindings: PCI: Add definition of PCIe WAKE# irq and PCI irq

2017-12-26 Thread Rafael J. Wysocki
On Wednesday, December 27, 2017 12:35:52 AM CET Rob Herring wrote: > On Tue, Dec 26, 2017 at 10:36:42AM +0800, Jeffy Chen wrote: > > We are going to handle PCIe WAKE# pin for PCI devices in the pci core, > > so add definitions of the optional PCIe WAKE# pin for PCI devices. > > > > Also add an def

Re: [PATCH]cpuidle: preventive check in cpuidle_select against crash

2017-12-26 Thread Rafael J. Wysocki
On Tue, Dec 26, 2017 at 8:26 AM, gaurav jindal wrote: > When selecting the idle state using cpuidle_select, there is no > check on cpuidle_curr_governor. In cpuidle_switch_governor, > cpuidle_currr_governor can be set to NULL to specify "disabled". How exactly?

Re: [PATCH] objtool: Fix clang enum conversion warning

2017-12-26 Thread Nick Desaulniers
I sent a similar one recently: https://patchwork.kernel.org/patch/10131815/ (maybe Josh is just forwarding me an earlier fix?) Reviewed-by: Nick Desaulniers On Tue, Dec 26, 2017 at 4:27 PM, Josh Poimboeuf wrote: > From: Lukas Bulwahn > > Fix the following clang enum conversion warning: > > a

Re: [PATCH v2] gpio: winbond: add driver

2017-12-26 Thread William Breathitt Gray
On Mon, Dec 25, 2017 at 03:48:16PM +0100, Maciej S. Szmigiero wrote: >On 24.12.2017 23:42, William Breathitt Gray wrote: >(..) >> By the way, don't hesitate to ask for more information on the ISA >> subsystem -- a lot of maintainers are unaware that it even exists since >> so few devices nowadays u

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread hpa
On December 26, 2017 3:19:00 PM PST, Alexandru Chirvasitu wrote: >On Tue, Dec 26, 2017 at 10:51:12AM -0800, Linus Torvalds wrote: >> [ Sorry, I was off-line on Christmas Eve due to festivities, and then >> yesterday because I've apparently caught a cold. >> >> Still not back to normal, but at

Re: [RFC PATCH v12 4/5] PCI / PM: Add support for the PCIe WAKE# signal for OF

2017-12-26 Thread Rob Herring
On Tue, Dec 26, 2017 at 10:36:45AM +0800, Jeffy Chen wrote: > Add of_pci_setup_wake_irq() and of_pci_teardown_wake_irq() to handle > the PCIe WAKE# interrupt. > > Also use the dedicated wakeirq infrastructure to simplify it. > > And add pci-of.c to enable/disable the wakeup irq in noirq stage to

Re: 4.14.9 boot error (regression), 4.14.8 is fine

2017-12-26 Thread Ozgur
27.12.2017, 00:52, "Randy Dunlap" : > On 12/26/2017 12:52 PM, Toralf Förster wrote: >>  at a headless server: Intel(R) Core(TM) i7-3930K CPU @ 3.20GHz, stable >> hardened Gentoo Linux >>  The hang does occur before any log message is written. >>  A ping command shows, that the server does reboot

Re: [PATCH net-next v5 1/6] net: tcp: Add trace events for TCP congestion window tracing

2017-12-26 Thread David Miller
From: Masami Hiramatsu Date: Fri, 22 Dec 2017 11:05:33 +0900 > This adds an event to trace TCP stat variables with > slightly intrusive trace-event. This uses ftrace/perf > event log buffer to trace those state, no needs to > prepare own ring-buffer, nor custom user apps. > > User can use ftrace

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread hpa
On December 26, 2017 3:19:00 PM PST, Alexandru Chirvasitu wrote: >On Tue, Dec 26, 2017 at 10:51:12AM -0800, Linus Torvalds wrote: >> [ Sorry, I was off-line on Christmas Eve due to festivities, and then >> yesterday because I've apparently caught a cold. >> >> Still not back to normal, but at

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread hpa
On December 26, 2017 3:19:00 PM PST, Alexandru Chirvasitu wrote: >On Tue, Dec 26, 2017 at 10:51:12AM -0800, Linus Torvalds wrote: >> [ Sorry, I was off-line on Christmas Eve due to festivities, and then >> yesterday because I've apparently caught a cold. >> >> Still not back to normal, but at

Re: [RFC PATCH v12 1/5] dt-bindings: PCI: Add definition of PCIe WAKE# irq and PCI irq

2017-12-26 Thread Rob Herring
On Tue, Dec 26, 2017 at 10:36:42AM +0800, Jeffy Chen wrote: > We are going to handle PCIe WAKE# pin for PCI devices in the pci core, > so add definitions of the optional PCIe WAKE# pin for PCI devices. > > Also add an definition of the optional PCI interrupt pin for PCI > devices to distinguish it

Re: [v2] ASoC: max98373: Added Amplifier Driver

2017-12-26 Thread Rob Herring
On Mon, Dec 25, 2017 at 07:10:10AM -0800, Ryan Lee wrote: Needs a commit message. > Signed-off-by: Ryan Lee > --- > > Changes since v1: > * Removed 'codec' from 'max98373_priv' structure > : Now 'max98373_set_clock' function use 'dai->codec.dev' > instead of using 'max98373

Re: [PATCH 2/3] dt-bindings: mtd: atmel-quadspi: add an optional property 'dmacap,memcpy'

2017-12-26 Thread Rob Herring
On Sun, Dec 24, 2017 at 05:36:05AM +0100, Cyrille Pitchen wrote: > The optional 'dmacap,memcpy' DT property tells the Atmel QSPI controller > driver to reserve some DMA channel then to use it to perform DMA > memcpy() during data transfers. This feature relies on the generic > bounce buffer helper

Re: PROBLEM: consolidated IDT invalidation causes kexec to reboot

2017-12-26 Thread Alexandru Chirvasitu
On Tue, Dec 26, 2017 at 10:51:12AM -0800, Linus Torvalds wrote: > [ Sorry, I was off-line on Christmas Eve due to festivities, and then > yesterday because I've apparently caught a cold. > > Still not back to normal, but at least I can sit in front of the > computer again ] > > On Mon, Dec 25,

Re: BUG warnings in 4.14.9

2017-12-26 Thread alexander . levin
On Tue, Dec 26, 2017 at 07:59:55PM +0100, Willy Tarreau wrote: >Guys, > >Chris reported the bug below and confirmed that reverting commit >9704f81 (ipv6: grab rt->rt6i_ref before allocating pcpu rt) seems to >have fixed the issue for him. This patch is a94b9367 in mainline. > >I personally have no

Re: [PATCH 09/11 v2] ARM: orion5x: constify gpio_led

2017-12-26 Thread kbuild test robot
Hi Arvind, Thank you for the patch! Yet something to improve: [auto build test ERROR on arm-soc/for-next] [also build test ERROR on v4.15-rc5 next-20171222] [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/lin

Re: [PATCH] mmc: Don't reference Linux-specific OF_GPIO_ACTIVE_LOW flag in DT binding

2017-12-26 Thread Rob Herring
On Sat, Dec 23, 2017 at 02:34:12AM +0200, Tuomas Tynkkynen wrote: > OF_GPIO_ACTIVE_LOW is a Linux implementation detail. The binding > document should be referring to GPIO_ACTIVE_LOW found in > include/dt-bindings/gpio/gpio.h. > > Signed-off-by: Tuomas Tynkkynen > --- > Documentation/devicetree/

  1   2   3   >