summaryrefslogtreecommitdiffstats
path: root/crypto/ansi_cprng.c
diff options
context:
space:
mode:
authorZlatko Calusic <zlatko.calusic@iskon.hr>2012-12-20 00:25:13 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2012-12-20 07:06:56 -0800
commitcda73a10eb3f493871ed39f468db50a65ebeddce (patch)
treefe486de989b58130ba20d534606f1091e367c8ec /crypto/ansi_cprng.c
parentf01af9f85855e38fbd601e033a8eac204cc4cc1c (diff)
downloadop-kernel-dev-cda73a10eb3f493871ed39f468db50a65ebeddce.zip
op-kernel-dev-cda73a10eb3f493871ed39f468db50a65ebeddce.tar.gz
mm: do not sleep in balance_pgdat if there's no i/o congestion
On a 4GB RAM machine, where Normal zone is much smaller than DMA32 zone, the Normal zone gets fragmented in time. This requires relatively more pressure in balance_pgdat to get the zone above the required watermark. Unfortunately, the congestion_wait() call in there slows it down for a completely wrong reason, expecting that there's a lot of writeback/swapout, even when there's none (much more common). After a few days, when fragmentation progresses, this flawed logic translates to a very high CPU iowait times, even though there's no I/O congestion at all. If THP is enabled, the problem occurs sooner, but I was able to see it even on !THP kernels, just by giving it a bit more time to occur. The proper way to deal with this is to not wait, unless there's congestion. Thanks to Mel Gorman, we already have the function that perfectly fits the job. The patch was tested on a machine which nicely revealed the problem after only 1 day of uptime, and it's been working great. Signed-off-by: Zlatko Calusic <zlatko.calusic@iskon.hr> Acked-by: Mel Gorman <mgorman@suse.de> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'crypto/ansi_cprng.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud