On Tue, Oct 19, 2021 at 07:09:42PM +0300, Nikolay Aleksandrov wrote:
> > I started this patch when I saw there is not limit for setting
> > multicast_membership_interval to 0, which will cause bridge remove the
> > mdb directly after adding. Do you think this is a problem.
> > 
> > And what about others? I don't think there is a meaning to set other 
> > intervals
> > to 0.
> > 
> 
> The problem is not if there is meaning, we cannot start restricting option 
> values now after
> they've become uapi (and have been for a very long time) because we can break 
> user-space even
> though chances are pretty low. I don't think this patch is acceptable, I 
> commented on the other
> patch issues but they don't matter because of this.

OK, I got your mean, we should not restrict the configurations based on whether
there is a meaning.

Thanks
Hangbin

Reply via email to