Date: Thu, 17 Jan 2008 04:59:57 GMT From: Kip Macy <kmacy@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 133452 for review Message-ID: <200801170459.m0H4xvqG038452@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=133452 Change 133452 by kmacy@pandemonium:kmacy:xen31 on 2008/01/17 04:59:46 bzero the the entire page allocated to the pv_chunk as opposed to just PG_ZERO bytes Affected files ... .. //depot/projects/xen31/sys/i386/xen/pmap.c#18 edit Differences ... ==== //depot/projects/xen31/sys/i386/xen/pmap.c#18 (text+ko) ==== @@ -300,6 +300,22 @@ CTASSERT(1 << PDESHIFT == sizeof(pd_entry_t)); CTASSERT(1 << PTESHIFT == sizeof(pt_entry_t)); +static __inline void +pagezero(void *page) +{ +#if defined(I686_CPU) + if (cpu_class == CPUCLASS_686) { +#if defined(CPU_ENABLE_SSE) + if (cpu_feature & CPUID_SSE2) + sse2_pagezero(page); + else +#endif + i686_pagezero(page); + } else +#endif + bzero(page, PAGE_SIZE); +} + void pd_set(struct pmap *pmap, int ptepindex, vm_paddr_t val, int type) { @@ -2051,7 +2067,7 @@ pc = (struct pv_chunk *)pmap_ptelist_alloc(&pv_vafree); pmap_qenter((vm_offset_t)pc, &m, 1); if ((m->flags & PG_ZERO) == 0) - bzero(pc, PG_ZERO); + pagezero(pc); pc->pc_pmap = pmap; pc->pc_map[0] = pc_freemask[0] & ~1ul; /* preallocated bit 0 */ for (field = 1; field < _NPCM; field++) @@ -3054,22 +3070,6 @@ PMAP_UNLOCK(dst_pmap); } -static __inline void -pagezero(void *page) -{ -#if defined(I686_CPU) - if (cpu_class == CPUCLASS_686) { -#if defined(CPU_ENABLE_SSE) - if (cpu_feature & CPUID_SSE2) - sse2_pagezero(page); - else -#endif - i686_pagezero(page); - } else -#endif - bzero(page, PAGE_SIZE); -} - /* * pmap_zero_page zeros the specified hardware page by mapping * the page into KVM and using bzero to clear its contents.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200801170459.m0H4xvqG038452>