________________________________________
From: Hefty, Sean [sean.he...@intel.com]
Sent: Friday, November 15, 2013 1:09 AM
To: Somnath Kotur; rol...@kernel.org
Cc: linux-rdma@vger.kernel.org
Subject: RE: [PATCH] IB/CMA: Initialize hop_limit to a known default macro 
instead of 1 as is done currently.

> -     route->path_rec->hop_limit = 1;
> +     route->path_rec->hop_limit = IPV6_DEFAULT_HOPLIMIT;

This is not an IPv6 route though.  AFAIK, RoCE/IBoE is not routable, so the hop 
limit should be set to 1.  Using a different value affects timeout values set 
by the IB CM.

Can you describe more of the reasoning behind this change?

Sean,
        This change was done precisely to enable us to do that i.e make it 
routable. This was the only suitable macro i could find that defines a certain 
default value for the hop limit and is used by the IPv6 layer in the kernel 
already. I'm open to other suggestions of values, but pegging to 1 means we are 
not allowing any scope for it to be routable in the future ..Would you agree ? 

Thanks
Som


--
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

Reply via email to