Commit 46012e8b authored by Eddie Wai's avatar Eddie Wai Committed by James Bottomley

[SCSI] bnx2i: Created an active linklist which holds bnx2i endpoints

This introduces a new active linklist which would link up all active
bnx2i_endpoints.  This will be used by subsequent patches that
follows.
Signed-off-by: default avatarEddie Wai <eddie.wai@broadcom.com>
Reviewed-by: default avatarMichael Chan <mchan@broadcom.com>
Reviewed-by: default avatarBenjamin Li <benli@broadcom.com>
Acked-by: default avatarAnil Veerabhadrappa <anilgv@broadcom.com>
Reviewed-by: default avatarMike Christie <michaelc@cs.wisc.edu>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@suse.de>
parent 6447f286
...@@ -299,6 +299,7 @@ struct iscsi_cid_queue { ...@@ -299,6 +299,7 @@ struct iscsi_cid_queue {
* @cid_que: iscsi cid queue * @cid_que: iscsi cid queue
* @ep_rdwr_lock: read / write lock to synchronize various ep lists * @ep_rdwr_lock: read / write lock to synchronize various ep lists
* @ep_ofld_list: connection list for pending offload completion * @ep_ofld_list: connection list for pending offload completion
* @ep_active_list: connection list for active offload endpoints
* @ep_destroy_list: connection list for pending offload completion * @ep_destroy_list: connection list for pending offload completion
* @mp_bd_tbl: BD table to be used with middle path requests * @mp_bd_tbl: BD table to be used with middle path requests
* @mp_bd_dma: DMA address of 'mp_bd_tbl' memory buffer * @mp_bd_dma: DMA address of 'mp_bd_tbl' memory buffer
...@@ -369,6 +370,7 @@ struct bnx2i_hba { ...@@ -369,6 +370,7 @@ struct bnx2i_hba {
rwlock_t ep_rdwr_lock; rwlock_t ep_rdwr_lock;
struct list_head ep_ofld_list; struct list_head ep_ofld_list;
struct list_head ep_active_list;
struct list_head ep_destroy_list; struct list_head ep_destroy_list;
/* /*
...@@ -645,6 +647,7 @@ enum { ...@@ -645,6 +647,7 @@ enum {
* @link: list head to link elements * @link: list head to link elements
* @hba: adapter to which this connection belongs * @hba: adapter to which this connection belongs
* @conn: iscsi connection this EP is linked to * @conn: iscsi connection this EP is linked to
* @cls_ep: associated iSCSI endpoint pointer
* @sess: iscsi session this EP is linked to * @sess: iscsi session this EP is linked to
* @cm_sk: cnic sock struct * @cm_sk: cnic sock struct
* @hba_age: age to detect if 'iscsid' issues ep_disconnect() * @hba_age: age to detect if 'iscsid' issues ep_disconnect()
...@@ -664,6 +667,7 @@ struct bnx2i_endpoint { ...@@ -664,6 +667,7 @@ struct bnx2i_endpoint {
struct list_head link; struct list_head link;
struct bnx2i_hba *hba; struct bnx2i_hba *hba;
struct bnx2i_conn *conn; struct bnx2i_conn *conn;
struct iscsi_endpoint *cls_ep;
struct cnic_sock *cm_sk; struct cnic_sock *cm_sk;
u32 hba_age; u32 hba_age;
u32 state; u32 state;
......
...@@ -386,6 +386,7 @@ static struct iscsi_endpoint *bnx2i_alloc_ep(struct bnx2i_hba *hba) ...@@ -386,6 +386,7 @@ static struct iscsi_endpoint *bnx2i_alloc_ep(struct bnx2i_hba *hba)
} }
bnx2i_ep = ep->dd_data; bnx2i_ep = ep->dd_data;
bnx2i_ep->cls_ep = ep;
INIT_LIST_HEAD(&bnx2i_ep->link); INIT_LIST_HEAD(&bnx2i_ep->link);
bnx2i_ep->state = EP_STATE_IDLE; bnx2i_ep->state = EP_STATE_IDLE;
bnx2i_ep->ep_iscsi_cid = (u16) -1; bnx2i_ep->ep_iscsi_cid = (u16) -1;
...@@ -678,7 +679,6 @@ bnx2i_find_ep_in_ofld_list(struct bnx2i_hba *hba, u32 iscsi_cid) ...@@ -678,7 +679,6 @@ bnx2i_find_ep_in_ofld_list(struct bnx2i_hba *hba, u32 iscsi_cid)
return ep; return ep;
} }
/** /**
* bnx2i_find_ep_in_destroy_list - find iscsi_cid in destroy list * bnx2i_find_ep_in_destroy_list - find iscsi_cid in destroy list
* @hba: pointer to adapter instance * @hba: pointer to adapter instance
...@@ -708,6 +708,39 @@ bnx2i_find_ep_in_destroy_list(struct bnx2i_hba *hba, u32 iscsi_cid) ...@@ -708,6 +708,39 @@ bnx2i_find_ep_in_destroy_list(struct bnx2i_hba *hba, u32 iscsi_cid)
return ep; return ep;
} }
/**
* bnx2i_ep_active_list_add - add an entry to ep active list
* @hba: pointer to adapter instance
* @ep: pointer to endpoint (transport indentifier) structure
*
* current active conn queue manager
*/
static void bnx2i_ep_active_list_add(struct bnx2i_hba *hba,
struct bnx2i_endpoint *ep)
{
write_lock_bh(&hba->ep_rdwr_lock);
list_add_tail(&ep->link, &hba->ep_active_list);
write_unlock_bh(&hba->ep_rdwr_lock);
}
/**
* bnx2i_ep_active_list_del - deletes an entry to ep active list
* @hba: pointer to adapter instance
* @ep: pointer to endpoint (transport indentifier) structure
*
* current active conn queue manager
*/
static void bnx2i_ep_active_list_del(struct bnx2i_hba *hba,
struct bnx2i_endpoint *ep)
{
write_lock_bh(&hba->ep_rdwr_lock);
list_del_init(&ep->link);
write_unlock_bh(&hba->ep_rdwr_lock);
}
/** /**
* bnx2i_setup_host_queue_size - assigns shost->can_queue param * bnx2i_setup_host_queue_size - assigns shost->can_queue param
* @hba: pointer to adapter instance * @hba: pointer to adapter instance
...@@ -784,6 +817,7 @@ struct bnx2i_hba *bnx2i_alloc_hba(struct cnic_dev *cnic) ...@@ -784,6 +817,7 @@ struct bnx2i_hba *bnx2i_alloc_hba(struct cnic_dev *cnic)
goto mp_bdt_mem_err; goto mp_bdt_mem_err;
INIT_LIST_HEAD(&hba->ep_ofld_list); INIT_LIST_HEAD(&hba->ep_ofld_list);
INIT_LIST_HEAD(&hba->ep_active_list);
INIT_LIST_HEAD(&hba->ep_destroy_list); INIT_LIST_HEAD(&hba->ep_destroy_list);
rwlock_init(&hba->ep_rdwr_lock); rwlock_init(&hba->ep_rdwr_lock);
...@@ -857,6 +891,7 @@ void bnx2i_free_hba(struct bnx2i_hba *hba) ...@@ -857,6 +891,7 @@ void bnx2i_free_hba(struct bnx2i_hba *hba)
iscsi_host_remove(shost); iscsi_host_remove(shost);
INIT_LIST_HEAD(&hba->ep_ofld_list); INIT_LIST_HEAD(&hba->ep_ofld_list);
INIT_LIST_HEAD(&hba->ep_active_list);
INIT_LIST_HEAD(&hba->ep_destroy_list); INIT_LIST_HEAD(&hba->ep_destroy_list);
pci_dev_put(hba->pcidev); pci_dev_put(hba->pcidev);
...@@ -1754,15 +1789,19 @@ static struct iscsi_endpoint *bnx2i_ep_connect(struct Scsi_Host *shost, ...@@ -1754,15 +1789,19 @@ static struct iscsi_endpoint *bnx2i_ep_connect(struct Scsi_Host *shost,
goto conn_failed; goto conn_failed;
} else } else
rc = cnic->cm_connect(bnx2i_ep->cm_sk, &saddr); rc = cnic->cm_connect(bnx2i_ep->cm_sk, &saddr);
if (rc) if (rc)
goto release_ep; goto release_ep;
bnx2i_ep_active_list_add(hba, bnx2i_ep);
if (bnx2i_map_ep_dbell_regs(bnx2i_ep)) if (bnx2i_map_ep_dbell_regs(bnx2i_ep))
goto release_ep; goto del_active_ep;
mutex_unlock(&hba->net_dev_lock); mutex_unlock(&hba->net_dev_lock);
return ep; return ep;
del_active_ep:
bnx2i_ep_active_list_del(hba, bnx2i_ep);
release_ep: release_ep:
if (bnx2i_tear_down_conn(hba, bnx2i_ep)) { if (bnx2i_tear_down_conn(hba, bnx2i_ep)) {
mutex_unlock(&hba->net_dev_lock); mutex_unlock(&hba->net_dev_lock);
...@@ -1931,6 +1970,7 @@ int bnx2i_hw_ep_disconnect(struct bnx2i_endpoint *bnx2i_ep) ...@@ -1931,6 +1970,7 @@ int bnx2i_hw_ep_disconnect(struct bnx2i_endpoint *bnx2i_ep)
del_timer_sync(&bnx2i_ep->ofld_timer); del_timer_sync(&bnx2i_ep->ofld_timer);
destroy_conn: destroy_conn:
bnx2i_ep_active_list_del(hba, bnx2i_ep);
if (bnx2i_tear_down_conn(hba, bnx2i_ep)) if (bnx2i_tear_down_conn(hba, bnx2i_ep))
ret = -EINVAL; ret = -EINVAL;
out: out:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment