Re: [PATCH] scsi:scsi_transport_fc:Add an additional flag to fc_host_fpin_rcv()

[James Smart <jsmart2021@gmail.com>]

On 2/8/2023 7:43 PM, Muneendra Kumar wrote:
> From: Muneendra <muneendra.kumar@broadcom.com>
> 
> The LLDD and the stack currently process FPINs received from the fabric,
> but the stack is not aware of any action taken by the driver to alleviate
> congestion. The current interface between the driver and the SCSI stack is
> limited to passing the notification mainly for statistics and heuristics.
> 
> The reaction to an FPIN could be handled either by the driver or by the
> stack (marginal path and failover). This patch enhances the interface to
> indicate if action on an FPIN has already been reacted to by the
> LLDDs or not.Add an additional flag to fc_host_fpin_rcv() to indicate
> if the FPIN has been acknowledged/reacted to by the driver.
> 
> Also added a new event code FCH_EVT_LINK_FPIN_ACK to notify to the user
> that the event has been acknowledged/reacted by the LLDD driver
> 
> Signed-off-by: Anil Gurumurthy <agurumurthy@marvell.com>
> Signed-off-by: Nilesh Javali <njavali@marvell.com>
> Signed-off-by: Muneendra <muneendra.kumar@broadcom.com>
> ---
>   drivers/scsi/lpfc/lpfc_els.c     |  2 +-
>   drivers/scsi/qla2xxx/qla_isr.c   |  2 +-
>   drivers/scsi/scsi_transport_fc.c | 10 +++++++---
>   include/scsi/scsi_transport_fc.h |  4 +++-
>   4 files changed, 12 insertions(+), 6 deletions(-)
> 

Reviewed-by: James Smart <jsmart2021@gmail.com>

Looks good.

Thanks

-- james