[PATCH v2 0/3] cxgbi iscsi driver

[PATCH v2 0/3] cxgbi iscsi driver

[kxie-ut6Up61K2wZBDgjK7y7TUQ]

[PATCH v2 0/3] cxgbi iscsi driver

From: Karen Xie <kxie-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>

This patchset includes:

1. added libcxgbi as the common library for both cxgb4i and cxgb3i driver
2. added cxgb4i iscsi initiator v5.
3. changed cxgb3i to use libcxgbi.

The version 2 includes the following changes:
- Made dbg_level static to avoid compiling errors when both cxgb3i and cxgb4i
  are built into the kernel.
- Deleted cxgbi_fill_iscsi_transport() and cxgbi_fill_scsi_host_template().
  The iscsi_transport and scsi_host_template are initialized in the drivers
  instead of being filled in by the library at runtime..
- Only register with iscsi class transport once per driver type. 

Thanks.
Karen

-- 
You received this message because you are subscribed to the Google Groups "open-iscsi" group.
To post to this group, send email to open-iscsi-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
To unsubscribe from this group, send email to open-iscsi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.

[PATCH v2 0/3] cxgbi iscsi driver

[kxie]

[PATCH v2 0/3] cxgbi iscsi driver

From: Karen Xie <kxie@chelsio.com>

Re-submission, missed netdev list last time.

This patchset includes:

1. added libcxgbi as the common library for both cxgb4i and cxgb3i driver
2. added cxgb4i iscsi initiator v5.
3. changed cxgb3i to use libcxgbi.

The version 2 includes the following changes:
- Made dbg_level static to avoid compiling errors when both cxgb3i and cxgb4i
  are built into the kernel.
- Deleted cxgbi_fill_iscsi_transport() and cxgbi_fill_scsi_host_template().
  The iscsi_transport and scsi_host_template are initialized in the drivers
  instead of being filled in by the library at runtime..
- Only register with iscsi class transport once per driver type. 

Thanks.
Karen

Re: [PATCH v2 0/3] cxgbi iscsi driver

[Joe Perches]

On Fri, 2010-08-13 at 08:19 -0700, kxie@chelsio.com wrote:
> [PATCH v2 0/3] cxgbi iscsi driver
> From: Karen Xie <kxie@chelsio.com>
> Re-submission, missed netdev list last time.

Hi Karen.

I presume this is not to be applied as you've
already received some feedback on it and that
you are sending it to netdev for any additional
comments.

Re: [PATCH v2 0/3] cxgbi iscsi driver

[Mike Christie]

On 08/13/2010 10:19 AM, kxie@chelsio.com wrote:
> [PATCH v2 0/3] cxgbi iscsi driver
>
> From: Karen Xie<kxie@chelsio.com>
>
> Re-submission, missed netdev list last time.
>
> This patchset includes:
>
> 1. added libcxgbi as the common library for both cxgb4i and cxgb3i driver
> 2. added cxgb4i iscsi initiator v5.
> 3. changed cxgb3i to use libcxgbi.
>
> The version 2 includes the following changes:
> - Made dbg_level static to avoid compiling errors when both cxgb3i and cxgb4i
>    are built into the kernel.
> - Deleted cxgbi_fill_iscsi_transport() and cxgbi_fill_scsi_host_template().
>    The iscsi_transport and scsi_host_template are initialized in the drivers
>    instead of being filled in by the library at runtime..
> - Only register with iscsi class transport once per driver type.
>

iSCSI and SCSI parts look ok now. The network code should be reviewed by 
the netdev people, because I am not qualified to review that. Thanks for 
your work Karen.

Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>