On 11/30/2017 5:11 AM, Ravi Kumar wrote:
> Signed-off-by: Ravi Kumar <ravi1.ku...@amd.com>
<...>
> +static void
> +axgbe_dev_interrupt_handler(void *param)
> +{
> + struct rte_eth_dev *dev = (struct rte_eth_dev *)param;
> + struct axgbe_port *pdata = dev->data->dev_private;
> +
> + pdata->phy_if.an_isr(pdata);
> +
> + rte_intr_enable(&pdata->pci_dev->intr_handle);
This is already inside the interrupt handler, shouldn't interrupts needs to be
enabled somewhere else to reach this point at first place.
And I have seen a few drivers does the same but do we need to call
rte_intr_enable() in interrupt handler? Will interrupts disabled after each
fire?
<...>
