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

From: Ingo Molnar
Date: Thu Mar 06 2008 - 06:34:41 EST



* Roland McGrath <roland@xxxxxxxxxx> wrote:

> The block-step test only tested that PTRACE_SINGLEBLOCK worked right.
> I just souped it up to also test that PTRACE_SINGLESTEP still works
> immediately afterwards. This still does not show any problem from
> this bug. The case that would be broken by it is rather more arcane.
> I haven't worked out the test case that fails with the bogon.

my interpretation of the bug would be that we fail to mask out the
block-step MSR bit [because we mask out bit 25 instead of bit 1], and
hence the bug would cause that MSR bit to stay enabled in other tasks
too.

So in theory the bug should manifest itself as block-step mode never
clearing itself, once activated. (but this would never leak into other
tasks because we've got the thread.debugctlmsr abstraction that protects
them)

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/