From owner-cvs-all Sat Aug 15 17:41:44 1998 Return-Path: Received: (from daemon@localhost) by hub.freebsd.org (8.8.8/8.8.8) id RAA10289 for cvs-all-outgoing; Sat, 15 Aug 1998 17:41:44 -0700 (PDT) (envelope-from owner-cvs-all) Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id RAA10284; Sat, 15 Aug 1998 17:41:43 -0700 (PDT) (envelope-from bde@FreeBSD.org) From: Bruce Evans Received: (from bde@localhost) by freefall.freebsd.org (8.8.8/8.8.5) id RAA24097; Sat, 15 Aug 1998 17:41:41 -0700 (PDT) Date: Sat, 15 Aug 1998 17:41:41 -0700 (PDT) Message-Id: <199808160041.RAA24097@freefall.freebsd.org> To: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: cvs commit: src/sys/i386/i386 mp_machdep.c pmap.c Sender: owner-cvs-all@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk bde 1998/08/15 17:41:40 PDT Modified files: sys/i386/i386 mp_machdep.c pmap.c Log: pmap.c: Cast pointers to (vm_offset_t) instead of to (u_long) (as before) or to (uintptr_t)(void *) (as would be more correct). Don't cast vm_offset_t's to (u_long) just to do arithmetic on them. mp_machdep.c: Cast pointers to (uintptr_t) instead of to (u_long). Don't forget to cast pointers to (void *) first or to recover from integral possible integral promotions, although this is too much work for machine-dependent code. vm code generally avoids warnings for pointer vs long size mismatches by using vm_offset_t to represent pointers; pmap.c often uses plain `unsigned int' instead of vm_offset_t and didn't use u_long elsewhere, but this style was messed up by code apparently imported from mp_machdep.c. Revision Changes Path 1.77 +4 -4 src/sys/i386/i386/mp_machdep.c 1.206 +8 -7 src/sys/i386/i386/pmap.c