Re: [RESEND PATCH] BNX2I - Bug fixes related to MTU change issue when there are active iscsi sessions

[Mike Christie <michaelc@cs.wisc.edu>]

On 04/08/2010 05:59 PM, Anil Veerabhadrappa wrote:
> James,
>     Please apply this patch to fix resources leak issue when device MTU
> is changed or ethtool selftest is executed while there are active iscsi
> sessions.
>
> Thanks,
> Anil
>
>
>      * bnx2i driver has to wait and cleanup all iscsi endpoints before
>        returning from bnx2i_stop(). This is to make sure all chip
>        resources are freed before chip is reset.
>      * As the requirements for 1G and 10G chipsets is different, added
>        per-device 'hba_shutdown_tmo' parameter to adapter structure
>      * If the connections are not torn down by the daemon within this
>        timeout period, 'cid's will be leaked in 10G device. 1G devices
>        are more flexible and do not leak any resources because the whole
>        chip ports gets reset when MTU is changed or ethtool selftest is run
>      * fixed a minor issue in bnx2i_ep_poll() which unnecessarily forced
>        error return code when driver timed out waiting for TCP connect
>        request to complete
>
> Signed-off-by: Anil Veerabhadrappa<anilgv@broadcom.com>


Looks ok now.

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