Thread (10 messages) flat view 10 messages, 3 authors, 2011-02-16
DORMANTno replies REVIEWED: 3 (3M)

1 review trailer.

[PATCH] ARM: errata: pl310 cache sync operation may be faulty

From: catalin.marinas@arm.com (Catalin Marinas)
Date: 2011-02-16 12:36:11

On Wed, 2011-02-16 at 12:34 +0000, Srinidhi KASAGAR wrote:
From 7a1fa2f8ec106bda4f940f5c9478ec16b2de6846 Mon Sep 17 00:00:00 2001
From: srinidhi kasagar <redacted>
Date: Mon, 14 Feb 2011 17:07:06 +0530
Subject: [PATCH] ARM: errata: pl310 cache sync operation may be faulty

The effect of cache sync operation is to drain the store
buffer and wait for all internal buffers to be empty. In
normal conditions, store buffer is able to merge the
normal memory writes within its 32-byte data buffers.
Due to this erratum present in r3p0, the effect of cache
sync operation on the store buffer still remains when
the operation completes. This means that the store buffer
is always asked to drain and this prevents it from merging
any further writes.

This can severely affect performance on the write traffic
esp. on Normal memory NC one.

The proposed workaround is to replace the normal offset of
cache sync operation(0x730) by another offset targeting an
unmapped PL310 register 0x740.

Signed-off-by: srinidhi kasagar <redacted>
Acked-by: Linus Walleij <redacted>
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help