Re: [PATCH 1/1] arm64: tegra: fix PPI interrupt flag

2017-04-03 Thread Marc Zyngier
On 03/04/17 13:55, Jon Hunter wrote: > Adding Marc ... > > On 03/04/17 12:19, Aniruddha Banerjee wrote: >> The interrupt flag for PPI should not be set to any value, since the >> register is read-only. Fix the flags for the PPI interrupts to >> IRQ_TYPE_NONE, so that there is no write to the

Re: [PATCH 1/1] arm64: tegra: fix PPI interrupt flag

2017-04-03 Thread Marc Zyngier
On 03/04/17 13:55, Jon Hunter wrote: > Adding Marc ... > > On 03/04/17 12:19, Aniruddha Banerjee wrote: >> The interrupt flag for PPI should not be set to any value, since the >> register is read-only. Fix the flags for the PPI interrupts to >> IRQ_TYPE_NONE, so that there is no write to the

Re: [PATCH] [net-next] stmmac: use netif_set_real_num_{rx,tx}_queues

2017-04-03 Thread Giuseppe CAVALLARO
Hello Joao On 3/30/2017 6:42 PM, Joao Pinto wrote: Às 5:35 PM de 3/30/2017, Niklas Cassel escreveu: On 03/30/2017 04:34 PM, Thierry Reding wrote: On Thu, Mar 30, 2017 at 09:45:36AM +0200, Corentin Labbe wrote: On Tue, Mar 28, 2017 at 06:01:05PM -0700, David Miller wrote: From: Arnd Bergmann

Re: [PATCH] [net-next] stmmac: use netif_set_real_num_{rx,tx}_queues

2017-04-03 Thread Giuseppe CAVALLARO
Hello Joao On 3/30/2017 6:42 PM, Joao Pinto wrote: Às 5:35 PM de 3/30/2017, Niklas Cassel escreveu: On 03/30/2017 04:34 PM, Thierry Reding wrote: On Thu, Mar 30, 2017 at 09:45:36AM +0200, Corentin Labbe wrote: On Tue, Mar 28, 2017 at 06:01:05PM -0700, David Miller wrote: From: Arnd Bergmann

Re: [PATCH v2 5/6] hpet: removing unused variable m in hpet_interrupt

2017-04-03 Thread Clemens Ladisch
Corentin Labbe wrote: > This patch fix the following warning: > drivers/char/hpet.c:146:17: attention : variable ‘m’ set but not used > [-Wunused-but-set-variable] > by removing the unused variable m in hpet_interrupt This patch might silence the warning, but it leaves the bug that actually

Re: [PATCH v2 5/6] hpet: removing unused variable m in hpet_interrupt

2017-04-03 Thread Clemens Ladisch
Corentin Labbe wrote: > This patch fix the following warning: > drivers/char/hpet.c:146:17: attention : variable ‘m’ set but not used > [-Wunused-but-set-variable] > by removing the unused variable m in hpet_interrupt This patch might silence the warning, but it leaves the bug that actually

Re: [PATCH 1/2] wireless: ath9k_htc: fix NULL-deref at probe

2017-04-03 Thread Kalle Valo
Kalle Valo writes: > Johan Hovold writes: > >> On Mon, Mar 13, 2017 at 01:44:20PM +0100, Johan Hovold wrote: >>> Make sure to check the number of endpoints to avoid dereferencing a >>> NULL-pointer or accessing memory beyond the endpoint array should a

Re: [PATCH 1/2] wireless: ath9k_htc: fix NULL-deref at probe

2017-04-03 Thread Kalle Valo
Kalle Valo writes: > Johan Hovold writes: > >> On Mon, Mar 13, 2017 at 01:44:20PM +0100, Johan Hovold wrote: >>> Make sure to check the number of endpoints to avoid dereferencing a >>> NULL-pointer or accessing memory beyond the endpoint array should a >>> malicious device lack the expected

Re: [PATCH v2 0/5] minitty: a minimal TTY layer alternative for embedded systems

2017-04-03 Thread Alan Cox
On Sun, 2017-04-02 at 11:55 -0400, Nicolas Pitre wrote: > On Sun, 2 Apr 2017, Andy Shevchenko wrote: > > > > > +Cc people, who have a key roles in all TTY stuff (btw, why you did > > miss them? > > I used what MAINTAINERS and get_maintainer.pl gave me. > I didn't see this until now as I'm mid

Re: [PATCH v2 0/5] minitty: a minimal TTY layer alternative for embedded systems

2017-04-03 Thread Alan Cox
On Sun, 2017-04-02 at 11:55 -0400, Nicolas Pitre wrote: > On Sun, 2 Apr 2017, Andy Shevchenko wrote: > > > > > +Cc people, who have a key roles in all TTY stuff (btw, why you did > > miss them? > > I used what MAINTAINERS and get_maintainer.pl gave me. > I didn't see this until now as I'm mid

Re: [PATCH 1/1] arm64: tegra: fix PPI interrupt flag

2017-04-03 Thread Jon Hunter
Adding Marc ... On 03/04/17 12:19, Aniruddha Banerjee wrote: > The interrupt flag for PPI should not be set to any value, since the > register is read-only. Fix the flags for the PPI interrupts to > IRQ_TYPE_NONE, so that there is no write to the read-only register. If the below matches the h/w

Re: [PATCH 1/1] arm64: tegra: fix PPI interrupt flag

2017-04-03 Thread Jon Hunter
Adding Marc ... On 03/04/17 12:19, Aniruddha Banerjee wrote: > The interrupt flag for PPI should not be set to any value, since the > register is read-only. Fix the flags for the PPI interrupts to > IRQ_TYPE_NONE, so that there is no write to the read-only register. If the below matches the h/w

Re: dwc2_hc_chhltd_intr_dma - ChHltd set errors?

2017-04-03 Thread Minas Harutyunyan
Hi, On 4/3/2017 9:23 AM, John Youn wrote: > On 03/31/2017 04:04 PM, John Stultz wrote: >> On Thu, Mar 2, 2017 at 12:00 PM, John Stultz wrote: >>> Hey John, >>> We've noticed that when using usb ethernet adapters on HiKey, we >>> occasionally see errors like: >>> >>>

Re: dwc2_hc_chhltd_intr_dma - ChHltd set errors?

2017-04-03 Thread Minas Harutyunyan
Hi, On 4/3/2017 9:23 AM, John Youn wrote: > On 03/31/2017 04:04 PM, John Stultz wrote: >> On Thu, Mar 2, 2017 at 12:00 PM, John Stultz wrote: >>> Hey John, >>> We've noticed that when using usb ethernet adapters on HiKey, we >>> occasionally see errors like: >>> >>> dwc2 f72c.usb:

Re: [Xen-devel] [PATCH v2] xen, kdump: handle pv domain in paddr_vmcoreinfo_note()

2017-04-03 Thread Daniel Kiper
On Fri, Mar 31, 2017 at 12:14:38PM +0200, Juergen Gross wrote: > For kdump to work correctly it needs the physical address of > vmcoreinfo_note. When running as dom0 this means the virtual address > has to be translated to the related machine address. > > paddr_vmcoreinfo_note() is meant to do the

Re: [Xen-devel] [PATCH v2] xen, kdump: handle pv domain in paddr_vmcoreinfo_note()

2017-04-03 Thread Daniel Kiper
On Fri, Mar 31, 2017 at 12:14:38PM +0200, Juergen Gross wrote: > For kdump to work correctly it needs the physical address of > vmcoreinfo_note. When running as dom0 this means the virtual address > has to be translated to the related machine address. > > paddr_vmcoreinfo_note() is meant to do the

[PATCH 2/3 v2] pinctrl: Add pincontrol driver for ARTPEC-6 SoC

2017-04-03 Thread Jesper Nilsson
Add pinctrl driver support for the Axis ARTPEC-6 SoC. There are only some pins that actually have different functions available, but all can control bias (pull-up/-down) and drive strength. Code originally written by Chris Paterson. Signed-off-by: Jesper Nilsson ---

[PATCH 2/3 v2] pinctrl: Add pincontrol driver for ARTPEC-6 SoC

2017-04-03 Thread Jesper Nilsson
Add pinctrl driver support for the Axis ARTPEC-6 SoC. There are only some pins that actually have different functions available, but all can control bias (pull-up/-down) and drive strength. Code originally written by Chris Paterson. Signed-off-by: Jesper Nilsson --- Changes since v1: - Get rid

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Michal Hocko
On Mon 03-04-17 15:37:07, Andrey Ryabinin wrote: > > > On 04/03/2017 11:47 AM, Michal Hocko wrote: > > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: > >> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin > >> wrote: > >>> zswap_frontswap_store() is called during memory

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Michal Hocko
On Mon 03-04-17 15:37:07, Andrey Ryabinin wrote: > > > On 04/03/2017 11:47 AM, Michal Hocko wrote: > > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: > >> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin > >> wrote: > >>> zswap_frontswap_store() is called during memory reclaim from > >>>

[PATCH 2/3] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

2017-04-03 Thread Mikko Perttunen
The Tegra186 CCPLEX_CLUSTER area contains memory-mapped registers that initiate CPU frequency/voltage transitions. Signed-off-by: Mikko Perttunen --- .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt | 22 ++ 1 file changed, 22 insertions(+) create

[PATCH 2/3] dt-bindings: Add bindings for nvidia,tegra186-ccplex-cluster

2017-04-03 Thread Mikko Perttunen
The Tegra186 CCPLEX_CLUSTER area contains memory-mapped registers that initiate CPU frequency/voltage transitions. Signed-off-by: Mikko Perttunen --- .../arm/tegra/nvidia,tegra186-ccplex-cluster.txt | 22 ++ 1 file changed, 22 insertions(+) create mode 100644

[PATCH 3/3] arm64: tegra: Add CCPLEX_CLUSTER area in Tegra186

2017-04-03 Thread Mikko Perttunen
The Tegra186 CCPLEX_CLUSTER area contains memory-mapped registers that initiate CPU frequency/voltage transitions. Signed-off-by: Mikko Perttunen --- arch/arm64/boot/dts/nvidia/tegra186.dtsi | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH 3/3] arm64: tegra: Add CCPLEX_CLUSTER area in Tegra186

2017-04-03 Thread Mikko Perttunen
The Tegra186 CCPLEX_CLUSTER area contains memory-mapped registers that initiate CPU frequency/voltage transitions. Signed-off-by: Mikko Perttunen --- arch/arm64/boot/dts/nvidia/tegra186.dtsi | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/arm64/boot/dts/nvidia/tegra186.dtsi

[PATCH 1/3] cpufreq: Add Tegra186 cpufreq driver

2017-04-03 Thread Mikko Perttunen
Add a new cpufreq driver for Tegra186 (and likely later). The CPUs are organized into two clusters, Denver and A57, with two and four cores respectively. CPU frequency can be adjusted by writing the desired rate divisor and a voltage hint to a special per-core register. The frequency of each core

[PATCH 1/3] cpufreq: Add Tegra186 cpufreq driver

2017-04-03 Thread Mikko Perttunen
Add a new cpufreq driver for Tegra186 (and likely later). The CPUs are organized into two clusters, Denver and A57, with two and four cores respectively. CPU frequency can be adjusted by writing the desired rate divisor and a voltage hint to a special per-core register. The frequency of each core

Re: [RFC][PATCHv2 1/8] printk: move printk_pending out of per-cpu

2017-04-03 Thread Petr Mladek
On Mon 2017-04-03 20:23:01, Sergey Senozhatsky wrote: > On (03/31/17 15:33), Peter Zijlstra wrote: > > On Fri, Mar 31, 2017 at 03:09:50PM +0200, Petr Mladek wrote: > > > On Wed 2017-03-29 18:25:04, Sergey Senozhatsky wrote: > > > > > > if (waitqueue_active(_wait)) { > > > > -

Re: [RFC][PATCHv2 1/8] printk: move printk_pending out of per-cpu

2017-04-03 Thread Petr Mladek
On Mon 2017-04-03 20:23:01, Sergey Senozhatsky wrote: > On (03/31/17 15:33), Peter Zijlstra wrote: > > On Fri, Mar 31, 2017 at 03:09:50PM +0200, Petr Mladek wrote: > > > On Wed 2017-03-29 18:25:04, Sergey Senozhatsky wrote: > > > > > > if (waitqueue_active(_wait)) { > > > > -

Re: [PATCH 08/11] dt-bindings: imx-ocotp: add compatible string for i.MX7D/S

2017-04-03 Thread Rob Herring
On Fri, Mar 31, 2017 at 7:44 AM, Srinivas Kandagatla wrote: > From: Peng Fan > > Add compatible string for i.MX7D/S > > Signed-off-by: Peng Fan > Cc: Shawn Guo > Cc: Srinivas Kandagatla

Re: [PATCH 08/11] dt-bindings: imx-ocotp: add compatible string for i.MX7D/S

2017-04-03 Thread Rob Herring
On Fri, Mar 31, 2017 at 7:44 AM, Srinivas Kandagatla wrote: > From: Peng Fan > > Add compatible string for i.MX7D/S > > Signed-off-by: Peng Fan > Cc: Shawn Guo > Cc: Srinivas Kandagatla > Cc: Maxime Ripard > Cc: Rob Herring > Cc: Mark Rutland > Signed-off-by: Srinivas Kandagatla > --- >

Re: [PATCH v3 01/20] net: stmmac: export stmmac_set_mac_addr/stmmac_get_mac_addr

2017-04-03 Thread Giuseppe CAVALLARO
Hello Alex do you can check if this has to be done for ST platforms? I do not remember that it was necessary when build as module so I cannot expect this should be only for dwmac-sun8i. Regards peppe On 4/3/2017 11:14 AM, Corentin Labbe wrote: Thoses symbol will be needed for the

Re: [PATCH v3 01/20] net: stmmac: export stmmac_set_mac_addr/stmmac_get_mac_addr

2017-04-03 Thread Giuseppe CAVALLARO
Hello Alex do you can check if this has to be done for ST platforms? I do not remember that it was necessary when build as module so I cannot expect this should be only for dwmac-sun8i. Regards peppe On 4/3/2017 11:14 AM, Corentin Labbe wrote: Thoses symbol will be needed for the

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Andrey Ryabinin
On 04/03/2017 03:37 PM, Andrey Ryabinin wrote: > > > On 04/03/2017 11:47 AM, Michal Hocko wrote: >> On Fri 31-03-17 10:00:30, Shakeel Butt wrote: >>> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin >>> wrote: zswap_frontswap_store() is called during memory

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Andrey Ryabinin
On 04/03/2017 03:37 PM, Andrey Ryabinin wrote: > > > On 04/03/2017 11:47 AM, Michal Hocko wrote: >> On Fri 31-03-17 10:00:30, Shakeel Butt wrote: >>> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin >>> wrote: zswap_frontswap_store() is called during memory reclaim from

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Andrey Ryabinin
On 04/03/2017 11:47 AM, Michal Hocko wrote: > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: >> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin >> wrote: >>> zswap_frontswap_store() is called during memory reclaim from >>> __frontswap_store() from swap_writepage() from

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Andrey Ryabinin
On 04/03/2017 11:47 AM, Michal Hocko wrote: > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: >> On Fri, Mar 31, 2017 at 8:30 AM, Andrey Ryabinin >> wrote: >>> zswap_frontswap_store() is called during memory reclaim from >>> __frontswap_store() from swap_writepage() from shrink_page_list(). >>>

Re: [PATCH v3 02/20] net: stmmac: add optional setup function

2017-04-03 Thread Giuseppe CAVALLARO
Hello Corentin On 4/3/2017 11:14 AM, Corentin Labbe wrote: Instead of adding more ifthen logic for adding a new mac_device_info setup function, it is easier to add a function pointer to the function needed. Signed-off-by: Corentin Labbe ---

Re: [PATCH v3 02/20] net: stmmac: add optional setup function

2017-04-03 Thread Giuseppe CAVALLARO
Hello Corentin On 4/3/2017 11:14 AM, Corentin Labbe wrote: Instead of adding more ifthen logic for adding a new mac_device_info setup function, it is easier to add a function pointer to the function needed. Signed-off-by: Corentin Labbe --- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

Re: [PATCH] KVM: mmu: Fix overlap with private memslots

2017-04-03 Thread Pankaj Gupta
> On 27.03.2017 08:23, Wanpeng Li wrote: > > From: Wanpeng Li > > > > Reported by syzkaller: > > > > pte_list_remove: 9714eb1f8078 0->BUG > > [ cut here ] > > kernel BUG at arch/x86/kvm/mmu.c:1157! > > invalid opcode:

Re: [PATCH] KVM: mmu: Fix overlap with private memslots

2017-04-03 Thread Pankaj Gupta
> On 27.03.2017 08:23, Wanpeng Li wrote: > > From: Wanpeng Li > > > > Reported by syzkaller: > > > > pte_list_remove: 9714eb1f8078 0->BUG > > [ cut here ] > > kernel BUG at arch/x86/kvm/mmu.c:1157! > > invalid opcode: [#1] SMP > > RIP:

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Michal Hocko
On Mon 03-04-17 14:57:11, Andrey Ryabinin wrote: > On 04/03/2017 11:47 AM, Michal Hocko wrote: > > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: [...] > >>> @@ -1017,9 +1018,7 @@ static int zswap_frontswap_store(unsigned type, > >>> pgoff_t offset, > >>> > >>> /* store */ > >>>

Re: [PATCH] mm/zswap: fix potential deadlock in zswap_frontswap_store()

2017-04-03 Thread Michal Hocko
On Mon 03-04-17 14:57:11, Andrey Ryabinin wrote: > On 04/03/2017 11:47 AM, Michal Hocko wrote: > > On Fri 31-03-17 10:00:30, Shakeel Butt wrote: [...] > >>> @@ -1017,9 +1018,7 @@ static int zswap_frontswap_store(unsigned type, > >>> pgoff_t offset, > >>> > >>> /* store */ > >>>

[PATCH v1] reset: Make optional stuff optional for all users

2017-04-03 Thread Andy Shevchenko
There is Device Tree oriented check for optional resource. Of course it will fail on non-DT platforms. Remove this check to make things optional for all users. Fixes: bb475230b8e5 ("reset: make optional functions really optional") Cc: Ramiro Oliveira Signed-off-by:

[PATCH v1] reset: Make optional stuff optional for all users

2017-04-03 Thread Andy Shevchenko
There is Device Tree oriented check for optional resource. Of course it will fail on non-DT platforms. Remove this check to make things optional for all users. Fixes: bb475230b8e5 ("reset: make optional functions really optional") Cc: Ramiro Oliveira Signed-off-by: Andy Shevchenko --- The

Re: [tip:x86/asm] debug: Fix __bug_table[] in arch linker scripts

2017-04-03 Thread Peter Zijlstra
On Mon, Apr 03, 2017 at 02:57:48AM -0700, h...@zytor.com wrote: > On April 3, 2017 1:27:49 AM PDT, tip-bot for Peter Zijlstra > wrote: > >diff --git a/arch/x86/kernel/vmlinux.lds.S > >b/arch/x86/kernel/vmlinux.lds.S > >index c74ae9c..c8a3b61 100644 > >---

Re: [tip:x86/asm] debug: Fix __bug_table[] in arch linker scripts

2017-04-03 Thread Peter Zijlstra
On Mon, Apr 03, 2017 at 02:57:48AM -0700, h...@zytor.com wrote: > On April 3, 2017 1:27:49 AM PDT, tip-bot for Peter Zijlstra > wrote: > >diff --git a/arch/x86/kernel/vmlinux.lds.S > >b/arch/x86/kernel/vmlinux.lds.S > >index c74ae9c..c8a3b61 100644 > >--- a/arch/x86/kernel/vmlinux.lds.S > >+++

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Eric Dumazet
On Mon, 2017-04-03 at 15:14 +0300, Denys Fedoryshchenko wrote: > On 2017-04-03 15:09, Eric Dumazet wrote: > > On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: > > > >> I modified patch a little as: > >> if (th->doff * 4 < sizeof(_tcph)) { > >> par->hotdrop = true; > >>

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Eric Dumazet
On Mon, 2017-04-03 at 15:14 +0300, Denys Fedoryshchenko wrote: > On 2017-04-03 15:09, Eric Dumazet wrote: > > On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: > > > >> I modified patch a little as: > >> if (th->doff * 4 < sizeof(_tcph)) { > >> par->hotdrop = true; > >>

[PATCH v3 2/3] usb: dwc3: make role-switching work with debugfs/mode

2017-04-03 Thread Roger Quadros
If dr_mode == "otg", we start by default in PERIPHERAL mode. Keep track of current role in "current_dr_role" whenever dwc3_set_mode() is called. When debugfs/mode is changed AND we're in dual-role mode, handle the switch by stopping and starting the respective host/gadget controllers.

[PATCH v3 2/3] usb: dwc3: make role-switching work with debugfs/mode

2017-04-03 Thread Roger Quadros
If dr_mode == "otg", we start by default in PERIPHERAL mode. Keep track of current role in "current_dr_role" whenever dwc3_set_mode() is called. When debugfs/mode is changed AND we're in dual-role mode, handle the switch by stopping and starting the respective host/gadget controllers.

[PATCH v3 0/3] usb: dwc3: dual-role support

2017-04-03 Thread Roger Quadros
Hi, Taking a simplistic approach this time. We don't use the OTG controller block at all. Instead we just rely on ID events via extcon framework. I tried to get rid of workqueue but unfortunately it causes too much pain when the UDC is unregistered. https://hastebin.com/upugaqogol.xml

[PATCH v3 0/3] usb: dwc3: dual-role support

2017-04-03 Thread Roger Quadros
Hi, Taking a simplistic approach this time. We don't use the OTG controller block at all. Instead we just rely on ID events via extcon framework. I tried to get rid of workqueue but unfortunately it causes too much pain when the UDC is unregistered. https://hastebin.com/upugaqogol.xml

[PATCH v3 3/3] usb: dwc3: Add dual-role support

2017-04-03 Thread Roger Quadros
If dr_mode is "otg" then support dual role mode of operation. Currently this mode is only supported when an extcon handle is present in the dwc3 device tree node. This is needed to get the ID status events of the port. We're using a workqueue to manage the dual-role state transitions as the

[PATCH v3 1/3] usb: udc: allow adding and removing the same gadget device

2017-04-03 Thread Roger Quadros
allow usb_del_gadget_udc() and usb add_gadget_udc() to be called repeatedly on the same gadget->dev structure. We need to clear the gadget->dev structure so that kobject_init() doesn't complain about already initialized object. Signed-off-by: Roger Quadros ---

[PATCH v3 3/3] usb: dwc3: Add dual-role support

2017-04-03 Thread Roger Quadros
If dr_mode is "otg" then support dual role mode of operation. Currently this mode is only supported when an extcon handle is present in the dwc3 device tree node. This is needed to get the ID status events of the port. We're using a workqueue to manage the dual-role state transitions as the

[PATCH v3 1/3] usb: udc: allow adding and removing the same gadget device

2017-04-03 Thread Roger Quadros
allow usb_del_gadget_udc() and usb add_gadget_udc() to be called repeatedly on the same gadget->dev structure. We need to clear the gadget->dev structure so that kobject_init() doesn't complain about already initialized object. Signed-off-by: Roger Quadros --- drivers/usb/gadget/udc/core.c | 1

Re: [PATCH 0/6] mm: make movable onlining suck less

2017-04-03 Thread Igor Mammedov
On Mon, 3 Apr 2017 13:55:46 +0200 Michal Hocko wrote: > On Thu 30-03-17 13:54:48, Michal Hocko wrote: > [...] > > Any thoughts, complains, suggestions? > > Anyting? I would really appreciate a feedback from IBM and Futjitsu guys > who have shaped this code last few years.

Re: [PATCH 0/6] mm: make movable onlining suck less

2017-04-03 Thread Igor Mammedov
On Mon, 3 Apr 2017 13:55:46 +0200 Michal Hocko wrote: > On Thu 30-03-17 13:54:48, Michal Hocko wrote: > [...] > > Any thoughts, complains, suggestions? > > Anyting? I would really appreciate a feedback from IBM and Futjitsu guys > who have shaped this code last few years. Also Igor and Vitaly

[PATCH v2 2/6] hpet: remove unused writeq/readq function definitions

2017-04-03 Thread Corentin Labbe
On all arch using hpet, only i386 miss writeq/readq. Instead of rewriting them, use linux/io-64-nonatomic-lo-hi.h which already have them. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-)

[PATCH v2 0/6] hpet: fix build warnings and style

2017-04-03 Thread Corentin Labbe
The original intent was to remove two build warnings, but finaly took the opportunity to fix some style issues. compile-tested for both x86/x86_64/ia64. Changes since v1: - add linux/io-64-nonatomic-lo-hi.h for writeq/readq as suggested by Clemens Ladisch Corentin Labbe (6): hpet: remove

[PATCH v2 2/6] hpet: remove unused writeq/readq function definitions

2017-04-03 Thread Corentin Labbe
On all arch using hpet, only i386 miss writeq/readq. Instead of rewriting them, use linux/io-64-nonatomic-lo-hi.h which already have them. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-) diff --git

[PATCH v2 0/6] hpet: fix build warnings and style

2017-04-03 Thread Corentin Labbe
The original intent was to remove two build warnings, but finaly took the opportunity to fix some style issues. compile-tested for both x86/x86_64/ia64. Changes since v1: - add linux/io-64-nonatomic-lo-hi.h for writeq/readq as suggested by Clemens Ladisch Corentin Labbe (6): hpet: remove

[PATCH v2 3/6] hpet: fix checkpatch complains about spaces

2017-04-03 Thread Corentin Labbe
This patch make checkpatch happy for spaces. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index 206d3a6..7a35739 100644 ---

[PATCH v2 3/6] hpet: fix checkpatch complains about spaces

2017-04-03 Thread Corentin Labbe
This patch make checkpatch happy for spaces. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index 206d3a6..7a35739 100644 --- a/drivers/char/hpet.c +++

[PATCH v2 4/6] hpet: replace printk by their pr_xxx counterparts

2017-04-03 Thread Corentin Labbe
This patch replace all printk by their pr_xxx counterparts. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index 7a35739..59e89e5

[PATCH v2 4/6] hpet: replace printk by their pr_xxx counterparts

2017-04-03 Thread Corentin Labbe
This patch replace all printk by their pr_xxx counterparts. Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index 7a35739..59e89e5 100644 --- a/drivers/char/hpet.c

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 14:12 +0200, Hans Verkuil wrote: > On 04/03/2017 02:12 PM, Smitha T Murthy wrote: > > On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: > >> On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > >>> Add V4L2 definition for HEVC compressed format > >>> > >>> Signed-off-by:

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 14:12 +0200, Hans Verkuil wrote: > On 04/03/2017 02:12 PM, Smitha T Murthy wrote: > > On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: > >> On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > >>> Add V4L2 definition for HEVC compressed format > >>> > >>> Signed-off-by:

[PATCH v2 5/6] hpet: removing unused variable m in hpet_interrupt

2017-04-03 Thread Corentin Labbe
This patch fix the following warning: drivers/char/hpet.c:146:17: attention : variable ‘m’ set but not used [-Wunused-but-set-variable] by removing the unused variable m in hpet_interrupt Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 3 +-- 1 file changed,

[PATCH v2 5/6] hpet: removing unused variable m in hpet_interrupt

2017-04-03 Thread Corentin Labbe
This patch fix the following warning: drivers/char/hpet.c:146:17: attention : variable ‘m’ set but not used [-Wunused-but-set-variable] by removing the unused variable m in hpet_interrupt Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH v2 6/6] hpet: fix style issue about braces and alignment

2017-04-03 Thread Corentin Labbe
This patch fix all style issue for braces and alignment Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index f6096e1..fe52f39 100644 ---

[PATCH v2 1/6] hpet: remove unused variable hpet in hpet_ioctl_common

2017-04-03 Thread Corentin Labbe
This patch fix the following warning: drivers/char/hpet.c:582:23: attention : variable ‘hpet’ set but not used [-Wunused-but-set-variable] by removing the unused variable hpet in hpet_ioctl_common Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 2 -- 1 file

[PATCH v2 6/6] hpet: fix style issue about braces and alignment

2017-04-03 Thread Corentin Labbe
This patch fix all style issue for braces and alignment Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c index f6096e1..fe52f39 100644 --- a/drivers/char/hpet.c +++

[PATCH v2 1/6] hpet: remove unused variable hpet in hpet_ioctl_common

2017-04-03 Thread Corentin Labbe
This patch fix the following warning: drivers/char/hpet.c:582:23: attention : variable ‘hpet’ set but not used [-Wunused-but-set-variable] by removing the unused variable hpet in hpet_ioctl_common Signed-off-by: Corentin Labbe --- drivers/char/hpet.c | 2 -- 1 file changed, 2 deletions(-)

Re: [PATCH v2] scsi: storvsc: Add support for FC rport.

2017-04-03 Thread Christoph Hellwig
> if (host->transportt == fc_transport_template) { > + struct fc_rport_identifiers ids = { > + .roles = FC_PORT_ROLE_FCP_TARGET, > + }; I don't think storvsc ever acts as FCP target.

Re: [PATCH v2] scsi: storvsc: Add support for FC rport.

2017-04-03 Thread Christoph Hellwig
> if (host->transportt == fc_transport_template) { > + struct fc_rport_identifiers ids = { > + .roles = FC_PORT_ROLE_FCP_TARGET, > + }; I don't think storvsc ever acts as FCP target.

Re: [Patch v3 10/11] [media] s5p-mfc: Add support for HEVC encoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:59 +0200, Andrzej Hajda wrote: > On 31.03.2017 11:06, Smitha T Murthy wrote: > > Add HEVC encoder support and necessary registers, V4L2 CIDs, > > and hevc encoder parameters > > > > Signed-off-by: Smitha T Murthy > > --- > >

Re: [Patch v3 10/11] [media] s5p-mfc: Add support for HEVC encoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:59 +0200, Andrzej Hajda wrote: > On 31.03.2017 11:06, Smitha T Murthy wrote: > > Add HEVC encoder support and necessary registers, V4L2 CIDs, > > and hevc encoder parameters > > > > Signed-off-by: Smitha T Murthy > > --- > > drivers/media/platform/s5p-mfc/regs-mfc-v10.h

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Denys Fedoryshchenko
On 2017-04-03 15:09, Eric Dumazet wrote: On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: I modified patch a little as: if (th->doff * 4 < sizeof(_tcph)) { par->hotdrop = true; WARN_ON_ONCE(!tcpinfo->option); return false; } And it did triggered WARN once at morning, and

Re: [PATCH RFC] remove custom Michael MIC implementation

2017-04-03 Thread Greg Kroah-Hartman
On Mon, Apr 03, 2017 at 07:03:59PM +1000, Tobin C. Harding wrote: > On Mon, Apr 03, 2017 at 08:19:40AM +0300, Kalle Valo wrote: > > + linux-wireless > > > > "Tobin C. Harding" writes: > > > > > On Fri, Mar 31, 2017 at 09:58:51AM +0200, Wolfram Sang wrote: > > >> > > >> > The

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Denys Fedoryshchenko
On 2017-04-03 15:09, Eric Dumazet wrote: On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: I modified patch a little as: if (th->doff * 4 < sizeof(_tcph)) { par->hotdrop = true; WARN_ON_ONCE(!tcpinfo->option); return false; } And it did triggered WARN once at morning, and

Re: [PATCH RFC] remove custom Michael MIC implementation

2017-04-03 Thread Greg Kroah-Hartman
On Mon, Apr 03, 2017 at 07:03:59PM +1000, Tobin C. Harding wrote: > On Mon, Apr 03, 2017 at 08:19:40AM +0300, Kalle Valo wrote: > > + linux-wireless > > > > "Tobin C. Harding" writes: > > > > > On Fri, Mar 31, 2017 at 09:58:51AM +0200, Wolfram Sang wrote: > > >> > > >> > The code is untested,

Re: [Patch v3 09/11] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:22 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add v4l2 controls for HEVC encoder > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > >

Re: [Patch v3 09/11] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:22 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add v4l2 controls for HEVC encoder > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > drivers/media/v4l2-core/v4l2-ctrls.c | 103

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Hans Verkuil
On 04/03/2017 02:12 PM, Smitha T Murthy wrote: > On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: >> On 03/31/2017 11:06 AM, Smitha T Murthy wrote: >>> Add V4L2 definition for HEVC compressed format >>> >>> Signed-off-by: Smitha T Murthy >>> Reviewed-by: Andrzej Hajda

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Hans Verkuil
On 04/03/2017 02:12 PM, Smitha T Murthy wrote: > On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: >> On 03/31/2017 11:06 AM, Smitha T Murthy wrote: >>> Add V4L2 definition for HEVC compressed format >>> >>> Signed-off-by: Smitha T Murthy >>> Reviewed-by: Andrzej Hajda >>> --- >>>

Re: [Patch v3 06/11] [media] s5p-mfc: Add support for HEVC decoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:13 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add support for codec definition and corresponding buffer > > requirements for HEVC decoder. > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda

Re: [Patch v3 06/11] [media] s5p-mfc: Add support for HEVC decoder

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:13 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add support for codec definition and corresponding buffer > > requirements for HEVC decoder. > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > >

Re: [Patch v3 05/11] [media] videodev2.h: Add v4l2 definition for HEVC

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:11 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > >

Re: [Patch v3 05/11] [media] videodev2.h: Add v4l2 definition for HEVC

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:11 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > include/uapi/linux/videodev2.h | 1 + > > 1 file changed,

Re: [PATCH] KVM: mmu: Fix overlap with private memslots

2017-04-03 Thread David Hildenbrand
On 27.03.2017 08:23, Wanpeng Li wrote: > From: Wanpeng Li > > Reported by syzkaller: > > pte_list_remove: 9714eb1f8078 0->BUG > [ cut here ] > kernel BUG at arch/x86/kvm/mmu.c:1157! > invalid opcode: [#1] SMP > RIP:

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > >

Re: [PATCH] KVM: mmu: Fix overlap with private memslots

2017-04-03 Thread David Hildenbrand
On 27.03.2017 08:23, Wanpeng Li wrote: > From: Wanpeng Li > > Reported by syzkaller: > > pte_list_remove: 9714eb1f8078 0->BUG > [ cut here ] > kernel BUG at arch/x86/kvm/mmu.c:1157! > invalid opcode: [#1] SMP > RIP:

Re: [Patch v3 07/11] Documentation: v4l: Documentation for HEVC v4l2 definition

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 10:10 +0200, Hans Verkuil wrote: > On 03/31/2017 11:06 AM, Smitha T Murthy wrote: > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > > --- > > Documentation/media/uapi/v4l/pixfmt-013.rst | 5 + > >

Re: [Patch v3 04/11] [media] s5p-mfc: Support MFCv10.10 buffer requirements

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 09:41 +0200, Andrzej Hajda wrote: > On 31.03.2017 11:06, Smitha T Murthy wrote: > > Aligning the luma_dpb_size, chroma_dpb_size, mv_size and me_buffer_size > > for MFCv10.10. > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda

Re: [Patch v3 04/11] [media] s5p-mfc: Support MFCv10.10 buffer requirements

2017-04-03 Thread Smitha T Murthy
On Mon, 2017-04-03 at 09:41 +0200, Andrzej Hajda wrote: > On 31.03.2017 11:06, Smitha T Murthy wrote: > > Aligning the luma_dpb_size, chroma_dpb_size, mv_size and me_buffer_size > > for MFCv10.10. > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej Hajda > -- > Regards > Andrzej >

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Eric Dumazet
On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: > I modified patch a little as: > if (th->doff * 4 < sizeof(_tcph)) { > par->hotdrop = true; > WARN_ON_ONCE(!tcpinfo->option); > return false; > } > > And it did triggered WARN once at morning, and didn't hit KASAN. I will >

Re: KASAN, xt_TCPMSS finally found nasty use-after-free bug? 4.10.8

2017-04-03 Thread Eric Dumazet
On Mon, 2017-04-03 at 11:10 +0300, Denys Fedoryshchenko wrote: > I modified patch a little as: > if (th->doff * 4 < sizeof(_tcph)) { > par->hotdrop = true; > WARN_ON_ONCE(!tcpinfo->option); > return false; > } > > And it did triggered WARN once at morning, and didn't hit KASAN. I will >

Re: in_irq_or_nmi() and RFC patch

2017-04-03 Thread Mel Gorman
On Thu, Mar 30, 2017 at 05:07:08PM +0200, Jesper Dangaard Brouer wrote: > On Thu, 30 Mar 2017 14:04:36 +0100 > Mel Gorman wrote: > > > On Wed, Mar 29, 2017 at 09:44:41PM +0200, Jesper Dangaard Brouer wrote: > > > > Regardless or using in_irq() (or in combi with

Re: in_irq_or_nmi() and RFC patch

2017-04-03 Thread Mel Gorman
On Thu, Mar 30, 2017 at 05:07:08PM +0200, Jesper Dangaard Brouer wrote: > On Thu, 30 Mar 2017 14:04:36 +0100 > Mel Gorman wrote: > > > On Wed, Mar 29, 2017 at 09:44:41PM +0200, Jesper Dangaard Brouer wrote: > > > > Regardless or using in_irq() (or in combi with in_nmi()) I get the > > > >

<    7   8   9   10   11   12   13   14   15   16   >