Hello, On Fri, Aug 28, 2015 at 07:13:22PM +0200, Michal Hocko wrote: > On Fri 28-08-15 11:25:29, Tejun Heo wrote: > > Currently, try_charge() tries to reclaim memory directly when the high > > limit is breached; however, this has a couple issues. > > > > * try_charge() can be invoked from any in-kernel allocation site and > > reclaim path may use considerable amount of stack. This can lead to > > stack overflows which are extremely difficult to reproduce. > > This is true but I haven't seen any reports for the stack overflow for > quite some time. So, this didn't really fix it but xfs had to punt things to workqueues to avoid stack overflows and IIRC it involved direct reclaim. Maybe it's too late but it probably is a good idea to punt this from the source. > I would just argue that this implementation has the same issue as the > other patch in the series which performs high-usage reclaim. I think > that each task should reclaim only its contribution which is trivial > to account. Hmm... I'll respond there. Thanks. -- tejun -- 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>