>
> The file drivers/scsi/pm8001/pm8001_hwi.c contains the piece of code
below,
> at
> the end of mpi_sata_completion() and mpi_sata_event(), which appears
wrong.
> While the test is opposite, the executed branch does exactly the same. I
believe
> one of the branches should spin_lockirqsave instead? not 100% sure how to
fix
> this.
>
> Roel
>
> } else if (t->uldd_task) {
> spin_unlock_irqrestore(&t->task_state_lock, flags);
> pm8001_ccb_task_free(pm8001_ha, t, ccb, tag);
> mb();/* ditto */
> spin_unlock_irq(&pm8001_ha->lock);
> t->task_done(t);
> spin_lock_irq(&pm8001_ha->lock);
> } else if (!t->uldd_task) {
> spin_unlock_irqrestore(&t->task_state_lock, flags);
> pm8001_ccb_task_free(pm8001_ha, t, ccb, tag);
> mb();/*ditto*/
> spin_unlock_irq(&pm8001_ha->lock);
> t->task_done(t);
> spin_lock_irq(&pm8001_ha->lock);
> }
> --
[Jack Wang] Current mainline code all task->uldd_task is not null, so we can
remove the below branch. Not sure why you said should use spin_lock_irqsave.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
