linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Peter Allworth <linsol@zeta.org.au>
To: Jesper Skov <jskov@redhat.com>
Cc: linuxppc-embedded@lists.linuxppc.org, linuxppc-dev@lists.linuxppc.org
Subject: more Re: Low memory problems in 8xx Linux
Date: Wed, 09 Feb 2000 12:11:50 +1100	[thread overview]
Message-ID: <38A0BED6.AD2E336C@zeta.org.au> (raw)
In-Reply-To: ot1z6of1ws.fsf@thinktwice.zoftcorp.dk


Jesper Skov wrote:

<snip>

> An alternative and slighly faster implementation might be to skip the
> _PAGE_PRESENT check. Any reason something like this wouldn't work?

<snip>

I had another look at the code and I now remember my rationale for
the _PAGE_PRESENT test. It wasn't to do with level 1 table entries,
Dan's code already takes care of that. It was so that a zeroed pte
couldn't get changed to a non-zero value, since that would make the
pte_none() function return the wrong result. (Moreover, it just seemed
wrong to mess with something that's marked "invalid".)

Admittedly I'm being a bit paranoid since the next thing the exception
handler is likely to do is set the page table entry to something valid.
But, can you be sure? What about a segmentation fault? ;)

Cheers,

PeterA.

"I've always considered a paranoid as a man with all the facts."
	-- Tales of Ordinary Madness

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/

  parent reply	other threads:[~2000-02-09  1:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-01-31 15:02 Low memory problems in 8xx Linux Marcus Sundberg
2000-02-01  2:51 ` Peter Allworth
2000-02-01 18:08   ` Marcus Sundberg
2000-02-01 19:47     ` Dan Malek
2000-02-01 23:30       ` Peter Allworth
2000-02-01 23:59         ` Dan Malek
2000-02-01 23:54       ` duncanp
2000-02-02  0:25         ` Peter Allworth
2000-02-08 10:01           ` Jesper Skov
2000-02-09  0:00             ` Peter Allworth
2000-02-09  8:43               ` Jesper Skov
2000-02-09 23:26                 ` Peter Allworth
2000-02-09  1:11             ` Peter Allworth [this message]
2000-02-09  5:52               ` more " Dan Malek
2000-02-09  8:40               ` Jesper Skov
2000-03-22 15:43   ` Residual data Claude Robitaille
2000-03-23 10:36     ` Gabriel Paubert

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=38A0BED6.AD2E336C@zeta.org.au \
    --to=linsol@zeta.org.au \
    --cc=jskov@redhat.com \
    --cc=linuxppc-dev@lists.linuxppc.org \
    --cc=linuxppc-embedded@lists.linuxppc.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).