Hi,
On 25/04/24 10:32, Chirag Shilwant via lists.yoctoproject.org wrote:
Hi Denys, On 25/04/24 01: 40, Denys Dmytriyenko wrote: > On Wed, Apr
24, 2024 at 08: 32: 52AM -0500, Ryan Eatmon wrote: >> >> On 4/22/2024
5: 32 PM, Denys Dmytriyenko wrote: >>> On Sun, Apr 21, 2024 at
02: 42: 23PM +0530,
ZjQcmQRYFpfptBannerStart
This message was sent from outside of Texas Instruments.
Do not click links or open attachments unless you recognize the source
of this email and know the content is safe. If you wish to report this
message to IT Security, please forward the message as an attachment to
[email protected]
ZjQcmQRYFpfptBannerEnd
Hi Denys,
On 25/04/24 01:40, Denys Dmytriyenko wrote:
> On Wed, Apr 24, 2024 at 08:32:52AM -0500, Ryan Eatmon wrote:
>>
>> On 4/22/2024 5:32 PM, Denys Dmytriyenko wrote:
>>> On Sun, Apr 21, 2024 at 02:42:23PM +0530, Chirag Shilwant 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
>>>>
>>>> ```
>>> Thanks, I'll do some additional testing with EFI a bit later.
>>>
>>>
>>>>>> - 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.
>>> I tested and submitted a different fix last night to adjust WIC templates to
>>> not mount boot partition under /boot directory[1]. I still believe doing it
>>> just for that one partition is better than disabling fstab updates globally.
>>>
>>> [1]
https://urldefense.com/v3/__https://patchwork.yoctoproject.org/project/ti/patch/[email protected]/__;!!G3vK!Qwf1x-GVI8r-vE0PXQX0UPdf9Wvl5Dp5bgpXJk3tcBfm8Dae64HW2APer6EWUrW4uAhOK3m7xVvsOm_KrFX_7d-6nDuRMT8M2qlU$
>> So with the above referenced patch in meta-ti, I'm assuming we do
>> not need this patch in meta-arago?
Ryan, We can ignore this patch.
Since we have integrated Denys patch in meta-ti, I will go ahead
and remove WIC_CREATE_EXTRA_ARG from meta-tisdk as well [1]
[1]:
https://git.ti.com/cgit/ti-sdk-linux/meta-tisdk/tree/recipes-core/images?h=scarthgap
> That's my pitch here.
>
> Chirag,
>
> Will that work for you and others?
Let me kickoff a build with your meta-ti patch [1] & see if that works
for us as well.
Denys,
As you pitched, it works for us as well.
I don't see the boot binaries under /rootfs/boot with your meta-ti patch [1]
Running `ls /boot/` in filesystem now lists the following,
```
root@am64xx-evm:~# ls /boot/
EFI Image Image-6.1.80-ti-g2e423244f8c0 dtb fitImage
fitImage-6.1.80-ti-g2e423244f8c0
```
[1]:
https://patchwork.yoctoproject.org/project/ti/patch/[email protected]/
Will get back in a day or two.
[1]:
https://urldefense.com/v3/__https://patchwork.yoctoproject.org/project/ti/patch/[email protected]/__;!!G3vK!Qwf1x-GVI8r-vE0PXQX0UPdf9Wvl5Dp5bgpXJk3tcBfm8Dae64HW2APer6EWUrW4uAhOK3m7xVvsOm_KrFX_7d-6nDuRMT8M2qlU$
>
>
>>> Can you please take a look and maybe test this option for your use case?
>>> Thanks.
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15311):
https://lists.yoctoproject.org/g/meta-arago/message/15311
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]]
-=-=-=-=-=-=-=-=-=-=-=-
