Hi Jingjing,

> -----Original Message-----
> From: Wu, Jingjing
> Sent: Thursday, May 3, 2018 11:09 AM
> To: Lu, Wenzhuo <wenzhuo...@intel.com>
> Cc: Wu, Jingjing <jingjing...@intel.com>; dev@dpdk.org; sta...@dpdk.org
> Subject: [PATCH] net/avf: fix Rx interrupt mapping
> 
> Vector used for rx mapping is different if WB_ON_ITR is supported. The
> mapping table need to be updated.
> 
> Fixes: d6bde6b5eae9 ("net/avf: enable Rx interrupt")
> 
> Signed-off-by: Jingjing Wu <jingjing...@intel.com>
> Cc: sta...@dpdk.org
> ---
>  drivers/net/avf/avf_ethdev.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/avf/avf_ethdev.c b/drivers/net/avf/avf_ethdev.c index
> a1ae3a2..db0eda4 100644
> --- a/drivers/net/avf/avf_ethdev.c
> +++ b/drivers/net/avf/avf_ethdev.c
> @@ -339,13 +339,13 @@ static int avf_config_rx_queues_irqs(struct
> rte_eth_dev *dev,
>               AVF_WRITE_FLUSH(hw);
>               /* map all queues to the same interrupt */
>               for (i = 0; i < dev->data->nb_rx_queues; i++)
> -                     vf->rxq_map[0] |= 1 << i;
> +                     vf->rxq_map[vf->msix_base] |= 1 << i;
>       } else {
>               if (!rte_intr_allow_others(intr_handle)) {
>                       vf->nb_msix = 1;
>                       vf->msix_base = AVF_MISC_VEC_ID;
>                       for (i = 0; i < dev->data->nb_rx_queues; i++) {
> -                             vf->rxq_map[0] |= 1 << i;
> +                             vf->rxq_map[AVF_MISC_VEC_ID] |= 1 << i;
Looks better using " vf->msix_base" here too.

>                               intr_handle->intr_vec[i] = AVF_MISC_VEC_ID;
>                       }
>                       PMD_DRV_LOG(DEBUG,
Better updating the debug log either.

> --
> 2.4.11

Reply via email to