[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: ptrace single-stepping change breaks Wine
- From: Davide Libenzi <davidel@xxxxxxxxxxxxxxx>
- Date: Fri, 31 Dec 2004 07:16:00 -0800 (PST)
- References: <Pine.LNX.4.58.0412292050550.22893@ppc970.osdl.org> <Pine.LNX.4.58.0412292055540.22893@ppc970.osdl.org> <Pine.LNX.4.58.0412292106400.454@bigblue.dev.mdolabs.com> <Pine.LNX.4.58.0412292256350.22893@ppc970.osdl.org> <Pine.LNX.4.58.0412300953470.2193@bigblue.dev.mdolabs.com> <53046857041230112742acccbe@mail.gmail.com> <Pine.LNX.4.58.0412301130540.22893@ppc970.osdl.org> <Pine.LNX.4.58.0412301436330.22893@ppc970.osdl.org> <m1mzvvjs3k.fsf@muc.de> <Pine.LNX.4.58.0412301628580.2280@ppc970.osdl.org> <20041231123538.GA18209@muc.de>
On Fri, 31 Dec 2004, Andi Kleen wrote:
> > - you couldn't even debug signal handlers, because they were _really_
> > hard to get into unless you knew where they were and put a breakpoint
> > on them.
>
> Ok I see this as being a problem. But I bet it could be fixed
> much simpler without doing all this complicated and likely-to-be-buggy
> popf parsing you added.
I don't think that the Wine problem resolution is due to the POPF
instruction handling. Basically Linus patch does a nice cleanup plus POPF
handling, so maybe the patch can be split.
> > - you couldn't see the instruction after a system call.
>
> Are you sure?
Yes, this was true with 2.4. Than it has been fixed some time ago. But
handling that revealed a fragile handling of ptrace event delivery we had
in do_syscall_trace(). Part of the Linus patch tries to solve the fact
that on one side strace wants things to happen in a certain way, way that
seem to break Wine. I think Linus cleanup of the ptrace event delivery can
get strace, Wine and single-step-after-syscall right, w/out POPF handling.
Then you guys can flame each other over the POPF handling ;)
- Davide
-
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/
Follow-Ups: