All of lore.kernel.org
 help / color / mirror / Atom feed
From: mark.rutland@arm.com (Mark Rutland)
To: linux-arm-kernel@lists.infradead.org
Subject: some question about TCR setting
Date: Mon, 17 Nov 2014 14:00:06 +0000	[thread overview]
Message-ID: <20141117140006.GB23769@leverpostej> (raw)
In-Reply-To: <20141117130306.GP4042@n2100.arm.linux.org.uk>

On Mon, Nov 17, 2014 at 01:03:06PM +0000, Russell King - ARM Linux wrote:
> On Mon, Nov 17, 2014 at 12:09:52PM +0000, Mark Rutland wrote:
> > On Sun, Nov 16, 2014 at 04:52:11PM +0000, Russell King - ARM Linux wrote:
> > > The access type is included on the bus along with the address and other
> > > attributes.  This includes whether it's sharable, and the cache attributes.
> > > A MMU page walker may be implemented such that it is capable of accessing
> > > L2 cache.
> > 
> > Or L1, depending on the implementation of the memory system. The logic
> > which walks the page tables is essentially another coherent observer, so
> > it doesn't necessarily matter precisely where it is attached.
> ...
> > To further this point, provided the TCR is programmed with the same
> > attributes the kernel uses to access the page tables, no cache
> > maintenance is necessary, as the CPU(s) and page table walker(s) are
> > coherent.
> 
> That depends on the implementation - whether the MMU page table walker
> can read from the L1 cache.  If you were correct in all cases, we could
> get rid of:
> 
>         ALT_UP (mcr     p15, 0, r0, c7, c10, 1)         @ flush_pte
> 
> in proc-v7-2level.S - but obviously we think that ARMv7 is unable to read
> from L1 when walking the page tables.
> 
> > This is what we do for ARMv7 and arm64.
> 
> From the code, it looks like we don't for ARMv7.

My bad, I was assuming ARMv7 with the multiprocessor extensions rather
than baseline ARMv7.

Thanks,
Mark,

      reply	other threads:[~2014-11-17 14:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-14  7:22 some question about TCR setting vichy
2014-11-14 10:42 ` Mark Rutland
2014-11-14 13:13   ` vichy
2014-11-14 13:55     ` Mark Rutland
2014-11-16 10:53       ` vichy
2014-11-16 16:52         ` Russell King - ARM Linux
2014-11-17  2:37           ` vichy
2014-11-17 17:10             ` Mark Rutland
2014-11-17 12:09           ` Mark Rutland
2014-11-17 13:03             ` Russell King - ARM Linux
2014-11-17 14:00               ` Mark Rutland [this message]

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=20141117140006.GB23769@leverpostej \
    --to=mark.rutland@arm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.