From: "Laszlo Ersek" <lersek@redhat.com>
To: Zhichao Gao <zhichao.gao@intel.com>, devel@edk2.groups.io
Cc: Jian J Wang <jian.j.wang@intel.com>, Hao Wu <hao.a.wu@intel.com>,
Ray Ni <ray.ni@intel.com>, Star Zeng <star.zeng@intel.com>,
Liming Gao <liming.gao@intel.com>,
Sean Brogan <sean.brogan@microsoft.com>,
Michael Turner <Michael.Turner@microsoft.com>,
Bret Barkelew <Bret.Barkelew@microsoft.com>
Subject: Re: [PATCH V4 2/2] MdeModulePkg/GraphicsConsoleDxe: Initialize the output mode
Date: Fri, 24 May 2019 11:40:44 +0200 [thread overview]
Message-ID: <67d7f12a-6c30-2922-74ac-a16790d0150c@redhat.com> (raw)
In-Reply-To: <20190524024009.31600-3-zhichao.gao@intel.com>
On 05/24/19 04:40, Zhichao Gao wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1412
>
> Original logic:
> Connect the graphics device -> connect it as graphics consoles
> and initialize its parameters(Mode = -1, invalid) -> connect it
> as console spliter and add the device to the list(use SetMode to
> set mode to the user defined mode or the best mode the devices
> supported if the mode is invalid. *clear the screen at this phase*)
>
> Changed logic:
> Connect the graphics device -> connect it as graphics consoles
> and initialize its parameters(initialize the mode to the user
> defined mode or the best mode. *directly set the mode value without
> using SetMode, that would not clear the screen) -> connect it as
> console spliter and add the device to the list(use SetMode to set
> mode to the user defined mode or the best mode the devices supported
> if the mode is invalid. *now the mode is already set, so it would
> not clear the screen*).
>
> Also remove the section of SetMode for debug version.
>
> Impact: as the text mode may not be an invalid value, the SetMode
> may have no chance to be called during reconnect the graphics device.
> That means the screen may not be cleaned after finishing reconnect
> operation. There is one common condition: shell command "recoonect -r".
>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Cc: Michael Turner <Michael.Turner@microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Zhichao Gao <zhichao.gao@intel.com>
> ---
> .../GraphicsConsoleDxe/GraphicsConsole.c | 42 ++++++++++++++-----
> .../GraphicsConsoleDxe/GraphicsConsoleDxe.inf | 2 +
> 2 files changed, 34 insertions(+), 10 deletions(-)
>
> diff --git a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
> index 26ea19f300..c042451a9b 100644
> --- a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
> +++ b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
> @@ -1,7 +1,7 @@
> /** @file
> This is the main routine for initializing the Graphics Console support routines.
>
> -Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.<BR>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
> @@ -384,6 +384,12 @@ GraphicsConsoleControllerDriverStart (
> EFI_GRAPHICS_OUTPUT_PROTOCOL_MODE *Mode;
> UINTN SizeOfInfo;
> EFI_GRAPHICS_OUTPUT_MODE_INFORMATION *Info;
> + INT32 PreferMode;
> + INT32 Index;
> + UINTN Column;
> + UINTN Row;
> + UINTN DefaultColumn;
> + UINTN DefaultRow;
>
> ModeNumber = 0;
>
> @@ -567,16 +573,32 @@ GraphicsConsoleControllerDriverStart (
> //
> Private->SimpleTextOutputMode.MaxMode = (INT32) MaxMode;
>
> - DEBUG_CODE_BEGIN ();
> - Status = GraphicsConsoleConOutSetMode (&Private->SimpleTextOutput, 0);
> - if (EFI_ERROR (Status)) {
> - goto Error;
> - }
> - Status = GraphicsConsoleConOutOutputString (&Private->SimpleTextOutput, (CHAR16 *)L"Graphics Console Started\n\r");
> - if (EFI_ERROR (Status)) {
> - goto Error;
> + //
> + // Initialize the Mode of graphics console devices
> + //
> + PreferMode = -1;
> + DefaultColumn = PcdGet32 (PcdConOutColumn);
> + DefaultRow = PcdGet32 (PcdConOutRow);
> + Column = 0;
> + Row = 0;
> + for (Index = 0; Index < (INT32)MaxMode; Index++) {
> + if (DefaultColumn != 0 && DefaultRow != 0) {
> + if ((Private->ModeData[Index].Columns == DefaultColumn) &&
> + (Private->ModeData[Index].Rows == DefaultRow)) {
> + PreferMode = Index;
> + break;
> + }
> + } else {
> + if ((Private->ModeData[Index].Columns > Column) &&
> + (Private->ModeData[Index].Rows > Row)) {
> + Column = Private->ModeData[Index].Columns;
> + Row = Private->ModeData[Index].Rows;
> + PreferMode = Index;
> + }
> }
> - DEBUG_CODE_END ();
> + }
> + Private->SimpleTextOutput.Mode->Mode = (INT32)PreferMode;
> + DEBUG ((DEBUG_INFO, "Graphics Console Started, Mode: %d\n", PreferMode));
>
> //
> // Install protocol interfaces for the Graphics Console device.
> diff --git a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.inf b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.inf
> index f7caa65aa9..bcfd306eee 100644
> --- a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.inf
> +++ b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.inf
> @@ -65,6 +65,8 @@
> [Pcd]
> gEfiMdeModulePkgTokenSpaceGuid.PcdVideoHorizontalResolution ## SOMETIMES_CONSUMES
> gEfiMdeModulePkgTokenSpaceGuid.PcdVideoVerticalResolution ## SOMETIMES_CONSUMES
> + gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow ## SOMETIMES_CONSUMES
> + gEfiMdeModulePkgTokenSpaceGuid.PcdConOutColumn ## SOMETIMES_CONSUMES
>
> [UserExtensions.TianoCore."ExtraFiles"]
> GraphicsConsoleDxeExtra.uni
>
Acked-by: Laszlo Ersek <lersek@redhat.com>
next prev parent reply other threads:[~2019-05-24 9:40 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-24 2:40 [PATCH V4 0/2] MdeModulePkg: Make the screen seamless Gao, Zhichao
2019-05-24 2:40 ` [PATCH V4 1/2] MdeModulePkg/ConSplitterDxe: Optimize the ConSplitterTextOutSetMode Gao, Zhichao
2019-05-27 1:38 ` Wu, Hao A
2019-05-24 2:40 ` [PATCH V4 2/2] MdeModulePkg/GraphicsConsoleDxe: Initialize the output mode Gao, Zhichao
2019-05-24 9:40 ` Laszlo Ersek [this message]
2019-05-27 1:51 ` Wu, Hao A
2019-05-27 2:15 ` [PATCH V4 0/2] MdeModulePkg: Make the screen seamless Wu, Hao A
2019-06-13 6:58 ` [edk2-devel] " Wu, Hao A
2019-06-13 10:00 ` Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=67d7f12a-6c30-2922-74ac-a16790d0150c@redhat.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox