On 10:16 Tue 15 Dec , Hal Rosenstock wrote:
>
> > patch: 3d20f82edd3246879063b77721d0bcef927bdc48
> >
> > opensm/osm_sa_path_record.c: separate router guid resolution code
> >
> > Move off subnet destination (router address) resolution code to separate
> > function to improve readability.
> >
> > Signed-off-by: Sasha Khapyorsky <sas...@voltaire.com>
> >
> > I further traced the problem to pr_rcv_build_pr and the patch below fixes
> > the
> > bug.
>
> On quick glance, I'm missing the connection between Sasha's patch and
> this routine setting something bad in the hop limit of the returned
> path record.
In this patch I moved *p_dgid initialization out from router only
section, as side effect it turns on this section:
/* Only set HopLimit if going through a router */
if (is_nonzero_gid)
p_pr->hop_flow_raw |= cl_hton32(IB_HOPLIMIT_MAX);
for all SA PR queries where DGID was selected (local subnet or not).
Sasha
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
