testers needed: loader: use display pixel density for font autoselection

[Toomas Soome]

> On 26. Feb 2021, at 17:56, Rodney W. Grimes <freebsd-rwg at gndrsh.dnsmgr.net> wrote:
> 
>>> On 26. Feb 2021, at 05:42, Rodney W. Grimes <freebsd at gndrsh.dnsmgr.net> wrote:
>>> 
>>>>> On 23. Feb 2021, at 17:53, Jakob Alvermark <jakob at alvermark.net> wrote:
>>>>> 
>>>>> On 2/23/21 12:27 PM, Toomas Soome via freebsd-current wrote:
>>>>>> hi!
>>>>>> 
>>>>>> 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?
>>>>>> 
>>>>>> The phab link ishttps://reviews.freebsd.org/D28849  <https://reviews.freebsd.org/D28849>
>>>>>> 
>>>>>> I have built loader binaries as well (bios and uefi):
>>>>>> loader_lua<http://148-52-235-80.sta.estpak.ee/loader_lua>
>>>>>> loader_lua.efi<http://148-52-235-80.sta.estpak.ee/loader_lua.efi>
>>>>>> 
>>>>>> To test, you should remove screen.font= line from loader.conf and test with different resolutions.
>>>>>> 
>>>>>> thanks,
>>>>>> toomas
>>>>> 
>>>>> 
>>>>> 
>>>>> Hi Toomas,
>>>>> 
>>>>> 
>>>>> I tested on five different setups.
>>>>> 
>>>>> Surface Pro 10.6"@1920x1080:
>>>>> 
>>>>> The loader menu looks different, the "FreeBSD" text is on the right side of the screen.
>>>> 
>>>> 
>>>> I think, this was the lua script bug we did fix not too long time ago.
>>>> 
>>>>> 
>>>>> Otherwise, the font size is what I would call a normal size.
>>>>> 
>>>>> 
>>>>> Acer laptop 11.6"@1366x768:
>>>>> 
>>>>> Menu looks fine. Almost fills the entire screen.
>>>>> 
>>>>> The font feels a little too big.
>>>> 
>>>> 
>>>> 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.
>>> 
>>> 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.
>>> 
>>> 
>> 
>> 
>> If there is EDID, then it is all good. I have seen cases we do not get it with available API.
> 
> Is there a way for me to know if the laoder found EDID data or not?
> It might be that the issue is that what ever loader is using for an API is not working.
> I based my "EDID is very common on laptop screens" on the fact that X11 almost always finds EDID.
> 

Yes, gop get / vbe list   — yes, it is inconsistent, should fix at some point… :)

we do use gop get active edid/get discovered edid protocol and vbe function 0x4f15.

rgds,
toomas