All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
To: Tom Tucker <tom-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
Cc: Ralph Campbell
	<ralph.campbell-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org>,
	Tom Tucker <tom-/Yg/VP3ZvrM@public.gmane.org>,
	"rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org"
	<rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>,
	"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"brandt-4OHPYypu0djtX7QSmKvirg@public.gmane.org"
	<brandt-4OHPYypu0djtX7QSmKvirg@public.gmane.org>,
	"swise-/Yg/VP3ZvrM@public.gmane.org"
	<swise-/Yg/VP3ZvrM@public.gmane.org>
Subject: Re: [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service
Date: Thu, 29 Jul 2010 14:13:17 -0600	[thread overview]
Message-ID: <20100729201317.GA7920@obsidianresearch.com> (raw)
In-Reply-To: <4C51D15F.8060708-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>

On Thu, Jul 29, 2010 at 02:07:11PM -0500, Tom Tucker wrote:
>>> Add an ib_iomem_get service that converts a vma to an array of
>>> physical addresses. This makes it easier for each device driver to
>>> add support for the reg_io_mr provider method.

Tom, could you say a few words on what this is for?

Also, I'd like to see a strong defence of this new user space API
particularly:
 1) Why can't this be done with the existing ibv_reg_mr, like huge
    pages are.
 2) How is it possible for userspace to know when it should use
    ibv_reg_mr vs ibv_reg_io_mr?
 3) Why can't ibv_dereg_io_mr be entirely handled by ibv_dereg_mr?

On first glance, this seems like a hugely bad API to me :)

Also, if new user space APIs are really needed, they do need man pages
too ...

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2010-07-29 20:13 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-29 16:24 [RFC PATCH 0/4] ibverbs: new verbs for registering I/O memory Tom Tucker
     [not found] ` <20100729162339.14674.15788.stgit-T4OLL4TyM9aNDNWfRnPdfg@public.gmane.org>
2010-07-29 16:25   ` [RFC PATCH 1/4] ibverbs: Add new provider verb for I/O memory registration Tom Tucker
2010-07-29 16:25   ` [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service Tom Tucker
     [not found]     ` <20100729162509.14674.34237.stgit-T4OLL4TyM9aNDNWfRnPdfg@public.gmane.org>
2010-07-29 18:22       ` Ralph Campbell
     [not found]         ` <1280427723.6820.56.camel-/vjeY7uYZjrPXfVEPVhPGq6RkeBMCJyt@public.gmane.org>
2010-07-29 19:07           ` Tom Tucker
     [not found]             ` <4C51D15F.8060708-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-07-29 19:33               ` Ralph Campbell
2010-07-29 20:13               ` Jason Gunthorpe [this message]
     [not found]                 ` <20100729201317.GA7920-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-07-29 20:29                   ` Tom Tucker
     [not found]                     ` <4C51E4B1.6040800-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-07-29 20:41                       ` Jason Gunthorpe
     [not found]                         ` <20100729204150.GC7920-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-07-29 20:54                           ` Tom Tucker
2010-07-29 22:49                           ` [Suspected SPAM] " Ralph Campbell
     [not found]                             ` <1280443789.6820.153.camel-/vjeY7uYZjrPXfVEPVhPGq6RkeBMCJyt@public.gmane.org>
2010-07-29 22:57                               ` Jason Gunthorpe
     [not found]                                 ` <20100729225705.GC26390-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-07-29 23:13                                   ` Ralph Campbell
2010-07-30  2:16                                   ` Tom Tucker
2010-07-29 21:20       ` Tom Tucker
2010-07-29 16:25   ` [RFC PATCH 3/4] uverbs_cmd: Add uverbs command definitions for reg_io_mr Tom Tucker
2010-07-29 16:25   ` [RFC PATCH 4/4] mthca: Add support for reg_io_mr and unreg_io_mr Tom Tucker

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=20100729201317.GA7920@obsidianresearch.com \
    --to=jgunthorpe-epgobjl8dl3ta4ec/59zmfatqe2ktcn/@public.gmane.org \
    --cc=brandt-4OHPYypu0djtX7QSmKvirg@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=ralph.campbell-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org \
    --cc=rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
    --cc=swise-/Yg/VP3ZvrM@public.gmane.org \
    --cc=tom-/Yg/VP3ZvrM@public.gmane.org \
    --cc=tom-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.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 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.