On Tue, Mar 10, 2026 at 07:22 PM, Denys Dmytriyenko wrote: > > 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 <[email protected]> > > > > >--- > > > > > 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
Yeah you are right, that the default base_do_compile() calls oe_runmake, which would work fine. But, the jailhouse recipe inherits module bbclass, "inherit module python3native bash-completion deploy " which overrides default do_compile() with module_do_compile. resulting in compilation errors if do_compile override is removed. If I don't use the do-compile override I am still getting build errors. I am unable to remove the module bbclass is required for building jailhouse kernel modules. > > > > 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 >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#19692): https://lists.yoctoproject.org/g/meta-ti/message/19692 Mute This Topic: https://lists.yoctoproject.org/mt/118225330/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
