From: Robin Holt <holt@sgi.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
andrea@qumranet.com, holt@sgi.com, avi@qumranet.com,
izike@qumranet.com, kvm-devel@lists.sourceforge.net,
a.p.zijlstra@chello.nl, steiner@sgi.com,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
daniel.blueman@quadrics.com, general@lists.openfabrics.org
Subject: Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 8 Feb 2008 17:36:37 -0600 [thread overview]
Message-ID: <20080208233636.GG26564@sgi.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081528070.4036@schroedinger.engr.sgi.com>
On Fri, Feb 08, 2008 at 03:32:19PM -0800, Christoph Lameter wrote:
> On Fri, 8 Feb 2008, Andrew Morton wrote:
>
> > What about ib_umem_get()?
>
> Ok. It pins using an elevated refcount. Same as XPmem right now. With that
> we effectively pin a page (page migration will fail) but we will
> continually be reclaiming the page and may repeatedly try to move it. We
> have issues with XPmem causing too many pages to be pinned and thus the
> OOM getting into weird behavior modes (OOM or stop lru scanning due to
> all_reclaimable set).
>
> An elevated refcount will also not be noticed by any of the schemes under
> consideration to improve LRU scanning performance.
Christoph, I am not sure what you are saying here. With v4 and later,
I thought we were able to use the rmap invalidation to remove the ref
count that XPMEM was holding and therefore be able to swapout. Did I miss
something? I agree the existing XPMEM does pin. I hope we are not saying
the XPMEM based upon these patches will not be able to swap/migrate.
Thanks,
Robin
WARNING: multiple messages have this Message-ID (diff)
From: Robin Holt <holt@sgi.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: andrea@qumranet.com, a.p.zijlstra@chello.nl, linux-mm@kvack.org,
izike@qumranet.com, steiner@sgi.com,
linux-kernel@vger.kernel.org, avi@qumranet.com,
kvm-devel@lists.sourceforge.net, daniel.blueman@quadrics.com,
holt@sgi.com, general@lists.openfabrics.org,
Andrew Morton <akpm@linux-foundation.org>
Subject: [ofa-general] Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 8 Feb 2008 17:36:37 -0600 [thread overview]
Message-ID: <20080208233636.GG26564@sgi.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081528070.4036@schroedinger.engr.sgi.com>
On Fri, Feb 08, 2008 at 03:32:19PM -0800, Christoph Lameter wrote:
> On Fri, 8 Feb 2008, Andrew Morton wrote:
>
> > What about ib_umem_get()?
>
> Ok. It pins using an elevated refcount. Same as XPmem right now. With that
> we effectively pin a page (page migration will fail) but we will
> continually be reclaiming the page and may repeatedly try to move it. We
> have issues with XPmem causing too many pages to be pinned and thus the
> OOM getting into weird behavior modes (OOM or stop lru scanning due to
> all_reclaimable set).
>
> An elevated refcount will also not be noticed by any of the schemes under
> consideration to improve LRU scanning performance.
Christoph, I am not sure what you are saying here. With v4 and later,
I thought we were able to use the rmap invalidation to remove the ref
count that XPMEM was holding and therefore be able to swapout. Did I miss
something? I agree the existing XPMEM does pin. I hope we are not saying
the XPMEM based upon these patches will not be able to swap/migrate.
Thanks,
Robin
WARNING: multiple messages have this Message-ID (diff)
From: Robin Holt <holt@sgi.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
andrea@qumranet.com, holt@sgi.com, avi@qumranet.com,
izike@qumranet.com, kvm-devel@lists.sourceforge.net,
a.p.zijlstra@chello.nl, steiner@sgi.com,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
daniel.blueman@quadrics.com, general@lists.openfabrics.org
Subject: Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 8 Feb 2008 17:36:37 -0600 [thread overview]
Message-ID: <20080208233636.GG26564@sgi.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081528070.4036@schroedinger.engr.sgi.com>
On Fri, Feb 08, 2008 at 03:32:19PM -0800, Christoph Lameter wrote:
> On Fri, 8 Feb 2008, Andrew Morton wrote:
>
> > What about ib_umem_get()?
>
> Ok. It pins using an elevated refcount. Same as XPmem right now. With that
> we effectively pin a page (page migration will fail) but we will
> continually be reclaiming the page and may repeatedly try to move it. We
> have issues with XPmem causing too many pages to be pinned and thus the
> OOM getting into weird behavior modes (OOM or stop lru scanning due to
> all_reclaimable set).
>
> An elevated refcount will also not be noticed by any of the schemes under
> consideration to improve LRU scanning performance.
Christoph, I am not sure what you are saying here. With v4 and later,
I thought we were able to use the rmap invalidation to remove the ref
count that XPMEM was holding and therefore be able to swapout. Did I miss
something? I agree the existing XPMEM does pin. I hope we are not saying
the XPMEM based upon these patches will not be able to swap/migrate.
Thanks,
Robin
--
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>
next prev parent reply other threads:[~2008-02-08 23:36 UTC|newest]
Thread overview: 230+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-08 22:06 [patch 0/6] MMU Notifiers V6 Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 1/6] mmu_notifier: Core code Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 2/6] mmu_notifier: Callbacks to invalidate address ranges Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 3/6] mmu_notifier: invalidate_page callbacks Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 4/6] mmu_notifier: Skeleton driver for a simple mmu_notifier Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 5/6] mmu_notifier: Support for drivers with revers maps (f.e. for XPmem) Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:06 ` [patch 6/6] mmu_rmap_notifier: Skeleton for complex driver that uses its own rmaps Christoph Lameter
2008-02-08 22:06 ` Christoph Lameter
2008-02-08 22:23 ` [patch 0/6] MMU Notifiers V6 Andrew Morton
2008-02-08 22:23 ` Andrew Morton
2008-02-08 22:23 ` [ofa-general] " Andrew Morton
2008-02-08 23:32 ` Christoph Lameter
2008-02-08 23:32 ` Christoph Lameter
2008-02-08 23:32 ` Christoph Lameter
2008-02-08 23:36 ` Robin Holt [this message]
2008-02-08 23:36 ` Robin Holt
2008-02-08 23:36 ` [ofa-general] " Robin Holt
2008-02-08 23:41 ` Christoph Lameter
2008-02-08 23:41 ` Christoph Lameter
2008-02-08 23:41 ` [ofa-general] " Christoph Lameter
2008-02-08 23:43 ` Robin Holt
2008-02-08 23:43 ` Robin Holt
2008-02-08 23:43 ` Robin Holt
2008-02-08 23:56 ` Andrew Morton
2008-02-08 23:56 ` Andrew Morton
2008-02-08 23:56 ` [ofa-general] " Andrew Morton
2008-02-09 0:05 ` Christoph Lameter
2008-02-09 0:05 ` Christoph Lameter
2008-02-09 0:05 ` [ofa-general] " Christoph Lameter
2008-02-09 0:12 ` Roland Dreier
2008-02-09 0:12 ` Roland Dreier
2008-02-09 0:12 ` Roland Dreier
2008-02-09 0:16 ` Christoph Lameter
2008-02-09 0:16 ` Christoph Lameter
2008-02-09 0:16 ` Christoph Lameter
2008-02-09 0:21 ` [ofa-general] trying to get of all lists R S
2008-02-09 0:22 ` [ofa-general] Re: [patch 0/6] MMU Notifiers V6 Roland Dreier
2008-02-09 0:22 ` Roland Dreier
2008-02-09 0:36 ` Christoph Lameter
2008-02-09 0:36 ` Christoph Lameter
2008-02-09 0:36 ` Christoph Lameter
2008-02-09 1:24 ` Andrea Arcangeli
2008-02-09 1:24 ` Andrea Arcangeli
2008-02-09 1:24 ` Andrea Arcangeli
2008-02-09 1:27 ` Christoph Lameter
2008-02-09 1:27 ` Christoph Lameter
2008-02-09 1:27 ` Christoph Lameter
2008-02-09 1:56 ` Andrea Arcangeli
2008-02-09 1:56 ` Andrea Arcangeli
2008-02-09 1:56 ` Andrea Arcangeli
2008-02-09 2:16 ` Christoph Lameter
2008-02-09 2:16 ` Christoph Lameter
2008-02-09 2:16 ` Christoph Lameter
2008-02-09 12:55 ` Rik van Riel
2008-02-09 12:55 ` Rik van Riel
2008-02-09 12:55 ` Rik van Riel
2008-02-09 21:46 ` Christoph Lameter
2008-02-09 21:46 ` Christoph Lameter
2008-02-09 21:46 ` Christoph Lameter
2008-02-11 22:40 ` Demand paging for memory regions (was Re: MMU Notifiers V6) Roland Dreier
2008-02-11 22:40 ` Roland Dreier
2008-02-11 22:40 ` [ofa-general] " Roland Dreier
2008-02-12 22:01 ` Steve Wise
2008-02-12 22:01 ` Steve Wise
2008-02-12 22:01 ` [ofa-general] " Steve Wise
2008-02-12 22:10 ` Christoph Lameter
2008-02-12 22:10 ` Christoph Lameter
2008-02-12 22:10 ` [ofa-general] " Christoph Lameter
2008-02-12 22:41 ` [ofa-general] Re: Demand paging for memory regions Roland Dreier
2008-02-12 22:41 ` Roland Dreier
2008-02-12 22:41 ` Roland Dreier
2008-02-12 23:14 ` Felix Marti
2008-02-12 23:14 ` Felix Marti
2008-02-12 23:14 ` Felix Marti
2008-02-13 0:57 ` Christoph Lameter
2008-02-13 0:57 ` Christoph Lameter
2008-02-13 0:57 ` Christoph Lameter
2008-02-14 15:09 ` Steve Wise
2008-02-14 15:09 ` Steve Wise
2008-02-14 15:09 ` Steve Wise
2008-02-14 15:53 ` Robin Holt
2008-02-14 15:53 ` Robin Holt
2008-02-14 16:23 ` Steve Wise
2008-02-14 16:23 ` Steve Wise
2008-02-14 16:23 ` Steve Wise
2008-02-14 17:48 ` Caitlin Bestler
2008-02-14 17:48 ` Caitlin Bestler
2008-02-14 17:48 ` Caitlin Bestler
2008-02-14 20:47 ` David Singleton
2008-02-15 9:55 ` Robin Holt
2008-02-14 19:39 ` Christoph Lameter
2008-02-14 19:39 ` Christoph Lameter
2008-02-14 19:39 ` Christoph Lameter
2008-02-14 20:17 ` Caitlin Bestler
2008-02-14 20:17 ` Caitlin Bestler
2008-02-14 20:17 ` Caitlin Bestler
2008-02-14 20:20 ` Christoph Lameter
2008-02-14 20:20 ` Christoph Lameter
2008-02-14 20:20 ` Christoph Lameter
2008-02-14 22:43 ` Caitlin Bestler
2008-02-14 22:43 ` Caitlin Bestler
2008-02-14 22:43 ` Caitlin Bestler
2008-02-14 22:48 ` Christoph Lameter
2008-02-14 22:48 ` Christoph Lameter
2008-02-14 22:48 ` Christoph Lameter
2008-02-15 1:26 ` Caitlin Bestler
2008-02-15 1:26 ` Caitlin Bestler
2008-02-15 1:26 ` Caitlin Bestler
2008-02-15 2:37 ` Christoph Lameter
2008-02-15 2:37 ` Christoph Lameter
2008-02-15 2:37 ` Christoph Lameter
2008-02-15 18:09 ` Caitlin Bestler
2008-02-15 18:09 ` Caitlin Bestler
2008-02-15 18:09 ` Caitlin Bestler
2008-02-15 18:45 ` Christoph Lameter
2008-02-15 18:45 ` Christoph Lameter
2008-02-15 18:45 ` Christoph Lameter
2008-02-15 18:53 ` Caitlin Bestler
2008-02-15 18:53 ` Caitlin Bestler
2008-02-15 18:53 ` Caitlin Bestler
2008-02-15 20:02 ` Christoph Lameter
2008-02-15 20:02 ` Christoph Lameter
2008-02-15 20:02 ` Christoph Lameter
2008-02-15 20:14 ` Caitlin Bestler
2008-02-15 20:14 ` Caitlin Bestler
2008-02-15 20:14 ` Caitlin Bestler
2008-02-15 22:50 ` Christoph Lameter
2008-02-15 22:50 ` Christoph Lameter
2008-02-15 22:50 ` Christoph Lameter
2008-02-15 23:50 ` Caitlin Bestler
2008-02-15 23:50 ` Caitlin Bestler
2008-02-15 23:50 ` Caitlin Bestler
2008-02-12 23:23 ` Jason Gunthorpe
2008-02-12 23:23 ` Jason Gunthorpe
2008-02-12 23:23 ` Jason Gunthorpe
2008-02-13 1:01 ` Christoph Lameter
2008-02-13 1:01 ` Christoph Lameter
2008-02-13 1:01 ` Christoph Lameter
2008-02-13 1:26 ` Jason Gunthorpe
2008-02-13 1:26 ` Jason Gunthorpe
2008-02-13 1:26 ` Jason Gunthorpe
2008-02-13 1:45 ` Steve Wise
2008-02-13 1:45 ` Steve Wise
2008-02-13 1:45 ` Steve Wise
2008-02-13 2:35 ` Christoph Lameter
2008-02-13 2:35 ` Christoph Lameter
2008-02-13 2:35 ` Christoph Lameter
2008-02-13 3:25 ` Jason Gunthorpe
2008-02-13 3:25 ` Jason Gunthorpe
2008-02-13 3:56 ` Patrick Geoffray
2008-02-13 3:56 ` Patrick Geoffray
2008-02-13 4:26 ` Jason Gunthorpe
2008-02-13 4:26 ` Jason Gunthorpe
2008-02-13 4:47 ` Patrick Geoffray
2008-02-13 4:47 ` Patrick Geoffray
2008-02-13 18:51 ` Christoph Lameter
2008-02-13 18:51 ` Christoph Lameter
2008-02-13 18:51 ` Christoph Lameter
2008-02-13 19:51 ` Jason Gunthorpe
2008-02-13 19:51 ` Jason Gunthorpe
2008-02-13 19:51 ` Jason Gunthorpe
2008-02-13 20:36 ` Christoph Lameter
2008-02-13 20:36 ` Christoph Lameter
2008-02-13 20:36 ` Christoph Lameter
2008-02-13 4:09 ` Christian Bell
2008-02-13 4:09 ` Christian Bell
2008-02-13 4:09 ` Christian Bell
2008-02-13 19:00 ` Christoph Lameter
2008-02-13 19:00 ` Christoph Lameter
2008-02-13 19:00 ` Christoph Lameter
2008-02-13 19:46 ` Christian Bell
2008-02-13 19:46 ` Christian Bell
2008-02-13 20:32 ` Christoph Lameter
2008-02-13 20:32 ` Christoph Lameter
2008-02-13 20:32 ` Christoph Lameter
2008-02-13 22:44 ` Kanoj Sarcar
2008-02-13 22:44 ` Kanoj Sarcar
2008-02-13 22:44 ` Kanoj Sarcar
2008-02-13 23:02 ` Christoph Lameter
2008-02-13 23:02 ` Christoph Lameter
2008-02-13 23:02 ` Christoph Lameter
2008-02-13 23:43 ` Kanoj Sarcar
2008-02-13 23:43 ` Kanoj Sarcar
2008-02-13 23:43 ` Kanoj Sarcar
2008-02-13 23:48 ` Jesse Barnes
2008-02-13 23:48 ` Jesse Barnes
2008-02-13 23:48 ` [ofa-general] " Jesse Barnes
2008-02-14 0:56 ` Andrea Arcangeli
2008-02-14 0:56 ` Andrea Arcangeli
2008-02-14 0:56 ` Andrea Arcangeli
2008-02-14 19:35 ` Christoph Lameter
2008-02-14 19:35 ` Christoph Lameter
2008-02-14 19:35 ` Christoph Lameter
2008-02-13 23:23 ` Pete Wyckoff
2008-02-13 23:23 ` Pete Wyckoff
2008-02-13 23:23 ` Pete Wyckoff
2008-02-14 0:01 ` Jason Gunthorpe
2008-02-14 0:01 ` Jason Gunthorpe
2008-02-27 22:11 ` Christoph Lameter
2008-02-27 22:11 ` Christoph Lameter
2008-02-27 22:11 ` Christoph Lameter
2008-02-13 1:55 ` Christian Bell
2008-02-13 1:55 ` Christian Bell
2008-02-13 1:55 ` Christian Bell
2008-02-13 2:19 ` Christoph Lameter
2008-02-13 2:19 ` Christoph Lameter
2008-02-13 2:19 ` Christoph Lameter
2008-02-13 0:56 ` Christoph Lameter
2008-02-13 0:56 ` Christoph Lameter
2008-02-13 0:56 ` Christoph Lameter
2008-02-13 12:11 ` Christoph Raisch
2008-02-13 12:11 ` Christoph Raisch
2008-02-13 12:11 ` Christoph Raisch
2008-02-13 19:02 ` Christoph Lameter
2008-02-13 19:02 ` Christoph Lameter
2008-02-13 19:02 ` Christoph Lameter
2008-02-09 0:12 ` [patch 0/6] MMU Notifiers V6 Andrew Morton
2008-02-09 0:12 ` Andrew Morton
2008-02-09 0:12 ` [ofa-general] " Andrew Morton
2008-02-09 0:18 ` Christoph Lameter
2008-02-09 0:18 ` Christoph Lameter
2008-02-09 0:18 ` [ofa-general] " Christoph Lameter
2008-02-13 14:31 ` Jack Steiner
2008-02-13 14:31 ` Jack Steiner
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=20080208233636.GG26564@sgi.com \
--to=holt@sgi.com \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=andrea@qumranet.com \
--cc=avi@qumranet.com \
--cc=clameter@sgi.com \
--cc=daniel.blueman@quadrics.com \
--cc=general@lists.openfabrics.org \
--cc=izike@qumranet.com \
--cc=kvm-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=steiner@sgi.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.