8eaf7dfcfc
We need to initialize qedf->fipvlan_compl in __qedf_probe so that if we receive an unsolicited FIP VLAN response, the system doesn't crash due to trying to complete an uninitialized completion. Also add a check to see if there are any waiters on the completion so we don't inadvertantly kick start the discovery process due to the unsolicited frame. Fixed the crash: <1>BUG: unable to handle kernel NULL pointer dereference at (null) <1>IP: [<ffffffff8105ed71>] __wake_up_common+0x31/0x90 <4>PGD 0 <4>Oops: 0000 [#1] SMP <4>last sysfs file: /sys/devices/system/cpu/online <4>CPU 7 <4>Modules linked in: autofs4 nfs lockd fscache auth_rpcgss nfs_acl sunrpc target_core_iblock target_core_file target_core_pscsi target_core_mod configfs bnx2fc cnic fcoe 8021q garp stp llc ipt_REJECT nf_conntrack_ipv4 nf_defrag_ipv4 iptable_filter ip_tables ip6t_REJECT nf_conntrack_ipv6 nf_defrag_ipv6 xt_state nf_conntrack ip6table_filter ip6_tables ipv6 vfat fat uinput ipmi_devintf microcode power_meter acpi_ipmi ipmi_si ipmi_msghandler iTCO_wdt iTCO_vendor_support dcdbas sg joydev sb_edac edac_core lpc_ich mfd_core shpchp tg3 ptp pps_core ext4 jbd2 mbcache sr_mod cdrom sd_mod crc_t10dif qedi(U) iscsi_boot_sysfs libiscsi scsi_transport_iscsi uio qedf(U) libfcoe libfc scsi_transport_fc scsi_tgt qede(U) qed(U) ahci megaraid_sas wmi dm_mirror dm_region_hash dm_log dm_mod [last unloaded: speedstep_lib] <4> <4>Pid: 1485, comm: qedf_11_ll2 Not tainted 2.6.32-642.el6.x86_64 #1 Dell Inc. PowerEdge R730/0599V5 <4>RIP: 0010:[<ffffffff8105ed71>] [<ffffffff8105ed71>] __wake_up_common+0x31/0x90 <4>RSP: 0018:ffff881068a83d50 EFLAGS: 00010086 <4>RAX: ffffffffffffffe8 RBX: ffff88106bf42de0 RCX: 0000000000000000 <4>RDX: 0000000000000000 RSI: 0000000000000003 RDI: ffff88106bf42de0 <4>RBP: ffff881068a83d90 R08: 0000000000000000 R09: 00000000fffffffe <4>R10: 0000000000000000 R11: 000000000000000b R12: 0000000000000286 <4>R13: ffff88106bf42de8 R14: 0000000000000000 R15: 0000000000000000 <4>FS: 0000000000000000(0000) GS:ffff88089c460000(0000) knlGS:0000000000000000 <4>CS: 0010 DS: 0018 ES: 0018 CR0: 000000008005003b <4>CR2: 0000000000000000 CR3: 0000000001a8d000 CR4: 00000000001407e0 <4>DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 <4>DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400 <4>Process qedf_11_ll2 (pid: 1485, threadinfo ffff881068a80000, task ffff881068a70040) <4>Stack: <4> ffff88106ef00090 0000000300000001 ffff881068a83d90 ffff88106bf42de0 <4><d> 0000000000000286 ffff88106bf42dd8 ffff88106bf40a50 0000000000000002 <4><d> ffff881068a83dc0 ffffffff810634c7 ffff881000000003 000000000000000b <4>Call Trace: <4> [<ffffffff810634c7>] complete+0x47/0x60 <4> [<ffffffffa01d37e7>] qedf_fip_recv+0x1c7/0x450 [qedf] <4> [<ffffffffa01cb3cb>] qedf_ll2_recv_thread+0x33b/0x510 [qedf] <4> [<ffffffffa01cb090>] ? qedf_ll2_recv_thread+0x0/0x510 [qedf] <4> [<ffffffff810a662e>] kthread+0x9e/0xc0 <4> [<ffffffff8100c28a>] child_rip+0xa/0x20 <4> [<ffffffff810a6590>] ? kthread+0x0/0xc0 <4> [<ffffffff8100c280>] ? child_rip+0x0/0x20 <4>Code: 41 56 41 55 41 54 53 48 83 ec 18 0f 1f 44 00 00 89 75 cc 89 55 c8 4c 8d 6f 08 48 8b 57 08 41 89 cf 4d 89 c6 48 8d 42 e8 49 39 d5 <48> 8b 58 18 74 3f 48 83 eb 18 eb 0a 0f 1f 00 48 89 d8 48 8d 5a <1>RIP [<ffffffff8105ed71>] __wake_up_common+0x31/0x90 <4> RSP <ffff881068a83d50> <4>CR2: 0000000000000000 Signed-off-by: Chad Dupuis <chad.dupuis@cavium.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
271 lines
7.6 KiB
C
271 lines
7.6 KiB
C
/*
|
|
* QLogic FCoE Offload Driver
|
|
* Copyright (c) 2016 Cavium Inc.
|
|
*
|
|
* This software is available under the terms of the GNU General Public License
|
|
* (GPL) Version 2, available from the file COPYING in the main directory of
|
|
* this source tree.
|
|
*/
|
|
#include <linux/if_ether.h>
|
|
#include <linux/if_vlan.h>
|
|
#include "qedf.h"
|
|
|
|
extern const struct qed_fcoe_ops *qed_ops;
|
|
/*
|
|
* FIP VLAN functions that will eventually move to libfcoe.
|
|
*/
|
|
|
|
void qedf_fcoe_send_vlan_req(struct qedf_ctx *qedf)
|
|
{
|
|
struct sk_buff *skb;
|
|
char *eth_fr;
|
|
int fr_len;
|
|
struct fip_vlan *vlan;
|
|
#define MY_FIP_ALL_FCF_MACS ((__u8[6]) { 1, 0x10, 0x18, 1, 0, 2 })
|
|
static u8 my_fcoe_all_fcfs[ETH_ALEN] = MY_FIP_ALL_FCF_MACS;
|
|
|
|
skb = dev_alloc_skb(sizeof(struct fip_vlan));
|
|
if (!skb)
|
|
return;
|
|
|
|
fr_len = sizeof(*vlan);
|
|
eth_fr = (char *)skb->data;
|
|
vlan = (struct fip_vlan *)eth_fr;
|
|
|
|
memset(vlan, 0, sizeof(*vlan));
|
|
ether_addr_copy(vlan->eth.h_source, qedf->mac);
|
|
ether_addr_copy(vlan->eth.h_dest, my_fcoe_all_fcfs);
|
|
vlan->eth.h_proto = htons(ETH_P_FIP);
|
|
|
|
vlan->fip.fip_ver = FIP_VER_ENCAPS(FIP_VER);
|
|
vlan->fip.fip_op = htons(FIP_OP_VLAN);
|
|
vlan->fip.fip_subcode = FIP_SC_VL_REQ;
|
|
vlan->fip.fip_dl_len = htons(sizeof(vlan->desc) / FIP_BPW);
|
|
|
|
vlan->desc.mac.fd_desc.fip_dtype = FIP_DT_MAC;
|
|
vlan->desc.mac.fd_desc.fip_dlen = sizeof(vlan->desc.mac) / FIP_BPW;
|
|
ether_addr_copy(vlan->desc.mac.fd_mac, qedf->mac);
|
|
|
|
vlan->desc.wwnn.fd_desc.fip_dtype = FIP_DT_NAME;
|
|
vlan->desc.wwnn.fd_desc.fip_dlen = sizeof(vlan->desc.wwnn) / FIP_BPW;
|
|
put_unaligned_be64(qedf->lport->wwnn, &vlan->desc.wwnn.fd_wwn);
|
|
|
|
skb_put(skb, sizeof(*vlan));
|
|
skb->protocol = htons(ETH_P_FIP);
|
|
skb_reset_mac_header(skb);
|
|
skb_reset_network_header(skb);
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, "Sending FIP VLAN "
|
|
"request.");
|
|
|
|
if (atomic_read(&qedf->link_state) != QEDF_LINK_UP) {
|
|
QEDF_WARN(&(qedf->dbg_ctx), "Cannot send vlan request "
|
|
"because link is not up.\n");
|
|
|
|
kfree_skb(skb);
|
|
return;
|
|
}
|
|
qed_ops->ll2->start_xmit(qedf->cdev, skb);
|
|
}
|
|
|
|
static void qedf_fcoe_process_vlan_resp(struct qedf_ctx *qedf,
|
|
struct sk_buff *skb)
|
|
{
|
|
struct fip_header *fiph;
|
|
struct fip_desc *desc;
|
|
u16 vid = 0;
|
|
ssize_t rlen;
|
|
size_t dlen;
|
|
|
|
fiph = (struct fip_header *)(((void *)skb->data) + 2 * ETH_ALEN + 2);
|
|
|
|
rlen = ntohs(fiph->fip_dl_len) * 4;
|
|
desc = (struct fip_desc *)(fiph + 1);
|
|
while (rlen > 0) {
|
|
dlen = desc->fip_dlen * FIP_BPW;
|
|
switch (desc->fip_dtype) {
|
|
case FIP_DT_VLAN:
|
|
vid = ntohs(((struct fip_vlan_desc *)desc)->fd_vlan);
|
|
break;
|
|
}
|
|
desc = (struct fip_desc *)((char *)desc + dlen);
|
|
rlen -= dlen;
|
|
}
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, "VLAN response, "
|
|
"vid=0x%x.\n", vid);
|
|
|
|
if (vid > 0 && qedf->vlan_id != vid) {
|
|
qedf_set_vlan_id(qedf, vid);
|
|
|
|
/* Inform waiter that it's ok to call fcoe_ctlr_link up() */
|
|
if (!completion_done(&qedf->fipvlan_compl))
|
|
complete(&qedf->fipvlan_compl);
|
|
}
|
|
}
|
|
|
|
void qedf_fip_send(struct fcoe_ctlr *fip, struct sk_buff *skb)
|
|
{
|
|
struct qedf_ctx *qedf = container_of(fip, struct qedf_ctx, ctlr);
|
|
struct ethhdr *eth_hdr;
|
|
struct vlan_ethhdr *vlan_hdr;
|
|
struct fip_header *fiph;
|
|
u16 op, vlan_tci = 0;
|
|
u8 sub;
|
|
|
|
if (!test_bit(QEDF_LL2_STARTED, &qedf->flags)) {
|
|
QEDF_WARN(&(qedf->dbg_ctx), "LL2 not started\n");
|
|
kfree_skb(skb);
|
|
return;
|
|
}
|
|
|
|
fiph = (struct fip_header *) ((void *)skb->data + 2 * ETH_ALEN + 2);
|
|
eth_hdr = (struct ethhdr *)skb_mac_header(skb);
|
|
op = ntohs(fiph->fip_op);
|
|
sub = fiph->fip_subcode;
|
|
|
|
if (!qedf->vlan_hw_insert) {
|
|
vlan_hdr = (struct vlan_ethhdr *)skb_push(skb, sizeof(*vlan_hdr)
|
|
- sizeof(*eth_hdr));
|
|
memcpy(vlan_hdr, eth_hdr, 2 * ETH_ALEN);
|
|
vlan_hdr->h_vlan_proto = htons(ETH_P_8021Q);
|
|
vlan_hdr->h_vlan_encapsulated_proto = eth_hdr->h_proto;
|
|
vlan_hdr->h_vlan_TCI = vlan_tci = htons(qedf->vlan_id);
|
|
}
|
|
|
|
/* Update eth_hdr since we added a VLAN tag */
|
|
eth_hdr = (struct ethhdr *)skb_mac_header(skb);
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2, "FIP frame send: "
|
|
"dest=%pM op=%x sub=%x vlan=%04x.", eth_hdr->h_dest, op, sub,
|
|
ntohs(vlan_tci));
|
|
if (qedf_dump_frames)
|
|
print_hex_dump(KERN_WARNING, "fip ", DUMP_PREFIX_OFFSET, 16, 1,
|
|
skb->data, skb->len, false);
|
|
|
|
qed_ops->ll2->start_xmit(qedf->cdev, skb);
|
|
}
|
|
|
|
/* Process incoming FIP frames. */
|
|
void qedf_fip_recv(struct qedf_ctx *qedf, struct sk_buff *skb)
|
|
{
|
|
struct ethhdr *eth_hdr;
|
|
struct fip_header *fiph;
|
|
struct fip_desc *desc;
|
|
struct fip_mac_desc *mp;
|
|
struct fip_wwn_desc *wp;
|
|
struct fip_vn_desc *vp;
|
|
size_t rlen, dlen;
|
|
uint32_t cvl_port_id;
|
|
__u8 cvl_mac[ETH_ALEN];
|
|
u16 op;
|
|
u8 sub;
|
|
|
|
eth_hdr = (struct ethhdr *)skb_mac_header(skb);
|
|
fiph = (struct fip_header *) ((void *)skb->data + 2 * ETH_ALEN + 2);
|
|
op = ntohs(fiph->fip_op);
|
|
sub = fiph->fip_subcode;
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2, "FIP frame received: "
|
|
"skb=%p fiph=%p source=%pM op=%x sub=%x", skb, fiph,
|
|
eth_hdr->h_source, op, sub);
|
|
if (qedf_dump_frames)
|
|
print_hex_dump(KERN_WARNING, "fip ", DUMP_PREFIX_OFFSET, 16, 1,
|
|
skb->data, skb->len, false);
|
|
|
|
/* Handle FIP VLAN resp in the driver */
|
|
if (op == FIP_OP_VLAN && sub == FIP_SC_VL_NOTE) {
|
|
qedf_fcoe_process_vlan_resp(qedf, skb);
|
|
qedf->vlan_hw_insert = 0;
|
|
kfree_skb(skb);
|
|
} else if (op == FIP_OP_CTRL && sub == FIP_SC_CLR_VLINK) {
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, "Clear virtual "
|
|
"link received.\n");
|
|
|
|
/* Check that an FCF has been selected by fcoe */
|
|
if (qedf->ctlr.sel_fcf == NULL) {
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC,
|
|
"Dropping CVL since FCF has not been selected "
|
|
"yet.");
|
|
return;
|
|
}
|
|
|
|
cvl_port_id = 0;
|
|
memset(cvl_mac, 0, ETH_ALEN);
|
|
/*
|
|
* We need to loop through the CVL descriptors to determine
|
|
* if we want to reset the fcoe link
|
|
*/
|
|
rlen = ntohs(fiph->fip_dl_len) * FIP_BPW;
|
|
desc = (struct fip_desc *)(fiph + 1);
|
|
while (rlen >= sizeof(*desc)) {
|
|
dlen = desc->fip_dlen * FIP_BPW;
|
|
switch (desc->fip_dtype) {
|
|
case FIP_DT_MAC:
|
|
mp = (struct fip_mac_desc *)desc;
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2,
|
|
"fd_mac=%pM\n", mp->fd_mac);
|
|
ether_addr_copy(cvl_mac, mp->fd_mac);
|
|
break;
|
|
case FIP_DT_NAME:
|
|
wp = (struct fip_wwn_desc *)desc;
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2,
|
|
"fc_wwpn=%016llx.\n",
|
|
get_unaligned_be64(&wp->fd_wwn));
|
|
break;
|
|
case FIP_DT_VN_ID:
|
|
vp = (struct fip_vn_desc *)desc;
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2,
|
|
"fd_fc_id=%x.\n", ntoh24(vp->fd_fc_id));
|
|
cvl_port_id = ntoh24(vp->fd_fc_id);
|
|
break;
|
|
default:
|
|
/* Ignore anything else */
|
|
break;
|
|
}
|
|
desc = (struct fip_desc *)((char *)desc + dlen);
|
|
rlen -= dlen;
|
|
}
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_LL2,
|
|
"cvl_port_id=%06x cvl_mac=%pM.\n", cvl_port_id,
|
|
cvl_mac);
|
|
if (cvl_port_id == qedf->lport->port_id &&
|
|
ether_addr_equal(cvl_mac,
|
|
qedf->ctlr.sel_fcf->fcf_mac)) {
|
|
fcoe_ctlr_link_down(&qedf->ctlr);
|
|
qedf_wait_for_upload(qedf);
|
|
fcoe_ctlr_link_up(&qedf->ctlr);
|
|
}
|
|
kfree_skb(skb);
|
|
} else {
|
|
/* Everything else is handled by libfcoe */
|
|
__skb_pull(skb, ETH_HLEN);
|
|
fcoe_ctlr_recv(&qedf->ctlr, skb);
|
|
}
|
|
}
|
|
|
|
void qedf_update_src_mac(struct fc_lport *lport, u8 *addr)
|
|
{
|
|
struct qedf_ctx *qedf = lport_priv(lport);
|
|
|
|
QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC,
|
|
"Setting data_src_addr=%pM.\n", addr);
|
|
ether_addr_copy(qedf->data_src_addr, addr);
|
|
}
|
|
|
|
u8 *qedf_get_src_mac(struct fc_lport *lport)
|
|
{
|
|
u8 mac[ETH_ALEN];
|
|
u8 port_id[3];
|
|
struct qedf_ctx *qedf = lport_priv(lport);
|
|
|
|
/* We need to use the lport port_id to create the data_src_addr */
|
|
if (is_zero_ether_addr(qedf->data_src_addr)) {
|
|
hton24(port_id, lport->port_id);
|
|
fc_fcoe_set_mac(mac, port_id);
|
|
qedf->ctlr.update_mac(lport, mac);
|
|
}
|
|
return qedf->data_src_addr;
|
|
}
|