Sorry for the virt_to_bus screw up.
On 09/08/2009 03:40 AM, Jayamohan Kallickal wrote:
+struct iscsi_cls_session *beiscsi_session_create(struct iscsi_endpoint *ep,
+ u16 cmds_max,
+ u16 qdepth,
+ u32 initial_cmdsn)
+{
+ struct Scsi_Host *shost;
+ if (!cls_session)
+ return NULL;
+ sess = cls_session->dd_data;
+ max_size = ALIGN(sizeof(struct be_cmd_bhs), 64) * cmds_max;
Instead of cmds_max you should use session->cmds_max like you do below.
+ task = sess->cmds[0];
+ io_task = task->dd_data;
The task and io_task assignment as not needed.
+ vaddr = pci_alloc_consistent(phba->pcidev,
+ max_size,
+ &bus_add);
+ pa_addr = (__u64) bus_add;
+
+ for (num_cmd = 0; num_cmd< sess->cmds_max; num_cmd++) {
+ task = sess->cmds[num_cmd];
+ io_task = task->dd_data;
+ io_task->cmd_bhs = vaddr;
+ io_task->bhs_pa.u.a64.address = pa_addr;
+ io_task->alloc_size = max_size;
+ vaddr += ALIGN(sizeof(struct be_cmd_bhs), 64);
+ pa_addr += ALIGN(sizeof(struct be_cmd_bhs), 64);
+ }
+ return cls_session;
+}
+
+/**
+ * beiscsi_session_destroy - destroys iscsi session
+ * @cls_session: pointer to iscsi cls session
+ *
+ * Destroys iSCSI session instance and releases
+ * resources allocated for it.
+ */
+void beiscsi_session_destroy(struct iscsi_cls_session *cls_session)
+{
+ struct iscsi_task *task;
+ struct beiscsi_io_task *io_task;
+ struct iscsi_session *sess = cls_session->dd_data;
+ struct Scsi_Host *shost = iscsi_session_to_shost(cls_session);
+ struct beiscsi_hba *phba = iscsi_host_priv(shost);
+
+ task = sess->cmds[0];
+ io_task = task->dd_data;
+ pci_free_consistent(phba->pcidev,
+ io_task->alloc_size,
+ io_task->cmd_bhs,
+ io_task->bhs_pa.u.a64.address);
+ iscsi_session_teardown(cls_session);
+}
+
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
