linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Kumar Gala <kumar.gala@freescale.com>
Cc: Michael Ellerman <michael@ozlabs.org>,
	Jimi Xenidis <jimix@watson.ibm.com>,
	jack@codezen.org, imunsie@au.ibm.com, linuxppc-dev@ozlabs.org,
	David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [PATCH 10/15] powerpc: Define slb0_limit() for BOOK3E
Date: Tue, 19 Apr 2011 08:50:39 +1000	[thread overview]
Message-ID: <1303167039.28876.172.camel@pasglop> (raw)
In-Reply-To: <5313E556-6BFA-48D0-921F-7CF91892514A@freescale.com>

On Mon, 2011-04-18 at 17:30 -0500, Kumar Gala wrote:
> > On Mon, 2011-04-18 at 07:42 -0500, Kumar Gala wrote:
> >> Let's rename this function to something 'linear_map'.  As on FSL
> >> Book-E 64 we do things a bit differently and have more covered in
> >> linear map than 1G
> > 
> > It's not quite linear_map. It's whatever can be accessed without
> taking
> > exceptions. IE. What is bolted.
> > 
> > It should probably go into a variable initialized by the mm code
> tho.
> > 
> > Cheers,
> > Ben.
> 
> Is this 'ppc64_rma_size' ? 

Not quite either :-)

The RMA (on server) is the amount of memory that is accessible in real
mode (with MMU off).

SLB0 is segment 0, which is 256M or 1T depending on what segment size we
use, and is a bolted segment in Linux. Thus all accesses to SLB0 are
guaranteed to not generate an SLB miss fault. Since on server our linear
mapping is bolted in the hash, what that means is that any access to
memory in segment 0 is guaranteed to not take an exception (ie not
clobber SRR0/1 for example).

We re-use the SLB0 concept on BookE to represent the bolted portion of
the linear mapping since it has the same characteristics, ie, accesses
will not cause an exception that can clobber SRR0/1.

The patch Michael posted is indeed a bit gross (it's my code btw :-) and
we'll need to clean it up, my idea is to have a global set by the
platform, but maybe we can have some generic code that performs a TLB
search for IPROT entries :-)

Cheers,
Ben.

  reply	other threads:[~2011-04-18 22:50 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-15  8:31 [PATCH 01/15] powerpc/a2: Add some #defines for A2 specific instructions Michael Ellerman
2011-04-15  8:31 ` [PATCH 02/15] of: Export of_irq_find_parent() Michael Ellerman
2011-04-15  8:31 ` [PATCH 03/15] powerpc: Add SCOM infrastructure Michael Ellerman
2011-04-15  8:31 ` [PATCH 04/15] powerpc/xics: xics.h relies on linux/interrupt.h Michael Ellerman
2011-04-15  8:31 ` [PATCH 05/15] powerpc/xics: Move irq_host matching into the ics backend Michael Ellerman
2011-04-15  8:32 ` [PATCH 06/15] powerpc/nvram: Search for nvram using compatible Michael Ellerman
2011-04-15  8:32 ` [PATCH 08/15] powerpc: Add A2 cpu support Michael Ellerman
2011-04-15  8:32 ` [PATCH 07/15] powerpc: Move CPU_FTRS_BASE_BOOK3E into cputable.h & update FTR masks Michael Ellerman
2011-04-18 12:37   ` Kumar Gala
2011-04-15  8:32 ` [PATCH 09/15] powerpc: Add TLB size detection for TYPE_3E MMUs Michael Ellerman
2011-04-15  8:32 ` [PATCH 10/15] powerpc: Define slb0_limit() for BOOK3E Michael Ellerman
2011-04-17  5:58   ` Olof Johansson
2011-04-18 12:42   ` Kumar Gala
2011-04-18 21:41     ` Benjamin Herrenschmidt
2011-04-18 22:30       ` Kumar Gala
2011-04-18 22:50         ` Benjamin Herrenschmidt [this message]
2011-04-15  8:32 ` [PATCH 11/15] powerpc: Index crit/dbg/mcheck stacks using cpu number on 64bit Michael Ellerman
2011-04-15  8:32 ` [PATCH 12/15] powerpc/book3e: Use way 3 for linear mapping bolted entry Michael Ellerman
2011-04-18 12:43   ` Kumar Gala
2011-04-18 21:40     ` Benjamin Herrenschmidt
2011-04-18 22:27       ` Kumar Gala
2011-04-18 22:44         ` Benjamin Herrenschmidt
2011-04-15  8:32 ` [PATCH 13/15] powerpc/book3e: Flush IPROT protected TLB entries leftover by firmware Michael Ellerman
2011-04-15  8:32 ` [PATCH 14/15] powerpc: Add WSP platform Michael Ellerman
2011-04-18 12:46   ` Kumar Gala
2011-04-18 21:39     ` Benjamin Herrenschmidt
2011-04-15  8:32 ` [PATCH 15/15] powerpc: Add early debug for WSP platforms Michael Ellerman

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=1303167039.28876.172.camel@pasglop \
    --to=benh@kernel.crashing.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=imunsie@au.ibm.com \
    --cc=jack@codezen.org \
    --cc=jimix@watson.ibm.com \
    --cc=kumar.gala@freescale.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=michael@ozlabs.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).