Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Jan 2007 10:17:14 GMT
From:      Roman Divacky <rdivacky@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 113103 for review
Message-ID:  <200701181017.l0IAHE0a033245@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=113103

Change 113103 by rdivacky@rdivacky_witten on 2007/01/18 10:17:04

	We have to lock the em in a case of child == 0 because its looked up
	thus shared.

Affected files ...

.. //depot/projects/linuxolator/src/sys/compat/linux/linux_emul.c#23 edit

Differences ...

==== //depot/projects/linuxolator/src/sys/compat/linux/linux_emul.c#23 (text+ko) ====

@@ -99,7 +99,7 @@
 		}
 	} else {
 		/* lookup the old one */
-		em = em_find(td->td_proc, EMUL_DONTLOCK);
+		em = em_find(td->td_proc, EMUL_DOLOCK);
 		KASSERT(em != NULL, ("proc_init: emuldata not found in exec case.\n"));
 	}
 
@@ -139,7 +139,8 @@
 		/* we might have a sleeping linux_schedtail */
 		wakeup(&p->p_emuldata);
 		PROC_UNLOCK(p);
-	}
+	} else
+	   	EMUL_UNLOCK(&emul_lock);
 
 	return (0);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200701181017.l0IAHE0a033245>