All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gleb Natapov <gleb@redhat.com>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	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>,
	Rik van Riel <riel@redhat.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	minchan.kim@gmail.com
Subject: Re: [PATCH v6] add MAP_UNLOCKED mmap flag
Date: Tue, 19 Jan 2010 12:40:58 +0200	[thread overview]
Message-ID: <20100119104058.GL14345@redhat.com> (raw)
In-Reply-To: <84144f021001190044s397c6665qb00af48235d2d818@mail.gmail.com>

On Tue, Jan 19, 2010 at 10:44:23AM +0200, Pekka Enberg wrote:
> On Tue, Jan 19, 2010 at 10:26 AM, Gleb Natapov <gleb@redhat.com> wrote:
> >> me that you're trying to use a big hammer (mlock) when a polite hint
> >> for the VM would probably be sufficient for it do its job.
> >>
> > I what to tell to VM "swap this, don't swap that" and as far as I see
> > there is no other way to do it currently.
> 
> Yeah, which is why I was suggesting that maybe posix_madvise() needs
> to be extended to have a MADV_NEED_BUT_LESS_IMPORTANT flag that can be
> used as a hint by mm/vmscan.c to first swap the guest address spaces.
> 
If such thing would exist may be I would have used it since swapping out
of a wrong page is not live or death matter in my case, but mlockall()
provides me with exactly what I need and without swapping out wrong
pages. Speaking about adding such madvise call wouldn't it be even
harder to justify? It obviously not good enough for real-time use and my
case, I admit, is unusual. Also if we start prioritise memory why stop
on binary, why not set value like "this memory is more important then
that memory by factor of 5"?

--
			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>

WARNING: multiple messages have this Message-ID (diff)
From: Gleb Natapov <gleb@redhat.com>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	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>,
	Rik van Riel <riel@redhat.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	minchan.kim@gmail.com
Subject: Re: [PATCH v6] add MAP_UNLOCKED mmap flag
Date: Tue, 19 Jan 2010 12:40:58 +0200	[thread overview]
Message-ID: <20100119104058.GL14345@redhat.com> (raw)
In-Reply-To: <84144f021001190044s397c6665qb00af48235d2d818@mail.gmail.com>

On Tue, Jan 19, 2010 at 10:44:23AM +0200, Pekka Enberg wrote:
> On Tue, Jan 19, 2010 at 10:26 AM, Gleb Natapov <gleb@redhat.com> wrote:
> >> me that you're trying to use a big hammer (mlock) when a polite hint
> >> for the VM would probably be sufficient for it do its job.
> >>
> > I what to tell to VM "swap this, don't swap that" and as far as I see
> > there is no other way to do it currently.
> 
> Yeah, which is why I was suggesting that maybe posix_madvise() needs
> to be extended to have a MADV_NEED_BUT_LESS_IMPORTANT flag that can be
> used as a hint by mm/vmscan.c to first swap the guest address spaces.
> 
If such thing would exist may be I would have used it since swapping out
of a wrong page is not live or death matter in my case, but mlockall()
provides me with exactly what I need and without swapping out wrong
pages. Speaking about adding such madvise call wouldn't it be even
harder to justify? It obviously not good enough for real-time use and my
case, I admit, is unusual. Also if we start prioritise memory why stop
on binary, why not set value like "this memory is more important then
that memory by factor of 5"?

--
			Gleb.

  reply	other threads:[~2010-01-19 10:40 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-18 13:37 [PATCH v6] add MAP_UNLOCKED mmap flag Gleb Natapov
2010-01-18 13:37 ` Gleb Natapov
2010-01-18 13:37 ` Gleb Natapov
     [not found] ` <20100118133755.GG30698-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-01-18 14:09   ` Pekka Enberg
2010-01-18 14:09     ` Pekka Enberg
2010-01-18 14:09     ` Pekka Enberg
2010-01-18 14:19     ` Gleb Natapov
2010-01-18 14:19       ` Gleb Natapov
2010-01-18 14:32       ` Alan Cox
2010-01-18 14:32         ` Alan Cox
2010-01-18 14:35         ` Gleb Natapov
2010-01-18 14:35           ` Gleb Natapov
2010-01-18 14:49         ` Peter Zijlstra
2010-01-18 14:49           ` Peter Zijlstra
2010-01-18 15:01           ` Gleb Natapov
2010-01-18 15:01             ` Gleb Natapov
2010-01-18 15:06             ` Peter Zijlstra
2010-01-18 15:06               ` Peter Zijlstra
2010-01-18 15:11               ` Avi Kivity
2010-01-18 15:11                 ` Avi Kivity
2010-01-18 15:14                 ` Peter Zijlstra
2010-01-18 15:14                   ` Peter Zijlstra
2010-01-18 15:19                   ` Avi Kivity
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:24                         ` Peter Zijlstra
2010-01-18 15:24                         ` Peter Zijlstra
2010-01-18 15:41                         ` Alan Cox
2010-01-18 15:41                           ` Alan Cox
2010-01-18 15:44                           ` Peter Zijlstra
2010-01-18 15:44                             ` Peter Zijlstra
2010-01-18 17:11                   ` Gleb Natapov
2010-01-18 17:11                     ` Gleb Natapov
2010-01-18 15:14               ` Gleb Natapov
2010-01-18 15:14                 ` Gleb Natapov
2010-01-19  0:12                 ` KOSAKI Motohiro
2010-01-19  0:12                   ` KOSAKI Motohiro
2010-01-18 16:05       ` Pekka Enberg
2010-01-18 16:05         ` Pekka Enberg
2010-01-18 17:08         ` Gleb Natapov
2010-01-18 17:08           ` Gleb Natapov
2010-01-18 18:09           ` Pekka Enberg
2010-01-18 18:09             ` Pekka Enberg
2010-01-18 18:19             ` Gleb Natapov
2010-01-18 18:19               ` Gleb Natapov
2010-01-18 19:10               ` Alan Cox
2010-01-18 19:10                 ` Alan Cox
2010-01-19  7:17                 ` Gleb Natapov
2010-01-19  7:17                   ` Gleb Natapov
2010-01-19  7:37                   ` Pekka Enberg
2010-01-19  7:37                     ` Pekka Enberg
2010-01-19  7:52                     ` Gleb Natapov
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:07                           ` Pekka Enberg
2010-01-19  8:07                           ` Pekka Enberg
2010-01-19  8:26                           ` Gleb Natapov
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  8:44                                 ` Pekka Enberg
2010-01-19  8:44                                 ` Pekka Enberg
2010-01-19 10:40                                 ` Gleb Natapov [this message]
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 12:48                                     ` Minchan Kim
2010-01-19 12:48                                     ` Minchan Kim
2010-01-19 13:18                                     ` Pekka Enberg
2010-01-19 13:18                                       ` Pekka Enberg
2010-01-19 13:26                                       ` Gleb Natapov
2010-01-19 13:26                                         ` Gleb Natapov
2010-01-20  0:24                                 ` KOSAKI Motohiro
2010-01-20  0:24                                   ` KOSAKI Motohiro
2010-01-19 11:54                       ` Alan Cox
2010-01-19 11:54                         ` Alan Cox
2010-01-19 12:07                         ` Gleb Natapov
2010-01-19 12:07                           ` Gleb Natapov
2010-01-19 13:21                           ` Alan Cox
2010-01-19 13:21                             ` Alan Cox
2010-01-19 14:07                       ` Minchan Kim
2010-01-19 14:07                         ` Minchan Kim
2010-01-19 14:14                         ` Gleb Natapov
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=20100119104058.GL14345@redhat.com \
    --to=gleb@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=andrew.c.morrow@gmail.com \
    --cc=hannes@cmpxchg.org \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=minchan.kim@gmail.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=penberg@cs.helsinki.fi \
    --cc=riel@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.