On Thu, Jan 19, 2017 at 10:33:52AM -0700, Jason Gunthorpe wrote: > On Thu, Jan 19, 2017 at 03:12:58PM +0200, Leon Romanovsky wrote: > > > Let's hope that we will find such brave man who will > > remove this line. The removal of this one line will > > require from him to do extensive testing and reviewing > > different code paths. > > Why? We already fully support this configuration for RC At minimum, i see here one new path which should be tested. Configure small MTU -> Join multicast group -> Configure large MTU Before removing min(..), in third step, we ensured that MTU won't exceed multicast MTU in all flows. After removing min(..), you will have a chance of flows there MTU limit won't work. And I just saying that it needs to be tested. > > Jason
Attachment:
signature.asc
Description: PGP signature
