Re: [PATCH] x86: fix typo(?) in step.c

From: Roland McGrath
Date: Thu Mar 06 2008 - 07:03:52 EST


> * Roland McGrath <roland@xxxxxxxxxx> wrote:
>
> > The bogon came in commit eee3af4a2c83a97fff107ddc445d9df6fded9ce4, the
> > introduction of the ptrace BTS stuff. Sorry I did not scour and cite
> > every problem in that patch, since I had NAK'd the entire thing as
> > needing more careful review and incremental introduction after 2.6.25.
>
> note that in -rc4 all those BTS ptrace extensions are disabled, see:

I know. That completely misses the point I just made:

As I said then, one of my concerns was with the low-level tweaks not yet
sufficiently baked, independent from my reservations about the ptrace
feature. Your #if'ing out of the user ABI additions for 2.6.25 does
nothing to remove the unknown new risks from all the tweaks with fingers in
the low-level arch stuff. This is the sort of thing I was concerned about.

You didn't revert the parts that ever could have caused problems for anyone
except those using the new ptrace extensions, i.e. changes to step.c,
context switch, whatever else was touched we've lost track of now. I keep
saying that those are not baked, 100% independent of the ptrace feature.
You don't seem to be hearing me.


Thanks,
Roland
--
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/