[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v9 06/13] efi: create new early memory allocator



Hello Daniel,

On 05/10/2016 00:02, Daniel Kiper wrote:
On Fri, Sep 30, 2016 at 03:46:54AM -0600, Jan Beulich wrote:
On 29.09.16 at 23:42, <daniel.kiper@xxxxxxxxxx> wrote:
+#else
+static void __init free_ebmalloc_unused_mem(void)
+{
+}
+#endif

Did you build test this for ARM? The function ought to be unused,
as ...

@@ -1251,6 +1301,8 @@ void __init efi_init_memory(void)
     } *extra, *extra_head = NULL;
 #endif

+    free_ebmalloc_unused_mem();

... the whole function here doesn't get built on ARM.

Julien - we're still awaiting your input on general aspects here.

Julien, ping? I would like to go forward with this patchset. Could
you tell us is current solution OK for you (of course except this
stupid build issue spotted by Jan)? Should anything change?

Sorry I have been in conferences for the past 2 weeks with limited review bandwidth. I will try to give a look today.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.