linux-um archives
 help / color / mirror / Atom feed
* [uml-devel] Please test 2.6.13-rc2 in TT mode! Big problem with syscall parameters clobbering
@ 2005-07-07 18:31 Blaisorblade
  2005-07-11 21:51 ` Blaisorblade
  0 siblings, 1 reply; 2+ messages in thread
From: Blaisorblade @ 2005-07-07 18:31 UTC (permalink / raw)
  To: user-mode-linux-devel, Bodo Stroesser, Jeff Dike
  Cc: user-mode-linux-user, Alexander Viro

[-- Attachment #1: Type: text/plain, Size: 2963 bytes --]

For this kernel (and for -rc1), I've been getting almost no reports (except a 
problem with GCC 2.95 I now solved) in TT mode, so I'd like some more 
testing, especially for users which have the latest SKAS patch installed. At 
least positive / negative reports.

But, for me, it hasn't been working at all, and fails with:

./vmlinux mode=tt
Checking PROT_EXEC mmap in /tmp...OK
switcheroo failed

, unless I apply this apparently 
silly patch:

diff -u linux-2.6.git-paolo/arch/um/sys-i386/unmap.c 
clean-linux-2.6.11-paolo/arch/um/sys-i386/unmap.c
--- linux-2.6.git-paolo/arch/um/sys-i386/unmap.c	2005-06-07 19:10:51.000000000 
+0200
+++ clean-linux-2.6.11-paolo/arch/um/sys-i386/unmap.c	2005-06-08 
20:01:21.000000000 +0200
@@ -15,7 +15,8 @@
 	if(munmap(to, size) < 0){
 		return(-1);
 	}
-	if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){
+	/* if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){ */
+	if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) == (void*) -1 ){
 		return(-1);
 	}
 	if(munmap(from, size) < 0){
diff -u linux-2.6.git-paolo/arch/um/sys-x86_64/unmap.c 
clean-linux-2.6.11-paolo/arch/um/sys-x86_64/unmap.c
--- linux-2.6.git-paolo/arch/um/sys-x86_64/unmap.c	2005-06-07 
19:10:51.000000000 +0200
+++ clean-linux-2.6.11-paolo/arch/um/sys-x86_64/unmap.c	2005-06-08 
20:01:18.000000000 +0200
@@ -15,7 +15,8 @@
 	if(munmap(to, size) < 0){
 		return(-1);
 	}
-	if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){
+	/* if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){ */
+	if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) == (void*) -1){
 		return(-1);
 	}
 	if(munmap(from, size) < 0){

It is silly because from the mmap2() API, kernel source, and so on, it will 
return either "to" or -1, and from strace output it always returns "to".

mmap2(0xa024f000, 532480, PROT_READ|PROT_WRITE|PROT_EXEC, MAP_SHARED|
MAP_FIXED, 3, 0) = 0xa024f000
//as you can see the return value is the first parameter!
write(1, "switcheroo failed\n", 18switcheroo failed
)     = 18
//but it does fail!

But on my system the comparison fails, because the compiler keeps "to" inside 
EBX and after the syscall return the content of EBX is altered!

So, this seems a kernel bug, but it does not happen in the previous form of 
the code (which didn't use a direct syscall but rather the glibc version)!

And my system works pretty well, I think;

* I get only some crashes from some KDE components, but I consider them 
normal,
* and a lot of problems with GDB and debugging (still looking for the cause).

This is with 2.6.11 + gentoo patchset + skas-v9-pre4 (which is used by many 
people). I've been suspicious of Gentoo, libc, gdb, the kernel, now I'm going 
to look at everything until I find something.
-- 
Inform me of my mistakes, so I can keep imitating Homer Simpson's "Doh!".
Paolo Giarrusso, aka Blaisorblade (Skype ID "PaoloGiarrusso", ICQ 215621894)
http://www.user-mode-linux.org/~blaisorblade

[-- Attachment #2: uml-fix-link-tt-mode-against-nptl.patch --]
[-- Type: text/x-diff, Size: 1214 bytes --]

diff -u linux-2.6.git-paolo/arch/um/sys-i386/unmap.c clean-linux-2.6.11-paolo/arch/um/sys-i386/unmap.c
--- linux-2.6.git-paolo/arch/um/sys-i386/unmap.c	2005-06-07 19:10:51.000000000 +0200
+++ clean-linux-2.6.11-paolo/arch/um/sys-i386/unmap.c	2005-06-08 20:01:21.000000000 +0200
@@ -15,7 +15,8 @@
 	if(munmap(to, size) < 0){
 		return(-1);
 	}
-	if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){
+	/* if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){ */
+	if(mmap2(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) == (void*) -1 ){
 		return(-1);
 	}
 	if(munmap(from, size) < 0){
diff -u linux-2.6.git-paolo/arch/um/sys-x86_64/unmap.c clean-linux-2.6.11-paolo/arch/um/sys-x86_64/unmap.c
--- linux-2.6.git-paolo/arch/um/sys-x86_64/unmap.c	2005-06-07 19:10:51.000000000 +0200
+++ clean-linux-2.6.11-paolo/arch/um/sys-x86_64/unmap.c	2005-06-08 20:01:18.000000000 +0200
@@ -15,7 +15,8 @@
 	if(munmap(to, size) < 0){
 		return(-1);
 	}
-	if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){
+	/* if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) != to){ */
+	if(mmap(to, size, prot, MAP_SHARED | MAP_FIXED, fd, 0) == (void*) -1){
 		return(-1);
 	}
 	if(munmap(from, size) < 0){

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-07-11 21:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-07-07 18:31 [uml-devel] Please test 2.6.13-rc2 in TT mode! Big problem with syscall parameters clobbering Blaisorblade
2005-07-11 21:51 ` Blaisorblade

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox