Skip to content

Commit db36c06

Browse files
Chris LeechJames Bottomley
authored andcommitted
[SCSI] libfc, libfcoe: FDISC ELS for NPIV
Add FDISC ELS handling to libfc and libfcoe, treat it the same as FLOGI where appropriate. Add checking for NPIV support in the FLOGI LS_ACC service parameters. Signed-off-by: Chris Leech <christopher.leech@intel.com> Signed-off-by: Robert Love <robert.w.love@intel.com> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
1 parent 8faecdd commit db36c06

4 files changed

Lines changed: 40 additions & 5 deletions

File tree

drivers/scsi/fcoe/libfcoe.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -449,7 +449,7 @@ static int fcoe_ctlr_encaps(struct fcoe_ctlr *fip,
449449
memset(mac, 0, sizeof(mac));
450450
mac->fd_desc.fip_dtype = FIP_DT_MAC;
451451
mac->fd_desc.fip_dlen = sizeof(*mac) / FIP_BPW;
452-
if (dtype != FIP_DT_FLOGI)
452+
if (dtype != FIP_DT_FLOGI && dtype != FIP_DT_FDISC)
453453
memcpy(mac->fd_mac, fip->data_src_addr, ETH_ALEN);
454454
else if (fip->spma)
455455
memcpy(mac->fd_mac, fip->ctl_src_addr, ETH_ALEN);
@@ -865,8 +865,8 @@ static void fcoe_ctlr_recv_els(struct fcoe_ctlr *fip, struct sk_buff *skb)
865865
goto drop;
866866
els_op = *(u8 *)(fh + 1);
867867

868-
if (els_dtype == FIP_DT_FLOGI && sub == FIP_SC_REP &&
869-
fip->flogi_oxid == ntohs(fh->fh_ox_id) &&
868+
if ((els_dtype == FIP_DT_FLOGI || els_dtype == FIP_DT_FDISC) &&
869+
sub == FIP_SC_REP && fip->flogi_oxid == ntohs(fh->fh_ox_id) &&
870870
els_op == ELS_LS_ACC && is_valid_ether_addr(granted_mac)) {
871871
fip->flogi_oxid = FC_XID_UNKNOWN;
872872
fip->update_mac(fip, fip->data_src_addr, granted_mac);

drivers/scsi/libfc/fc_lport.c

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1449,6 +1449,9 @@ static void fc_lport_flogi_resp(struct fc_seq *sp, struct fc_frame *fp,
14491449
e_d_tov = ntohl(flp->fl_csp.sp_e_d_tov);
14501450
if (csp_flags & FC_SP_FT_EDTR)
14511451
e_d_tov /= 1000000;
1452+
1453+
lport->npiv_enabled = !!(csp_flags & FC_SP_FT_NPIV_ACC);
1454+
14521455
if ((csp_flags & FC_SP_FT_FPORT) == 0) {
14531456
if (e_d_tov > lport->e_d_tov)
14541457
lport->e_d_tov = e_d_tov;
@@ -1498,7 +1501,8 @@ void fc_lport_enter_flogi(struct fc_lport *lport)
14981501
if (!fp)
14991502
return fc_lport_error(lport, fp);
15001503

1501-
if (!lport->tt.elsct_send(lport, FC_FID_FLOGI, fp, ELS_FLOGI,
1504+
if (!lport->tt.elsct_send(lport, FC_FID_FLOGI, fp,
1505+
lport->vport ? ELS_FDISC : ELS_FLOGI,
15021506
fc_lport_flogi_resp, lport, lport->e_d_tov))
15031507
fc_lport_error(lport, NULL);
15041508
}

include/scsi/fc/fc_els.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,10 +248,12 @@ struct fc_els_csp {
248248
/*
249249
* sp_features
250250
*/
251-
#define FC_SP_FT_CIRO 0x8000 /* continuously increasing rel. off. */
251+
#define FC_SP_FT_NPIV 0x8000 /* multiple N_Port_ID support (FLOGI) */
252+
#define FC_SP_FT_CIRO 0x8000 /* continuously increasing rel off (PLOGI) */
252253
#define FC_SP_FT_CLAD 0x8000 /* clean address (in FLOGI LS_ACC) */
253254
#define FC_SP_FT_RAND 0x4000 /* random relative offset */
254255
#define FC_SP_FT_VAL 0x2000 /* valid vendor version level */
256+
#define FC_SP_FT_NPIV_ACC 0x2000 /* NPIV assignment (FLOGI LS_ACC) */
255257
#define FC_SP_FT_FPORT 0x1000 /* F port (1) vs. N port (0) */
256258
#define FC_SP_FT_ABB 0x0800 /* alternate BB_credit management */
257259
#define FC_SP_FT_EDTR 0x0400 /* E_D_TOV Resolution is nanoseconds */

include/scsi/fc_encode.h

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,31 @@ static inline void fc_flogi_fill(struct fc_lport *lport, struct fc_frame *fp)
198198
sp->sp_bb_data = htons((u16) lport->mfs);
199199
cp = &flogi->fl_cssp[3 - 1]; /* class 3 parameters */
200200
cp->cp_class = htons(FC_CPC_VALID | FC_CPC_SEQ);
201+
if (lport->does_npiv)
202+
sp->sp_features = htons(FC_SP_FT_NPIV);
203+
}
204+
205+
/**
206+
* fc_fdisc_fill - Fill in a fdisc request frame.
207+
*/
208+
static inline void fc_fdisc_fill(struct fc_lport *lport, struct fc_frame *fp)
209+
{
210+
struct fc_els_csp *sp;
211+
struct fc_els_cssp *cp;
212+
struct fc_els_flogi *fdisc;
213+
214+
fdisc = fc_frame_payload_get(fp, sizeof(*fdisc));
215+
memset(fdisc, 0, sizeof(*fdisc));
216+
fdisc->fl_cmd = (u8) ELS_FDISC;
217+
put_unaligned_be64(lport->wwpn, &fdisc->fl_wwpn);
218+
put_unaligned_be64(lport->wwnn, &fdisc->fl_wwnn);
219+
sp = &fdisc->fl_csp;
220+
sp->sp_hi_ver = 0x20;
221+
sp->sp_lo_ver = 0x20;
222+
sp->sp_bb_cred = htons(10); /* this gets set by gateway */
223+
sp->sp_bb_data = htons((u16) lport->mfs);
224+
cp = &fdisc->fl_cssp[3 - 1]; /* class 3 parameters */
225+
cp->cp_class = htons(FC_CPC_VALID | FC_CPC_SEQ);
201226
}
202227

203228
/**
@@ -296,6 +321,10 @@ static inline int fc_els_fill(struct fc_lport *lport,
296321
fc_flogi_fill(lport, fp);
297322
break;
298323

324+
case ELS_FDISC:
325+
fc_fdisc_fill(lport, fp);
326+
break;
327+
299328
case ELS_LOGO:
300329
fc_logo_fill(lport, fp);
301330
break;

0 commit comments

Comments
 (0)