On Wed, Aug 7, 2013 at 6:09 AM, Matt Fleming [off-list ref] wrote:

On Tue, 06 Aug, at 08:45:00PM, Roy Franz wrote:

quoted

Rename them to be more similar, as low_free() could be used to free
memory allocated by both high_alloc() and low_alloc().
high_alloc() -> efi_high_alloc()
low_alloc()  -> efi_low_alloc()
low_free()   -> efi_free()

Signed-off-by: Roy Franz <redacted>
---
 arch/x86/boot/compressed/eboot.c       |   19 ++++++++++---------
 drivers/firmware/efi/efi-stub-helper.c |   16 ++++++++--------
 2 files changed, 18 insertions(+), 17 deletions(-)

[...]

quoted

@@ -163,10 +163,10 @@ fail:
 }

 /*
- * Allocate at the lowest possible address.
+ * Allocate at the lowest possible address, that is not below 'min'
  */
-static efi_status_t low_alloc(efi_system_table_t *sys_table_arg,
-             unsigned long size, unsigned long align,
+static efi_status_t efi_low_alloc(efi_system_table_t *sys_table_arg,
+                           unsigned long size, unsigned long align,
                            unsigned long *addr)
 {
      unsigned long map_size, desc_size;

Whoops, did this change to the comment slip in from some other patch?

--
Matt Fleming, Intel Open Source Technology Center

Yes it did - that should be in a later patch that makes that change.

Roy