[PATCH 1/8] ARM: Improve the L2 cache performance when PL310 is used

[Jason McMullan]

On Tue, May 4, 2010 at 12:44 PM, Catalin Marinas
<catalin.marinas at arm.com> wrote:
> With this L2 cache controller, the cache maintenance by PA and sync
> operations are atomic and do not require a "wait" loop or spinlocks.
> This patch conditionally defines the cache_wait() function and locking
> primitives (rather than duplicating the functions or file).
>
> Since L2x0 cache controllers do not work with ARMv7 CPUs, the patch
> automatically enables CACHE_PL310 when CPU_V7 is defined.
>
> [snip snip snip]
>  static inline void l2x0_inv_all(void)
> @@ -107,11 +134,11 @@ static inline void l2x0_inv_all(void)
>        unsigned long flags;
>
>        /* invalidate all ways */
> -       spin_lock_irqsave(&l2x0_lock, flags);
> +       _l2x0_lock(&l2x0_lock, flags);
>        writel(0xff, l2x0_base + L2X0_INV_WAY);
> -       cache_wait(l2x0_base + L2X0_INV_WAY, 0xff);
> +       cache_wait_always(l2x0_base + L2X0_INV_WAY, 0xff);
>        cache_sync();
> -       spin_unlock_irqrestore(&l2x0_lock, flags);
> +       _l2x0_unlock(&l2x0_lock, flags);
>  }

So, ah, shouldn't you be using a mask of 0xffff for 16-way PL310s?

And I think we have a potential patch collision in the near future.

Could you integrate in my [arm l2x0] patch I posted today on the list?

It supports 16-way PL310s, and PL210s with fewer than 7 ways.

--
Jason S. McMullan
Netronome Systems, Inc.