diff mbox series

[2/4] genericarm64.conf: add AMD Kria KV260 serial consoles

Message ID 20241029080544.5486-2-mikko.rapeli@linaro.org (mailing list archive)
State New
Headers show
Series [1/4] README.hardware.md: add genericarm64 maintainers | expand

Commit Message

Mikko Rapeli Oct. 29, 2024, 8:05 a.m. UTC
Current practice is to add new device serial devices to
SERIAL_CONSOLES. Enables testing and test automation with
the board. Note that SERIAL_CONSOLES get added to /etc/inittab
and missing physical serial consoles causes respawn warnings
at runtime:

INIT: Id "S0" respawning too fast: disabled for 5 minutes
INIT: Id "S1" respawning too fast: disabled for 5 minutes

Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
---
 meta-yocto-bsp/conf/machine/genericarm64.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ross Burton Oct. 29, 2024, 11:58 a.m. UTC | #1
On 29 Oct 2024, at 08:05, Mikko Rapeli via lists.yoctoproject.org <mikko.rapeli=linaro.org@lists.yoctoproject.org> wrote:
> 
> Current practice is to add new device serial devices to
> SERIAL_CONSOLES. Enables testing and test automation with
> the board. Note that SERIAL_CONSOLES get added to /etc/inittab
> and missing physical serial consoles causes respawn warnings
> at runtime:
> 
> INIT: Id "S0" respawning too fast: disabled for 5 minutes
> INIT: Id "S1" respawning too fast: disabled for 5 minutes

The use of ttyrun should avoid the respawning issue, have you confirmed this still happens? If so, any idea why?

Ross
Mikko Rapeli Oct. 29, 2024, 12:10 p.m. UTC | #2
On Tue, Oct 29, 2024 at 11:58:09AM +0000, Ross Burton wrote:
> On 29 Oct 2024, at 08:05, Mikko Rapeli via lists.yoctoproject.org <mikko.rapeli=linaro.org@lists.yoctoproject.org> wrote:
> > 
> > Current practice is to add new device serial devices to
> > SERIAL_CONSOLES. Enables testing and test automation with
> > the board. Note that SERIAL_CONSOLES get added to /etc/inittab
> > and missing physical serial consoles causes respawn warnings
> > at runtime:
> > 
> > INIT: Id "S0" respawning too fast: disabled for 5 minutes
> > INIT: Id "S1" respawning too fast: disabled for 5 minutes
> 
> The use of ttyrun should avoid the respawning issue, have you confirmed this still happens? If so, any idea why?

This is happening with very recent poky master builds with sysvinit, the default.

https://ledge.validation.linaro.org/scheduler/job/95628

    Starting syslogd/klogd: done
    
    INIT: Id "S1" respawning too fast: disabled for 5 minutes
    
    INIT: Id "S0" respawning too fast: disabled for 5 minutes
    Poky (Yocto Project Reference Distro) 5.1 genericarm64 /dev/ttyPS1

Alternatively testing could focus on systemd init, or the issue is just ignored.
Or udev based agetty starter is developed. For generic images, it should not be
necessary to define all serial consoles via SERIAL_CONSOLES at build time.
Kernel has a lot more drivers enabled.

Cheers,

-Mikko
Ross Burton Oct. 29, 2024, 12:30 p.m. UTC | #3
On 29 Oct 2024, at 12:10, Mikko Rapeli <mikko.rapeli@linaro.org> wrote:
> 
> On Tue, Oct 29, 2024 at 11:58:09AM +0000, Ross Burton wrote:
>> On 29 Oct 2024, at 08:05, Mikko Rapeli via lists.yoctoproject.org <mikko.rapeli=linaro.org@lists.yoctoproject.org> wrote:
>>> 
>>> Current practice is to add new device serial devices to
>>> SERIAL_CONSOLES. Enables testing and test automation with
>>> the board. Note that SERIAL_CONSOLES get added to /etc/inittab
>>> and missing physical serial consoles causes respawn warnings
>>> at runtime:
>>> 
>>> INIT: Id "S0" respawning too fast: disabled for 5 minutes
>>> INIT: Id "S1" respawning too fast: disabled for 5 minutes
>> 
>> The use of ttyrun should avoid the respawning issue, have you confirmed this still happens? If so, any idea why?
> 
> This is happening with very recent poky master builds with sysvinit, the default.
> 
> https://ledge.validation.linaro.org/scheduler/job/95628
> 
>    Starting syslogd/klogd: done
> 
>    INIT: Id "S1" respawning too fast: disabled for 5 minutes
> 
>    INIT: Id "S0" respawning too fast: disabled for 5 minutes
>    Poky (Yocto Project Reference Distro) 5.1 genericarm64 /dev/ttyPS1
> 
> Alternatively testing could focus on systemd init, or the issue is just ignored.
> Or udev based agetty starter is developed. For generic images, it should not be
> necessary to define all serial consoles via SERIAL_CONSOLES at build time.
> Kernel has a lot more drivers enabled.

Better solutions for sysv appreciated: hardcoding the list at boot time _is_ madness.  Something involving a udev triggers sounds like a better solution long term, for sure.

However, ttyrun really should be stopping this, so I’m curious what the problem is and whether the xilinx tty driver is a bit broken.

Ross
diff mbox series

Patch

diff --git a/meta-yocto-bsp/conf/machine/genericarm64.conf b/meta-yocto-bsp/conf/machine/genericarm64.conf
index b6dfb8042d..a77ffb0431 100644
--- a/meta-yocto-bsp/conf/machine/genericarm64.conf
+++ b/meta-yocto-bsp/conf/machine/genericarm64.conf
@@ -28,7 +28,7 @@  WKS_FILE ?= "genericarm64.wks.in"
 EFI_PROVIDER ?= "${@bb.utils.contains("DISTRO_FEATURES", "systemd", "systemd-boot", "grub-efi", d)}"
 
 # Try to bring up a selection of physical or virtual serial consoles
-SERIAL_CONSOLES ?= "115200;ttyAMA0 115200;hvc0 115200;ttyS0 115200;ttyS1 115200;ttyS2"
+SERIAL_CONSOLES ?= "115200;ttyPS0 115200;ttyPS1 115200;ttyAMA0 115200;hvc0 115200;ttyS0 115200;ttyS1 115200;ttyS2"
 
 # Allow u-boot to be built for use with qemu-system-aarch64.
 # This u-boot is _not_ suitable for use with real hardware, and the expectation