| Message ID | 20250728165437.571427-3-afd@ti.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-ti,scarthgap/master,1/3] wic: sdimage-2part-efi: Add rootwait kernel param | expand |
On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > For all our platforms booting with EFI there is only one option today. > The timeout gives the user 3 seconds to choose a different option, but > as there is only one, this just causes a needless 3 second delay in > boot time. Remove this delay and boot the only option immediately. > If we ever have more than one option we can always add this back. Well, menu delay in grub is not only used for selecting among available predefined options, but also for interactively editing the command line on the fly before booting. > Signed-off-by: Andrew Davis <afd@ti.com> > --- > meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > index 7252411c..c1089762 100644 > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > @@ -3,6 +3,6 @@ > # supports EFI. Boot files are located in the first vfat partition with extra > # reserved space. We cannot use a GPT here. > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > -- > 2.39.2
On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: > > On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > > For all our platforms booting with EFI there is only one option today. > > The timeout gives the user 3 seconds to choose a different option, but > > as there is only one, this just causes a needless 3 second delay in > > boot time. Remove this delay and boot the only option immediately. > > If we ever have more than one option we can always add this back. > > Well, menu delay in grub is not only used for selecting among available > predefined options, but also for interactively editing the command line > on the fly before booting. > for development, this sounds good, for production not so much. > > > Signed-off-by: Andrew Davis <afd@ti.com> > > --- > > meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > index 7252411c..c1089762 100644 > > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > @@ -3,6 +3,6 @@ > > # supports EFI. Boot files are located in the first vfat partition with extra > > # reserved space. We cannot use a GPT here. > > > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > > part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > > -- > > 2.39.2 > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#18840): https://lists.yoctoproject.org/g/meta-ti/message/18840 > Mute This Topic: https://lists.yoctoproject.org/mt/114416911/1997914 > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: > On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via > lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: > > > > On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > > > For all our platforms booting with EFI there is only one option today. > > > The timeout gives the user 3 seconds to choose a different option, but > > > as there is only one, this just causes a needless 3 second delay in > > > boot time. Remove this delay and boot the only option immediately. > > > If we ever have more than one option we can always add this back. > > > > Well, menu delay in grub is not only used for selecting among available > > predefined options, but also for interactively editing the command line > > on the fly before booting. > > > > for development, this sounds good, for production not so much. Sure. And we also disable e.g. U-boot boot prompt delay all the time for our customers during production hardening stage, among many other similar items. But it is done for the final product - I don't think it should be done in a generic BSP layer that many use for development. > > > Signed-off-by: Andrew Davis <afd@ti.com> > > > --- > > > meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > index 7252411c..c1089762 100644 > > > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > @@ -3,6 +3,6 @@ > > > # supports EFI. Boot files are located in the first vfat partition with extra > > > # reserved space. We cannot use a GPT here. > > > > > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > > > part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > > > -- > > > 2.39.2
On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote: > > On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: > > On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via > > lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: > > > > > > On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > > > > For all our platforms booting with EFI there is only one option today. > > > > The timeout gives the user 3 seconds to choose a different option, but > > > > as there is only one, this just causes a needless 3 second delay in > > > > boot time. Remove this delay and boot the only option immediately. > > > > If we ever have more than one option we can always add this back. > > > > > > Well, menu delay in grub is not only used for selecting among available > > > predefined options, but also for interactively editing the command line > > > on the fly before booting. > > > > > > > for development, this sounds good, for production not so much. > > Sure. And we also disable e.g. U-boot boot prompt delay all the time for our > customers during production hardening stage, among many other similar items. > But it is done for the final product - I don't think it should be done in a > generic BSP layer that many use for development. Right, it could be documented somewhere in README or some such would help the consumers of the layer. > > > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > > > --- > > > > meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > > index 7252411c..c1089762 100644 > > > > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > > > > @@ -3,6 +3,6 @@ > > > > # supports EFI. Boot files are located in the first vfat partition with extra > > > > # reserved space. We cannot use a GPT here. > > > > > > > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > > part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > > > > part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > > > > -- > > > > 2.39.2
On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote: > On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote: >> >> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: >>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via >>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: >>>> >>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: >>>>> For all our platforms booting with EFI there is only one option today. >>>>> The timeout gives the user 3 seconds to choose a different option, but >>>>> as there is only one, this just causes a needless 3 second delay in >>>>> boot time. Remove this delay and boot the only option immediately. >>>>> If we ever have more than one option we can always add this back. >>>> >>>> Well, menu delay in grub is not only used for selecting among available >>>> predefined options, but also for interactively editing the command line >>>> on the fly before booting. >>>> >>> >>> for development, this sounds good, for production not so much. >> >> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our >> customers during production hardening stage, among many other similar items. >> But it is done for the final product - I don't think it should be done in a >> generic BSP layer that many use for development. > > Right, it could be documented somewhere in README or some such would help > the consumers of the layer. Is there already something in the system that we could use to drive a variable that we can stick into the file? Like EXTRA_IMAGE_FEATURES? I know there used to be a debug-tweaks that got broken out into multiple features. Maybe create a new one? TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", "wks-debug-timeout", "3", "0 ",d)}' bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" Would that work? >> >> >>>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>>> --- >>>>> meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>> index 7252411c..c1089762 100644 >>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>> @@ -3,6 +3,6 @@ >>>>> # supports EFI. Boot files are located in the first vfat partition with extra >>>>> # reserved space. We cannot use a GPT here. >>>>> >>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" >>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" >>>>> part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M >>>>> part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid >>>>> -- >>>>> 2.39.2 >>>>> >>>>> >>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>> Links: You receive all messages sent to this group. >>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848 >>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054 >>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org >>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com] >>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>>
On Mon, Jul 28, 2025 at 1:33 PM Ryan Eatmon <reatmon@ti.com> wrote: > > > > On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote: > > On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote: > >> > >> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: > >>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via > >>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: > >>>> > >>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > >>>>> For all our platforms booting with EFI there is only one option today. > >>>>> The timeout gives the user 3 seconds to choose a different option, but > >>>>> as there is only one, this just causes a needless 3 second delay in > >>>>> boot time. Remove this delay and boot the only option immediately. > >>>>> If we ever have more than one option we can always add this back. > >>>> > >>>> Well, menu delay in grub is not only used for selecting among available > >>>> predefined options, but also for interactively editing the command line > >>>> on the fly before booting. > >>>> > >>> > >>> for development, this sounds good, for production not so much. > >> > >> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our > >> customers during production hardening stage, among many other similar items. > >> But it is done for the final product - I don't think it should be done in a > >> generic BSP layer that many use for development. > > > > Right, it could be documented somewhere in README or some such would help > > the consumers of the layer. > > Is there already something in the system that we could use to drive a > variable that we can stick into the file? Like EXTRA_IMAGE_FEATURES? I > know there used to be a debug-tweaks that got broken out into multiple > features. Maybe create a new one? > > TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", > "wks-debug-timeout", "3", "0 ",d)}' > > bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 > rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > Would that work? probably, it can make it easier for sure. > > > >> > >> > >>>>> Signed-off-by: Andrew Davis <afd@ti.com> > >>>>> --- > >>>>> meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>> > >>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>> index 7252411c..c1089762 100644 > >>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>> @@ -3,6 +3,6 @@ > >>>>> # supports EFI. Boot files are located in the first vfat partition with extra > >>>>> # reserved space. We cannot use a GPT here. > >>>>> > >>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > >>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > >>>>> part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > >>>>> part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > >>>>> -- > >>>>> 2.39.2 > >>>>> > >>>>> > >>>>> -=-=-=-=-=-=-=-=-=-=-=- > >>>>> Links: You receive all messages sent to this group. > >>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848 > >>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054 > >>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org > >>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com] > >>>>> -=-=-=-=-=-=-=-=-=-=-=- > >>>>> > > -- > Ryan Eatmon reatmon@ti.com > ----------------------------------------- > Texas Instruments, Inc. - LCPD - MGTS
On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote: > > > On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote: > >On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote: > >> > >>On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: > >>>On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via > >>>lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: > >>>> > >>>>On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > >>>>>For all our platforms booting with EFI there is only one option today. > >>>>>The timeout gives the user 3 seconds to choose a different option, but > >>>>>as there is only one, this just causes a needless 3 second delay in > >>>>>boot time. Remove this delay and boot the only option immediately. > >>>>>If we ever have more than one option we can always add this back. > >>>> > >>>>Well, menu delay in grub is not only used for selecting among available > >>>>predefined options, but also for interactively editing the command line > >>>>on the fly before booting. > >>>> > >>> > >>>for development, this sounds good, for production not so much. > >> > >>Sure. And we also disable e.g. U-boot boot prompt delay all the time for our > >>customers during production hardening stage, among many other similar items. > >>But it is done for the final product - I don't think it should be done in a > >>generic BSP layer that many use for development. > > > >Right, it could be documented somewhere in README or some such would help > >the consumers of the layer. > > Is there already something in the system that we could use to drive > a variable that we can stick into the file? Like > EXTRA_IMAGE_FEATURES? I know there used to be a debug-tweaks that > got broken out into multiple features. Maybe create a new one? > > TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", > "wks-debug-timeout", "3", "0 ",d)}' Should probably make it more generic, not wks. I'd imagine it can be shared for U-boot and EFI/GRUB, so maybe bootloader-timeout or something? > bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 > rootwait ${TI_WKS_BOOTLOADER_APPEND}" > > > Would that work? > > > >> > >> > >>>>>Signed-off-by: Andrew Davis <afd@ti.com> > >>>>>--- > >>>>> meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>> > >>>>>diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>>index 7252411c..c1089762 100644 > >>>>>--- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>>+++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in > >>>>>@@ -3,6 +3,6 @@ > >>>>> # supports EFI. Boot files are located in the first vfat partition with extra > >>>>> # reserved space. We cannot use a GPT here. > >>>>> > >>>>>-bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > >>>>>+bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" > >>>>> part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M > >>>>> part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid > >>>>>-- > >>>>>2.39.2 > >>>>> > >>>>> > >>>>>-=-=-=-=-=-=-=-=-=-=-=- > >>>>>Links: You receive all messages sent to this group. > >>>>>View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848 > >>>>>Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054 > >>>>>Group Owner: meta-ti+owner@lists.yoctoproject.org > >>>>>Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com] > >>>>>-=-=-=-=-=-=-=-=-=-=-=- > >>>>>
On 7/28/25 5:59 PM, Denys Dmytriyenko wrote: > On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote: >> >> >> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote: >>> On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote: >>>> >>>> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: >>>>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via >>>>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote: >>>>>> >>>>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote: >>>>>>> For all our platforms booting with EFI there is only one option today. >>>>>>> The timeout gives the user 3 seconds to choose a different option, but >>>>>>> as there is only one, this just causes a needless 3 second delay in >>>>>>> boot time. Remove this delay and boot the only option immediately. >>>>>>> If we ever have more than one option we can always add this back. >>>>>> >>>>>> Well, menu delay in grub is not only used for selecting among available >>>>>> predefined options, but also for interactively editing the command line >>>>>> on the fly before booting. >>>>>> >>>>> >>>>> for development, this sounds good, for production not so much. >>>> >>>> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our >>>> customers during production hardening stage, among many other similar items. >>>> But it is done for the final product - I don't think it should be done in a >>>> generic BSP layer that many use for development. >>> >>> Right, it could be documented somewhere in README or some such would help >>> the consumers of the layer. >> >> Is there already something in the system that we could use to drive >> a variable that we can stick into the file? Like >> EXTRA_IMAGE_FEATURES? I know there used to be a debug-tweaks that >> got broken out into multiple features. Maybe create a new one? >> >> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", >> "wks-debug-timeout", "3", "0 ",d)}' > > Should probably make it more generic, not wks. I'd imagine it can be shared > for U-boot and EFI/GRUB, so maybe bootloader-timeout or something? > Right now we have both a 2 second delay in U-Boot, and then a 3 second delay in the loaded EFI chainloader (GRUB). Which means after all the work we do to push down our boot times, we still just sit for 5 seconds waiting in case some developer has a terminal attached and wants to modify some option.. If you are a developer then you should know how to add back the boot delay, for everyone else it should be default disabled. Having a "bootloader-timeout" var we can set would be nice, but it should set the U-Boot timeout and the EFI timeout should still be default 0 IMHO. No reason for two different delays in boot to wait for extra dev uart input. I've got a patchset in the works to switch us away from GRUB to uefi-kernel anyway, which means direct kernel boot through EFI. No middle-man EFI chainloader like GRUB or systemd-boot needed. Just U-Boot -> Kernel like it should be on an embedded system. Andrew > >> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 >> rootwait ${TI_WKS_BOOTLOADER_APPEND}" >> >> >> Would that work? >> >> >>>> >>>> >>>>>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>>>>> --- >>>>>>> meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- >>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>> >>>>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>> index 7252411c..c1089762 100644 >>>>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>> @@ -3,6 +3,6 @@ >>>>>>> # supports EFI. Boot files are located in the first vfat partition with extra >>>>>>> # reserved space. We cannot use a GPT here. >>>>>>> >>>>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" >>>>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" >>>>>>> part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M >>>>>>> part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid >>>>>>> -- >>>>>>> 2.39.2 >>>>>>> >>>>>>> >>>>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>>>> Links: You receive all messages sent to this group. >>>>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848 >>>>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054 >>>>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org >>>>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com] >>>>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>>>>
On 7/29/2025 1:03 PM, Andrew Davis wrote: > On 7/28/25 5:59 PM, Denys Dmytriyenko wrote: >> On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote: >>> >>> >>> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote: >>>> On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> >>>> wrote: >>>>> >>>>> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote: >>>>>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via >>>>>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> >>>>>> wrote: >>>>>>> >>>>>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via >>>>>>> lists.yoctoproject.org wrote: >>>>>>>> For all our platforms booting with EFI there is only one option >>>>>>>> today. >>>>>>>> The timeout gives the user 3 seconds to choose a different >>>>>>>> option, but >>>>>>>> as there is only one, this just causes a needless 3 second delay in >>>>>>>> boot time. Remove this delay and boot the only option immediately. >>>>>>>> If we ever have more than one option we can always add this back. >>>>>>> >>>>>>> Well, menu delay in grub is not only used for selecting among >>>>>>> available >>>>>>> predefined options, but also for interactively editing the >>>>>>> command line >>>>>>> on the fly before booting. >>>>>>> >>>>>> >>>>>> for development, this sounds good, for production not so much. >>>>> >>>>> Sure. And we also disable e.g. U-boot boot prompt delay all the >>>>> time for our >>>>> customers during production hardening stage, among many other >>>>> similar items. >>>>> But it is done for the final product - I don't think it should be >>>>> done in a >>>>> generic BSP layer that many use for development. >>>> >>>> Right, it could be documented somewhere in README or some such would >>>> help >>>> the consumers of the layer. >>> >>> Is there already something in the system that we could use to drive >>> a variable that we can stick into the file? Like >>> EXTRA_IMAGE_FEATURES? I know there used to be a debug-tweaks that >>> got broken out into multiple features. Maybe create a new one? >>> >>> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", >>> "wks-debug-timeout", "3", "0 ",d)}' >> >> Should probably make it more generic, not wks. I'd imagine it can be >> shared >> for U-boot and EFI/GRUB, so maybe bootloader-timeout or something? >> > > Right now we have both a 2 second delay in U-Boot, and then a 3 second > delay > in the loaded EFI chainloader (GRUB). Which means after all the work we > do to > push down our boot times, we still just sit for 5 seconds waiting in > case some > developer has a terminal attached and wants to modify some option.. If you > are a developer then you should know how to add back the boot delay, for > everyone else it should be default disabled. Agreed. It would just be nice to have some nice easy way in the local.conf to add the delay back in if you want. > Having a "bootloader-timeout" var we can set would be nice, but it should > set the U-Boot timeout and the EFI timeout should still be default 0 IMHO. > No reason for two different delays in boot to wait for extra dev uart > input. > > I've got a patchset in the works to switch us away from GRUB to uefi-kernel > anyway, which means direct kernel boot through EFI. No middle-man EFI > chainloader like GRUB or systemd-boot needed. Just U-Boot -> Kernel like > it should be on an embedded system. > > Andrew > >> >>> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 >>> rootwait ${TI_WKS_BOOTLOADER_APPEND}" >>> >>> >>> Would that work? >>> >>> >>>>> >>>>> >>>>>>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>>>>>> --- >>>>>>>> meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- >>>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>>> >>>>>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>>> b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>>> index 7252411c..c1089762 100644 >>>>>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in >>>>>>>> @@ -3,6 +3,6 @@ >>>>>>>> # supports EFI. Boot files are located in the first vfat >>>>>>>> partition with extra >>>>>>>> # reserved space. We cannot use a GPT here. >>>>>>>> >>>>>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait >>>>>>>> ${TI_WKS_BOOTLOADER_APPEND}" >>>>>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait >>>>>>>> ${TI_WKS_BOOTLOADER_APPEND}" >>>>>>>> part --source bootimg-efi >>>>>>>> --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label >>>>>>>> boot --active --align 1024 --use-uuid --fixed-size 128M >>>>>>>> part / --source rootfs --fstype=ext4 --label root --align 1024 >>>>>>>> --use-uuid >>>>>>>> -- >>>>>>>> 2.39.2 >>>>>>>> >>>>>>>> >>>>>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>>>>> Links: You receive all messages sent to this group. >>>>>>>> View/Reply Online (#18848): >>>>>>>> https://lists.yoctoproject.org/g/meta-ti/message/18848 >>>>>>>> Mute This Topic: >>>>>>>> https://lists.yoctoproject.org/mt/114416911/6551054 >>>>>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org >>>>>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub >>>>>>>> [reatmon@ti.com] >>>>>>>> -=-=-=-=-=-=-=-=-=-=-=- >>>>>>>> >
diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in index 7252411c..c1089762 100644 --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in @@ -3,6 +3,6 @@ # supports EFI. Boot files are located in the first vfat partition with extra # reserved space. We cannot use a GPT here. -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}" part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
For all our platforms booting with EFI there is only one option today. The timeout gives the user 3 seconds to choose a different option, but as there is only one, this just causes a needless 3 second delay in boot time. Remove this delay and boot the only option immediately. If we ever have more than one option we can always add this back. Signed-off-by: Andrew Davis <afd@ti.com> --- meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)