<div dir="ltr"><div>Recheck-request: iol-intel-Performance</div><div><br></div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-family:verdana,sans-serif">Thanks</span></div><div><span style="font-family:verdana,sans-serif">Amiya</span><br></div></div></div></div><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, 4 Sept 2026 at 14:52, Amiya Ranjan Mohakud <<a href="mailto:amiyaranjan.mohakud@gmail.com">amiyaranjan.mohakud@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">npa_aura_pool_init() allocates the pool init mbox message twice on<br>
non-CN20K platforms. The if/else block at line 106-111 allocates via<br>
the model-specific branch, but the original unconditional allocation<br>
at line 112 was not removed when the CN20K branch was added. This<br>
overwrites pool_init_req, and the orphaned first message is sent to<br>
the AF with uninitialized aura_id, ctype, and op fields, which can<br>
trigger any undesired behavior on CN10K.<br>
<br>
Remove the duplicate allocation.<br>
<br>
Fixes: 143a419edf35 ("common/cnxk: use new NPA aq enq mbox for CN20K")<br>
Cc: <a href="mailto:asekhar@marvell.com" target="_blank">asekhar@marvell.com</a><br>
Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
<br>
Signed-off-by: Amiya Ranjan Mohakud <<a href="mailto:amiyaranjan.mohakud@gmail.com" target="_blank">amiyaranjan.mohakud@gmail.com</a>><br>
---<br>
 drivers/common/cnxk/roc_npa.c | 1 -<br>
 1 file changed, 1 deletion(-)<br>
<br>
diff --git a/drivers/common/cnxk/roc_npa.c b/drivers/common/cnxk/roc_npa.c<br>
index 4a3e96a97a..87f7637e9f 100644<br>
--- a/drivers/common/cnxk/roc_npa.c<br>
+++ b/drivers/common/cnxk/roc_npa.c<br>
@@ -110,7 +110,6 @@ npa_aura_pool_init(struct mbox *m_box, uint32_t aura_id, struct npa_aura_s *aura<br>
        } else {<br>
                pool_init_req = mbox_alloc_msg_npa_aq_enq(mbox);<br>
        }<br>
-       pool_init_req = mbox_alloc_msg_npa_aq_enq(mbox);<br>
        if (pool_init_req == NULL)<br>
                goto exit;<br>
        pool_init_req->aura_id = aura_id;<br>
-- <br>
2.39.5 (Apple Git-154)<br>
<br>
</blockquote></div>