On Wed, Mar 09, 2011 at 02:00:16PM -0500, Hal Rosenstock wrote: > >> >> IBA 1.2.1 v1 p.830 line 9 the otherwise sentence. > >> > > >> > That is talking about PortInfo. NodeInfo does not support that > >> > attribute modifier language, my patch affects SANodeRecord. > >> > >> My bad :-( I was thinking of PortInfoRecord, not NodeRecord, for some > >> reason... > > > > No worries, what do you think about my patch, does it align with what > > IBA intends? > > Yes from my brief scan/understanding of what it's trying to > accomplish. I haven't probed/analyzed the actual fix itself. I guess > LocalPortNum in NodeRecord was not so important to anyone before now.
Right, plus this only comes up if you connect both ports on a HCA into the same subnet, otherwise opensm works correctly.. Alex? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
