Re: [PATCH 10/24] HWPOISON: remove the free buddy page handler

From: Wu Fengguang
Date: Wed Dec 02 2009 - 08:31:26 EST


On Wed, Dec 02, 2009 at 09:13:30PM +0800, Andi Kleen wrote:
> On Wed, Dec 02, 2009 at 11:12:41AM +0800, Wu Fengguang wrote:
> > The buddy page has already be handled in the very beginning.
> > So remove redundant code.
>
> I think I prefer the table to be complete, even if some of the
> cases might not happen currently. A BUG() would be reasonable though.

I'd prefer not to carry around some useless bytes in kernel.

What if we replace it with a comment line in the table?

Thanks,
Fengguang
--
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/