<div dir="auto">Thank you very much for the review and Ack.<div dir="auto"><br></div><div dir="auto">I’ll make sure to include version change notes under a notes section in future patch versions, as per the contribution guide.</div><div dir="auto"><br></div><div dir="auto">Also noted about maintainers, I had added them in v1 but missed them in v2. I’ll make sure to always use `get-maintainer.sh` going forward.</div><div dir="auto"><br></div><div dir="auto">Thanks again for the guidance and support.</div><div dir="auto"><br></div><div dir="auto">Best regards,  </div><div dir="auto">Khadem Ullah</div><div dir="auto"><br></div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Jun 26, 2025, 18:33 Dariusz Sosnowski <<a href="mailto:dsosnowski@nvidia.com">dsosnowski@nvidia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thank you very much for changes and detailed descriptions.<br>
It helped a lot during review.<br>
<br>
Acked-by: Dariusz Sosnowski <<a href="mailto:dsosnowski@nvidia.com" target="_blank" rel="noreferrer">dsosnowski@nvidia.com</a>><br>
<br>
On Thu, Jun 26, 2025 at 09:07:02AM -0400, Khadem Ullah wrote:<br>
> v2:<br>
>  - Added missing check for AGE + CT conflict in flow_dv_query().<br>
>  - Removed unnecessary null check from flow_aso_age_get_by_idx().<br>
>  - Added Fixes tag for LTS tracking.<br>
>  - Ensured .mailmap and Signed-off-by addresses match.<br>
<br>
In case of any future contribution would you be able to put the changes<br>
between versions in notes section of the patch?<br>
You can find the details here: <a href="https://doc.dpdk.org/guides/contributing/patches.html#creating-patches" rel="noreferrer noreferrer" target="_blank">https://doc.dpdk.org/guides/contributing/patches.html#creating-patches</a><br>
<br>
Also, in the future would you be able to send patches to all relevant<br>
maintainers? We have a script, ./devtools/get-maintainer.sh,<br>
which extracts the info from MAINTAINERS file.<br>
You can find more info here: <a href="https://doc.dpdk.org/guides/contributing/patches.html#sending-patches" rel="noreferrer noreferrer" target="_blank">https://doc.dpdk.org/guides/contributing/patches.html#sending-patches</a><br>
<br>
> <br>
> This patch fixes a segmentation fault that occurs when querying the<br>
> AGE action of a flow rule that uses indirect connection tracking (CT).<br>
> <br>
> Background:<br>
> AGE and CT indices share a union in the mlx5 flow struct. When using CT<br>
> without age, the age index is invalid. Querying AGE in this case leads<br>
> to a crash due to reading an invalid pointer.<br>
> <br>
> Fix:<br>
> Add a check in `flow_dv_query()` to prevent AGE queries on indirect CT<br>
> actions. This is the correct fix rather than null-checking the pool.<br>
> <br>
> Steps to reproduce:<br>
>  1. Create an indirect CT action:<br>
>     flow indirect_action 0 create ingress action conntrack / end<br>
> <br>
>  2. Create a root rule with jump:<br>
>     flow create 0 ingress pattern eth / ipv4 / tcp / end actions jump group 3 / end<br>
> <br>
>  3. Create a group 3 rule using the indirect action:<br>
>     flow create 0 group 3 ingress pattern eth / ipv4 / tcp / end actions indirect 0 / jump group 5 / end<br>
> <br>
>  4. Create a group 5 rule matching CT state:<br>
>     flow create 0 group 5 ingress pattern eth / ipv4 / tcp / conntrack is 1 / end actions queue index 5 / end<br>
> <br>
>  5. Querying the first rule causes segfault:<br>
>     flow query 0 1 age<br>
> <br>
> Fixes: 2d084f69aa26 ("net/mlx5: add translation of connection tracking action")<br>
> Cc: <a href="mailto:stable@dpdk.org" target="_blank" rel="noreferrer">stable@dpdk.org</a><br>
> <br>
> Signed-off-by: Khadem Ullah <<a href="mailto:14pwcse1224@uetpeshawar.edu.pk" target="_blank" rel="noreferrer">14pwcse1224@uetpeshawar.edu.pk</a>><br>
</blockquote></div>