Re: [PATCH 4/4] mm: Do not define PFN_SECTION_SHIFT if!CONFIG_SPARSEMEM

From: Dave Hansen
Date: Tue May 03 2011 - 12:10:30 EST


On Mon, 2011-05-02 at 23:22 +0200, Daniel Kiper wrote:
> Do not define PFN_SECTION_SHIFT if !CONFIG_SPARSEMEM.
>
> Signed-off-by: Daniel Kiper <dkiper@xxxxxxxxxxxx>

I'd like if this was a bit easier to verify that it didn't break
anything. Basically, we should probably limit direct use of
PFN_SECTION_SHIFT to inside #ifdefs in headers.

But, if something is truly using this today, it's probably broken. It's
easy enough to work around if someone hits it, I guess.

Acked-by: Dave Hansen <dave@xxxxxxxxxxxxxxxxxx>

-- Dave

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/