On Sun, 2017-12-10 at 12:36 +0800, Xin Long wrote: > The new patch works to me, just two questions: > 1. should it use "idev->cnf.mtu6" here for mld ? No idea why some parts of IPv6 would use a different view of device mtu. Maybe others can comment. > > 2. 'if (int < unsigned int)' is still not nice, though in 'if > (AVAILABLE(skb) < sizeof())' > AVAILABLE(skb) seems always to return >= 0 after your patch. Really if AVAILABLE(skb) was negative, a bug already happened. -- To unsubscribe from this list: send the line "unsubscribe linux-sctp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html