Date: Thu, 24 Jan 2008 09:13:53 GMT From: Rafal Jaworowski <raj@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 133990 for review Message-ID: <200801240913.m0O9DrrN014930@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=133990 Change 133990 by raj@raj_mimi on 2008/01/24 09:12:59 Correct a nasty bug in mmu_booke_page_init(). Due to a missing argument, the pv_list was not being initialized properly and we had a null pointer dereference upon first use... Affected files ... .. //depot/projects/e500/sys/powerpc/booke/pmap.c#7 edit Differences ... ==== //depot/projects/e500/sys/powerpc/booke/pmap.c#7 (text+ko) ==== @@ -288,7 +288,7 @@ void mmu_booke_object_init_pt(mmu_t, pmap_t, vm_offset_t, vm_object_t, vm_pindex_t, vm_size_t); boolean_t mmu_booke_page_exists_quick(mmu_t, pmap_t, vm_page_t); -void mmu_booke_page_init(vm_page_t); +void mmu_booke_page_init(mmu_t, vm_page_t); void mmu_booke_pinit(mmu_t, pmap_t); void mmu_booke_pinit0(mmu_t, pmap_t); void mmu_booke_protect(mmu_t, pmap_t, vm_offset_t, vm_offset_t, vm_prot_t); @@ -1969,7 +1969,7 @@ * Initialize a vm_page's machine-dependent fields. */ void -mmu_booke_page_init(vm_page_t m) +mmu_booke_page_init(mmu_t mmu, vm_page_t m) { TAILQ_INIT(&m->md.pv_list); }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200801240913.m0O9DrrN014930>