From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.groups.io with SMTP id smtpd.web10.5636.1624443869186924726 for ; Wed, 23 Jun 2021 03:24:29 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=fBfU++pt; spf=pass (domain: redhat.com, ip: 216.205.24.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1624443868; 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=tsD6M/SGWrn53P3t19xYDlEI+1nAhtdvnv0Lr6L7Clc=; b=fBfU++ptfn34uexdAqke7tPO5zaNFF45KS1lWLOF1QqJcN8G2OgxbsI4FKcUJVg517sy7p SBh6lB85WCayYdji5CEN+VkZFrwtMn+qPrLkTX9LbPKZpCXDURiPT/Dlafsoc46Eu4iDW6 3LbIrWKNzzjNw8D4ds4oS1k76KkVprg= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-45-eRKvHJjUPVefHMkr-xyuzA-1; Wed, 23 Jun 2021 06:24:25 -0400 X-MC-Unique: eRKvHJjUPVefHMkr-xyuzA-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 5770C802B9F; Wed, 23 Jun 2021 10:24:24 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-200.ams2.redhat.com [10.36.112.200]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5B5FB60C05; Wed, 23 Jun 2021 10:24:23 +0000 (UTC) Subject: Re: [edk2-devel] SIMPLE_TEXT_OUTPUT_PROTOCOL and Video Resolution To: devel@edk2.groups.io, afish@apple.com, df7729@gmail.com References: <1688FC67A39CB5DB.19105@groups.io> From: "Laszlo Ersek" Message-ID: <77881bee-4efb-df6a-8cef-9a24eee343d3@redhat.com> Date: Wed, 23 Jun 2021 12:24:21 +0200 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=windows-1252 Content-Language: en-US Content-Transfer-Encoding: 7bit On 06/17/21 01:22, Andrew Fish via groups.io wrote: > > >> On Jun 16, 2021, at 2:45 PM, David F. wrote: >> >> Also, I found if there are 2 GOP handles and you change the mode of >> one, the other one doesn't reflect the change (but still doesn't solve >> anything with the original question), are you supposed to set the mode >> on every handle to keep that part in sync? >> > > A common implementation is to have the Conspliter [1] driver that produces virtual handles that aggregate how many actual devices you have and manages policy. > > You should grab the protocols on the gST->ConsoleOutHandle as these are the Spec defined active console devices. > > > In terms on Simple Text Output Protocol on Graphics this is the default driver in edk2 [2]. These are the config knobs you can set from your DSC file to control defaults. > > [Pcd] > gEfiMdeModulePkgTokenSpaceGuid.PcdVideoHorizontalResolution ## SOMETIMES_CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdVideoVerticalResolution ## SOMETIMES_CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow ## SOMETIMES_CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdConOutColumn ## SOMETIMES_CONSUMES > Note: If you have serial active that may mess with the Conspliter and force it to pick a lower resolution since it has to find the best match between the serial and graphics Simple Text In. So try the Graphics without the serial terminal connect to see if it does different stuff. Regarding the serial terminal aspect, we discussed the following patch (maybe: workaround?) for that, many years ago, off-list: diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c index a98b690c8b95..ded5513c74a7 100644 --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c @@ -115,9 +115,44 @@ TERMINAL_DEV mTerminalDevTemplate = { }; TERMINAL_CONSOLE_MODE_DATA mTerminalConsoleModeData[] = { - {80, 25}, - {80, 50}, - {100, 31}, + { 80, 25 }, // from graphics resolution 640 x 480 + { 80, 50 }, // from graphics resolution 640 x 960 + { 100, 25 }, // from graphics resolution 800 x 480 + { 100, 31 }, // from graphics resolution 800 x 600 + { 104, 32 }, // from graphics resolution 832 x 624 + { 120, 33 }, // from graphics resolution 960 x 640 + { 128, 31 }, // from graphics resolution 1024 x 600 + { 128, 40 }, // from graphics resolution 1024 x 768 + { 144, 45 }, // from graphics resolution 1152 x 864 + { 144, 45 }, // from graphics resolution 1152 x 870 + { 160, 37 }, // from graphics resolution 1280 x 720 + { 160, 40 }, // from graphics resolution 1280 x 760 + { 160, 40 }, // from graphics resolution 1280 x 768 + { 160, 42 }, // from graphics resolution 1280 x 800 + { 160, 50 }, // from graphics resolution 1280 x 960 + { 160, 53 }, // from graphics resolution 1280 x 1024 + { 170, 40 }, // from graphics resolution 1360 x 768 + { 170, 40 }, // from graphics resolution 1366 x 768 + { 175, 55 }, // from graphics resolution 1400 x 1050 + { 180, 47 }, // from graphics resolution 1440 x 900 + { 200, 47 }, // from graphics resolution 1600 x 900 + { 200, 63 }, // from graphics resolution 1600 x 1200 + { 210, 55 }, // from graphics resolution 1680 x 1050 + { 240, 56 }, // from graphics resolution 1920 x 1080 + { 240, 63 }, // from graphics resolution 1920 x 1200 + { 240, 75 }, // from graphics resolution 1920 x 1440 + { 250, 105 }, // from graphics resolution 2000 x 2000 + { 256, 80 }, // from graphics resolution 2048 x 1536 + { 256, 107 }, // from graphics resolution 2048 x 2048 + { 320, 75 }, // from graphics resolution 2560 x 1440 + { 320, 84 }, // from graphics resolution 2560 x 1600 + { 320, 107 }, // from graphics resolution 2560 x 2048 + { 350, 110 }, // from graphics resolution 2800 x 2100 + { 400, 126 }, // from graphics resolution 3200 x 2400 + { 480, 113 }, // from graphics resolution 3840 x 2160 + { 512, 113 }, // from graphics resolution 4096 x 2160 + { 960, 227 }, // from graphics resolution 7680 x 4320 + { 1024, 227 }, // from graphics resolution 8192 x 4320 // // New modes can be added here. // but the discussion didn't go anywhere over several months, so we've been carrying this patch downstream-only ever since. Thanks Laszlo > > > [1] https://github.com/tianocore/edk2/tree/master/MdeModulePkg/Universal/Console/ConSplitterDxe > > [2] https://github.com/tianocore/edk2/tree/master/MdeModulePkg/Universal/Console/GraphicsConsoleDxe > > > Thanks, > > Andrew Fish > >> On Tue, Jun 15, 2021 at 11:33 PM David F. via groups.io >> wrote: >>> >>> Hello, >>> >>> I've found that most implementation of UEFI don't automatically change >>> the resolution when setting the mode with STOP (Simple Text Output >>> Protocol) . You can use GOP to change it after the mode but that >>> causes other problems. For example, using surface pro 7 in this case, >>> with 4K screen. The default text mode is 342x96 which puts it in >>> 2736x1824 mode which you'd expect and the text is tiny. But now you >>> set the mode to 0 which is 80x25 and it actually sets the mode to >>> 2736x1824 if not already in that resolution and uses a 80x25 area in >>> the center of the screen, still tiny text you can hardly read. If you >>> then say you want GOP in 640x480 mode (which is available as GOP mode >>> 1 on this system, it will make the font larger but you can't see >>> anything because it's still offset to the middle of the 2736x1824 area >>> and you're only seeing the 640x480 upper left of that area on the >>> screen. Likewise if you have it in 342x96 so it's fully in the upper >>> left corner of the screen and change the mode to say 800x600 >>> (available as GOP mode 2 on this system) it will make the text >>> readable but the text can go off the screen in both directions because >>> it's still 342x96 when the 100x31 STOP mode would be the correct one >>> (which happens to be mode 2 on this system). >>> >>> Shouldn't setting the STOP mode handle adjusting the resolution since >>> that's the main reason you want to change the mode so the size shown >>> on the screen changes to something you can read. >>> >>> Any tricks? I've tried a bunch of things, resetting the controller, >>> using the Reset() protocol function, and other things but nothing >>> works. As soon as you use STOP to set the mode, it is back to high >>> resolution and using an area centered in the screen and changing the >>> resolution after that leaves it in the area centered in the high res >>> screen and not in the upper left area. >>> >>> Thanks. >>> >> >> >> >> >> > > > > > > >