<div dir="auto"><div><br><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Le jeu. 6 nov. 2025, 20:29, Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu,  6 Nov 2025 22:08:06 +0530<br>
Hemant Agrawal <<a href="mailto:hemant.agrawal@nxp.com" target="_blank" rel="noreferrer">hemant.agrawal@nxp.com</a>> wrote:<br>
<br>
> +static void<br>
> +dpaa2_clear_queue_active_dps(struct dpaa2_queue *q, int num_lcores)<br>
> +{<br>
> +     int i;<br>
> +<br>
> +     for (i = 0; i < num_lcores; i++) {<br>
> +             struct queue_storage_info_t *qs = q->q_storage[i];<br>
> +<br>
> +             if (!qs)<br>
> +                     continue;<br>
> +<br>
> +             if (qs->active_dqs) {<br>
> +                     clear_swp_active_dqs(qs->active_dpio_id);<br>
> +                     qs->active_dqs = NULL;<br>
> +             }<br>
> +     }<br>
> +}<br>
> +<br>
<br>
Why not use RTE_LCORE_FOREACH() here?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">For the loop, I did it the same way as in dpaa2_queue_storage_alloc(), to stay aligned with the rest of the driver instead of using RTE_LCORE_FOREACH().</div><div dir="auto"><br></div><div dir="auto">Thanks Hemant for upstreaming my patch. However, you added my email in .mailmap as a new entry — it should instead be added as a second email under the existing Maxime Leroy entry.</div><div dir="auto"><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>