On Tue, 2 Dec 2008, Kumar Gala wrote:
> Added 85xx specifc smp_ops structure.  We use ePAPR style boot release
> and the MPIC for IPIs at this point.
>
> Additionally added routines for secondary cpu entry and initializtion.
>
> @@ -740,6 +750,9 @@ finish_tlb_load:
> #else
>       rlwimi  r12, r11, 26, 27, 31    /* extract WIMGE from pte */
> #endif
> +#ifdef CONFIG_SMP
> +     ori     r12, r12, MAS2_M
> +#endif
>       mtspr   SPRN_MAS2, r12

Wouldn't it be more efficient to set _PAGE_COHERENT when the pte is created
vs setting MAS2_M each time it's loaded?

Is it correct to set MAS2_M for all pages, even uncached ones?

The code for ioremap() has this:

         /* Non-cacheable page cannot be coherent */
         if (flags & _PAGE_NO_CACHE)
                 flags &= ~_PAGE_COHERENT;

It seems odd that ioremap would explictly unset _PAGE_COHERENT when the
code that sets the tlb will just force it back on.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to