lkml.org 
[lkml]   [2009]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 00/11] Hardware Breakpoint interfaces

* K.Prasad <prasad@linux.vnet.ibm.com> wrote:

> On Wed, Mar 11, 2009 at 12:34:43PM -0400, Alan Stern wrote:
> > On Wed, 11 Mar 2009, K.Prasad wrote:
> >
> > > The hardware breakpoint interfaces haven't been put under any CONFIG_
> > > till now, but I think we should bring them under a new config, say
> > > CONFIG_HW_BREAKPOINT. It would help create a dependancy for
> > > CONFIG_KSYM_TRACER too.
> >
> > With these patches, ptrace is dependent on hw-breakpoint.
> > You can't disable CONFIG_HW_BREAKPOINT without breaking
> > ptrace.
> >
> > Alan Stern
> >
>
> Agreed. We might have to retain the old code for ptrace and
> put the new implementation under #ifdef CONFIG_HW_BREAKPOINT
> to get them working. What do you think?

With the simple reservation mechanism i suggested i have no
problem with having HW_BREAKPOINT enabled [selected]
unconditionally on x86.

Your ptrace changes are an improvement in terms of code quality
so as long as the facility is simple and obvious, it's a step
forward.

#ifdefs are ugly and hard to maintain - especially in such a
rarely used and still critical API as ptrace.

Ingo


\
 
 \ /
  Last update: 2009-03-11 18:33    [W:0.082 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site