All of lore.kernel.org
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH] ARM: Merge v7 and v8 outer cache operations
Date: Fri, 27 Mar 2015 15:11:48 +0100	[thread overview]
Message-ID: <20150327151148.05ce2a2b@lilith> (raw)
In-Reply-To: <20150212155652.GI1522@leverpostej>

Hello Mark,

On Thu, 12 Feb 2015 15:56:52 +0000, Mark Rutland <mark.rutland@arm.com>
wrote:
> On Sat, Jan 31, 2015 at 03:08:54AM +0000, fenghua at phytium.com.cn wrote:
> > From: David Feng <fenghua@phytium.com.cn>
> > 
> > Armv7 and Armv8 allow outer cache exist, it is outside of the architecture
> > defined cache hierarchy and can not be manipulated by architecture defined
> > instructions. It's processor specific.
> > This patch merge v7_outer_cache_* and v8 l3_cache_*.
> 
> This commit message is a little misleading, though it probably makes
> sense to have something of this sort ARMv8. Info dump below.
> 
> Recently the ARMv8 architecture reference manual was clarified to
> mention that any such system caches _must_ respect maintenance by VA,
> and are affected by the architected instructions for this. The arm64
> Linux port relies on this property.
> 
> Set/Way maintenance will not affect system caches. So if you want to
> flush/empty the entire cache hierarchy, you will need to rely on a
> mechanism specific to the outer cache implementation (rather than one
> specific to the processor).
> 
> Additionally, the interconnect and cache hierarchies in ARMv8
> implementations are becoming more complex, and it is more likely that
> dirty lines may migrate arbitrarily between CPUs and the system caches.
> Due to this you will need to ensure that CPU caches are disabled and
> empty before system cache maintenance is performed (I don't know whether
> your current sequences for ARMv7 ensure that).

So, does the commit message require rewriting?

> Thanks,
> Mark.

Amicalement,
-- 
Albert.

  reply	other threads:[~2015-03-27 14:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-31  3:08 [U-Boot] [RFC PATCH] ARM: Merge v7 and v8 outer cache operations fenghua at phytium.com.cn
2015-02-12 15:37 ` Tom Rini
2015-02-12 15:56 ` Mark Rutland
2015-03-27 14:11   ` Albert ARIBAUD [this message]
2015-03-30  9:39     ` Mark Rutland
2015-04-02  6:57       ` Albert ARIBAUD

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150327151148.05ce2a2b@lilith \
    --to=albert.u.boot@aribaud.net \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.