Date: Wed, 21 May 2025 14:37:02 -0300 From: Renato Botelho <garga@FreeBSD.org> To: Toomas Soome <tsoome@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 3bcf8e6db8e9 - main - loader: loader can pick too large font (check if one side is larger) Message-ID: <ebda746f-3463-4c1d-b79f-44e2eecc0668@FreeBSD.org> In-Reply-To: <692b5cb8-be2e-4197-abec-d8889712f2a8__49356.5424041421$1747847616$gmane$org@FreeBSD.org> References: <202505130746.54D7khTN010139__37381.4146196068$1747122471$gmane$org@gitrepo.freebsd.org> <692b5cb8-be2e-4197-abec-d8889712f2a8__49356.5424041421$1747847616$gmane$org@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 21/05/25 14:13, Renato Botelho wrote: > On 13/05/25 04:46, Toomas Soome wrote: >> The branch main has been updated by tsoome: >> >> URL: https://cgit.FreeBSD.org/src/commit/? >> id=3bcf8e6db8e9b30af0a1d445e6e413a332066434 >> >> commit 3bcf8e6db8e9b30af0a1d445e6e413a332066434 >> Author: Toomas Soome <tsoome@FreeBSD.org> >> AuthorDate: 2025-05-13 07:40:39 +0000 >> Commit: Toomas Soome <tsoome@FreeBSD.org> >> CommitDate: 2025-05-13 07:44:35 +0000 >> >> loader: loader can pick too large font (check if one side is larger) >> The 5e25f7b09977 still allows one side to be too large, fix it. >> --- >> stand/common/gfx_fb.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/stand/common/gfx_fb.c b/stand/common/gfx_fb.c >> index ad38a657e233..af72ab1a4c99 100644 >> --- a/stand/common/gfx_fb.c >> +++ b/stand/common/gfx_fb.c >> @@ -2082,7 +2082,7 @@ gfx_get_font(teken_unit_t rows, teken_unit_t >> cols, teken_unit_t height, >> * Skip too large fonts. >> */ >> font = fl->font_data; >> - if (height / font->vfbd_height < rows && >> + if (height / font->vfbd_height < rows || >> width / font->vfbd_width < cols) >> continue; > I still got huge fonts if I remove screen.font from loader.conf. What > kind of data you want me to collect from my system to help you to get it > fixed? > My upgrade script had a reverse logic and didn't update loader.efi. It's working as expected. Thank you! -- Renato Botelho
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ebda746f-3463-4c1d-b79f-44e2eecc0668>