Date: Mon, 03 Mar 2008 10:20:53 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: rrs@FreeBSD.org Cc: perforce@FreeBSD.org Subject: Re: PERFORCE change 136737 for review Message-ID: <20080303.102053.1091030337.imp@bsdimp.com> In-Reply-To: <200803031313.m23DDJv0034582@repoman.freebsd.org> References: <200803031313.m23DDJv0034582@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <200803031313.m23DDJv0034582@repoman.freebsd.org> "Randall R. Stewart" <rrs@FreeBSD.org> writes: : http://perforce.freebsd.org/chv.cgi?CH=136737 : : Change 136737 by rrs@rrs-mips2-jnpr on 2008/03/03 13:13:06 : : Found real source of PG_G problem, the proper intial : value for a CACHED page is PG_CACHE not PG_CACHED. : PG_CACHED is defined in vm to be 1, where as PG_CACHE : is 0x18 (the proper value for us and defined in pte.h). : : Affected files ... : : .. //depot/projects/mips2-jnpr/src/sys/mips/include/pte.h#5 edit : : Differences ... : : ==== //depot/projects/mips2-jnpr/src/sys/mips/include/pte.h#5 (text+ko) ==== : : @@ -110,8 +110,8 @@ : #define PG_UNCACHED 0x00000010 : #define PG_CACHE 0x00000018 : #define PG_CACHEMODE 0x00000038 : -#define PG_ROPAGE (PG_V | PG_RO | PG_CACHED) /* Write protected */ : -#define PG_RWPAGE (PG_V | PG_M | PG_CACHED) /* Not wr-prot not clean */ : +#define PG_ROPAGE (PG_V | PG_RO | PG_CACHE) /* Write protected */ : +#define PG_RWPAGE (PG_V | PG_M | PG_CACHE) /* Not wr-prot not clean */ : #define PG_CWPAGE (PG_V | PG_CACHED) /* Not wr-prot but clean */ : #define PG_IOPAGE (PG_G | PG_V | PG_M | PG_UNCACHED) : #define PG_FRAME 0x3fffffc0 : We should audit all the PG_CACHE vs PG_CACHED stuff, and maybe change PG_CACHED to PG_PAGE_CACHED and PG_UNCACHED to PG_PAGE_UNCACHED... Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080303.102053.1091030337.imp>