> -----Original Message----- > From: Nipun Gupta <[email protected]> > Sent: Tuesday, June 6, 2023 12:57 PM > To: Harman Kalra <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected] > Cc: [email protected]; [email protected]; > [email protected] > Subject: Re: [EXT] [PATCH v5 3/5] eal/interrupts: add IRQ count in interrupt > handle > > > > On 6/6/2023 12:48 PM, Harman Kalra wrote: > > > >> -----Original Message----- > >> From: Nipun Gupta <[email protected]> > >> Sent: Thursday, May 25, 2023 3:38 PM > >> To: [email protected]; [email protected]; > [email protected]; > >> Harman Kalra <[email protected]>; [email protected]; > >> [email protected] > >> Cc: [email protected]; [email protected]; > >> [email protected]; Nipun Gupta <[email protected]> > >> Subject: [EXT] [PATCH v5 3/5] eal/interrupts: add IRQ count in > >> interrupt handle > >> > >> External Email > >> > >> --------------------------------------------------------------------- > >> - Have total number of IRQ count support in interrupt handle. > >> In case of VFIO this IRQ count is returned when > >> VFIO_DEVICE_GET_IRQ_INFO ioctl is invoked. This IRQ_count can used by > >> the devices to store/provide total number of interrupts available and > >> to enable or disable these interrupts. > >> > > > > Hi Nipun, > > > > We already have "max_intr" field for the same purpose and its > > respective APIs > > plt_intr_max_intr_set()/plt_intr_max_intr_get() > > Hi Harman, > > If we have a look into rte_intr_efd_enable() API, 'max_intr' being set in this > API. So once a driver is using the interrupts the 'max_intr' > would be overwritten. 'nb_intr' which is described as "Max vector count" > seems more relevant to me here and I have used 'nb_intr' to have the total > interrupt count available and sent out the updated series for CDX bus. Please > let me know in case you have separate thoughts on this. >
Hi Nipun, Got your point, we are aligned on this. Just to bring to your notice, In pci_vfio_setup_interrupts(), if irq.count > nb_intr we are updating nb_intr as part of rte_intr_event_list_update() Thanks Harman > Thanks, > Nipun > > > > > Thanks > > Harman
