[M3commit] CVS Update: cm3

Jay Krell jkrell at elego.de
Wed Jan 28 18:44:42 CET 2009


CVSROOT:	/usr/cvs
Changes by:	jkrell at birch.	09/01/28 18:44:42

Modified files:
	cm3/m3-libs/m3core/src/runtime/ALPHA_OSF/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/AMD64_DARWIN/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/FreeBSD4/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/I386_DARWIN/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/LINUXLIBC6/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/NetBSD2_i386/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/PPC_DARWIN/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/SOLgnu/: RTThread.m3 
	cm3/m3-libs/m3core/src/runtime/SOLsun/: RTThread.m3 

Log message:
	eliminate more gratuitous platform differences
	
	Some platforms uses + to combine mprotect protections, some used Word.Or.
	Use + because it is terser, assuming these are all independent flags.
	
	Some platforms used WITH i = foo; ASSERT(i = 0);
	Some used VAR i := foo; ASSERT(i = 0);
	Use VAR, in order to elminate indentation.
	
	some platforms said IF (page_size = 0) and some said IF page_size = 0.
	Use the second shorter form. This code kinda looks like it has a race condition,
	but probably not, since it is the code that is involved in creating threads,
	there cannot be multiple threads until after it runs.
	
	Platforms still vary as to what protections they use on the last stack page.




More information about the M3commit mailing list