Re: [PATCH 3/8] ARM: dts: aspeed: Add I2C buses

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: > Now with an upstream i2c bus driver, we can add  the 14 i2c buses that > exist in ASPEED G4 and G5 generation SoCs. > > It also adds aliases for the 14 built-in I2C busses to ensure userspace > sees the numbering staring from zero and counti

Re: [PATCH] fsl/fman: remove of_node

2017-10-03 Thread David Miller
From: Madalin-cristian Bucur Date: Tue, 3 Oct 2017 08:49:31 + > My patch removes the of_node that was set to a device that was not an > of_device, preventing duplicated probing of both the real of_device > and the "fake" one created through this assignment. > > I understand that the DSA issu

Re: [PATCH RESEND net 0/9] Fixes, cleanup and modernization for some legacy ethernet NIC drivers

2017-10-03 Thread David Miller
From: Finn Thain Date: Mon, 2 Oct 2017 21:07:17 -0400 (EDT) > This patch series fixes some logging bugs and adds some missing message > severity levels. > > There are also cleanup patches for dead code and some Kconfig cruft. > > Custom debug message logging is converted to netif_* calls to re

Re: [PATCH] kbuild: comments cleanup in Makefile.lib

2017-10-03 Thread Masahiro Yamada
Hi Cao, 2017-09-19 20:36 GMT+09:00 Cao jin : > It has: > 1. Move comments close to what it want to comment. > 2. Comments cleanup & improvement. > > Signed-off-by: Cao jin > --- > scripts/Makefile.lib | 19 ++- > 1 file changed, 6 insertions(+), 13 deletions(-) > > diff --git a/

Re: [PATCH 4/8] ARM: dts: aspeed-romulus: Add I2C devices

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: > Enable the buses that are in use and the devices that are attached. > Currently that is just the battery backed RTC. >  > Some of these buses are for hotplugged cards, such as PCIe cards. Others > do not yet have upstream drivers, so there ar

Re: [PATCH 5/8] ARM: dts: aspeed-palmetto: Add I2C devices

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: > Enable the buses that are in use and the devices that are attached. > Currently that includes the battery backed RTC, temperature > measurement > and EEPROM. > > Some of these buses are for hotplugged cards, such as PCIe cards. > Others do n

Re: [PATCH 6/8] ARM: dts: aspeed-ast2500: Add I2C devices

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: > Enable the buses that are in use and the devices that are attached. > Currently that includes temperature measurement and EEPROM. >  > Signed-off-by: Joel Stanley Reviewed-by: Andrew Jeffery > --- >  arch/arm/boot/dts/aspeed-ast2500-evb.d

Re: [PATCH 7/8] ARM: dts: aspeed: Add aliases for UARTs

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: > Existing userspace expects the console (UART5) to be at /dev/ttyS4.  To > ensure the UARTs show up where users expect them, we give them fixed > aliases starting at 0. >  > Signed-off-by: Joel Stanley Reviewed-by: Andrew Jeffery > --- >  

Re: [PATCH 2/8] ARM: dts: aspeed: Reorder ADC node

2017-10-03 Thread Joel Stanley
On Wed, Oct 4, 2017 at 1:05 PM, Andrew Jeffery wrote: > On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: >> --- a/arch/arm/boot/dts/aspeed-g4.dtsi >> +++ b/arch/arm/boot/dts/aspeed-g4.dtsi >> @@ -129,6 +129,14 @@ >> }; >> }; >> >> +

Re: [PATCH 2/8] ARM: dts: aspeed: Reorder ADC node

2017-10-03 Thread Andrew Jeffery
On Wed, 2017-10-04 at 14:43 +0930, Joel Stanley wrote: > > > > > > + adc: adc@1e6e9000 { > > > + compatible = "aspeed,ast2500-adc"; > > > + reg = <0x1e6e9000 0xb0>; > > > > Did you intend to change the size cell value here

Re: [PATCH 4/8] ARM: dts: aspeed-romulus: Add I2C devices

2017-10-03 Thread Joel Stanley
On Wed, Oct 4, 2017 at 2:32 PM, Andrew Jeffery wrote: > On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: >> Enable the buses that are in use and the devices that are attached. >> Currently that is just the battery backed RTC. >> >> Some of these buses are for hotplugged cards, such as PCIe c

Re: [PATCH 8/8] ARM: dts: aspeed: Clean up UART nodes

2017-10-03 Thread Andrew Jeffery
On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: >  - Shorten size of reg property so it covers only the implemented >  registers >  >  - Add VUART compatible >  >  - Move stray uart1 in g5 definition >  >  - Remove outdated current-speed property. Different bootloaders use >  different speed

[PATCH] kconfig: Document the 'symbol' struct

2017-10-03 Thread Ulf Magnusson
Visibility and choices in particular might be a bit tricky to figure out. Also fix existing comment to point out that P_MENU is also used for menus. Signed-off-by: Ulf Magnusson --- scripts/kconfig/expr.h | 45 - 1 file changed, 44 insertions(+), 1 de

Re: [patch V2 22/29] lockup_detector: Make watchdog_nmi_reconfigure() two stage

2017-10-03 Thread Michael Ellerman
Thomas Gleixner writes: > On Tue, 3 Oct 2017, Thomas Gleixner wrote: >> On Tue, 3 Oct 2017, Thomas Gleixner wrote: >> > On Tue, 3 Oct 2017, Michael Ellerman wrote: >> > > Hmm, I tried that patch, it makes the warning go away. But then I >> > > triggered a deliberate hard lockup and got nothing. >

Re: [PATCH 8/8] ARM: dts: aspeed: Clean up UART nodes

2017-10-03 Thread Joel Stanley
On Wed, Oct 4, 2017 at 2:58 PM, Andrew Jeffery wrote: > On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote: >> - uart6: serial@1e787000 { >> - compatible = "ns16550a"; >> - reg = <0x1e787000 0x1000>; >> +

Re: [PATCHv3] mm: Account pud page tables

2017-10-03 Thread Vlastimil Babka
On 10/02/2017 10:04 AM, Kirill A. Shutemov wrote: > On machine with 5-level paging support a process can allocate > significant amount of memory and stay unnoticed by oom-killer and > memory cgroup. The trick is to allocate a lot of PUD page tables. > We don't account PUD page tables, only PMD and

[PATCH] PM / OPP: Add missing of_node_put(np)

2017-10-03 Thread Viresh Kumar
From: Tobias Jordan The for_each_available_child_of_node() loop in _of_add_opp_table_v2() doesn't drop the reference to "np" on errors. Fix that. Fixes: 274659029c9d ("PM / OPP: Add support to parse "operating-points-v2" bindings") Cc: 4.3+ # 4.3+ Signed-off-by: Tobias Jordan [ VK: Improved c

Re: [RFC 5/5] pm: remove kernel thread freezing

2017-10-03 Thread Hannes Reinecke
On 10/03/2017 08:53 PM, Luis R. Rodriguez wrote: > Now that all filesystems which used to rely on kthread > freezing have been converted to filesystem freeze/thawing > we can remove the kernel kthread freezer. > > Signed-off-by: Luis R. Rodriguez > --- > Documentation/power/freezing-of-tasks.txt

Re: usb/sound/bcd2000: warning in bcd2000_init_device

2017-10-03 Thread Takashi Iwai
On Tue, 03 Oct 2017 19:42:21 +0200, Greg Kroah-Hartman wrote: > > On Tue, Oct 03, 2017 at 12:50:08PM -0400, Alan Stern wrote: > > On Tue, 3 Oct 2017, Takashi Iwai wrote: > > > > > > It's a dev_WARN because it indicates a potentially serious error in the > > > > driver: The driver has submitted a

[RFC] perf callchain: Compare dsos (as well) for CCKEY_FUNCTION

2017-10-03 Thread Ravi Bangoria
Two functions from different binaries can have same start address. Thus, comparing only start address in match_chain() leads to inconsistent callchains. Fix this by adding a check for dsos as well. Ex, https://www.spinics.net/lists/linux-perf-users/msg04067.html Reported-by: Alexander Pozdneev S

Re: [PATCH RESEND net 0/9] Fixes, cleanup and modernization for some legacy ethernet NIC drivers

2017-10-03 Thread Finn Thain
On Tue, 3 Oct 2017, David Miller wrote: > From: Finn Thain > Date: Mon, 2 Oct 2017 21:07:17 -0400 (EDT) > > > This patch series fixes some logging bugs and adds some missing > > message severity levels. > > > > There are also cleanup patches for dead code and some Kconfig cruft. > > > > Cust

Re: [PATCH] fix security_release_secctx seems broken

2017-10-03 Thread James Morris
On Tue, 19 Sep 2017, Casey Schaufler wrote: > Subject: [PATCH] fix security_release_secctx seems broken > > security_inode_getsecurity() provides the text string value > of a security attribute. It does not provide a "secctx". > The code in xattr_getsecurity() that calls security_inode_getsecurit

Re: [BUGFIX PATCH] kprobes/x86: Remove IRQ disabling from jprobe handlers

2017-10-03 Thread Masami Hiramatsu
On Tue, 3 Oct 2017 19:11:00 +0200 Ingo Molnar wrote: > > * Masami Hiramatsu wrote: > > > On Tue, 3 Oct 2017 11:33:44 +0200 > > Ingo Molnar wrote: > > > > > > > > * Masami Hiramatsu wrote: > > > > > > > Jprobe actually doesn't need to disable IRQs while calling > > > > handlers, because Do

Re: [PATCH 3/4] staging: bcm2835-camera: pr_err() strings should end with newlines

2017-10-03 Thread Stefan Wahren
Am 03.10.2017 um 13:43 schrieb Arvind Yadav: > pr_err() messages should end with a new-line to avoid other messages > being concatenated. > > Signed-off-by: Arvind Yadav Acked-by: Stefan Wahren > --- > drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.c | 2 +- > 1 file changed, 1 insert

Re: [PATCH] block: drop "sending ioctl to a partition" message

2017-10-03 Thread Christoph Hellwig
On Tue, Oct 03, 2017 at 06:22:23PM +0200, Paolo Bonzini wrote: > On 21/09/2017 16:49, Paolo Bonzini wrote: > > After the first few months, the message has not led to many bug reports. > > It's been almost five years now, and in practice the main source of > > it seems to be MTIOCGET that someone is

Re: [PATCH v2] nvme-pci: Use PCI bus address for data/queues in CMB

2017-10-03 Thread Christoph Hellwig
On Mon, Oct 02, 2017 at 11:21:29AM -0600, Keith Busch wrote: > Yah, calling this a DMA address was a misnomer and confusing. Abhishek, can you test if this works for you?

Re: [Part2 Patch v4.2] crypto: ccp: Add Platform Security Processor (PSP) device support

2017-10-03 Thread P J P
+-- On Mon, 2 Oct 2017, Brijesh Singh wrote --+ | +int psp_dev_init(struct sp_device *sp) | +{ | + struct device *dev = sp->dev; | + struct psp_device *psp; | + int ret; | + | + ret = -ENOMEM; | + psp = psp_alloc_struct(sp); | + if (!psp) | + goto e_err; | + | +

[PATCH 6/7] phy: samsung: Remove dependency on obsolete SoC support

2017-10-03 Thread Marek Szyprowski
Support for Exynos4212 SoCs has been removed by commit bca9085e0ae9 ("ARM: dts: exynos: remove Exynos4212 support (dead code)"), so there is no need to keep remaining dead code related to this SoC version. Signed-off-by: Marek Szyprowski --- drivers/phy/samsung/Kconfig | 2 +- 1 file changed, 1

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Mika Westerberg
On Tue, Oct 03, 2017 at 08:54:30PM +0300, Andy Shevchenko wrote: > I'm not sure about this approach. It might be some other broken BIOS > discovered with some other driver (like pinctrl-baytrail.c as an > hypothetical example). Let's deal that separately if it ever happens. > Alternative is to de

Re: [PATCH V10 0/3] PM / Domains: Performance state support

2017-10-03 Thread Viresh Kumar
On 03-10-17, 09:52, Ulf Hansson wrote: > We sorted out things at LPC! > > However, the last weeks discussions at Linaro connect, raised a couple > of more concerns with the current approach. Let me summarize them > here. > > 1) > The ->dev_get_performance_state(), which currently translates > fre

Re: [PATCH] pinctrl: cherryview: fix issues caused by dynamic gpio irqs mapping

2017-10-03 Thread Mika Westerberg
On Tue, Oct 03, 2017 at 12:00:49PM -0500, Grygorii Strashko wrote: > New GPIO IRQs are allocated and mapped dynamically by default when > GPIO IRQ infrastructure is used by cherryview-pinctrl driver. > This causes issues on some Intel platforms [1][2] with broken BIOS which > hardcodes Linux IRQ nu

[PATCH v2 1/9] ARM: dts: aspeed: Move pinctrl subnodes to improve readability

2017-10-03 Thread Joel Stanley
From: Andrew Jeffery Moving the subnodes out of the pinctrl node declaration to a reference allows easier access to the remaining parts of the devicetree. Signed-off-by: Andrew Jeffery Reviewed-by: Xo Wang Acked-by: Andrew Jeffery Signed-off-by: Joel Stanley --- arch/arm/boot/dts/aspeed-g4.

[PATCH v2 3/9] ARM: dts: aspeed: Add I2C buses

2017-10-03 Thread Joel Stanley
Now with an upstream i2c bus driver, we can add the 14 i2c buses that exist in ASPEED G4 and G5 generation SoCs. It also adds aliases for the 14 built-in I2C busses to ensure userspace sees the numbering staring from zero and counting up. Acked-by: Andrew Jeffery Reviewed-by: Brendan Higgins S

Re: [alsa-devel] [PATCH RESEND1 00/12] ALSA: vsnd: Add Xen para-virtualized frontend driver

2017-10-03 Thread Oleksandr Andrushchenko
gentle reminder On 09/26/2017 02:35 PM, Oleksandr Andrushchenko wrote: Clemens, Sakamoto-san, could you please review the below if you by chance have a minute? Thank you, Oleksandr On 09/19/2017 11:57 AM, Oleksandr Andrushchenko wrote: Hi, all! We did some work on implementing the idea with

[PATCH v2 0/9] ARM: dts: aspeed: Device tree updates

2017-10-03 Thread Joel Stanley
This series adds support for the i2c buses and vuart in the ASPEED device trees, as well as making a few cleanups. v2 fixes issues pointed out by Andrew's reivew, and adds Brendan's acks. Thanks for the review both of you! There's one more patch than v1 to seperate out fixing the UARTs from chang

[PATCH v2 4/9] ARM: dts: aspeed-romulus: Add I2C devices

2017-10-03 Thread Joel Stanley
Enable the buses that are in use and the devices that are attached. Currently that is just the battery backed RTC. Some of these buses are for hotplugged cards, such as PCIe cards. Others do not yet have upstream drivers, so there are no devices attached. Reviewed-by: Brendan Higgins Signed-off-

[PATCH v2 2/9] ARM: dts: aspeed: Reorder ADC node

2017-10-03 Thread Joel Stanley
We try to keep the nodes in address order. The ADC node was out of place. Signed-off-by: Joel Stanley --- v2: - Don't introduce new clk bindings --- arch/arm/boot/dts/aspeed-g4.dtsi | 16 arch/arm/boot/dts/aspeed-g5.dtsi | 16 2 files changed, 16 insertions(+)

[PATCH v2 7/9] ARM: dts: aspeed: Add aliases for UARTs

2017-10-03 Thread Joel Stanley
Existing userspace expects the console (UART5) to be at /dev/ttyS4. To ensure the UARTs show up where users expect them, we give them fixed aliases starting at 0. Reviewed-by: Andrew Jeffery Signed-off-by: Joel Stanley --- arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts | 4 arch/arm/boot/d

[PATCH v2 8/9] ARM: dts: aspeed: Correctly order UART nodes

2017-10-03 Thread Joel Stanley
Order them all by address. Signed-off-by: Joel Stanley --- arch/arm/boot/dts/aspeed-g4.dtsi | 48 +++ arch/arm/boot/dts/aspeed-g5.dtsi | 61 2 files changed, 54 insertions(+), 55 deletions(-) diff --git a/arch/arm/boot/dts/asp

[PATCH v2 5/9] ARM: dts: aspeed-palmetto: Add I2C devices

2017-10-03 Thread Joel Stanley
Enable the buses that are in use and the devices that are attached. Currently that includes the battery backed RTC, temperature measurement and EEPROM. Some of these buses are for hotplugged cards, such as PCIe cards. Others do not yet have upstream drivers, so there are no devices attached. Revi

[PATCH v2 9/9] ARM: dts: aspeed: Clean up UART nodes

2017-10-03 Thread Joel Stanley
- Shorten size of reg property so it covers only the implemented registers - Add VUART compatible, and change node name to serial@ - Remove outdated current-speed property. Different bootloaders use different speeds, so this is no longer helpful Signed-off-by: Joel Stanley --- v2: - move

[PATCH] powerpc/perf: Fix for core/nest imc call trace on cpuhotplug

2017-10-03 Thread Anju T Sudhakar
Nest/core pmu units are enabled only when it is used. A reference count is maintained for the events which uses the nest/core pmu units. Currently in *_imc_counters_release function a WARN() is used for notification of any underflow of ref count.

Re: [PATCH v3 00/20] Speculative page faults

2017-10-03 Thread Laurent Dufour
On 25/09/2017 18:27, Alexei Starovoitov wrote: > On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour > wrote: >> Despite the unprovable lockdep warning raised by Sergey, I didn't get any >> feedback on this series. >> >> Is there a chance to get it moved upstream ? > > what is the status ? > We're e

[PATCH v2 6/9] ARM: dts: aspeed-ast2500: Add I2C devices

2017-10-03 Thread Joel Stanley
Enable the buses that are in use and the devices that are attached. Currently that includes temperature measurement and EEPROM. Reviewed-by: Brendan Higgins Reviewed-by: Andrew Jeffery Signed-off-by: Joel Stanley --- arch/arm/boot/dts/aspeed-ast2500-evb.dts | 19 +++ 1 file cha

Re: [PATCH v2 9/9] ARM: dts: aspeed: Clean up UART nodes

2017-10-03 Thread Joel Stanley
On Wed, Oct 4, 2017 at 4:19 PM, Joel Stanley wrote: > diff --git a/arch/arm/boot/dts/aspeed-g5.dtsi > b/arch/arm/boot/dts/aspeed-g5.dtsi > index 61cc2d25143a..cef51dcc1002 100644 > --- a/arch/arm/boot/dts/aspeed-g5.dtsi > +++ b/arch/arm/boot/dts/aspeed-g5.dtsi > @@ -297,9 +297,9 @@ >

Re: [Part2 PATCH v4.1 07/29] crypto: ccp: Add Secure Encrypted Virtualization (SEV) command support

2017-10-03 Thread P J P
+-- On Mon, 2 Oct 2017, Brijesh Singh wrote --+ | +static int sev_cmd_buffer_len(int cmd) | +{ | + switch (cmd) { | + case SEV_CMD_INIT: return sizeof(struct sev_data_init); | + case SEV_CMD_PLATFORM_STATUS: return sizeof(struct sev_data_status); | + case SEV_CMD_PEK_

Re: [PATCH 01/18] sound: use ARRAY_SIZE

2017-10-03 Thread Takashi Iwai
On Sun, 01 Oct 2017 21:30:39 +0200, Jérémy Lefaure wrote: > > Using the ARRAY_SIZE macro improves the readability of the code. > > Found with Coccinelle with the following semantic patch: > @r depends on (org || report)@ > type T; > T[] E; > position p; > @@ > ( > (sizeof(E)@p /sizeof(*E)) > | >

Re: [patch V2 22/29] lockup_detector: Make watchdog_nmi_reconfigure() two stage

2017-10-03 Thread Thomas Gleixner
On Tue, 3 Oct 2017, Thomas Gleixner wrote: > On Tue, 3 Oct 2017, Michael Ellerman wrote: > > Hi Thomas, > > Unfortunately this is hitting the WARN_ON in start_wd_cpu() on powerpc > > because we're calling it multiple times for the boot CPU. > > > > The first call is via: > > > > start_wd_on_cpu

Re: [PATCH v3] dma-debug: fix incorrect pfn calculation

2017-10-03 Thread Christoph Hellwig
On Mon, Oct 02, 2017 at 06:30:41PM +0800, Miles Chen wrote: > ARCHs like metag and xtensa define their mappings (non-vmalloc and > non-linear) for dma allocation. metag basically is a reimplementation of the vmalloc map mechanism that should be easy to consolidate into the common one :( xtensa ha

Re: [PATCH] mm,hugetlb,migration: don't migrate kernelcore hugepages

2017-10-03 Thread Michal Hocko
On Tue 03-10-17 07:42:25, Alexandru Moise wrote: > On Mon, Oct 02, 2017 at 06:15:00PM +0200, Michal Hocko wrote: [...] > > I really fail to see why kernel vs. movable zones play any role here. > > Zones should be mostly an implementation detail which userspace > > shouldn't really care about. > >

Re: regression in 4.14-rc2 caused by apparmor: add base infastructure for socket mediation

2017-10-03 Thread John Johansen
On 10/02/2017 11:48 PM, Vlastimil Babka wrote: > On 10/03/2017 07:15 AM, James Bottomley wrote: >> On Mon, 2017-10-02 at 21:11 -0700, John Johansen wrote: >>> On 10/02/2017 09:02 PM, James Bottomley wrote: The specific problem is that dnsmasq refuses to start on openSUSE Leap 42.2.

Re: [RFC] yamldt v0.5, now a DTS compiler too

2017-10-03 Thread Geert Uytterhoeven
Hi Pantelis, On Mon, Oct 2, 2017 at 9:46 PM, Pantelis Antoniou wrote: > # Note that the YAML example must be validated against this binding > # to be an accepted entry Indeed ;-) > yaml: | > sk11@0: > compatible: "skel,sk11" > reg: 0 > sip-max

[BUGFIX PATCH] kprobes/x86: Remove IRQ disabling from jprobe handlers

2017-10-03 Thread Masami Hiramatsu
Jprobe actually doesn't need to disable IRQs while calling handlers, because Documentation/kprobes.txt says: - Probe handlers are run with preemption disabled. Depending on the architecture and optimization state, handlers may also run with interrupts disabled (e.g., kretprobe handlers and

Re: [PATCH] of: do not leak console options

2017-10-03 Thread Petr Mladek
On Tue 2017-09-26 15:25:10, Sergey Senozhatsky wrote: > Do not strdup() console options. It seems that the only reason for > it to be strdup()-ed was a compilation warning: printk, UART and > console drivers, for some reason, expect char pointer instead of > const char pointer. So we can just pass

Re: [PATCH V7 3/6] block: pass flags to blk_queue_enter()

2017-10-03 Thread Ming Lei
On Mon, Oct 02, 2017 at 06:40:32AM -0700, Christoph Hellwig wrote: > On Sat, Sep 30, 2017 at 02:12:11PM +0800, Ming Lei wrote: > > We need to pass PREEMPT flags to blk_queue_enter() > > for allocating request with RQF_PREEMPT in the > > following patch. > > I don't like having another name space f

[RFC PATCH] mm, hugetlb: drop hugepages_treat_as_movable sysctl

2017-10-03 Thread Michal Hocko
From: Michal Hocko hugepages_treat_as_movable has been introduced by 396faf0303d2 ("Allow huge page allocations to use GFP_HIGH_MOVABLE") to allow hugetlb allocations from ZONE_MOVABLE even when hugetlb pages were not migrateable. The purpose of the movable zone was different at the time. It aime

Re: [RFC] yamldt v0.5, now a DTS compiler too

2017-10-03 Thread Pantelis Antoniou
Hi Geert, On Tue, 2017-10-03 at 09:17 +0200, Geert Uytterhoeven wrote: > Hi Pantelis, > > On Mon, Oct 2, 2017 at 9:46 PM, Pantelis Antoniou > wrote: > > # Note that the YAML example must be validated against this binding > > # to be an accepted entry > > Indeed ;-) > > > yaml: |

Re: usb/sound/line6: trying to register non-static key in podhd_disconnect

2017-10-03 Thread Takashi Iwai
On Mon, 25 Sep 2017 14:40:08 +0200, Andrey Konovalov wrote: > > Hi! > > I've got the following report while fuzzing the kernel with syzkaller. > > On commit e19b205be43d11bff638cad4487008c48d21c103 (4.14-rc2). > > INFO: trying to register non-static key. > the code is fine but needs lockdep ann

[PATCH 0/1] XDP Program for Ip forward

2017-10-03 Thread cjacob
The patch below implements port to port forwarding through route table and arp table lookup for ipv4 packets using bpf_redirect helper function and lpm_trie map. This has an improved performance over the normal kernel stack ip forward. Implementation details. --- The program

[PATCH 1/1] xdp: Sample xdp program implementing ip forward

2017-10-03 Thread cjacob
Implements port to port forwarding with route table and arp table lookup for ipv4 packets using bpf_redirect helper function and lpm_trie map. Signed-off-by: cjacob --- samples/bpf/Makefile|4 + samples/bpf/xdp3_kern.c | 204 +++ samples/bpf/xdp3_user.c | 649 +

Re: Patch "KVM: VMX: avoid double list add with VT-d posted interrupts" has been added to the 4.13-stable tree

2017-10-03 Thread Stefan Lippers-Hollmann
Hi On 2017-10-02, gre...@linuxfoundation.org wrote: > This is a note to let you know that I've just added the patch titled > > KVM: VMX: avoid double list add with VT-d posted interrupts > > to the 4.13-stable tree which can be found at: > > http://www.kernel.org/git/?p=linux/kernel/git

Re: [PATCH v3 1/3] crypto: engine - permit to enqueue aead_request

2017-10-03 Thread Fabien DESSENNE
On 22/09/17 11:09, Herbert Xu wrote: > On Fri, Aug 18, 2017 at 11:19:04AM +0200, Fabien Dessenne wrote: >> The current crypto engine allows ablkcipher_request and ahash_request to >> be enqueued. Extend this to aead_request. >> >> Signed-off-by: Fabien Dessenne > I'd like to see the crypto_engine

Re: usb/sound/bcd2000: warning in bcd2000_init_device

2017-10-03 Thread Takashi Iwai
On Mon, 25 Sep 2017 14:39:51 +0200, Andrey Konovalov wrote: > > Hi! > > I've got the following report while fuzzing the kernel with syzkaller. > > On commit e19b205be43d11bff638cad4487008c48d21c103 (4.14-rc2). > > It seems that there's no check of the endpoint type. > > usb 1-1: BOGUS urb xfer

Re: [PATCH V10 0/3] PM / Domains: Performance state support

2017-10-03 Thread Ulf Hansson
On 20 September 2017 at 00:32, Viresh Kumar wrote: > Hi Ulf, > > This version contains the changes we discussed during the LPC. > > Some platforms have the capability to configure the performance state of > their power domains. The process of configuring the performance state is > pretty much plat

Re: [RFC PATCH] mm, hugetlb: drop hugepages_treat_as_movable sysctl

2017-10-03 Thread Mel Gorman
On Tue, Oct 03, 2017 at 09:26:19AM +0200, Michal Hocko wrote: > From: Michal Hocko > > hugepages_treat_as_movable has been introduced by 396faf0303d2 ("Allow > huge page allocations to use GFP_HIGH_MOVABLE") to allow hugetlb > allocations from ZONE_MOVABLE even when hugetlb pages were not > migra

Fix a typo in Documentation/sysctl/vm.txt

2017-10-03 Thread Kangmin Park
--- linux/Documentation/sysctl/vm.txt.orig 2017-09-07 00:19:43.260477784 +0900 +++ linux/Documentation/sysctl/vm.txt 2017-09-07 00:19:56.498431846 +0900 @@ -793,7 +793,7 @@ with no ill effects: errors and warnings swappiness This control is used to define how aggressive the kernel will swap -mem

Re: [PATCH v3 00/20] Speculative page faults

2017-10-03 Thread Laurent Dufour
On 03/10/2017 03:27, Michael Ellerman wrote: > Laurent Dufour writes: > >> Hi Andrew, >> >> On 28/09/2017 22:38, Andrew Morton wrote: >>> On Thu, 28 Sep 2017 14:29:02 +0200 Laurent Dufour >>> wrote: >>> > Laurent's [0/n] provides some nice-looking performance benefits for > workloads wh

Re: [PATCH v2 0/5] Simplify panel bridge cleanup

2017-10-03 Thread Daniel Vetter
On Mon, Oct 02, 2017 at 11:32:30AM +0200, Benjamin Gaignard wrote: > The goal of this series is to simplify driver code when they need to clean up > a previously allocated panel bridge. > Few drivers have "is_panel_bridge" flag to be able to distinguish a > drm_panel_bridge from "simple" drm_bridge

Re: Patch "KVM: VMX: avoid double list add with VT-d posted interrupts" has been added to the 4.13-stable tree

2017-10-03 Thread Paolo Bonzini
On 03/10/2017 09:46, Stefan Lippers-Hollmann wrote: > Hi > > On 2017-10-02, gre...@linuxfoundation.org wrote: >> This is a note to let you know that I've just added the patch titled >> >> KVM: VMX: avoid double list add with VT-d posted interrupts >> >> to the 4.13-stable tree which can be fou

Re: [PATCH] PM: ARM: locomo: Drop suspend and resume bus type callbacks

2017-10-03 Thread Ulf Hansson
On 26 September 2017 at 22:45, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > None of the locomo drivers in the tree implements the suspend and > resume callbacks from struct locomo_driver, so drop them and drop > the corresponding callbacks from locomo_bus_type. > > Signed-off-by: Rafael

Re: [PATCH V2] PCI: AER: fix deadlock in do_recovery

2017-10-03 Thread Christoph Hellwig
On Mon, Oct 02, 2017 at 05:14:09PM -0700, Govindarajulu Varadarajan wrote: > > I would avoid increasing the size of pci_dev. The list_head would be empty > after > we call aer_pci_walk_bus(). We already have pci_bus *subordinate to link all > the > 'device's. Making list_head available to others

Re: [PATCH 05/18] net: use ARRAY_SIZE

2017-10-03 Thread Andy Shevchenko
On Tue, Oct 3, 2017 at 4:22 AM, Jérémy Lefaure wrote: > On Mon, 2 Oct 2017 16:07:36 +0300 > Andy Shevchenko wrote: > >> > + {&gainctrl_lut_core0_rev0, ARRAY_SIZE(gainctrl_lut_core0_rev0), >> > 26, 192, >> > +32}, >> >> For all such cases I would rather put on one line disregard che

Re: [PATCH V7 5/6] block: support PREEMPT_ONLY

2017-10-03 Thread Ming Lei
On Mon, Oct 02, 2017 at 04:27:59PM +, Bart Van Assche wrote: > On Sat, 2017-09-30 at 14:12 +0800, Ming Lei wrote: > > +void blk_set_preempt_only(struct request_queue *q, bool preempt_only) > > +{ > > + blk_mq_freeze_queue(q); > > + if (preempt_only) > > + queue_flag_set_unlocked(Q

Re: [PATCH V7 6/6] SCSI: set block queue at preempt only when SCSI device is put into quiesce

2017-10-03 Thread Ming Lei
On Mon, Oct 02, 2017 at 06:52:25AM -0700, Christoph Hellwig wrote: > > + /* > > +* Simply quiesing SCSI device isn't safe, it is easy > > +* to use up requests because all these allocated requests > > +* can't be dispatched when device is put in QIUESCE. > > +* Then no request can

Re: [PATCH V7 5/6] block: support PREEMPT_ONLY

2017-10-03 Thread Ming Lei
On Mon, Oct 02, 2017 at 06:50:24AM -0700, Christoph Hellwig wrote: > > +void blk_set_preempt_only(struct request_queue *q, bool preempt_only) > > +{ > > + blk_mq_freeze_queue(q); > > + if (preempt_only) > > + queue_flag_set_unlocked(QUEUE_FLAG_PREEMPT_ONLY, q); > > + else > > +

Re: [PATCH RFC V4 5/6] perf top: switch to backward overwrite mode

2017-10-03 Thread Jiri Olsa
On Mon, Oct 02, 2017 at 05:19:41PM +, Liang, Kan wrote: > > > > On Fri, Sep 29, 2017 at 07:47:56AM -0700, kan.li...@intel.com wrote: > > > From: Kan Liang > > > > > > perf_top__mmap_read has severe performance issue in Knights > > > Landing/Mill, when monitoring in heavy load system. It cost

Re: [PATCH 0/6] Boot-time switching between 4- and 5-level paging for 4.15, Part 1

2017-10-03 Thread Kirill A. Shutemov
On Fri, Sep 29, 2017 at 05:08:15PM +0300, Kirill A. Shutemov wrote: > The first bunch of patches that prepare kernel to boot-time switching > between paging modes. > > Please review and consider applying. Ping? -- Kirill A. Shutemov

[PATCH] drivers/fsi: make a couple of functions static

2017-10-03 Thread Colin King
From: Colin Ian King The functions fsi_slave_report_and_clear_errors and fsi_slave_handle_error are local to the source and do not need to be in global scope, so make them static. Cleans up sparse warnings: symbol 'fsi_slave_report_and_clear_errors' was not declared. Should it be static? symbol

Re: [PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init

2017-10-03 Thread Peter De Schrijver
Acked-By: Peter De Schrijver I would assume this also applies to many PLL structs for other Tegra SoCs? Peter. On Tue, Oct 03, 2017 at 01:42:08AM +0530, Bhumika Goyal wrote: > These structures are only passed to the functions tegra_clk_register_pll, > tegra_clk_register_pll{e/u} or tegra_periph

[PATCH v2 2/2] dmaengine: omap-dma: Implement protection for invalid max_burst

2017-10-03 Thread Peter Ujfalusi
Set the device's max_burst to 16777215 (EN is 24bit unsigned value) so clients can take this into consideration when setting up the transfer. During slave transfer preparation check if the requested maxburst is valid. Signed-off-by: Peter Ujfalusi Cc: Russell King --- drivers/dma/omap-dma.c |

[PATCH v2 1/2] dmaengine: edma: Implement protection for invalid max_burst

2017-10-03 Thread Peter Ujfalusi
Set the device's max_burst to 32767 (CIDX is 16bit signed value) so clients can take this into consideration when setting up the transfer. During slave transfer preparation check if the requested maxburst is valid. Signed-off-by: Peter Ujfalusi --- drivers/dma/edma.c | 5 + 1 file changed,

[PATCH v2 0/2] dmaengine: edma/omap-dma: max_burst validity checking

2017-10-03 Thread Peter Ujfalusi
Hi, Changes since v1: - Fixed typo in commit message - Separated from the 'dmaengine: core/edma/omap-dma: maximum SG len reporting' series Both eDMA and sDMA have limit on the maximum burst size. The two patch implements protection against too big burst size for both driver. Regards, Peter --

Re: [PATCH 1/3] scsi: ufs: Change HCI marco to actual bit position

2017-10-03 Thread Alim Akhtar
Hi Bart, On 10/02/2017 10:11 PM, Bart Van Assche wrote: > On Sat, 2017-09-30 at 11:52 +0530, Alim Akhtar wrote: >> Currently UFS HCI uses UFS_BIT() macro to get various bit >> position for the hardware registers status bits. Which makes >> code longer instead of shorter. This macro does not improv

Re: sysbench throughput degradation in 4.13+

2017-10-03 Thread Peter Zijlstra
On Mon, Oct 02, 2017 at 11:53:12PM +0100, Matt Fleming wrote: > On Wed, 27 Sep, at 01:58:20PM, Rik van Riel wrote: > > > > I like the simplicity of your approach! I hope it does not break > > stuff like netperf... > > > > I have been working on the patch below, which is much less optimistic > >

[PATCH] ARM: vexpress/spc: Delete an error message for a failed memory allocation in two functions

2017-10-03 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 3 Oct 2017 10:30:07 +0200 Omit extra messages for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- arch/arm/mach-vexpress/spc.c | 8 ++-- 1 file changed, 2 insertions

[PATCH] firmware: google: make structure gsmi_dev static

2017-10-03 Thread Colin King
From: Colin Ian King The structure gsmi_dev is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: symbol 'gsmi_dev' was not declared. Should it be static? Signed-off-by: Colin Ian King --- drivers/firmware/google/gsmi.c | 2 +- 1 file cha

Re: [PATCH] binder: fix memory corruption in binder_transaction binder

2017-10-03 Thread Amit Pundir
Hi, On 12 September 2017 at 13:50, Martijn Coenen wrote: > Hi Amit, > > Can you try with the patch I sent to LKML recently, "[PATCH v2 10/13] > ANDROID: binder: call poll_wait() unconditionally."? This fixes a > problem in binder's poll() implementation that only causes issues > under certain rac

Re: [BUG] fs/dcache: might_sleep is called under a spinlock

2017-10-03 Thread Jia-Ju Bai
Thanks for your detailed explanation :) I will improve my static analysis tool. Thanks, Jia-Ju Bai On 2017/10/3 11:19, Al Viro wrote: On Tue, Oct 03, 2017 at 10:38:25AM +0800, Jia-Ju Bai wrote: According to fs/dcache.c, might_sleep is called under a spinlock, and the function call path is: d_

RE: [PATCH] fsl/fman: remove of_node

2017-10-03 Thread Madalin-cristian Bucur
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Tuesday, October 03, 2017 2:05 AM > To: Madalin-cristian Bucur > Subject: Re: [PATCH] fsl/fman: remove of_node > > From: Madalin Bucur > Date: Mon, 2 Oct 2017 13:31:37 +0300 > > > The FMan MAC driver allocate

Re: [PATCH -v2 13/18] sched/fair: Propagate an effective runnable_load_avg

2017-10-03 Thread Peter Zijlstra
On Mon, Oct 02, 2017 at 06:46:32PM +0100, Dietmar Eggemann wrote: > > +/* > > + * Recomputes the group entity based on the current state of its group > > + * runqueue. > > + */ > > +static void update_cfs_group(struct sched_entity *se) > > update_cfs_share(s)() is still mentioned in the function

Re: [PATCH V5 1/7] blk-mq: issue rq directly in blk_mq_request_bypass_insert()

2017-10-03 Thread Christoph Hellwig
This patch does two many things at once and needs a split. I also don't really understand why it's in this series and not your dm-mpath performance one. > +static void blk_mq_request_direct_insert(struct blk_mq_hw_ctx *hctx, > + struct request *rq) > +{ > +

Re: [PATCH] extcon: Split out extcon header file for consumer and provider device

2017-10-03 Thread Charles Keepax
On Fri, Sep 29, 2017 at 09:01:45AM +0900, Chanwoo Choi wrote: > The extcon has two type of extcon devices as following. > - 'extcon provider deivce' adds new extcon device and detect the >state/properties of external connector. Also, it notifies the >state/properties to the extcon consumer

Re: [PATCH V5 4/7] blk-mq: introduce blk_mq_dequeue_from_ctx()

2017-10-03 Thread Christoph Hellwig
On Sat, Sep 30, 2017 at 06:27:17PM +0800, Ming Lei wrote: > This function is introduced for dequeuing request > from sw queue so that we can dispatch it in > scheduler's way. > > More importantly, some SCSI devices may set > q->queue_depth, which is a per-request_queue limit, > and applied on pend

Re: [PATCH v3 01/12] pinctrl: move gpio-axp209 to pinctrl

2017-10-03 Thread Chen-Yu Tsai
On Tue, Oct 3, 2017 at 4:18 AM, Maxime Ripard wrote: > On Mon, Oct 02, 2017 at 12:08:43PM +, Quentin Schulz wrote: >> To prepare the driver for the upcoming pinctrl features, move the GPIO >> driver AXP209 from GPIO to pinctrl subsystem. >> >> Signed-off-by: Quentin Schulz > > Acked-by: Maxim

Re: [PATCH v3 04/12] pinctrl: axp209: add programmable gpio_status_offset

2017-10-03 Thread Chen-Yu Tsai
On Tue, Oct 3, 2017 at 4:38 AM, Maxime Ripard wrote: > On Mon, Oct 02, 2017 at 12:08:46PM +, Quentin Schulz wrote: >> To prepare for patches that will add support for a new PMIC that has a >> different GPIO input status register, add a gpio_status_offset within >> axp20x_pctl structure and use

Re: [PATCH REBASED 0/6] rwsem: Implement down_read_killable()

2017-10-03 Thread Peter Zijlstra
On Fri, Sep 29, 2017 at 07:05:45PM +0300, Kirill Tkhai wrote: > The series introduces new down_read_killable() primitive, which > is similar to down_read(), but it may be interrupted by a signal. > The most touched is architectures code. Also, it marks a new user > of the primitive, which is iterat

Re: [PATCH 1/2] RAS/CEC: Use the right length for "cec_disable"

2017-10-03 Thread Borislav Petkov
On Mon, Oct 02, 2017 at 05:42:56PM +0200, Thomas Gleixner wrote: > On Mon, 2 Oct 2017, Borislav Petkov wrote: > > From: Nicolas Iooss > > > > parse_cec_param() compares a string with "cec_disable" using only 7 > > characters of the 11-character-long string. Fix the length. > > > > Signed-off-by:

Re: Modular BIG_KEYS (was: Re: [PATCH v4] security/keys: rewrite all of big_key crypto)

2017-10-03 Thread David Howells
Eric Biggers wrote: > (I also still need to convince myself that there aren't any race conditions > in key type unregistering. It's a little weird how it changes the key type > to the ".dead" key type, rather than pinning the key type in memory while > it's still used.) Keys are converted to th

Re: [PATCH v9 1/4] rockchip/rga: v4l2 m2m support

2017-10-03 Thread Jacob Chen
Hi Hans, 2017-09-22 20:02 GMT+08:00 Hans Verkuil : > Hi Jacob, > > Sorry for the delay, but here is my review. > > I noticed that there is no patch for the MAINTAINERS file, please add an entry > there for this driver. > > On 14/09/17 03:19, Jacob Chen wrote: >> Rockchip RGA is a separate 2D raste

Re: [PATCH V5 6/7] blk-mq-sched: improve dispatching from sw queue

2017-10-03 Thread Christoph Hellwig
On Sat, Sep 30, 2017 at 06:27:19PM +0800, Ming Lei wrote: > SCSI devices use host-wide tagset, and the shared > driver tag space is often quite big. Meantime > there is also queue depth for each lun(.cmd_per_lun), > which is often small. > > So lots of requests may stay in sw queue, and we > alway

<    1   2   3   4   5   6   7   8   9   10   >