[PATCH 06/12] arm64: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit. Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc

[PATCH 09/12] xtensa: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc

[PATCH 12/12] perf/breakpoint: Clean up and consolidate modify_user_hw_breakpoint_check()

Remove the dance around old and new attributes. Just don't modify the previous breakpoint at all until we have verified everything. Reported-by: Linus Torvalds <torva...@linux-foundation.org> Original-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederic Weisb

[PATCH 10/12] perf/breakpoint: Remove default hw_breakpoint_arch_parse()

All architectures have implemented it, we can now remove the poor weak version. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp&

[PATCH 08/12] sh: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org>

[PATCH 12/12] perf/breakpoint: Clean up and consolidate modify_user_hw_breakpoint_check()

Remove the dance around old and new attributes. Just don't modify the previous breakpoint at all until we have verified everything. Reported-by: Linus Torvalds Original-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc

[PATCH 10/12] perf/breakpoint: Remove default hw_breakpoint_arch_parse()

All architectures have implemented it, we can now remove the poor weak version. Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris

[PATCH 08/12] sh: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc

[PATCH 11/12] perf/breakpoint: Pass new breakpoint type to modify_breakpoint_slot()

ing breakpoint before we actually know if the modification will be validated. In order to prepare for that, allow modify_breakpoint_slot() to take the new breakpoint type. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: And

[PATCH 11/12] perf/breakpoint: Pass new breakpoint type to modify_breakpoint_slot()

ing breakpoint before we actually know if the modification will be validated. In order to prepare for that, allow modify_breakpoint_slot() to take the new breakpoint type. Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Mol

[PATCH 03/12] x86: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit. Original-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-fou

[PATCH 02/12] perf/breakpoint: Pass arch breakpoint struct to arch_check_bp_in_kernelspace()

the probing struct instead. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar

[PATCH 03/12] x86: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit. Original-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo

[PATCH 02/12] perf/breakpoint: Pass arch breakpoint struct to arch_check_bp_in_kernelspace()

the probing struct instead. Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc

[PATCH 04/12] powerpc: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org>

[PATCH 07/12] sh: Remove "struct arch_hw_breakpoint::name" unused field

This field seem to be unused, perhaps a leftover from old code... Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Fel

[PATCH 04/12] powerpc: Implement hw_breakpoint_arch_parse()

Migrate to the new API in order to remove arch_validate_hwbkpt_settings() that clumsily mixes up architecture validation and commit Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc

[PATCH 07/12] sh: Remove "struct arch_hw_breakpoint::name" unused field

This field seem to be unused, perhaps a leftover from old code... Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc

[PATCH 01/12] perf/breakpoint: Split attribute parse and commit

then need to implement the new API. Reported-by: Linus Torvalds <torva...@linux-foundation.org> Original-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirs

[PATCH 01/12] perf/breakpoint: Split attribute parse and commit

then need to implement the new API. Reported-by: Linus Torvalds Original-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Fi

[PATCH 00/12] breakpoint: Rework arch validation v2

attributes progresses. If something goes wrong in the middle, the architecture data changes aren't rolled back and we are left with a halfway fiddled breakpoint. This set fixes the various misdesigns that back this bad behaviour. Thanks, Frederic --- Frederic Weisbecker (12): perf

[PATCH 00/12] breakpoint: Rework arch validation v2

attributes progresses. If something goes wrong in the middle, the architecture data changes aren't rolled back and we are left with a halfway fiddled breakpoint. This set fixes the various misdesigns that back this bad behaviour. Thanks, Frederic --- Frederic Weisbecker (12): perf

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Tue, May 15, 2018 at 09:58:03PM -0700, Andy Lutomirski wrote: > > > > On May 15, 2018, at 8:11 PM, Frederic Weisbecker <frede...@kernel.org> > > wrote: > > > >> On Wed, May 09, 2018 at 11:17:03AM +0200, Peter Zijlstra wrote: > >>>

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Tue, May 15, 2018 at 09:58:03PM -0700, Andy Lutomirski wrote: > > > > On May 15, 2018, at 8:11 PM, Frederic Weisbecker > > wrote: > > > >> On Wed, May 09, 2018 at 11:17:03AM +0200, Peter Zijlstra wrote: > >>> On Sun, May 06, 2018 at 09:19:54PM

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Wed, May 09, 2018 at 11:17:03AM +0200, Peter Zijlstra wrote: > On Sun, May 06, 2018 at 09:19:54PM +0200, Frederic Weisbecker wrote: > > arch/arm/include/asm/hw_breakpoint.h | 5 - > > arch/arm/kernel/hw_breakpoint.c | 22 +++--- > > a

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Wed, May 09, 2018 at 11:17:03AM +0200, Peter Zijlstra wrote: > On Sun, May 06, 2018 at 09:19:54PM +0200, Frederic Weisbecker wrote: > > arch/arm/include/asm/hw_breakpoint.h | 5 - > > arch/arm/kernel/hw_breakpoint.c | 22 +++--- > > a

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Tue, May 15, 2018 at 08:57:47AM +0200, Ingo Molnar wrote: > > * Peter Zijlstra <pet...@infradead.org> wrote: > > > On Sun, May 06, 2018 at 09:19:54PM +0200, Frederic Weisbecker wrote: > > > arch/arm/include/asm/hw_breakpoint.h | 5 - > >

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Tue, May 15, 2018 at 08:57:47AM +0200, Ingo Molnar wrote: > > * Peter Zijlstra wrote: > > > On Sun, May 06, 2018 at 09:19:54PM +0200, Frederic Weisbecker wrote: > > > arch/arm/include/asm/hw_breakpoint.h | 5 - > > > arch/arm/ker

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Mon, May 07, 2018 at 12:46:06AM +, Joel Fernandes wrote: > On Sun, May 6, 2018 at 12:22 PM Frederic Weisbecker <frede...@kernel.org> > wrote: > > > arch_validate_hwbkpt_settings() mixes up attribute check and commit into > > a single code entity. Therefor

Re: [PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

On Mon, May 07, 2018 at 12:46:06AM +, Joel Fernandes wrote: > On Sun, May 6, 2018 at 12:22 PM Frederic Weisbecker > wrote: > > > arch_validate_hwbkpt_settings() mixes up attribute check and commit into > > a single code entity. Therefore the validation ma

Re: [PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

On Wed, May 09, 2018 at 12:32:57PM +0100, Mark Rutland wrote: > On Tue, May 08, 2018 at 12:13:23PM +0100, Mark Rutland wrote: > > Hi Frederick, > > > > On Sun, May 06, 2018 at 09:19:50PM +0200, Frederic Weisbecker wrote: > > > The breakpoint code mixes up attribute

Re: [PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

On Wed, May 09, 2018 at 12:32:57PM +0100, Mark Rutland wrote: > On Tue, May 08, 2018 at 12:13:23PM +0100, Mark Rutland wrote: > > Hi Frederick, > > > > On Sun, May 06, 2018 at 09:19:50PM +0200, Frederic Weisbecker wrote: > > > The breakpoint code mixes up attribute

[tip:irq/core] softirq/sh: Use nmi_count() on /proc/interrupts print out

Commit-ID: 30d723340c30ff9afe200ef5ecbbdc77e6d1f816 Gitweb: https://git.kernel.org/tip/30d723340c30ff9afe200ef5ecbbdc77e6d1f816 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:18 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/sh: Use nmi_count() on /proc/interrupts print out

Commit-ID: 30d723340c30ff9afe200ef5ecbbdc77e6d1f816 Gitweb: https://git.kernel.org/tip/30d723340c30ff9afe200ef5ecbbdc77e6d1f816 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:18 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:27 +0200 softirq/sh: Use

[tip:irq/core] softirq/s390: Move default mutators of overwritten softirq mask to s390

Commit-ID: 48bda43eabb8d086204f543cf8bbad696b8c6391 Gitweb: https://git.kernel.org/tip/48bda43eabb8d086204f543cf8bbad696b8c6391 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:26 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/s390: Move default mutators of overwritten softirq mask to s390

Commit-ID: 48bda43eabb8d086204f543cf8bbad696b8c6391 Gitweb: https://git.kernel.org/tip/48bda43eabb8d086204f543cf8bbad696b8c6391 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:26 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:28 +0200 softirq/s390: Move

[tip:irq/core] softirq/x86: Switch to generic local_softirq_pending() implementation

Commit-ID: 1a8bc8f8d6a7980a75edbd29578fbce09359 Gitweb: https://git.kernel.org/tip/1a8bc8f8d6a7980a75edbd29578fbce09359 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:25 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/x86: Switch to generic local_softirq_pending() implementation

Commit-ID: 1a8bc8f8d6a7980a75edbd29578fbce09359 Gitweb: https://git.kernel.org/tip/1a8bc8f8d6a7980a75edbd29578fbce09359 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:25 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:28 +0200 softirq/x86: Switch

[tip:irq/core] softirq/sparc: Switch to generic local_softirq_pending() implementation

Commit-ID: 424f7d3e3b950c88a4127b7dfa78ea54e287413e Gitweb: https://git.kernel.org/tip/424f7d3e3b950c88a4127b7dfa78ea54e287413e Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:24 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/sparc: Switch to generic local_softirq_pending() implementation

Commit-ID: 424f7d3e3b950c88a4127b7dfa78ea54e287413e Gitweb: https://git.kernel.org/tip/424f7d3e3b950c88a4127b7dfa78ea54e287413e Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:24 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:28 +0200 softirq/sparc

[tip:irq/core] softirq/powerpc: Switch to generic local_softirq_pending() implementation

Commit-ID: 1321a5de1ecb0d2981394ff2111c75c4dcb0c237 Gitweb: https://git.kernel.org/tip/1321a5de1ecb0d2981394ff2111c75c4dcb0c237 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:23 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/powerpc: Switch to generic local_softirq_pending() implementation

Commit-ID: 1321a5de1ecb0d2981394ff2111c75c4dcb0c237 Gitweb: https://git.kernel.org/tip/1321a5de1ecb0d2981394ff2111c75c4dcb0c237 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:23 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:28 +0200 softirq/powerpc

[tip:irq/core] softirq/parisc: Switch to generic local_softirq_pending() implementation

Commit-ID: 03979f8a72e6576248e7b9e3abb72a760312dd7d Gitweb: https://git.kernel.org/tip/03979f8a72e6576248e7b9e3abb72a760312dd7d Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:22 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/parisc: Switch to generic local_softirq_pending() implementation

Commit-ID: 03979f8a72e6576248e7b9e3abb72a760312dd7d Gitweb: https://git.kernel.org/tip/03979f8a72e6576248e7b9e3abb72a760312dd7d Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:22 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:27 +0200 softirq/parisc

[tip:irq/core] softirq/ia64: Switch to generic local_softirq_pending() implementation

Commit-ID: a58bdf25b98bf765b4b732f2c56097ddcb9f2d5a Gitweb: https://git.kernel.org/tip/a58bdf25b98bf765b4b732f2c56097ddcb9f2d5a Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:21 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/ia64: Switch to generic local_softirq_pending() implementation

Commit-ID: a58bdf25b98bf765b4b732f2c56097ddcb9f2d5a Gitweb: https://git.kernel.org/tip/a58bdf25b98bf765b4b732f2c56097ddcb9f2d5a Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:21 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:27 +0200 softirq/ia64

[tip:irq/core] softirq/core: Consolidate default local_softirq_pending() implementations

Commit-ID: 0fd7d86285290ccebc0dc6eb536b6b043dd6a1e4 Gitweb: https://git.kernel.org/tip/0fd7d86285290ccebc0dc6eb536b6b043dd6a1e4 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:20 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/core: Consolidate default local_softirq_pending() implementations

Commit-ID: 0fd7d86285290ccebc0dc6eb536b6b043dd6a1e4 Gitweb: https://git.kernel.org/tip/0fd7d86285290ccebc0dc6eb536b6b043dd6a1e4 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:20 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:27 +0200 softirq/core

[tip:irq/core] softirq/core: Turn default irq_cpustat_t to standard per-cpu

Commit-ID: 0f6f47bacba514f4e9f61de0d85940dfb41498cc Gitweb: https://git.kernel.org/tip/0f6f47bacba514f4e9f61de0d85940dfb41498cc Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:19 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/core: Turn default irq_cpustat_t to standard per-cpu

Commit-ID: 0f6f47bacba514f4e9f61de0d85940dfb41498cc Gitweb: https://git.kernel.org/tip/0f6f47bacba514f4e9f61de0d85940dfb41498cc Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:19 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:27 +0200 softirq/core: Turn

[tip:irq/core] softirq/sparc: Convert local_softirq_pending() to use per-cpu op

Commit-ID: 49892dbc2cb349f78eccfc1f55eac0ec718f44bb Gitweb: https://git.kernel.org/tip/49892dbc2cb349f78eccfc1f55eac0ec718f44bb Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:17 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/sparc: Convert local_softirq_pending() to use per-cpu op

Commit-ID: 49892dbc2cb349f78eccfc1f55eac0ec718f44bb Gitweb: https://git.kernel.org/tip/49892dbc2cb349f78eccfc1f55eac0ec718f44bb Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:17 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:26 +0200 softirq/sparc

[tip:irq/core] softirq/ia64: Convert local_softirq_pending() to per-cpu ops

Commit-ID: 2e5c4632dcc0365a97c36817a368507e6a4c89b2 Gitweb: https://git.kernel.org/tip/2e5c4632dcc0365a97c36817a368507e6a4c89b2 Author: Frederic Weisbecker <frede...@kernel.org> AuthorDate: Tue, 8 May 2018 15:38:16 +0200 Committer: Ingo Molnar <mi...@kernel.org> CommitDate:

[tip:irq/core] softirq/ia64: Convert local_softirq_pending() to per-cpu ops

Commit-ID: 2e5c4632dcc0365a97c36817a368507e6a4c89b2 Gitweb: https://git.kernel.org/tip/2e5c4632dcc0365a97c36817a368507e6a4c89b2 Author: Frederic Weisbecker AuthorDate: Tue, 8 May 2018 15:38:16 +0200 Committer: Ingo Molnar CommitDate: Mon, 14 May 2018 11:25:26 +0200 softirq/ia64

Re: [PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

On Wed, May 09, 2018 at 07:51:28PM +, Andy Lutomirski wrote: > On Wed, May 9, 2018 at 4:33 AM Mark Rutland <mark.rutl...@arm.com> wrote: > > > On Tue, May 08, 2018 at 12:13:23PM +0100, Mark Rutland wrote: > > > Hi Frederick, > > > > > > On Su

Re: [PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

On Wed, May 09, 2018 at 07:51:28PM +, Andy Lutomirski wrote: > On Wed, May 9, 2018 at 4:33 AM Mark Rutland wrote: > > > On Tue, May 08, 2018 at 12:13:23PM +0100, Mark Rutland wrote: > > > Hi Frederick, > > > > > > On Sun, May 06, 2018 at 09:1

[GIT PULL] softirq: Consolidate and optimize softirq mask v4

The actual subject is "softirq: Consolidate and optimize softirq mask v4" :-) Sorry...

[GIT PULL] softirq: Consolidate and optimize softirq mask v4

The actual subject is "softirq: Consolidate and optimize softirq mask v4" :-) Sorry...

[PATCH 03/11] sh: Use nmi_count() on /proc/interrupts print out

Use nmi_count() instead of accessing directly the irq_stat structure. Its implementation is going to change to use per-CPU, so defer the guts to standard API instead. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Z

[PATCH 03/11] sh: Use nmi_count() on /proc/interrupts print out

Use nmi_count() instead of accessing directly the irq_stat structure. Its implementation is going to change to use per-CPU, so defer the guts to standard API instead. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc

[PATCH 02/11] sparc: Convert local_softirq_pending() to use per-cpu op

In order to consolidate and optimize generic softirq mask accesses, we first need to convert architectures to use per-cpu operations when possible. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradea

[PATCH 02/11] sparc: Convert local_softirq_pending() to use per-cpu op

In order to consolidate and optimize generic softirq mask accesses, we first need to convert architectures to use per-cpu operations when possible. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc

[PATCH 05/11] softirq: Consolidate default local_softirq_pending() implementations

Consolidate and optimize default softirq mask API implementations. Per-CPU operations are expected to be faster and a few architectures already rely on them to implement local_softirq_pending() and related accessors/mutators. Those will be migrated to the new generic code. Signed-off-by: Frederic

[PATCH 05/11] softirq: Consolidate default local_softirq_pending() implementations

Consolidate and optimize default softirq mask API implementations. Per-CPU operations are expected to be faster and a few architectures already rely on them to implement local_softirq_pending() and related accessors/mutators. Those will be migrated to the new generic code. Signed-off-by: Frederic

[PATCH 06/11] ia64: Switch to generic local_softirq_pending() implementation

Benefit from the generic softirq mask implementation that rely on per-CPU mutators instead of working with raw operators on top of this_cpu_ptr(). Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradea

[PATCH 06/11] ia64: Switch to generic local_softirq_pending() implementation

Benefit from the generic softirq mask implementation that rely on per-CPU mutators instead of working with raw operators on top of this_cpu_ptr(). Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc

[PATCH 11/11] softirq/s390: Move default mutators of overwritten softirq mask to s390

org> Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Sebastian Andrzej Siewior <bige...@linutronix.de> Cc: David S. Miller <da...@davem

[PATCH 10/11] x86: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: S

[PATCH 11/11] softirq/s390: Move default mutators of overwritten softirq mask to s390

s390 is now the last architecture that entirely overwrites local_softirq_pending() and uses the according default definitions of set_softirq_pending() and or_softirq_pending(). Just move these to s390 to debloat the generic code complexity. Suggested-by: Peter Zijlstra Signed-off-by: Frederic

[PATCH 10/11] x86: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael

[PATCH 08/11] powerpc: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Acked-by: Michael Ellerman <m...@ellerman.id.au> Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradead

[PATCH 07/11] parisc: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: S

[PATCH 08/11] powerpc: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Acked-by: Michael Ellerman Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc: Benjamin Herrenschmidt Cc

[PATCH 07/11] parisc: Switch to generic local_softirq_pending() implementation

Remove the ad-hoc implementation, the generic code now allows us not to reinvent the wheel. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael

[PATCH 09/11] sparc: Switch to generic local_softirq_pending() implementation

Benefit from the generic softirq mask implementation that rely on per-CPU mutators instead of working with raw operators on top of this_cpu_ptr(). Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradea

[PATCH 09/11] sparc: Switch to generic local_softirq_pending() implementation

Benefit from the generic softirq mask implementation that rely on per-CPU mutators instead of working with raw operators on top of this_cpu_ptr(). Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc

[PATCH 04/11] softirq: Turn default irq_cpustat_t to standard per-cpu

In order to optimize and consolidate softirq mask accesses, let's convert the default irq_cpustat_t implementation to per-CPU standard API. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradead.org&g

[PATCH 04/11] softirq: Turn default irq_cpustat_t to standard per-cpu

In order to optimize and consolidate softirq mask accesses, let's convert the default irq_cpustat_t implementation to per-CPU standard API. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc

[PATCH 01/11] ia64: Convert local_softirq_pending() to per-cpu ops

In order to consolidate and optimize generic softirq mask accesses, we first need to convert architectures to use per-cpu operations when possible. Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Peter Zijlstra <pet...@infradea

[PATCH 01/11] ia64: Convert local_softirq_pending() to per-cpu ops

In order to consolidate and optimize generic softirq mask accesses, we first need to convert architectures to use per-cpu operations when possible. Signed-off-by: Frederic Weisbecker Cc: Thomas Gleixner Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Sebastian Andrzej Siewior Cc: David S. Miller Cc

[GIT PULL]

an provide instead a generic efficient version that most of them can use. In fact s390 is the only exception because the field is stored in lowcore. Thanks, Frederic --- Frederic Weisbecker (11): ia64: Convert local_softirq_pending() to per-cpu ops sparc: Convert local_softi

[GIT PULL]

an provide instead a generic efficient version that most of them can use. In fact s390 is the only exception because the field is stored in lowcore. Thanks, Frederic --- Frederic Weisbecker (11): ia64: Convert local_softirq_pending() to per-cpu ops sparc: Convert local_softi

[PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

this misdesigned function and call directly the arch check and commit functions instead. This allows us to later avoid commiting a breakpoint to architecture when its slot couldn't be allocated. Original-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederic Weisbecker

[PATCH 8/9] perf/breakpoint: Split breakpoint "check" and "commit"

this misdesigned function and call directly the arch check and commit functions instead. This allows us to later avoid commiting a breakpoint to architecture when its slot couldn't be allocated. Original-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy

[PATCH 1/9] x86/breakpoint: Split validation into "check" and "commit"

-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal

[PATCH 1/9] x86/breakpoint: Split validation into "check" and "commit"

-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc

[PATCH 5/9] xtensa: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Tho

[PATCH 5/9] xtensa: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman

[PATCH 9/9] perf/breakpoint: Only commit breakpoint to arch upon slot reservation success

the attribute check and commit code, we can avoid commiting a breakpoint to the architecture until its slot reservation has been accepted and completed. Reported-by: Linus Torvalds <torva...@linux-foundation.org> Original-patch-by: Andy Lutomirski <l...@kernel.org> Signed-off-by: Frederi

[PATCH 9/9] perf/breakpoint: Only commit breakpoint to arch upon slot reservation success

the attribute check and commit code, we can avoid commiting a breakpoint to the architecture until its slot reservation has been accepted and completed. Reported-by: Linus Torvalds Original-patch-by: Andy Lutomirski Signed-off-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc

[PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Tho

[PATCH 0/9] breakpoint: Rework arch validation

/linux/kernel/git/frederic/linux-dynticks.git perf/breakpoint HEAD: 0a788eff09988e58708c4a3ce7b8441ce3e7df95 Thanks, Frederic --- Frederic Weisbecker (9): x86/breakpoint: Split validation into "check" and "commit" sh: Remove "struct arch_hw_break

[PATCH 4/9] arm: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman

[PATCH 0/9] breakpoint: Rework arch validation

/linux/kernel/git/frederic/linux-dynticks.git perf/breakpoint HEAD: 0a788eff09988e58708c4a3ce7b8441ce3e7df95 Thanks, Frederic --- Frederic Weisbecker (9): x86/breakpoint: Split validation into "check" and "commit" sh: Remove "struct arch_hw_break

[PATCH 7/9] powerpc: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Tho

[PATCH 6/9] arm64: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Tho

[PATCH 7/9] powerpc: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman

[PATCH 6/9] arm64: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker Cc: Linus Torvalds Cc: Andy Lutomirski Cc: Yoshinori Sato Cc: Rich Felker Cc: Ingo Molnar Cc: Thomas Gleixner Cc: Will Deacon Cc: Mark Rutland Cc: Max Filippov Cc: Chris Zankel Cc: Catalin Marinas Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman

[PATCH 3/9] sh: Split breakpoint validation into "check" and "commit"

-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Felker <dal...@libc.org> Cc: Ingo Molnar <mi...@kernel.org> Cc: Tho

[PATCH 2/9] sh: Remove "struct arch_hw_breakpoint::name" unused field

This field seem to be unused, perhaps a leftover from old code... Signed-off-by: Frederic Weisbecker <frede...@kernel.org> Cc: Linus Torvalds <torva...@linux-foundation.org> Cc: Andy Lutomirski <l...@kernel.org> Cc: Yoshinori Sato <ys...@users.sourceforge.jp> Cc: Rich Fel

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