@@ -1001,10 +1001,10 @@ efc_domain_dispatch_frame(void *arg, struct efc_hw_sequence *seq)
if (node->hold_frames || !list_empty(&node->pend_frames)) {
/* add frame to node's pending list */
- spin_lock_irqsave(&node->pend_frames_lock, flags);
+ spin_lock(&node->pend_frames_lock);
INIT_LIST_HEAD(&seq->list_entry);
list_add_tail(&seq->list_entry, &node->pend_frames);
- spin_unlock_irqrestore(&node->pend_frames_lock, flags);
+ spin_unlock(&node->pend_frames_lock);
rc = EFC_HW_SEQ_HOLD;
goto out_release;
}
Calling a nested spin_lock_irqsave() will overwrite the original "flags" so that they can not be enabled again at the end. Fixes: 3146240f19bf ("scsi: elx: libefc: FC Domain state machine interfaces") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/scsi/elx/libefc/efc_domain.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)