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
