From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH v6.1] infiniband: dma-mapping: Use unsigned long for dma_attrs Date: Wed, 13 Jul 2016 13:17:54 +0200 Message-ID: <57862362.4050608@samsung.com> References: <1468399300-5399-17-git-send-email-k.kozlowski@samsung.com> <1468408466-7369-1-git-send-email-k.kozlowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1468408466-7369-1-git-send-email-k.kozlowski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Andrew Morton Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org, Bartlomiej Zolnierkiewicz , Bart Van Assche , Doug Ledford , Sean Hefty , Hal Rosenstock , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org On 07/13/2016 01:14 PM, Krzysztof Kozlowski wrote: > Split out subsystem specific changes for easier reviews. This will be > squashed with main commit. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Update only to this patch from the set. > > Changes since v6: > 1. Rename attrs->dma_attrs arguments in map/unmap functions (suggested > by Bart Van Assche). > --- > drivers/infiniband/core/umem.c | 7 +++---- > include/rdma/ib_verbs.h | 13 +++++++------ > 2 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > index fe4d2e1a8b58..c68746ce6624 100644 > --- a/drivers/infiniband/core/umem.c > +++ b/drivers/infiniband/core/umem.c > @@ -37,7 +37,6 @@ > #include > #include > #include > -#include > #include > #include > > @@ -92,12 +91,12 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > unsigned long npages; > int ret; > int i; > - DEFINE_DMA_ATTRS(attrs); > + unsigned long dma_attrs = 0; > struct scatterlist *sg, *sg_list_start; > int need_release = 0; > > if (dmasync) > - dma_set_attr(DMA_ATTR_WRITE_BARRIER, &attrs); > + dma_attrs |= DMA_ATTR_WRITE_BARRIER; > > if (!size) > return ERR_PTR(-EINVAL); > @@ -215,7 +214,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > umem->sg_head.sgl, > umem->npages, > DMA_BIDIRECTIONAL, > - &attrs); > + dma_attrs); > > if (umem->nmap <= 0) { > ret = -ENOMEM; > diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h > index 156673a80704..ae9cfcac20f4 100644 > --- a/include/rdma/ib_verbs.h > +++ b/include/rdma/ib_verbs.h > @@ -2910,7 +2910,7 @@ static inline void ib_dma_unmap_single(struct ib_device *dev, > static inline u64 ib_dma_map_single_attrs(struct ib_device *dev, > void *cpu_addr, size_t size, > enum dma_data_direction direction, > - struct dma_attrs *attrs) > + unsigned long attrs) > { > return dma_map_single_attrs(dev->dma_device, cpu_addr, size, > direction, attrs); > @@ -2919,7 +2919,7 @@ static inline u64 ib_dma_map_single_attrs(struct ib_device *dev, > static inline void ib_dma_unmap_single_attrs(struct ib_device *dev, > u64 addr, size_t size, > enum dma_data_direction direction, > - struct dma_attrs *attrs) > + unsigned long attrs) Oh crap, I didn't rename it everywhere. Let me do a respin... BR, Krzysztof -- 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