Re: uvcvideo logging kernel warnings on device disconnect

2017-04-17 Thread Daniel Axtens
Hi, >> > I hate to pester, but wondered if you had found anything obvious. >> > I really do appreciate you taking the time to look. >> >> Sorry, I haven't had the chance and now will not be able to until >> January > > Did you mean January 2017 or 2018 ? :-) I stumbled across this problem

Re: uvcvideo logging kernel warnings on device disconnect

2017-04-17 Thread Daniel Axtens
Hi, >> > I hate to pester, but wondered if you had found anything obvious. >> > I really do appreciate you taking the time to look. >> >> Sorry, I haven't had the chance and now will not be able to until >> January > > Did you mean January 2017 or 2018 ? :-) I stumbled across this problem

[PATCH 2/2] [media] uvcvideo: Kill video URBs on disconnect

2017-04-17 Thread Daniel Axtens
When an in-use webcam is disconnected, I noticed the following messages: uvcvideo: Failed to resubmit video URB (-19). -19 is -ENODEV, which does make sense given that the device has disappeared. We could put a case for -ENODEV like we have with -ENOENT, -ECONNRESET and -ESHUTDOWN, but the

[PATCH 2/2] [media] uvcvideo: Kill video URBs on disconnect

2017-04-17 Thread Daniel Axtens
When an in-use webcam is disconnected, I noticed the following messages: uvcvideo: Failed to resubmit video URB (-19). -19 is -ENODEV, which does make sense given that the device has disappeared. We could put a case for -ENODEV like we have with -ENOENT, -ECONNRESET and -ESHUTDOWN, but the

[PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-17 Thread Daniel Axtens
Currently, disconnecting a USB webcam while it is in use prints out a number of warnings, such as: WARNING: CPU: 2 PID: 3118 at /build/linux-ezBi1T/linux-4.8.0/fs/sysfs/group.c:237 sysfs_remove_group+0x8b/0x90 sysfs group a7cd0780 not found for kobject 'event13' This has been noticed

[PATCH 1/2] [media] uvcvideo: Refactor teardown of uvc on USB disconnect

2017-04-17 Thread Daniel Axtens
Currently, disconnecting a USB webcam while it is in use prints out a number of warnings, such as: WARNING: CPU: 2 PID: 3118 at /build/linux-ezBi1T/linux-4.8.0/fs/sysfs/group.c:237 sysfs_remove_group+0x8b/0x90 sysfs group a7cd0780 not found for kobject 'event13' This has been noticed

[tip:perf/core] perf util: Hint missing file when tool tips fail to load

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 570eda03213a216a88566c0da7bfe175832cfaa4 Gitweb: http://git.kernel.org/tip/570eda03213a216a88566c0da7bfe175832cfaa4 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:16 -0700 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] perf util: Hint missing file when tool tips fail to load

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 570eda03213a216a88566c0da7bfe175832cfaa4 Gitweb: http://git.kernel.org/tip/570eda03213a216a88566c0da7bfe175832cfaa4 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:16 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 11:52:51 -0300

[tip:perf/core] Revert "perf tools: Fix include of linux/mman.h"

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 16eb81365b70266c17d1141ef9b32c3110b22d17 Gitweb: http://git.kernel.org/tip/16eb81365b70266c17d1141ef9b32c3110b22d17 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:19 -0700 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] Revert "perf tools: Fix include of linux/mman.h"

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 16eb81365b70266c17d1141ef9b32c3110b22d17 Gitweb: http://git.kernel.org/tip/16eb81365b70266c17d1141ef9b32c3110b22d17 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:19 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 11:54:46 -0300

[tip:perf/core] tools build: Fix feature detection redefinion of build flags

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 9961aa665b70e47d6c80141c4a2482266010f246 Gitweb: http://git.kernel.org/tip/9961aa665b70e47d6c80141c4a2482266010f246 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:14 -0700 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] tools build: Fix feature detection redefinion of build flags

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 9961aa665b70e47d6c80141c4a2482266010f246 Gitweb: http://git.kernel.org/tip/9961aa665b70e47d6c80141c4a2482266010f246 Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:14 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 11:48:48 -0300

[tip:perf/core] perf tools: Disable JVMTI if no ELF support available

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: e5e992a7c184c2121adf37bdf292a516af81dbbb Gitweb: http://git.kernel.org/tip/e5e992a7c184c2121adf37bdf292a516af81dbbb Author: David Carrillo-Cisneros AuthorDate: Wed, 12 Apr 2017 10:07:45 -0700 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] perf tools: Disable JVMTI if no ELF support available

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: e5e992a7c184c2121adf37bdf292a516af81dbbb Gitweb: http://git.kernel.org/tip/e5e992a7c184c2121adf37bdf292a516af81dbbb Author: David Carrillo-Cisneros AuthorDate: Wed, 12 Apr 2017 10:07:45 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 11:47:43 -0300

[tip:perf/core] perf stat: Fix bug in handling events in error state

2017-04-17 Thread tip-bot for Stephane Eranian
Commit-ID: db49a71798a38f3ddf3f3462703328dca39b1ac7 Gitweb: http://git.kernel.org/tip/db49a71798a38f3ddf3f3462703328dca39b1ac7 Author: Stephane Eranian AuthorDate: Wed, 12 Apr 2017 11:23:01 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu,

[tip:perf/core] perf stat: Fix bug in handling events in error state

2017-04-17 Thread tip-bot for Stephane Eranian
Commit-ID: db49a71798a38f3ddf3f3462703328dca39b1ac7 Gitweb: http://git.kernel.org/tip/db49a71798a38f3ddf3f3462703328dca39b1ac7 Author: Stephane Eranian AuthorDate: Wed, 12 Apr 2017 11:23:01 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 10:40:36 -0300 perf

[tip:perf/core] perf trace: Add usage of --no-syscalls in man page

2017-04-17 Thread tip-bot for Ravi Bangoria
Commit-ID: 739cf305512cb852e852099f9e12dd66bf4df076 Gitweb: http://git.kernel.org/tip/739cf305512cb852e852099f9e12dd66bf4df076 Author: Ravi Bangoria AuthorDate: Thu, 13 Apr 2017 11:32:12 +0530 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] perf trace: Add usage of --no-syscalls in man page

2017-04-17 Thread tip-bot for Ravi Bangoria
Commit-ID: 739cf305512cb852e852099f9e12dd66bf4df076 Gitweb: http://git.kernel.org/tip/739cf305512cb852e852099f9e12dd66bf4df076 Author: Ravi Bangoria AuthorDate: Thu, 13 Apr 2017 11:32:12 +0530 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 13 Apr 2017 10:54:04 -0300 perf trace:

[tip:perf/core] perf tools: Pass PYTHON config to feature detection

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 7be6b3166ebf2c10c28ef5777d1b31a937ed8f7a Gitweb: http://git.kernel.org/tip/7be6b3166ebf2c10c28ef5777d1b31a937ed8f7a Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:13 -0700 Committer: Arnaldo Carvalho de Melo

[tip:perf/core] perf tools: Pass PYTHON config to feature detection

2017-04-17 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 7be6b3166ebf2c10c28ef5777d1b31a937ed8f7a Gitweb: http://git.kernel.org/tip/7be6b3166ebf2c10c28ef5777d1b31a937ed8f7a Author: David Carrillo-Cisneros AuthorDate: Tue, 11 Apr 2017 23:49:13 -0700 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 12 Apr 2017 10:45:21 -0300

Re: [PATCH] powerpc/mm: some cleanup of do_page_fault()

2017-04-17 Thread Aneesh Kumar K.V
Christophe Leroy writes: > This patch is a bunch of small cleanups of the do_page_fault() > function: > 1/ Function store_updates_sp() checks whether the faulting > instruction is a store updating r1. Therefore we can limit its calls > to stores exceptions > 2/ Only the

Re: [PATCH] powerpc/mm: some cleanup of do_page_fault()

2017-04-17 Thread Aneesh Kumar K.V
Christophe Leroy writes: > This patch is a bunch of small cleanups of the do_page_fault() > function: > 1/ Function store_updates_sp() checks whether the faulting > instruction is a store updating r1. Therefore we can limit its calls > to stores exceptions > 2/ Only the get_user() in

答复: [PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread 郝加节
CC: linux-arm-kernel Many thanks Jiajie. -邮件原件- 发件人: jiajie@mediatek.com [mailto:jiajie@mediatek.com] 发送时间: 2017年4月17日 16:10 收件人: ulf.hans...@linaro.org 抄送: linus.wall...@linaro.org; Chaotian Jing (井朝天) ; Yong Mao (毛勇) ; Eddie

答复: [PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread 郝加节
CC: linux-arm-kernel Many thanks Jiajie. -邮件原件- 发件人: jiajie@mediatek.com [mailto:jiajie@mediatek.com] 发送时间: 2017年4月17日 16:10 收件人: ulf.hans...@linaro.org 抄送: linus.wall...@linaro.org; Chaotian Jing (井朝天) ; Yong Mao (毛勇) ; Eddie Huang (黃智傑) ; srv_heupstream ;

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Greg Kroah-Hartman
On Mon, Apr 17, 2017 at 01:37:00PM +0530, Amit Pundir wrote: > On 17 April 2017 at 12:26, Greg Kroah-Hartman > wrote: > > On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: > >> On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: > >> > This is the start of

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Greg Kroah-Hartman
On Mon, Apr 17, 2017 at 01:37:00PM +0530, Amit Pundir wrote: > On 17 April 2017 at 12:26, Greg Kroah-Hartman > wrote: > > On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: > >> On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: > >> > This is the start of the stable review cycle for

Re: pinctrl: imx: Checking reuse of grouping functionality

2017-04-17 Thread Gary Bisson
Hi Markus, All, On Sat, Apr 15, 2017 at 9:55 PM, SF Markus Elfring wrote: > Hello, > > A preprocessor symbol like “CONFIG_GENERIC_PINCTRL_GROUPS” was introduced > by the commit c033a718f615b6b3ddc83ce3e0a217c30bd09cb5 (“pinctrl: stricten up > generic group code”)

Re: pinctrl: imx: Checking reuse of grouping functionality

2017-04-17 Thread Gary Bisson
Hi Markus, All, On Sat, Apr 15, 2017 at 9:55 PM, SF Markus Elfring wrote: > Hello, > > A preprocessor symbol like “CONFIG_GENERIC_PINCTRL_GROUPS” was introduced > by the commit c033a718f615b6b3ddc83ce3e0a217c30bd09cb5 (“pinctrl: stricten up > generic group code”) on 2017-01-03. Some functions

[PATCH] regulator: tps65023: Fix inverted core enable logic.

2017-04-17 Thread Richard Cochran
Commit 43530b69d758328d3ffe6ab98fd640463e8e3667 ("regulator: Use regmap_read/write(), regmap_update_bits functions directly") intended to replace working inline helper functions with standard regmap calls. However, it also inverted the set/clear logic of the "CORE ADJ Allowed" bit. That patch

[PATCH] regulator: tps65023: Fix inverted core enable logic.

2017-04-17 Thread Richard Cochran
Commit 43530b69d758328d3ffe6ab98fd640463e8e3667 ("regulator: Use regmap_read/write(), regmap_update_bits functions directly") intended to replace working inline helper functions with standard regmap calls. However, it also inverted the set/clear logic of the "CORE ADJ Allowed" bit. That patch

Re: your mail

2017-04-17 Thread Michal Hocko
On Mon 17-04-17 14:47:20, Joonsoo Kim wrote: > On Sat, Apr 15, 2017 at 02:17:31PM +0200, Michal Hocko wrote: > > Hi, > > here I 3 more preparatory patches which I meant to send on Thursday but > > forgot... After more thinking about pfn walkers I have realized that > > the current code doesn't

Re: your mail

2017-04-17 Thread Michal Hocko
On Mon 17-04-17 14:47:20, Joonsoo Kim wrote: > On Sat, Apr 15, 2017 at 02:17:31PM +0200, Michal Hocko wrote: > > Hi, > > here I 3 more preparatory patches which I meant to send on Thursday but > > forgot... After more thinking about pfn walkers I have realized that > > the current code doesn't

Re: [GIT PULL 0/7] perf/core improvements and fixes

2017-04-17 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > Em Thu, Apr 13, 2017 at 04:35:29PM -0300, Arnaldo Carvalho de Melo escreveu: > > Hi Ingo, > > > > Please consider pulling, > > > > - Arnaldo > > > > Test results at the end of this message, as usual. > > Ooops, here is the missing

Re: [GIT PULL 0/7] perf/core improvements and fixes

2017-04-17 Thread Ingo Molnar
* Arnaldo Carvalho de Melo wrote: > Em Thu, Apr 13, 2017 at 04:35:29PM -0300, Arnaldo Carvalho de Melo escreveu: > > Hi Ingo, > > > > Please consider pulling, > > > > - Arnaldo > > > > Test results at the end of this message, as usual. > > Ooops, here is the missing output from

Re: [kbuild-all] [tip:x86/cpu 8/12] arch/x86/kernel/cpu/intel_rdt.c:63: error: unknown field 'cache' specified in initializer

2017-04-17 Thread Thomas Gleixner
On Mon, 17 Apr 2017, Fengguang Wu wrote: > > So the compiler fails to handle the anon union, which was introduced in > > 05b93417ce5b924. No idea why, but that concept is not new and widely used > > in the kernel already. > > It looks like a problem with gcc 4.4: > >Bug 42875 - gcc

Re: [kbuild-all] [tip:x86/cpu 8/12] arch/x86/kernel/cpu/intel_rdt.c:63: error: unknown field 'cache' specified in initializer

2017-04-17 Thread Thomas Gleixner
On Mon, 17 Apr 2017, Fengguang Wu wrote: > > So the compiler fails to handle the anon union, which was introduced in > > 05b93417ce5b924. No idea why, but that concept is not new and widely used > > in the kernel already. > > It looks like a problem with gcc 4.4: > >Bug 42875 - gcc

[PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread jiajie.hao
From: Jiajie Hao The host may issue an I/O abort by writing to the CCCR at any time during I/O read operation via CMD52. And host may need suspend transcation during write busy stage in SDIO suspend/resume scenario. >From other side, a card may accept CMD52 during data

[PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread jiajie.hao
From: Jiajie Hao The host may issue an I/O abort by writing to the CCCR at any time during I/O read operation via CMD52. And host may need suspend transcation during write busy stage in SDIO suspend/resume scenario. >From other side, a card may accept CMD52 during data transfer phase. Previous

[PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread jiajie.hao
The host may issue an I/O abort by writing to the CCCR at any time during I/O read operation via CMD52. And host may need suspend transcation during write busy stage in SDIO suspend/resume scenario. >From other side, a card may accept CMD52 during data transfer phase. Previous implement would

[PATCH] mmc: sdio: Fix sdio wait busy implement limitation

2017-04-17 Thread jiajie.hao
The host may issue an I/O abort by writing to the CCCR at any time during I/O read operation via CMD52. And host may need suspend transcation during write busy stage in SDIO suspend/resume scenario. >From other side, a card may accept CMD52 during data transfer phase. Previous implement would

Re: [PATCH v6 03/11] powerpc/powernv: Detect supported IMC units and its events

2017-04-17 Thread Anju T Sudhakar
Hi Michael, On Thursday 13 April 2017 05:13 PM, Michael Ellerman wrote: Anju T Sudhakar writes: On Thursday 06 April 2017 02:07 PM, Stewart Smith wrote: Madhavan Srinivasan writes: --- a/arch/powerpc/platforms/powernv/opal-imc.c +++

Re: [PATCH v6 03/11] powerpc/powernv: Detect supported IMC units and its events

2017-04-17 Thread Anju T Sudhakar
Hi Michael, On Thursday 13 April 2017 05:13 PM, Michael Ellerman wrote: Anju T Sudhakar writes: On Thursday 06 April 2017 02:07 PM, Stewart Smith wrote: Madhavan Srinivasan writes: --- a/arch/powerpc/platforms/powernv/opal-imc.c +++ b/arch/powerpc/platforms/powernv/opal-imc.c @@ -33,6

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Amit Pundir
On 17 April 2017 at 12:26, Greg Kroah-Hartman wrote: > On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: >> On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: >> > This is the start of the stable review cycle for the 3.18.49 release. >> > There are 145

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Amit Pundir
On 17 April 2017 at 12:26, Greg Kroah-Hartman wrote: > On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: >> On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: >> > This is the start of the stable review cycle for the 3.18.49 release. >> > There are 145 patches in this series, all will

Re: [PATCH v2 11/11] arm64: allwinner: a64: enable Wi-Fi for Pine64

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 06:56:16PM +0200, Andreas Färber wrote: > > one provides I/O voltage). > > > > Add device node for it. > > > > Signed-off-by: Icenowy Zheng > > --- > > arch/arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 16 > > 1 file changed, 16

Re: [PATCH v2 11/11] arm64: allwinner: a64: enable Wi-Fi for Pine64

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 06:56:16PM +0200, Andreas Färber wrote: > > one provides I/O voltage). > > > > Add device node for it. > > > > Signed-off-by: Icenowy Zheng > > --- > > arch/arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 16 > > 1 file changed, 16 insertions(+) > > >

Re: [PATCH v2] arm64: allwinner: h5: add support for Orange Pi Prime board

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 12:38:07AM +0800, Icenowy Zheng wrote: > Orange Pi Prime is a new Allwinner H5-based SBC by Xunlong. > > It's like a Orange Pi Plus 2E with H3 replaced with H5, eMMC replaced > with onboard SPI NOR Flash and wireless card changed to Realtek > RTL8723BS (with Bluetooth

Re: [PATCH v2] arm64: allwinner: h5: add support for Orange Pi Prime board

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 12:38:07AM +0800, Icenowy Zheng wrote: > Orange Pi Prime is a new Allwinner H5-based SBC by Xunlong. > > It's like a Orange Pi Plus 2E with H3 replaced with H5, eMMC replaced > with onboard SPI NOR Flash and wireless card changed to Realtek > RTL8723BS (with Bluetooth

Re: [PATCH 2/2] arm64: allwinner: a64: enable EHCI0/OHCI0 for Pine64

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 09:15:55PM +0800, Icenowy Zheng wrote: > As we have USB0 controller switch available on A64, we should now enable > the EHCI0/OHCI0 controllers for Pine64. > > Signed-off-by: Icenowy Zheng Applied both, thanks! Maxime -- Maxime Ripard, Free Electrons

Re: [PATCH 2/2] arm64: allwinner: a64: enable EHCI0/OHCI0 for Pine64

2017-04-17 Thread Maxime Ripard
On Fri, Apr 14, 2017 at 09:15:55PM +0800, Icenowy Zheng wrote: > As we have USB0 controller switch available on A64, we should now enable > the EHCI0/OHCI0 controllers for Pine64. > > Signed-off-by: Icenowy Zheng Applied both, thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and

Re: [PATCH 4/5] ARM: sun8i: h3: add operating-points-v2 table for CPU

2017-04-17 Thread Maxime Ripard
On Mon, Apr 17, 2017 at 05:00:00AM +0800, Icenowy Zheng wrote: > > > 于 2017年4月17日 GMT+08:00 上午4:57:40, Maxime Ripard > 写到: > >On Tue, Apr 11, 2017 at 09:28:55PM +0800, icen...@aosc.io wrote: > >> 在 2017-04-11 17:13,Maxime Ripard 写道: > >> > On Sun, Apr 09, 2017

Re: [PATCH 4/5] ARM: sun8i: h3: add operating-points-v2 table for CPU

2017-04-17 Thread Maxime Ripard
On Mon, Apr 17, 2017 at 05:00:00AM +0800, Icenowy Zheng wrote: > > > 于 2017年4月17日 GMT+08:00 上午4:57:40, Maxime Ripard > 写到: > >On Tue, Apr 11, 2017 at 09:28:55PM +0800, icen...@aosc.io wrote: > >> 在 2017-04-11 17:13,Maxime Ripard 写道: > >> > On Sun, Apr 09, 2017 at 02:50:24AM +0800, Icenowy

Re: [PATCH] usb: gadget: remove redundant self assignment

2017-04-17 Thread Peter Chen
On Sun, Apr 16, 2017 at 08:12:50PM -0700, Stefan Agner wrote: > The assignment ret = ret is redundant and can be removed. > > Signed-off-by: Stefan Agner > --- > A very similar patch has been applied already last year, but there is > a second such assignment... > > -- > Stefan

Re: [PATCH] usb: gadget: remove redundant self assignment

2017-04-17 Thread Peter Chen
On Sun, Apr 16, 2017 at 08:12:50PM -0700, Stefan Agner wrote: > The assignment ret = ret is redundant and can be removed. > > Signed-off-by: Stefan Agner > --- > A very similar patch has been applied already last year, but there is > a second such assignment... > > -- > Stefan > >

Re: [patch 17/20] PCI: Use cpu_hotplug_disable() instead of get_online_cpus()

2017-04-17 Thread Thomas Gleixner
On Mon, 17 Apr 2017, Peter Zijlstra wrote: > On Sat, Apr 15, 2017 at 07:01:24PM +0200, Thomas Gleixner wrote: > > +++ b/drivers/pci/pci-driver.c > > @@ -320,10 +320,19 @@ static long local_pci_probe(void *_ddi) > > return 0; > > } > > > > +static bool pci_physfn_is_probed(struct pci_dev

Re: [patch 17/20] PCI: Use cpu_hotplug_disable() instead of get_online_cpus()

2017-04-17 Thread Thomas Gleixner
On Mon, 17 Apr 2017, Peter Zijlstra wrote: > On Sat, Apr 15, 2017 at 07:01:24PM +0200, Thomas Gleixner wrote: > > +++ b/drivers/pci/pci-driver.c > > @@ -320,10 +320,19 @@ static long local_pci_probe(void *_ddi) > > return 0; > > } > > > > +static bool pci_physfn_is_probed(struct pci_dev

Re: [PATCH v7 1/7] mm/page_alloc: don't reserve ZONE_HIGHMEM for ZONE_MOVABLE request

2017-04-17 Thread Minchan Kim
Hi Joonsoo, On Tue, Apr 11, 2017 at 12:17:14PM +0900, js1...@gmail.com wrote: > From: Joonsoo Kim > > Freepage on ZONE_HIGHMEM doesn't work for kernel memory so it's not that > important to reserve. When ZONE_MOVABLE is used, this problem would > theorectically cause to

Re: [PATCH v7 1/7] mm/page_alloc: don't reserve ZONE_HIGHMEM for ZONE_MOVABLE request

2017-04-17 Thread Minchan Kim
Hi Joonsoo, On Tue, Apr 11, 2017 at 12:17:14PM +0900, js1...@gmail.com wrote: > From: Joonsoo Kim > > Freepage on ZONE_HIGHMEM doesn't work for kernel memory so it's not that > important to reserve. When ZONE_MOVABLE is used, this problem would > theorectically cause to decrease usable memory

Re: [PATCH] Revert "arm64: Increase the max granular size"

2017-04-17 Thread Imran Khan
On 4/12/2017 7:30 PM, Chalamarla, Tirumalesh wrote: > > > On 4/11/17, 10:13 PM, "linux-arm-kernel on behalf of Imran Khan" > kim...@codeaurora.org> wrote: > > On 4/7/2017 7:36 AM, Ganesh Mahendran wrote: > > 2017-04-06 23:58

Re: [PATCH] Revert "arm64: Increase the max granular size"

2017-04-17 Thread Imran Khan
On 4/12/2017 7:30 PM, Chalamarla, Tirumalesh wrote: > > > On 4/11/17, 10:13 PM, "linux-arm-kernel on behalf of Imran Khan" > kim...@codeaurora.org> wrote: > > On 4/7/2017 7:36 AM, Ganesh Mahendran wrote: > > 2017-04-06 23:58 GMT+08:00 Catalin Marinas : > >> On Thu, Apr 06, 2017 at

Re: [RFC 0/8] Copy Offload with Peer-to-Peer PCI Memory

2017-04-17 Thread Benjamin Herrenschmidt
On Sun, 2017-04-16 at 23:13 -0600, Logan Gunthorpe wrote: > > > > > I'm still not 100% why do you need a "p2mem device" mind you ... > > Well, you don't "need" it but it is a design choice that I think makes a > lot of sense for the following reasons: > > 1) p2pmem is in fact a device on the

Re: [RFC 0/8] Copy Offload with Peer-to-Peer PCI Memory

2017-04-17 Thread Benjamin Herrenschmidt
On Sun, 2017-04-16 at 23:13 -0600, Logan Gunthorpe wrote: > > > > > I'm still not 100% why do you need a "p2mem device" mind you ... > > Well, you don't "need" it but it is a design choice that I think makes a > lot of sense for the following reasons: > > 1) p2pmem is in fact a device on the

Re: [PATCH 2/2] staging:skein: skein_base.h, skein_block.h: move macros into appropriate header files

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:47:34AM +0200, Karim Eshapa wrote: > Macros more related to BLK operations. That doesn't make any sense to me, can you be more explicit? > Signed-off-by: Karim Eshapa > --- > drivers/staging/skein/skein_base.h | 28

Re: [PATCH 2/2] staging:skein: skein_base.h, skein_block.h: move macros into appropriate header files

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:47:34AM +0200, Karim Eshapa wrote: > Macros more related to BLK operations. That doesn't make any sense to me, can you be more explicit? > Signed-off-by: Karim Eshapa > --- > drivers/staging/skein/skein_base.h | 28 >

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Greg Kroah-Hartman
On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: > On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 3.18.49 release. > > There are 145 patches in this series, all will be posted as a response > > to this one. If anyone has

Re: [PATCH 3.18 000/145] 3.18.49-stable review

2017-04-17 Thread Greg Kroah-Hartman
On Sun, Apr 16, 2017 at 04:30:32PM -0700, Guenter Roeck wrote: > On 04/16/2017 03:48 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 3.18.49 release. > > There are 145 patches in this series, all will be posted as a response > > to this one. If anyone has

Re: [PATCH 3/4] restrict unprivileged TIOCSTI tty ioctl

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:05AM -0400, Matt Brown wrote: > this patch depends on patch 1 and 2 > > enforces restrictions on unprivileged users injecting commands > into other processes in the same tty session using the TIOCSTI ioctl > > Signed-off-by: Matt Brown > --- >

Re: [PATCH v4 2/2] vfio/type1: Prune vfio_pin_page_external()

2017-04-17 Thread Peter Xu
On Sun, Apr 16, 2017 at 07:42:39PM -0600, Alex Williamson wrote: > With vfio_lock_acct() testing the locked memory limit under mmap_sem, > it's redundant to do it here for a single page. We can also reorder > our tests such that we can avoid testing for reserved pages if we're > not doing

Re: [PATCH 3/4] restrict unprivileged TIOCSTI tty ioctl

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:05AM -0400, Matt Brown wrote: > this patch depends on patch 1 and 2 > > enforces restrictions on unprivileged users injecting commands > into other processes in the same tty session using the TIOCSTI ioctl > > Signed-off-by: Matt Brown > --- > drivers/tty/tty_io.c

Re: [PATCH v4 2/2] vfio/type1: Prune vfio_pin_page_external()

2017-04-17 Thread Peter Xu
On Sun, Apr 16, 2017 at 07:42:39PM -0600, Alex Williamson wrote: > With vfio_lock_acct() testing the locked memory limit under mmap_sem, > it's redundant to do it here for a single page. We can also reorder > our tests such that we can avoid testing for reserved pages if we're > not doing

Re: [PATCH 2/4] add tiocsti_restrict variable

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:04AM -0400, Matt Brown wrote: > adding extern variable tiocsti_restrict to allow it to be included > in sysctl > > Signed-off-by: Matt Brown > --- > include/linux/tty.h | 2 ++ > 1 file changed, 2 insertions(+) I'm all for breaking up changes to be

Re: [PATCH 2/4] add tiocsti_restrict variable

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:04AM -0400, Matt Brown wrote: > adding extern variable tiocsti_restrict to allow it to be included > in sysctl > > Signed-off-by: Matt Brown > --- > include/linux/tty.h | 2 ++ > 1 file changed, 2 insertions(+) I'm all for breaking up changes to be tiny logical

Re: [PATCH 1/4] added SECURITY_TIOCSTI_RESTRICT kernel config

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:03AM -0400, Matt Brown wrote: > adding the kernel config SECURITY_TIOCSTI_RESTRICT in order to allow > the user to restrict unprivileged command injection using TIOCSTI > tty ioctls "unpriviledged command injection"? That sounds a bit "odd", don't you think? > >

Re: [PATCH 1/4] added SECURITY_TIOCSTI_RESTRICT kernel config

2017-04-17 Thread Greg KH
On Mon, Apr 17, 2017 at 02:07:03AM -0400, Matt Brown wrote: > adding the kernel config SECURITY_TIOCSTI_RESTRICT in order to allow > the user to restrict unprivileged command injection using TIOCSTI > tty ioctls "unpriviledged command injection"? That sounds a bit "odd", don't you think? > >

Re: [patch 20/20] cpu/hotplug: Convert hotplug locking to percpu rwsem

2017-04-17 Thread Peter Zijlstra
On Sat, Apr 15, 2017 at 07:01:27PM +0200, Thomas Gleixner wrote: > There are no more (known) nested calls to get_online_cpus() so it's > possible to remove the nested call magic and convert the mutex to a > percpu-rwsem, which speeds up get/put_online_cpus() significantly for the > uncontended

Re: [patch 20/20] cpu/hotplug: Convert hotplug locking to percpu rwsem

2017-04-17 Thread Peter Zijlstra
On Sat, Apr 15, 2017 at 07:01:27PM +0200, Thomas Gleixner wrote: > There are no more (known) nested calls to get_online_cpus() so it's > possible to remove the nested call magic and convert the mutex to a > percpu-rwsem, which speeds up get/put_online_cpus() significantly for the > uncontended

Re: [PATCH v4 1/2] vfio/type1: Remove locked page accounting workqueue

2017-04-17 Thread Peter Xu
On Sun, Apr 16, 2017 at 07:42:27PM -0600, Alex Williamson wrote: [...] > -static void vfio_lock_acct(struct task_struct *task, long npage) > +static int vfio_lock_acct(struct task_struct *task, long npage, bool > lock_cap) > { > - struct vwork *vwork; > struct mm_struct *mm; >

Re: [PATCH v4 1/2] vfio/type1: Remove locked page accounting workqueue

2017-04-17 Thread Peter Xu
On Sun, Apr 16, 2017 at 07:42:27PM -0600, Alex Williamson wrote: [...] > -static void vfio_lock_acct(struct task_struct *task, long npage) > +static int vfio_lock_acct(struct task_struct *task, long npage, bool > lock_cap) > { > - struct vwork *vwork; > struct mm_struct *mm; >

Re: [patch 17/20] PCI: Use cpu_hotplug_disable() instead of get_online_cpus()

2017-04-17 Thread Peter Zijlstra
On Sat, Apr 15, 2017 at 07:01:24PM +0200, Thomas Gleixner wrote: > +++ b/drivers/pci/pci-driver.c > @@ -320,10 +320,19 @@ static long local_pci_probe(void *_ddi) > return 0; > } > > +static bool pci_physfn_is_probed(struct pci_dev *dev) > +{ > +#ifdef CONFIG_ATS > + return

Re: [patch 17/20] PCI: Use cpu_hotplug_disable() instead of get_online_cpus()

2017-04-17 Thread Peter Zijlstra
On Sat, Apr 15, 2017 at 07:01:24PM +0200, Thomas Gleixner wrote: > +++ b/drivers/pci/pci-driver.c > @@ -320,10 +320,19 @@ static long local_pci_probe(void *_ddi) > return 0; > } > > +static bool pci_physfn_is_probed(struct pci_dev *dev) > +{ > +#ifdef CONFIG_ATS > + return

Re: [RfC PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-17 Thread Ilia Mirkin
On Tue, Apr 11, 2017 at 10:18 AM, Ilia Mirkin wrote: >> However, I totally agree with Alex that someone with a BE machine >> should review the whole stack before we could be confident with anything. > > Here's what I'm confident about: xf86-video-nouveau worked just fine >

Re: [RfC PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-17 Thread Ilia Mirkin
On Tue, Apr 11, 2017 at 10:18 AM, Ilia Mirkin wrote: >> However, I totally agree with Alex that someone with a BE machine >> should review the whole stack before we could be confident with anything. > > Here's what I'm confident about: xf86-video-nouveau worked just fine > on top of kernel 4.3 on

[PATCH] net: cx89x0: move attribute declaration before struct keyword

2017-04-17 Thread Stefan Agner
The attribute declaration is typically before the definition. Move the __maybe_unused attribute declaration before the struct keyword. Signed-off-by: Stefan Agner --- drivers/net/ethernet/cirrus/cs89x0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] net: cx89x0: move attribute declaration before struct keyword

2017-04-17 Thread Stefan Agner
The attribute declaration is typically before the definition. Move the __maybe_unused attribute declaration before the struct keyword. Signed-off-by: Stefan Agner --- drivers/net/ethernet/cirrus/cs89x0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] ARM: dts: dra7: Add power hold and power controller properties to palmas

2017-04-17 Thread Keerthy
On Friday 14 April 2017 07:51 PM, Tony Lindgren wrote: > * Keerthy [170412 21:55]: >> Add power hold and power controller properties to palmas node. >> This is needed to shutdown pmic correctly on boards with >> powerhold set. > > Is this OK to wait for v4.12 or is this

Re: [PATCH] ARM: dts: dra7: Add power hold and power controller properties to palmas

2017-04-17 Thread Keerthy
On Friday 14 April 2017 07:51 PM, Tony Lindgren wrote: > * Keerthy [170412 21:55]: >> Add power hold and power controller properties to palmas node. >> This is needed to shutdown pmic correctly on boards with >> powerhold set. > > Is this OK to wait for v4.12 or is this needed as a fix for >

[PATCH 2/4] add tiocsti_restrict variable

2017-04-17 Thread Matt Brown
adding extern variable tiocsti_restrict to allow it to be included in sysctl Signed-off-by: Matt Brown --- include/linux/tty.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/linux/tty.h b/include/linux/tty.h index 1017e904..7011102 100644 --- a/include/linux/tty.h

[PATCH 2/4] add tiocsti_restrict variable

2017-04-17 Thread Matt Brown
adding extern variable tiocsti_restrict to allow it to be included in sysctl Signed-off-by: Matt Brown --- include/linux/tty.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/linux/tty.h b/include/linux/tty.h index 1017e904..7011102 100644 --- a/include/linux/tty.h +++

Patchset to Restrict Unprivileged TIOCSTI TTY Command Injection

2017-04-17 Thread Matt Brown
The following patchset reproduces GRKERNSEC_HARDEN_TTY functionality from the grsecurity project in-kernel. The purpose of this feature is to restrict unprivileged users from injecting commands into other processes in the same tty session by using the TIOCSTI ioctl. It creates the kernel

Patchset to Restrict Unprivileged TIOCSTI TTY Command Injection

2017-04-17 Thread Matt Brown
The following patchset reproduces GRKERNSEC_HARDEN_TTY functionality from the grsecurity project in-kernel. The purpose of this feature is to restrict unprivileged users from injecting commands into other processes in the same tty session by using the TIOCSTI ioctl. It creates the kernel

[PATCH v2 3/3] arm64: dts: exynos: Add support for s6e3hf2 panel device on TM2e board

2017-04-17 Thread Hoegeun Kwon
This patch add the panel device tree node for s6e3hf2 display controller to TM2e dts. Signed-off-by: Hoegeun Kwon --- arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts | 12 1 file changed, 12 insertions(+) diff --git

[PATCH 1/4] added SECURITY_TIOCSTI_RESTRICT kernel config

2017-04-17 Thread Matt Brown
adding the kernel config SECURITY_TIOCSTI_RESTRICT in order to allow the user to restrict unprivileged command injection using TIOCSTI tty ioctls Signed-off-by: Matt Brown --- security/Kconfig | 12 1 file changed, 12 insertions(+) diff --git a/security/Kconfig

[PATCH v2 3/3] arm64: dts: exynos: Add support for s6e3hf2 panel device on TM2e board

2017-04-17 Thread Hoegeun Kwon
This patch add the panel device tree node for s6e3hf2 display controller to TM2e dts. Signed-off-by: Hoegeun Kwon --- arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2e.dts

[PATCH 1/4] added SECURITY_TIOCSTI_RESTRICT kernel config

2017-04-17 Thread Matt Brown
adding the kernel config SECURITY_TIOCSTI_RESTRICT in order to allow the user to restrict unprivileged command injection using TIOCSTI tty ioctls Signed-off-by: Matt Brown --- security/Kconfig | 12 1 file changed, 12 insertions(+) diff --git a/security/Kconfig b/security/Kconfig

[PATCH 3/4] restrict unprivileged TIOCSTI tty ioctl

2017-04-17 Thread Matt Brown
this patch depends on patch 1 and 2 enforces restrictions on unprivileged users injecting commands into other processes in the same tty session using the TIOCSTI ioctl Signed-off-by: Matt Brown --- drivers/tty/tty_io.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 4/4] added kernel.tiocsti_restrict sysctl

2017-04-17 Thread Matt Brown
this patch depends on patches 1, 2 and 3 Allow CONFIG_SECURITY_TIOCSTI_RESTRICT to be controlled via sysctl Signed-off-by: Matt Brown --- kernel/sysctl.c | 12 1 file changed, 12 insertions(+) diff --git a/kernel/sysctl.c b/kernel/sysctl.c index acf0a5a..68d1363

[PATCH 3/4] restrict unprivileged TIOCSTI tty ioctl

2017-04-17 Thread Matt Brown
this patch depends on patch 1 and 2 enforces restrictions on unprivileged users injecting commands into other processes in the same tty session using the TIOCSTI ioctl Signed-off-by: Matt Brown --- drivers/tty/tty_io.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 4/4] added kernel.tiocsti_restrict sysctl

2017-04-17 Thread Matt Brown
this patch depends on patches 1, 2 and 3 Allow CONFIG_SECURITY_TIOCSTI_RESTRICT to be controlled via sysctl Signed-off-by: Matt Brown --- kernel/sysctl.c | 12 1 file changed, 12 insertions(+) diff --git a/kernel/sysctl.c b/kernel/sysctl.c index acf0a5a..68d1363 100644 ---

[PATCH V2 02/17] thermal: cpu_cooling: rearrange globals

2017-04-17 Thread Viresh Kumar
Just to make it look better. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index be29489dd247..ce94aafed25d 100644 ---

[PATCH V2 02/17] thermal: cpu_cooling: rearrange globals

2017-04-17 Thread Viresh Kumar
Just to make it look better. Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index be29489dd247..ce94aafed25d 100644 ---

<    5   6   7   8   9   10   11   >