On 9/26/2023 2:13 PM, Ferruh Yigit wrote: > On 9/4/2023 5:56 AM, wa...@3snic.com wrote: >> From: Renyong Wan <wa...@3snic.com> >> >> The sssnic PMD (**librte_pmd_sssnic**) provides poll mode driver support >> for 3SNIC 9x0 serials family of Ethernet adapters. >> >> Supported NICs are: >> >> - 3S910 Dual Port SFP28 10/25GbE Ethernet adapter >> - 3S920 Quad Port SFP28 10/25GbE Ethernet adapter >> - 3S920 Quad Port QSFP28 100GbE Ethernet adapter >> >> Features of sssnic PMD are: >> >> - Link status >> - Link status event >> - Queue start/stop >> - Rx interrupt >> - Scattered Rx >> - TSO >> - LRO >> - Promiscuous mode >> - Allmulticast mode >> - Unicast MAC filter >> - Multicast MAC filte >> - RSS hash >> - RSS key update >> - RSS reta update >> - Inner RSS >> - VLAN filter >> - VLAN offload >> - L3 checksum offload >> - L4 checksum offload >> - Inner L3 checksum >> - Inner L4 checksum >> - Basic stats >> - Extended stats >> - Stats per queue >> - Flow control >> - FW version >> - Generic flow API >> >> --- >> v2: >> * Fixed 'Title underline too short' in doc/guides/nics/sssnic.rst. >> * Removed error.h from including files. >> * Fixed variable 'cmd_len' is uninitialized when used. >> * Fixed 'EINVAL' undeclared. >> * Fixed wrong format of printing uint64_t. >> * Fixed 'mask->hdr.src_addr' will always evaluate to 'true'. >> >> v3: >> * Fixed dereferencing type-punned pointer in base/sssnic_mbox.c. >> >> v4: >> * Fixed dereferencing type-punned pointer in base/sssnic_eventq.c. >> * Fixed coding style issue of COMPLEX_MACRO. >> * Fixed coding style issue of REPEATED_WORD. >> >> v5: >> * Fixed rebase mistake. >> * Fixed incorrect path in MAINTAINERS file. >> --- >> >> Renyong Wan (32): >> net/sssnic: add build and doc infrastructure >> net/sssnic: add log type and log macros >> net/sssnic: support probe and remove >> net/sssnic: initialize hardware base >> net/sssnic: add event queue >> net/sssnic/base: add message definition and utility >> net/sssnic/base: add mailbox support >> net/sssnic/base: add work queue >> net/sssnic/base: add control queue >> net/sssnic: add dev configure and infos get >> net/sssnic: add dev MAC ops >> net/sssnic: support dev link status >> net/sssnic: support link status event >> net/sssnic: support Rx queue setup and release >> net/sssnic: support Tx queue setup and release >> net/sssnic: support Rx queue start and stop >> net/sssnic: support Tx queue start and stop >> net/sssnic: add Rx interrupt support >> net/sssnic: support dev start and stop >> net/sssnic: support dev close and reset >> net/sssnic: add allmulticast and promiscuous ops >> net/sssnic: add basic and extended stats ops >> net/sssnic: support Rx packet burst >> net/sssnic: support Tx packet burst >> net/sssnic: add RSS support >> net/sssnic: support dev MTU set >> net/sssnic: support dev queue info get >> net/sssnic: support dev firmware version get >> net/sssnic: add dev flow control ops >> net/sssnic: support VLAN offload and filter >> net/sssnic: add generic flow ops >> net/sssnic: add VF dev support >> > > Hi Renyong, > > Driver mostly looks good, I did put some minor comments, I guess driver > can be merged soon with those issues addressed. > > Also agree on Stephen's comments, if you apply them in next version, > please feel free to keep Stephen's ack in next version. > >
Hi Renyong, v5 was almost there, will there be a new version for the v24.07?
