From owner-freebsd-current@freebsd.org Fri Feb 26 07:29:33 2021 Return-Path: Delivered-To: freebsd-current@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id BC837558562 for ; Fri, 26 Feb 2021 07:29:33 +0000 (UTC) (envelope-from tsoome@me.com) Received: from mr85p00im-zteg06012001.me.com (mr85p00im-zteg06012001.me.com [17.58.23.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4Dn1WK34JZz4twB for ; Fri, 26 Feb 2021 07:29:33 +0000 (UTC) (envelope-from tsoome@me.com) Received: from nazgul.lan (148-52-235-80.sta.estpak.ee [80.235.52.148]) by mr85p00im-zteg06012001.me.com (Postfix) with ESMTPSA id 741FBA004D3; Fri, 26 Feb 2021 07:29:24 +0000 (UTC) From: Toomas Soome Message-Id: Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\)) Subject: Re: testers needed: loader: use display pixel density for font autoselection Date: Fri, 26 Feb 2021 09:29:20 +0200 In-Reply-To: <202102260342.11Q3gkZU028499@gndrsh.dnsmgr.net> Cc: Jakob Alvermark , FreeBSD Developers , freebsd-current To: rgrimes@freebsd.org References: <202102260342.11Q3gkZU028499@gndrsh.dnsmgr.net> X-Mailer: Apple Mail (2.3654.60.0.2.21) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.369, 18.0.761 definitions=2021-02-26_01:2021-02-24, 2021-02-26 signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 malwarescore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 mlxscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-2006250000 definitions=main-2102260057 X-Rspamd-Queue-Id: 4Dn1WK34JZz4twB X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.34 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 Feb 2021 07:29:33 -0000 > On 26. Feb 2021, at 05:42, Rodney W. Grimes = wrote: >=20 >>> On 23. Feb 2021, at 17:53, Jakob Alvermark = wrote: >>>=20 >>> On 2/23/21 12:27 PM, Toomas Soome via freebsd-current wrote: >>>> hi! >>>>=20 >>>> I have done some work to make font pickup a bit smarter (hopefully = better;), but my own ability to test is limited to one bugged supermicro = and one MBP with retina display? >>>>=20 >>>> The phab link ishttps://reviews.freebsd.org/D28849 = >>>>=20 >>>> I have built loader binaries as well (bios and uefi): >>>> loader_lua >>>> loader_lua.efi >>>>=20 >>>> To test, you should remove screen.font=3D line from loader.conf and = test with different resolutions. >>>>=20 >>>> thanks, >>>> toomas >>>=20 >>>=20 >>>=20 >>> Hi Toomas, >>>=20 >>>=20 >>> I tested on five different setups. >>>=20 >>> Surface Pro 10.6"@1920x1080: >>>=20 >>> The loader menu looks different, the "FreeBSD" text is on the right = side of the screen. >>=20 >>=20 >> I think, this was the lua script bug we did fix not too long time = ago. >>=20 >>>=20 >>> Otherwise, the font size is what I would call a normal size. >>>=20 >>>=20 >>> Acer laptop 11.6"@1366x768: >>>=20 >>> Menu looks fine. Almost fills the entire screen. >>>=20 >>> The font feels a little too big. >>=20 >>=20 >> The laptop built in displays usually do not give out EDID (we get = physical dimensions from EDID), so there we fall back to try to get = 80x25 terminal method. >=20 > I am having a hard time with that statement. EDID is very common on = laptop screens, infact I can not recall ever not seeing EDID on a = laptops builtin screen. > My 11" acer 1400 has EDID in it. >=20 >=20 If there is EDID, then it is all good. I have seen cases we do not get = it with available API. >>>=20 >>> Thinkpad built in 13"@1920x1080: >>>=20 >>> Menu looks fine, but a little slow. >>>=20 >>> The font size is a little to big for my liking. When drm loads and = mirrors the screen to my external 27" it looks comically large. >>>=20 >>=20 >> There is another issue - once DRM will kick in, we should re-consider = the console attributes, like fonts, but at this time, the kernel itself = only can use what was built in (8x16), or what loader was offering = (default if present). So it is up to user to act there. >=20 > It would be really nice if DRM could pick up what the resolution and = font was when it loaded! it should do more, my supermicro X10SAE is ony doing 800x600 with UEFI, = it has dell 27=E2=80=9D 4k monitor connected. VBE can get 1600x1200 from = the same set. What I would like to see is, once KMS is attached (i915), = I=E2=80=99d like to get bets possible resolution and appropriate font. = But thats something for future work. >=20 >>=20 >>>=20 >>> Thinkpad external 24"@1920x1200: >>>=20 >>> Menu looks OK, uses about a quarter of the screen. >>>=20 >>> Font size is fine, but once drm loads it looks a bit squeezed (like = thin and tall), but I guess that's drm detecting the built in 1920x1080, = and the external display is stretched. >>>=20 >>>=20 >>> Thinkpad external 27"@3840x2160: >>>=20 >>> Menu looks OK, uses about a quarter of the screen. >>>=20 >>> Font size is fine. >>>=20 >>> Looking at the dmesg though, it says: VT(efifb): resolution = 1920x1080 >>>=20 >>>=20 >>> Jakob >>>=20 >>=20 >>=20 >> Those cases .. I suppose the menu was still at left side, not in = middle? The thing there is, our menu is designed for 80x25 screen, with = respective constants.=20 >=20 > SO again... why are we deviating from that causing us issues? You have misunderstood - the current menu code *is* built for 80x25 - = the menu frame size is fixed, logo/brand locations are constants and so = on. >=20 >>=20 >> many thanks for testing, >=20 > I have downloaded your modified loader, and put it in place, it shall = get tested on my next reboot, which should be soon as 13-BETA4 should be = popping out soon. >=20 thank you! rgds, toomas