Commit 78d0264e authored by Christoph Hellwig's avatar Christoph Hellwig

irq_poll: don't disable new irq_poll instances

There is no good reason to start out disabled - drivers can control if
the poll instance can be scheduled by simply not scheduling it yet.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarBart Van Assche <bart.vanassche@sandisk.com>
parent 511cbce2
...@@ -5581,7 +5581,6 @@ static void beiscsi_eeh_resume(struct pci_dev *pdev) ...@@ -5581,7 +5581,6 @@ static void beiscsi_eeh_resume(struct pci_dev *pdev)
pbe_eq = &phwi_context->be_eq[i]; pbe_eq = &phwi_context->be_eq[i];
irq_poll_init(&pbe_eq->iopoll, be_iopoll_budget, irq_poll_init(&pbe_eq->iopoll, be_iopoll_budget,
be_iopoll); be_iopoll);
irq_poll_enable(&pbe_eq->iopoll);
} }
i = (phba->msix_enabled) ? i : 0; i = (phba->msix_enabled) ? i : 0;
...@@ -5754,7 +5753,6 @@ static int beiscsi_dev_probe(struct pci_dev *pcidev, ...@@ -5754,7 +5753,6 @@ static int beiscsi_dev_probe(struct pci_dev *pcidev,
pbe_eq = &phwi_context->be_eq[i]; pbe_eq = &phwi_context->be_eq[i];
irq_poll_init(&pbe_eq->iopoll, be_iopoll_budget, irq_poll_init(&pbe_eq->iopoll, be_iopoll_budget,
be_iopoll); be_iopoll);
irq_poll_enable(&pbe_eq->iopoll);
} }
i = (phba->msix_enabled) ? i : 0; i = (phba->msix_enabled) ? i : 0;
......
...@@ -3708,7 +3708,6 @@ static ssize_t ipr_store_iopoll_weight(struct device *dev, ...@@ -3708,7 +3708,6 @@ static ssize_t ipr_store_iopoll_weight(struct device *dev,
for (i = 1; i < ioa_cfg->hrrq_num; i++) { for (i = 1; i < ioa_cfg->hrrq_num; i++) {
irq_poll_init(&ioa_cfg->hrrq[i].iopoll, irq_poll_init(&ioa_cfg->hrrq[i].iopoll,
ioa_cfg->iopoll_weight, ipr_iopoll); ioa_cfg->iopoll_weight, ipr_iopoll);
irq_poll_enable(&ioa_cfg->hrrq[i].iopoll);
} }
} }
spin_unlock_irqrestore(shost->host_lock, lock_flags); spin_unlock_irqrestore(shost->host_lock, lock_flags);
...@@ -10407,7 +10406,6 @@ static int ipr_probe(struct pci_dev *pdev, const struct pci_device_id *dev_id) ...@@ -10407,7 +10406,6 @@ static int ipr_probe(struct pci_dev *pdev, const struct pci_device_id *dev_id)
for (i = 1; i < ioa_cfg->hrrq_num; i++) { for (i = 1; i < ioa_cfg->hrrq_num; i++) {
irq_poll_init(&ioa_cfg->hrrq[i].iopoll, irq_poll_init(&ioa_cfg->hrrq[i].iopoll,
ioa_cfg->iopoll_weight, ipr_iopoll); ioa_cfg->iopoll_weight, ipr_iopoll);
irq_poll_enable(&ioa_cfg->hrrq[i].iopoll);
} }
} }
......
...@@ -170,8 +170,7 @@ EXPORT_SYMBOL(irq_poll_enable); ...@@ -170,8 +170,7 @@ EXPORT_SYMBOL(irq_poll_enable);
* @poll_fn: The handler to invoke * @poll_fn: The handler to invoke
* *
* Description: * Description:
* Initialize this irq_poll structure. Before being actively used, the * Initialize and enable this irq_poll structure.
* driver must call irq_poll_enable().
**/ **/
void irq_poll_init(struct irq_poll *iop, int weight, irq_poll_fn *poll_fn) void irq_poll_init(struct irq_poll *iop, int weight, irq_poll_fn *poll_fn)
{ {
...@@ -179,7 +178,6 @@ void irq_poll_init(struct irq_poll *iop, int weight, irq_poll_fn *poll_fn) ...@@ -179,7 +178,6 @@ void irq_poll_init(struct irq_poll *iop, int weight, irq_poll_fn *poll_fn)
INIT_LIST_HEAD(&iop->list); INIT_LIST_HEAD(&iop->list);
iop->weight = weight; iop->weight = weight;
iop->poll = poll_fn; iop->poll = poll_fn;
set_bit(IRQ_POLL_F_SCHED, &iop->state);
} }
EXPORT_SYMBOL(irq_poll_init); EXPORT_SYMBOL(irq_poll_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment