From: kernel test robot <lkp@intel.com>
To: Alok Prasad <palok@marvell.com>, jgg@ziepe.ca, dledford@redhat.com
Cc: kbuild-all@lists.01.org, michal.kalderon@marvell.com,
ariel.elior@marvell.com, smalin@marvell.com,
linux-rdma@vger.kernel.org, Alok Prasad <palok@marvell.com>,
Ariel Elior <aelior@marvell.com>
Subject: Re: [for-rc] RDMA/qedr: qedr crash while running rdma-tool.
Date: Sun, 22 Aug 2021 10:45:36 +0800 [thread overview]
Message-ID: <202108221042.6jppRmfo-lkp@intel.com> (raw)
In-Reply-To: <20210821074339.16614-1-palok@marvell.com>
[-- Attachment #1: Type: text/plain, Size: 4811 bytes --]
Hi Alok,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on rdma/for-next]
[also build test WARNING on v5.14-rc6 next-20210820]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Alok-Prasad/RDMA-qedr-qedr-crash-while-running-rdma-tool/20210821-154459
base: https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/f9b6462f18a87caead9b362d4cdd049504ac3c62
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Alok-Prasad/RDMA-qedr-qedr-crash-while-running-rdma-tool/20210821-154459
git checkout f9b6462f18a87caead9b362d4cdd049504ac3c62
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/infiniband/hw/qedr/verbs.c: In function 'qedr_query_qp':
>> drivers/infiniband/hw/qedr/verbs.c:2754:35: warning: implicit conversion from 'enum qed_roce_qp_state' to 'enum ib_qp_state' [-Wenum-conversion]
2754 | qp_attr->qp_state = QED_ROCE_QP_STATE_RTS;
| ^
vim +2754 drivers/infiniband/hw/qedr/verbs.c
2735
2736 int qedr_query_qp(struct ib_qp *ibqp,
2737 struct ib_qp_attr *qp_attr,
2738 int attr_mask, struct ib_qp_init_attr *qp_init_attr)
2739 {
2740 struct qed_rdma_query_qp_out_params params;
2741 struct qedr_qp *qp = get_qedr_qp(ibqp);
2742 struct qedr_dev *dev = qp->dev;
2743 int rc = 0;
2744
2745 memset(¶ms, 0, sizeof(params));
2746 memset(qp_attr, 0, sizeof(*qp_attr));
2747 memset(qp_init_attr, 0, sizeof(*qp_init_attr));
2748
2749 if (qp->qed_qp)
2750 rc = dev->ops->rdma_query_qp(dev->rdma_ctx,
2751 qp->qed_qp, ¶ms);
2752
2753 if (qp->qp_type == IB_QPT_GSI)
> 2754 qp_attr->qp_state = QED_ROCE_QP_STATE_RTS;
2755 else
2756 qp_attr->qp_state = qedr_get_ibqp_state(params.state);
2757
2758 qp_attr->cur_qp_state = qedr_get_ibqp_state(params.state);
2759 qp_attr->path_mtu = ib_mtu_int_to_enum(params.mtu);
2760 qp_attr->path_mig_state = IB_MIG_MIGRATED;
2761 qp_attr->rq_psn = params.rq_psn;
2762 qp_attr->sq_psn = params.sq_psn;
2763 qp_attr->dest_qp_num = params.dest_qp;
2764
2765 qp_attr->qp_access_flags = qedr_to_ib_qp_acc_flags(¶ms);
2766
2767 qp_attr->cap.max_send_wr = qp->sq.max_wr;
2768 qp_attr->cap.max_recv_wr = qp->rq.max_wr;
2769 qp_attr->cap.max_send_sge = qp->sq.max_sges;
2770 qp_attr->cap.max_recv_sge = qp->rq.max_sges;
2771 qp_attr->cap.max_inline_data = dev->attr.max_inline;
2772 qp_init_attr->cap = qp_attr->cap;
2773
2774 qp_attr->ah_attr.type = RDMA_AH_ATTR_TYPE_ROCE;
2775 rdma_ah_set_grh(&qp_attr->ah_attr, NULL,
2776 params.flow_label, qp->sgid_idx,
2777 params.hop_limit_ttl, params.traffic_class_tos);
2778 rdma_ah_set_dgid_raw(&qp_attr->ah_attr, ¶ms.dgid.bytes[0]);
2779 rdma_ah_set_port_num(&qp_attr->ah_attr, 1);
2780 rdma_ah_set_sl(&qp_attr->ah_attr, 0);
2781 qp_attr->timeout = params.timeout;
2782 qp_attr->rnr_retry = params.rnr_retry;
2783 qp_attr->retry_cnt = params.retry_cnt;
2784 qp_attr->min_rnr_timer = params.min_rnr_nak_timer;
2785 qp_attr->pkey_index = params.pkey_index;
2786 qp_attr->port_num = 1;
2787 rdma_ah_set_path_bits(&qp_attr->ah_attr, 0);
2788 rdma_ah_set_static_rate(&qp_attr->ah_attr, 0);
2789 qp_attr->alt_pkey_index = 0;
2790 qp_attr->alt_port_num = 0;
2791 qp_attr->alt_timeout = 0;
2792 memset(&qp_attr->alt_ah_attr, 0, sizeof(qp_attr->alt_ah_attr));
2793
2794 qp_attr->sq_draining = (params.state == QED_ROCE_QP_STATE_SQD) ? 1 : 0;
2795 qp_attr->max_dest_rd_atomic = params.max_dest_rd_atomic;
2796 qp_attr->max_rd_atomic = params.max_rd_atomic;
2797 qp_attr->en_sqd_async_notify = (params.sqd_async) ? 1 : 0;
2798
2799 DP_DEBUG(dev, QEDR_MSG_QP, "QEDR_QUERY_QP: max_inline_data=%d\n",
2800 qp_attr->cap.max_inline_data);
2801 return rc;
2802 }
2803
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 73335 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [for-rc] RDMA/qedr: qedr crash while running rdma-tool.
Date: Sun, 22 Aug 2021 10:45:36 +0800 [thread overview]
Message-ID: <202108221042.6jppRmfo-lkp@intel.com> (raw)
In-Reply-To: <20210821074339.16614-1-palok@marvell.com>
[-- Attachment #1: Type: text/plain, Size: 4923 bytes --]
Hi Alok,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on rdma/for-next]
[also build test WARNING on v5.14-rc6 next-20210820]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Alok-Prasad/RDMA-qedr-qedr-crash-while-running-rdma-tool/20210821-154459
base: https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/f9b6462f18a87caead9b362d4cdd049504ac3c62
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Alok-Prasad/RDMA-qedr-qedr-crash-while-running-rdma-tool/20210821-154459
git checkout f9b6462f18a87caead9b362d4cdd049504ac3c62
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/infiniband/hw/qedr/verbs.c: In function 'qedr_query_qp':
>> drivers/infiniband/hw/qedr/verbs.c:2754:35: warning: implicit conversion from 'enum qed_roce_qp_state' to 'enum ib_qp_state' [-Wenum-conversion]
2754 | qp_attr->qp_state = QED_ROCE_QP_STATE_RTS;
| ^
vim +2754 drivers/infiniband/hw/qedr/verbs.c
2735
2736 int qedr_query_qp(struct ib_qp *ibqp,
2737 struct ib_qp_attr *qp_attr,
2738 int attr_mask, struct ib_qp_init_attr *qp_init_attr)
2739 {
2740 struct qed_rdma_query_qp_out_params params;
2741 struct qedr_qp *qp = get_qedr_qp(ibqp);
2742 struct qedr_dev *dev = qp->dev;
2743 int rc = 0;
2744
2745 memset(¶ms, 0, sizeof(params));
2746 memset(qp_attr, 0, sizeof(*qp_attr));
2747 memset(qp_init_attr, 0, sizeof(*qp_init_attr));
2748
2749 if (qp->qed_qp)
2750 rc = dev->ops->rdma_query_qp(dev->rdma_ctx,
2751 qp->qed_qp, ¶ms);
2752
2753 if (qp->qp_type == IB_QPT_GSI)
> 2754 qp_attr->qp_state = QED_ROCE_QP_STATE_RTS;
2755 else
2756 qp_attr->qp_state = qedr_get_ibqp_state(params.state);
2757
2758 qp_attr->cur_qp_state = qedr_get_ibqp_state(params.state);
2759 qp_attr->path_mtu = ib_mtu_int_to_enum(params.mtu);
2760 qp_attr->path_mig_state = IB_MIG_MIGRATED;
2761 qp_attr->rq_psn = params.rq_psn;
2762 qp_attr->sq_psn = params.sq_psn;
2763 qp_attr->dest_qp_num = params.dest_qp;
2764
2765 qp_attr->qp_access_flags = qedr_to_ib_qp_acc_flags(¶ms);
2766
2767 qp_attr->cap.max_send_wr = qp->sq.max_wr;
2768 qp_attr->cap.max_recv_wr = qp->rq.max_wr;
2769 qp_attr->cap.max_send_sge = qp->sq.max_sges;
2770 qp_attr->cap.max_recv_sge = qp->rq.max_sges;
2771 qp_attr->cap.max_inline_data = dev->attr.max_inline;
2772 qp_init_attr->cap = qp_attr->cap;
2773
2774 qp_attr->ah_attr.type = RDMA_AH_ATTR_TYPE_ROCE;
2775 rdma_ah_set_grh(&qp_attr->ah_attr, NULL,
2776 params.flow_label, qp->sgid_idx,
2777 params.hop_limit_ttl, params.traffic_class_tos);
2778 rdma_ah_set_dgid_raw(&qp_attr->ah_attr, ¶ms.dgid.bytes[0]);
2779 rdma_ah_set_port_num(&qp_attr->ah_attr, 1);
2780 rdma_ah_set_sl(&qp_attr->ah_attr, 0);
2781 qp_attr->timeout = params.timeout;
2782 qp_attr->rnr_retry = params.rnr_retry;
2783 qp_attr->retry_cnt = params.retry_cnt;
2784 qp_attr->min_rnr_timer = params.min_rnr_nak_timer;
2785 qp_attr->pkey_index = params.pkey_index;
2786 qp_attr->port_num = 1;
2787 rdma_ah_set_path_bits(&qp_attr->ah_attr, 0);
2788 rdma_ah_set_static_rate(&qp_attr->ah_attr, 0);
2789 qp_attr->alt_pkey_index = 0;
2790 qp_attr->alt_port_num = 0;
2791 qp_attr->alt_timeout = 0;
2792 memset(&qp_attr->alt_ah_attr, 0, sizeof(qp_attr->alt_ah_attr));
2793
2794 qp_attr->sq_draining = (params.state == QED_ROCE_QP_STATE_SQD) ? 1 : 0;
2795 qp_attr->max_dest_rd_atomic = params.max_dest_rd_atomic;
2796 qp_attr->max_rd_atomic = params.max_rd_atomic;
2797 qp_attr->en_sqd_async_notify = (params.sqd_async) ? 1 : 0;
2798
2799 DP_DEBUG(dev, QEDR_MSG_QP, "QEDR_QUERY_QP: max_inline_data=%d\n",
2800 qp_attr->cap.max_inline_data);
2801 return rc;
2802 }
2803
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 73335 bytes --]
next prev parent reply other threads:[~2021-08-22 2:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-21 7:43 [for-rc] RDMA/qedr: qedr crash while running rdma-tool Alok Prasad
2021-08-21 11:55 ` Leon Romanovsky
2021-08-24 6:19 ` Alok Prasad
2021-10-22 15:49 ` Kamal Heib
2021-10-23 16:48 ` [EXT] " Alok Prasad
2021-08-22 2:45 ` kernel test robot [this message]
2021-08-22 2:45 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202108221042.6jppRmfo-lkp@intel.com \
--to=lkp@intel.com \
--cc=aelior@marvell.com \
--cc=ariel.elior@marvell.com \
--cc=dledford@redhat.com \
--cc=jgg@ziepe.ca \
--cc=kbuild-all@lists.01.org \
--cc=linux-rdma@vger.kernel.org \
--cc=michal.kalderon@marvell.com \
--cc=palok@marvell.com \
--cc=smalin@marvell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.