On Mon, Mar 16, 2009 at 4:05 AM, Piotr Ziecik <ko...@semihalf.com> wrote: > BestComm, a DMA engine in MPC52xx SoC, requires snooping when > CPU caches are enabled to work properly. > > Adding CPU_FTR_NEED_COHERENT fixes NFS problems on MPC52xx machines > introduced by 'powerpc/mm: Fix handling of _PAGE_COHERENT in BAT setup code'. > > Signed-off-by: Piotr Ziecik <ko...@semihalf.com> > --- > > This patch was succesfully tested on our MPC5200-based. > Ben, Gerhard could you ack this patch, as it implements your suggestion. > > arch/powerpc/include/asm/cputable.h | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/arch/powerpc/include/asm/cputable.h > b/arch/powerpc/include/asm/cputable.h > index b18358f..22dc2d8 100644 > --- a/arch/powerpc/include/asm/cputable.h > +++ b/arch/powerpc/include/asm/cputable.h > @@ -241,9 +241,11 @@ extern const char *powerpc_base_platform; > /* We need to mark all pages as being coherent if we're SMP or we have a > * 74[45]x and an MPC107 host bridge. Also 83xx and PowerQUICC II > * require it for PCI "streaming/prefetch" to work properly. > + * This is also required by 52xx family. > */ > #if defined(CONFIG_SMP) || defined(CONFIG_MPC10X_BRIDGE) \ > - || defined(CONFIG_PPC_83xx) || defined(CONFIG_8260) > + || defined(CONFIG_PPC_83xx) || defined(CONFIG_8260) \ > + || defined(CONFIG_PPC_MPC52xx) > #define CPU_FTR_COMMON CPU_FTR_NEED_COHERENT > #else > #define CPU_FTR_COMMON 0
Aside from the fact that MPC10X_BRIDGE, PPC83xx and 8260 are already doing it, adding the feature bit this way isn't multiplatform friendly. Essentially it means that all selected platforms will have CPU_FTR_NEED_COHERENT enabled if CONFIG_PPC_MPC52xx is enabled. Adding a new section to cputable.c for PVR=0x8082201x and applying the bit there would contain the impact to only G2 cores, but it's not perfect. Some of the 82xx parts have the same PVR and so would also be impacted. Maybe I'm fretting over nothing. Ben, what's the right thing to do here? Otherwise, I've pulled this into my tree and while I'm not affected by the bug, I don't see any new problems with the patch either. g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev