From: Jack Steiner <steiner@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: cacheble to uncachble change
Date: Tue, 27 Apr 2004 14:24:41 +0000 [thread overview]
Message-ID: <20040427142441.GA19680@sgi.com> (raw)
In-Reply-To: <408D5C58.E07A5FBE@email.mot.com>
On Tue, Apr 27, 2004 at 05:52:28AM -0500, Robin Holt wrote:
> On Mon, Apr 26, 2004 at 05:03:23PM -0700, David Mosberger wrote:
> > >>>>> On Mon, 26 Apr 2004 16:35:55 -0500, Robin Holt <holt@sgi.com> said:
> >
> >
> > Are you just re-stating my caveat about memory-attribute-aliasing or
> > are you saying something else? If the latter, I'm not following. If
> > the former, I certainly agree: memory attribute-aliasing leads to
> > really nasty-to-track-down bugs. Hence, you want to make sure
> > _upfront_ that it doesn't occur.
>
> Restating. Don't you love the person who plays the master of the obvious
> role. I started writing the email and was at the same time looking for
> examples of kernel code from 2.4 which we had found that was speculating
IIRC, one place that got us in trouble in 2.4 was in free_one_pgd(). The code
prefetches a dirty cacheline that is one cache line BEYOND the
end of the PT page. The line is marked dirty (prefetchw()) in the cache even
though the function does not actually modify it. The line will subsequently be
written back to memory. If the following page is in the same granule &
is being used uncached (memory-attribute-aliasing), bad things will
happen......
> addresses and was going to hopefully point out how difficult it was
> for us to track down. Unfortunately, I never found the bugs in our
> internal bug tracking system. I am not sure we ever actually found
> the source of the corruption, only that it was occuring.
>
> Sorry,
> Robin Holt
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Thanks
Jack Steiner (steiner@sgi.com) 651-683-5302
Principal Engineer SGI - Silicon Graphics, Inc.
next prev parent reply other threads:[~2004-04-27 14:24 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-04-26 18:57 cacheble to uncachble change Mario Smarduch
2004-04-26 19:25 ` David Mosberger
2004-04-26 21:24 ` Jim Hull
2004-04-26 21:35 ` Robin Holt
2004-04-26 21:46 ` David Mosberger
2004-04-27 0:03 ` David Mosberger
2004-04-27 10:52 ` Robin Holt
2004-04-27 14:24 ` Jack Steiner [this message]
2004-04-27 19:50 ` David Mosberger
2004-04-27 21:31 ` Jack Steiner
2004-04-27 21:48 ` David Mosberger
2004-04-27 22:35 ` Mario Smarduch
2004-04-27 22:45 ` David Mosberger
2004-04-27 23:53 ` Luck, Tony
2004-04-28 1:43 ` Robin Holt
2004-04-28 15:52 ` Mario Smarduch
2004-04-29 4:49 ` David Mosberger
2004-04-29 13:39 ` Smarduch Mario-CMS063
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=20040427142441.GA19680@sgi.com \
--to=steiner@sgi.com \
--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