Hi Ferruh & Thomas, > -----Original Message----- > From: Ferruh Yigit <[email protected]> > Sent: Thursday, May 6, 2021 2:21 AM > To: Bing Zhao <[email protected]>; Slava Ovsiienko > <[email protected]>; Matan Azrad <[email protected]>; NBU- > Contact-Thomas Monjalon <[email protected]> > Cc: [email protected]; Ori Kam <[email protected]>; Raslan Darawsheh > <[email protected]> > Subject: Re: [dpdk-dev] [PATCH v7 17/17] doc: update mlx5 support > for conntrack > > External email: Use caution opening links or attachments > > > On 5/5/2021 1:23 PM, Bing Zhao wrote: > > In the release notes and mlx5 NIC document, the support and > limitation > > of connection tracking are added. > > > > A new NIC feature seems added, better to be acked beyond the mlx5 > scope. > > > Signed-off-by: Bing Zhao <[email protected]> > > Acked-by: Viacheslav Ovsiienko <[email protected]> > > --- > > doc/guides/nics/features/default.ini | 1 + > > doc/guides/nics/features/mlx5.ini | 1 + > > doc/guides/nics/mlx5.rst | 14 ++++++++++++++ > > doc/guides/rel_notes/release_21_05.rst | 2 ++ > > 4 files changed, 18 insertions(+) > > > > diff --git a/doc/guides/nics/features/default.ini > > b/doc/guides/nics/features/default.ini > > index 8046bd121e..0deb4ef547 100644 > > --- a/doc/guides/nics/features/default.ini > > +++ b/doc/guides/nics/features/default.ini > > @@ -66,6 +66,7 @@ Module EEPROM dump = > > Registers dump = > > LED = > > Multiprocess aware = > > +Connection tracking = > > FreeBSD = > > Linux = > > Windows = > > Need to update 'doc/guides/nics/features.rst' too, to describe new > feature. > > I will drop new feature part in next-net, it can be sent as separate > patch later. Will keep driver and release notes documentation.
http://patches.dpdk.org/project/dpdk/patch/[email protected]/ A new patch with the updated description is in the link below. I checked the file of the "features.rst" again, it seems that only the API for ethdev are described. Even if conntrack is a new feature and provided by new HW, but the only way to operate on it is to use rte_flow API now. So I am not quite sure if it is possible and proper to list the feature in the driver feature lists table. What do you think? BR. Bing
