lkml.org 
[lkml]   [2017]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v2 00/26] IB: Optimize DMA mapping
Date


> -----Original Message-----
> From: Bart Van Assche [mailto:Bart.VanAssche@sandisk.com]
> Sent: Friday, January 13, 2017 5:00 PM
> To: Estrin, Alex <alex.estrin@intel.com>; dledford@redhat.com
> Cc: linux-kernel@vger.kernel.org; linux-rdma@vger.kernel.org;
> gregkh@linuxfoundation.org
> Subject: Re: [PATCH v2 00/26] IB: Optimize DMA mapping
>
> On Fri, 2017-01-13 at 21:08 +0000, Estrin, Alex wrote:
> > It didn't fix the failure... Apparently there is an issue with generic
> > mapping itself.
>
> Hello Alex,
>
> The generic mapping code in lib/dma-virt.c works fine with at least the rxe
> driver. Additionally, as far as I can see the mapping code in lib/dma-virt.c
> is identical to the mapping in drivers/infiniband/sw/rdmavt/dma.c. What is
> not clear to me however is how my patches can have caused the SDMA engine
> error messages? As far as I can see the code in
> drivers/infiniband/hw/hfi1/sdma.c uses PCI DMA mapping operations instead of
> the generic dma_virt_ops. Is that correct?

Hi Bart,
If you refer to code like this
from sdma.c:
/* dma map the coalesce buffer */
addr = dma_map_single(&dd->pcidev->dev,
.....
please see hfi1/verbs.c @ hfi1_register_ib_device()

'dma_device' field is initialized here
-> ibdev->dma_device = &dd->pcidev->dev;
followed by
ret = rvt_register_device(&dd->verbs_dev.rdi);
-> set_dma_ops(rdi->ibdev.dma_device, &dma_virt_ops);

Thanks,
Alex.

\
 
 \ /
  Last update: 2017-01-14 03:06    [W:0.154 / U:7.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site