On 5/24/23 3:03 PM, [email protected] wrote:
From: Randolph Sapp <[email protected]>
Adding serial consoles to this variable makes systemd attempt to start a
Getty instance on them. Getty has no retry limit and no retry timout.
I'm sure you can guess where this is going.
This fixes the endless start / stop Getty on ttyS0/ttyS3 messages that
will flood the log until it's full.
In the case of Jailhouse demos, the getty systemd service should be
invoked upon demo start and stopped upon end of the demo. It should not
be listed as a system serial interface since it will not be present
during normal device operation.
Before anyone asks, SERIAL_CONSOLES_CHECK only works on the sysVinit
script so it's not worth discussing as a solution here.
Sounds like the better solution then is to fix systemd to timeout
on SERIAL_CONSOLES_CHECK. We are not the only board with multiple
possible terminals in OE right now. Making systemd match sysv here
is something worth doing.
Not tested but a bbappends that changes the service reset might do:
--- a/meta/recipes-core/systemd/systemd-serialgetty/[email protected]
+++ b/meta/recipes-core/systemd/systemd-serialgetty/[email protected]
@@ -32,7 +32,7 @@ Before=rescue.service
Environment="TERM=@TERM@"
ExecStart=-/sbin/agetty -8 -L %I @BAUDRATE@ $TERM
Type=idle
-Restart=always
+Restart=on-success
UtmpIdentifier=%I
TTYPath=/dev/%I
TTYReset=yes
Andrew
Signed-off-by: Randolph Sapp <[email protected]>
---
meta-ti-bsp/conf/machine/include/am62xx.inc | 3 ---
meta-ti-bsp/conf/machine/include/k3.inc | 2 +-
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/meta-ti-bsp/conf/machine/include/am62xx.inc
b/meta-ti-bsp/conf/machine/include/am62xx.inc
index c98da5f5..1274d3b5 100644
--- a/meta-ti-bsp/conf/machine/include/am62xx.inc
+++ b/meta-ti-bsp/conf/machine/include/am62xx.inc
@@ -25,6 +25,3 @@ TFA_BOARD = "lite"
TFA_K3_SYSTEM_SUSPEND = "1"
OPTEEMACHINE = "k3-am62x"
-
-# Normally AM62 boards use ttyS2, but our Jailhouse inmate may use ttyS3, so
try both
-SERIAL_CONSOLES = "115200;ttyS2 115200;ttyS3"
diff --git a/meta-ti-bsp/conf/machine/include/k3.inc
b/meta-ti-bsp/conf/machine/include/k3.inc
index c6b55239..97c6da94 100644
--- a/meta-ti-bsp/conf/machine/include/k3.inc
+++ b/meta-ti-bsp/conf/machine/include/k3.inc
@@ -55,5 +55,5 @@ WKS_FILE ?= "${@bb.utils.contains("MACHINE_FEATURES", "efi",
"sdimage-2part-efi.
do_image_wic[depends] += "virtual/bootloader:do_deploy"
do_image_complete[mcdepends] += "mc::k3r5:ti-sci-fw:do_deploy"
-SERIAL_CONSOLES = "115200;ttyS0 115200;ttyS2"
+SERIAL_CONSOLES = "115200;ttyS2"
SERIAL_CONSOLES_CHECK = "${SERIAL_CONSOLES}"
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#16627):
https://lists.yoctoproject.org/g/meta-ti/message/16627
Mute This Topic: https://lists.yoctoproject.org/mt/99116754/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
