Re: [PATCH 01/27] ptrace: arch_has_single_step

From: Ingo Molnar
Date: Tue Nov 27 2007 - 05:39:20 EST



* Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:

> On Sun, 25 Nov 2007 13:55:07 -0800 (PST) Roland McGrath <roland@xxxxxxxxxx> 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.
>
> I think I'll duck this lot for now in view of the (relatively small)
> amount of followup.
>
> I did do an experimental will-it-apply and got a tremendous number of
> rejects against the x86 git tree, almost all of which went away when
> `patch -l' was used. Seems that someone has gone on a whitespace
> rampage through arch/x86/ia32/ptrace32.c and arch/x86/ia32/ptrace64.c.
>
> That's kinda/sorta OK, but `patch -l' is a bit more inclined to
> misplace hunks and it's all a bit risky.

actually, Thomas tried a tentative merge two days ago or so and the
result was pretty OK and functional. I'll try to do a more formal merge
and see what comes up.

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