[dpdk-dev] [PATCH v2 3/4] pci: add req handler field to generic pci device
Andrew Rybchenko
arybchenko at solarflare.com
Mon Oct 1 11:46:23 CEST 2018
On 9/30/18 5:16 PM, Jeff Guo wrote:
> There are some extended interrupt types in vfio pci device except from the
> existing interrupts, such as err and req notifier, they could be useful for
> device error monitoring. And these corresponding interrupt handler is
> different from the other interrupt handler that register in PMDs, so a new
> interrupt handler should be added. This patch will add specific req handler
> in generic pci device.
>
> Signed-off-by: Jeff Guo <jia.guo at intel.com>
> ---
> v2->v1:
> no change
> ---
> drivers/bus/pci/rte_bus_pci.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/bus/pci/rte_bus_pci.h b/drivers/bus/pci/rte_bus_pci.h
> index 0d1955f..c45a820 100644
> --- a/drivers/bus/pci/rte_bus_pci.h
> +++ b/drivers/bus/pci/rte_bus_pci.h
> @@ -66,6 +66,7 @@ struct rte_pci_device {
> uint16_t max_vfs; /**< sriov enable if not zero */
> enum rte_kernel_driver kdrv; /**< Kernel driver passthrough */
> char name[PCI_PRI_STR_SIZE+1]; /**< PCI location (ASCII) */
> + struct rte_intr_handle req_notifier_handler;/**< Req notifier handle */
It is uninitialized now everywhere the structure is used. In the best case
it is initialized with zeros (if the structure is memset or partly
initialized
on declaration) and zero FD is a valid file descriptor. Is it OK?
> };
>
> /**
More information about the dev
mailing list