From: Maciej Żenczykowski <zenczykow...@gmail.com> Date: Tue, 27 Sep 2016 05:16:45 -0700
> From: Maciej Żenczykowski <m...@google.com> > > Signed-off-by: Maciej Żenczykowski <m...@google.com> I was about to ask you to submit this patch alongside an actual addition of a proc_dointvec_minmax user to the table. But upon second checking there is one, indirectly via addrconf_sysctl_mtu(). I should never have to search and figure things out like that. It should be obvious or explained in the commit message. You must explain in detail what the dependency is, why you have to make this change, how you made it, and why you made it that way. Especially in this case, your empty commit message is really not appropriate. Thanks.