Fyi, This is not a k3 only issue. We fix this for pretty much all ti products. We've fixed this for years on our own layer by defining our own wic image which removes the /boot.
Setting --no-fstab-update wouldn't break our solution, though i do wonder if it's too heavy handed. https://support.criticallink.com/gitweb/?p=meta-mitysom.git;a=blob;f=wic/sdimage-bootpart-nomountboot.wks;h=1e2c3927de62cd33382733fd4fdd24047a7d8fd9;hb=refs/heads/kirkstone https://support.criticallink.com/gitweb/?p=meta-mitysom.git;a=blob;f=conf/machine/mitysom.inc;h=85256ad4c9039af390411e9811f6f49e570269d4;hb=refs/heads/kirkstone#l3 Ps. Not sure if anyone from meta-edgeai is on here but why would you rename the boot partition to BOOT and change the rootfs name... Anyways I digress. On Sat, Apr 20, 2024, 11:12 PM Chirag Shilwant via lists.yoctoproject.org <[email protected]> wrote: > + Andreas > On 21/04/24 08:37, Denys Dmytriyenko wrote: > > On Sat, Apr 20, 2024 at 06:23:18PM +0530, Chirag Shilwant wrote: > >> - Ideally running `ls /boot/` in filesystem should list contents from > /rootfs/boot partition > >> but currently it lists the contents of /boot partition of the WIC Image. > >> > >> Eg - Incase of AM64x, > >> ``` > >> root@am64xx-evm:~# ls /boot/ > >> EFI Image tiboot3-am64x-gp-evm.bin > tiboot3-am64x_sr2-hs-evm.bin tiboot3-am64x_sr2-hs-fs-evm.bin tiboot3.bin > tispl.bin u-boot.img uEnv.txt > > Will this change affect EFI? As there's EFI directory in that partition. > > The EFI partition under /rootfs/boot will still be present. > Following is the output of `ls /boot` with this patch, > > ``` > root@am64xx-evm:~# ls /boot > EFI Image Image-6.1.80-ti-g2e423244f8c0 dtb fitImage > fitImage-6.1.80-ti-g2e423244f8c0 > > ``` > > > > > > > >> ``` > >> > >> - To fix this issue, add --no-fstab-update flag to WIC_CREATE_EXTRA_ARG > in arago-image.inc > >> to prevent WIC image from adding an additional entry in /etc/fstab. > > This flag is global and will prevent all other partitions from populating > > /etc/fstab properly. Sure, meta-ti only has /boot and / defined, but > other > > layers could have own wic files with e.g. /home or /data partitions. > Mounting > > those will require a custom /etc/fstab instead of wic creating > corresponding > > entries automatically. > > > > > Apparently, this change was being carried in processor sdk layer from > quite a long time (from dunfell) to resolve a customer issue reported by > Andreas. > Diving into our internal JIRA system showed that this fix was suggested > by Andreas > & was resolved by the SDK team back then by keeping this change in > processor sdk layer [1]. > Not sure why this was in processor sdk layer though. Maybe Andreas can > provide additional details. > > Btw, I also see other SOC vendors (meta-freescale) adding > --no-fstab-update flag to WIC_CREATE_EXTRA_ARG [2] > So, if we are planning to pull in this change should it go somewhere in > meta-ti (maybe k3.inc [3])? > > [1]: > > https://git.ti.com/cgit/processor-sdk/meta-processor-sdk/commit/recipes-core/images/tisdk-default-image.bbappend?h=dunfell&id=58975a30e8f90efbffb97e49d5baac02971644da > [2]: > > https://git.yoctoproject.org/meta-freescale/tree/conf/machine/include/imx-base.inc?h=scarthgap#n614 > [3]: > > https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/conf/machine/include/k3.inc?h=scarthgap > > > >> Signed-off-by: Chirag Shilwant <[email protected]> > >> --- > >> meta-arago-distro/recipes-core/images/arago-image.inc | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >> diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc > b/meta-arago-distro/recipes-core/images/arago-image.inc > >> index 00308b57..2a34a709 100644 > >> --- a/meta-arago-distro/recipes-core/images/arago-image.inc > >> +++ b/meta-arago-distro/recipes-core/images/arago-image.inc > >> @@ -31,4 +31,6 @@ BAD_RECOMMENDATIONS += " \ > >> " > >> BAD_RECOMMENDATIONS += "${@oe.utils.conditional("INIT_MANAGER", > "sysvinit", "", "busybox-syslog", d)}" > >> > >> +WIC_CREATE_EXTRA_ARGS += " --no-fstab-update" > >> + > >> inherit core-image remove-net-rules > >> -- > >> 2.34.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#15301): https://lists.yoctoproject.org/g/meta-arago/message/15301 Mute This Topic: https://lists.yoctoproject.org/mt/105635090/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
