mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: moreau francis <francis_moreau2000@yahoo.fr>
To: Andy Whitcroft <apw@shadowen.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	alan@lxorguk.ukuu.org.uk, linux-kernel@vger.kernel.org
Subject: Re : Re : Re : sparsemem usage
Date: Fri, 11 Aug 2006 12:46:38 +0000 (GMT)	[thread overview]
Message-ID: <20060811124638.99233.qmail@web25812.mail.ukl.yahoo.com> (raw)
In-Reply-To: <44DC3F47.70508@shadowen.org>

Andy Whitcroft wrote:
> It does produce real numbers, it tells you how many reserved pages you
> have.  The places that this is triggered we are interested in why we
> have no memory left.  We are not interested in how many pages are known
> but reserved as against how many pages are backed by page*'s but are
> really holes; they are mearly pages we can't use out of the total we are
> tracking.  We care about how many are not reserved, and how many of
> those are available.
> 
> It would be 'as simple' as adding a PG_real page bit except for two things:
> 
> 1) page flags bits are seriously short supply; there are some 24
> available of which 22 are in use.  Any new user of a bit would have to
> be an extremely valuable change with major benefit to the kernel, and
> 

It's indeed an issue. Could we instead use a combination of flags
that can't happen together. For example PG_Free|PG_Reserved ?

> 2) if you were to try and populate a PG_real flag it would need to be
> populated for _all_ architectures (and there are a lot) for it to be of
> any use.  As you have already noted there is no consistent way to find
> out whether a page is ram so it would be major exercise to get these
> bits setup during boot.
> 
> I think we should take (2) as a hint here.  If we don't have a
> consistent interface for finding whether a page is real or not, we
> obviously have no general need of that information in the kernel.
> 

or maybe _because_ we don't have a consistent interface for finding 
whether a page is real or not, we end up with a strange thing called
page_is_ram() which could be the same for all arch and be implemented
very simply.

BTW, can you try in a linux tree:

$ grep -r page_is_ram arch/

and see how it's implemented...

thanks

Francis

  reply	other threads:[~2006-08-11 12:46 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-10 14:46 moreau francis
2006-08-10 15:05 ` KAMEZAWA Hiroyuki
2006-08-10 15:23   ` Re : " moreau francis
2006-08-10 15:21 ` Andy Whitcroft
2006-08-10 15:37   ` Re : " moreau francis
2006-08-11  8:26     ` Andy Whitcroft
2006-08-11 12:46       ` moreau francis [this message]
2006-08-11 12:52         ` Re : " Andy Whitcroft
2006-08-16 12:56           ` Re : " moreau francis
  -- strict thread matches above, loose matches on Subject: below --
2006-08-03  9:47 Andy Whitcroft
2006-08-03 12:46 ` Re : " moreau francis
2006-08-03 13:13   ` Andy Whitcroft

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=20060811124638.99233.qmail@web25812.mail.ukl.yahoo.com \
    --to=francis_moreau2000@yahoo.fr \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=apw@shadowen.org \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@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

all inboxes | Powered by JetHome®