[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <201009062034.36437.rjw@sisk.pl>
Date: Mon, 6 Sep 2010 20:34:36 +0200
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
Cc: Hugh Dickins <hughd@...gle.com>,
Andrew Morton <akpm@...ux-foundation.org>,
KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
Ondrej Zary <linux@...nbow-software.org>,
Andrea Gelmini <andrea.gelmini@...il.com>,
Balbir Singh <balbir@...ibm.com>,
Andrea Arcangeli <aarcange@...hat.com>,
Nigel Cunningham <nigel@...onice.net>, linux-mm@...ck.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/4] swap: prevent reuse during hibernation
On Monday, September 06, 2010, KAMEZAWA Hiroyuki wrote:
> On Mon, 6 Sep 2010 01:12:38 -0700 (PDT)
> Hugh Dickins <hughd@...gle.com> wrote:
>
> > Move the hibernation check from scan_swap_map() into try_to_free_swap():
> > to catch not only the common case when hibernation's allocation itself
> > triggers swap reuse, but also the less likely case when concurrent page
> > reclaim (shrink_page_list) might happen to try_to_free_swap from a page.
> >
> > Hibernation already clears __GFP_IO from the gfp_allowed_mask, to stop
> > reclaim from going to swap: check that to prevent swap reuse too.
> >
> > Signed-off-by: Hugh Dickins <hughd@...gle.com>
> > Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
> > Cc: KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>
> > Cc: "Rafael J. Wysocki" <rjw@...k.pl>
> > Cc: Ondrej Zary <linux@...nbow-software.org>
> > Cc: Andrea Gelmini <andrea.gelmini@...il.com>
> > Cc: Balbir Singh <balbir@...ibm.com>
> > Cc: Andrea Arcangeli <aarcange@...hat.com>
> > Cc: Nigel Cunningham <nigel@...onice.net>
> > Cc: stable@...nel.org
>
> Hmm...seems better.
>
> Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
Acked-by: Rafael J. Wysocki <rjw@...k.pl>
--
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