mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-10-01 22:54:01 +00:00
qed: Remove unnecessary synchronize_irq() before free_irq()
Calling synchronize_irq() right before free_irq() is quite useless. On one hand the IRQ can easily fire again before free_irq() is entered, on the other hand free_irq() itself calls synchronize_irq() internally (in a race condition free way), before any state associated with the IRQ is freed. Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn> Link: https://lore.kernel.org/r/20220516072646.1651109-1-chi.minghao@zte.com.cn Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
parent
a968c799eb
commit
29fd3ca177
1 changed files with 0 additions and 1 deletions
|
@ -823,7 +823,6 @@ static void qed_slowpath_irq_free(struct qed_dev *cdev)
|
|||
for_each_hwfn(cdev, i) {
|
||||
if (!cdev->hwfns[i].b_int_requested)
|
||||
break;
|
||||
synchronize_irq(cdev->int_params.msix_table[i].vector);
|
||||
free_irq(cdev->int_params.msix_table[i].vector,
|
||||
&cdev->hwfns[i].sp_dpc);
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue