lkml.org 
[lkml]   [2009]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] hibernate / memory hotplug: always use for_each_populated_zone()
    On Tue, Jul 21, 2009 at 07:29:58AM +1000, Nigel Cunningham wrote:
    > Hi.
    >
    > Gerald Schaefer wrote:
    > > From: Gerald Schaefer <gerald.schaefer@de.ibm.com>
    > >
    > > Use for_each_populated_zone() instead of for_each_zone() in hibernation
    > > code. This fixes a bug on s390, where we allow both config options
    > > HIBERNATION and MEMORY_HOTPLUG, so that we also have a ZONE_MOVABLE
    > > here. We only allow hibernation if no memory hotplug operation was
    > > performed, so in fact both features can only be used exclusively, but
    > > this way we don't need 2 differently configured (distribution) kernels.
    > >
    > > If we have an unpopulated ZONE_MOVABLE, we allow hibernation but run
    > > into a BUG_ON() in memory_bm_test/set/clear_bit() because hibernation
    > > code iterates through all zones, not only the populated zones, in
    > > several places. For example, swsusp_free() does for_each_zone() and
    > > then checks for pfn_valid(), which is true even if the zone is not
    > > populated, resulting in a BUG_ON() later because the pfn cannot be
    > > found in the memory bitmap.
    >
    > I agree with your logic and patch, but doesn't this also imply that the
    > s390 implementation pfn_valid should be changed to return false for
    > those pages?

    For CONFIG_SPARSEMEM, which s390 uses, there is no architecture specific
    pfn_valid() implementation.
    Also it looks like the semantics of pfn_valid() aren't clear.
    At least for sparsemem it means nothing but "the memmap for the section
    this page belongs to exists". So it just means the struct page for the
    pfn exists.
    We still have pfn_present() for CONFIG_SPARSEMEM. But that just means
    "some pages in the section this pfn belongs to are present."
    So it looks like checking for pfn_valid() and afterwards checking
    for PG_Reserved (?) might give what one would expect.
    Looks all a bit confusing to me.
    Or maybe it's just me who is confused? :)


    \
     
     \ /
      Last update: 2009-07-21 09:17    [W:2.803 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site