On 2020/7/13 下午6:22, Zhu, Lingshan wrote:

On 7/13/2020 4:33 PM, Jason Wang wrote:

quoted

On 2020/7/12 下午10:49, Zhu Lingshan wrote:

quoted

This commit replaced irq_request/free() with helpers in vDPA
core, so that it can request/free irq and setup irq offloading
on order.

Signed-off-by: Zhu Lingshan <redacted>
---
  drivers/vdpa/ifcvf/ifcvf_main.c | 11 ++++++-----
  1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/vdpa/ifcvf/ifcvf_main.c 

b/drivers/vdpa/ifcvf/ifcvf_main.c
index f5a60c1..65b84e1 100644

--- a/drivers/vdpa/ifcvf/ifcvf_main.c
+++ b/drivers/vdpa/ifcvf/ifcvf_main.c

@@ -47,11 +47,12 @@ static void ifcvf_free_irq(struct ifcvf_adapter 

*adapter, int queues)
  {
      struct pci_dev *pdev = adapter->pdev;
      struct ifcvf_hw *vf = &adapter->vf;
+    struct vdpa_device *vdpa = &adapter->vdpa;
      int i;
          for (i = 0; i < queues; i++)
-        devm_free_irq(&pdev->dev, vf->vring[i].irq, &vf->vring[i]);
+        vdpa_free_vq_irq(&pdev->dev, vdpa, vf->vring[i].irq, i, 
&vf->vring[i]);
        ifcvf_free_irq_vectors(pdev);
  }

@@ -60,6 +61,7 @@ static int ifcvf_request_irq(struct ifcvf_adapter 

*adapter)
  {
      struct pci_dev *pdev = adapter->pdev;
      struct ifcvf_hw *vf = &adapter->vf;
+    struct vdpa_device *vdpa = &adapter->vdpa;
      int vector, i, ret, irq;
        ret = pci_alloc_irq_vectors(pdev, IFCVF_MAX_INTR,

@@ -73,6 +75,7 @@ static int ifcvf_request_irq(struct ifcvf_adapter 

*adapter)
           pci_name(pdev));
      vector = 0;
      irq = pci_irq_vector(pdev, vector);
+    /* config interrupt */

Unnecessary changes.

This is to show we did not setup this irq offloading for config 
interrupt, only setup irq offloading for data vq. But can remove this 
since we have config_msix_name in code to show what it is

Btw, any reason for not making config interrupt work for irq offloading? 
I don't see any thing that blocks this.

Thanks

Thanks BR Zhu Lingshan

quoted