On Tue 27-06-17 06:47:51, Christoph Hellwig wrote: > On Tue, Jun 27, 2017 at 10:49:50AM +0200, Michal Hocko wrote: > > Christoph, Darrick > > could you have a look at this patch please? Andrew has put it into mmotm > > but I definitely do not want it passes your attention. > > I don't think what we have to gain from it. Callsite for KM_MAYFAIL > should handler failures, but the current behavior seems to be doing fine > too. Last time I've asked I didnd't get any reply so let me ask again. Some of those allocations seem to be small (e.g. by a random look xlog_cil_init allocates struct xfs_cil which is 576B and struct xfs_cil_ctx 176B). Those do not fail currently under most conditions and it will retry allocation with the OOM killer if there is no progress. As you know that failing those is acceptable, wouldn't it be better to simply fail them and do not disrupt the system with the oom killer? -- Michal Hocko SUSE Labs -- 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>
