diff mbox

[dpdk-dev,v2,1/4] net/i40e: fix VF RX queue interrupt enabling

Message ID 1516781813-50818-2-git-send-email-wenzhuo.lu@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Helin Zhang
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail Compilation issues

Commit Message

Wenzhuo Lu Jan. 24, 2018, 8:16 a.m. UTC 
  When using UIO, after enabling the interrupt to get the PF
message, VF RX queue interrupt is not working.
It's expected, as UIO doesn't support multiple interrupt.
So, PMD should not try to enable RX queue interrupt. Then
APP can know the RX queue interrupt is not enabled and only
choose the polling mode.

Fixes: ae19955e7c86 ("i40evf: support reporting PF reset")
CC: stable@dpdk.org

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---
 drivers/net/i40e/i40e_ethdev_vf.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Zhang, Helin Jan. 26, 2018, 3:32 p.m. UTC | #1 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Wenzhuo Lu
> Sent: Wednesday, January 24, 2018 4:17 PM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo; stable@dpdk.org
> Subject: [dpdk-dev] [PATCH v2 1/4] net/i40e: fix VF RX queue interrupt
> enabling
> 
> When using UIO, after enabling the interrupt to get the PF message, VF RX
> queue interrupt is not working.
> It's expected, as UIO doesn't support multiple interrupt.
> So, PMD should not try to enable RX queue interrupt. Then APP can know the
> RX queue interrupt is not enabled and only choose the polling mode.
> 
> Fixes: ae19955e7c86 ("i40evf: support reporting PF reset")
> CC: stable@dpdk.org
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
Zhang, Helin Jan. 26, 2018, 3:40 p.m. UTC | #2 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Zhang, Helin
> Sent: Friday, January 26, 2018 11:32 PM
> To: Lu, Wenzhuo; dev@dpdk.org
> Cc: Lu, Wenzhuo; stable@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2 1/4] net/i40e: fix VF RX queue interrupt
> enabling
> 
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Wenzhuo Lu
> > Sent: Wednesday, January 24, 2018 4:17 PM
> > To: dev@dpdk.org
> > Cc: Lu, Wenzhuo; stable@dpdk.org
> > Subject: [dpdk-dev] [PATCH v2 1/4] net/i40e: fix VF RX queue interrupt
> > enabling
> >
> > When using UIO, after enabling the interrupt to get the PF message, VF
> > RX queue interrupt is not working.
> > It's expected, as UIO doesn't support multiple interrupt.
> > So, PMD should not try to enable RX queue interrupt. Then APP can know
> > the RX queue interrupt is not enabled and only choose the polling mode.
> >
> > Fixes: ae19955e7c86 ("i40evf: support reporting PF reset")
> > CC: stable@dpdk.org
> >
> > Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
> Acked-by: Helin Zhang <helin.zhang@intel.com>
Applied to dpdk-next-net-intel, with commit log changes. Thanks!

/Helin
diff mbox

Patch

diff --git a/drivers/net/i40e/i40e_ethdev_vf.c b/drivers/net/i40e/i40e_ethdev_vf.c
index 0cca0d3..60783f9 100644
--- a/drivers/net/i40e/i40e_ethdev_vf.c
+++ b/drivers/net/i40e/i40e_ethdev_vf.c
@@ -1976,7 +1976,8 @@  static int eth_i40evf_pci_remove(struct rte_pci_device *pci_dev)
 					dev->data->nb_tx_queues);
 
 	/* check and configure queue intr-vector mapping */
-	if (dev->data->dev_conf.intr_conf.rxq != 0) {
+	if (rte_intr_cap_multiple(intr_handle) &&
+	    dev->data->dev_conf.intr_conf.rxq) {
 		intr_vector = dev->data->nb_rx_queues;
 		if (rte_intr_efd_enable(intr_handle, intr_vector))
 			return -1;