[openib-general] [PATCH] OpenSM: Support configurable SL per partition

[Hal Rosenstock]

Hi Pat,

On Tue, 2006-07-11 at 17:14, Patrick Mullaney wrote:
> This will avoid an invalid warning about service level value if sl=0 is
> used in the partition config file. Can you include something like it in
> your original patch?

Yes, SL 0 is valid so this warning should not be output. I will include
this patch when I commit this functionality which will hopefully be over
the next few days. Thanks for catching this!

-- Hal

> 
> Thanks.
> Pat
> 
> --------------------------------------------------------------------------------------------------
> 
> --- opensm.old/osm_prtn_config.c	2006-07-11 16:06:33.000000000
> -0400
> +++ opensm/osm_prtn_config.c	2006-07-11 16:08:00.000000000 -0400
> @@ -52,6 +52,7 @@
>  #include <string.h>
>  #include <errno.h>
>  #include <ctype.h>
> +#include <limits.h>
>  
>  #include <iba/ib_types.h>
>  #include <opensm/osm_base.h>
> @@ -159,7 +160,7 @@ static int partition_add_flag(unsigned l
>  	} else if (!strncmp(flag, "sl", len)) {
>  		unsigned sl;
>  
> -		if (!val || (sl = strtoul(val, NULL, 0)) == 0 || sl >
> 15)
> +		if (!val || (sl = strtoul(val, NULL, 0)) == UINT_MAX ||
> sl > 15)
>  			PARSEWARN(conf->p_log, lineno,
>  				"flag \'sl\' requires valid value"
>  				" - skipped.\n");
>