[tip:perf/core] perf script: Mark inlined frames and do not print DSO for them

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 9628b56dc1240ce0faa3bd9b7a3390fa4451c59f Gitweb: https://git.kernel.org/tip/9628b56dc1240ce0faa3bd9b7a3390fa4451c59f Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:02 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf script: Mark inlined frames and do not print DSO for them

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 9628b56dc1240ce0faa3bd9b7a3390fa4451c59f Gitweb: https://git.kernel.org/tip/9628b56dc1240ce0faa3bd9b7a3390fa4451c59f Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:02 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:56 -0300 perf script:

[tip:perf/core] perf report: Remove code to handle inline frames from browsers

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 2a704fc8db7b0080a67d9f4f4cb2a7bcaf79949d Gitweb: https://git.kernel.org/tip/2a704fc8db7b0080a67d9f4f4cb2a7bcaf79949d Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:55 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf report: Remove code to handle inline frames from browsers

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 2a704fc8db7b0080a67d9f4f4cb2a7bcaf79949d Gitweb: https://git.kernel.org/tip/2a704fc8db7b0080a67d9f4f4cb2a7bcaf79949d Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:55 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf report:

[tip:perf/core] perf callchain: Create real callchain entries for inlined frames

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 11ea2515f32e783b9a7984c148e742c377383915 Gitweb: https://git.kernel.org/tip/11ea2515f32e783b9a7984c148e742c377383915 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:59 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf callchain: Create real callchain entries for inlined frames

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 11ea2515f32e783b9a7984c148e742c377383915 Gitweb: https://git.kernel.org/tip/11ea2515f32e783b9a7984c148e742c377383915 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:59 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf

[tip:perf/core] perf callchain: Mark inlined frames in output by " (inlined)" suffix

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 8932f8071cae8a12dfd5f49224ee176b0da4 Gitweb: https://git.kernel.org/tip/8932f8071cae8a12dfd5f49224ee176b0da4 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:01 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf callchain: Mark inlined frames in output by " (inlined)" suffix

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 8932f8071cae8a12dfd5f49224ee176b0da4 Gitweb: https://git.kernel.org/tip/8932f8071cae8a12dfd5f49224ee176b0da4 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:01 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:56 -0300 perf

[tip:perf/core] perf callchain: Store srcline in callchain_cursor_node

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 40a342cda2cd9bc8f7bf81c5ce1a141584760757 Gitweb: https://git.kernel.org/tip/40a342cda2cd9bc8f7bf81c5ce1a141584760757 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:56 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf report: Fall-back to function name comparison for -g srcline

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: cbe50f61727f538f05e63186c2e0022182a3a28f Gitweb: https://git.kernel.org/tip/cbe50f61727f538f05e63186c2e0022182a3a28f Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:00 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf callchain: Store srcline in callchain_cursor_node

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 40a342cda2cd9bc8f7bf81c5ce1a141584760757 Gitweb: https://git.kernel.org/tip/40a342cda2cd9bc8f7bf81c5ce1a141584760757 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:56 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf

[tip:perf/core] perf report: Fall-back to function name comparison for -g srcline

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: cbe50f61727f538f05e63186c2e0022182a3a28f Gitweb: https://git.kernel.org/tip/cbe50f61727f538f05e63186c2e0022182a3a28f Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:33:00 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf report:

[tip:perf/core] perf callchain: Refactor inline_list to operate on symbols

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: fea0cf842c7aa08950063264ab1cfbce4ba38c1b Gitweb: https://git.kernel.org/tip/fea0cf842c7aa08950063264ab1cfbce4ba38c1b Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:57 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf callchain: Refactor inline_list to operate on symbols

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: fea0cf842c7aa08950063264ab1cfbce4ba38c1b Gitweb: https://git.kernel.org/tip/fea0cf842c7aa08950063264ab1cfbce4ba38c1b Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:57 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf

[tip:perf/core] perf callchain: Refactor inline_list to store srcline string directly

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 2be8832f3c51cf9e36a3e80ff57f4137505c2ba4 Gitweb: https://git.kernel.org/tip/2be8832f3c51cf9e36a3e80ff57f4137505c2ba4 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:58 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue,

[tip:perf/core] perf callchain: Refactor inline_list to store srcline string directly

2017-10-25 Thread tip-bot for Milian Wolff
Commit-ID: 2be8832f3c51cf9e36a3e80ff57f4137505c2ba4 Gitweb: https://git.kernel.org/tip/2be8832f3c51cf9e36a3e80ff57f4137505c2ba4 Author: Milian Wolff AuthorDate: Mon, 9 Oct 2017 22:32:58 +0200 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 24 Oct 2017 09:59:55 -0300 perf

Re: [RFC 10/14] bootsplash: Add animation support

2017-10-25 Thread Randy Dunlap
On 10/25/17 05:45, Max Staudt wrote: > @@ -117,17 +128,45 @@ static bool is_fb_compatible(struct fb_info *info) > */ > void bootsplash_render_full(struct fb_info *info) > { > + bool is_update = false; > + > mutex_lock(_global.data_lock); > > - if (!is_fb_compatible(info)) > -

Re: [RFC 10/14] bootsplash: Add animation support

2017-10-25 Thread Randy Dunlap
On 10/25/17 05:45, Max Staudt wrote: > @@ -117,17 +128,45 @@ static bool is_fb_compatible(struct fb_info *info) > */ > void bootsplash_render_full(struct fb_info *info) > { > + bool is_update = false; > + > mutex_lock(_global.data_lock); > > - if (!is_fb_compatible(info)) > -

Re: [PATCH v3 2/2] ima: check signature enforcement against cmdline param instead of CONFIG

2017-10-25 Thread Mimi Zohar
On Wed, 2017-10-25 at 13:05 -0200, Bruno E. O. Meneguele wrote: > On 24-10, Mimi Zohar wrote: > > On Tue, 2017-10-24 at 15:37 -0200, Bruno E. O. Meneguele wrote: > > > When the user requests MODULE_CHECK policy and its kernel is compiled > > > with CONFIG_MODULE_SIG_FORCE not set, all modules

Re: [PATCH v3 2/2] ima: check signature enforcement against cmdline param instead of CONFIG

2017-10-25 Thread Mimi Zohar
On Wed, 2017-10-25 at 13:05 -0200, Bruno E. O. Meneguele wrote: > On 24-10, Mimi Zohar wrote: > > On Tue, 2017-10-24 at 15:37 -0200, Bruno E. O. Meneguele wrote: > > > When the user requests MODULE_CHECK policy and its kernel is compiled > > > with CONFIG_MODULE_SIG_FORCE not set, all modules

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-25 Thread Dave Hansen
On 10/25/2017 02:30 AM, Michal Hocko wrote: >> >> 7f6c1780-7f6c17e0 rw-s 00:06 20559 /dev/dax12.0 >> Size: 6144 kB >> . >> . >> . >> Ptes@2MB:6144 kB > This says how but it doesn't tell why and who is going to use the > information and what for.

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-25 Thread Dave Hansen
On 10/25/2017 02:30 AM, Michal Hocko wrote: >> >> 7f6c1780-7f6c17e0 rw-s 00:06 20559 /dev/dax12.0 >> Size: 6144 kB >> . >> . >> . >> Ptes@2MB:6144 kB > This says how but it doesn't tell why and who is going to use the > information and what for.

[PATCH v2 1/9] irqchip: stm32: select GENERIC_IRQ_CHIP

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds GENERIC_IRQ_CHIP to stm32 exti config. Signed-off-by: Ludovic Barre --- drivers/irqchip/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig index

[PATCH v2 1/9] irqchip: stm32: select GENERIC_IRQ_CHIP

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds GENERIC_IRQ_CHIP to stm32 exti config. Signed-off-by: Ludovic Barre --- drivers/irqchip/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig index 9d8a1dd..c067aae 100644 --- a/drivers/irqchip/Kconfig

[PATCH v2 4/9] irqchip: stm32: add stm32h7 support

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre stm32h7 has up to 96 inputs (3 banks of 32 inputs max). Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 42 1 file changed, 42 insertions(+) diff --git

[PATCH v2 4/9] irqchip: stm32: add stm32h7 support

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre stm32h7 has up to 96 inputs (3 banks of 32 inputs max). Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 42 1 file changed, 42 insertions(+) diff --git a/drivers/irqchip/irq-stm32-exti.c

[PATCH v2 3/9] dt-bindings: interrupt-controllers: add compatible string for stm32h7

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch updates stm32-exti documentation with stm32h7-exti compatible string. Signed-off-by: Ludovic Barre --- .../devicetree/bindings/interrupt-controller/st,stm32-exti.txt| 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH v2 3/9] dt-bindings: interrupt-controllers: add compatible string for stm32h7

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch updates stm32-exti documentation with stm32h7-exti compatible string. Signed-off-by: Ludovic Barre --- .../devicetree/bindings/interrupt-controller/st,stm32-exti.txt| 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH v2 8/9] ARM: dts: stm32: add system config bank node for stm32h743

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds system config support for stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/boot/dts/stm32h743.dtsi

[PATCH v2 8/9] ARM: dts: stm32: add system config bank node for stm32h743

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds system config support for stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/boot/dts/stm32h743.dtsi b/arch/arm/boot/dts/stm32h743.dtsi index 6b2fb4c..b17aa5c 100644

[PATCH v2 9/9] ARM: dts: stm32: add support of exti on stm32h743 pinctrl

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds support of external interrupt (exti) on all gpio ports of stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743-pinctrl.dtsi | 24 1 file changed, 24 insertions(+) diff

[PATCH v2 9/9] ARM: dts: stm32: add support of exti on stm32h743 pinctrl

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds support of external interrupt (exti) on all gpio ports of stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743-pinctrl.dtsi | 24 1 file changed, 24 insertions(+) diff --git

[PATCH v2 5/9] irqchip: stm32: fix initial values

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre -After cold boot, imr default value depends on hardware configuration. -After hot reboot the registers must be cleared to avoid residue. Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 9 + 1 file

[PATCH v2 5/9] irqchip: stm32: fix initial values

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre -After cold boot, imr default value depends on hardware configuration. -After hot reboot the registers must be cleared to avoid residue. Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH v2 2/9] irqchip: stm32: add multi-bank management

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre -Prepare to manage multi-bank of external interrupts (N banks of 32 inputs). -Prepare to manage registers offsets by compatible (registers offsets could be different follow per stm32 platform). Signed-off-by: Ludovic Barre ---

[PATCH v2 2/9] irqchip: stm32: add multi-bank management

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre -Prepare to manage multi-bank of external interrupts (N banks of 32 inputs). -Prepare to manage registers offsets by compatible (registers offsets could be different follow per stm32 platform). Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 151

[PATCH v2 0/9] irqchip: stm32: add stm32h7 support

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This series adds: -Management of multi-bank of external interrupts stm32h7 has up to 96 inputs (3 banks of 32 inputs). -Fix initial value after cold/hot boot (wakeup issue). Changes v2: -Remove irq_mask and adds const on struct stm32_exti_bank -Add

[PATCH v2 0/9] irqchip: stm32: add stm32h7 support

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This series adds: -Management of multi-bank of external interrupts stm32h7 has up to 96 inputs (3 banks of 32 inputs). -Fix initial value after cold/hot boot (wakeup issue). Changes v2: -Remove irq_mask and adds const on struct stm32_exti_bank -Add wrapper functions pending

Re: [fstests PATCH v2] generic: add test for DAX MAP_SYNC support

2017-10-25 Thread Ross Zwisler
On Wed, Oct 25, 2017 at 03:19:22PM +0300, Amir Goldstein wrote: > On Sun, Oct 22, 2017 at 9:56 AM, Amir Goldstein wrote: > > On Sat, Oct 21, 2017 at 12:25 AM, Ross Zwisler > > wrote: > >> Add a test that exercises DAX's new MAP_SYNC flag. > >> >

Re: [fstests PATCH v2] generic: add test for DAX MAP_SYNC support

2017-10-25 Thread Ross Zwisler
On Wed, Oct 25, 2017 at 03:19:22PM +0300, Amir Goldstein wrote: > On Sun, Oct 22, 2017 at 9:56 AM, Amir Goldstein wrote: > > On Sat, Oct 21, 2017 at 12:25 AM, Ross Zwisler > > wrote: > >> Add a test that exercises DAX's new MAP_SYNC flag. > >> > >> This test creates a file and writes to it via

RE: [PATCH] wimax/i2400m: Remove VLAIS

2017-10-25 Thread Perez-Gonzalez, Inaky
>On Tue, 2017-10-24 at 21:00 +, Perez-Gonzalez, Inaky wrote: >> > ping any comments on this? >> >> None from me; I don't have access to this HW anymore, so I can't >> validate if the change would work or not. > I still have a 5350 around somewhere, I can make sure firmware loads if > you

RE: [PATCH] wimax/i2400m: Remove VLAIS

2017-10-25 Thread Perez-Gonzalez, Inaky
>On Tue, 2017-10-24 at 21:00 +, Perez-Gonzalez, Inaky wrote: >> > ping any comments on this? >> >> None from me; I don't have access to this HW anymore, so I can't >> validate if the change would work or not. > I still have a 5350 around somewhere, I can make sure firmware loads if > you

[PATCH v2 6/9] irqchip: stm32: move the wakeup on interrupt mask

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre Move irq_set_wake on interrupt mask, needed to wake up from low power mode as the event mask is not able to do so. Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 10 +- 1 file changed, 5

[PATCH v2 6/9] irqchip: stm32: move the wakeup on interrupt mask

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre Move irq_set_wake on interrupt mask, needed to wake up from low power mode as the event mask is not able to do so. Signed-off-by: Ludovic Barre --- drivers/irqchip/irq-stm32-exti.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH v2 7/9] ARM: dts: stm32: add exti support for stm32h743

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds support of external interrupt (exti) for stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/stm32h743.dtsi

[PATCH v2 7/9] ARM: dts: stm32: add exti support for stm32h743

2017-10-25 Thread Ludovic Barre
From: Ludovic Barre This patch adds support of external interrupt (exti) for stm32h743. Signed-off-by: Ludovic Barre --- arch/arm/boot/dts/stm32h743.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/stm32h743.dtsi b/arch/arm/boot/dts/stm32h743.dtsi index

Re: [GIT PULL 00/15] perf/core inlining improvements

2017-10-25 Thread Ingo Molnar
/git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core > (2017-10-24 10:53:04 +0200) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git > tags/perf-core-for-mingo-4.15-20171025 > > for you to fet

Re: [GIT PULL 00/15] perf/core inlining improvements

2017-10-25 Thread Ingo Molnar
cm/linux/kernel/git/acme/linux into perf/core > (2017-10-24 10:53:04 +0200) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git > tags/perf-core-for-mingo-4.15-20171025 > > for you to fetch changes up to d8a88d

Re: [PATCH v2 0/2] mtd: nand: wait for tWHR, and fix the setup_data_interface of Denali

2017-10-25 Thread Masahiro Yamada
Hi Marc, 2017-10-19 23:58 GMT+09:00 Marc Gonzalez : > On 13/10/2017 10:34, Masahiro Yamada wrote: > >> 2017-10-04 20:05, Marc Gonzalez wrote: >> >>> On 29/09/2017 16:33, Masahiro Yamada wrote: >>> tango_nand.c is the only driver that sets NAND_WAIT_TCCS.

Re: [PATCH v2 0/2] mtd: nand: wait for tWHR, and fix the setup_data_interface of Denali

2017-10-25 Thread Masahiro Yamada
Hi Marc, 2017-10-19 23:58 GMT+09:00 Marc Gonzalez : > On 13/10/2017 10:34, Masahiro Yamada wrote: > >> 2017-10-04 20:05, Marc Gonzalez wrote: >> >>> On 29/09/2017 16:33, Masahiro Yamada wrote: >>> tango_nand.c is the only driver that sets NAND_WAIT_TCCS. Now, there is completely no

Re: [PATCH] net: sunrpc: svcauth_gss: use BUG_ON instead of if condition followed by BUG

2017-10-25 Thread Jeff Layton
On Tue, 2017-10-24 at 15:12 -0500, Gustavo A. R. Silva wrote: > Quoting "J. Bruce Fields" : > > > On Tue, Oct 24, 2017 at 02:18:52PM -0400, Jeff Layton wrote: > > > On Tue, 2017-10-24 at 13:53 -0400, J. Bruce Fields wrote: > > > > On Tue, Oct 24, 2017 at 01:26:49PM -0400,

Re: [PATCH] net: sunrpc: svcauth_gss: use BUG_ON instead of if condition followed by BUG

2017-10-25 Thread Jeff Layton
On Tue, 2017-10-24 at 15:12 -0500, Gustavo A. R. Silva wrote: > Quoting "J. Bruce Fields" : > > > On Tue, Oct 24, 2017 at 02:18:52PM -0400, Jeff Layton wrote: > > > On Tue, 2017-10-24 at 13:53 -0400, J. Bruce Fields wrote: > > > > On Tue, Oct 24, 2017 at 01:26:49PM -0400, Weston Andros Adamson

Re: Detecting page cache trashing state

2017-10-25 Thread Daniel Walker
On 09/28/2017 08:49 AM, Ruslan Ruslichenko -X (rruslich - GLOBALLOGIC INC at Cisco) wrote: Hi Johannes, Hopefully I was able to rebase the patch on top v4.9.26 (latest supported version by us right now) and test a bit. The overall idea definitely looks promising, although I have one question

Re: Detecting page cache trashing state

2017-10-25 Thread Daniel Walker
On 09/28/2017 08:49 AM, Ruslan Ruslichenko -X (rruslich - GLOBALLOGIC INC at Cisco) wrote: Hi Johannes, Hopefully I was able to rebase the patch on top v4.9.26 (latest supported version by us right now) and test a bit. The overall idea definitely looks promising, although I have one question

Re: iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread SF Markus Elfring
> If that is the only unlock in the function, then it is probably > best to keep things as is. In general gotos are considered > better then multiple unlocks, but not having either is > even better. Thanks for your quick feedback. >> How do you think about to use the following code variant

Re: iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread SF Markus Elfring
> If that is the only unlock in the function, then it is probably > best to keep things as is. In general gotos are considered > better then multiple unlocks, but not having either is > even better. Thanks for your quick feedback. >> How do you think about to use the following code variant

Re: [PATCH 0/2] ARM-OMAP3: Fine-tuning for omap3xxx_hwmod_init()

2017-10-25 Thread Tony Lindgren
* SF Markus Elfring [171020 08:17]: > From: Markus Elfring > Date: Fri, 20 Oct 2017 17:08:09 +0200 > > Two update suggestions were taken into account > from static source code analysis. Thanks applying into

Re: [PATCH 0/2] ARM-OMAP3: Fine-tuning for omap3xxx_hwmod_init()

2017-10-25 Thread Tony Lindgren
* SF Markus Elfring [171020 08:17]: > From: Markus Elfring > Date: Fri, 20 Oct 2017 17:08:09 +0200 > > Two update suggestions were taken into account > from static source code analysis. Thanks applying into omap-for-v4.15/coccinelle. Regards, Tony

Re: [PATCH] kernel/kprobes: add check to avoid kprobe memory leak

2017-10-25 Thread Masami Hiramatsu
On Tue, 24 Oct 2017 20:17:02 +0800 JianKang Chen wrote: > The function register_kretprobe is used to initialize a struct > kretprobe and allocate a list table for kprobe instance. > However,in this function, there is a memory leak. > > The test case: > > static struct

Re: [PATCH] kernel/kprobes: add check to avoid kprobe memory leak

2017-10-25 Thread Masami Hiramatsu
On Tue, 24 Oct 2017 20:17:02 +0800 JianKang Chen wrote: > The function register_kretprobe is used to initialize a struct > kretprobe and allocate a list table for kprobe instance. > However,in this function, there is a memory leak. > > The test case: > > static struct kretprobe rp; > struct

Re: [PATCH 0/9] Enable dw-mmc multi-card support

2017-10-25 Thread Ulf Hansson
[...] > Unfortunate patch 8 and 9 does *not* solve the fundamental problem > with multiple slots. You can't solve it without involving the mmc > core, as explained in earlier discussions [1]. > This link got lost, so here it is: [1] https://www.spinics.net/lists/linux-mmc/msg46549.html [...]

Re: [PATCH 0/9] Enable dw-mmc multi-card support

2017-10-25 Thread Ulf Hansson
[...] > Unfortunate patch 8 and 9 does *not* solve the fundamental problem > with multiple slots. You can't solve it without involving the mmc > core, as explained in earlier discussions [1]. > This link got lost, so here it is: [1] https://www.spinics.net/lists/linux-mmc/msg46549.html [...]

Linux Foundation Technical Advisory Board election results 2017

2017-10-25 Thread H. Peter Anvin
The TAB elections are now complete. Thank you to all the candidates for putting their names forward. With 73 ballots cast, the top 5 candidates received: Steve Rostedt 58 Jonathan Corbet 57 Greg Kroah-Hartman 57 Tim Bird39 Ted Ts'o37 It was

Linux Foundation Technical Advisory Board election results 2017

2017-10-25 Thread H. Peter Anvin
The TAB elections are now complete. Thank you to all the candidates for putting their names forward. With 73 ballots cast, the top 5 candidates received: Steve Rostedt 58 Jonathan Corbet 57 Greg Kroah-Hartman 57 Tim Bird39 Ted Ts'o37 It was

Re: [PATCH 0/9] Enable dw-mmc multi-card support

2017-10-25 Thread Ulf Hansson
Hi Liming, Sorry for jumping into the discussion like this. [...] >> >> This series of commits enables the multi-card support for the dw-mmc >> >> controller. It includes two parts as below. >> >> >> >> The first part (patches 1-7) reverts the series of recent commits >> >> that removed the

Re: [PATCH 0/9] Enable dw-mmc multi-card support

2017-10-25 Thread Ulf Hansson
Hi Liming, Sorry for jumping into the discussion like this. [...] >> >> This series of commits enables the multi-card support for the dw-mmc >> >> controller. It includes two parts as below. >> >> >> >> The first part (patches 1-7) reverts the series of recent commits >> >> that removed the

Re: [PATCH] xen/gntdev: avoid out of bounds access in case of partial gntdev_mmap()

2017-10-25 Thread Boris Ostrovsky
On 10/25/2017 11:08 AM, Juergen Gross wrote: > In case gntdev_mmap() succeeds only partially in mapping grant pages > it will leave some vital information uninitialized needed later for > cleanup. This will lead to an out of bounds array access when unmapping > the already mapped pages. > > So

Re: [PATCH] xen/gntdev: avoid out of bounds access in case of partial gntdev_mmap()

2017-10-25 Thread Boris Ostrovsky
On 10/25/2017 11:08 AM, Juergen Gross wrote: > In case gntdev_mmap() succeeds only partially in mapping grant pages > it will leave some vital information uninitialized needed later for > cleanup. This will lead to an out of bounds array access when unmapping > the already mapped pages. > > So

Re: [PATCH] fs, mm: account filp and names caches to kmemcg

2017-10-25 Thread Johannes Weiner
On Wed, Oct 25, 2017 at 04:12:21PM +0200, Michal Hocko wrote: > On Wed 25-10-17 09:11:51, Johannes Weiner wrote: > > On Wed, Oct 25, 2017 at 09:15:22AM +0200, Michal Hocko wrote: > [...] > > > ... we shouldn't make it more loose though. > > > > Then we can end this discussion right now. I pointed

Re: [PATCH] fs, mm: account filp and names caches to kmemcg

2017-10-25 Thread Johannes Weiner
On Wed, Oct 25, 2017 at 04:12:21PM +0200, Michal Hocko wrote: > On Wed 25-10-17 09:11:51, Johannes Weiner wrote: > > On Wed, Oct 25, 2017 at 09:15:22AM +0200, Michal Hocko wrote: > [...] > > > ... we shouldn't make it more loose though. > > > > Then we can end this discussion right now. I pointed

Re: [PATCH] net: xfrm_user: use BUG_ON instead of if condition followed by BUG

2017-10-25 Thread Gustavo A. R. Silva
Hi all, Quoting Steffen Klassert : On Wed, Oct 25, 2017 at 01:22:22PM +0900, David Miller wrote: From: Herbert Xu Date: Wed, 25 Oct 2017 12:05:41 +0800 > On Tue, Oct 24, 2017 at 05:48:42PM +0900, David Miller wrote: >> >> This

Re: [PATCH] net: xfrm_user: use BUG_ON instead of if condition followed by BUG

2017-10-25 Thread Gustavo A. R. Silva
Hi all, Quoting Steffen Klassert : On Wed, Oct 25, 2017 at 01:22:22PM +0900, David Miller wrote: From: Herbert Xu Date: Wed, 25 Oct 2017 12:05:41 +0800 > On Tue, Oct 24, 2017 at 05:48:42PM +0900, David Miller wrote: >> >> This discussion has happened before. >> >> But I'll explain the

Re: [PATCH] Input: synaptics-rmi4: Limit the range of what GPIOs are buttons

2017-10-25 Thread Dmitry Torokhov
On Wed, Oct 25, 2017 at 03:37:35PM +0200, Benjamin Tissoires wrote: > On Oct 24 2017 or thereabouts, Andrew Duggan wrote: > > By convention the first 6 bits of F30 Ctrl 2 and 3 are used to signify > > I really do not like the "by convention". What if future firmware > developers are willing to

Re: [PATCH] Input: synaptics-rmi4: Limit the range of what GPIOs are buttons

2017-10-25 Thread Dmitry Torokhov
On Wed, Oct 25, 2017 at 03:37:35PM +0200, Benjamin Tissoires wrote: > On Oct 24 2017 or thereabouts, Andrew Duggan wrote: > > By convention the first 6 bits of F30 Ctrl 2 and 3 are used to signify > > I really do not like the "by convention". What if future firmware > developers are willing to

Re: [PATCH] PM / QoS: Fix device resume latency PM QoS

2017-10-25 Thread Rafael J. Wysocki
On Wed, Oct 25, 2017 at 9:27 AM, Ramesh Thomas wrote: > On 2017-10-24 at 13:23:23 +0200, Rafael J. Wysocki wrote: >> On Tue, Oct 24, 2017 at 10:49 AM, Rafael J. Wysocki >> wrote: >> > On Tuesday, October 24, 2017 7:54:09 AM CEST Ramesh Thomas wrote:

Re: [PATCH] PM / QoS: Fix device resume latency PM QoS

2017-10-25 Thread Rafael J. Wysocki
On Wed, Oct 25, 2017 at 9:27 AM, Ramesh Thomas wrote: > On 2017-10-24 at 13:23:23 +0200, Rafael J. Wysocki wrote: >> On Tue, Oct 24, 2017 at 10:49 AM, Rafael J. Wysocki >> wrote: >> > On Tuesday, October 24, 2017 7:54:09 AM CEST Ramesh Thomas wrote: >> >> On 2017-10-20 at 13:27:34 +0200, Rafael

[PATCH v2] tracing: Allocate mask_str buffer dynamically

2017-10-25 Thread changbin . du
From: Changbin Du The default NR_CPUS can be very large, but actual possible nr_cpu_ids usually is very small. For my x86 distribution, the NR_CPUS is 8192 and nr_cpu_ids is 4. About 2 pages are wasted. Most machines don't have so many CPUs, so define a array with NR_CPUS

[PATCH v2] tracing: Allocate mask_str buffer dynamically

2017-10-25 Thread changbin . du
From: Changbin Du The default NR_CPUS can be very large, but actual possible nr_cpu_ids usually is very small. For my x86 distribution, the NR_CPUS is 8192 and nr_cpu_ids is 4. About 2 pages are wasted. Most machines don't have so many CPUs, so define a array with NR_CPUS just wastes memory. So

Re: [PATCH] media: pvrusb2: Convert timers to use timer_setup()

2017-10-25 Thread Mike Isely
Ack'ed (separate formal reply) -Mike On Wed, 25 Oct 2017, Kees Cook wrote: > Eek, sorry, this uses timer_setup_on_stack() which is only in -next. > If you can Ack this, I can carry it in the timer tree. > > Thanks! > > -Kees > > On Tue, Oct 24, 2017 at 5:22 PM, Kees Cook

Re: [PATCH] media: pvrusb2: Convert timers to use timer_setup()

2017-10-25 Thread Mike Isely
Ack'ed (separate formal reply) -Mike On Wed, 25 Oct 2017, Kees Cook wrote: > Eek, sorry, this uses timer_setup_on_stack() which is only in -next. > If you can Ack this, I can carry it in the timer tree. > > Thanks! > > -Kees > > On Tue, Oct 24, 2017 at 5:22 PM, Kees Cook wrote: > > In

Re: [PATCH] drivers/net: netronome: Convert timers to use timer_setup()

2017-10-25 Thread Jakub Kicinski
On Wed, 25 Oct 2017 03:51:38 -0700, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Jakub Kicinski

Re: [PATCH] drivers/net: netronome: Convert timers to use timer_setup()

2017-10-25 Thread Jakub Kicinski
On Wed, 25 Oct 2017 03:51:38 -0700, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Jakub Kicinski > Cc: "David S. Miller"

Re: [PATCH] iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread Hans de Goede
Hi, On 25-10-17 18:15, SF Markus Elfring wrote: IMHO, if you do this, you should rework the function so that there is a single unlock call at the end, not a separate one in in error label. Thanks for your update suggestion. Does it indicate that I may propose similar source code adjustments

Re: [PATCH] iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread Hans de Goede
Hi, On 25-10-17 18:15, SF Markus Elfring wrote: IMHO, if you do this, you should rework the function so that there is a single unlock call at the end, not a separate one in in error label. Thanks for your update suggestion. Does it indicate that I may propose similar source code adjustments

Re: [PATCH] media: pvrusb2: Convert timers to use timer_setup()

2017-10-25 Thread Mike Isely
Acked-By: Mike Isely On Tue, 24 Oct 2017, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Mike Isely

Re: [PATCH] media: pvrusb2: Convert timers to use timer_setup()

2017-10-25 Thread Mike Isely
Acked-By: Mike Isely On Tue, 24 Oct 2017, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Mike Isely > Cc: Mauro

Re: [RFC PATCH 0/9] V4L2 Jobs API WIP

2017-10-25 Thread Hans Verkuil
On 10/25/2017 05:48 PM, Laurent Pinchart wrote: > Hello, > > On Monday, 23 October 2017 11:45:01 EEST Alexandre Courbot wrote: >> On Thu, Oct 19, 2017 at 11:43 PM, Sakari Ailus wrote: >>> On Thu, Sep 28, 2017 at 06:50:18PM +0900, Alexandre Courbot wrote: Hi everyone,

Re: [RFC PATCH 0/9] V4L2 Jobs API WIP

2017-10-25 Thread Hans Verkuil
On 10/25/2017 05:48 PM, Laurent Pinchart wrote: > Hello, > > On Monday, 23 October 2017 11:45:01 EEST Alexandre Courbot wrote: >> On Thu, Oct 19, 2017 at 11:43 PM, Sakari Ailus wrote: >>> On Thu, Sep 28, 2017 at 06:50:18PM +0900, Alexandre Courbot wrote: Hi everyone, Here is a new

Re: [PATCH] iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread SF Markus Elfring
> IMHO, if you do this, you should rework the function so that there is a > single unlock call > at the end, not a separate one in in error label. Thanks for your update suggestion. Does it indicate that I may propose similar source code adjustments in this software area? > Could e.g. change

Re: [PATCH] iio/accel/bmc150: Improve unlocking of a mutex in two functions

2017-10-25 Thread SF Markus Elfring
> IMHO, if you do this, you should rework the function so that there is a > single unlock call > at the end, not a separate one in in error label. Thanks for your update suggestion. Does it indicate that I may propose similar source code adjustments in this software area? > Could e.g. change

Re: [PATCH] Input: gtco - fix potential out-of-bound access

2017-10-25 Thread Dmitry Torokhov
On Wed, Oct 25, 2017 at 12:33:38PM +0200, Andrey Konovalov wrote: > On Tue, Oct 24, 2017 at 6:39 PM, Dmitry Torokhov > wrote: > > On Tue, Oct 24, 2017 at 01:04:03PM +0200, Andrey Konovalov wrote: > >> On Tue, Oct 24, 2017 at 7:28 AM, Dmitry Torokhov > >>

Re: [PATCH] Input: gtco - fix potential out-of-bound access

2017-10-25 Thread Dmitry Torokhov
On Wed, Oct 25, 2017 at 12:33:38PM +0200, Andrey Konovalov wrote: > On Tue, Oct 24, 2017 at 6:39 PM, Dmitry Torokhov > wrote: > > On Tue, Oct 24, 2017 at 01:04:03PM +0200, Andrey Konovalov wrote: > >> On Tue, Oct 24, 2017 at 7:28 AM, Dmitry Torokhov > >> wrote: > >> >

[PATCH 03/15] perf callchain: Refactor inline_list to operate on symbols

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is a requirement to create real callchain entries for inlined frames. Since the list of inlines usually contains the target symbol too, i.e. the location where the frames get inlined to, we alias that symbol and reuse it as-is is. This ensures that

[PATCH 03/15] perf callchain: Refactor inline_list to operate on symbols

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is a requirement to create real callchain entries for inlined frames. Since the list of inlines usually contains the target symbol too, i.e. the location where the frames get inlined to, we alias that symbol and reuse it as-is is. This ensures that other dependent

[PATCH 04/15] perf callchain: Refactor inline_list to store srcline string directly

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is a preparation for the creation of real callchain entries for inlined frames. The rest of the perf code uses the srcline string. As such, using that also for the srcline API allows us to simplify some of the upcoming code. Most notably, it will

[PATCH 07/15] perf callchain: Mark inlined frames in output by " (inlined)" suffix

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff The original patch that introduced inline frame output in the various browsers used this suffix already. The new centralized approach that uses fake symbols for inlined frames was missing this approach so far. Instead of changing the symbol name itself,

[PATCH 04/15] perf callchain: Refactor inline_list to store srcline string directly

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is a preparation for the creation of real callchain entries for inlined frames. The rest of the perf code uses the srcline string. As such, using that also for the srcline API allows us to simplify some of the upcoming code. Most notably, it will allow us to cache the

[PATCH 07/15] perf callchain: Mark inlined frames in output by " (inlined)" suffix

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff The original patch that introduced inline frame output in the various browsers used this suffix already. The new centralized approach that uses fake symbols for inlined frames was missing this approach so far. Instead of changing the symbol name itself, we only print the

[PATCH 02/15] perf callchain: Store srcline in callchain_cursor_node

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is mostly a preparation to enable the creation of full callchain nodes for inline frames. Such frames will reference the IP of the non-inlined frame, but hold the symbol and srcline for an inlined location. As such, we won't be able to query the

[PATCH 02/15] perf callchain: Store srcline in callchain_cursor_node

2017-10-25 Thread Arnaldo Carvalho de Melo
From: Milian Wolff This is mostly a preparation to enable the creation of full callchain nodes for inline frames. Such frames will reference the IP of the non-inlined frame, but hold the symbol and srcline for an inlined location. As such, we won't be able to query the srcline on-demand based on

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