Date: Sat, 7 Aug 2010 16:53:51 GMT From: Edward Tomasz Napierala <trasz@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 182037 for review Message-ID: <201008071653.o77GrpNq027411@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://p4web.freebsd.org/@@182037?ac=10 Change 182037 by trasz@trasz_victim on 2010/08/07 16:53:34 When containers are enabled, run pagedaemon at least every second to recompute RSS. Affected files ... .. //depot/projects/soc2009/trasz_limits/TODO#25 edit .. //depot/projects/soc2009/trasz_limits/sys/vm/vm_pageout.c#12 edit Differences ... ==== //depot/projects/soc2009/trasz_limits/TODO#25 (text+ko) ==== @@ -62,8 +62,6 @@ container mechanism. This is a part of HRL, and it's outside the scope of the containers project. This means the code is full of "#ifdef CONTAINERS". - - RSS is accounted for only when pagedaemon is running, i.e. when memory shortage occurs. - - Don't account resources for system processes. It's not as easy as checking for P_SYSTEM flag, as the flag will be set for AIO kernel processes. ==== //depot/projects/soc2009/trasz_limits/sys/vm/vm_pageout.c#12 (text+ko) ==== @@ -1627,7 +1627,11 @@ while (TRUE) { mtx_lock(&vm_daemon_mtx); +#ifdef CONTAINERS + msleep(&vm_daemon_needed, &vm_daemon_mtx, PPAUSE, "psleep", hz); +#else msleep(&vm_daemon_needed, &vm_daemon_mtx, PPAUSE, "psleep", 0); +#endif swapout_flags = vm_pageout_req_swapout; vm_pageout_req_swapout = 0; mtx_unlock(&vm_daemon_mtx);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201008071653.o77GrpNq027411>