Re: [meta-ti] [master][PATCH v2 3/3] ti-jailhouse: Fix buildpaths QA warning

[Denys Dmytriyenko <denis@denix.org>]

On Tue, Mar 10, 2026 at 01:58:45AM -0700, Sadik via lists.yoctoproject.org wrote:
> On Tue, Mar 10, 2026 at 12:28 AM, Denys Dmytriyenko wrote:
> 
> >
> > On Mon, Mar 09, 2026 at 12:08:36PM -0500, Ryan Eatmon via
> > lists.yoctoproject.org wrote:
> > > 
> > > 
> > > On 3/9/2026 12:01 PM, Mahammed Sadik Shaik wrote:
> > > >Add -ffile-prefix-map flags to KCFLAGS to remove build path
> > > >references from kernel module compilation, ensuring reproducible
> > > >builds and preventing absolute path leakage in binaries.
> > > >
> > > >This resolves buildpaths QA warnings during packaging.
> > > >
> > > >Signed-off-by: Mahammed Sadik Shaik <s-sadik@ti.com>
> > > >---
> > > >  meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc | 4 ++--
> > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > >
> > > >diff --git a/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
> > b/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
> > > >index 91076c9e..8308f0e5 100644
> > > >--- a/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
> > > >+++ b/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
> > > >@@ -80,8 +80,8 @@ USER_SPACE_CFLAGS = '${CFLAGS}
> > -DLIBEXECDIR=\\\"${libexecdir}\\\" \
> > > >  TOOLS_SRC_DIR = "${S}/tools"
> > > >-EXTRA_OEMAKE = "ARCH=${JH_ARCH} CROSS_COMPILE=${TARGET_PREFIX} CC="${CC}"
> > KDIR=${STAGING_KERNEL_BUILDDIR}"
> > > >-
> > > >+EXTRA_OEMAKE = "ARCH=${JH_ARCH} CROSS_COMPILE=${TARGET_PREFIX} CC="${CC}"
> > KDIR=${STAGING_KERNEL_BUILDDIR} \
> > > >+    KCFLAGS='-ffile-prefix-map=${STAGING_KERNEL_BUILDDIR}/=
> > -ffile-prefix-map=${STAGING_KERNEL_DIR}/='"
> > > >  do_compile() {
> > > >          oe_runmake V=1
> > > >  }
> > 
> > Can you move V=1 into EXTRA_OEMAKE and remove overriding do_compile()
> 
> Thanks for the review Denys, Apologies for multiple replies was facing some issues with relply all in mail.
> 
> Moving V=1 into EXTRA_OEMAKE is fine, but we can't remove the do_compile() override. . This is insufficient for jailhouse, which requires building:
> 1. Kernel modules (driver/jailhouse.ko)
> 2. Hypervisor binary (hypervisor/jailhouse.bin)
> 3. Userspace tools (tools/jailhouse, jailhouse-*)
> 4. Inmate demos (inmates/demos/*) — causing the install failure for all the demos during the build.
> 
> The do_compile() override is needed to run a plain oe_runmake which hits jailhouse's top-level Makefile and builds the full project.

The default do_compile() does exactly that - calls oe_runmake:
https://git.openembedded.org/openembedded-core/tree/meta/classes-global/base.bbclass#n399


> If necessary I can send an V3 by moving V=1 into EXTRA_OEMAKE, but that doesn't serve our purpose of completely overriding the do_compile
> And also I couldn't find an existing bbclass that does "run the top-level Makefile and build everything."
> 
> > 
> > 
> > > Thank you.  I was going to comment on the v1 to not disable buildpaths.
> > 
> > +1