From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759725AbcINAy1 (ORCPT ); Tue, 13 Sep 2016 20:54:27 -0400 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:47354 "HELO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1755414AbcINAyV (ORCPT ); Tue, 13 Sep 2016 20:54:21 -0400 From: "Rafael J. Wysocki" To: Anisse Astier Cc: linux-mm@kvack.org, linux-pm@vger.kernel.org, "Kirill A . Shutemov" , Laura Abbott , Mel Gorman , Alan Cox , Andi Kleen , Andrew Morton , Brad Spengler , Dave Hansen , David Rientjes , Jianyu Zhan , Kees Cook , Len Brown , Linus Torvalds , Mathias Krause , Michal Hocko , PaX Team , Pavel Machek , Peter Zijlstra , Vlastimil Babka , Yves-Alexis Perez , linux-kernel@vger.kernel.org, Kees Cook , Pavel Machek Subject: Re: [PATCH] PM / Hibernate: allow hibernation with PAGE_POISONING_ZERO Date: Wed, 14 Sep 2016 03:00:29 +0200 Message-ID: <1856521.PhkV49ibzK@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/4.8.0-rc2+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1473410612-6207-1-git-send-email-anisse@astier.eu> References: <1473410612-6207-1-git-send-email-anisse@astier.eu> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday, September 09, 2016 10:43:32 AM Anisse Astier wrote: > PAGE_POISONING_ZERO disables zeroing new pages on alloc, they are > poisoned (zeroed) as they become available. > In the hibernate use case, free pages will appear in the system without > being cleared, left there by the loading kernel. > > This patch will make sure free pages are cleared on resume when > PAGE_POISONING_ZERO is enabled. We free the pages just after resume > because we can't do it later: going through any device resume code might > allocate some memory and invalidate the free pages bitmap. > > Thus we don't need to disable hibernation when PAGE_POISONING_ZERO is > enabled. > > Signed-off-by: Anisse Astier > Cc: Kirill A. Shutemov > Cc: Laura Abbott > Cc: Mel Gorman > Cc: Rafael J. Wysocki Applied (with the tags from Pavel and Kees). Thanks, Rafael