On Thu, Jan 07, 2010 at 08:50:47AM -0800, Sean Hefty wrote: > >> > + route->path_rec->hop_limit = 2; > >> > > > >The reason is that ib_init_ah_from_path() will not set IB_AH_GRH for > >hop_limit smaller then 2, and since that GRH is required in RoCEE, and > >since this is specific to RoCEE, I put 2 to make pass this. > > A hop limit of 2 seems wrong though. Isn't there some change that > could be made to ib_init_ah_from_path() that would work? It seems > like the PR itself should indicate somehow whether the path is for > IB or Ethernet. (Special LID values or some flag...)
I agree, on the wire the hop limit in the GRH should be 0 or 0xFF. 0 is preferred since there is no support for routing. 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
