<div class="__aliyun_email_body_block"><div style="font-family: Tahoma, Arial, STHeitiSC-Light, SimSun"><div style="clear: both;"><p class="ds-markdown-paragraph" style="margin: 16px 0px; font-variant-ligatures: normal; font-variant-caps: normal; letter-spacing: normal; text-align: start; text-indent: 0px; white-space: normal; word-spacing: 0px; background-color: rgb(255, 255, 255); text-decoration-style: initial; text-decoration-color: initial;">Hi Stephen, Thanks for the review. I'll resubmit the patch shortly.<div ></div><p class="ds-markdown-paragraph" style="margin: 16px 0px; font-variant-ligatures: normal; font-variant-caps: normal; letter-spacing: normal; text-align: start; text-indent: 0px; white-space: normal; word-spacing: 0px; background-color: rgb(255, 255, 255); text-decoration-style: initial; text-decoration-color: initial;"><div class="">Thank you.</div><div class=""> </div></div><blockquote _quote="1" style="margin-right: 0px; margin-top: 0px; margin-bottom: 0px;"><div class="alimail-quote"><div style="clear: both;">------------------------------------------------------------------</div><div style="clear: both;">发件人：Stephen Hemminger <stephen@networkplumber.org></div><div style="clear: both;">发送时间：2026年1月25日(周日) 03:08</div><div style="clear: both;">收件人：Dimon<dimon.zhao@nebula-matrix.com></div><div style="clear: both;">抄　送：dev<dev@dpdk.org></div><div style="clear: both;">主　题：Re: [PATCH v1 0/4] improve NBL memory safety and mailbox reliability</div><div style="clear: both;"> </div>On Thu, 22 Jan 2026 19:16:23 -0800 Dimon Zhao <dimon.zhao@nebula-matrix.com> wrote: > Includes fixes for mbuf/memzone leaks and mailbox state handling. > > Dimon Zhao (4): > net/nbl: fix memzone leak in queue release > net/nbl: optimize mbuf headroom usage in packet transmission > net/nbl: fix mbuf double-free in queue cleanup > net/nbl: improve exception handling for the mailbox > > drivers/net/nbl/nbl_dev/nbl_dev.c | 2 +- > drivers/net/nbl/nbl_hw/nbl_channel.c | 94 ++++++++++++++++---- > drivers/net/nbl/nbl_hw/nbl_channel.h | 11 ++- > drivers/net/nbl/nbl_hw/nbl_resource.h | 2 + > drivers/net/nbl/nbl_hw/nbl_txrx.c | 122 +++++++++++++++++++------- > drivers/net/nbl/nbl_hw/nbl_txrx_ops.h | 2 +- > 6 files changed, 179 insertions(+), 54 deletions(-) > Some of the automated review comments are valid, some are not. Please resubmit, or I can fix during merge Fix: 1. Remove unnecessary NULL checks before `rte_free()` and `rte_memzone_free()` 4. Fix grammar in log message ("does not" instead of "do not") Ok as is, but would be good to change: 2. Add blank lines between variable declarations and statements 5. Use "Tx" instead of "transmission" in commit subject Ignore: 3. Use explicit NULL comparisons (`!= NULL`) instead of implicit boolean tests</div></blockquote><div style="line-height: 20px; clear: both;"> </div></div></div>