linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v5 27/26] IB/hfi1: Remove fast registration from the code
@ 2015-10-29 15:33 Sagi Grimberg
       [not found] ` <1446132812-20170-1-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Sagi Grimberg @ 2015-10-29 15:33 UTC (permalink / raw)
  To: Doug Ledford; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

The driver does not support it anyway, and the support
should be added to a generic layer shared by both hfi1,
qib and softroce drivers.

Signed-off-by: Sagi Grimberg <sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
 drivers/staging/rdma/hfi1/keys.c  |   55 -------------------------------------
 drivers/staging/rdma/hfi1/mr.c    |   33 +---------------------
 drivers/staging/rdma/hfi1/verbs.c |    9 +-----
 drivers/staging/rdma/hfi1/verbs.h |    8 -----
 4 files changed, 3 insertions(+), 102 deletions(-)

diff --git a/drivers/staging/rdma/hfi1/keys.c b/drivers/staging/rdma/hfi1/keys.c
index 82c21b1..cb4e608 100644
--- a/drivers/staging/rdma/hfi1/keys.c
+++ b/drivers/staging/rdma/hfi1/keys.c
@@ -354,58 +354,3 @@ bail:
 	rcu_read_unlock();
 	return 0;
 }
-
-/*
- * Initialize the memory region specified by the work request.
- */
-int hfi1_fast_reg_mr(struct hfi1_qp *qp, struct ib_fast_reg_wr *wr)
-{
-	struct hfi1_lkey_table *rkt = &to_idev(qp->ibqp.device)->lk_table;
-	struct hfi1_pd *pd = to_ipd(qp->ibqp.pd);
-	struct hfi1_mregion *mr;
-	u32 rkey = wr->rkey;
-	unsigned i, n, m;
-	int ret = -EINVAL;
-	unsigned long flags;
-	u64 *page_list;
-	size_t ps;
-
-	spin_lock_irqsave(&rkt->lock, flags);
-	if (pd->user || rkey == 0)
-		goto bail;
-
-	mr = rcu_dereference_protected(
-		rkt->table[(rkey >> (32 - hfi1_lkey_table_size))],
-		lockdep_is_held(&rkt->lock));
-	if (unlikely(mr == NULL || qp->ibqp.pd != mr->pd))
-		goto bail;
-
-	if (wr->page_list_len > mr->max_segs)
-		goto bail;
-
-	ps = 1UL << wr->page_shift;
-	if (wr->length > ps * wr->page_list_len)
-		goto bail;
-
-	mr->user_base = wr->iova_start;
-	mr->iova = wr->iova_start;
-	mr->lkey = rkey;
-	mr->length = wr->length;
-	mr->access_flags = wr->access_flags;
-	page_list = wr->page_list->page_list;
-	m = 0;
-	n = 0;
-	for (i = 0; i < wr->page_list_len; i++) {
-		mr->map[m]->segs[n].vaddr = (void *) page_list[i];
-		mr->map[m]->segs[n].length = ps;
-		if (++n == HFI1_SEGSZ) {
-			m++;
-			n = 0;
-		}
-	}
-
-	ret = 0;
-bail:
-	spin_unlock_irqrestore(&rkt->lock, flags);
-	return ret;
-}
diff --git a/drivers/staging/rdma/hfi1/mr.c b/drivers/staging/rdma/hfi1/mr.c
index bd64e4f..402bd64 100644
--- a/drivers/staging/rdma/hfi1/mr.c
+++ b/drivers/staging/rdma/hfi1/mr.c
@@ -344,9 +344,10 @@ out:
 
 /*
  * Allocate a memory region usable with the
- * IB_WR_FAST_REG_MR send work request.
+ * IB_WR_REG_MR send work request.
  *
  * Return the memory region on success, otherwise return an errno.
+ * FIXME: IB_WR_REG_MR is not supported
  */
 struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
 			    enum ib_mr_type mr_type,
@@ -364,36 +365,6 @@ struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
 	return &mr->ibmr;
 }
 
-struct ib_fast_reg_page_list *
-hfi1_alloc_fast_reg_page_list(struct ib_device *ibdev, int page_list_len)
-{
-	unsigned size = page_list_len * sizeof(u64);
-	struct ib_fast_reg_page_list *pl;
-
-	if (size > PAGE_SIZE)
-		return ERR_PTR(-EINVAL);
-
-	pl = kzalloc(sizeof(*pl), GFP_KERNEL);
-	if (!pl)
-		return ERR_PTR(-ENOMEM);
-
-	pl->page_list = kzalloc(size, GFP_KERNEL);
-	if (!pl->page_list)
-		goto err_free;
-
-	return pl;
-
-err_free:
-	kfree(pl);
-	return ERR_PTR(-ENOMEM);
-}
-
-void hfi1_free_fast_reg_page_list(struct ib_fast_reg_page_list *pl)
-{
-	kfree(pl->page_list);
-	kfree(pl);
-}
-
 /**
  * hfi1_alloc_fmr - allocate a fast memory region
  * @pd: the protection domain for this memory region
diff --git a/drivers/staging/rdma/hfi1/verbs.c b/drivers/staging/rdma/hfi1/verbs.c
index 981e6c1..6e2da7e 100644
--- a/drivers/staging/rdma/hfi1/verbs.c
+++ b/drivers/staging/rdma/hfi1/verbs.c
@@ -380,9 +380,7 @@ static int post_one_send(struct hfi1_qp *qp, struct ib_send_wr *wr)
 	 * undefined operations.
 	 * Make sure buffer is large enough to hold the result for atomics.
 	 */
-	if (wr->opcode == IB_WR_FAST_REG_MR) {
-		return -EINVAL;
-	} else if (qp->ibqp.qp_type == IB_QPT_UC) {
+	if (qp->ibqp.qp_type == IB_QPT_UC) {
 		if ((unsigned) wr->opcode >= IB_WR_RDMA_READ)
 			return -EINVAL;
 	} else if (qp->ibqp.qp_type != IB_QPT_RC) {
@@ -417,9 +415,6 @@ static int post_one_send(struct hfi1_qp *qp, struct ib_send_wr *wr)
 	if (qp->ibqp.qp_type != IB_QPT_UC &&
 	    qp->ibqp.qp_type != IB_QPT_RC)
 		memcpy(&wqe->ud_wr, ud_wr(wr), sizeof(wqe->ud_wr));
-	else if (wr->opcode == IB_WR_FAST_REG_MR)
-		memcpy(&wqe->fast_reg_wr, fast_reg_wr(wr),
-			sizeof(wqe->fast_reg_wr));
 	else if (wr->opcode == IB_WR_RDMA_WRITE_WITH_IMM ||
 		 wr->opcode == IB_WR_RDMA_WRITE ||
 		 wr->opcode == IB_WR_RDMA_READ)
@@ -2065,8 +2060,6 @@ int hfi1_register_ib_device(struct hfi1_devdata *dd)
 	ibdev->reg_user_mr = hfi1_reg_user_mr;
 	ibdev->dereg_mr = hfi1_dereg_mr;
 	ibdev->alloc_mr = hfi1_alloc_mr;
-	ibdev->alloc_fast_reg_page_list = hfi1_alloc_fast_reg_page_list;
-	ibdev->free_fast_reg_page_list = hfi1_free_fast_reg_page_list;
 	ibdev->alloc_fmr = hfi1_alloc_fmr;
 	ibdev->map_phys_fmr = hfi1_map_phys_fmr;
 	ibdev->unmap_fmr = hfi1_unmap_fmr;
diff --git a/drivers/staging/rdma/hfi1/verbs.h b/drivers/staging/rdma/hfi1/verbs.h
index cf5a3c9..159ec08 100644
--- a/drivers/staging/rdma/hfi1/verbs.h
+++ b/drivers/staging/rdma/hfi1/verbs.h
@@ -353,7 +353,6 @@ struct hfi1_swqe {
 		struct ib_rdma_wr rdma_wr;
 		struct ib_atomic_wr atomic_wr;
 		struct ib_ud_wr ud_wr;
-		struct ib_fast_reg_wr fast_reg_wr;
 	};
 	u32 psn;                /* first packet sequence number */
 	u32 lpsn;               /* last packet sequence number */
@@ -1026,13 +1025,6 @@ struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
 			    enum ib_mr_type mr_type,
 			    u32 max_entries);
 
-struct ib_fast_reg_page_list *hfi1_alloc_fast_reg_page_list(
-				struct ib_device *ibdev, int page_list_len);
-
-void hfi1_free_fast_reg_page_list(struct ib_fast_reg_page_list *pl);
-
-int hfi1_fast_reg_mr(struct hfi1_qp *qp, struct ib_fast_reg_wr *wr);
-
 struct ib_fmr *hfi1_alloc_fmr(struct ib_pd *pd, int mr_access_flags,
 			      struct ib_fmr_attr *fmr_attr);
 
-- 
1.7.1

--
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

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 28/26] IB/ipath: Remove fast registration from the code
       [not found] ` <1446132812-20170-1-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
@ 2015-10-29 15:33   ` Sagi Grimberg
       [not found]     ` <1446132812-20170-2-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
  2015-10-29 15:44   ` [PATCH v5 27/26] IB/hfi1: " Doug Ledford
  1 sibling, 1 reply; 5+ messages in thread
From: Sagi Grimberg @ 2015-10-29 15:33 UTC (permalink / raw)
  To: Doug Ledford; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

The driver does not support it anyway, and the support
should be added to a generic layer shared by both hfi1,
qib and softroce drivers.

Signed-off-by: Sagi Grimberg <sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
 drivers/staging/rdma/ipath/ipath_verbs.c |    3 ---
 drivers/staging/rdma/ipath/ipath_verbs.h |    1 -
 2 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rdma/ipath/ipath_verbs.c b/drivers/staging/rdma/ipath/ipath_verbs.c
index 15633ec..29e9179 100644
--- a/drivers/staging/rdma/ipath/ipath_verbs.c
+++ b/drivers/staging/rdma/ipath/ipath_verbs.c
@@ -399,9 +399,6 @@ static int ipath_post_one_send(struct ipath_qp *qp, struct ib_send_wr *wr)
 	if (qp->ibqp.qp_type != IB_QPT_UC &&
 	    qp->ibqp.qp_type != IB_QPT_RC)
 		memcpy(&wqe->ud_wr, ud_wr(wr), sizeof(wqe->ud_wr));
-	else if (wr->opcode == IB_WR_FAST_REG_MR)
-		memcpy(&wqe->fast_reg_wr, fast_reg_wr(wr),
-			sizeof(wqe->fast_reg_wr));
 	else if (wr->opcode == IB_WR_RDMA_WRITE_WITH_IMM ||
 		 wr->opcode == IB_WR_RDMA_WRITE ||
 		 wr->opcode == IB_WR_RDMA_READ)
diff --git a/drivers/staging/rdma/ipath/ipath_verbs.h b/drivers/staging/rdma/ipath/ipath_verbs.h
index ed102a2..0a90a56 100644
--- a/drivers/staging/rdma/ipath/ipath_verbs.h
+++ b/drivers/staging/rdma/ipath/ipath_verbs.h
@@ -280,7 +280,6 @@ struct ipath_swqe {
 	union {
 		struct ib_send_wr wr;   /* don't use wr.sg_list */
 		struct ib_ud_wr ud_wr;
-		struct ib_fast_reg_wr fast_reg_wr;
 		struct ib_rdma_wr rdma_wr;
 		struct ib_atomic_wr atomic_wr;
 	};
-- 
1.7.1

--
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

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v5 27/26] IB/hfi1: Remove fast registration from the code
       [not found] ` <1446132812-20170-1-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
  2015-10-29 15:33   ` [PATCH 28/26] IB/ipath: " Sagi Grimberg
@ 2015-10-29 15:44   ` Doug Ledford
  1 sibling, 0 replies; 5+ messages in thread
From: Doug Ledford @ 2015-10-29 15:44 UTC (permalink / raw)
  To: Sagi Grimberg; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

[-- Attachment #1: Type: text/plain, Size: 6692 bytes --]

On 10/29/2015 11:33 AM, Sagi Grimberg wrote:
> The driver does not support it anyway, and the support
> should be added to a generic layer shared by both hfi1,
> qib and softroce drivers.
> 
> Signed-off-by: Sagi Grimberg <sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

Thanks, applied.

> ---
>  drivers/staging/rdma/hfi1/keys.c  |   55 -------------------------------------
>  drivers/staging/rdma/hfi1/mr.c    |   33 +---------------------
>  drivers/staging/rdma/hfi1/verbs.c |    9 +-----
>  drivers/staging/rdma/hfi1/verbs.h |    8 -----
>  4 files changed, 3 insertions(+), 102 deletions(-)
> 
> diff --git a/drivers/staging/rdma/hfi1/keys.c b/drivers/staging/rdma/hfi1/keys.c
> index 82c21b1..cb4e608 100644
> --- a/drivers/staging/rdma/hfi1/keys.c
> +++ b/drivers/staging/rdma/hfi1/keys.c
> @@ -354,58 +354,3 @@ bail:
>  	rcu_read_unlock();
>  	return 0;
>  }
> -
> -/*
> - * Initialize the memory region specified by the work request.
> - */
> -int hfi1_fast_reg_mr(struct hfi1_qp *qp, struct ib_fast_reg_wr *wr)
> -{
> -	struct hfi1_lkey_table *rkt = &to_idev(qp->ibqp.device)->lk_table;
> -	struct hfi1_pd *pd = to_ipd(qp->ibqp.pd);
> -	struct hfi1_mregion *mr;
> -	u32 rkey = wr->rkey;
> -	unsigned i, n, m;
> -	int ret = -EINVAL;
> -	unsigned long flags;
> -	u64 *page_list;
> -	size_t ps;
> -
> -	spin_lock_irqsave(&rkt->lock, flags);
> -	if (pd->user || rkey == 0)
> -		goto bail;
> -
> -	mr = rcu_dereference_protected(
> -		rkt->table[(rkey >> (32 - hfi1_lkey_table_size))],
> -		lockdep_is_held(&rkt->lock));
> -	if (unlikely(mr == NULL || qp->ibqp.pd != mr->pd))
> -		goto bail;
> -
> -	if (wr->page_list_len > mr->max_segs)
> -		goto bail;
> -
> -	ps = 1UL << wr->page_shift;
> -	if (wr->length > ps * wr->page_list_len)
> -		goto bail;
> -
> -	mr->user_base = wr->iova_start;
> -	mr->iova = wr->iova_start;
> -	mr->lkey = rkey;
> -	mr->length = wr->length;
> -	mr->access_flags = wr->access_flags;
> -	page_list = wr->page_list->page_list;
> -	m = 0;
> -	n = 0;
> -	for (i = 0; i < wr->page_list_len; i++) {
> -		mr->map[m]->segs[n].vaddr = (void *) page_list[i];
> -		mr->map[m]->segs[n].length = ps;
> -		if (++n == HFI1_SEGSZ) {
> -			m++;
> -			n = 0;
> -		}
> -	}
> -
> -	ret = 0;
> -bail:
> -	spin_unlock_irqrestore(&rkt->lock, flags);
> -	return ret;
> -}
> diff --git a/drivers/staging/rdma/hfi1/mr.c b/drivers/staging/rdma/hfi1/mr.c
> index bd64e4f..402bd64 100644
> --- a/drivers/staging/rdma/hfi1/mr.c
> +++ b/drivers/staging/rdma/hfi1/mr.c
> @@ -344,9 +344,10 @@ out:
>  
>  /*
>   * Allocate a memory region usable with the
> - * IB_WR_FAST_REG_MR send work request.
> + * IB_WR_REG_MR send work request.
>   *
>   * Return the memory region on success, otherwise return an errno.
> + * FIXME: IB_WR_REG_MR is not supported
>   */
>  struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
>  			    enum ib_mr_type mr_type,
> @@ -364,36 +365,6 @@ struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
>  	return &mr->ibmr;
>  }
>  
> -struct ib_fast_reg_page_list *
> -hfi1_alloc_fast_reg_page_list(struct ib_device *ibdev, int page_list_len)
> -{
> -	unsigned size = page_list_len * sizeof(u64);
> -	struct ib_fast_reg_page_list *pl;
> -
> -	if (size > PAGE_SIZE)
> -		return ERR_PTR(-EINVAL);
> -
> -	pl = kzalloc(sizeof(*pl), GFP_KERNEL);
> -	if (!pl)
> -		return ERR_PTR(-ENOMEM);
> -
> -	pl->page_list = kzalloc(size, GFP_KERNEL);
> -	if (!pl->page_list)
> -		goto err_free;
> -
> -	return pl;
> -
> -err_free:
> -	kfree(pl);
> -	return ERR_PTR(-ENOMEM);
> -}
> -
> -void hfi1_free_fast_reg_page_list(struct ib_fast_reg_page_list *pl)
> -{
> -	kfree(pl->page_list);
> -	kfree(pl);
> -}
> -
>  /**
>   * hfi1_alloc_fmr - allocate a fast memory region
>   * @pd: the protection domain for this memory region
> diff --git a/drivers/staging/rdma/hfi1/verbs.c b/drivers/staging/rdma/hfi1/verbs.c
> index 981e6c1..6e2da7e 100644
> --- a/drivers/staging/rdma/hfi1/verbs.c
> +++ b/drivers/staging/rdma/hfi1/verbs.c
> @@ -380,9 +380,7 @@ static int post_one_send(struct hfi1_qp *qp, struct ib_send_wr *wr)
>  	 * undefined operations.
>  	 * Make sure buffer is large enough to hold the result for atomics.
>  	 */
> -	if (wr->opcode == IB_WR_FAST_REG_MR) {
> -		return -EINVAL;
> -	} else if (qp->ibqp.qp_type == IB_QPT_UC) {
> +	if (qp->ibqp.qp_type == IB_QPT_UC) {
>  		if ((unsigned) wr->opcode >= IB_WR_RDMA_READ)
>  			return -EINVAL;
>  	} else if (qp->ibqp.qp_type != IB_QPT_RC) {
> @@ -417,9 +415,6 @@ static int post_one_send(struct hfi1_qp *qp, struct ib_send_wr *wr)
>  	if (qp->ibqp.qp_type != IB_QPT_UC &&
>  	    qp->ibqp.qp_type != IB_QPT_RC)
>  		memcpy(&wqe->ud_wr, ud_wr(wr), sizeof(wqe->ud_wr));
> -	else if (wr->opcode == IB_WR_FAST_REG_MR)
> -		memcpy(&wqe->fast_reg_wr, fast_reg_wr(wr),
> -			sizeof(wqe->fast_reg_wr));
>  	else if (wr->opcode == IB_WR_RDMA_WRITE_WITH_IMM ||
>  		 wr->opcode == IB_WR_RDMA_WRITE ||
>  		 wr->opcode == IB_WR_RDMA_READ)
> @@ -2065,8 +2060,6 @@ int hfi1_register_ib_device(struct hfi1_devdata *dd)
>  	ibdev->reg_user_mr = hfi1_reg_user_mr;
>  	ibdev->dereg_mr = hfi1_dereg_mr;
>  	ibdev->alloc_mr = hfi1_alloc_mr;
> -	ibdev->alloc_fast_reg_page_list = hfi1_alloc_fast_reg_page_list;
> -	ibdev->free_fast_reg_page_list = hfi1_free_fast_reg_page_list;
>  	ibdev->alloc_fmr = hfi1_alloc_fmr;
>  	ibdev->map_phys_fmr = hfi1_map_phys_fmr;
>  	ibdev->unmap_fmr = hfi1_unmap_fmr;
> diff --git a/drivers/staging/rdma/hfi1/verbs.h b/drivers/staging/rdma/hfi1/verbs.h
> index cf5a3c9..159ec08 100644
> --- a/drivers/staging/rdma/hfi1/verbs.h
> +++ b/drivers/staging/rdma/hfi1/verbs.h
> @@ -353,7 +353,6 @@ struct hfi1_swqe {
>  		struct ib_rdma_wr rdma_wr;
>  		struct ib_atomic_wr atomic_wr;
>  		struct ib_ud_wr ud_wr;
> -		struct ib_fast_reg_wr fast_reg_wr;
>  	};
>  	u32 psn;                /* first packet sequence number */
>  	u32 lpsn;               /* last packet sequence number */
> @@ -1026,13 +1025,6 @@ struct ib_mr *hfi1_alloc_mr(struct ib_pd *pd,
>  			    enum ib_mr_type mr_type,
>  			    u32 max_entries);
>  
> -struct ib_fast_reg_page_list *hfi1_alloc_fast_reg_page_list(
> -				struct ib_device *ibdev, int page_list_len);
> -
> -void hfi1_free_fast_reg_page_list(struct ib_fast_reg_page_list *pl);
> -
> -int hfi1_fast_reg_mr(struct hfi1_qp *qp, struct ib_fast_reg_wr *wr);
> -
>  struct ib_fmr *hfi1_alloc_fmr(struct ib_pd *pd, int mr_access_flags,
>  			      struct ib_fmr_attr *fmr_attr);
>  
> 


-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 28/26] IB/ipath: Remove fast registration from the code
       [not found]     ` <1446132812-20170-2-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
@ 2015-10-29 15:45       ` Doug Ledford
       [not found]         ` <56323F02.8040803-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Doug Ledford @ 2015-10-29 15:45 UTC (permalink / raw)
  To: Sagi Grimberg; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

[-- Attachment #1: Type: text/plain, Size: 1816 bytes --]

On 10/29/2015 11:33 AM, Sagi Grimberg wrote:
> The driver does not support it anyway, and the support
> should be added to a generic layer shared by both hfi1,
> qib and softroce drivers.

Thanks, applied.

> Signed-off-by: Sagi Grimberg <sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> ---
>  drivers/staging/rdma/ipath/ipath_verbs.c |    3 ---
>  drivers/staging/rdma/ipath/ipath_verbs.h |    1 -
>  2 files changed, 0 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/rdma/ipath/ipath_verbs.c b/drivers/staging/rdma/ipath/ipath_verbs.c
> index 15633ec..29e9179 100644
> --- a/drivers/staging/rdma/ipath/ipath_verbs.c
> +++ b/drivers/staging/rdma/ipath/ipath_verbs.c
> @@ -399,9 +399,6 @@ static int ipath_post_one_send(struct ipath_qp *qp, struct ib_send_wr *wr)
>  	if (qp->ibqp.qp_type != IB_QPT_UC &&
>  	    qp->ibqp.qp_type != IB_QPT_RC)
>  		memcpy(&wqe->ud_wr, ud_wr(wr), sizeof(wqe->ud_wr));
> -	else if (wr->opcode == IB_WR_FAST_REG_MR)
> -		memcpy(&wqe->fast_reg_wr, fast_reg_wr(wr),
> -			sizeof(wqe->fast_reg_wr));
>  	else if (wr->opcode == IB_WR_RDMA_WRITE_WITH_IMM ||
>  		 wr->opcode == IB_WR_RDMA_WRITE ||
>  		 wr->opcode == IB_WR_RDMA_READ)
> diff --git a/drivers/staging/rdma/ipath/ipath_verbs.h b/drivers/staging/rdma/ipath/ipath_verbs.h
> index ed102a2..0a90a56 100644
> --- a/drivers/staging/rdma/ipath/ipath_verbs.h
> +++ b/drivers/staging/rdma/ipath/ipath_verbs.h
> @@ -280,7 +280,6 @@ struct ipath_swqe {
>  	union {
>  		struct ib_send_wr wr;   /* don't use wr.sg_list */
>  		struct ib_ud_wr ud_wr;
> -		struct ib_fast_reg_wr fast_reg_wr;
>  		struct ib_rdma_wr rdma_wr;
>  		struct ib_atomic_wr atomic_wr;
>  	};
> 


-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 28/26] IB/ipath: Remove fast registration from the code
       [not found]         ` <56323F02.8040803-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
@ 2015-10-29 15:49           ` Doug Ledford
  0 siblings, 0 replies; 5+ messages in thread
From: Doug Ledford @ 2015-10-29 15:49 UTC (permalink / raw)
  To: Sagi Grimberg; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

[-- Attachment #1: Type: text/plain, Size: 598 bytes --]

On 10/29/2015 11:45 AM, Doug Ledford wrote:
> On 10/29/2015 11:33 AM, Sagi Grimberg wrote:
>> The driver does not support it anyway, and the support
>> should be added to a generic layer shared by both hfi1,
>> qib and softroce drivers.
> 
> Thanks, applied.

Thanks for the quick turnaround Sagi, this has my compiles working again
with the final 26/26 patch back in place and lets me move on to other
patchsets.

I've pushed everything up to this point to k.o and github.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2015-10-29 15:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-29 15:33 [PATCH v5 27/26] IB/hfi1: Remove fast registration from the code Sagi Grimberg
     [not found] ` <1446132812-20170-1-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-29 15:33   ` [PATCH 28/26] IB/ipath: " Sagi Grimberg
     [not found]     ` <1446132812-20170-2-git-send-email-sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-29 15:45       ` Doug Ledford
     [not found]         ` <56323F02.8040803-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-10-29 15:49           ` Doug Ledford
2015-10-29 15:44   ` [PATCH v5 27/26] IB/hfi1: " Doug Ledford

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).