[ofa-general] [PATCH] opensm/osm_ucast_mgr.c: simplify fwd tables setup flow

Sasha Khapyorsky sashak at voltaire.com
Sun Aug 30 03:02:53 PDT 2009


On 18:59 Sat 29 Aug     , Hal Rosenstock wrote:
> >
> > diff --git a/opensm/opensm/osm_ucast_mgr.c b/opensm/opensm/osm_ucast_mgr.c
> > index b7e3893..39d825c 100644
> > --- a/opensm/opensm/osm_ucast_mgr.c
> > +++ b/opensm/opensm/osm_ucast_mgr.c
> > @@ -1007,6 +1007,7 @@ int osm_ucast_mgr_process(IN osm_ucast_mgr_t * p_mgr)
> >                /* If configured routing algorithm failed, use default
> > MinHop */
> >                osm_ucast_mgr_build_lid_matrices(p_mgr);
> >                ucast_mgr_build_lfts(p_mgr);
> > +               osm_ucast_mgr_set_fwd_tables(p_mgr);
> 
> 
> 
> Shouldn't this be osm_ucast_mgr_set_fwd_table ?

Yes it should be, but I renamed this later, now it is
osm_ucast_mgr_set_fwd_tables() (since it sets all tables and not per
switch as before). By mistake I pushed this last change patch before
renaming so we have broken patch in the history (thing I'm trying to
avoid normally).

Sasha



More information about the general mailing list