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.ki...@oracle.com> 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@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to