diff mbox series

[master,kirkstone] conf: include TI_COMMON_DEPLOY in PSEUDO_IGNORE_PATHS

Message ID 20231020121530.351358-1-matthias.schiffer@ew.tq-group.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [master,kirkstone] conf: include TI_COMMON_DEPLOY in PSEUDO_IGNORE_PATHS | expand

Commit Message

Matthias Schiffer Oct. 20, 2023, 12:15 p.m. UTC
Since the introduction of TI_COMMON_DEPLOY, we've been seeing frequent
Pseudo errors [1] breaking image build during development, in particular
with partial rebuilds due to the externalsrc bbclass.

The relevant part of the error log seems to be the following:

    ERROR: Task (/home/schifferm/Devel/src/ci-meta-tq-kirkstone/sources/meta
        -tq/meta-dumpling/recipes-images/images/tq-image-generic-debug.bb:do
	_rootfs) failed with exit code '1'
    Pseudo log:
    path mismatch [1 link]: ino 12215614 db '/home/schifferm/Devel/src/ci-me
        ta-tq-kirkstone/build-mba64xx/deploy-ti/images/tqma64xxl-mbax4xxl/tq
	-image-generic-debug-tqma64xxl-mbax4xxl-20231020091917.testdata.json
	' req '/home/schifferm/Devel/src/ci-meta-tq-kirkstone/build-mba64xx/
	tmp/work/tqma64xxl_mbax4xxl-tq-linux/tq-image-generic-debug/1.0-r0/r
	ootfs/usr/share/stress-ng/example-jobs/device.job;65324e26'.

My understanding is that all deploy dirs should be ignored by Pseudo, as
openembedded-core/meta/bitbake.conf includes the whole of DEPLOY_DIR in
PSEUDO_IGNORE_PATHS. With the introduction of TI_COMMON_DEPLOY,
DEPLOY_DIR_IMAGE is not covered by that setting anymore, so we add it in
meta-ti to avoid the issue.

[1] https://wiki.yoctoproject.org/wiki/Pseudo_Abort

Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>
---
 meta-ti-bsp/conf/layer.conf | 1 +
 1 file changed, 1 insertion(+)

Comments

Denys Dmytriyenko Oct. 20, 2023, 4:35 p.m. UTC | #1
On Fri, Oct 20, 2023 at 02:15:30PM +0200, Matthias Schiffer wrote:
> Since the introduction of TI_COMMON_DEPLOY, we've been seeing frequent
> Pseudo errors [1] breaking image build during development, in particular
> with partial rebuilds due to the externalsrc bbclass.

Indeed! We've been chasing a similar issue with TISDK bundle in Arago (which 
is a tarball of several images, SDKs and other prebuilt stuff). It was more 
severe though and would always fail on the second run - I posted a cleanup 
patch to address that and thought it was the end of it...

But I just saw this issue pop up in a different place last night - in a 
regular image do_rootfs. Like you said, it was a partial rebuild due to 
some changes in the dependency.

Somehow I completely missed PSEUDO_IGNORE_PATHS setting! A bit heavy-handed, 
as it would have masked our issues with TISDK bundle.


> The relevant part of the error log seems to be the following:
> 
>     ERROR: Task (/home/schifferm/Devel/src/ci-meta-tq-kirkstone/sources/meta
>         -tq/meta-dumpling/recipes-images/images/tq-image-generic-debug.bb:do
> 	_rootfs) failed with exit code '1'
>     Pseudo log:
>     path mismatch [1 link]: ino 12215614 db '/home/schifferm/Devel/src/ci-me
>         ta-tq-kirkstone/build-mba64xx/deploy-ti/images/tqma64xxl-mbax4xxl/tq
> 	-image-generic-debug-tqma64xxl-mbax4xxl-20231020091917.testdata.json
> 	' req '/home/schifferm/Devel/src/ci-meta-tq-kirkstone/build-mba64xx/
> 	tmp/work/tqma64xxl_mbax4xxl-tq-linux/tq-image-generic-debug/1.0-r0/r
> 	ootfs/usr/share/stress-ng/example-jobs/device.job;65324e26'.
> 
> My understanding is that all deploy dirs should be ignored by Pseudo, as
> openembedded-core/meta/bitbake.conf includes the whole of DEPLOY_DIR in
> PSEUDO_IGNORE_PATHS. With the introduction of TI_COMMON_DEPLOY,
> DEPLOY_DIR_IMAGE is not covered by that setting anymore, so we add it in
> meta-ti to avoid the issue.
> 
> [1] https://wiki.yoctoproject.org/wiki/Pseudo_Abort
> 
> Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>

Acked-by: Denys Dmytriyenko <denys@konsulko.com>


> ---
>  meta-ti-bsp/conf/layer.conf | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-ti-bsp/conf/layer.conf b/meta-ti-bsp/conf/layer.conf
> index 460be02a..ffa47ae6 100644
> --- a/meta-ti-bsp/conf/layer.conf
> +++ b/meta-ti-bsp/conf/layer.conf
> @@ -31,3 +31,4 @@ HOSTTOOLS_NONFATAL += "truncate xxd comm"
>  TI_COMMON_DEPLOY ?= "${TOPDIR}/deploy-ti"
>  DEPLOY_DIR_IMAGE:ti-soc = "${TI_COMMON_DEPLOY}/images/${MACHINE}"
>  SDK_DEPLOY:ti-soc = "${TI_COMMON_DEPLOY}/sdk"
> +PSEUDO_IGNORE_PATHS:append:ti-soc = ",${TI_COMMON_DEPLOY}"
> --
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/layer.conf b/meta-ti-bsp/conf/layer.conf
index 460be02a..ffa47ae6 100644
--- a/meta-ti-bsp/conf/layer.conf
+++ b/meta-ti-bsp/conf/layer.conf
@@ -31,3 +31,4 @@  HOSTTOOLS_NONFATAL += "truncate xxd comm"
 TI_COMMON_DEPLOY ?= "${TOPDIR}/deploy-ti"
 DEPLOY_DIR_IMAGE:ti-soc = "${TI_COMMON_DEPLOY}/images/${MACHINE}"
 SDK_DEPLOY:ti-soc = "${TI_COMMON_DEPLOY}/sdk"
+PSEUDO_IGNORE_PATHS:append:ti-soc = ",${TI_COMMON_DEPLOY}"