All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roland Dreier <rdreier@cisco.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	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,
	Robin Holt <holt@sgi.com>,
	general@lists.openfabrics.org
Subject: Re: [ofa-general] Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 08 Feb 2008 16:12:42 -0800	[thread overview]
Message-ID: <adaprv70yyt.fsf@cisco.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081603430.4543@schroedinger.engr.sgi.com> (Christoph Lameter's message of "Fri, 8 Feb 2008 16:05:00 -0800 (PST)")

 > We have done several rounds of discussion on linux-kernel about this so 
 > far and the IB folks have not shown up to join in. I have tried to make 
 > this as general as possible.

Sorry, this has been on my "things to look at" list for a while, but I
haven't gotten a chance to really understand where things are yet.

In general, this MMU notifier stuff will only be useful to a subset of
InfiniBand/RDMA hardware.  Some adapters are smart enough to handle
changing the IO virtual -> bus/physical mapping on the fly, but some
aren't.  For the dumb adapters, I think the current ib_umem_get() is
pretty close to as good as we can get: we have to keep the physical
pages pinned for as long as the adapter is allowed to DMA into the
memory region.

For the smart adapters, we just need a chance to change the adapter's
page table when the kernel/CPU's mapping changes, and naively, this
stuff looks like it would work.

Andrew, does that help?

- R.

WARNING: multiple messages have this Message-ID (diff)
From: Roland Dreier <rdreier@cisco.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: andrea@qumranet.com, a.p.zijlstra@chello.nl, izike@qumranet.com,
	steiner@sgi.com, linux-kernel@vger.kernel.org, avi@qumranet.com,
	linux-mm@kvack.org, daniel.blueman@quadrics.com,
	Robin Holt <holt@sgi.com>,
	general@lists.openfabrics.org,
	Andrew Morton <akpm@linux-foundation.org>,
	kvm-devel@lists.sourceforge.net
Subject: Re: [ofa-general] Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 08 Feb 2008 16:12:42 -0800	[thread overview]
Message-ID: <adaprv70yyt.fsf@cisco.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081603430.4543@schroedinger.engr.sgi.com> (Christoph Lameter's message of "Fri, 8 Feb 2008 16:05:00 -0800 (PST)")

 > We have done several rounds of discussion on linux-kernel about this so 
 > far and the IB folks have not shown up to join in. I have tried to make 
 > this as general as possible.

Sorry, this has been on my "things to look at" list for a while, but I
haven't gotten a chance to really understand where things are yet.

In general, this MMU notifier stuff will only be useful to a subset of
InfiniBand/RDMA hardware.  Some adapters are smart enough to handle
changing the IO virtual -> bus/physical mapping on the fly, but some
aren't.  For the dumb adapters, I think the current ib_umem_get() is
pretty close to as good as we can get: we have to keep the physical
pages pinned for as long as the adapter is allowed to DMA into the
memory region.

For the smart adapters, we just need a chance to change the adapter's
page table when the kernel/CPU's mapping changes, and naively, this
stuff looks like it would work.

Andrew, does that help?

- R.

WARNING: multiple messages have this Message-ID (diff)
From: Roland Dreier <rdreier@cisco.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	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,
	Robin Holt <holt@sgi.com>,
	general@lists.openfabrics.org
Subject: Re: [ofa-general] Re: [patch 0/6] MMU Notifiers V6
Date: Fri, 08 Feb 2008 16:12:42 -0800	[thread overview]
Message-ID: <adaprv70yyt.fsf@cisco.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0802081603430.4543@schroedinger.engr.sgi.com> (Christoph Lameter's message of "Fri, 8 Feb 2008 16:05:00 -0800 (PST)")

Sorry, this has been on my "things to look at" list for a while, but I
haven't gotten a chance to really understand where things are yet.

In general, this MMU notifier stuff will only be useful to a subset of
InfiniBand/RDMA hardware.  Some adapters are smart enough to handle
changing the IO virtual -> bus/physical mapping on the fly, but some
aren't.  For the dumb adapters, I think the current ib_umem_get() is
pretty close to as good as we can get: we have to keep the physical
pages pinned for as long as the adapter is allowed to DMA into the
memory region.

For the smart adapters, we just need a chance to change the adapter's
page table when the kernel/CPU's mapping changes, and naively, this
stuff looks like it would work.

Andrew, does that help?

- R.

--
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:[~2008-02-09  0:13 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
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 [this message]
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=adaprv70yyt.fsf@cisco.com \
    --to=rdreier@cisco.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=holt@sgi.com \
    --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.