Re: kexec load failure introduced by "x86, memblock: Replace e820_/_earlystring with memblock_"

From: H. Peter Anvin
Date: Mon Sep 27 2010 - 19:27:42 EST


On 09/27/2010 04:20 PM, Yinghai Lu wrote:
>
> x86 own version for find_area?
>

No, double no.

Same kind of crap: overloading an interface with semantics it shouldn't
have. The right thing is to introduce a new interface with carries the
explicitly needed policy with it... e.g. memblock_find_in_range_lowest().

That interface would have the explicit semantics of returning the lowest
possible address, as opposed to any suitable address (which may change
if policy requirements change.)

The other question is why does kexec need this in the first place? Is
this due to a design bug in kexec or is there some fundamental reason
for this?

-hpa
--
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/