lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20121206202325.GA1498@cmpxchg.org> Date: Thu, 6 Dec 2012 15:23:25 -0500 From: Johannes Weiner <hannes@...xchg.org> To: Linus Torvalds <torvalds@...ux-foundation.org> Cc: Bruno Wolff III <bruno@...ff.to>, Thorsten Leemhuis <fedora@...mhuis.info>, Mel Gorman <mgorman@...e.de>, Andrew Morton <akpm@...ux-foundation.org>, Rik van Riel <riel@...hat.com>, George Spelvin <linux@...izon.com>, Johannes Hirte <johannes.hirte@....tu-ilmenau.de>, Tomas Racek <tracek@...hat.com>, Jan Kara <jack@...e.cz>, Dave Hansen <dave@...ux.vnet.ibm.com>, Josh Boyer <jwboyer@...il.com>, Valdis Kletnieks <Valdis.Kletnieks@...edu>, Jiri Slaby <jslaby@...e.cz>, Zdenek Kabelac <zkabelac@...hat.com>, linux-mm <linux-mm@...ck.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, John Ellson <john.ellson@...cast.net> Subject: Re: kswapd craziness in 3.7 On Thu, Dec 06, 2012 at 11:31:21AM -0800, Linus Torvalds wrote: > Ok, people seem to be reporting success. > > I've applied Johannes' last patch with the new tested-by tags. > > Johannes (or anybody else, for that matter), please holler LOUDLY if > you disagreed.. (or if I used the wrong version of the patch, there's > been several, afaik). I just went back one more time and of course that's when I spot that I forgot to remove the zone congestion clearing that depended on the now removed checks to ensure the zone is balanced. It's not too big of a deal, just the /risk/ of increased CPU use from reclaim because we go back to scanning zones that we previously deemed congested and slept a little bit before continuing reclaim. Sorry, I should have seen that earlier. Removing it is a low risk fix, the clearing was kinda redundant anyway (the preliminary zone check clears it for OK zones, so does the reclaim loop under the same criteria), letting it stay is probably more problematic for 3.8 than just dropping it... --- From: Johannes Weiner <hannes@...xchg.org> Subject: [patch] mm: vmscan: fix inappropriate zone congestion clearing c702418 ("mm: vmscan: do not keep kswapd looping forever due to individual uncompactable zones") removed zone watermark checks from the compaction code in kswapd but left in the zone congestion clearing, which now happens unconditionally on higher order reclaim. This messes up the reclaim throttling logic for zones with dirty/writeback pages, where zones should only lose their congestion status when their watermarks have been restored. Remove the clearing from the zone compaction section entirely. The preliminary zone check and the reclaim loop in kswapd will clear it if the zone is considered balanced. Signed-off-by: Johannes Weiner <hannes@...xchg.org> --- mm/vmscan.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index 124bbfe..b7ed376 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2827,9 +2827,6 @@ static unsigned long balance_pgdat(pg_data_t *pgdat, int order, if (zone_watermark_ok(zone, order, low_wmark_pages(zone), *classzone_idx, 0)) zones_need_compaction = 0; - - /* If balanced, clear the congested flag */ - zone_clear_flag(zone, ZONE_CONGESTED); } if (zones_need_compaction) -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists