lkml.org 
[lkml]   [2007]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 01/27] ptrace: arch_has_single_step
    On Sun, Nov 25, 2007 at 01:55:07PM -0800, Roland McGrath wrote:
    > This defines the new macro arch_has_single_step() in linux/ptrace.h, a
    > default for when asm/ptrace.h does not define it. It declares the new
    > user_enable_single_step and user_disable_single_step functions.
    > This is not used yet, but paves the way to harmonize on this interface
    > for the arch-specific calls on all machines.

    Why should arch_has_single_step be a function-like macro? I can't thing
    of a case were this wouln't be a compile-time constant. And given that
    this is hopefully a transitionary ifdef because eventually all architectures
    would use the generic code I'd prefer ifdefs in the code that clearly mark
    this as transitional in this case.

    > +static inline void user_enable_single_step(struct task_struct *task)

    > +static inline void user_disable_single_step(struct task_struct *task)

    And I don't think these should be provided at all as generic stubs. If
    an arch doesn't use the generic code it simply shouldn't compile the
    code using this.

    Whats the reason for the user_ prefix btw, most architectures seems to
    have these functions already anyway, just without the user_ prefix.

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-11-25 23:25    [W:0.021 / U:188.108 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site