[master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings

All of lore.kernel.org
 help / color / mirror / Atom feed

* [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings
@ 2026-03-11 19:05 Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path Mahammed Sadik Shaik
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Mahammed Sadik Shaik @ 2026-03-11 19:05 UTC (permalink / raw)
  To: meta-ti; +Cc: Ryan Eatmon, Prudhvi Veeramachaneni, Mahammed Sadik Shaik

This series fixes build failures and QA warnings in the jailhouse recipes:

1. Fix JH_RAMFS image path to match current naming convention
2. Fix buildpaths QA warning and remove do_compile override
3. Inhibit debug package split for bare-metal inmate binaries

These changes resolve build failures when searching for initramfs images
and eliminate QA warnings related to buildpaths and debug packages.

v3 <-> v4:
  - Fix EXTRA_OEMAKE and USER_SPACE_CFLAGS formatting per Ryan's feedback

Link to V3-:https://lists.yoctoproject.org/g/meta-ti/message/19701

v2 <-> v3:
  - Move V=1 into EXTRA_OEMAKE as suggested by Denys
  - Switch from 'module' to 'module-base' + 'kernel-module-split' inheritance
  - Remove setuptools3 inherit (not needed)
  - Remove do_compile() override as suggested by Denys

Link to V2-:https://lists.yoctoproject.org/g/meta-ti/message/19680

v1 <-> v2:
  - Split buildpaths fix into separate commit
  - Improved commit message descriptions
  - Added KCFLAGS with -ffile-prefix-map for reproducible builds

Link to V1-:https://lists.yoctoproject.org/g/meta-ti/message/19674

Mahammed Sadik Shaik (3):
  jailhouse: Fix JH_RAMFS image path
  jailhouse-inmate: Inhibit debug package split
  ti-jailhouse: Fix buildpaths QA warning and cleanup

 .../recipes-ti/jailhouse/jailhouse-inmate.bb  |  3 ++
 .../recipes-ti/jailhouse/jailhouse_git.bb     |  8 ++---
 .../recipes-ti/jailhouse/ti-jailhouse.inc     | 36 +++++++++++--------
 3 files changed, 28 insertions(+), 19 deletions(-)

-- 
2.34.1



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path
  2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
@ 2026-03-11 19:05 ` Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 2/3] jailhouse-inmate: Inhibit debug package split Mahammed Sadik Shaik
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Mahammed Sadik Shaik @ 2026-03-11 19:05 UTC (permalink / raw)
  To: meta-ti; +Cc: Ryan Eatmon, Prudhvi Veeramachaneni, Mahammed Sadik Shaik

Update JH_RAMFS_IMAGE references to use correct .cpio
extension instead of .rootfs.cpio to match current image
naming convention

Signed-off-by: Mahammed Sadik Shaik <s-sadik@ti.com>
---
 meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
index 0113a916..7d908215 100644
--- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
+++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
@@ -55,11 +55,11 @@ do_install() {
 	install -d ${D}/boot
 	if [ -n "${JH_RAMFS_IMAGE}" ]
 	then
-		if [ -f ${DEPLOY_DIR_IMAGE}/${JH_RAMFS_IMAGE}*-${MACHINE}.rootfs.cpio ]
+		if [ -f ${DEPLOY_DIR_IMAGE}/${JH_RAMFS_IMAGE}*-${MACHINE}.cpio ]
 		then
-			install -m 0644 ${DEPLOY_DIR_IMAGE}/${JH_RAMFS_IMAGE}*-${MACHINE}.rootfs.cpio ${D}/boot
+			install -m 0644 ${DEPLOY_DIR_IMAGE}/${JH_RAMFS_IMAGE}*-${MACHINE}.cpio ${D}/boot
 		else
-			bberror "Could not find JH_RAMFS_IMAGE (${JH_RAMFS_IMAGE}*-${MACHINE}.rootfs.cpio)!"
+			bberror "Could not find JH_RAMFS_IMAGE (${JH_RAMFS_IMAGE}*-${MACHINE}.cpio)!"
 			bberror "Please make sure that \"cpio\" is in IMAGE_FSTYPES."
 		fi
 	fi
@@ -72,7 +72,7 @@ do_install() {
 		./jailhouse-cell-linux -w ${D}${JH_DATADIR}/${JH_INMATE_DTB} \
 			-a ${JH_ARCH} -c "${JH_CMDLINE}" \
 			-d ../configs/${JH_ARCH}/dts/${JH_INMATE_DTB} \
-			-i ${D}/boot/${JH_RAMFS_IMAGE}*-${MACHINE}.rootfs.cpio \
+			-i ${D}/boot/${JH_RAMFS_IMAGE}*-${MACHINE}.cpio \
 			${D}${CELL_DIR}/${JH_LINUX_DEMO_CELL} \
 			${DEPLOY_DIR_IMAGE}/Image \
 			| tr -cd '\11\12\15\40-\176' \
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [master][PATCH v4 2/3] jailhouse-inmate: Inhibit debug package split
  2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path Mahammed Sadik Shaik
@ 2026-03-11 19:05 ` Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 3/3] ti-jailhouse: Fix buildpaths QA warning and cleanup Mahammed Sadik Shaik
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Mahammed Sadik Shaik @ 2026-03-11 19:05 UTC (permalink / raw)
  To: meta-ti; +Cc: Ryan Eatmon, Prudhvi Veeramachaneni, Mahammed Sadik Shaik

Inmate binaries are bare-metal applications without standard debug
symbols that can be meaningfully split. Inhibit debug package
creation to avoid unnecessary packaging overhead and potential
QA warnings.

Signed-off-by: Mahammed Sadik Shaik <s-sadik@ti.com>
---
 meta-ti-extras/recipes-ti/jailhouse/jailhouse-inmate.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse-inmate.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse-inmate.bb
index 39654a53..c6b3b85b 100644
--- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse-inmate.bb
+++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse-inmate.bb
@@ -1,5 +1,8 @@
 require ti-jailhouse.inc
 
+# Inmate binaries are bare-metal; no debug package needed
+INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
+
 do_install() {
 
 	install -d ${D}${bindir}
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [master][PATCH v4 3/3] ti-jailhouse: Fix buildpaths QA warning and cleanup
  2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path Mahammed Sadik Shaik
  2026-03-11 19:05 ` [master][PATCH v4 2/3] jailhouse-inmate: Inhibit debug package split Mahammed Sadik Shaik
@ 2026-03-11 19:05 ` Mahammed Sadik Shaik
  2026-03-11 19:19 ` [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings PRC Automation
  2026-03-11 20:07 ` [meta-ti] " Denys Dmytriyenko
  4 siblings, 0 replies; 6+ messages in thread
From: Mahammed Sadik Shaik @ 2026-03-11 19:05 UTC (permalink / raw)
  To: meta-ti; +Cc: Ryan Eatmon, Prudhvi Veeramachaneni, Mahammed Sadik Shaik

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.

inherit module-base kernel-module-split for building kernel modules

move V=1 into EXTRA_OEMAKE and remove overriding do_compile()

This resolves buildpaths QA warnings during packaging.

Signed-off-by: Mahammed Sadik Shaik <s-sadik@ti.com>
---
 .../recipes-ti/jailhouse/ti-jailhouse.inc     | 36 +++++++++++--------
 1 file changed, 21 insertions(+), 15 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..08c2814e 100644
--- a/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
+++ b/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc
@@ -47,7 +47,7 @@ B = "${S}"
 
 DEPENDS = "virtual/kernel dtc-native python3-mako-native python3-mako make-native"
 
-inherit module python3native bash-completion deploy setuptools3
+inherit module-base kernel-module-split python3native bash-completion deploy
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 COMPATIBLE_MACHINE = "(ti-soc)"
@@ -66,22 +66,28 @@ JH_SYSCONFIG_CELL:am62lxx ?= "k3-am62l3-evm.cell"
 JH_SYSCONFIG_CELL:j7 ?= "k3-j721e-evm.cell"
 JH_SYSCONFIG_CELL:j7200-evm ?= "k3-j7200-evm.cell"
 
-do_configure() {
-        if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ];
-        then
-            cp ${STAGING_DIR_HOST}/${CELLCONF_DIR}/*.c ${S}/configs/
-        fi
-}
+EXTRA_OEMAKE = "\
+    V=1 \
+    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}/=' \
+"
 
-USER_SPACE_CFLAGS = '${CFLAGS} -DLIBEXECDIR=\\\"${libexecdir}\\\" \
-                    -DJAILHOUSE_VERSION=\\\"$JAILHOUSE_VERSION\\\" \
-                    -Wall -Wextra -Wmissing-declarations -Wmissing-prototypes -Werror \
-                    -I../driver'
+USER_SPACE_CFLAGS = "\
+    ${CFLAGS} \
+    -DLIBEXECDIR=\\\"${libexecdir}\\\" \
+    -DJAILHOUSE_VERSION=\\\"$JAILHOUSE_VERSION\\\" \
+    -Wall -Wextra -Wmissing-declarations -Wmissing-prototypes -Werror \
+    -I../driver \
+"
 
 TOOLS_SRC_DIR = "${S}/tools"
 
-EXTRA_OEMAKE = "ARCH=${JH_ARCH} CROSS_COMPILE=${TARGET_PREFIX} CC="${CC}" KDIR=${STAGING_KERNEL_BUILDDIR}"
-
-do_compile() {
-        oe_runmake V=1
+do_configure() {
+    if [ -d ${STAGING_DIR_HOST}/${CELLCONF_DIR} ]; then
+        cp ${STAGING_DIR_HOST}/${CELLCONF_DIR}/*.c ${S}/configs/
+    fi
 }
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings
  2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
                   ` (2 preceding siblings ...)
  2026-03-11 19:05 ` [master][PATCH v4 3/3] ti-jailhouse: Fix buildpaths QA warning and cleanup Mahammed Sadik Shaik
@ 2026-03-11 19:19 ` PRC Automation
  2026-03-11 20:07 ` [meta-ti] " Denys Dmytriyenko
  4 siblings, 0 replies; 6+ messages in thread
From: PRC Automation @ 2026-03-11 19:19 UTC (permalink / raw)
  To: Mahammed Sadik Shaik; +Cc: meta-ti, Ryan Eatmon, Prudhvi Veeramachaneni

meta-ti / na / 20260311190537.3340422-1-s-sadik

PRC Results: PASS

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
WARN - [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path
    WARN: Revised patch missing a comment. (REVISION-1)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
WARN - [master][PATCH v4 2/3] jailhouse-inmate: Inhibit debug package split
    WARN: Revised patch missing a comment. (REVISION-1)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
WARN - [master][PATCH v4 3/3] ti-jailhouse: Fix buildpaths QA warning and cleanup
    WARN: Revised patch missing a comment. (REVISION-1)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887



=========================================================
  apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings
- Submitter: From: Mahammed Sadik Shaik <s-sadik@ti.com>
- Date: Date: Thu, 12 Mar 2026 00:35:34 +0530
- Num Patches: 3
- Mailing List (public inbox) Commit SHA: 64e82420e6f9ab1890df7679aa20d92ae8d44fce

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: master-wip
- Commit Author: Ryan Eatmon <reatmon@ti.com>
- Commit Subject: conf: Add falcon support natively
- Commit SHA: 2976bdb6413a80438ce1fe7d4056105f6385f019

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
master
=====================
PASS



=========================================================
  yocto-check-layers: PASS
=========================================================
master - PASS
=====================
All checks passed





^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-ti] [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings
  2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
                   ` (3 preceding siblings ...)
  2026-03-11 19:19 ` [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings PRC Automation
@ 2026-03-11 20:07 ` Denys Dmytriyenko
  4 siblings, 0 replies; 6+ messages in thread
From: Denys Dmytriyenko @ 2026-03-11 20:07 UTC (permalink / raw)
  To: s-sadik; +Cc: meta-ti, Ryan Eatmon, Prudhvi Veeramachaneni

On Thu, Mar 12, 2026 at 12:35:34AM +0530, Sadik via lists.yoctoproject.org wrote:
> This series fixes build failures and QA warnings in the jailhouse recipes:
> 
> 1. Fix JH_RAMFS image path to match current naming convention
> 2. Fix buildpaths QA warning and remove do_compile override
> 3. Inhibit debug package split for bare-metal inmate binaries
> 
> These changes resolve build failures when searching for initramfs images
> and eliminate QA warnings related to buildpaths and debug packages.
> 
> v3 <-> v4:
>   - Fix EXTRA_OEMAKE and USER_SPACE_CFLAGS formatting per Ryan's feedback

LGTM - Ack for the series.


> Link to V3-:https://lists.yoctoproject.org/g/meta-ti/message/19701
> 
> v2 <-> v3:
>   - Move V=1 into EXTRA_OEMAKE as suggested by Denys
>   - Switch from 'module' to 'module-base' + 'kernel-module-split' inheritance
>   - Remove setuptools3 inherit (not needed)
>   - Remove do_compile() override as suggested by Denys
> 
> Link to V2-:https://lists.yoctoproject.org/g/meta-ti/message/19680
> 
> v1 <-> v2:
>   - Split buildpaths fix into separate commit
>   - Improved commit message descriptions
>   - Added KCFLAGS with -ffile-prefix-map for reproducible builds
> 
> Link to V1-:https://lists.yoctoproject.org/g/meta-ti/message/19674
> 
> Mahammed Sadik Shaik (3):
>   jailhouse: Fix JH_RAMFS image path
>   jailhouse-inmate: Inhibit debug package split
>   ti-jailhouse: Fix buildpaths QA warning and cleanup
> 
>  .../recipes-ti/jailhouse/jailhouse-inmate.bb  |  3 ++
>  .../recipes-ti/jailhouse/jailhouse_git.bb     |  8 ++---
>  .../recipes-ti/jailhouse/ti-jailhouse.inc     | 36 +++++++++++--------
>  3 files changed, 28 insertions(+), 19 deletions(-)
> 
> -- 
> 2.34.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-03-11 20:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-11 19:05 [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings Mahammed Sadik Shaik
2026-03-11 19:05 ` [master][PATCH v4 1/3] jailhouse: Fix JH_RAMFS image path Mahammed Sadik Shaik
2026-03-11 19:05 ` [master][PATCH v4 2/3] jailhouse-inmate: Inhibit debug package split Mahammed Sadik Shaik
2026-03-11 19:05 ` [master][PATCH v4 3/3] ti-jailhouse: Fix buildpaths QA warning and cleanup Mahammed Sadik Shaik
2026-03-11 19:19 ` [master][PATCH v4 0/3] jailhouse: Fix build issues and QA warnings PRC Automation
2026-03-11 20:07 ` [meta-ti] " Denys Dmytriyenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.