[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