On Thu, Nov 23, 2017 at 09:31:14AM +0800, Ching Huang wrote: > @@ -829,12 +833,15 @@ arcmsr_request_irq(struct pci_dev *pdev, > unsigned long flags; > int nvec, i; > > + if (msix_enable == 0) > + goto msi_int0; I feel like this goto is not very beautiful, but I can't actually apply this patch? Which tree is this written against? I'm using linux-next. regards, dan carpenter