On Tue, Apr 17, 2018 at 03:16:27PM -0400, David Miller wrote: > From: Hangbin Liu <liuhang...@gmail.com> > Date: Tue, 17 Apr 2018 20:52:54 +0800 > > > Like tos inherit, ttl inherit should also means inherit the inner protocol's > > ttl values, which actually not implemented in vxlan yet. > > > > But we could not treat ttl == 0 as "use the inner TTL", because that would > > be > > used also when the "ttl" option is not specified and that would be a > > behavior > > change, and breaking real use cases. > > > > So add a different attribute IFLA_VXLAN_TTL_INHERIT when "ttl inherit" is > > specified. > > > > --- > > v2: As suggested by Stefano, clean up function ip_tunnel_get_ttl(). > > > > Suggested-by: Jiri Benc <jb...@redhat.com> > > Signed-off-by: Hangbin Liu <liuhang...@gmail.com> > > I already applied V1 of your patch. > > Furthermore, this commit message would cause your signoffs and other tags > to be removed due to the "---" deliminator. > > I generally encourage people to leave the change history text _in_ the > commit message anyways. It is useful information for the future.
Thanks for the reminding. I will keep this in mind. Cheers Hangbin
