svn commit: r215309 - in head/sys: amd64/amd64 i386/i386 sys vm

Konstantin Belousov kib at FreeBSD.org
Sun Nov 14 18:24:13 UTC 2010


Author: kib
Date: Sun Nov 14 18:24:12 2010
New Revision: 215309
URL: http://svn.freebsd.org/changeset/base/215309

Log:
  Use symbolic names instead of hardcoding values for magic p_osrel constants.
  
  MFC after:   1 week

Modified:
  head/sys/amd64/amd64/trap.c
  head/sys/i386/i386/trap.c
  head/sys/sys/param.h
  head/sys/vm/vm_mmap.c

Modified: head/sys/amd64/amd64/trap.c
==============================================================================
--- head/sys/amd64/amd64/trap.c	Sun Nov 14 18:05:03 2010	(r215308)
+++ head/sys/amd64/amd64/trap.c	Sun Nov 14 18:24:12 2010	(r215309)
@@ -415,9 +415,9 @@ trap(struct trapframe *frame)
 					 * This check also covers the images
 					 * without the ABI-tag ELF note.
 					 */
-					if (SV_CURPROC_ABI() ==
-					    SV_ABI_FREEBSD &&
-					    p->p_osrel >= 700004) {
+					if (SV_CURPROC_ABI() == SV_ABI_FREEBSD
+					    && p->p_osrel >=
+					    __FreeBSD_version_SIGSEGV) {
 						i = SIGSEGV;
 						ucode = SEGV_ACCERR;
 					} else {

Modified: head/sys/i386/i386/trap.c
==============================================================================
--- head/sys/i386/i386/trap.c	Sun Nov 14 18:05:03 2010	(r215308)
+++ head/sys/i386/i386/trap.c	Sun Nov 14 18:24:12 2010	(r215309)
@@ -461,9 +461,9 @@ trap(struct trapframe *frame)
 					 * This check also covers the images
 					 * without the ABI-tag ELF note.
 					 */
-					if (SV_CURPROC_ABI() ==
-					    SV_ABI_FREEBSD &&
-					    p->p_osrel >= 700004) {
+					if (SV_CURPROC_ABI() == SV_ABI_FREEBSD
+					    && p->p_osrel >=
+					    __FreeBSD_version_SIGSEGV) {
 						i = SIGSEGV;
 						ucode = SEGV_ACCERR;
 					} else {

Modified: head/sys/sys/param.h
==============================================================================
--- head/sys/sys/param.h	Sun Nov 14 18:05:03 2010	(r215308)
+++ head/sys/sys/param.h	Sun Nov 14 18:24:12 2010	(r215309)
@@ -60,6 +60,11 @@
 #undef __FreeBSD_version
 #define __FreeBSD_version 900025	/* Master, propagated to newvers */
 
+#ifdef _KERNEL
+#define	__FreeBSD_version_SIGSEGV	700004
+#define	__FreeBSD_version_MAP_ANON	800104
+#endif
+
 #ifndef LOCORE
 #include <sys/types.h>
 #endif

Modified: head/sys/vm/vm_mmap.c
==============================================================================
--- head/sys/vm/vm_mmap.c	Sun Nov 14 18:05:03 2010	(r215308)
+++ head/sys/vm/vm_mmap.c	Sun Nov 14 18:24:12 2010	(r215309)
@@ -232,7 +232,7 @@ mmap(td, uap)
 
 	/* Make sure mapping fits into numeric range, etc. */
 	if ((uap->len == 0 && !SV_CURPROC_FLAG(SV_AOUT) &&
-	     curproc->p_osrel >= 800104) ||
+	     curproc->p_osrel >= __FreeBSD_version_MAP_ANON) ||
 	    ((flags & MAP_ANON) && (uap->fd != -1 || pos != 0)))
 		return (EINVAL);
 


More information about the svn-src-all mailing list