* [DistroKit] [PATCH] platforms: revert to default systemd loglevel @ 2023-04-24 19:29 Roland Hieber 2023-04-24 19:55 ` Ahmad Fatoum 2023-04-26 20:15 ` Roland Hieber 0 siblings, 2 replies; 5+ messages in thread From: Roland Hieber @ 2023-04-24 19:29 UTC (permalink / raw) To: distrokit; +Cc: Roland Hieber By decreasing the systemd loglevel to "warning", the journal no longer contains useful lines like which units were started and stopped when, which is quite sparse information for debugging. Remove the systemd.log_level option from all kernel command lines so that systemd defaults to loglevel "info", same as loglevel=5 for kernel messages. Keep the systemd.show_status option on "auto" so that systemd only produces console output when the boot takes a significant amount of time or produces error messages; so in effect this commit only changes the verbosity of the journal, not the console. Signed-off-by: Roland Hieber <rhi@pengutronix.de> --- configs/platform-mips/run | 2 +- .../barebox-am335x-defaultenv/nv/linux.bootargs.loglevel | 2 +- .../barebox-mx6-defaultenv/nv/linux.bootargs.loglevel | 2 +- .../barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel | 2 +- .../barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel | 2 +- .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- configs/platform-v7a/run | 2 +- configs/platform-v7a/run-nfs | 2 +- .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- configs/platform-v7a_noneon/run | 2 +- configs/platform-v7a_noneon/run-nfs | 2 +- .../barebox-common-defaultenv/nv/linux.bootargs.loglevel | 2 +- .../barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel | 2 +- configs/platform-x86_64/run | 2 +- 14 files changed, 14 insertions(+), 14 deletions(-) diff --git a/configs/platform-mips/run b/configs/platform-mips/run index edfce51a96b5..cf8a535167e3 100755 --- a/configs/platform-mips/run +++ b/configs/platform-mips/run @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) fi -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" # Machine to emulate QEMU_ARGS=( -M malta -m 256 -serial stdio -monitor null -device qemu-xhci,id=xhci) diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a/run b/configs/platform-v7a/run index 3f62b237ac12..004706ddaf59 100755 --- a/configs/platform-v7a/run +++ b/configs/platform-v7a/run @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" fi -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then security_model="mapped-file" diff --git a/configs/platform-v7a/run-nfs b/configs/platform-v7a/run-nfs index d1675fb47bfd..c78e36ca1ea0 100755 --- a/configs/platform-v7a/run-nfs +++ b/configs/platform-v7a/run-nfs @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ -smp 1 \ - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" diff --git a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v7a_noneon/run b/configs/platform-v7a_noneon/run index 8af190f11eb4..0bb06bc60868 100755 --- a/configs/platform-v7a_noneon/run +++ b/configs/platform-v7a_noneon/run @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" fi -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then security_model="mapped-file" diff --git a/configs/platform-v7a_noneon/run-nfs b/configs/platform-v7a_noneon/run-nfs index d1675fb47bfd..c78e36ca1ea0 100755 --- a/configs/platform-v7a_noneon/run-nfs +++ b/configs/platform-v7a_noneon/run-nfs @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ -smp 1 \ - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" diff --git a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel index 705752ca416d..d45c34464422 100644 --- a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel +++ b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel @@ -1 +1 @@ -loglevel=5 systemd.log_level=warning systemd.show_status=auto +loglevel=5 systemd.show_status=auto diff --git a/configs/platform-x86_64/run b/configs/platform-x86_64/run index f3f4c34653a8..e7f30fdd3048 100755 --- a/configs/platform-x86_64/run +++ b/configs/platform-x86_64/run @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) fi -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then security_model="mapped-file" -- 2.39.2 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] platforms: revert to default systemd loglevel 2023-04-24 19:29 [DistroKit] [PATCH] platforms: revert to default systemd loglevel Roland Hieber @ 2023-04-24 19:55 ` Ahmad Fatoum 2023-04-24 20:26 ` Roland Hieber 2023-04-26 20:15 ` Roland Hieber 1 sibling, 1 reply; 5+ messages in thread From: Ahmad Fatoum @ 2023-04-24 19:55 UTC (permalink / raw) To: Roland Hieber, distrokit Hello Roland, On 24.04.23 21:29, Roland Hieber wrote: > By decreasing the systemd loglevel to "warning", the journal no longer > contains useful lines like which units were started and stopped when, > which is quite sparse information for debugging. Remove the > systemd.log_level option from all kernel command lines so that systemd > defaults to loglevel "info", same as loglevel=5 for kernel messages. > > Keep the systemd.show_status option on "auto" so that systemd only > produces console output when the boot takes a significant amount of > time or produces error messages; so in effect this commit only changes > the verbosity of the journal, not the console. For debugging, you can override the loglevel, e.g. in barebox with: nv linux.bootargs.loglevel="" In the normal case, less output = more chance to spot new boot warnings/ errors, so I'd prefer we leave that as-is. Cheers, Ahmad > > Signed-off-by: Roland Hieber <rhi@pengutronix.de> > --- > configs/platform-mips/run | 2 +- > .../barebox-am335x-defaultenv/nv/linux.bootargs.loglevel | 2 +- > .../barebox-mx6-defaultenv/nv/linux.bootargs.loglevel | 2 +- > .../barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel | 2 +- > .../barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel | 2 +- > .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- > configs/platform-v7a/run | 2 +- > configs/platform-v7a/run-nfs | 2 +- > .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- > configs/platform-v7a_noneon/run | 2 +- > configs/platform-v7a_noneon/run-nfs | 2 +- > .../barebox-common-defaultenv/nv/linux.bootargs.loglevel | 2 +- > .../barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel | 2 +- > configs/platform-x86_64/run | 2 +- > 14 files changed, 14 insertions(+), 14 deletions(-) > > diff --git a/configs/platform-mips/run b/configs/platform-mips/run > index edfce51a96b5..cf8a535167e3 100755 > --- a/configs/platform-mips/run > +++ b/configs/platform-mips/run > @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then > QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) > fi > > -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" > > # Machine to emulate > QEMU_ARGS=( -M malta -m 256 -serial stdio -monitor null -device qemu-xhci,id=xhci) > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a/run b/configs/platform-v7a/run > index 3f62b237ac12..004706ddaf59 100755 > --- a/configs/platform-v7a/run > +++ b/configs/platform-v7a/run > @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then > echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" > fi > > -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > security_model="mapped-file" > diff --git a/configs/platform-v7a/run-nfs b/configs/platform-v7a/run-nfs > index d1675fb47bfd..c78e36ca1ea0 100755 > --- a/configs/platform-v7a/run-nfs > +++ b/configs/platform-v7a/run-nfs > @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ > -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ > -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ > -smp 1 \ > - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" > + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" > > diff --git a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v7a_noneon/run b/configs/platform-v7a_noneon/run > index 8af190f11eb4..0bb06bc60868 100755 > --- a/configs/platform-v7a_noneon/run > +++ b/configs/platform-v7a_noneon/run > @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then > echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" > fi > > -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > security_model="mapped-file" > diff --git a/configs/platform-v7a_noneon/run-nfs b/configs/platform-v7a_noneon/run-nfs > index d1675fb47bfd..c78e36ca1ea0 100755 > --- a/configs/platform-v7a_noneon/run-nfs > +++ b/configs/platform-v7a_noneon/run-nfs > @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ > -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ > -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ > -smp 1 \ > - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" > + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" > > diff --git a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > index 705752ca416d..d45c34464422 100644 > --- a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > +++ b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > @@ -1 +1 @@ > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > +loglevel=5 systemd.show_status=auto > diff --git a/configs/platform-x86_64/run b/configs/platform-x86_64/run > index f3f4c34653a8..e7f30fdd3048 100755 > --- a/configs/platform-x86_64/run > +++ b/configs/platform-x86_64/run > @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then > QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) > fi > > -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > security_model="mapped-file" -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] platforms: revert to default systemd loglevel 2023-04-24 19:55 ` Ahmad Fatoum @ 2023-04-24 20:26 ` Roland Hieber 2023-05-03 15:24 ` Ahmad Fatoum 0 siblings, 1 reply; 5+ messages in thread From: Roland Hieber @ 2023-04-24 20:26 UTC (permalink / raw) To: Ahmad Fatoum; +Cc: distrokit On Mon, Apr 24, 2023 at 09:55:12PM +0200, Ahmad Fatoum wrote: > Hello Roland, > > On 24.04.23 21:29, Roland Hieber wrote: > > By decreasing the systemd loglevel to "warning", the journal no longer > > contains useful lines like which units were started and stopped when, > > which is quite sparse information for debugging. Remove the > > systemd.log_level option from all kernel command lines so that systemd > > defaults to loglevel "info", same as loglevel=5 for kernel messages. > > > > Keep the systemd.show_status option on "auto" so that systemd only > > produces console output when the boot takes a significant amount of > > time or produces error messages; so in effect this commit only changes > > the verbosity of the journal, not the console. > > For debugging, you can override the loglevel, e.g. in barebox with: > > nv linux.bootargs.loglevel="" > > In the normal case, less output = more chance to spot new boot warnings/ > errors, so I'd prefer we leave that as-is. With the current state in DistroKit of systemd.log-level=warning, info messages and below don't even make it into the journal. With systemd.show_status=auto on the kernel command line, systemd is still quiet on the console (i.e., fast) until a unit fails or there is a significant delay in boot, which probably does what you wanted? - Roland > > Cheers, > Ahmad > > > > > Signed-off-by: Roland Hieber <rhi@pengutronix.de> > > --- > > configs/platform-mips/run | 2 +- > > .../barebox-am335x-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > .../barebox-mx6-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > .../barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > .../barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > configs/platform-v7a/run | 2 +- > > configs/platform-v7a/run-nfs | 2 +- > > .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > configs/platform-v7a_noneon/run | 2 +- > > configs/platform-v7a_noneon/run-nfs | 2 +- > > .../barebox-common-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > .../barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel | 2 +- > > configs/platform-x86_64/run | 2 +- > > 14 files changed, 14 insertions(+), 14 deletions(-) > > > > diff --git a/configs/platform-mips/run b/configs/platform-mips/run > > index edfce51a96b5..cf8a535167e3 100755 > > --- a/configs/platform-mips/run > > +++ b/configs/platform-mips/run > > @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then > > QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) > > fi > > > > -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > > +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" > > > > # Machine to emulate > > QEMU_ARGS=( -M malta -m 256 -serial stdio -monitor null -device qemu-xhci,id=xhci) > > diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a/run b/configs/platform-v7a/run > > index 3f62b237ac12..004706ddaf59 100755 > > --- a/configs/platform-v7a/run > > +++ b/configs/platform-v7a/run > > @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then > > echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" > > fi > > > > -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > > +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" > > > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > > security_model="mapped-file" > > diff --git a/configs/platform-v7a/run-nfs b/configs/platform-v7a/run-nfs > > index d1675fb47bfd..c78e36ca1ea0 100755 > > --- a/configs/platform-v7a/run-nfs > > +++ b/configs/platform-v7a/run-nfs > > @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ > > -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ > > -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ > > -smp 1 \ > > - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" > > + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" > > > > diff --git a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v7a_noneon/run b/configs/platform-v7a_noneon/run > > index 8af190f11eb4..0bb06bc60868 100755 > > --- a/configs/platform-v7a_noneon/run > > +++ b/configs/platform-v7a_noneon/run > > @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then > > echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" > > fi > > > > -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > > +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" > > > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > > security_model="mapped-file" > > diff --git a/configs/platform-v7a_noneon/run-nfs b/configs/platform-v7a_noneon/run-nfs > > index d1675fb47bfd..c78e36ca1ea0 100755 > > --- a/configs/platform-v7a_noneon/run-nfs > > +++ b/configs/platform-v7a_noneon/run-nfs > > @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ > > -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ > > -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ > > -smp 1 \ > > - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" > > + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" > > > > diff --git a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > > index 705752ca416d..d45c34464422 100644 > > --- a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > > +++ b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel > > @@ -1 +1 @@ > > -loglevel=5 systemd.log_level=warning systemd.show_status=auto > > +loglevel=5 systemd.show_status=auto > > diff --git a/configs/platform-x86_64/run b/configs/platform-x86_64/run > > index f3f4c34653a8..e7f30fdd3048 100755 > > --- a/configs/platform-x86_64/run > > +++ b/configs/platform-x86_64/run > > @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then > > QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) > > fi > > > > -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" > > +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" > > > > if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then > > security_model="mapped-file" > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > > -- Roland Hieber, Pengutronix e.K. | r.hieber@pengutronix.de | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] platforms: revert to default systemd loglevel 2023-04-24 20:26 ` Roland Hieber @ 2023-05-03 15:24 ` Ahmad Fatoum 0 siblings, 0 replies; 5+ messages in thread From: Ahmad Fatoum @ 2023-05-03 15:24 UTC (permalink / raw) To: Roland Hieber; +Cc: distrokit On 24.04.23 22:26, Roland Hieber wrote: > On Mon, Apr 24, 2023 at 09:55:12PM +0200, Ahmad Fatoum wrote: >> Hello Roland, >> >> On 24.04.23 21:29, Roland Hieber wrote: >>> By decreasing the systemd loglevel to "warning", the journal no longer >>> contains useful lines like which units were started and stopped when, >>> which is quite sparse information for debugging. Remove the >>> systemd.log_level option from all kernel command lines so that systemd >>> defaults to loglevel "info", same as loglevel=5 for kernel messages. >>> >>> Keep the systemd.show_status option on "auto" so that systemd only >>> produces console output when the boot takes a significant amount of >>> time or produces error messages; so in effect this commit only changes >>> the verbosity of the journal, not the console. >> >> For debugging, you can override the loglevel, e.g. in barebox with: >> >> nv linux.bootargs.loglevel="" >> >> In the normal case, less output = more chance to spot new boot warnings/ >> errors, so I'd prefer we leave that as-is. > > With the current state in DistroKit of systemd.log-level=warning, info > messages and below don't even make it into the journal. With > systemd.show_status=auto on the kernel command line, systemd is still > quiet on the console (i.e., fast) until a unit fails or there is a > significant delay in boot, which probably does what you wanted? Thanks for the clarification. Makes sense. > > - Roland > >> >> Cheers, >> Ahmad >> >>> >>> Signed-off-by: Roland Hieber <rhi@pengutronix.de> >>> --- >>> configs/platform-mips/run | 2 +- >>> .../barebox-am335x-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> .../barebox-mx6-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> .../barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> .../barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> configs/platform-v7a/run | 2 +- >>> configs/platform-v7a/run-nfs | 2 +- >>> .../barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> configs/platform-v7a_noneon/run | 2 +- >>> configs/platform-v7a_noneon/run-nfs | 2 +- >>> .../barebox-common-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> .../barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel | 2 +- >>> configs/platform-x86_64/run | 2 +- >>> 14 files changed, 14 insertions(+), 14 deletions(-) >>> >>> diff --git a/configs/platform-mips/run b/configs/platform-mips/run >>> index edfce51a96b5..cf8a535167e3 100755 >>> --- a/configs/platform-mips/run >>> +++ b/configs/platform-mips/run >>> @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then >>> QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) >>> fi >>> >>> -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" >>> +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" >>> >>> # Machine to emulate >>> QEMU_ARGS=( -M malta -m 256 -serial stdio -monitor null -device qemu-xhci,id=xhci) >>> diff --git a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a/barebox-am335x-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a/barebox-mx6-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a/barebox-rpi2-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a/barebox-stm32mp-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a/run b/configs/platform-v7a/run >>> index 3f62b237ac12..004706ddaf59 100755 >>> --- a/configs/platform-v7a/run >>> +++ b/configs/platform-v7a/run >>> @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then >>> echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" >>> fi >>> >>> -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" >>> +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" >>> >>> if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then >>> security_model="mapped-file" >>> diff --git a/configs/platform-v7a/run-nfs b/configs/platform-v7a/run-nfs >>> index d1675fb47bfd..c78e36ca1ea0 100755 >>> --- a/configs/platform-v7a/run-nfs >>> +++ b/configs/platform-v7a/run-nfs >>> @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ >>> -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ >>> -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ >>> -smp 1 \ >>> - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" >>> + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" >>> >>> diff --git a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v7a_noneon/barebox-vexpress-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v7a_noneon/run b/configs/platform-v7a_noneon/run >>> index 8af190f11eb4..0bb06bc60868 100755 >>> --- a/configs/platform-v7a_noneon/run >>> +++ b/configs/platform-v7a_noneon/run >>> @@ -62,7 +62,7 @@ if [ -z "${QEMU_NET}" ]; then >>> echo "Forwarding SSH port 127.0.0.1:${SSH_EXTERNAL_PORT} -> qemu:${SSH_INTERNAL_PORT}" >>> fi >>> >>> -BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" >>> +BASE_CMDLINE="console=ttyAMA0,115200 loglevel=5 systemd.show_status=auto" >>> >>> if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then >>> security_model="mapped-file" >>> diff --git a/configs/platform-v7a_noneon/run-nfs b/configs/platform-v7a_noneon/run-nfs >>> index d1675fb47bfd..c78e36ca1ea0 100755 >>> --- a/configs/platform-v7a_noneon/run-nfs >>> +++ b/configs/platform-v7a_noneon/run-nfs >>> @@ -16,5 +16,5 @@ ${QEMU_EXEC} ${QEMU_NOGRAPHIC} \ >>> -kernel ${PTXDIST_PLATFORMDIR}/images/linuximage \ >>> -dtb ${PTXDIST_PLATFORMDIR}/images/vexpress-v2p-ca9.dtb \ >>> -smp 1 \ >>> - -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.log_level=warning systemd.show_status=auto ip=dhcp" >>> + -append "console=ttyAMA0 root=/dev/nfs nfsroot=10.0.2.2:/root,v3,tcp,port=2049,mountport=2049 rw mem=1024M rootwait loglevel=5 systemd.show_status=auto ip=dhcp" >>> >>> diff --git a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v8a/barebox-common-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel >>> index 705752ca416d..d45c34464422 100644 >>> --- a/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel >>> +++ b/configs/platform-v8a/barebox-rock3a-defaultenv/nv/linux.bootargs.loglevel >>> @@ -1 +1 @@ >>> -loglevel=5 systemd.log_level=warning systemd.show_status=auto >>> +loglevel=5 systemd.show_status=auto >>> diff --git a/configs/platform-x86_64/run b/configs/platform-x86_64/run >>> index f3f4c34653a8..e7f30fdd3048 100755 >>> --- a/configs/platform-x86_64/run >>> +++ b/configs/platform-x86_64/run >>> @@ -65,7 +65,7 @@ if [ -z "${QEMU_NET}" ]; then >>> QEMU_NET=(-netdev user,id=net1,hostfwd=tcp:127.0.0.1:${TELNET_EXTERNAL_PORT}-:${TELNET_INTERNAL_PORT},hostfwd=tcp:127.0.0.1:${SSH_EXTERNAL_PORT}-:${SSH_INTERNAL_PORT} ) >>> fi >>> >>> -BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.log_level=warning systemd.show_status=auto" >>> +BASE_CMDLINE="console=ttyS0,115200 loglevel=5 systemd.show_status=auto" >>> >>> if [ -d "${PTXDIST_PLATFORMDIR}/root/.virtfs_metadata" ]; then >>> security_model="mapped-file" >> >> -- >> Pengutronix e.K. | | >> Steuerwalder Str. 21 | http://www.pengutronix.de/ | >> 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | >> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | >> >> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] platforms: revert to default systemd loglevel 2023-04-24 19:29 [DistroKit] [PATCH] platforms: revert to default systemd loglevel Roland Hieber 2023-04-24 19:55 ` Ahmad Fatoum @ 2023-04-26 20:15 ` Roland Hieber 1 sibling, 0 replies; 5+ messages in thread From: Roland Hieber @ 2023-04-26 20:15 UTC (permalink / raw) To: distrokit On Mon, Apr 24, 2023 at 09:29:48PM +0200, Roland Hieber wrote: > By decreasing the systemd loglevel to "warning", the journal no longer > contains useful lines like which units were started and stopped when, > which is quite sparse information for debugging. Remove the > systemd.log_level option from all kernel command lines so that systemd > defaults to loglevel "info", same as loglevel=5 for kernel messages. > > Keep the systemd.show_status option on "auto" so that systemd only > produces console output when the boot takes a significant amount of > time or produces error messages; so in effect this commit only changes > the verbosity of the journal, not the console. > > Signed-off-by: Roland Hieber <rhi@pengutronix.de> I've included this in v2 of the RAUC series: <https://lore.distrokit.org/distrokit/20230426201414.2044409-2-rhi@pengutronix.de/T/#u> -- Roland Hieber, Pengutronix e.K. | r.hieber@pengutronix.de | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-05-03 15:24 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-04-24 19:29 [DistroKit] [PATCH] platforms: revert to default systemd loglevel Roland Hieber 2023-04-24 19:55 ` Ahmad Fatoum 2023-04-24 20:26 ` Roland Hieber 2023-05-03 15:24 ` Ahmad Fatoum 2023-04-26 20:15 ` Roland Hieber
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox