On Wed, Mar 9, 2016 at 7:22 AM, Martin K. Petersen <martin.petersen@xxxxxxxxxx> wrote: >>>>>> "Lars-Peter" == Lars-Peter Clausen <lars@xxxxxxxxxx> writes: > > Lars-Peter> Calling synchronize_irq() right before free_irq() is quite > Lars-Peter> useless. On one hand the IRQ can easily fire again before > Lars-Peter> free_irq() is entered, on the other hand free_irq() itself > Lars-Peter> calls synchronize_irq() internally (in a race condition free > Lars-Peter> way), before any state associated with the IRQ is freed. > > Broadcom folks, please review. Martin, Please consider this patch as Ack-by: Sreekanth Reddy <sreekanth.reddy@xxxxxxxxxxxx> Thanks, Sreekanth > > -- > Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html