[ofa-general] Re: [PATCH] opensm/opensm/osm_subnet.c: add checks for HOQ and Leaf HOQ input values

Sasha Khapyorsky sashak at voltaire.com
Wed Apr 9 03:01:08 PDT 2008


Hi Ira,

On 16:48 Tue 08 Apr     , weiny2 at llnl.gov wrote:
> As per Hal's comments change the alternate value for [leaf] HOQ to be
> "infinity" when the user specifies a value larger than "infinity".

Actually I would prefer original version of the patch. The main reason
is that infinite packet life time is really dangerous thing - in case
when a fabric is routed with credit loops (very common case with default
min-hops routing) it leads to total fabric stuck and not just to some
performance degradation.

So I think it is safer to reject invalid value and to set the default
(log an error, etc.i). As it was done in the original version of the
patch.

Hal, do you agree?

Sasha



More information about the general mailing list