On Fri, Jun 13, 2014 at 08:52:22AM +0400, Konstantin Khlebnikov wrote: > On Fri, Jun 13, 2014 at 8:36 AM, Chen Yucong <sla...@gmail.com> wrote: > > shrink_page_list() has too many arguments that have already reached ten. > > Some of those arguments and temporary variables introduces extra 80 bytes > > on the stack. This patch wraps five parameters into shrink_result and > > removes > > some temporary variables, thus making the relative functions to consume > > fewer > > stack space. > > I think it's better to put them into struct scan_control. > Reset them before calling shrinker or take a snapshot to get delta.
scan_control applies to the whole reclaim invocation*, it would be confusing as hell to have things in there that only apply to certain sublevels. Please don't do that. If you on the other hand take snapshots and accumulate them over the whole run, it might actually make sense to move sc->nr_scanned and sc->nr_reclaimed into shrink_results instead. But I'm not sure it's worth the extra snapshotting code, given that we don't actually need the accumulated numbers at the outer levels right now. * sc->swappiness being the recent exception, I'll send a fix for that. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
