public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: "Luck, Tony" <tony.luck@intel.com>
To: linux-ia64@vger.kernel.org
Subject: RE: [Linux-ia64] Fix for for memory leak in IA32 mmap
Date: Tue, 05 Mar 2002 21:49:09 +0000	[thread overview]
Message-ID: <marc-linux-ia64-105590701905230@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590701905222@msgid-missing>

> 3)  I wasn't intending to keep a bitmap for all of IA32 addressable
> memory.  I intend to keep a list of all the partial pages, basically
> the first and last page of an `mmap' request, and then keep a bitmap
> for each of the 4K chunks inside of that page.  Maintaining the bitmap
> will be a little tricky to handle all of the `mmap/munmap' possibilities
> but it shouldn't be all that hard to get it right.

Aha! A list of the partial pages makes sense. I thought that the "list" was
going to be a list of mmap'd ranges ... which would be tougher to get right
in the face of pathalogical programs.  I agree that these pathalogical
programs deserve to fail, but they shouldn't take out the system (or tie it
up with incredibly long lists of objects).

You might keep an eye on the complexity of the code for maintaining the
lists
versus just having a (2-level) bitmap for the whole of the IA-32 address
space.  There are some tricky cases (especially when you factor in support
for 64K kernel page size as well as 16K ... so your partial page bitmaps
have different lengths for different configurations).

-Tony


  parent reply	other threads:[~2002-03-05 21:49 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-03-05 15:13 [Linux-ia64] Fix for for memory leak in IA32 mmap Don Dugger
2002-03-05 17:34 ` David Mosberger
2002-03-05 17:46 ` Don Dugger
2002-03-05 18:59 ` David Mosberger
2002-03-05 19:44 ` Luck, Tony
2002-03-05 20:06 ` n0ano
2002-03-05 20:09 ` David Mosberger
2002-03-05 21:49 ` Luck, Tony [this message]
2002-03-05 22:18 ` n0ano

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=marc-linux-ia64-105590701905230@msgid-missing \
    --to=tony.luck@intel.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