Re: [Suspected SPAM] Re: [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service

2010-07-29 Thread Ralph Campbell
On Thu, 2010-07-29 at 13:41 -0700, Jason Gunthorpe wrote: On Thu, Jul 29, 2010 at 03:29:37PM -0500, Tom Tucker wrote: 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.

Re: [Suspected SPAM] Re: [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service

2010-07-29 Thread Jason Gunthorpe
You would need to modify ib_umem_get() to check for the VM_PFNMAP flag and build the struct ib_umem similar to the proposed ib_iomem_get(). However, the page reference counting/sharing issue would need to be solved. I think there are kernel level callbacks for this that could be used. But in

Re: [Suspected SPAM] Re: [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service

2010-07-29 Thread Ralph Campbell
On Thu, 2010-07-29 at 15:57 -0700, Jason Gunthorpe wrote: You would need to modify ib_umem_get() to check for the VM_PFNMAP flag and build the struct ib_umem similar to the proposed ib_iomem_get(). However, the page reference counting/sharing issue would need to be solved. I think there

Re: [Suspected SPAM] Re: [RFC PATCH 2/4] uverbs: Add common ib_iomem_get service

2010-07-29 Thread Tom Tucker
On 7/29/10 5:57 PM, Jason Gunthorpe wrote: You would need to modify ib_umem_get() to check for the VM_PFNMAP flag and build the struct ib_umem similar to the proposed ib_iomem_get(). However, the page reference counting/sharing issue would need to be solved. I think there are kernel level