[Stgt-devel] [PATCH 10/20] iser connection transport data
    Pete Wyckoff 
    pw
       
    Wed Nov 14 20:05:03 CET 2007
    
    
  
tomof at acm.org wrote on Wed, 14 Nov 2007 23:33 +0900:
> Moving connection allocation to transport would work better for iSER?
> 
> This patch is against the lastest git tree
> (09532079071d65ec62dae982324ab07a895e16f1).
[..]
> -struct tcp_conn_info {
> +struct iscsi_tcp_connection {
>  	int fd;
> +
> +	struct iscsi_connection iscsi_conn;
>  };
Sure, good idea.  It will create a bit of churn against the later
patches but easily fixed.  Note that iser adds a task->trans_data
too; you could treat it similarly.
Also note the ep_conn_init() we need.  It is called _after_ login
negotiation, not at allocation time, unlike your new conn_init().
May want to change the name so people aren't confused about this.
ep_task_init() is okay; it is called at task allocation time.
		-- Pete
    
    
More information about the stgt
mailing list