| commit | 6d6c5c1eb39badb85be1df9b1d6aa5b345dac75f | |
| author | Lars-Peter Clausen <lars@metafoo.de> | |
| Mon, 28 Mar 2016 09:31:26 +0000 (28 11:31 +0200) | ||
| committer | Doug Ledford <dledford@redhat.com> | |
| Fri, 13 May 2016 23:39:33 +0000 (13 19:39 -0400) | ||
| tree | e84ea1965ece541191c6a0bc23d120a6b6236296 | treesnapshot (tar.gz zip) |
| parent | 2016de7ba013462d56fb344a5a8ed93f7a4c460a | commitdiff |
i40iw: 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.
Patch was generated using the following semantic patch:
// <smpl>
@@
expression irq;
@@
-synchronize_irq(irq);
free_irq(irq, ...);
// </smpl>
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Acked-by: Faisal Latif <faisal.latif#intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
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.
Patch was generated using the following semantic patch:
// <smpl>
@@
expression irq;
@@
-synchronize_irq(irq);
free_irq(irq, ...);
// </smpl>
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Acked-by: Faisal Latif <faisal.latif#intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
| drivers/infiniband/hw/i40iw/i40iw_main.c | diffblobblamehistory |