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.web09.1515.1624476211297627301 for ; Wed, 23 Jun 2021 12:23:31 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=YF8U5L6T; 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=1624476210; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=r+R4mUA4d47TdOx4vjrediZay+ef1Gq/1rmEn/MRT8A=; b=YF8U5L6TtPBQ2a/yCoVOMAACEdzi0jmkpLhRtE3xSqRRT+qbc3NGoymMZMAFLdK6A0SmCK AVFPTcKUB/IwPek8TUMX0jZGwi6WBxXvIma1C5hf0FgrNYI0bEazeEf+p72FcxLXLdl9TQ Z4quuiWqFMjcK7FONfkgsq6SE70m7jY= 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-57-OlfpyWh-OPmw9l36IhoyHQ-1; Wed, 23 Jun 2021 15:23:27 -0400 X-MC-Unique: OlfpyWh-OPmw9l36IhoyHQ-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 7A8C718414A1; Wed, 23 Jun 2021 19:23:26 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-37.ams2.redhat.com [10.36.112.37]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 6D193608BA; Wed, 23 Jun 2021 19:23:25 +0000 (UTC) Subject: Re: [edk2-devel] SIMPLE_TEXT_OUTPUT_PROTOCOL and Video Resolution To: Andrew Fish , edk2-devel-groups-io Cc: df7729@gmail.com References: <1688FC67A39CB5DB.19105@groups.io> <77881bee-4efb-df6a-8cef-9a24eee343d3@redhat.com> <56E7A804-5FC0-4CB1-B59D-4A7612180879@apple.com> From: "Laszlo Ersek" Message-ID: <65e1ed1c-ef97-3e44-1498-2ee326e93083@redhat.com> Date: Wed, 23 Jun 2021 21:23:23 +0200 MIME-Version: 1.0 In-Reply-To: <56E7A804-5FC0-4CB1-B59D-4A7612180879@apple.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 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=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 06/23/21 18:57, Andrew Fish wrote: > > >> On Jun 23, 2021, at 3:24 AM, Laszlo Ersek wrote: >> >> 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: >> > > Laszlo, > > This reminds me I have some hacks in our terminal driver too. > 1) Add a mode that matches the Mode 2 of the GraphicsConsole driver. > a) This kind of assumes BDS connects the graphics 1st. > b) We could generalized this to match the higher modes of the ConSpliter? No capacity to think about this :) > 2) I added a PCD to make the Mode 2 match the current size terminal. > a) The EFI Terminal driver sends { ESC, '[', '1', '8', 't', 0 } to the terminal emulator app and gets the current size as a response. > b) I mostly use this for OVMF with no Graphics console. Nice :) > 3) I think 1) and 2) may require a PCD to the ConSpliter to pick the max mode vs. a known mode. It might be possible to hijack the current PCD values for this. > 4) At one point I added some code with 1) that would auto resize the terminal window to match the graphics screen resolution. I think this was one of the most unpopular things I’ve ever done at work. I guess I’m a just open a bunch of windows and switch between them person, but some people optimize the windows layout and get really angry if you resize windows on them. I even posted a variant of this to the list years ago, and we've been carrying it downstream since. It's based on a dynamic boolean PCD, and the PCD can be controlled on the QEMU command line with the "-fw_cfg" switch (so QEMU need not know any internals). By default, nothing changes, but if the PCD is set to TRUE, then TerminalDxe emits xterm control codes for resizing xterm. In general it's a really annoying feature, but for debugging issues in the UEFI shell (with a graphics console enabled too), it's very useful to have an auto-resized, super-wide terminal window. It's a debugging aid for sure. Thanks Laszlo > > Thanks, > > Andrew Fish > >> 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. >>>>> >>>> >>>> >>>> >>>> >>>> >>> >>> >>> >>> >>> >>> >>> >> >> >> >> > >