Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(283)

Issue 4124009: sys-apps/baselayout: set min_filelist_kbytes to 50M (Closed)

Created:
10 years, 1 month ago by Mandeep Singh Baines
Modified:
9 years, 7 months ago
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, adlr, anush
Visibility:
Public.

Description

sys-apps/baselayout: set min_filelist_kbytes to 50M To avoid thrashing, we are going to protect 50M of file cache. BUG=7561 TEST=Verified that the value gets set. $ cat /proc/sys/vm/min_filelist_kbytes 50000 Change-Id: If804c776a72478ed31f8e81e7d773b28bf458d05 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=74386c7

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, --1 lines) Patch
A + sys-apps/baselayout/baselayout-2.0.1-r209.ebuild View 0 chunks +-1 lines, --1 lines 0 comments Download
M sys-apps/baselayout/files/sysctl.conf View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mandeep Singh Baines
10 years, 1 month ago (2010-10-29 00:39:12 UTC) #1
Will Drewry
LGTM!
10 years, 1 month ago (2010-10-29 00:44:14 UTC) #2
hughd
10 years, 1 month ago (2010-10-29 00:56:55 UTC) #3
LGTM (I'm taking it on trust that what looks like an ebuild non-change is
actually something sensible that the review server cannot show!)

Powered by Google App Engine
This is Rietveld 408576698