Re: [PATCH v2 18/18] arm64: select ARCH_SUPPORTS_LTO_CLANG

2017-11-20 Thread Paul E. McKenney
On Mon, Nov 20, 2017 at 08:28:06PM +0100, Peter Zijlstra wrote: > On Mon, Nov 20, 2017 at 06:05:55PM +, Will Deacon wrote: > > This is a thorny issue, but RCU (specifically rcu_dereference but probably > > also some READ_ONCEs) relies on being able to utilise syntactic dependency > > chains to

Re: 4.14: WARNING: CPU: 4 PID: 2895 at block/blk-mq.c:1144 with virtio-blk

2017-11-20 Thread Jens Axboe
On 11/20/2017 01:49 PM, Christian Borntraeger wrote: > > > On 11/20/2017 08:42 PM, Jens Axboe wrote: >> On 11/20/2017 12:29 PM, Christian Borntraeger wrote: >>> >>> >>> On 11/20/2017 08:20 PM, Bart Van Assche wrote: On Fri, 2017-11-17 at 15:42 +0100, Christian Borntraeger wrote: > This i

Re: [PATCH v2 18/18] arm64: select ARCH_SUPPORTS_LTO_CLANG

2017-11-20 Thread Paul E. McKenney
On Mon, Nov 20, 2017 at 08:32:56PM +0100, Peter Zijlstra wrote: > On Mon, Nov 20, 2017 at 06:05:55PM +, Will Deacon wrote: > > Although the current direction of the C++ committee is to prefer > > that dependencies are explicitly "marked", this is not deemed to be > > acceptable for the kernel (

[PATCH] auxdisplay: img-ascii-lcd: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/auxdisplay/img-ascii-lcd.o see include/linux/module.h for more information This adds the license as "GPL", which matches the header of the file. MODULE_DESCRIPTI

[PATCH] gpio: ath79: add missing MODULE_DESCRIPTION/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTION is

Re: [PATCH 17/30] x86, kaiser: map debug IDT tables

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: > On Fri, Nov 10, 2017 at 11:31 AM, Dave Hansen > wrote: > > > > From: Dave Hansen > > > > The IDT is another structure which the CPU references via a > > virtual address. It also obviously needs these to handle an > > interrupt in userspace, so these

Re: [PATCH 2/3] mmc: tmio: use mmc_can_gpio_cd() instead of checking TMIO_MMC_USE_GPIO_CD

2017-11-20 Thread Wolfram Sang
> I am puzzled by this driver. And now imagine that it is already cleaned up :) I will check these CD patches tomorrow. I need a fresh brain for that. Thanks for the work! signature.asc Description: PGP signature

[PATCH] gpio: iop: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-iop.o see include/linux/module.h for more information This adds the license as "GPL", which matches the header of the file. MODULE_DESCRIPTION and MODU

[PATCH] iio: accel: kxsd9-i2c: add missing MODULE_AUTHOR/DESCRIPTION/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/iio/accel/kxsd9-i2c.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTION

Re: [PATCH 12/30] x86, kaiser: map GDT into user page tables

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: > On Mon, Nov 20, 2017 at 12:22 PM, Thomas Gleixner wrote: > > On Fri, 10 Nov 2017, Dave Hansen wrote: > >> __set_fixmap(get_cpu_gdt_ro_index(cpu), get_cpu_gdt_paddr(cpu), > >> prot); > >> + > >> + /* CPU 0's mapping is done in kaiser_init() *

[PATCH] iio: adc: qcom-vadc: add missing MODULE_DESCRIPTION/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/iio/adc/qcom-vadc-common.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRI

[PATCH] [media] soc_camera: soc_scale_crop: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/soc_camera/soc_scale_crop.o see include/linux/module.h for more information This adds the license as "GPL", which matches the header of the file.

[PATCH] media: tegra-cec: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/tegra-cec/tegra_cec.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MO

[PATCH] mtd: nand: denali_pci: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/mtd/nand/denali_pci.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTION

[PATCHv2] drm: adv7511/33: Fix adv7511_cec_init() failure handling

2017-11-20 Thread Hans Verkuil
If the device tree for a board did not specify a cec clock, then adv7511_cec_init would return an error, which would cause adv7511_probe() to fail and thus there is no HDMI output. There is no need to have adv7511_probe() fail if the CEC initialization fails, so just change adv7511_cec_init() to a

Re: Autoselect patches for stable (Was: Re: [PATCH AUTOSEL for 4.9 36/56] drm/i915: Fix the level 0 max_wm hack on VLV/CHV)

2017-11-20 Thread Alex Deucher
On Mon, Nov 20, 2017 at 8:13 AM, Daniel Vetter wrote: > On Mon, Nov 20, 2017 at 01:39:31PM +0100, Daniel Vetter wrote: >> On Mon, Nov 20, 2017 at 11:21:52AM +, Emil Velikov wrote: >> > Hi all, >> > >> > Since I'm going slightly off-topic, I've tweaked the subject line and >> > trimmed some of

[PATCH] net: phy: cortina: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/net/phy/cortina.o see include/linux/module.h for more information This adds the license as "GPL", which matches the header of the file. MODULE_DESCRIPTION and MO

Re: [PATCH] fs: Safe rcu access to hlist.

2017-11-20 Thread Luc Van Oostenryck
On Mon, Nov 20, 2017 at 12:42:53PM -0800, Matthew Wilcox wrote: > > I disagree. The notion of whether a pointer is protected by RCU or not > is definitely not transient. Sure. But what about the memory it points to? It's just 'normal' kernel memory, there is nowhere something like some 'RCU memo

[PATCH] power: reset: zx-reboot: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/power/reset/zx-reboot.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTI

[PATCH] pinctrl: pxa: pxa2xx: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/pinctrl/pxa/pinctrl-pxa2xx.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DES

Re: [PATCH 02/16] x86/dumpstack: Add get_stack_info() support for the SYSENTER stack

2017-11-20 Thread Josh Poimboeuf
On Mon, Nov 20, 2017 at 12:46:13PM -0800, Andy Lutomirski wrote: > On Mon, Nov 20, 2017 at 12:42 PM, Josh Poimboeuf wrote: > > On Mon, Nov 20, 2017 at 09:07:33AM -0800, Andy Lutomirski wrote: > >> +bool in_SYSENTER_stack(unsigned long *stack, struct stack_info *info) > > > > Can you make it lowerc

Re: [PATCH v2 16/18] arm64: crypto: disable LTO for aes-ce-cipher.c

2017-11-20 Thread Sami Tolvanen
On Mon, Nov 20, 2017 at 03:25:31PM +, Ard Biesheuvel wrote: > However, under LTO this all changes, and it is no longer guaranteed > that the NEON registers are only touched between the kernel mode > neon begin/end calls. LTO operates on LLVM IR, so disabling LTO for this file should make sure

[PATCH] mfd: kempld-core: Use resource_size function on resource object

2017-11-20 Thread Vasyl Gomonovych
drivers/mfd/kempld-core.c:461:13-16: WARNING: Suspicious code. resource_size is maybe missing with ioport Generated by: scripts/coccinelle/api/resource_size.cocci Signed-off-by: Vasyl Gomonovych --- drivers/mfd/kempld-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

Re: [PATCH v2 00/18] Add support for clang LTO

2017-11-20 Thread Sami Tolvanen
On Mon, Nov 20, 2017 at 03:21:40PM +, Mark Rutland wrote: > Would it be possible to split this into: > > (1) (basic) arm64 clang support > (2) gold support (no LTO) > (3) LTO support > > ... with any necessary workarounds added as-required to the relevant > series? Absolutely. I'll split the

Re: [PATCH 2/7] kbuild: Add P= command line flag to run checkpatch

2017-11-20 Thread Knut Omang
On Tue, 2017-11-21 at 01:18 +0900, Masahiro Yamada wrote: > 2017-11-17 2:01 GMT+09:00 Knut Omang : > > Add interpretation of a new environment variable P={1,2} in spirit of the > > C= option, but executing checkpatch instead of sparse. > > > > Signed-off-by: Knut Omang > > Reviewed-by: Håkon Bugg

Re: [PATCH 02/16] x86/dumpstack: Add get_stack_info() support for the SYSENTER stack

2017-11-20 Thread Andy Lutomirski
On Mon, Nov 20, 2017 at 1:00 PM, Josh Poimboeuf wrote: > On Mon, Nov 20, 2017 at 12:46:13PM -0800, Andy Lutomirski wrote: >> On Mon, Nov 20, 2017 at 12:42 PM, Josh Poimboeuf wrote: >> > On Mon, Nov 20, 2017 at 09:07:33AM -0800, Andy Lutomirski wrote: >> >> +bool in_SYSENTER_stack(unsigned long *s

Re: [PATCH v8 2/3] arm: dts: add Nuvoton NPCM750 device tree

2017-11-20 Thread Rob Herring
On Fri, Nov 17, 2017 at 11:07:46AM -0800, Brendan Higgins wrote: > Add a common device tree for all Nuvoton NPCM750 BMCs and a board > specific device tree for the NPCM750 (Poleg) evaluation board. > > Signed-off-by: Brendan Higgins > Reviewed-by: Tomer Maimon > Reviewed-by: Avi Fishman > Revie

Re: [PATCH 2/7] kbuild: Add P= command line flag to run checkpatch

2017-11-20 Thread Knut Omang
On Mon, 2017-11-20 at 21:08 +0100, Luc Van Oostenryck wrote: > On Mon, Nov 20, 2017 at 12:48:35PM -0700, Jim Davis wrote: > > > > I'd be nice if people could just specify CHECK and CHECKFLAGS to run > > their favorite checker, but currently CHECKFLAGS seems hardwired for > > running sparse. So so

Re: [PATCH] dt-bindings: trivial-devices: Remove fsl,mc13892

2017-11-20 Thread Rob Herring
On Sat, Nov 18, 2017 at 03:22:32AM +0100, Jonathan Neuschäfer wrote: > This device's bindings are not trivial: Additional properties are > documented in in Documentation/devicetree/bindings/mfd/mc13xxx.txt. > > Signed-off-by: Jonathan Neuschäfer > --- > Documentation/devicetree/bindings/trivial-

Re: [PATCH v3 2/3] leds: Add driver for Qualcomm LPG

2017-11-20 Thread Bjorn Andersson
On Sun 19 Nov 13:36 PST 2017, Jacek Anaszewski wrote: > Hi Bjorn, > > Thanks for the patch. Please refer to my comments in the code. > > On 11/15/2017 08:13 AM, Bjorn Andersson wrote: [..] > > diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig > > index 52ea34e337cd..ccc3aa4b2474 100644 >

Re: mm/percpu.c: use smarter memory allocation for struct pcpu_alloc_info (crisv32 hang)

2017-11-20 Thread Guenter Roeck
On Mon, Nov 20, 2017 at 03:21:32PM -0500, Nicolas Pitre wrote: > On Mon, 20 Nov 2017, Guenter Roeck wrote: > > > On Mon, Nov 20, 2017 at 01:18:38PM -0500, Nicolas Pitre wrote: > > > On Sun, 19 Nov 2017, Guenter Roeck wrote: > > > > > > > On 11/19/2017 08:08 PM, Nicolas Pitre wrote: > > > > > On S

Re: [PATCH] KVM: VMX: Fix rflags cache during vCPU reset

2017-11-20 Thread Paolo Bonzini
On 20/11/2017 10:17, Wanpeng Li wrote: >[ cut here ] >WARNING: CPU: 6 PID: 24431 at /home/kernel/linux/arch/x86/kvm//x86.c:7302 > kvm_arch_vcpu_ioctl_run+0x651/0x2ea0 [kvm] >CPU: 6 PID: 24431 Comm: reprotest Tainted: GW OE 4.14.0+ #26 >RIP: 0010:k

Re: [PATCH 3.18 00/38] 3.18.83-stable review

2017-11-20 Thread Shuah Khan
On 11/19/2017 07:29 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.83 release. > There are 38 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 know. > > Responses s

Re: [PATCH 4.4 00/59] 4.4.100-stable review

2017-11-20 Thread Shuah Khan
On 11/19/2017 07:32 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.100 release. > There are 59 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 know. > > Responses s

[PATCH 1/2] of: documentation: add bindings documentation for TS-7970

2017-11-20 Thread Sebastien Bourdelin
This adds the documentation for the TS-7970 by Technologic Systems. Signed-off-by: Sebastien Bourdelin --- Documentation/devicetree/bindings/arm/technologic.txt | 6 ++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/technologic.txt b/Documentation/device

[PATCH 2/2] ARM: dts: TS-7970: add basic device tree

2017-11-20 Thread Sebastien Bourdelin
These device trees add support for TS-7970 by Technologic Systems. More details here: https://wiki.embeddedarm.com/wiki/TS-7970 Signed-off-by: Sebastien Bourdelin --- arch/arm/boot/dts/Makefile| 2 + arch/arm/boot/dts/imx6dl-ts7970.dts | 54 +++ arch/arm/boot/dts/imx6q-ts797

Re: [PATCH 4.9 00/72] 4.9.64-stable review

2017-11-20 Thread Shuah Khan
On 11/19/2017 07:38 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.64 release. > There are 72 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 know. > > Responses sh

Re: [PATCH 4.13 00/28] 4.13.15-stable review

2017-11-20 Thread Shuah Khan
On 11/19/2017 07:43 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.13.15 release. > There are 28 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 know. > > Responses s

Re: [PATCH 4.14 00/31] 4.14.1-stable review

2017-11-20 Thread Shuah Khan
On 11/19/2017 07:59 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.1 release. > There are 31 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 know. > > Responses sh

[PATCH] Make asus-nb-wmi support the Zenbook UX430UQ

2017-11-20 Thread Kiernan Hager
This patch adds support for the Zenbook UX430UQ to the asus_nb_wmi driver. It also renames "quirk_asus_ux330uak" to "quirk_asus_forceals" because it is now used for more than one model of computer, and should thus have a more general name. Signed-off-by: Kiernan Hager --- drivers/platform/x86/as

Re: [PATCH] fs: Safe rcu access to hlist.

2017-11-20 Thread Paul E. McKenney
On Mon, Nov 20, 2017 at 09:58:02PM +0100, Luc Van Oostenryck wrote: > On Mon, Nov 20, 2017 at 12:42:53PM -0800, Matthew Wilcox wrote: > > > > I disagree. The notion of whether a pointer is protected by RCU or not > > is definitely not transient. > > Sure. But what about the memory it points to?

Re: [PATCH 2/7] kbuild: Add P= command line flag to run checkpatch

2017-11-20 Thread Luc Van Oostenryck
On Mon, Nov 20, 2017 at 10:10:12PM +0100, Knut Omang wrote: > On Mon, 2017-11-20 at 21:08 +0100, Luc Van Oostenryck wrote: > > > > It should be noted though that CHECKFLAGS contains very very few > > sparse specific things. It's mainly flags for the compiler > > coming from KBUILD_CFLAGS (which o

[PATCH] selftest: sync: Use common error handling code in sync_file_info()

2017-11-20 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 20 Nov 2017 22:20:37 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- tools/testing/selftests/sync/sync.c |

[PATCH] Fix style issues in olpc_dcon

2017-11-20 Thread zebmccorkle
From: Zebulon McCorkle The olpc_dcon driver had some slight style issues, mostly pertaining to indentation in function calls and definitions. I've solved those, and plan to work on the issues in the TODO. Signed-off-by: Zebulon McCorkle --- drivers/staging/olpc_dcon/olpc_dcon.c | 30 +

[PATCH] proc: fix /proc/*/map_files lookup

2017-11-20 Thread Alexey Dobriyan
Current code does: if (sscanf(dentry->d_name.name, "%lx-%lx", start, end) != 2) However sscanf() is broken garbage. It silently accepts whitespace between format specifiers (did you know that?). It silently accepts valid strings which result in integer overflow. Do not use sscanf() for

Re: [PATCH 02/16] x86/dumpstack: Add get_stack_info() support for the SYSENTER stack

2017-11-20 Thread Josh Poimboeuf
On Mon, Nov 20, 2017 at 01:07:16PM -0800, Andy Lutomirski wrote: > >> but, more importantly, the OOPS unwinder will just bail without this > >> patch. With the patch, we get a valid unwind, except that everything > >> has a ? in front. > > > > Hm. I can't even fathom how that's possible. Are yo

Re: [patches] Re: [PATCH] dt-bindings: Add a RISC-V SBI firmware node

2017-11-20 Thread Palmer Dabbelt
On Mon, 20 Nov 2017 12:28:56 PST (-0800), j.neuschae...@gmx.net wrote: On Mon, Nov 20, 2017 at 11:50:00AM -0800, Palmer Dabbelt wrote: The RISC-V privileged ISA mandates the presence of an SBI, but there's no reason not to put it in the device tree. This would allow us to possibly remove the SB

Re: [PATCH v2 16/18] arm64: crypto: disable LTO for aes-ce-cipher.c

2017-11-20 Thread Alex Matveev
On Mon, 20 Nov 2017 15:20:14 + Mark Rutland wrote: > On Wed, Nov 15, 2017 at 01:34:26PM -0800, Sami Tolvanen wrote: > > CONFIG_LTO_CLANG requires the use of clang's integrated assembler, > > which doesn't understand the inline assembly in aes-ce-cipher.c. > > Disable LTO for the file to work

Re: [PATCH v2 1/4] dt-bindings: Add vendor prefix for ilitek

2017-11-20 Thread Rob Herring
On Sun, Nov 19, 2017 at 02:12:05PM -0600, David Lechner wrote: > This adds the vendor prefix ilitek for ILI Technology Corporation (ILITEK). > > This prefix is already used several places and I will be adding more. > > Signed-off-by: David Lechner > --- > > v2 changes: > * New patch in v2 > >

Re: [PATCH v2 2/4] dt-bindings: Add binding for Ilitek ILI9225 display panels

2017-11-20 Thread Rob Herring
On Sun, Nov 19, 2017 at 02:12:06PM -0600, David Lechner wrote: > This adds a new binding for display panels that use an Ilitek ILI9225 > controller. > > The "ilitek,ili9225-2.2in-176x220" device listed has no identification > markings whatsoever and an hour of googling turned up nothing, hence the

Re: [PATCH 02/16] x86/dumpstack: Add get_stack_info() support for the SYSENTER stack

2017-11-20 Thread Andy Lutomirski
On Mon, Nov 20, 2017 at 1:27 PM, Josh Poimboeuf wrote: > On Mon, Nov 20, 2017 at 01:07:16PM -0800, Andy Lutomirski wrote: >> >> but, more importantly, the OOPS unwinder will just bail without this >> >> patch. With the patch, we get a valid unwind, except that everything >> >> has a ? in front.

Re: [PATCH v2 16/18] arm64: crypto: disable LTO for aes-ce-cipher.c

2017-11-20 Thread Ard Biesheuvel
On 20 November 2017 at 21:29, Alex Matveev wrote: > On Mon, 20 Nov 2017 15:20:14 + > Mark Rutland wrote: > >> On Wed, Nov 15, 2017 at 01:34:26PM -0800, Sami Tolvanen wrote: >> > CONFIG_LTO_CLANG requires the use of clang's integrated assembler, >> > which doesn't understand the inline assembl

Re: Dell Vostro 3360 multimedia keys

2017-11-20 Thread Oleksandr Natalenko
Hi, Pali et al. Thanks for the detailed explanation. I've managed to craft the following ugly hack: (I explain my thoughts on it under the snippet) === commit a2baeaba1657a47359efea61865aa012da509e03 Author: Oleksandr Natalenko Date: Mon Nov 20 13:25:43 2017 +0100 dell-wmi: support Dell

Re: [PATCH] qed: fix unnecessary call to memset cocci warnings

2017-11-20 Thread Vasyl Gomonovych
It doesn't apply becouse of identical one qed: use kzalloc instead of kmalloc and memset.

Re: [PATCH PREEMPT RT] rt-mutex: fix deadlock in device mapper

2017-11-20 Thread Mikulas Patocka
On Mon, 20 Nov 2017, Sebastian Siewior wrote: > On 2017-11-18 19:37:10 [+0100], Mike Galbraith wrote: > > Below is my 2012 3.0-rt version of that for reference; at that time we > > were using slab, and slab_lock referenced below was a local_lock.  The > > comment came from crash analysis of a de

[PATCH] cpufreq: mediatek: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/cpufreq/mediatek-cpufreq.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRI

[PATCH] mmc: renesas_sdhi: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/mmc/host/renesas_sdhi_core.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DES

Re: [RESEND PATCH v3 1/2] documentation: Add compatibles for Amlogic Meson AXG pin controllers

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 06:23:53PM +0800, Yixun Lan wrote: > From: Xingyu Chen > > Add compatibles for Amlogic Meson AXG pin controllers > > Reviewed-by: Neil Armstrong > Signed-off-by: Xingyu Chen > Signed-off-by: Yixun Lan > --- > Documentation/devicetree/bindings/pinctrl/meson,pinctrl.txt

Re: [PATCH PREEMPT RT] rt-mutex: fix deadlock in device mapper

2017-11-20 Thread Mikulas Patocka
On Sat, 18 Nov 2017, Mike Galbraith wrote: > On Fri, 2017-11-17 at 15:57 +0100, Sebastian Siewior wrote: > > On 2017-11-13 12:56:53 [-0500], Mikulas Patocka wrote: > > > Hi > > Hi, > > > > > I'm submitting this patch for the CONFIG_PREEMPT_RT patch. It fixes > > > deadlocks in device mapper wh

[PATCH] soc: qcom: rmtfs_mem: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/soc/qcom/rmtfs_mem.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTION

[PATCH] phy: qcom-ufs: add missing MODULE_DESCRIPTION/LICENSE

2017-11-20 Thread Jesse Chan
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESC

Re: [PATCH v2 1/5] dt-bindings: i2c: update documentation for the Meson-AXG

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 10:54:11PM +0800, Yixun Lan wrote: > From: Jian Hu > > Update the doc to explicitly add Meson-AXG to support list > > Signed-off-by: Jian Hu > Signed-off-by: Yixun Lan > --- > Documentation/devicetree/bindings/i2c/i2c-meson.txt | 6 +- > 1 file changed, 5 insertion

Re: [PATCH v5 1/8] dt-bindings: fsi: Add SBEFIFO documentation

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 01:46:50PM -0600, Eddie James wrote: > From: "Edward A. James" > > Document the bindings for the SBE CFAM device. SBE devices are > located on a CFAM off an FSI bus. > > Signed-off-by: Edward A. James > --- > .../devicetree/bindings/fsi/ibm,p9-sbefifo.txt | 35 > ++

Re: [PATCH v5 5/8] dt-bindings: fsi: Add OCC documentation

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 01:46:54PM -0600, Eddie James wrote: > From: "Edward A. James" > > Document the bindings for the P9 OCC device. OCC devices are accessed > through the SBEFIFO. > > Signed-off-by: Edward A. James > --- > Documentation/devicetree/bindings/fsi/ibm,p9-occ.txt | 18 +

Re: RFC: Copying Device Tree File into reserved area of VMLINUX before deployment

2017-11-20 Thread Frank Rowand
Hi Ulf, Rob, On 11/20/17 15:19, Ulf Samuelsson wrote: > > > On 2017-11-20 05:32, Frank Rowand wrote: >> Hi Ulf, >> >> >> On 11/19/17 23:23, Frank Rowand wrote: >>> adding devicetree list, devicetree maintainers >>> >>> On 11/18/17 12:59, Ulf Samuelsson wrote: I noticed when checking out the

Re: Autoselect patches for stable (Was: Re: [PATCH AUTOSEL for 4.9 36/56] drm/i915: Fix the level 0 max_wm hack on VLV/CHV)

2017-11-20 Thread Dave Airlie
On 20 November 2017 at 23:13, Daniel Vetter wrote: > On Mon, Nov 20, 2017 at 01:39:31PM +0100, Daniel Vetter wrote: >> On Mon, Nov 20, 2017 at 11:21:52AM +, Emil Velikov wrote: >> > Hi all, >> > >> > Since I'm going slightly off-topic, I've tweaked the subject line and >> > trimmed some of the

KASAN help, please (Re: [PATCH 00/16] Entry stuff, in decent shape now)

2017-11-20 Thread Andy Lutomirski
On Mon, Nov 20, 2017 at 9:07 AM, Andy Lutomirski wrote: > This sets up stack switching, including for SYSCALL. I think it's > in decent shape. > > Known issues: > - KASAN is likely to be busted. This could be fixed either by teaching >KASAN that cpu_entry_area contains valid stacks (I have

Re: [PATCH] dt-bindings: Add a RISC-V SBI firmware node

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 11:50:00AM -0800, Palmer Dabbelt wrote: > The RISC-V privileged ISA mandates the presence of an SBI, but there's > no reason not to put it in the device tree. This would allow us to > possibly remove the SBI later. If it is mandatory, then it should not be in DT. DT is not

Re: [PATCH v3 3/3] DT: leds: Add Qualcomm Light Pulse Generator binding

2017-11-20 Thread Bjorn Andersson
On Mon 20 Nov 12:35 PST 2017, Jacek Anaszewski wrote: > On 11/20/2017 08:58 PM, Bjorn Andersson wrote: > > On Sun 19 Nov 13:35 PST 2017, Jacek Anaszewski wrote: > > > >> Hi Bjorn, > >> > >> Thanks for the update. > >> > >> On 11/15/2017 08:13 AM, Bjorn Andersson wrote: > >>> This adds the binding

Re: [PATCH] dt-bindings: Add an enable method to RISC-V

2017-11-20 Thread Rob Herring
On Mon, Nov 20, 2017 at 11:50:22AM -0800, Palmer Dabbelt wrote: > RISC-V doesn't currently specify a mechanism for enabling or disabling > CPUs. Instead, we assume that all CPUs are enabled on boot, and if > someone wants to save power we instead put a CPU to sleep via a WFI > loop. > > This patc

Re: [PATCH v7 3/5] fw_cfg: do DMA read operation

2017-11-20 Thread Michael S. Tsirkin
On Mon, Nov 20, 2017 at 10:55:17AM +0100, Marc-André Lureau wrote: > Modify fw_cfg_read_blob() to use DMA if the device supports it. > Return errors, because the operation may fail. > > The DMA operation is expected to run synchronously with today qemu, > but the specification states that it may b

[PATCH 0/2] staging: pi433: fix logging and naming issues

2017-11-20 Thread Marcin Ciupak
These patches fix messages logging and device naming issues when probing for new radio module. Those two changes are send as one patchset since both modify the same part of code (second patch depends on first one) Marcin Ciupak (2): staging: pi433: fix (NULL device *) in log message staging:

Re: [PATCH 01/16] x86/asm/64: Allocate and enable the SYSENTER stack

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: > This will simplify future changes that want scratch variables early in > the SYSENTER handler -- they'll be able to spill registers to the > stack. It also lets us get rid of a SWAPGS_UNSAFE_STACK user. > > This does not depend on CONFIG_IA32_EMULATI

[PATCH 1/2] staging: pi433: fix (NULL device *) in log message

2017-11-20 Thread Marcin Ciupak
(NULL device *) is printed in log message in pi433_probe and pi433_get_minor functions due to device->dev being used prior to call to device_create function. Signed-off-by: Marcin Ciupak --- drivers/staging/pi433/pi433_if.c | 26 +- 1 file changed, 13 insertions(+), 13 de

Re: [PATCH] gpio: ath79: add missing MODULE_DESCRIPTION/LICENSE

2017-11-20 Thread Alban
On Mon, 20 Nov 2017 12:54:26 -0800 Jesse Chan wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o > see include/linux/module.h for more information > > This adds the license as "G

Re: [PATCH 02/16] x86/dumpstack: Add get_stack_info() support for the SYSENTER stack

2017-11-20 Thread Josh Poimboeuf
On Mon, Nov 20, 2017 at 01:30:12PM -0800, Andy Lutomirski wrote: > On Mon, Nov 20, 2017 at 1:27 PM, Josh Poimboeuf wrote: > > On Mon, Nov 20, 2017 at 01:07:16PM -0800, Andy Lutomirski wrote: > >> >> but, more importantly, the OOPS unwinder will just bail without this > >> >> patch. With the patch

Re: [PATCH 03/16] x86/gdt: Put per-cpu GDT remaps in ascending order

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: > We currently have CPU 0's GDT at the top of the GDT range and > higher-numbered CPUs at lower addreses. This happens because the > fixmap is upside down (index 0 is the top of the fixmap). > > Flip it so that GDTs are in ascending order by virtual ad

[PATCH 2/2] staging: pi433: fix naming when more than one radio is used

2017-11-20 Thread Marcin Ciupak
When using more than one hardware radio module pi433_probe fails as the same name is used for all modules. Create unique name by adding minor number to the device name. Signed-off-by: Marcin Ciupak --- drivers/staging/pi433/pi433_if.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-)

Re: [PATCH v2] iommu/ipmmu-vmsa: Do not replace bus IOMMU ops on driver init.

2017-11-20 Thread Alex Williamson
On Mon, 20 Nov 2017 14:25:14 + Liviu Dudau wrote: > On Fri, Oct 13, 2017 at 04:48:45PM +0100, Robin Murphy wrote: > > Hi Joerg, > > Hi, > > > > > On 20/09/17 15:13, Liviu Dudau wrote: > > > If the IPMMU driver is compiled in the kernel it will replace the > > > platform bus IOMMU ops o

Re: [PATCH 04/16] x86/fixmap: Generalize the GDT fixmap mechanism

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: Just a few nits. > /* Provide the fixmap address of the remapped GDT */ > static inline struct desc_struct *get_cpu_gdt_ro(int cpu) > { > - unsigned int idx = get_cpu_gdt_ro_index(cpu); > - return (struct desc_struct *)__fix_to_virt(idx); > +

Re: [PATCH 1/2] Documentation: add Sorting section to kernel-api

2017-11-20 Thread Randy Dunlap
On 11/20/17 10:09, Jonathan Corbet wrote: > On Wed, 15 Nov 2017 19:19:19 -0800 > Randy Dunlap wrote: > >> Add sort() and list_sort() to the kernel API documentation in a >> new "Sorting" section. > > Neither of these patches applies to current docs-next. Seems they > conflict with the math func

[PATCH] irq-imgpdc: Use resource_size function on resource object

2017-11-20 Thread Vasyl Gomonovych
drivers/irqchip/irq-imgpdc.c:327:20-23: WARNING: Suspicious code. resource_size is maybe missing with res_regs Generated by: scripts/coccinelle/api/resource_size.cocci Signed-off-by: Vasyl Gomonovych --- drivers/irqchip/irq-imgpdc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

Re: [PATCH v2 16/18] arm64: crypto: disable LTO for aes-ce-cipher.c

2017-11-20 Thread Alex Matveev
Sami, this seems to be better solution for aes-ce-cipher.c problem. Regards, Alex From 6bcdd763b56ce10a77a79373a46fc0e8d9026178 Mon Sep 17 00:00:00 2001 From: Alex Matveev Date: Mon, 20 Nov 2017 21:30:38 + Subject: [PATCH] arm64: crypto: fix typo in aes_sub() Clang's integrated assembler c

Re: [PATCH v5 5/8] dt-bindings: fsi: Add OCC documentation

2017-11-20 Thread Eddie James
On 11/20/2017 03:39 PM, Rob Herring wrote: On Mon, Nov 20, 2017 at 01:46:54PM -0600, Eddie James wrote: From: "Edward A. James" Document the bindings for the P9 OCC device. OCC devices are accessed through the SBEFIFO. Signed-off-by: Edward A. James --- Documentation/devicetree/bindings/

Re: [PATCH] phy: qcom-ufs: add missing MODULE_DESCRIPTION/LICENSE

2017-11-20 Thread Bjorn Andersson
On Mon 20 Nov 13:33 PST 2017, Jesse Chan wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in > drivers/phy/qualcomm/phy-qcom-ufs.o > see include/linux/module.h for more information > > This adds the licens

Re: [PATCH PREEMPT RT] rt-mutex: fix deadlock in device mapper

2017-11-20 Thread Mikulas Patocka
On Mon, 20 Nov 2017, Mikulas Patocka wrote: > > > On Mon, 20 Nov 2017, Sebastian Siewior wrote: > > > On 2017-11-18 19:37:10 [+0100], Mike Galbraith wrote: > > > Below is my 2012 3.0-rt version of that for reference; at that time we > > > were using slab, and slab_lock referenced below was a

[PATCH] ASoC: fsl_ssi: remove duplicated flag setting in fsl_ssi_setup_reg_vals()

2017-11-20 Thread Maciej S. Szmigiero
We don't need to set CCSR_SSI_SIER_RFF0_EN / CCSR_SSI_SIER_TFE0_EN bits in reg->rx.sier / reg->tx.sier variables in a non-AC'97 mode considering we had just initialized these variables to these very values unconditionally a few lines earlier. Signed-off-by: Maciej S. Szmigiero --- sound/soc/fsl/

Re: [PATCH] tpm: Add explicit chip->ops locking for sysfs attributes.

2017-11-20 Thread Jarkko Sakkinen
On Thu, Nov 16, 2017 at 01:25:01PM -0800, Guenter Roeck wrote: > Add explicit chip->ops locking for all sysfs attributes. > This lets us support those attributes on tpm2 devices. > > Signed-off-by: Guenter Roeck > --- > drivers/char/tpm/tpm-chip.c | 4 -- > drivers/char/tpm/tpm-sysfs.c | 125

[PATCH] ASoC: fsl_ssi: call _fsl_ssi_set_dai_fmt() just once in AC'97 mode

2017-11-20 Thread Maciej S. Szmigiero
In AC'97 mode we configure and start SSI RX / TX on probe path via a call to _fsl_ssi_set_dai_fmt() function. We don't need to call this function again later and in fact don't want to do it since this function temporarily sets STCR, SRCR and SCR to some intermediate values. We need to make sure, h

[PATCH 1/2] ASoC: fsl_ssi: AC'97 ops need regmap, clock and cleaning up on failure

2017-11-20 Thread Maciej S. Szmigiero
AC'97 ops (register read / write) need SSI regmap and clock, so they have to be set after them. We also need to set these ops back to NULL if we fail the probe. Signed-off-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(

Nokia N900: Funny screen effects, v4.14 with certain configs

2017-11-20 Thread Pavel Machek
Hi! I'm trying to find common configuration for N900, N9 and N950. ...and I found that N900, at least with DRM configuration, is flakey. Depending on exact .config, and exact way I boot it (timing of 0x) display either works or just shows moving horizontal stripes. In particular, ondemand cp

[PATCH 2/2] ASoC: fsl_ssi: serialize AC'97 register access operations

2017-11-20 Thread Maciej S. Szmigiero
AC'97 register access operations (both read and write) on SSI use a one, shared set of SSI registers for AC'97 register address and data. This means that only one such access is possible at a time and so all these operations need to be serialized. Since an AC'97 register access operation in this d

Re: [PATCH] proc: fix /proc/*/map_files lookup

2017-11-20 Thread Andrew Morton
On Tue, 21 Nov 2017 00:27:06 +0300 Alexey Dobriyan wrote: > Current code does: > > if (sscanf(dentry->d_name.name, "%lx-%lx", start, end) != 2) > > However sscanf() is broken garbage. > > It silently accepts whitespace between format specifiers > (did you know that?). > > It silently ac

Re: [PATCH v5 00/11] Intel SGX Driver

2017-11-20 Thread Jarkko Sakkinen
On Wed, Nov 15, 2017 at 11:35:42AM +0100, Thomas Gleixner wrote: > On Mon, 13 Nov 2017, Jarkko Sakkinen wrote: > > Any reason why you did not CC the x86 maintainers and other x86 developers > who have had opinions on the first attempt to get SGX into the kernel? > > This is not some driver for a

Re: [PATCH] afs: remove redundant assignment of dvnode to itself

2017-11-20 Thread David Howells
Colin King wrote: > From: Colin Ian King > > The assignment of dvnode to itself is redundant and can be removed. > Cleans up warning detected by cppcheck: > > fs/afs/dir.c:975: (warning) Redundant assignment of 'dvnode' to itself. > > Fixes: d2ddc776a458 ("afs: Overhaul volume and server reco

Re: [PATCH] afs: cell: Remove unnecessary code in afs_lookup_cell

2017-11-20 Thread David Howells
Gustavo A. R. Silva wrote: > Due to recent changes this piece of code is no longer needed. > > Addresses-Coverity-ID: 1462033 > Link: https://lkml.kernel.org/r/4923.1510957...@warthog.procyon.org.uk > Signed-off-by: Gustavo A. R. Silva Applied, thanks. David

Re: [intel-sgx-kernel-dev] [PATCH v5 08/11] intel_sgx: in-kernel launch enclave

2017-11-20 Thread Jarkko Sakkinen
On Tue, Nov 14, 2017 at 10:05:05PM +0200, Jarkko Sakkinen wrote: > On Tue, Nov 14, 2017 at 09:05:09AM -0800, Sean Christopherson wrote: > > Unless there is some conflict you are worried about, "signing_key.pem" is > > preferable as the default name so that the key is ignored via the top-level > > .

Re: [PATCH 05/16] x86/asm: Fix assumptions that the HW TSS is at the beginning of cpu_tss

2017-11-20 Thread Thomas Gleixner
On Mon, 20 Nov 2017, Andy Lutomirski wrote: > I'm going to move SYSENTER_stack to the beginning of cpu_tss to help > detect overflow. Before this can happen, I need to fix several code > paths that hardcode assumptions about the old layout. > > Signed-off-by: Andy Lutomirski Reviewed-by: Thoma

Re: [PATCH v3 1/3] mm/mempolicy: remove redundant check in get_nodes

2017-11-20 Thread David Rientjes
On Fri, 17 Nov 2017, Yisheng Xie wrote: > We have already checked whether maxnode is a page worth of bits, by: > maxnode > PAGE_SIZE*BITS_PER_BYTE > > So no need to check it once more. > > Acked-by: Vlastimil Babka > Signed-off-by: Yisheng Xie Acked-by: David Rientjes

Re: [PATCH v5 08/11] intel_sgx: in-kernel launch enclave

2017-11-20 Thread Jarkko Sakkinen
On Wed, Nov 15, 2017 at 12:50:06PM +0100, Peter Zijlstra wrote: > On Mon, Nov 13, 2017 at 09:45:25PM +0200, Jarkko Sakkinen wrote: > > TinyCrypt (https://github.com/01org/tinycrypt) is used as AES > > implementation, which is not timing resistant. Eventually this needs to > > be replaced with AES-N

<    2   3   4   5   6   7   8   9   >