[ofa-general] Re: [PATCH] opensm/osm_mcast_tbl.c: osm_mcast_tbl_get_block returns boolean

Sasha Khapyorsky sashak at voltaire.com
Wed Aug 12 02:03:29 PDT 2009


On 09:43 Fri 07 Aug     , Hal Rosenstock wrote:
> 
> so use TRUE/FALSE rather than IB_INVALID_PARMETER
> 
> Signed-off-by: Hal Rosenstock <hal.rosenstock at gmail.com>
> ---
> diff --git a/opensm/opensm/osm_mcast_tbl.c b/opensm/opensm/osm_mcast_tbl.c
> index 82850be..38c06c1 100644
> --- a/opensm/opensm/osm_mcast_tbl.c
> +++ b/opensm/opensm/osm_mcast_tbl.c
> @@ -273,7 +273,7 @@ osm_mcast_tbl_get_block(IN osm_mcast_tbl_t * const p_tbl,
>  	mlid_start_ho = (uint16_t) (block_num * IB_MCAST_BLOCK_SIZE);
>  
>  	if (mlid_start_ho + IB_MCAST_BLOCK_SIZE - 1 > p_tbl->max_mlid_ho)
> -		return (IB_INVALID_PARAMETER);
> +		return (TRUE);

In this case p_block array is not initialized, so just returning 'TRUE'
is not a good idea.

Actually if we are hitting this case it can indicate an inconsistent
mcast_tbl initialization - I would suggest to rework this part and
likely to drop this check at all.

Sasha



More information about the general mailing list