[PATCH v4 8/8] interrupts: warn on leaked event FDs

Burakov, Anatoly anatoly.burakov at intel.com
Fri Oct 23 16:48:35 CEST 2026


On 10/23/2026 4:41 PM, David Marchand wrote:
> On Fri, 23 Oct 2026 at 15:52, Anatoly Burakov <anatoly.burakov at intel.com> wrote:
>> diff --git a/lib/eal/linux/eal_interrupts.c b/lib/eal/linux/eal_interrupts.c
>> index aadb7b84a22..f6f51ba53c0 100644
>> --- a/lib/eal/linux/eal_interrupts.c
>> +++ b/lib/eal/linux/eal_interrupts.c
>> @@ -1667,6 +1667,8 @@ rte_intr_efd_disable(struct rte_intr_handle *intr_handle)
>>                  for (i = 0; i < (uint32_t)rte_intr_nb_efd_get(intr_handle); i++)
>>                          close(rte_intr_efds_index_get(intr_handle, i));
>>          }
>> +       for (i = 0; i < (uint32_t)rte_intr_nb_efd_get(intr_handle); i++)
>> +               rte_intr_efds_index_set(intr_handle, i, -1);
> 
> We don't want to waive for types != VFIO_MSIX, do we?
> 

Looking into this further, I suspectwe want to check for != VDEV 
instead. All handle types except VDEV will write into the handle.

> 
>>          rte_intr_nb_efd_set(intr_handle, 0);
>>          rte_intr_max_intr_set(intr_handle, 0);
>>   }
> 
> The rest lgtm.
> 
> 


-- 
Thanks,
Anatoly


More information about the dev mailing list