diff options
author | Vasu Dev <vasu.dev@intel.com> | 2010-07-20 18:21:22 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:06:03 -0400 |
commit | cf4aebcafb44a8810af10006dd4a5fcfb07bb810 (patch) | |
tree | 5621b30b8d64e8670e2b1b1772621fe5495b6375 /drivers/scsi/fcoe | |
parent | 54a5b21da9d4d3f58770da5d1c244db9724659ee (diff) |
[SCSI] Revert "[SCSI] fcoe: Fix using VLAN ID in creating lport's WWWN/WWPN"
This reverts commit cc0136c2e9c10e889cb36e39710c0eb10707b396.
That commit introduced vlan id info to WWPN but WWPN needs to
remain static as an unique port identifier in the fabric, therefore
variable fabric vlan id info doesn't need to be coded inside WWPN.
After this revert, port arg to fcoe_wwn_from_mac is always zero
but still leaving it as-is okay to later allow users to use NAA 2
scheme with this additional port arg.
Note with this patch, existing zoning using WWPN would require
re-zoning this time only and later no more re-zoning due to any
vlan id changes.
Signed-off-by: Vasu Dev <vasu.dev@intel.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/fcoe')
-rw-r--r-- | drivers/scsi/fcoe/fcoe.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c index cf9d718c731c..ddd2ca27e003 100644 --- a/drivers/scsi/fcoe/fcoe.c +++ b/drivers/scsi/fcoe/fcoe.c | |||
@@ -647,7 +647,6 @@ static int fcoe_netdev_config(struct fc_lport *lport, struct net_device *netdev) | |||
647 | u64 wwnn, wwpn; | 647 | u64 wwnn, wwpn; |
648 | struct fcoe_interface *fcoe; | 648 | struct fcoe_interface *fcoe; |
649 | struct fcoe_port *port; | 649 | struct fcoe_port *port; |
650 | int vid = 0; | ||
651 | 650 | ||
652 | /* Setup lport private data to point to fcoe softc */ | 651 | /* Setup lport private data to point to fcoe softc */ |
653 | port = lport_priv(lport); | 652 | port = lport_priv(lport); |
@@ -677,20 +676,12 @@ static int fcoe_netdev_config(struct fc_lport *lport, struct net_device *netdev) | |||
677 | fcoe_link_speed_update(lport); | 676 | fcoe_link_speed_update(lport); |
678 | 677 | ||
679 | if (!lport->vport) { | 678 | if (!lport->vport) { |
680 | /* | ||
681 | * Use NAA 1&2 (FC-FS Rev. 2.0, Sec. 15) to generate WWNN/WWPN: | ||
682 | * For WWNN, we use NAA 1 w/ bit 27-16 of word 0 as 0. | ||
683 | * For WWPN, we use NAA 2 w/ bit 27-16 of word 0 from VLAN ID | ||
684 | */ | ||
685 | if (netdev->priv_flags & IFF_802_1Q_VLAN) | ||
686 | vid = vlan_dev_vlan_id(netdev); | ||
687 | |||
688 | if (fcoe_get_wwn(netdev, &wwnn, NETDEV_FCOE_WWNN)) | 679 | if (fcoe_get_wwn(netdev, &wwnn, NETDEV_FCOE_WWNN)) |
689 | wwnn = fcoe_wwn_from_mac(fcoe->ctlr.ctl_src_addr, 1, 0); | 680 | wwnn = fcoe_wwn_from_mac(fcoe->ctlr.ctl_src_addr, 1, 0); |
690 | fc_set_wwnn(lport, wwnn); | 681 | fc_set_wwnn(lport, wwnn); |
691 | if (fcoe_get_wwn(netdev, &wwpn, NETDEV_FCOE_WWPN)) | 682 | if (fcoe_get_wwn(netdev, &wwpn, NETDEV_FCOE_WWPN)) |
692 | wwpn = fcoe_wwn_from_mac(fcoe->ctlr.ctl_src_addr, | 683 | wwpn = fcoe_wwn_from_mac(fcoe->ctlr.ctl_src_addr, |
693 | 2, vid); | 684 | 2, 0); |
694 | fc_set_wwpn(lport, wwpn); | 685 | fc_set_wwpn(lport, wwpn); |
695 | } | 686 | } |
696 | 687 | ||