On Tue, Feb 19, 2019 at 08:57:40AM -0600, Shiraz Saleem wrote:
> Combine contiguous regions of PAGE_SIZE pages
> into single scatter list entries while adding
> to the scatter table. This minimizes the number
> of the entries in the scatter list and reduces
> the DMA mapping overhead, particularly with the
> IOMMU.
>
> Suggested-by: Jason Gunthorpe <jgg@xxxxxxxx>
> Reviewed-by: Michael J. Ruhl <michael.j.ruhl@xxxxxxxxx>
> Signed-off-by: Shiraz Saleem <shiraz.saleem@xxxxxxxxx>
> ---
> drivers/infiniband/core/umem.c | 92 ++++++++++++++++++++++++++++++++++--------
> include/rdma/ib_umem.h | 17 ++++----
> 2 files changed, 84 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c
> index b69d3ef..e8611b7 100644
> --- a/drivers/infiniband/core/umem.c
> +++ b/drivers/infiniband/core/umem.c
> @@ -39,25 +39,22 @@
> #include <linux/export.h>
> #include <linux/hugetlb.h>
> #include <linux/slab.h>
> +#include <linux/pagemap.h>
> #include <rdma/ib_umem_odp.h>
>
> #include "uverbs.h"
>
> -
> static void __ib_umem_release(struct ib_device *dev, struct ib_umem *umem, int dirty)
> {
> - struct scatterlist *sg;
> + struct sg_page_iter sg_iter;
> struct page *page;
> - int i;
>
> if (umem->nmap > 0)
> - ib_dma_unmap_sg(dev, umem->sg_head.sgl,
> - umem->npages,
> + ib_dma_unmap_sg(dev, umem->sg_head.sgl, umem->sg_nents,
> DMA_BIDIRECTIONAL);
>
> - for_each_sg(umem->sg_head.sgl, sg, umem->npages, i) {
> -
> - page = sg_page(sg);
> + for_each_sg_page(umem->sg_head.sgl, &sg_iter, umem->sg_nents, 0) {
> + page = sg_page_iter_page(&sg_iter);
> if (!PageDirty(page) && umem->writable && dirty)
> set_page_dirty_lock(page);
> put_page(page);
> @@ -66,6 +63,67 @@ static void __ib_umem_release(struct ib_device *dev, struct ib_umem *umem, int d
> sg_free_table(&umem->sg_head);
> }
>
> +/* ib_umem_add_sg_table - Add N contiguous pages to scatter table
> + *
> + * sg: current scatterlist entry
> + * page_list: array of npage struct page pointers
> + * npages: number of pages in page_list
> + * nents: [out] number of entries in the scatterlist
> + *
> + * Return new end of scatterlist
> + */
> +struct scatterlist *ib_umem_add_sg_table(struct scatterlist *sg,
> + struct page **page_list,
> + unsigned long npages,
> + int *nents)
It produces compilation warnings for me, you forgot to put static in
front of function declaration.
500b1c079f73 (HEAD -> build) RDMA/umem: Remove hugetlb flag
drivers/infiniband/core/umem.c:75:21: warning: no previous prototype for
_ib_umem_add_sg_table_ [-Wmissing-prototypes]
struct scatterlist *ib_umem_add_sg_table(struct scatterlist *sg,
^~~~~~~~~~~~~~~~~~~~
Thanks
Attachment:
signature.asc
Description: PGP signature
