<div dir="auto">Hi Stephen, <div dir="auto">Can we add only the check that fixes the segfault, or do you mean that it should be fixed at the PMD level?<br><div dir="auto"><br></div><div dir="auto">Best regards,</div><div dir="auto">Khadem</div></div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, Jul 22, 2025, 18:39 Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, 22 Jul 2025 07:54:39 -0400<br>
Khadem Ullah <<a href="mailto:14pwcse1224@uetpeshawar.edu.pk" target="_blank" rel="noreferrer">14pwcse1224@uetpeshawar.edu.pk</a>> wrote:<br>
<br>
> +     if (rte_eal_process_type() == RTE_PROC_SECONDARY &&<br>
> +             (dev == NULL || dev->data == NULL ||<br>
> +             dev->data->dev_private == NULL ||<br>
<br>
dev can't be NULL and checking it here will cause a Coverity warning.<br>
<br>
There are many other ethdev calls that will fail if primary dies.<br>
stats, xstats, rx/tx burst, ...<br>
<br>
I don't think it is good idea to add checks here.<br>
</blockquote></div>