public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer
@ 2009-02-25  1:28 Karen Xie
  2009-02-25 16:30 ` Mike Christie
  0 siblings, 1 reply; 3+ messages in thread
From: Karen Xie @ 2009-02-25  1:28 UTC (permalink / raw)
  To: open-iscsi, linux-scsi; +Cc: michaelc, James.Bottomley, kxie

[PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer

From: Karen Xie <kxie@chelsio.com>

change iscsi_session_failure() to take a pointer to iscsi_session instead of iscsi_cls_session.

Signed-off-by: Mike Christie<michaelc@cs.wisc.edu>
Signed-off-by: Karen Xie <kxie@chelsio.com>
---

 drivers/scsi/libiscsi.c |    5 ++---
 include/scsi/libiscsi.h |    2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)


diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c
index 809d32d..5ad87e8 100644
--- a/drivers/scsi/libiscsi.c
+++ b/drivers/scsi/libiscsi.c
@@ -1034,10 +1034,9 @@ struct iscsi_task *iscsi_itt_to_ctask(struct iscsi_conn *conn, itt_t itt)
 }
 EXPORT_SYMBOL_GPL(iscsi_itt_to_ctask);
 
-void iscsi_session_failure(struct iscsi_cls_session *cls_session,
+void iscsi_session_failure(struct iscsi_session *session,
 			   enum iscsi_err err)
 {
-	struct iscsi_session *session = cls_session->dd_data;
 	struct iscsi_conn *conn;
 	struct device *dev;
 	unsigned long flags;
@@ -2043,7 +2042,7 @@ EXPORT_SYMBOL_GPL(iscsi_host_alloc);
 
 static void iscsi_notify_host_removed(struct iscsi_cls_session *cls_session)
 {
-	iscsi_session_failure(cls_session, ISCSI_ERR_INVALID_HOST);
+	iscsi_session_failure(cls_session->dd_data, ISCSI_ERR_INVALID_HOST);
 }
 
 /**
diff --git a/include/scsi/libiscsi.h b/include/scsi/libiscsi.h
index 7360e19..07c3cd3 100644
--- a/include/scsi/libiscsi.h
+++ b/include/scsi/libiscsi.h
@@ -382,7 +382,7 @@ extern void iscsi_conn_stop(struct iscsi_cls_conn *, int);
 extern int iscsi_conn_bind(struct iscsi_cls_session *, struct iscsi_cls_conn *,
 			   int);
 extern void iscsi_conn_failure(struct iscsi_conn *conn, enum iscsi_err err);
-extern void iscsi_session_failure(struct iscsi_cls_session *cls_session,
+extern void iscsi_session_failure(struct iscsi_session *session,
 				  enum iscsi_err err);
 extern int iscsi_conn_get_param(struct iscsi_cls_conn *cls_conn,
 				enum iscsi_param param, char *buf);

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer
  2009-02-25  1:28 [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer Karen Xie
@ 2009-02-25 16:30 ` Mike Christie
  2009-02-25 17:05   ` James Bottomley
  0 siblings, 1 reply; 3+ messages in thread
From: Mike Christie @ 2009-02-25 16:30 UTC (permalink / raw)
  To: Karen Xie; +Cc: open-iscsi, linux-scsi, James.Bottomley

Karen Xie wrote:
> [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer
> 
> From: Karen Xie <kxie@chelsio.com>
> 

I guess the From: got messed up. I am not sure if James cares or not.

James, I actually wrote the patch for Karen, and she has been using and 
testing it out with her patches so that is why/how my signed off is 
first but the "From:" above has her.


> change iscsi_session_failure() to take a pointer to iscsi_session instead of iscsi_cls_session.
> 
> Signed-off-by: Mike Christie<michaelc@cs.wisc.edu>
> Signed-off-by: Karen Xie <kxie@chelsio.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer
  2009-02-25 16:30 ` Mike Christie
@ 2009-02-25 17:05   ` James Bottomley
  0 siblings, 0 replies; 3+ messages in thread
From: James Bottomley @ 2009-02-25 17:05 UTC (permalink / raw)
  To: Mike Christie; +Cc: Karen Xie, open-iscsi, linux-scsi

On Wed, 2009-02-25 at 10:30 -0600, Mike Christie wrote:
> Karen Xie wrote:
> > [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer
> > 
> > From: Karen Xie <kxie@chelsio.com>
> > 
> 
> I guess the From: got messed up. I am not sure if James cares or not.

I'll fix it, don't worry.

James



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-02-25 17:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-25  1:28 [PATCH 1/2 2.6.29-rc] libiscsi - change iscsi_session_failure() to take iscsi_session pointer Karen Xie
2009-02-25 16:30 ` Mike Christie
2009-02-25 17:05   ` James Bottomley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox