Re: [PATCH v3 02/13] scsi: fnic: Add and use fnic number
From: Martin K. Petersen
Date: Fri Nov 24 2023 - 20:26:11 EST
Hi Karan,
> Add fnic_num in fnic.h to identify fnic in a multi-fnic environment.
> Increment and set the fnic number during driver load in fnic_probe.
> Replace the host number with fnic number in debugfs.
I agree with Hannes. Everything else in the stack will be using the host
number. Your change will make it harder to correlate a SCSI error
message to an fnic driver instance.
If you absolutely need an instance number I suggest you add it as an
orthogonal value instead replacing the host. Also, we typically use an
idr for enumerating things like this.
--
Martin K. Petersen Oracle Linux Engineering