diff mbox

arm64: struct debug_info: Check CONFIG_HAVE_HW_BREAKPOINT

Message ID 20170316221043.GA29802@credmon.mdns.org (mailing list archive)
State New, archived
Headers show

Commit Message

Chris Redmon March 16, 2017, 10:10 p.m. UTC
Check if CONFIG_HAVE_HW_BREAKPOINT is enabled before compiling in extra
data required for hardware breakpoints. Compiling out this code when hw
breakpoints are disabled saves about 272 bytes per struct task_struct.

Signed-off-by: Chris Redmon <credmonster@gmail.com>
---
 arch/arm64/include/asm/processor.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Catalin Marinas March 21, 2017, 3:56 p.m. UTC | #1
On Thu, Mar 16, 2017 at 06:10:43PM -0400, Chris Redmon wrote:
> Check if CONFIG_HAVE_HW_BREAKPOINT is enabled before compiling in extra
> data required for hardware breakpoints. Compiling out this code when hw
> breakpoints are disabled saves about 272 bytes per struct task_struct.
> 
> Signed-off-by: Chris Redmon <credmonster@gmail.com>

Queued for 4.12. Thanks.
diff mbox

Patch

diff --git a/arch/arm64/include/asm/processor.h b/arch/arm64/include/asm/processor.h
index c97b8bd..9428b93 100644
--- a/arch/arm64/include/asm/processor.h
+++ b/arch/arm64/include/asm/processor.h
@@ -50,6 +50,7 @@ 
 #define ARCH_LOW_ADDRESS_LIMIT	(arm64_dma_phys_limit - 1)
 
 struct debug_info {
+#ifdef CONFIG_HAVE_HW_BREAKPOINT
 	/* Have we suspended stepping by a debugger? */
 	int			suspended_step;
 	/* Allow breakpoints and watchpoints to be disabled for this thread. */
@@ -58,6 +59,7 @@  struct debug_info {
 	/* Hardware breakpoints pinned to this task. */
 	struct perf_event	*hbp_break[ARM_MAX_BRP];
 	struct perf_event	*hbp_watch[ARM_MAX_WRP];
+#endif
 };
 
 struct cpu_context {