Re: [PATCH 04/16] Add minimum address parameter to efi_low_alloc()

From: Grant Likely
Date: Fri Aug 30 2013 - 18:56:25 EST


On Fri, Aug 30, 2013 at 11:12 PM, Roy Franz <roy.franz@xxxxxxxxxx> wrote:
> On Fri, Aug 30, 2013 at 6:01 AM, Grant Likely <grant.likely@xxxxxxxxxxxx> wrote:
>> On Fri, 9 Aug 2013 16:26:05 -0700, Roy Franz <roy.franz@xxxxxxxxxx> wrote:
>>> This allows allocations to be made low in memory while
>>> avoiding allocations at the base of memory.
>>
>> Your commit message should include /why/ the change is needed. From the
>> above I understand what the patch does, but I don't understand why it is
>> necessary.
>
> This was used to avoid relocating the zImage so low in memory that it
> would conflict
> with memory range that it was decompressed into. This is now handled
> by allocating
> the memory for the decompressed kernel, so the lower limit is no
> longer required.
> I'll remove it, as it is not necessary any more.

Even better! :-)

g.
--
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/