<!DOCTYPE html>   <div style="display:none !important;display:none;visibility:hidden;mso-hide:all;font-size:1px;color:#ffffff;line-height:1px;height:0px;max-height:0px;opacity:0;overflow:hidden;"> > -----Original Message----- > From: Stephen Hemminger <stephen@ networkplumber. org> > Sent: Wednesday, April 23, 2025 9: 01 PM > To: edwin. brossette@ 6wind. com > Cc: dev@ dpdk. org; olivier. matz@ 6wind. com; didier. pallard@ 6wind. com;</div>   <div style="display:none !important;display:none;visibility:hidden;mso-hide:all;font-size:1px;color:#ffffff;line-height:1px;max-height:0px;opacity:0;overflow:hidden;">ZjQcmQRYFpfptBannerStart</div>  <![if !((ie)|(mso))]> <div dir="ltr" lang="en" id="pfptBannerv5yd6th" style="all: revert !important; display:block !important; text-align: left !important; margin:16px 0px 16px 0px !important; padding:8px 16px 8px 16px !important; border-radius: 4px !important; min-width: 200px !important; background-color: #D2D0D0 !important; background-color: #D2D0D0; border-top: 4px solid #8d8c8c !important; border-top: 4px solid #8d8c8c;"> <div id="pfptBannerv5yd6th" style="all: unset !important; float:left !important; display:block !important; margin: 0px 0px 1px 0px !important; max-width: 600px !important;"> <div id="pfptBannerv5yd6th" style="all: unset !important; display:block !important; visibility: visible !important; background-color: #D2D0D0 !important; color:#000000 !important; color:#000000; font-family: 'Arial', sans-serif !important; font-family: 'Arial', sans-serif; font-weight:bold !important; font-weight:bold; font-size:14px !important; line-height:18px !important; line-height:18px"> Prioritize security for external emails: </div> <div id="pfptBannerv5yd6th" style="all: unset !important; display:block !important; visibility: visible !important; background-color: #D2D0D0 !important; color:#000000 !important; color:#000000; font-weight:normal; font-family: 'Arial', sans-serif !important; font-family: 'Arial', sans-serif; font-size:12px !important; line-height:18px !important; line-height:18px; margin-top:2px !important;"> Confirm sender and content safety before clicking links or opening attachments </div> </div> <div id="pfptBannerv5yd6th" style="all: unset !important; float: right !important; display: block !important; display: block; margin: 0px 0px 0px 16px !important; text-align: right !important; width: fit-content !important;"> <a id="pfptBannerv5yd6th" href="https://us-phishalarm-ewt.proofpoint.com/EWT/v1/CRVmXkqW!ui3X2N84rjX7-6H8OTxazkNDjFJ0kvQsO4gPyKe8ks2dilRBSghAMrur7RbkStR2M7HAgg2na0jbuoSR6ul8oPbJS6ZmrrwjpOM7ZMymFuoPB9Kwi9T7TkjAWAAqXD7GCaflBzU$" style="all: unset !important; display: inline-block !important; text-decoration: none"> <div class="pfptPrimaryButtonv5yd6th" style="display: inline-block !important; display: inline-block; visibility: visible !important; opacity: 1 !important; color: #000000 !important; color: #000000; font-family: 'Arial', sans-serif !important; font-family: 'Arial', sans-serif; font-size: 14px !important; font-weight: normal !important; text-decoration: none !important; border-radius: 2px !important; padding: 7.5px 16px !important; margin: 3px 0 3px 16px !important; white-space: nowrap !important; width: fit-content !important; border: 1px solid #666666"> Report Suspicious </div> </a> </div> <div style="clear: both !important; display: block !important; visibility: hidden !important; line-height: 0 !important; font-size: 0.01px !important; height: 0px"> </div> </div> <![endif]> <div style="display:none !important;display:none;visibility:hidden;mso-hide:all;font-size:1px;color:#ffffff;line-height:1px;max-height:0px;opacity:0;overflow:hidden;">ZjQcmQRYFpfptBannerEnd</div>   <html> <head> <style> #pfptBannerv5yd6th { all: revert !important; display: block !important; visibility: visible !important; opacity: 1 !important; background-color: #D2D0D0 !important; max-width: none !important; max-height: none !important } .pfptPrimaryButtonv5yd6th:hover, .pfptPrimaryButtonv5yd6th:focus { background-color: #afaeae !important; } .pfptPrimaryButtonv5yd6th:active { background-color: #8d8c8c !important; } </style>  </head><body><pre style="font-family: sans-serif; font-size: 100%; white-space: pre-wrap; word-wrap: break-word"> > -----Original Message----- > From: Stephen Hemminger <stephen@networkplumber.org> > Sent: Wednesday, April 23, 2025 9:01 PM > To: edwin.brossette@6wind.com > Cc: dev@dpdk.org; olivier.matz@6wind.com; didier.pallard@6wind.com; > lauren.hardy@6wind.com; Devendra Singh Rawat > <dsinghrawat@marvell.com>; Alok Prasad <palok@marvell.com>; > stable@dpdk.org > Subject: [EXTERNAL] Re: [PATCH 2/5] net/qede: fix bad sanity check on Rx > queue release > > On Tue, 22 Apr 2025 17: 51: 40 +0200 edwin. brossette@ 6wind. com wrote: > > From: Edwin Brossette <edwin. brossette@ 6wind. com> > > As per the rte_mbuf > API: the driver is responsible of initializing all > the required fields. This > ZjQcmQRYFpfptBannerStart Prioritize security for external emails: > Confirm sender and content safety before clicking links or opening attachments > <https://us-phishalarm- > ewt.proofpoint.com/EWT/v1/CRVmXkqW!tO3Z1f8UAnVatG- > 9ub1azhlXOoqqvdVa2NGs146PaCzt5I1OZmqvjWYIEBzLlqf3wHpb0a3UlSO8Py2t > m0U$> > Report Suspicious > > ZjQcmQRYFpfptBannerEnd > On Tue, 22 Apr 2025 17:51:40 +0200 > edwin.brossette@6wind.com wrote: > > > From: Edwin Brossette <edwin.brossette@6wind.com> > > > > As per the rte_mbuf API: the driver is responsible of initializing all > > the required fields. This is not done at qede alloc, meaning there can > > be garbage data in mbufs memory, although this garbage data should be > > overwritten when the mbufs are used. Since a sanity check is done when > > freeing the queues, its possible some remaining garbage data causes a > > panic when trying to release the queues if some mbufs are being > > processed. > > > > Use rte_pktmbuf_raw_free() instead of rte_pktmbuf_free() as the sanity > > check is more relaxed. > > > > Fixes: 2ea6f76aff40 ("qede: add core driver") > > CC: stable@dpdk.org > > Patch looks fine, but DPDK is trying to follow the inclusive naming guidelines. > The term "sanity check" is on the not allowed list. > > I will reword the commit message. I will wait for next version with this fix. @Devendra Singh Rawat @Alok Prasad Please review this series. I am planning to merge next version in next week. > > <a href="https://urldefense.proofpoint.com/v2/url?u=https- >">https://urldefense.proofpoint.com/v2/url?u=https- ></a>3A__inclusivenaming.org_word-2Dlists_tier-2D2_sanity- > 2Dcheck_&d=DwICAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=1DGob4H4rxz6H8uIToz > GOCa0s5f4wCNtTa4UUKvcsvI&m=LCVRTeR2PgdHSbOJP9MjCM9h0sZ_33FUs4ox > nlqJOIWbNhI01OIvorgdTnNQJtrZ&s=zsvx54N2PbMxHZ7Y_zO1uP_QviEQoQDcv > 0Iak-rJzEk&e= </pre></body></html>