> -----Original Message-----
> From: Ferruh Yigit <ferruh.yi...@intel.com>
> Sent: Saturday, November 14, 2020 2:54 AM
> To: Bill Zhou <do...@mellanox.com>; Slava Ovsiienko
> <viachesl...@nvidia.com>; Matan Azrad <ma...@nvidia.com>; NBU-Contact-
> Thomas Monjalon <tho...@monjalon.net>
> Cc: sta...@dpdk.org; dev@dpdk.org; Raslan Darawsheh <rasl...@nvidia.com>;
> Bill Zhou <dongz...@nvidia.com>
> Subject: Re: [dpdk-dev] [PATCH] net/mlx5: fix switch port id when representor
> in
> bonding
>
> On 11/4/2020 6:04 AM, Bill Zhou wrote:
> > From: Bill Zhou <dongz...@nvidia.com>
> >
> > In the bonding configurations the port switch id for representors was
> > composed of pf index in bonding as the 1 msb and the representor's
> > index as the remaining 15 lsbs. The special corner case for the host
> > PF representor on BF setups with representor id 0xFFFF was missed as well.
> >
> > The new switch port id consists of 4 msbs for the pf bonding index and
> > the remaining 12 lsbs for the representor index. The switch port id
> > ranges for each type of representors are as follows:
> >
> > Uplink representor(AKA master): 0xFFFF Host PF representor:
> > 0x<pf_bond>FFF VF representor: 0x<pf_bond>[0-FFE]
> >
> > Fixes: bee57a0a3565 ("net/mlx5: update switch port id in bonding
> > configuration")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Bill Zhou <dongz...@nvidia.com>
> > Acked-by: Viacheslav Ovsiienko <viachesl...@nvidia.com>
>
> Sign off tag is updated as following in the next-net, to keep consistent to
> previously used name:
> Signed-off-by: Dong Zhou <dongz...@nvidia.com>
Yes, it's fine.
