Re: [PATCH v3 1/2] Revert "scsi: fcoe: Fix potential deadlock on &fip->ctlr_lock"

From: Hannes Reinecke
Date: Fri Feb 09 2024 - 05:19:26 EST

Next message: Hannes Reinecke: "Re: [PATCH v3 2/2] fnic: move fnic_fnic_flush_tx() to a work queue"
Previous message: Christian Brauner: "Re: [PATCH] eventfd: strictly check the count parameter of eventfd_write to avoid inputting illegal strings"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On 2/7/24 23:20, Lee Duncan wrote:

From: Lee Duncan <lduncan@xxxxxxxx>

This reverts commit 1a1975551943f681772720f639ff42fbaa746212

This commit causes interrupts to be lost for FCoE devices,
since it changed sping locks from "bh" to "irqsave".

Instead, a work queue should be used, and will be addressed
in a separate patch.

Fixes: 1a1975551943f681772720f639ff42fbaa746212

Please use the correct fixes tag:

Fixes: 1a1975551943 ("scsi: fcoe: Fix potential deadlock on &fip->ctlr_lock")

Otherwise looks good.

Cheers,

Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@xxxxxxx +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), GF: Ivo Totev, Andrew McDonald,
Werner Knoblich

Next message: Hannes Reinecke: "Re: [PATCH v3 2/2] fnic: move fnic_fnic_flush_tx() to a work queue"
Previous message: Christian Brauner: "Re: [PATCH] eventfd: strictly check the count parameter of eventfd_write to avoid inputting illegal strings"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]