Re: [PATCH 4/6] hw_breakpoints: Breakpoints arch ability don't need perf events
On Tue, May 24, 2011 at 11:52:25PM +0200, Frederic Weisbecker wrote: The breakpoint support ability in an arch is not related to the fact perf events is built or not. HAVE_HW_BREAKPOINT only shows an ability so this dependency makes no sense anymore. Archs that select HAVE_HW_BREAKPOINT already ensure that perf event is built. Remove that dependency. Signed-off-by: Frederic Weisbecker fweis...@gmail.com Cc: Ingo Molnar mi...@elte.hu Cc: Peter Zijlstra a.p.zijls...@chello.nl Cc: Will Deacon will.dea...@arm.com Cc: Prasad pra...@linux.vnet.ibm.com Cc: Paul Mundt let...@linux-sh.org --- arch/Kconfig |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/Kconfig b/arch/Kconfig index f78c2be..ce4be89 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -149,7 +149,6 @@ config HAVE_DEFAULT_NO_SPIN_MUTEXES config HAVE_HW_BREAKPOINT bool - depends on PERF_EVENTS config HAVE_MIXED_BREAKPOINTS_REGS bool -- Just a thought you might want to consider... The need to keep the ability (HAVE_HW_BREAKPOINT) and the user-choice to enable hardware breakpoints (through HW_BREAKPOINT) in separate config options isn't very clear to me (and is a bit confusing with very similar names). Why not make HAVE_HW_BREAKPOINT selectable by the user (which in turn would turn on PERF_EVENTS) for a given architecture? Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 4/6] hw_breakpoints: Breakpoints arch ability don't need perf events
On Mon, Jul 04, 2011 at 07:02:23PM +0530, K.Prasad wrote: On Tue, May 24, 2011 at 11:52:25PM +0200, Frederic Weisbecker wrote: The breakpoint support ability in an arch is not related to the fact perf events is built or not. HAVE_HW_BREAKPOINT only shows an ability so this dependency makes no sense anymore. Archs that select HAVE_HW_BREAKPOINT already ensure that perf event is built. Remove that dependency. Signed-off-by: Frederic Weisbecker fweis...@gmail.com Cc: Ingo Molnar mi...@elte.hu Cc: Peter Zijlstra a.p.zijls...@chello.nl Cc: Will Deacon will.dea...@arm.com Cc: Prasad pra...@linux.vnet.ibm.com Cc: Paul Mundt let...@linux-sh.org --- arch/Kconfig |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/Kconfig b/arch/Kconfig index f78c2be..ce4be89 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -149,7 +149,6 @@ config HAVE_DEFAULT_NO_SPIN_MUTEXES config HAVE_HW_BREAKPOINT bool - depends on PERF_EVENTS config HAVE_MIXED_BREAKPOINTS_REGS bool -- Just a thought you might want to consider... The need to keep the ability (HAVE_HW_BREAKPOINT) and the user-choice to enable hardware breakpoints (through HW_BREAKPOINT) in separate config options isn't very clear to me (and is a bit confusing with very similar names). Why not make HAVE_HW_BREAKPOINT selectable by the user (which in turn would turn on PERF_EVENTS) for a given architecture? But then how kconfig knows if the allows that? You need to know if the arch has the ability to support breakpoints. This is a commin pattern in Linux Kconfig. Things are often seperated between ability (some constant value provided by the arch) and the user choice that depends on that ability. The advantage of doing this is that you can centralize the generic dependencies, help menu, etc... into a single place. Thanks, K.Prasad ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
[PATCH 4/6] hw_breakpoints: Breakpoints arch ability don't need perf events
The breakpoint support ability in an arch is not related to the fact perf events is built or not. HAVE_HW_BREAKPOINT only shows an ability so this dependency makes no sense anymore. Archs that select HAVE_HW_BREAKPOINT already ensure that perf event is built. Remove that dependency. Signed-off-by: Frederic Weisbecker fweis...@gmail.com Cc: Ingo Molnar mi...@elte.hu Cc: Peter Zijlstra a.p.zijls...@chello.nl Cc: Will Deacon will.dea...@arm.com Cc: Prasad pra...@linux.vnet.ibm.com Cc: Paul Mundt let...@linux-sh.org --- arch/Kconfig |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/Kconfig b/arch/Kconfig index f78c2be..ce4be89 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -149,7 +149,6 @@ config HAVE_DEFAULT_NO_SPIN_MUTEXES config HAVE_HW_BREAKPOINT bool - depends on PERF_EVENTS config HAVE_MIXED_BREAKPOINTS_REGS bool -- 1.7.3.2 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
