Linux PARISC architecture development
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Mikulas Patocka <mikulas@artax.karlin.mff.cuni.cz>
Cc: John David Anglin <dave@hiauly1.hia.nrc.ca>,
	kyle@mcmartin.ca, linux-parisc@vger.kernel.org
Subject: Re: PA caches (was: C8000 cpu upgrade problem)
Date: Wed, 27 Oct 2010 11:35:24 -0500	[thread overview]
Message-ID: <1288197324.6886.2.camel@mulgrave.site> (raw)
In-Reply-To: <alpine.DEB.1.10.1010271820320.18202@artax.karlin.mff.cuni.cz>

On Wed, 2010-10-27 at 18:28 +0200, Mikulas Patocka wrote:
> 
> On Wed, 27 Oct 2010, James Bottomley wrote:
> 
> > On Wed, 2010-10-27 at 10:06 +0200, Mikulas Patocka wrote:
> > > 
> > > On Tue, 26 Oct 2010, James Bottomley wrote:
> > > 
> > > > On Tue, 2010-10-26 at 21:29 -0400, John David Anglin wrote:
> > > > > > - shared memory --- there is SHMLBA boundary that causes that all
> > > > > mappings 
> > > > > > are aligned to this boundary --- it is **WRONG** in the current
> > > > > kernel!!! 
> > > > > > It is only 4MB and should be 16MB!!!
> > > > > 
> > > > > James has said that the max for all PA-RISC implementations is
> > > > > 4 MB.  The value is returned by the PDC_CACHE call.  Maybe a BUG_ON is
> > > > > called for.  The alias boundary can be determined by the alias field
> > > > > in the D_conf return value.
> > > > 
> > > > Why is it I get blamed for everything cache related on parisc?  The
> > > 
> > > You don't get blamed, we're just trying to find bugs :)
> > > 
> > > > statement in the manuals that the equivalency modulus is 16MB was left
> > > > for future expansion.  However, given PA8900 is the last in the series,
> > > > there is no future expansion.  John Marvin (I think it was) from the HP
> > > > processor group confirmed that the largest equivalency modulus for any
> > > > produced parisc processor is 4MB, so that's what we use in the kernel.
> > > > 
> > > > James
> > > 
> > > The largest L2 cache size is 64MB --- so if the cache is 4-way 
> > > associative, the equivalency distance is 16MB (as the manual says).
> > > 
> > > If the equivalency distance were 4MB, the L2 cache would have to be 16-way 
> > > (or the cache would have to be physically indexed and you wouldn't have to 
> > > care about its consistency at all).
> > 
> > The L2 cache has no equivalency modulus ... it's not virtually indexed.
> > 
> > James
> 
> Why is Kyle than suggesting that I am lucky because I have no L2 cache 
> (and therefore, Linux runs faster)?
> 
> Why are people talking here about flushing 32MB or 64MB L2 on fork()?
> 
> Or is it that you need to flush only L1 cache but the architecture forces 
> flush of both caches?

There's only a couple of flush instructions: fic and fdc ... they have
to flush all caches.  We did argue the toss on this with the HP
processor people since aliasing, which is primarily where we need
flushes for control, only occurs in the L1 cache.  However, they pointed
out that if they made fic and fdc L1 specific, we'd have no control over
DMA type ops which have to reach physical memory.

> I'd still like to see if someone with PA8800 or PA8900 with L2 ran that 
> shared memory experiment to actually *prove* that L2 is physically indexed 
> and that the L1 equivalency modulus is 4MB. I.e. not rely on what you 
> heard somewhere, but rely on what you see.

We already did all of that years ago just trying to make the pa8x00
chips work with linux ... they didn't for about 18 months.

James



  reply	other threads:[~2010-10-27 16:35 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20101024020337.725094D30@hiauly1.hia.nrc.ca>
2010-10-24  3:03 ` C8000 cpu upgrade problem Mikulas Patocka
2010-10-24  3:43   ` Kyle McMartin
2010-10-26  2:16     ` PA caches (was: C8000 cpu upgrade problem) Mikulas Patocka
2010-10-26  3:04       ` Kyle McMartin
2010-10-26  4:30         ` John David Anglin
2010-10-26 16:02         ` Mikulas Patocka
2010-10-27  1:29           ` John David Anglin
2010-10-27  2:40             ` John David Anglin
2010-10-27  4:50             ` James Bottomley
2010-10-27  8:06               ` Mikulas Patocka
2010-10-27  8:35                 ` Mikulas Patocka
2010-10-27 14:18                   ` James Bottomley
2010-10-27 14:07                 ` James Bottomley
2010-10-27 16:28                   ` Mikulas Patocka
2010-10-27 16:35                     ` James Bottomley [this message]
2010-10-27 16:50                       ` Mikulas Patocka
2010-10-27 17:07                         ` James Bottomley
2010-10-28  6:04                           ` John David Anglin
2010-10-28 16:55                             ` James Bottomley
2010-10-27  9:04               ` sym53c8xx_2 data corruption Mikulas Patocka
2010-10-27 14:46                 ` James Bottomley
2010-10-27 16:19                   ` Mikulas Patocka
2010-10-27 16:37                     ` James Bottomley
2010-10-28  5:59                     ` Grant Grundler
2010-12-18 20:13       ` PA caches (was: C8000 cpu upgrade problem) John David Anglin
2010-10-24  4:01   ` C8000 cpu upgrade problem John David Anglin
2010-10-26  2:04     ` Mikulas Patocka

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=1288197324.6886.2.camel@mulgrave.site \
    --to=james.bottomley@hansenpartnership.com \
    --cc=dave@hiauly1.hia.nrc.ca \
    --cc=kyle@mcmartin.ca \
    --cc=linux-parisc@vger.kernel.org \
    --cc=mikulas@artax.karlin.mff.cuni.cz \
    /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