<div class="__aliyun_email_body_block"><div  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun"><div  class="">Hi Stephen,</div><div  class=""><br ></div><div  class="">Thank you for the review. I have updated the patch as requested:</div><div  class=""><div  class="">1. Enhanced the commit message</div><div  class="">2. Updated doc/guides/nics/nbl.rst<br ></div><div  class="">3. Added an entry in the release notes<br ></div><div  class=""><br ></div><div  class="">This is not a bug fix, so no Fixes tag is needed.<br ></div><div  class="">v2 has been submitted.</div><p  class="ds-markdown-paragraph" style="margin-right: 0px; margin-bottom: 16px; margin-left: 0px; font-variant-ligatures: normal; font-variant-caps: normal; letter-spacing: normal; text-align: start; text-indent: 0px; word-spacing: 0px; white-space: normal; background-color: rgb(255, 255, 255); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; margin-top: 0px !important;"><span  style="color: rgb(15, 17, 21); font-family: quote-cjk-patch, Inter, system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, 'Open Sans', 'Helvetica Neue', sans-serif; font-size: 16px; font-style: normal; font-weight: 400; text-transform: none;"><br ></span></p><p  class="ds-markdown-paragraph" style="margin-top: 16px; margin-right: 0px; margin-left: 0px; font-variant-ligatures: normal; font-variant-caps: normal; letter-spacing: normal; text-align: start; text-indent: 0px; word-spacing: 0px; white-space: normal; background-color: rgb(255, 255, 255); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; margin-bottom: 0px !important;"><span  style="color: rgb(15, 17, 21); font-family: quote-cjk-patch, Inter, system-ui, -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, 'Open Sans', 'Helvetica Neue', sans-serif; font-size: 16px; font-style: normal; font-weight: 400; text-transform: none;"><br ></span></p><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><div  style="clear: both;"><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><div  style="clear: both;"><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><div  style="clear: both;"><span  style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun;"><br ></span></div><div  style="clear: both;"><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;">发送时间:2026年2月12日(周四) 01:13</div><div  style="clear: both;">收件人:Dimon<dimon.zhao@nebula-matrix.com></div><div  style="clear: both;">抄 送:dev<dev@dpdk.org>; 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 v1 1/1] net/nbl: add igb_uio support for NBL VF devices</div><div  style="clear: both;"><br ></div>On Tue, 10 Feb 2026 18:40:29 -0800<br >Dimon Zhao <dimon.zhao@nebula-matrix.com> wrote:<br ><br >> Due to chip design limitations, only VF support<br >> the igb_uio driver. PF are not compatible.<br >> <br >> Signed-off-by: Dimon Zhao <dimon.zhao@nebula-matrix.com><br ><br >The patch makes sense, but I would like to see more before merging:<br > - more verbose commit message to help users and the next developer <br >   works on this. It also should have proper grammar and be a complete sentence.<br ><br > - update to the documentation files for this driver.<br ><br > - add a release note if needed.<br ><br > - does this fix a bug? does it need a Fixes tag.</div></blockquote><div  style="line-height: 20px; clear: both;"><br ></div></div></div>