<div class="__aliyun_email_body_block"><div  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun"><div  style="clear: both; font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><span >Hi Stephen,<br ></span></span><div  style="clear: both;"><br ></div><div  style="clear: both;">Thank you for the review.<br ></div><div  style="clear: both;"><br ></div><div  style="clear: both;">I will fix the documentation to match the features implemented in the code.<br ></div><div  style="clear: both;"><br ></div><div  style="clear: both;">To clarify, the nbl driver does support all the features listed in the documentation.</div><div  style="clear: both;">There was a misunderstanding on my part about the submission process - I thought we were supposed to submit basic functionality first,</div><div  style="clear: both;">and then add features like TSO, checksum offload, and FEC in subsequent patches after the initial driver merge.<br ></div><div  style="clear: both;"><br ></div><div ><span >Thank you.</span></div><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><div  style="clear: both; font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><blockquote  _quote="1" style="margin-right: 0px; margin-top: 0px; margin-bottom: 0px; font-family: Tahoma, Arial, STHeiti, SimSun; font-size: 14px; color: rgb(0, 0, 0);"><div  class="alimail-quote"><div  style="clear: both;">------------------------------------------------------------------</div><div  style="clear: both;">发件人：Stephen Hemminger <stephen@networkplumber.org></div><div  style="clear: both;">发送时间：2025年10月21日(周二) 22:21</div><div  style="clear: both;">收件人：Dimon<dimon.zhao@nebula-matrix.com></div><div  style="clear: both;">抄　送：dev<dev@dpdk.org>; Thomas Monjalon<thomas@monjalon.net>; Kyo Liu<kyo.liu@nebula-matrix.com>; Leon<leon.yu@nebula-matrix.com>; Sam<sam.chen@nebula-matrix.com></div><div  style="clear: both;">主　题：Re: [PATCH v13 01/17] net/nbl: add doc and minimum nbl build framework</div><div  style="clear: both;"><br ></div>As part of driver review I audit the documentation in features<br >against what is actually implemented. Many new drivers seem to cut and paste<br >from existing drivers and not implement features.<br ><br >> diff --git a/doc/guides/nics/features/nbl.ini b/doc/guides/nics/features/nbl.ini<br >> new file mode 100644<br >> index 0000000000..90ec593759<br >> --- /dev/null<br >> +++ b/doc/guides/nics/features/nbl.ini<br >> @@ -0,0 +1,31 @@<br >> +;<br >> +; Supported features of the 'nbl' network poll mode driver.<br >> +;<br >> +; Refer to default.ini for the full list of available PMD features.<br >> +;<br >> +[Features]<br >> +Speed capabilities   = Y<br >> +Link speed configuration = Y<br >> +Link status          = Y<br >> +FEC                  = Y<br ><br >There is no ethdev ops for fec_get in this driver.<br >It doesn't support FEC<br ><br >> +Queue start/stop     = Y<br ><br >There is no operation for tx_queue_start or rx_queue_start<br ><br >> +Burst mode info      = Y<br ><br >Driver has prototype for txrx_burst_mode_get but no implementation<br >or ethdev_ops.<br ><br >> +MTU update           = Y<br >> +Scattered Rx         = Y<br >> +TSO                  = Y<br >> +Promiscuous mode     = Y<br >> +Allmulticast mode    = Y<br >> +Unicast MAC filter   = Y<br >> +RSS reta update      = Y<br >> +VLAN filter          = Y<br ><br >No ethdev_ops for vlan_filter_set<br ><br >> +VLAN offload         = P<br ><br >Driver does not report RTE_ETH_VLAN_STRIP_OFFLOAS<br ><br >> +L3 checksum offload  = P<br >> +L4 checksum offload  = P<br ><br >Driver does not report RTE_ETH_RX_OFFLOAD_TCP_CKSUM etc.<br ><br >> +Basic stats          = Y<br >> +Extended stats       = Y<br >> +Stats per queue      = Y<br >> +FW version           = Y<br ><br >Not implemented.<br ><br >> +Module EEPROM dump   = Y<br ><br >Not implemented.<br ><br >> +Linux                = Y<br >> +ARMv8                = Y<br >> +x86-64               = Y<br ><br >Please fix the documentation and/or code to match features<br >and resubmit</div></blockquote><div  style="line-height: 20px; clear: both;"><br ></div></div></div>
<div style="clear: both;"><span style="font-size: 10px; color: rgb(216, 216, 216);">本邮件所含信息及其任何附件为保密信息且可能属于专有信息。任何非指定接收人均无权访问本邮件。如果您不是该邮件的指定接收人，那么任何对本邮件内容进行披露，复制或使用的行为均是禁止的。如果您不是该邮件的指定接收人，请您立即通过邮件通知 compliance@nebula-matrix.com并立即删除您错误接受的邮件。</span></div><div style="clear: both;"><span style="font-size: 10px; color: rgb(216, 216, 216);">The information in this message and any attachments is confidential and may be privileged.  Access to this email by anyone other than the intended recipient is not authorized.  If you are not the intended recipient, disclosure, copying or use of the contents of this email is prohibited.  If you are not the intended recipient, please notify  compliance@nebula-matrix.com immediately by email, and please destroy the email you received in error.</span></div>