[ofw] [PATCH] ibal: set timeout and retry values for RMPP messages

Hal Rosenstock hal.rosenstock at gmail.com
Fri Jun 18 13:25:34 PDT 2010


On Fri, Jun 18, 2010 at 2:10 PM, Hefty, Sean <sean.hefty at intel.com> wrote:
> Define default timeout and retry values for RMPP response messages
> if none are provided by the user.  Also mark GETTABLE requests as
> needing RMPP.

GETTABLE does not need RMPP. It's only the response which does.

-- Hal

>
> From: Sean Hefty <sean.hefty at intel.com>
> ---
> diff --git a/trunk/core/al/al_mad.c b/trunk/core/al/al_mad.c
> index 4576fa8..51310f9 100644
> --- a/trunk/core/al/al_mad.c
> +++ b/trunk/core/al/al_mad.c
> @@ -61,6 +61,7 @@
>
>  #define AL_RMPP_WINDOW                 16                              /* Max size of RMPP window */
>  #define AL_REASSEMBLY_TIMEOUT  5000                    /* 5 seconds */
> +#define AL_RMPP_RETRIES                        5
>
>  static void
>  __cleanup_mad_disp(
> @@ -1452,6 +1453,12 @@ __init_send_mad(
>                        return IB_INVALID_SETTING;
>                }
>
> +               if( !p_mad_element->timeout_ms )
> +                       p_mad_element->timeout_ms = AL_REASSEMBLY_TIMEOUT;
> +
> +               if( !h_send->retry_cnt )
> +                       h_send->retry_cnt = AL_RMPP_RETRIES;
> +
>                p_rmpp_hdr->rmpp_version = rmpp_version;
>                p_rmpp_hdr->rmpp_type = IB_RMPP_TYPE_DATA;
>                ib_rmpp_set_resp_time( p_rmpp_hdr, IB_RMPP_NO_RESP_TIME );
> @@ -1570,6 +1577,7 @@ __does_send_req_rmpp(
>                {
>                        switch( p_mad_element->p_mad_buf->method )
>                        {
> +                       case IB_MAD_METHOD_GETTABLE:
>                        case IB_MAD_METHOD_GETTABLE_RESP:
>                        case IB_MAD_METHOD_GETMULTI:
>                        case IB_MAD_METHOD_GETMULTI_RESP:
>
> _______________________________________________
> ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
>


More information about the ofw mailing list