RE: [RFC 07/11] Add support for memory registeration verbs

2016-09-14 Thread Kalderon, Michal
> > +struct qedr_mr *__qedr_alloc_mr(struct ib_pd *ibpd, int > > +max_page_list_len) { > > + struct qedr_pd *pd = get_qedr_pd(ibpd); > > + struct qedr_dev *dev = get_qedr_dev(ibpd->device); > > + struct qedr_mr *mr; > > + int rc = -ENOMEM; > > + > > + DP_VERBOSE(dev, QEDR_MSG_MR, > > +

RE: [RFC 07/11] Add support for memory registeration verbs

2016-09-14 Thread Kalderon, Michal
> >>> +struct qedr_mr *__qedr_alloc_mr(struct ib_pd *ibpd, int > >>> +max_page_list_len) { > >>> + struct qedr_pd *pd = get_qedr_pd(ibpd); > >>> + struct qedr_dev *dev = get_qedr_dev(ibpd->device); > >>> + struct qedr_mr *mr; > >>> + int rc = -ENOMEM; > >>> + > >>> + DP_VERBOSE(dev, QEDR_MSG_MR, >

Re: [RFC 07/11] Add support for memory registeration verbs

2016-09-14 Thread Sagi Grimberg
+struct qedr_mr *__qedr_alloc_mr(struct ib_pd *ibpd, int +max_page_list_len) { + struct qedr_pd *pd = get_qedr_pd(ibpd); + struct qedr_dev *dev = get_qedr_dev(ibpd->device); + struct qedr_mr *mr; + int rc = -ENOMEM; + + DP_VERBOSE(dev, QEDR_MSG_MR, +

RE: [RFC 07/11] Add support for memory registeration verbs

2016-09-14 Thread Amrani, Ram
> > +static inline struct qedr_ah *get_qedr_ah(struct ib_ah *ibah) { > > + return container_of(ibah, struct qedr_ah, ibah); } > > Little surprising to find that here... how is the ah related to this patch? Thanks, Sagi. Will move into a proper location.

Re: [RFC 07/11] Add support for memory registeration verbs

2016-09-13 Thread Sagi Grimberg
+struct qedr_mr *__qedr_alloc_mr(struct ib_pd *ibpd, int max_page_list_len) +{ + struct qedr_pd *pd = get_qedr_pd(ibpd); + struct qedr_dev *dev = get_qedr_dev(ibpd->device); + struct qedr_mr *mr; + int rc = -ENOMEM; + + DP_VERBOSE(dev, QEDR_MSG_MR, +

Re: [RFC 07/11] Add support for memory registeration verbs

2016-09-13 Thread Sagi Grimberg
+static inline struct qedr_ah *get_qedr_ah(struct ib_ah *ibah) +{ + return container_of(ibah, struct qedr_ah, ibah); +} Little surprising to find that here... how is the ah related to this patch?