Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754909AbYKKSHd (ORCPT ); Tue, 11 Nov 2008 13:07:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754103AbYKKSHZ (ORCPT ); Tue, 11 Nov 2008 13:07:25 -0500 Received: from vms173005pub.verizon.net ([206.46.173.5]:43418 "EHLO vms173005pub.verizon.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754032AbYKKSHZ (ORCPT ); Tue, 11 Nov 2008 13:07:25 -0500 Date: Tue, 11 Nov 2008 13:07:15 -0500 (EST) From: Len Brown Subject: Re: [PATCH -next] Hibernate: Take overlapping zones into account In-reply-to: <200811081356.13060.rjw@sisk.pl> X-X-Sender: lenb@localhost.localdomain To: "Rafael J. Wysocki" Cc: Pavel Machek , Dave Hansen , Andy Whitcroft , LKML , pm list Message-id: MIME-version: 1.0 Content-type: TEXT/PLAIN; charset=US-ASCII References: <200811081356.13060.rjw@sisk.pl> User-Agent: Alpine 2.00 (LFD 1167 2008-08-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1945 Lines: 54 On Sat, 8 Nov 2008, Rafael J. Wysocki wrote: > Hi Len, > > Please add this patch to the suspend branch. Please re-send a checkpatch clean version. Also, the comments on this patch (and the next) are mal-formed. The commit message must be before the first "---" and the throw-away comments must be after it, rather thant the reverse. I ask others to get this right, so it seems most fair that I do the same for you -- otherwise your next patch will have the same problems and others will follow your example of not sending the "perfect patch" thanks, -Len > --- > From: Rafael J. Wysocki > Subject: Hibernate: Take overlapping zones into account > > It has been requested to make hibernation work with memory > hotplugging enabled and for this purpose the hibernation code has to > be reworked to take the possible overlapping of zones into account. > Thus, rework the hibernation memory bitmaps code to prevent > duplication of PFNs from occuring and add checks to make sure that > one page frame will not be marked as saveable many times. > > Additionally, use list.h lists instead of open-coded lists to > implement the memory bitmaps. > > Signed-off-by: Rafael J. Wysocki > Cc: Pavel Machek > Cc: Dave Hansen > Cc: Andy Whitcroft > --- > kernel/power/snapshot.c | 331 ++++++++++++++++++++++++------------------------ > 1 file changed, 169 insertions(+), 162 deletions(-) > > Index: linux-2.6/kernel/power/snapshot.c > =================================================================== > --- linux-2.6.orig/kernel/power/snapshot.c > +++ linux-2.6/kernel/power/snapshot.c > @@ -25,6 +25,7 @@ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/