From nobody Fri Sep 23 21:35:51 2022 X-Original-To: dev-commits-src-all@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 4MZ56x5VNMz4d0gf; Fri, 23 Sep 2022 21:35:53 +0000 (UTC) (envelope-from jhb@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 "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4MZ56x3VTpz3ckR; Fri, 23 Sep 2022 21:35:53 +0000 (UTC) (envelope-from jhb@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1663968953; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pVHmiEHskqVM3XviEZ7NR7mFWF0DzxGvZ4jUadzruCQ=; b=qnOY78X7ZFxZUSJjObEWSS6m2brJ5UmrJmg+FoJd23dtP8x4D6Cp28TfgW/lVj0IUu6d5y cLDOla7e7haiIqzdMKOYZ2n7ZYm9oY32o04jS7ssUcoi+4YJY6gPf+d+5xXBIUlcOLzTQo DRuuAXO2bWoGWHOxRgUL/uCtaxy+1ZfmL/dmGaP1Kbcnz/T7nYEXwk/n6GXVjMWYBeyL+B dkQWF0GXOEPzks6Gnj4+fjZ8i92adqDWm3Kgmbq0RVP29j/y3L93TY9CmRZFNWYgZpZuaR j+f78DfLoXBDJxX485AZteECzrVQWUPIh4DSTna+evaGmcdWJ1zm1YmYQS257g== Received: from [10.0.1.4] (ralph.baldwin.cx [66.234.199.215]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (Client did not present a certificate) (Authenticated sender: jhb) by smtp.freebsd.org (Postfix) with ESMTPSA id 4MZ56w6RQrz1SJ3; Fri, 23 Sep 2022 21:35:52 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Message-ID: <7e773164-4317-ba17-1f26-1f382cf46fb3@FreeBSD.org> Date: Fri, 23 Sep 2022 14:35:51 -0700 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.13.1 Subject: Re: git: df065f699f1f - main - stand: More sensible defaults when ConOut is missing Content-Language: en-US From: John Baldwin To: Warner Losh , src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org References: <202208270418.27R4IkeL078154@gitrepo.freebsd.org> <0fccddc3-126d-f7b6-3b69-5fc1cbdb2775@FreeBSD.org> In-Reply-To: <0fccddc3-126d-f7b6-3b69-5fc1cbdb2775@FreeBSD.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1663968953; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=pVHmiEHskqVM3XviEZ7NR7mFWF0DzxGvZ4jUadzruCQ=; b=G4fQeK17we/jVp67I1mfoGU//rCOCj9gc2Uxy1QG7APCWhvyo7StuegV7NgMHtVAylQVbA u1uE/SInqaaNBhYJq4JH8IWitGR1TthD3yJX6hXbhv7koGw4aPPKg3450qJILueqU3kb5H a5QKnbiRP3q3B3cdQ12/SKGv9zDwapQQpK3uB+RydmooE7MJ0/R7jNsg5tOIApeN2I71ee K9gjCWaF/kw+52rvxW3BnqA6xyui2CRAaYIVNxnJMl5ZrTE70gGHQzZMsKJriXdZ9x9UPd 0jIlY71MGuCnUUb+CITNyN26J7soVkicNmBbFt4JW/sMGTIrTMj7tsIQ3bGUiQ== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1663968953; a=rsa-sha256; cv=none; b=FZ8CzmSmAxxbxX+Yn+P20iIgRRU6QLwGnGe5ZKgBfkajlLhsFwEHfbSIHdsusSbmapmx39 rqgsp9quL2UYjXu3MqmMiCDEFXb8VH6VnkP3NVfxbakxEQpxFcXYhS+29M/cnTXFpLmiec 0dM4pf1Ps/jzwYdy0dhNFLU4o/c7/FU6b9X8yJHt/tc6RneS7L7iWBnlOh2iknSrJB1Y1S /qoilf/kbcHssM3QvYVcX8MVXe7L6PU3PN8Ex6xk6yhu1bt/95YncSEeQ9226ooyC6rdyN mBj7LxUe8TUXYMJUdjd2g4s67FWejNC4kjNof/RzGyaxIkSMgd5qRal/16yXTg== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N On 9/23/22 2:29 PM, John Baldwin wrote: > On 8/26/22 9:18 PM, Warner Losh wrote: >> The branch main has been updated by imp: >> >> URL: https://cgit.FreeBSD.org/src/commit/?id=df065f699f1ff819bb9607c44a6754275ab335ed >> >> commit df065f699f1ff819bb9607c44a6754275ab335ed >> Author: Warner Losh >> AuthorDate: 2022-08-26 21:46:33 +0000 >> Commit: Warner Losh >> CommitDate: 2022-08-27 04:17:56 +0000 >> >> stand: More sensible defaults when ConOut is missing >> >> When ConOut is missing, we used to default to serial. Except we did it >> in the worst way possible by just setting the howto bits and not >> updating the console setting, which lead to weird behavior where we'd >> get some things on the video port, others on serial. >> >> Instead, set console to "efi,comconsole" for this case. Also set >> RB_MULTIPLE always (so we get dual consoles from the kernel) and or in >> RB_SERIAL when we can't find GOPs that suggest the precense of a video >> console. This will put output in the most places and have a sensible >> default for 'primary' console. >> >> Sponsored by: Netflix >> Reviewed by: emaste, manu >> Differential Revision: https://reviews.freebsd.org/D36299 > > One possibly surprising result of this is that I did not get dual console > output on my rpi after this. (Curiously this only affected my arm64 image > but not my armv7 image.) Loader output goes to both, but kernel output is > only on the video console (which I don't normally use for my pi). (Also, > none of the ANSI escape sequences used by the loader work on the pi's video > console, so once the menu starts it just looks like raw ANSI code garbage > until the kernel starts booting.) > > Not sure if this warrants UPDATING as the effect is that the serial console > seems to stop working? The lack of working dual console output is perhaps > the only real bug. Not sure what is up there. FWIW, using 'console="comconsole,efi"' did not work as a workaround. Same results (used the video console only for the kernel). Had to use 'console="comconsole"' in /boot/loader.conf. -- John Baldwin