From nobody Thu Jan 29 16:31:20 2026 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4f24Qh2tdSz6Qx2f for ; Thu, 29 Jan 2026 16:31:24 +0000 (UTC) (envelope-from michaelo@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R12" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4f24Qh2KNcz3Gnb; Thu, 29 Jan 2026 16:31:24 +0000 (UTC) (envelope-from michaelo@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1769704284; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=68wzoKmc8u+4vgcfQdGGWNH0jOiRY+hlaXAcxQhro34=; b=k/sy7s/8rjRKrsi62w+v736+WB4Bg6u0D9ibn2+jM6MonTu1RoPgARDzaRHNsYXt51/lPB ARcaXecy6P8Ll97+QA8suhxPdOBVY7vU059SYRrcuNvvHyBX0C0uRwSNsPLWrzEcX11ww8 M0D9D23UswAZ0rNMTCwaw3J3JOdUccL89tDURzEbHVJxgw/Ziz1T4Xx5fmn13grl8IfF7i azYQeyjfJZI6KO1ng4blcB4Vamx3niJoaFgTGTAru4SrsFXx5qyiGKiiEJmEIM87olVM3F MAToWWGtNBLnwmALNIjzFPxbtZ0nAZb53Ex9JFKOmkPiUjvpQxmDCFu8UrgIfA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1769704284; a=rsa-sha256; cv=none; b=af4zHcjrACokIZrEKz5aTDXlwkOP0CP8LFavI6RQq9xH3KXC+/oCJNM9QTPWjkoSin7F+s P16Iyh/Q31Gd0RH2ifn2RrYEtOS3f3JMLnYjp58ppGjwRl+re79Ze+A2JTSSVITJ7lEENq nBX1M/TRSckSfdL9g5rde8LkMkGRIYn3mFow/tRojbCQpIScT7F1EhUVJehBQNuXHKoHoM HIuDUdjXPNVxKxslqGhLJuHbz7+edrhY7WVN1virR+CzHWA5t83q0YTYCSp8nWIOxi8JoC 265W/NQQIAejNfdj4cqJHg+QTQ7Esp1yI7yn5LC0sX41MyC2jpfhOpsQTgc4uw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1769704284; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=68wzoKmc8u+4vgcfQdGGWNH0jOiRY+hlaXAcxQhro34=; b=gey9E5WdTux689YgWVBnZM6hsoEMV6T3jsSt/sH4hQfAGSZ+8DxT3PcLiZspWI0Rqgo6+B qelvizg67adg9SzVCT08ISvQKGBn60EkEOHyEHJcEtyuf9mlcYDNT4PMy/fB+2PdBlcYrM xz1yKramG0ImtqNmxUuY+Vc/D1MWR+hWYwMdcAqJ7U/C0+OYC4P+56WoHMyntBjagnmWMw vvOuD1XxGHhai7Bosn4RwpDT8QQqi/4I2xSRzoooBW5aTFRFInLPl1e3eWs4h1u8zma/OV wuqWTluOIbxz62J8t1x4BwEFtnEUaDTGqaSCO3xv67yR842cG5N5d6MZMDAKUg== Received: from [192.168.1.23] (unknown [147.161.251.33]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: michaelo/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id 4f24Qg5YZyzHfj; Thu, 29 Jan 2026 16:31:23 +0000 (UTC) (envelope-from michaelo@FreeBSD.org) Message-ID: <40a3f9a2-3adc-470b-8dcb-63b166e2cf14@FreeBSD.org> Date: Thu, 29 Jan 2026 17:31:20 +0100 List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org MIME-Version: 1.0 User-Agent: Thunderbird Daily Subject: Re: git: 56970c3c4b0b - main - loader: Prefer comma-separated list of values for "console" Content-Language: en-US To: Mark Millard , dev-commits-src-main@FreeBSD.org Cc: Warner Losh References: <697b1e0e.27b69.ae3352c@gitrepo.freebsd.org> From: Michael Osipov In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 2026-01-29 17:22, Mark Millard wrote: > On 1/29/26 00:45, Michael Osipov wrote: >> The branch main has been updated by michaelo: >> >> URL: https://deu01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcgit.freebsd.org%2Fsrc%2Fcommit%2F%3Fid%3D56970c3c4b0bc61d972837661a31cc2b9e8e8d7c&data=05%7C02%7Cmichael.osipov%40innomotics.com%7C599792c98cac445815a108de5f529e81%7C698c6ffb74e34a84be68f22d8d3201a3%7C0%7C0%7C639053005625974477%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=PsvT9MHa5qWyE2CBQGLbDG30GT8AWpltNOu3JrYO%2BK4%3D&reserved=0 >> >> commit 56970c3c4b0bc61d972837661a31cc2b9e8e8d7c >> Author: Michael Osipov >> AuthorDate: 2026-01-28 09:02:33 +0000 >> Commit: Michael Osipov >> CommitDate: 2026-01-29 08:44:45 +0000 >> >> loader: Prefer comma-separated list of values for "console" >> >> PR: 292595 >> Reviewed by: imp >> MFC after: 1 week >> Differential Revision: https://deu01.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.freebsd.org%2FD54923&data=05%7C02%7Cmichael.osipov%40innomotics.com%7C599792c98cac445815a108de5f529e81%7C698c6ffb74e34a84be68f22d8d3201a3%7C0%7C0%7C639053005625994409%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=fqqWtXQep%2FyB%2BinUOiod%2FO%2BqDOOQ4UC5O4KzvM36C7Q%3D&reserved=0 >> --- >> release/tools/azure.conf | 2 +- >> release/tools/openstack.conf | 2 +- >> stand/efi/loader/main.c | 4 ++-- >> stand/i386/loader/main.c | 4 ++-- >> 4 files changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/release/tools/azure.conf b/release/tools/azure.conf >> index 981e40566a1c..4fa6ba4d924d 100644 >> --- a/release/tools/azure.conf >> +++ b/release/tools/azure.conf >> @@ -59,7 +59,7 @@ autoboot_delay="-1" >> beastie_disable="YES" >> loader_logo="none" >> hw.memtest.tests="0" >> -console="comconsole efi vidconsole" >> +console="comconsole,efi,vidconsole" >> comconsole_speed="115200" >> boot_multicons="YES" >> boot_serial="YES" >> diff --git a/release/tools/openstack.conf b/release/tools/openstack.conf >> index 9ce65a75c3c5..6cfb02bab9af 100644 >> --- a/release/tools/openstack.conf >> +++ b/release/tools/openstack.conf >> @@ -33,7 +33,7 @@ vm_extra_pre_umount() { >> echo 'autoboot_delay="-1"' >> ${DESTDIR}/boot/loader.conf >> echo 'beastie_disable="YES"' >> ${DESTDIR}/boot/loader.conf >> echo 'boot_multicons="YES"' >> ${DESTDIR}/boot/loader.conf >> - echo 'console="comconsole vidconsole"' >> ${DESTDIR}/boot/loader.conf >> + echo 'console="comconsole,vidconsole"' >> ${DESTDIR}/boot/loader.conf >> echo 'comconsole_speed="115200"' >> ${DESTDIR}/boot/loader.conf >> >> # Reboot quickly, Don't wait at the panic screen >> diff --git a/stand/efi/loader/main.c b/stand/efi/loader/main.c >> index 22dbd10a0f37..7ff2cf188509 100644 >> --- a/stand/efi/loader/main.c >> +++ b/stand/efi/loader/main.c >> @@ -1316,10 +1316,10 @@ main(int argc, CHAR16 *argv[]) >> setenv("console", "comconsole", 1); >> break; >> case VID_SER_BOTH: >> - setenv("console", "efi comconsole", 1); >> + setenv("console", "efi,comconsole", 1); >> break; >> case SER_VID_BOTH: >> - setenv("console", "comconsole efi", 1); >> + setenv("console", "comconsole,efi", 1); >> break; >> /* case VIDEO_ONLY can't happen -- it's the first if above */ >> } > > [Note: I CC'd Warner just because I'm not sure anyone else knows for > sure about this area's complicated history and issues.] > > > This looks possibly wrong to me for efi for all but amd64 --but for the > references to comconsole, not the space vs. comma issue. > > > stand: efi create eficom console device. > (2023-05-11) > > QUOTE of > console=comconsole talks to the hardware directly. This is available > only on amd64. It is not available anywhere else (and so requires > changes for people doing comconsole on aarch64) > > console=eficom talks to the console via EFI protocols. It's available > on amd64, aarch64 and riscv64. It's the first port that we find, though > it can be overriden by efi_com_port (which should be set to the UID of > the serial port, not the I/O port, despite the name). devinfo -v > will give the UID to uartX mapping. > > This is an incompatible change for HYPER-V on amd64. It only works with > eficom console, so you'll need to change your configuration in > loader.conf. No compatibility hack will ever be provided for this (since > it requires renamig, which the loader cannot reliably do). > > It's also an incompatible change for aarch64. comconsole will need to > change to eficom. There might be a comconsole "shim" for this. > END QUOTE > > > stand: add comconsole backwards compatibility shim for aarch64 > (2023-05-11) > > QUOTE of > Add a compat shim for the "comconsole" name so that people with a > "console=comconsole" in their loader.conf on aarch64 will continue to > work (though with a warning). > > This is only aarch64: it will never be there for amd64 (where comconsole > always means talk to the hardware directly). To do that is too hard. > END QUOTE > > > But that last (f93416d67743) is only for: > > +#if defined(__aarch64__) && __FreeBSD_version < 1500000 > +/* Hack for backward compatibility -- but only for a while */ > +extern struct console comconsole; > +#endif > . . . > +#if defined(__aarch64__) && __FreeBSD_version < 1500000 > + &comconsole, > +#endif > . . . (and so on) . . . > > So, as of FreeBSD 15+, aarch64 does not have comconsole as far as I can > tell. > > > It looks to me like for FreeBSD 15+ eficom should be referenced mostly, > instead of comconsole, comconsole being only for amd64. > > For FreeBSD 14.3+ and stable/14 it may be that comconsole should be used > for amd64 and aarch64. (stable/13 does not have the commits at all.) > > > There is also a prior: > > if (efi_has_gop()) > how |= RB_MULTIPLE; > else > how |= RB_MULTIPLE | RB_SERIAL; > setenv("console", "efi,comconsole", 1); > > and: > > #if defined(__riscv) > /* > * This workaround likely is papering over a real issue > */ > if ((uhowto & RB_SERIAL) != 0) > setenv("console", "comconsole", 1); > #endif > > that may well both be wrong for stand/efi/loader/main.c too. Hi Mark, thanks for raising. I do not fully understand the point you are trying to make. This change is a spinoff from https://reviews.freebsd.org/D54843 where we identified that comma is the preferred separator, but space is allowed as well. For consistency reasons I have updated everything to comma. I didn't change the actual values. Can you clarify? Michael