On Sun, May 10, 2009 at 2:42 AM, Eli Dorfman (Voltaire)
<[email protected]> wrote:
> Hal Rosenstock wrote:
>> Signed-off-by: Hal Rosenstock <[email protected]>
>> ---
>> diff --git a/opensm/opensm/osm_port.c b/opensm/opensm/osm_port.c
>> index 2e6c642..17bac73 100644
>> --- a/opensm/opensm/osm_port.c
>> +++ b/opensm/opensm/osm_port.c
>> @@ -381,7 +381,7 @@ uint8_t osm_physp_calc_link_op_vls(IN osm_log_t * p_log,
>>               op_vls = p_subn->opt.max_op_vls;
>>
>>       if (op_vls == 0) {
>> -             OSM_LOG(p_log, OSM_LOG_DEBUG, "ERR 4102: "
>> +             OSM_LOG(p_log, OSM_LOG_DEBUG,
>>                       "Invalid OP_VLS = 0. Forcing correction to 1 (VL0)\n");
>
> In this case I think that level should be changed to ERROR since this is not 
> the normal behavior.

Sasha has been adamant that any device supplied data errors use
something other than ERROR log level.

-- Hal

>
>>               op_vls = 1;
>>       }
> _______________________________________________
> general mailing list
> [email protected]
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
>
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to