[stgt] [PATCH RFC] new iser code

Pete Wyckoff pw at padd.com
Tue Jul 13 03:06:23 CEST 2010


alexandern at Voltaire.COM wrote on Mon, 12 Jul 2010 17:38 +0300:
> Pete Wyckoff wrote:
> > 
> > Why did you duplicate so many shared iscsi functions?
> > iser_scsi_cmd_done etc. should not know about the particulars of the
> > transport.  You got rid of the whole transport abstraction?
> 
> There are few reasons. First, my initial motivation was to escape 
> the api of iscsi sub-transports, because it did not suit well for iser. 
> Then, the login and text code was not generic and became unusable
> outside the above-mentioned api.
> So i circumvented iscsi transport abstraction for iser, and duplicated
> some other code, but if we formulate a new transport api 
> suitable for both iscsi/tcp and iser then it can be brought back.
> Actually this is one possibility for a new design, and i'd be happy 
> to discuss it.
> 
> Perhaps you missed my explanations, posted previously on the list:
> http://lists.wpkg.org/pipermail/stgt/2010-June/003842.html

Thanks for the pointer.  I think I fell off the stgt list.

All your reasons do make sense to me too.  It is a shame that the
code can't be more shared, but I understand why you want to decouple
some of it.  I'll try to take a look at your new event processing
loop some time.

		-- Pete
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html



More information about the stgt mailing list