[ofa-general] Re: [PATCH 2/2] IB/IPoIB: Separate IB events to groups and handle each according to level of severity

[Hal Rosenstock]

Olga,

On Thu, 2008-05-22 at 18:28 +0300, Olga Shern (Voltaire) wrote:
> Hal, 
> You pointed out that we cannot rely on the assumption that on SM
> failover there is not path change.
> In the previous patch we only flush multicast.
> What Moni changed in this patch is that on SM failover (SM change
> event), we will flush not only multicast but also all paths but
> without destroying ah.

I missed that in the patch :-( It addresses the first level of concern
in terms of the unicast paths but leaves open the path parameter changes
(rate, etc.) as the address handles are preserved as Moni stated in
other words. I agree it's in the right direction. I would like to see
the whole problem solved. Is the cost of recreating the AHs too much or
is something else leading towards preserving the AHs ? That's what's
needed to be resolved for a complete solution.

-- Hal

> Olga