Message ID | 1552138669-22239-1-git-send-email-devesh.sharma@broadcom.com (mailing list archive) |
---|---|
State | Not Applicable |
Headers | show |
Series | perftest: add Broadcom's netxtreme pci ids | expand |
On Sat, Mar 9, 2019 at 7:08 PM Devesh Sharma <devesh.sharma@broadcom.com> wrote: > > perftest needs to match the vendor id and device > id to set the supported inline size for a given > adapter. Making entries for all the broadcom boards > to allow wqes with inline data. > > Signef-off-by: Devesh Sharma <devesh.sharma@broadcom.com> > --- > src/perftest_parameters.c | 25 +++++++++++++++++++++++++ > src/perftest_parameters.h | 3 ++- > 2 files changed, 27 insertions(+), 1 deletion(-) > > diff --git a/src/perftest_parameters.c b/src/perftest_parameters.c > index 5749267..e682155 100755 > --- a/src/perftest_parameters.c > +++ b/src/perftest_parameters.c > @@ -1543,6 +1543,29 @@ enum ctx_device ib_dev_name(struct ibv_context *context) > case 32882 : dev_fname = QLOGIC_AH; break; > case 32883 : dev_fname = QLOGIC_AH; break; > case 32912 : dev_fname = QLOGIC_AH; break; > + case 5638 : dev_fname = NETXTREME; break; > + case 5652 : dev_fname = NETXTREME; break; > + case 5824 : dev_fname = NETXTREME; break; > + case 5825 : dev_fname = NETXTREME; break; > + case 5827 : dev_fname = NETXTREME; break; > + case 5839 : dev_fname = NETXTREME; break; > + case 5846 : dev_fname = NETXTREME; break; > + case 5847 : dev_fname = NETXTREME; break; > + case 5848 : dev_fname = NETXTREME; break; > + case 5849 : dev_fname = NETXTREME; break; > + case 5855 : dev_fname = NETXTREME; break; > + case 5858 : dev_fname = NETXTREME; break; > + case 5859 : dev_fname = NETXTREME; break; > + case 5861 : dev_fname = NETXTREME; break; > + case 5867 : dev_fname = NETXTREME; break; > + case 5869 : dev_fname = NETXTREME; break; > + case 5871 : dev_fname = NETXTREME; break; > + case 5872 : dev_fname = NETXTREME; break; > + case 5873 : dev_fname = NETXTREME; break; > + case 5968 : dev_fname = NETXTREME; break; > + case 55296 : dev_fname = NETXTREME; break; > + case 55298 : dev_fname = NETXTREME; break; > + case 55300 : dev_fname = NETXTREME; break; > default : dev_fname = UNKNOWN; > } > } > @@ -1708,6 +1731,8 @@ static void ctx_set_max_inline(struct ibv_context *context,struct perftest_param > ((user_param->connection_type == XRC) ? DEF_INLINE_SEND_XRC : DEF_INLINE_SEND_RC_UC) ; break; > default : user_param->inline_size = 0; > } > + if (current_dev == NETXTREME) > + user_param->inline_size = 96; > > } else { > user_param->inline_size = 0; > diff --git a/src/perftest_parameters.h b/src/perftest_parameters.h > index a8c4610..f15fb73 100755 > --- a/src/perftest_parameters.h > +++ b/src/perftest_parameters.h > @@ -291,7 +291,8 @@ enum ctx_device { > CONNECTX5EX = 16, > CONNECTX6 = 17, > BLUEFIELD = 18, > - INTEL_ALL = 19 > + INTEL_ALL = 19, > + NETXTREME = 20 > }; > > /* Units for rate limiter */ > -- > 1.8.3.1 > Hi List, Can someone cc the correct maintainer for perftest so that this path goes in there?
On Tue, Mar 12, 2019 at 11:42:15AM +0530, Devesh Sharma wrote: > On Sat, Mar 9, 2019 at 7:08 PM Devesh Sharma <devesh.sharma@broadcom.com> wrote: > > > > perftest needs to match the vendor id and device > > id to set the supported inline size for a given > > adapter. Making entries for all the broadcom boards > > to allow wqes with inline data. > > > > Signef-off-by: Devesh Sharma <devesh.sharma@broadcom.com> > > --- > > src/perftest_parameters.c | 25 +++++++++++++++++++++++++ > > src/perftest_parameters.h | 3 ++- > > 2 files changed, 27 insertions(+), 1 deletion(-) > > > > diff --git a/src/perftest_parameters.c b/src/perftest_parameters.c > > index 5749267..e682155 100755 > > --- a/src/perftest_parameters.c > > +++ b/src/perftest_parameters.c > > @@ -1543,6 +1543,29 @@ enum ctx_device ib_dev_name(struct ibv_context *context) > > case 32882 : dev_fname = QLOGIC_AH; break; > > case 32883 : dev_fname = QLOGIC_AH; break; > > case 32912 : dev_fname = QLOGIC_AH; break; > > + case 5638 : dev_fname = NETXTREME; break; > > + case 5652 : dev_fname = NETXTREME; break; > > + case 5824 : dev_fname = NETXTREME; break; > > + case 5825 : dev_fname = NETXTREME; break; > > + case 5827 : dev_fname = NETXTREME; break; > > + case 5839 : dev_fname = NETXTREME; break; > > + case 5846 : dev_fname = NETXTREME; break; > > + case 5847 : dev_fname = NETXTREME; break; > > + case 5848 : dev_fname = NETXTREME; break; > > + case 5849 : dev_fname = NETXTREME; break; > > + case 5855 : dev_fname = NETXTREME; break; > > + case 5858 : dev_fname = NETXTREME; break; > > + case 5859 : dev_fname = NETXTREME; break; > > + case 5861 : dev_fname = NETXTREME; break; > > + case 5867 : dev_fname = NETXTREME; break; > > + case 5869 : dev_fname = NETXTREME; break; > > + case 5871 : dev_fname = NETXTREME; break; > > + case 5872 : dev_fname = NETXTREME; break; > > + case 5873 : dev_fname = NETXTREME; break; > > + case 5968 : dev_fname = NETXTREME; break; > > + case 55296 : dev_fname = NETXTREME; break; > > + case 55298 : dev_fname = NETXTREME; break; > > + case 55300 : dev_fname = NETXTREME; break; > > default : dev_fname = UNKNOWN; > > } > > } > > @@ -1708,6 +1731,8 @@ static void ctx_set_max_inline(struct ibv_context *context,struct perftest_param > > ((user_param->connection_type == XRC) ? DEF_INLINE_SEND_XRC : DEF_INLINE_SEND_RC_UC) ; break; > > default : user_param->inline_size = 0; > > } > > + if (current_dev == NETXTREME) > > + user_param->inline_size = 96; > > > > } else { > > user_param->inline_size = 0; > > diff --git a/src/perftest_parameters.h b/src/perftest_parameters.h > > index a8c4610..f15fb73 100755 > > --- a/src/perftest_parameters.h > > +++ b/src/perftest_parameters.h > > @@ -291,7 +291,8 @@ enum ctx_device { > > CONNECTX5EX = 16, > > CONNECTX6 = 17, > > BLUEFIELD = 18, > > - INTEL_ALL = 19 > > + INTEL_ALL = 19, > > + NETXTREME = 20 > > }; > > > > /* Units for rate limiter */ > > -- > > 1.8.3.1 > > > > Hi List, > > Can someone cc the correct maintainer for perftest so that this path > goes in there? Last time, it was Simon Raviv <simonra@mellanox.com>, but I'm tired to wait for any change in that area and I'll take your PR. Thanks
On Tue, Mar 12, 2019 at 12:09 PM Leon Romanovsky <leon@kernel.org> wrote: > > On Tue, Mar 12, 2019 at 11:42:15AM +0530, Devesh Sharma wrote: > > On Sat, Mar 9, 2019 at 7:08 PM Devesh Sharma <devesh.sharma@broadcom.com> wrote: > > > > > > perftest needs to match the vendor id and device > > > id to set the supported inline size for a given > > > adapter. Making entries for all the broadcom boards > > > to allow wqes with inline data. > > > > > > Signef-off-by: Devesh Sharma <devesh.sharma@broadcom.com> > > > --- > > > src/perftest_parameters.c | 25 +++++++++++++++++++++++++ > > > src/perftest_parameters.h | 3 ++- > > > 2 files changed, 27 insertions(+), 1 deletion(-) > > > > > > diff --git a/src/perftest_parameters.c b/src/perftest_parameters.c > > > index 5749267..e682155 100755 > > > --- a/src/perftest_parameters.c > > > +++ b/src/perftest_parameters.c > > > @@ -1543,6 +1543,29 @@ enum ctx_device ib_dev_name(struct ibv_context *context) > > > case 32882 : dev_fname = QLOGIC_AH; break; > > > case 32883 : dev_fname = QLOGIC_AH; break; > > > case 32912 : dev_fname = QLOGIC_AH; break; > > > + case 5638 : dev_fname = NETXTREME; break; > > > + case 5652 : dev_fname = NETXTREME; break; > > > + case 5824 : dev_fname = NETXTREME; break; > > > + case 5825 : dev_fname = NETXTREME; break; > > > + case 5827 : dev_fname = NETXTREME; break; > > > + case 5839 : dev_fname = NETXTREME; break; > > > + case 5846 : dev_fname = NETXTREME; break; > > > + case 5847 : dev_fname = NETXTREME; break; > > > + case 5848 : dev_fname = NETXTREME; break; > > > + case 5849 : dev_fname = NETXTREME; break; > > > + case 5855 : dev_fname = NETXTREME; break; > > > + case 5858 : dev_fname = NETXTREME; break; > > > + case 5859 : dev_fname = NETXTREME; break; > > > + case 5861 : dev_fname = NETXTREME; break; > > > + case 5867 : dev_fname = NETXTREME; break; > > > + case 5869 : dev_fname = NETXTREME; break; > > > + case 5871 : dev_fname = NETXTREME; break; > > > + case 5872 : dev_fname = NETXTREME; break; > > > + case 5873 : dev_fname = NETXTREME; break; > > > + case 5968 : dev_fname = NETXTREME; break; > > > + case 55296 : dev_fname = NETXTREME; break; > > > + case 55298 : dev_fname = NETXTREME; break; > > > + case 55300 : dev_fname = NETXTREME; break; > > > default : dev_fname = UNKNOWN; > > > } > > > } > > > @@ -1708,6 +1731,8 @@ static void ctx_set_max_inline(struct ibv_context *context,struct perftest_param > > > ((user_param->connection_type == XRC) ? DEF_INLINE_SEND_XRC : DEF_INLINE_SEND_RC_UC) ; break; > > > default : user_param->inline_size = 0; > > > } > > > + if (current_dev == NETXTREME) > > > + user_param->inline_size = 96; > > > > > > } else { > > > user_param->inline_size = 0; > > > diff --git a/src/perftest_parameters.h b/src/perftest_parameters.h > > > index a8c4610..f15fb73 100755 > > > --- a/src/perftest_parameters.h > > > +++ b/src/perftest_parameters.h > > > @@ -291,7 +291,8 @@ enum ctx_device { > > > CONNECTX5EX = 16, > > > CONNECTX6 = 17, > > > BLUEFIELD = 18, > > > - INTEL_ALL = 19 > > > + INTEL_ALL = 19, > > > + NETXTREME = 20 > > > }; > > > > > > /* Units for rate limiter */ > > > -- > > > 1.8.3.1 > > > > > > > Hi List, > > > > Can someone cc the correct maintainer for perftest so that this path > > goes in there? > > Last time, it was Simon Raviv <simonra@mellanox.com>, but I'm tired to wait for any > change in that area and I'll take your PR. > > Thanks > -----BEGIN PGP SIGNATURE----- > > iQIcBAEBAgAGBQJch1QAAAoJEORje4g2clinWX0QAIEzu3pBcDZUGLZ7K8OYYPJL > oQgwb7fzvFr/tEMk3M0NcFiF4AKPilURGZtU/JhkWoguPhF8/kg2/1jZK5LWAX+I > Pjvr423huH4EHM7sURANS0sRIAxWCu4Wnyyh2ywD19P4h7lkIIr/WprxxGOPKybJ > dAh5JH74kBmsczyx09FcO997MZsCq2qeX3bC7SzP8LBkkRp027rWCjbehbiQgNIy > 7aO+bvNSNDmArfbfZEf0q8EUAROsMH+/z6Nb3CWGU6RFOs62k3qK3CB7iN9lTeZ1 > BuXvmxe/vfS1XaYbhqTvCy7x57wwuAd5x4ysWXG4zaE/NfeaTrFNvnpzpbhn9FGk > u+HmwpCMIL6fDVliATSoC42aQgG7L5OZ8XcuOsTzYDUuzPC88R4E17Zi2G2k3wda > ZPzFyvDs77pM1eFxPvEaXsPz/ZIulPse/FDoDiNGRyNJJRszFi3RIOwmrBezpcP7 > bXIHKSNJCj3gMWuI5tJrk9X63Nr8efqlgWfxutyOn/BbA+M9YkO3UzmQ0pPh2aO7 > Pr35HDe4aYtIwbAPc+Xzr9N7uJnV6ZqjTfJy/earXi8wO4uuhNcl+fuBNh2XPSEF > akTCkGIL0nGK3d4VgPwLQH5zeZAzJUR1/59cT8yQI49SWdba4jfptyY029nxBaiZ > 0LfCLPFeUwsqwtJemlvH > =Txuq > -----END PGP SIGNATURE----- Thanks a lot Leon, this helps! -Regards Devesh
Hello, I didn't see the PR, for some reason it went to spam in my email, I will check it with the IT. I saw Leon merged it. Thanks for your contribution. Regards, Simon Raviv | Performance SW Engineer | Mellanox Technologies Ltd. Work: +972 74 7238014 | Cell: +972 50 6474912 -----Original Message----- From: Devesh Sharma <devesh.sharma@broadcom.com> Sent: Tuesday, March 12, 2019 8:59 AM To: Leon Romanovsky <leon@kernel.org> Cc: Simon Raviv <simonra@mellanox.com>; linux-rdma <linux-rdma@vger.kernel.org>; Jason Gunthorpe <jgg@mellanox.com> Subject: Re: [PATCH] perftest: add Broadcom's netxtreme pci ids On Tue, Mar 12, 2019 at 12:09 PM Leon Romanovsky <leon@kernel.org> wrote: > > On Tue, Mar 12, 2019 at 11:42:15AM +0530, Devesh Sharma wrote: > > On Sat, Mar 9, 2019 at 7:08 PM Devesh Sharma <devesh.sharma@broadcom.com> wrote: > > > > > > perftest needs to match the vendor id and device id to set the > > > supported inline size for a given adapter. Making entries for all > > > the broadcom boards to allow wqes with inline data. > > > > > > Signef-off-by: Devesh Sharma <devesh.sharma@broadcom.com> > > > --- > > > src/perftest_parameters.c | 25 +++++++++++++++++++++++++ > > > src/perftest_parameters.h | 3 ++- > > > 2 files changed, 27 insertions(+), 1 deletion(-) > > > > > > diff --git a/src/perftest_parameters.c b/src/perftest_parameters.c > > > index 5749267..e682155 100755 > > > --- a/src/perftest_parameters.c > > > +++ b/src/perftest_parameters.c > > > @@ -1543,6 +1543,29 @@ enum ctx_device ib_dev_name(struct ibv_context *context) > > > case 32882 : dev_fname = QLOGIC_AH; break; > > > case 32883 : dev_fname = QLOGIC_AH; break; > > > case 32912 : dev_fname = QLOGIC_AH; break; > > > + case 5638 : dev_fname = NETXTREME; break; > > > + case 5652 : dev_fname = NETXTREME; break; > > > + case 5824 : dev_fname = NETXTREME; break; > > > + case 5825 : dev_fname = NETXTREME; break; > > > + case 5827 : dev_fname = NETXTREME; break; > > > + case 5839 : dev_fname = NETXTREME; break; > > > + case 5846 : dev_fname = NETXTREME; break; > > > + case 5847 : dev_fname = NETXTREME; break; > > > + case 5848 : dev_fname = NETXTREME; break; > > > + case 5849 : dev_fname = NETXTREME; break; > > > + case 5855 : dev_fname = NETXTREME; break; > > > + case 5858 : dev_fname = NETXTREME; break; > > > + case 5859 : dev_fname = NETXTREME; break; > > > + case 5861 : dev_fname = NETXTREME; break; > > > + case 5867 : dev_fname = NETXTREME; break; > > > + case 5869 : dev_fname = NETXTREME; break; > > > + case 5871 : dev_fname = NETXTREME; break; > > > + case 5872 : dev_fname = NETXTREME; break; > > > + case 5873 : dev_fname = NETXTREME; break; > > > + case 5968 : dev_fname = NETXTREME; break; > > > + case 55296 : dev_fname = NETXTREME; break; > > > + case 55298 : dev_fname = NETXTREME; break; > > > + case 55300 : dev_fname = NETXTREME; break; > > > default : dev_fname = UNKNOWN; > > > } > > > } > > > @@ -1708,6 +1731,8 @@ static void ctx_set_max_inline(struct ibv_context *context,struct perftest_param > > > ((user_param->connection_type == XRC) ? DEF_INLINE_SEND_XRC : DEF_INLINE_SEND_RC_UC) ; break; > > > default : user_param->inline_size = 0; > > > } > > > + if (current_dev == NETXTREME) > > > + user_param->inline_size = 96; > > > > > > } else { > > > user_param->inline_size = 0; diff --git > > > a/src/perftest_parameters.h b/src/perftest_parameters.h index > > > a8c4610..f15fb73 100755 > > > --- a/src/perftest_parameters.h > > > +++ b/src/perftest_parameters.h > > > @@ -291,7 +291,8 @@ enum ctx_device { > > > CONNECTX5EX = 16, > > > CONNECTX6 = 17, > > > BLUEFIELD = 18, > > > - INTEL_ALL = 19 > > > + INTEL_ALL = 19, > > > + NETXTREME = 20 > > > }; > > > > > > /* Units for rate limiter */ > > > -- > > > 1.8.3.1 > > > > > > > Hi List, > > > > Can someone cc the correct maintainer for perftest so that this path > > goes in there? > > Last time, it was Simon Raviv <simonra@mellanox.com>, but I'm tired to > wait for any change in that area and I'll take your PR. > > Thanks > -----BEGIN PGP SIGNATURE----- > > iQIcBAEBAgAGBQJch1QAAAoJEORje4g2clinWX0QAIEzu3pBcDZUGLZ7K8OYYPJL > oQgwb7fzvFr/tEMk3M0NcFiF4AKPilURGZtU/JhkWoguPhF8/kg2/1jZK5LWAX+I > Pjvr423huH4EHM7sURANS0sRIAxWCu4Wnyyh2ywD19P4h7lkIIr/WprxxGOPKybJ > dAh5JH74kBmsczyx09FcO997MZsCq2qeX3bC7SzP8LBkkRp027rWCjbehbiQgNIy > 7aO+bvNSNDmArfbfZEf0q8EUAROsMH+/z6Nb3CWGU6RFOs62k3qK3CB7iN9lTeZ1 > BuXvmxe/vfS1XaYbhqTvCy7x57wwuAd5x4ysWXG4zaE/NfeaTrFNvnpzpbhn9FGk > u+HmwpCMIL6fDVliATSoC42aQgG7L5OZ8XcuOsTzYDUuzPC88R4E17Zi2G2k3wda > ZPzFyvDs77pM1eFxPvEaXsPz/ZIulPse/FDoDiNGRyNJJRszFi3RIOwmrBezpcP7 > bXIHKSNJCj3gMWuI5tJrk9X63Nr8efqlgWfxutyOn/BbA+M9YkO3UzmQ0pPh2aO7 > Pr35HDe4aYtIwbAPc+Xzr9N7uJnV6ZqjTfJy/earXi8wO4uuhNcl+fuBNh2XPSEF > akTCkGIL0nGK3d4VgPwLQH5zeZAzJUR1/59cT8yQI49SWdba4jfptyY029nxBaiZ > 0LfCLPFeUwsqwtJemlvH > =Txuq > -----END PGP SIGNATURE----- Thanks a lot Leon, this helps! -Regards Devesh
diff --git a/src/perftest_parameters.c b/src/perftest_parameters.c index 5749267..e682155 100755 --- a/src/perftest_parameters.c +++ b/src/perftest_parameters.c @@ -1543,6 +1543,29 @@ enum ctx_device ib_dev_name(struct ibv_context *context) case 32882 : dev_fname = QLOGIC_AH; break; case 32883 : dev_fname = QLOGIC_AH; break; case 32912 : dev_fname = QLOGIC_AH; break; + case 5638 : dev_fname = NETXTREME; break; + case 5652 : dev_fname = NETXTREME; break; + case 5824 : dev_fname = NETXTREME; break; + case 5825 : dev_fname = NETXTREME; break; + case 5827 : dev_fname = NETXTREME; break; + case 5839 : dev_fname = NETXTREME; break; + case 5846 : dev_fname = NETXTREME; break; + case 5847 : dev_fname = NETXTREME; break; + case 5848 : dev_fname = NETXTREME; break; + case 5849 : dev_fname = NETXTREME; break; + case 5855 : dev_fname = NETXTREME; break; + case 5858 : dev_fname = NETXTREME; break; + case 5859 : dev_fname = NETXTREME; break; + case 5861 : dev_fname = NETXTREME; break; + case 5867 : dev_fname = NETXTREME; break; + case 5869 : dev_fname = NETXTREME; break; + case 5871 : dev_fname = NETXTREME; break; + case 5872 : dev_fname = NETXTREME; break; + case 5873 : dev_fname = NETXTREME; break; + case 5968 : dev_fname = NETXTREME; break; + case 55296 : dev_fname = NETXTREME; break; + case 55298 : dev_fname = NETXTREME; break; + case 55300 : dev_fname = NETXTREME; break; default : dev_fname = UNKNOWN; } } @@ -1708,6 +1731,8 @@ static void ctx_set_max_inline(struct ibv_context *context,struct perftest_param ((user_param->connection_type == XRC) ? DEF_INLINE_SEND_XRC : DEF_INLINE_SEND_RC_UC) ; break; default : user_param->inline_size = 0; } + if (current_dev == NETXTREME) + user_param->inline_size = 96; } else { user_param->inline_size = 0; diff --git a/src/perftest_parameters.h b/src/perftest_parameters.h index a8c4610..f15fb73 100755 --- a/src/perftest_parameters.h +++ b/src/perftest_parameters.h @@ -291,7 +291,8 @@ enum ctx_device { CONNECTX5EX = 16, CONNECTX6 = 17, BLUEFIELD = 18, - INTEL_ALL = 19 + INTEL_ALL = 19, + NETXTREME = 20 }; /* Units for rate limiter */