Re: [PATCH 09/11] lpfc: Add capability to register Platform Name ID to fabric

[Krzysztof Kozlowski <krzk@kernel.org>]

On 28/10/2025 00:54, Justin Tee wrote:
> FC-LS and FC-GS specifications outline fields for registering a platform
> name identifier (PNI) to the fabric.  The PNI assists fabrics with
> identifying the physical server source of frames in the fabric.
> 
> lpfc generates a PNI based partially on the uuid specific for the system.
> Initial attempts to extract a uuid are made from SMBIOS's System
> Information 08h uuid entry.  If SMBIOS DMI does not exist, then Open
> Firmware Device-Tree's virtual partition uuid property is used.  Otherwise,
> a PNI is not generated and PNI registration with the fabric is skipped.
> 
> The PNI is submitted in FLOGI and FDISC frames.  After successful fabric
> login, the RSPNI_PNI CT frame is submitted to the fabric to register the OS
> host name tying it to the PNI.
> 
> Signed-off-by: Justin Tee <justin.tee@broadcom.com>

Still does not match From address.

> ---
>  drivers/scsi/lpfc/lpfc.h           |  4 ++
>  drivers/scsi/lpfc/lpfc_ct.c        | 36 +++++++++++++
>  drivers/scsi/lpfc/lpfc_els.c       | 18 +++++--
>  drivers/scsi/lpfc/lpfc_hbadisc.c   |  2 +
>  drivers/scsi/lpfc/lpfc_hw.h        | 25 ++++++++-
>  drivers/scsi/lpfc/lpfc_nportdisc.c |  4 --
>  drivers/scsi/lpfc/lpfc_sli.c       | 83 ++++++++++++++++++++++++++++++
>  7 files changed, 164 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/scsi/lpfc/lpfc.h b/drivers/scsi/lpfc/lpfc.h
> index 8459cf568c12..f892bb2d119a 100644
> --- a/drivers/scsi/lpfc/lpfc.h
> +++ b/drivers/scsi/lpfc/lpfc.h
> @@ -633,6 +633,7 @@ struct lpfc_vport {
>  #define FC_CT_RSPN_ID		0x8	 /* RSPN_ID accepted by switch */
>  #define FC_CT_RFT_ID		0x10	 /* RFT_ID accepted by switch */
>  #define FC_CT_RPRT_DEFER	0x20	 /* Defer issuing FDMI RPRT */
> +#define FC_CT_RSPNI_PNI		0x40	 /* RSPNI_PNI accepted by switch */
>  
>  	struct list_head fc_nodes;
>  	spinlock_t fc_nodes_list_lock; /* spinlock for fc_nodes list */
> @@ -1077,6 +1078,9 @@ struct lpfc_hba {
>  
>  	uint32_t nport_event_cnt;	/* timestamp for nlplist entry */
>  
> +	unsigned long pni;		/* 64-bit Platform Name Identifier */
> +#define PATH_UUID_IBM "ibm,partition-uuid"

Where did you document the ABI?

Best regards,
Krzysztof