Re: Ping: [PATCH 0/6] tools/include: adjustments to the population of xen/

[Bertrand Marquis <Bertrand.Marquis@xxxxxxx>]

Hi,

> On 1 Oct 2020, at 17:29, Bertrand Marquis <bertrand.marquis@xxxxxxx> wrote:
> 
> Hi Jan,
> 
>> On 1 Oct 2020, at 17:03, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>> 
>> On 10.09.2020 14:09, Jan Beulich wrote:
>>> While looking at what it would take to move around libelf/
>>> in the hypervisor subtree, I've run into this rule, which I
>>> think can do with a few improvements and some simplification.
>>> 
>>> 1: adjust population of acpi/
>>> 2: fix (drop) dependencies of when to populate xen/
>>> 3: adjust population of public headers into xen/
>>> 4: properly install Arm public headers
>>> 5: adjust x86-specific population of xen/
>>> 6: drop remaining -f from ln invocations
>> 
>> May I ask for an ack or otherwise here?
> 
> This is going the right way but with this serie (on top of current staging
> status), I have a compilation error in Yocto while compiling qemu:
> In file included from 
> /media/extend-drive/bermar01/Development/xen-dev/yocto-build/build/dom0-fvp.prj/tmp/work/armv8a-poky-linux/qemu/5.1.0-r0/recipe-sysroot/usr/include/xenguest.h:25,
> |                  from 
> /media/extend-drive/bermar01/Development/xen-dev/yocto-build/build/dom0-fvp.prj/tmp/work/armv8a-poky-linux/qemu/5.1.0-r0/qemu-5.1.0/hw/i386/xen/xen_platform.c:41:
> | 
> /media/extend-drive/bermar01/Development/xen-dev/yocto-build/build/dom0-fvp.prj/tmp/work/armv8a-poky-linux/qemu/5.1.0-r0/recipe-sysroot/usr/include/xenctrl_dom.h:19:10:
>  fatal error: xen/libelf/libelf.h: No such file or directory
> |    19 | #include <xen/libelf/libelf.h>
> |       |          ^~~~~~~~~~~~~~~~~~~~~
> | compilation terminated.
> | 
> /media/extend-drive/bermar01/Development/xen-dev/yocto-build/build/dom0-fvp.prj/tmp/work/armv8a-poky-linux/qemu/5.1.0-r0/qemu-5.1.0/rules.mak:69:
>  recipe for target 'hw/i386/xen/xen_platform.o’ failed
> 
> Xen is using xenctrl_dom.h which need the libelf.h header from xen.

Actually this is not coming from your serie and this is actually a problem 
already present on master.

Regards
Bertrand