On Mon, 12 Jun 2017 21:35:17 -0400 "Liam R. Howlett" <Liam.Howlett@xxxxxxxxxx> wrote: > > > > > If there's no message stating any > > > configuration issue, then many admins would probably think something is > > > seriously broken and it's not just a simple typo of K vs M. > > > > > > Even though this doesn't catch all errors, I think it's a worth while > > > change since this is currently a silent failure which results in a > > > system crash. > > > > Seriously, this warning just doesn't help in _most_ miscofigurations. It > > just focuses on one particular which really requires to misconfigure > > really badly. And there are way too many other ways to screw your system > > that way, yet we do not warn about many of those. So just try to step > > back and think whether this is something we actually do care about and > > if yes then try to come up with a more reasonable warning which would > > cover a wider range of misconfigurations. > > Understood. Again, I appreciate all the time you have taken on my > patch and explaining your points. I will look at this again as you > have suggested. So do we want to drop mm-hugetlb-warn-the-user-when-issues-arise-on-boot-due-to-hugepages.patch? I'd be inclined to keep it if Liam found it a bit useful - it does have some overhead, but half the patch is in __init code... -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
