linux-kernel - Re: [PATCH 21/34] kswapd: assign new_order and new_classzone

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LSU.2.00.1207221213100.1896@eggly.anvils>
Date:	Sun, 22 Jul 2012 12:25:14 -0700 (PDT)
From:	Hugh Dickins <hughd@...gle.com>
To:	Mel Gorman <mgorman@...e.de>
cc:	Stable <stable@...r.kernel.org>, linux-mm@...ck.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 21/34] kswapd: assign new_order and new_classzone_idx
 after wakeup in sleeping

On Thu, 19 Jul 2012, Mel Gorman wrote:
> From: "Alex,Shi" <alex.shi@...el.com>
> 
> commit d2ebd0f6b89567eb93ead4e2ca0cbe03021f344b upstream.

Thanks for assembling these, Mel: I was checking through to see if
I was missing any, and noticed that this one has the wrong upstream
SHA1: the one you give here is the same as in 20/34, but it should be

commit f0dfcde099453aa4c0dc42473828d15a6d492936 upstream.

I got quite confused by 30/34 too: interesting definition of "partial
backport" :) I've no objection, but "substitute" might be clearer there.

Hugh

> 
> Stable note: Fixes https://bugzilla.redhat.com/show_bug.cgi?id=712019. This
> 	patch reduces kswapd CPU usage.
> 
> There 2 places to read pgdat in kswapd.  One is return from a successful
> balance, another is waked up from kswapd sleeping.  The new_order and
> new_classzone_idx represent the balance input order and classzone_idx.
> 
> But current new_order and new_classzone_idx are not assigned after
> kswapd_try_to_sleep(), that will cause a bug in the following scenario.
> 
> 1: after a successful balance, kswapd goes to sleep, and new_order = 0;
>    new_classzone_idx = __MAX_NR_ZONES - 1;
> 
> 2: kswapd waked up with order = 3 and classzone_idx = ZONE_NORMAL
> 
> 3: in the balance_pgdat() running, a new balance wakeup happened with
>    order = 5, and classzone_idx = ZONE_NORMAL
> 
> 4: the first wakeup(order = 3) finished successufly, return order = 3
>    but, the new_order is still 0, so, this balancing will be treated as a
>    failed balance.  And then the second tighter balancing will be missed.
> 
> So, to avoid the above problem, the new_order and new_classzone_idx need
> to be assigned for later successful comparison.
> 
> Signed-off-by: Alex Shi <alex.shi@...el.com>
> Acked-by: Mel Gorman <mgorman@...e.de>
> Reviewed-by: Minchan Kim <minchan.kim@...il.com>
> Tested-by: Pádraig Brady <P@...igBrady.com>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> Signed-off-by: Mel Gorman <mgorman@...e.de>
> ---
>  mm/vmscan.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/mm/vmscan.c b/mm/vmscan.c
> index bf85e4d..b8c1fc0 100644
> --- a/mm/vmscan.c
> +++ b/mm/vmscan.c
> @@ -2905,6 +2905,8 @@ static int kswapd(void *p)
>  						balanced_classzone_idx);
>  			order = pgdat->kswapd_max_order;
>  			classzone_idx = pgdat->classzone_idx;
> +			new_order = order;
> +			new_classzone_idx = classzone_idx;
>  			pgdat->kswapd_max_order = 0;
>  			pgdat->classzone_idx = pgdat->nr_zones - 1;
>  		}
> -- 
> 1.7.9.2
> 
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@...ck.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@...ck.org"> email@...ck.org </a>
>