On Tue, Aug 23, 2011 at 06:33:17PM +0100, Al Viro wrote:
>       * SYSCALL is not terminally broken wrt restarts.  My apologies for
> misreading what was going on.
>       * SYSENTER with Linus' patch does work just fine wrt restarts + ptrace
>       * SYSCALL is losing ptrace-made changes to arguments when it restarts.
> Might or might not be a problem for somebody.

BTW, that one (irrelevant to UML even if we do end up coping with SYSCALL
there) might be worth spelling it out:

tracer:
        ptrace(tracee, PTRACE_SYSCALL);
tracee:
        recvfrom(..., &addrlen);
tracer:
        ptrace(tracee, PTRACE_POKEUSER, EBP, &len2);
        ptrace(tracee, PTRACE_DETACH, 0, 0);
tracee:
        completes recvfrom(), using &len2 instead of the &addrlen

That works just fine, regardless of the way syscall is entered; yes, including
SYSCALL - there we take care to handle ptrace on the way in.  However, if it's
SYSCALL and (ex-)tracee takes a restart, the second time around we'll have the
original value of 6th argument used.  Changes made by POKEUSER are lost.

It's not a problem with int 0x80 or SYSENTER (now, with int 0x80 instead of
jmp).  It's probably not going to be a real issue for anyone, but I pity the
poor bastard stuck with debugging that if it *does* become someone's problem.

------------------------------------------------------------------------------
Get a FREE DOWNLOAD! and learn more about uberSVN rich system, 
user administration capabilities and model configuration. Take 
the hassle out of deploying and managing Subversion and the 
tools developers use with it. http://p.sf.net/sfu/wandisco-d2d-2
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Reply via email to