> On Thu, 2010-11-18 at 13:32 +1100, Michael Neuling wrote: > > In:=20 > > powerpc/mm: Fix pgtable cache cleanup with CONFIG_PPC_SUBPAGE_PROT > > commit d28513bc7f675d28b479db666d572e078ecf182d > > Author: David Gibson <da...@gibson.dropbear.id.au> > >=20 > > subpage_protection() was changed to to take an mm rather a pgdir but it > > didn't change calling site in hashpage_preload(). The change wasn't > > noticed at compile time since hashpage_preload() used a void* as the > > parameter to subpage_protection(). > ...=20 > >=20 > > diff --git a/arch/powerpc/mm/hash_utils_64.c b/arch/powerpc/mm/hash_utils= > _64.c > > index 83f534d..5e95844 100644 > > --- a/arch/powerpc/mm/hash_utils_64.c > > +++ b/arch/powerpc/mm/hash_utils_64.c > > @@ -1123,7 +1123,7 @@ void hash_preload(struct mm_struct *mm, unsigned lo= > ng ea, > > else > > #endif /* CONFIG_PPC_HAS_HASH_64K */ > > rc =3D __hash_page_4K(ea, access, vsid, ptep, trap, local, ssiz e, > > - subpage_protection(pgdir, ea)); > > + subpage_protection(mm, ea)); > > Type checking is fun :) > > This is stable material no?
Yes. In the bit you snipped was a: cc: sta...@kernel.org (only 2.6.33 and newer) Fortunately it's not in 2.6.32 so the bug missed the distros. Mikey _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
