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@xxxxxxxxxxxx>
Sent: Tuesday, March 12, 2019 8:59 AM
To: Leon Romanovsky <leon@xxxxxxxxxx>
Cc: Simon Raviv <simonra@xxxxxxxxxxxx>; linux-rdma <linux-rdma@xxxxxxxxxxxxxxx>; Jason Gunthorpe <jgg@xxxxxxxxxxxx>
Subject: Re: [PATCH] perftest: add Broadcom's netxtreme pci ids
On Tue, Mar 12, 2019 at 12:09 PM Leon Romanovsky <leon@xxxxxxxxxx> 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@xxxxxxxxxxxx> 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@xxxxxxxxxxxx>
> > > ---
> > > 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@xxxxxxxxxxxx>, 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
