RE: [PATCH v4 4/5] scsi: fnic: Turn off FDMI ACTIVE flags on link down

[Karan Tilak Kumar (kartilak)]

On Friday, June 13, 2025 1:29 PM, John Meneghini <jmeneghi@xxxxxxxxxx> wrote:
>
> Hi Karan.
>
> You've got two patches in this series with the same Fixes: tag.
>
> [PATCH v4 2/5] scsi: fnic: Fix crash in fnic_wq_cmpl_handler when FDMI times out
> Fixes: 09c1e6ab4ab2 ("scsi: fnic: Add and integrate support for FDMI")
>
> [PATCH v4 4/5] scsi: fnic: Turn off FDMI ACTIVE flags on link down
> Fixes: 09c1e6ab4ab2 ("scsi: fnic: Add and integrate support for FDMI")
>
> both of these patches modify the same file:
>
> drivers/scsi/fnic/fdls_disc.c
>
> So I recommend you squash patch 4/5 and patch 2/5 into one.
>
> Thanks,
>
> /John

Thanks for your proposal, John.

I'm a bit hesitant to squash them into one since each patch fixes unrelated bugs.
The reason they have the same fixes tag is because we found two separate issues in the same commit.

I get your concern, however. What do you think about the following?

Move this patch to the beginning and increment driver version to 1.8.0.1:
[PATCH v4 2/5] scsi: fnic: Fix crash in fnic_wq_cmpl_handler when FDMI times out

Move this patch to the next and increment driver version to 1.8.0.2:
[PATCH v4 4/5] scsi: fnic: Turn off FDMI ACTIVE flags on link down

Move this patch to the next:
[PATCH v4 3/5] scsi: fnic: Add and improve logs in FDMI and FDMI ABTS paths

Ideally, I would have liked to squash this above patch with the first patch in this series, since it's easier to debug if and when an issue occurs in that path.
I separated them since it would be easier to review. I could add a Fixes tag here as well, but I'm not sure about that since they are just log messages,
and they are not really fixes.

Move this patch to the end:
[PATCH v4 1/5] scsi: fnic: Set appropriate logging level for log message

Regards,
Karan