linux-api.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Gleb Natapov <gleb@redhat.com>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: linux-mm@kvack.org, kosaki.motohiro@jp.fujitsu.com,
	linux-kernel@vger.kernel.org, linux-api@vger.kernel.org,
	akpm@linux-foundation.org, andrew.c.morrow@gmail.com,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Subject: Re: [PATCH v6] add MAP_UNLOCKED mmap flag
Date: Mon, 18 Jan 2010 19:08:16 +0200	[thread overview]
Message-ID: <20100118170816.GA22111@redhat.com> (raw)
In-Reply-To: <84144f021001180805q4d1203b8qab8ccb1de87b2866@mail.gmail.com>

On Mon, Jan 18, 2010 at 06:05:38PM +0200, Pekka Enberg wrote:
> On Mon, Jan 18, 2010 at 4:19 PM, Gleb Natapov <gleb@redhat.com> wrote:
> > The specific use cases were discussed in the thread following previous
> > version of the patch. I can describe my specific use case in a change log
> > and I can copy what Andrew said about his case, but is it really needed in
> > a commit message itself? It boils down to greater control over when and
> > where application can get major fault. There are applications that need
> > this kind of control. As of use of mlockall(MCL_FUTURE) how can I make
> > sure that all memory allocated behind my application's back (by dynamic
> > linker, libraries, stack) will be locked otherwise?
> 
> Again, why do you want to MCL_FUTURE but then go and use MAP_UNLOCKED?
I need to have all my memory locked except one big (bigger then main
memory) chunk. I either need to rewrite my application and all libraries
to use memory allocator that return locked memory, may be even rewrite
dynamic loader to use this allocator to lock executable code too, or run
mlockall(MCL_FUTURE|MCL_CURRENT) at startup and exempt one allocation
from this rule. Note that I can't allocate it locked and then unlock
since allocation will fail. Actually for me it hangs kernel last I
checked.

> "Greater control" is not an argument for adding a new API that needs
> to be maintained forever, a real world use case is.
> 
If there is real world use case for mlockall() there is real use case for
this too. People seems to be trying to convince me that I don't need
mlockall() without proposing alternatives. The only alternative I see
lock everything from userspace.

> And yes, this stuff needs to be in the changelog. Whether you want to
> spell it out or post an URL to some previous discussion is up to you.
The discussion was here just a couple of days ago. Here is the link
were I describe my use case: http://marc.info/?l=linux-mm&m=126345374125942&w=2
If you think it needs to be spelled out in commit log I'll do it.

--
			Gleb.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2010-01-18 17:08 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-18 13:37 [PATCH v6] add MAP_UNLOCKED mmap flag Gleb Natapov
     [not found] ` <20100118133755.GG30698-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-01-18 14:09   ` Pekka Enberg
2010-01-18 14:19     ` Gleb Natapov
2010-01-18 14:32       ` Alan Cox
2010-01-18 14:35         ` Gleb Natapov
2010-01-18 14:49         ` Peter Zijlstra
2010-01-18 15:01           ` Gleb Natapov
2010-01-18 15:06             ` Peter Zijlstra
2010-01-18 15:11               ` Avi Kivity
2010-01-18 15:14                 ` Peter Zijlstra
2010-01-18 15:19                   ` Avi Kivity
     [not found]                     ` <4B547C09.8010906-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-01-18 15:24                       ` Peter Zijlstra
2010-01-18 15:41                         ` Alan Cox
2010-01-18 15:44                           ` Peter Zijlstra
2010-01-18 17:11                   ` Gleb Natapov
2010-01-18 15:14               ` Gleb Natapov
2010-01-19  0:12                 ` KOSAKI Motohiro
2010-01-18 16:05       ` Pekka Enberg
2010-01-18 17:08         ` Gleb Natapov [this message]
2010-01-18 18:09           ` Pekka Enberg
2010-01-18 18:19             ` Gleb Natapov
2010-01-18 19:10               ` Alan Cox
2010-01-19  7:17                 ` Gleb Natapov
2010-01-19  7:37                   ` Pekka Enberg
2010-01-19  7:52                     ` Gleb Natapov
     [not found]                       ` <20100119075205.GI14345-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-01-19  8:07                         ` Pekka Enberg
2010-01-19  8:26                           ` Gleb Natapov
     [not found]                             ` <20100119082638.GK14345-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-01-19  8:44                               ` Pekka Enberg
2010-01-19 10:40                                 ` Gleb Natapov
     [not found]                                 ` <84144f021001190044s397c6665qb00af48235d2d818-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-01-19 12:48                                   ` Minchan Kim
2010-01-19 13:18                                     ` Pekka Enberg
2010-01-19 13:26                                       ` Gleb Natapov
2010-01-20  0:24                                 ` KOSAKI Motohiro
2010-01-19 11:54                       ` Alan Cox
2010-01-19 12:07                         ` Gleb Natapov
2010-01-19 13:21                           ` Alan Cox
2010-01-19 14:07                       ` Minchan Kim
2010-01-19 14:14                         ` Gleb Natapov

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=20100118170816.GA22111@redhat.com \
    --to=gleb@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=andrew.c.morrow@gmail.com \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=penberg@cs.helsinki.fi \
    /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).