From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: linux-ia64@vger.kernel.org
Subject: RE: ia64 mmu_gather question
Date: Thu, 02 Aug 2007 07:57:04 +0000 [thread overview]
Message-ID: <1186041426.5495.599.camel@localhost.localdomain> (raw)
In-Reply-To: <1186026055.5495.585.camel@localhost.localdomain>
On Thu, 2007-08-02 at 17:10 +1000, Benjamin Herrenschmidt wrote:
> On Wed, 2007-08-01 at 22:38 -0700, Luck, Tony wrote:
> > No locking. But we do have race detection. After we chase the
> > PGD>PUD>PMD>PTE
> > pointers we insert the TLB entry. Then we retrace the pointer chain
> > and
> > make sure that the pte we find is still the same. If it isn't, then
> > we
> > purge the entry we just inserted and go for a full page fault.
> >
> > Time to tell bed-time stories to my daughter. More tomorrow (if
> > someone
> > else doesn't fill in the rest of the answers before I get back to
> > this).
>
> Ok, that's what I think I understood from the asm. However, what
> prevents the very unlikely race where you insert a stale pgtable
> mapping entry, and before you backtrack and remove it, another
> CPU accesses the stale PTE ?
I was thinking about a case where the TLB is shared (SMT) between linux
logical CPUs (threads) but ia64 is not SMT right ? Thus the TLB is
split ,and the "other" CPU can't see the stale translation... should be
allright then.
Cheers,
Ben.
next prev parent reply other threads:[~2007-08-02 7:57 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-02 3:40 ia64 mmu_gather question Benjamin Herrenschmidt
2007-08-02 5:38 ` Luck, Tony
2007-08-02 7:09 ` Benjamin Herrenschmidt
2007-08-02 7:57 ` Benjamin Herrenschmidt [this message]
2007-08-02 17:16 ` Luck, Tony
2007-08-02 21:46 ` Benjamin Herrenschmidt
2007-08-02 21:56 ` Luck, Tony
2007-08-02 22:00 ` Benjamin Herrenschmidt
2007-08-02 22:07 ` David Mosberger-Tang
2007-08-02 22:32 ` Benjamin Herrenschmidt
2007-08-03 3:22 ` Tony Luck
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=1186041426.5495.599.camel@localhost.localdomain \
--to=benh@kernel.crashing.org \
--cc=linux-ia64@vger.kernel.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