MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++ 1 file changed, 3 insertions(+)
The commit b807174fecacf4c9f8400cab4d6fb3f580284021
("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for the
high display resolutions. Support these modes in the TerminalDxe as
well, so it would be possible to select the text mode that would take
all the available screen space.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
index e2d779c783..6b7b970516 100644
--- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
@@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA mTerminalConsoleModeData[] = {
{ 80, 25 },
{ 80, 50 },
{ 100, 31 },
+ { 128, 40 },
+ { 160, 42 },
+ { 240, 56 },
//
// New modes can be added here.
//
--
2.25.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93935): https://edk2.groups.io/g/devel/message/93935
Mute This Topic: https://groups.io/mt/93777396/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-This looks like a reasonable update to support platforms that have both
graphical consoles and serial consoles.
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> Sent: Monday, September 19, 2022 1:57 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> Aladyshev <aladyshev22@gmail.com>
> Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for the
> high display resolutions. Support these modes in the TerminalDxe as
> well, so it would be possible to select the text mode that would take
> all the available screen space.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> ---
> MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> index e2d779c783..6b7b970516 100644
> --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA mTerminalConsoleModeData[] = {
> { 80, 25 },
>
> { 80, 50 },
>
> { 100, 31 },
>
> + { 128, 40 },
>
> + { 160, 42 },
>
> + { 240, 56 },
>
> //
>
> // New modes can be added here.
>
> //
>
> --
> 2.25.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#93935): https://edk2.groups.io/g/devel/message/93935
> Mute This Topic: https://groups.io/mt/93777396/1643496
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [michael.d.kinney@intel.com]
> -=-=-=-=-=-=
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93965): https://edk2.groups.io/g/devel/message/93965
Mute This Topic: https://groups.io/mt/93777396/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Konstantin:
Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
I try Putty in my desktop. It can't display 240 * 56 in one screen.
Thanks
Liming
> -----邮件原件-----
> 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> 发送时间: 2022年9月20日 5:02
> 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> <michael.d.kinney@intel.com>
> 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> This looks like a reasonable update to support platforms that have both
> graphical consoles and serial consoles.
>
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
>
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Konstantin Aladyshev
> > Sent: Monday, September 19, 2022 1:57 AM
> > To: devel@edk2.groups.io
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > Aladyshev <aladyshev22@gmail.com>
> > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> >
> > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> the
> > high display resolutions. Support these modes in the TerminalDxe as
> > well, so it would be possible to select the text mode that would take
> > all the available screen space.
> >
> > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > ---
> > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > index e2d779c783..6b7b970516 100644
> > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> mTerminalConsoleModeData[] = {
> > { 80, 25 },
> >
> > { 80, 50 },
> >
> > { 100, 31 },
> >
> > + { 128, 40 },
> >
> > + { 160, 42 },
> >
> > + { 240, 56 },
> >
> > //
> >
> > // New modes can be added here.
> >
> > //
> >
> > --
> > 2.25.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#93935):
> https://edk2.groups.io/g/devel/message/93935
> > Mute This Topic: https://groups.io/mt/93777396/1643496
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [michael.d.kinney@intel.com]
> > -=-=-=-=-=-=
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94024): https://edk2.groups.io/g/devel/message/94024
Mute This Topic: https://groups.io/mt/93818296/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Tera Term works.
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> Sent: Tuesday, September 20, 2022 6:26 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> Konstantin:
> Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> I try Putty in my desktop. It can't display 240 * 56 in one screen.
>
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > 发送时间: 2022年9月20日 5:02
> > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> >
> > This looks like a reasonable update to support platforms that have both
> > graphical consoles and serial consoles.
> >
> > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> >
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > Konstantin Aladyshev
> > > Sent: Monday, September 19, 2022 1:57 AM
> > > To: devel@edk2.groups.io
> > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > Aladyshev <aladyshev22@gmail.com>
> > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > >
> > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > the
> > > high display resolutions. Support these modes in the TerminalDxe as
> > > well, so it would be possible to select the text mode that would take
> > > all the available screen space.
> > >
> > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > ---
> > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > 1 file changed, 3 insertions(+)
> > >
> > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > index e2d779c783..6b7b970516 100644
> > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > mTerminalConsoleModeData[] = {
> > > { 80, 25 },
> > >
> > > { 80, 50 },
> > >
> > > { 100, 31 },
> > >
> > > + { 128, 40 },
> > >
> > > + { 160, 42 },
> > >
> > > + { 240, 56 },
> > >
> > > //
> > >
> > > // New modes can be added here.
> > >
> > > //
> > >
> > > --
> > > 2.25.1
> > >
> > >
> > >
> > > -=-=-=-=-=-=
> > > Groups.io Links: You receive all messages sent to this group.
> > > View/Reply Online (#93935):
> > https://edk2.groups.io/g/devel/message/93935
> > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > Group Owner: devel+owner@edk2.groups.io
> > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > [michael.d.kinney@intel.com]
> > > -=-=-=-=-=-=
> > >
>
>
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94026): https://edk2.groups.io/g/devel/message/94026
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
I have tested it in the following way:
- launch OVMF in Linux and connect with vnc
- change resolution via the form browser (Device Manager -> OVMF
Platform Configuration) from the default 640x480 to the 1920x1080
- "reset"
- after the reset the system will boot to the 1920x1080 resolution and
text mode 80x25
- check "mode" command output
Before the commit this command displays only these modes:
```
Shell> mode
Available modes for console output device.
Col 80 Row 25 *
Col 80 Row 50
Col 100 Row 31
```
But with this commit the high resolution modes are present in the output:
```
Shell> mode
Available modes for console output device.
Col 80 Row 25 *
Col 80 Row 50
Col 100 Row 31
Col 128 Row 40
Col 160 Row 42
Col 240 Row 56
```
And in my case setting the appropriate "mode 240 56" at runtime works just fine:
```
Shell> mode 240 56
```
After that the text output starts using the full screen.
Best regards,
Konstantin Aladyshev
On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
<michael.d.kinney@intel.com> wrote:
>
> Tera Term works.
>
> Mike
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > Sent: Tuesday, September 20, 2022 6:26 PM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> >
> > Konstantin:
> > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > 发送时间: 2022年9月20日 5:02
> > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > >
> > > This looks like a reasonable update to support platforms that have both
> > > graphical consoles and serial consoles.
> > >
> > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > >
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > Konstantin Aladyshev
> > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > To: devel@edk2.groups.io
> > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > Aladyshev <aladyshev22@gmail.com>
> > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > >
> > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > the
> > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > well, so it would be possible to select the text mode that would take
> > > > all the available screen space.
> > > >
> > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > ---
> > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > 1 file changed, 3 insertions(+)
> > > >
> > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > index e2d779c783..6b7b970516 100644
> > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > mTerminalConsoleModeData[] = {
> > > > { 80, 25 },
> > > >
> > > > { 80, 50 },
> > > >
> > > > { 100, 31 },
> > > >
> > > > + { 128, 40 },
> > > >
> > > > + { 160, 42 },
> > > >
> > > > + { 240, 56 },
> > > >
> > > > //
> > > >
> > > > // New modes can be added here.
> > > >
> > > > //
> > > >
> > > > --
> > > > 2.25.1
> > > >
> > > >
> > > >
> > > > -=-=-=-=-=-=
> > > > Groups.io Links: You receive all messages sent to this group.
> > > > View/Reply Online (#93935):
> > > https://edk2.groups.io/g/devel/message/93935
> > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > Group Owner: devel+owner@edk2.groups.io
> > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > [michael.d.kinney@intel.com]
> > > > -=-=-=-=-=-=
> > > >
> >
> >
> >
> >
> >
> >
> >
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94056): https://edk2.groups.io/g/devel/message/94056
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
The other way to get full screen text on graphical console is to disable the serial console
in the Boot Maintenance Manager.
If the serial console is disabled then ConSplitterDxe will only see the text modes supported
by GraphicsConsoleDxe which will include these higher text modes.
Is the issue that you are trying to solve supporting larger serial console modes or just
wanting to use the largest possible text mode at a given graphics resolution?
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> Sent: Wednesday, September 21, 2022 1:24 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> <ray.ni@intel.com>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> I have tested it in the following way:
> - launch OVMF in Linux and connect with vnc
> - change resolution via the form browser (Device Manager -> OVMF
> Platform Configuration) from the default 640x480 to the 1920x1080
> - "reset"
> - after the reset the system will boot to the 1920x1080 resolution and
> text mode 80x25
> - check "mode" command output
>
> Before the commit this command displays only these modes:
> ```
> Shell> mode
> Available modes for console output device.
> Col 80 Row 25 *
> Col 80 Row 50
> Col 100 Row 31
> ```
> But with this commit the high resolution modes are present in the output:
> ```
> Shell> mode
> Available modes for console output device.
> Col 80 Row 25 *
> Col 80 Row 50
> Col 100 Row 31
> Col 128 Row 40
> Col 160 Row 42
> Col 240 Row 56
> ```
> And in my case setting the appropriate "mode 240 56" at runtime works just fine:
> ```
> Shell> mode 240 56
> ```
> After that the text output starts using the full screen.
>
> Best regards,
> Konstantin Aladyshev
>
> On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
> <michael.d.kinney@intel.com> wrote:
> >
> > Tera Term works.
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > > Sent: Tuesday, September 20, 2022 6:26 PM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > >
> > > Konstantin:
> > > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> > >
> > > Thanks
> > > Liming
> > > > -----邮件原件-----
> > > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > 发送时间: 2022年9月20日 5:02
> > > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > >
> > > > This looks like a reasonable update to support platforms that have both
> > > > graphical consoles and serial consoles.
> > > >
> > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > Konstantin Aladyshev
> > > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > > Aladyshev <aladyshev22@gmail.com>
> > > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > >
> > > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > > the
> > > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > > well, so it would be possible to select the text mode that would take
> > > > > all the available screen space.
> > > > >
> > > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > > ---
> > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > > 1 file changed, 3 insertions(+)
> > > > >
> > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > index e2d779c783..6b7b970516 100644
> > > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > > mTerminalConsoleModeData[] = {
> > > > > { 80, 25 },
> > > > >
> > > > > { 80, 50 },
> > > > >
> > > > > { 100, 31 },
> > > > >
> > > > > + { 128, 40 },
> > > > >
> > > > > + { 160, 42 },
> > > > >
> > > > > + { 240, 56 },
> > > > >
> > > > > //
> > > > >
> > > > > // New modes can be added here.
> > > > >
> > > > > //
> > > > >
> > > > > --
> > > > > 2.25.1
> > > > >
> > > > >
> > > > >
> > > > > -=-=-=-=-=-=
> > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > View/Reply Online (#93935):
> > > > https://edk2.groups.io/g/devel/message/93935
> > > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > [michael.d.kinney@intel.com]
> > > > > -=-=-=-=-=-=
> > > > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> >
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94083): https://edk2.groups.io/g/devel/message/94083
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
> Is the issue that you are trying to solve supporting larger serial console modes or just wanting to use the largest possible text mode at a given graphics resolution?
I spotted this issue when I wanted to use the largest possible text
mode in UEFI shell at high graphics resolutions. Is it not enough
reason for this commit?
> The other way to get full screen text on the graphical console is to disable the serial console in the Boot Maintenance Manager.
Can you please elaborate this? I've tried to disable UART in "Boot
Maintenance Manager -> Console Options" like this:
```
Console Input Device Select:
PciRoot(0x0)/Pci(0x1,0x0)/Acpi(PNP0303,0x0) [X]
PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
Console Output Device Select:
PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
Console Standard Error Device Select:
PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
```
But after reset, the console is 80x25 and all UART options are checked again.
Also there is a menu "Boot Maintenance Manager -> Console Options ->
Console Output Mode Select" that has "Set Console Output Mode", and it
is always "80x25" without any options for me.
Best regards,
Konstantin Aladyshev
On Wed, Sep 21, 2022 at 7:28 PM Kinney, Michael D
<michael.d.kinney@intel.com> wrote:
>
> The other way to get full screen text on graphical console is to disable the serial console
> in the Boot Maintenance Manager.
>
> If the serial console is disabled then ConSplitterDxe will only see the text modes supported
> by GraphicsConsoleDxe which will include these higher text modes.
>
> Is the issue that you are trying to solve supporting larger serial console modes or just
> wanting to use the largest possible text mode at a given graphics resolution?
>
> Mike
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> > Sent: Wednesday, September 21, 2022 1:24 AM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > <ray.ni@intel.com>
> > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> >
> > I have tested it in the following way:
> > - launch OVMF in Linux and connect with vnc
> > - change resolution via the form browser (Device Manager -> OVMF
> > Platform Configuration) from the default 640x480 to the 1920x1080
> > - "reset"
> > - after the reset the system will boot to the 1920x1080 resolution and
> > text mode 80x25
> > - check "mode" command output
> >
> > Before the commit this command displays only these modes:
> > ```
> > Shell> mode
> > Available modes for console output device.
> > Col 80 Row 25 *
> > Col 80 Row 50
> > Col 100 Row 31
> > ```
> > But with this commit the high resolution modes are present in the output:
> > ```
> > Shell> mode
> > Available modes for console output device.
> > Col 80 Row 25 *
> > Col 80 Row 50
> > Col 100 Row 31
> > Col 128 Row 40
> > Col 160 Row 42
> > Col 240 Row 56
> > ```
> > And in my case setting the appropriate "mode 240 56" at runtime works just fine:
> > ```
> > Shell> mode 240 56
> > ```
> > After that the text output starts using the full screen.
> >
> > Best regards,
> > Konstantin Aladyshev
> >
> > On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
> > <michael.d.kinney@intel.com> wrote:
> > >
> > > Tera Term works.
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > > > Sent: Tuesday, September 20, 2022 6:26 PM
> > > > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > >
> > > > Konstantin:
> > > > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > > > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> > > >
> > > > Thanks
> > > > Liming
> > > > > -----邮件原件-----
> > > > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > > 发送时间: 2022年9月20日 5:02
> > > > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > > > <michael.d.kinney@intel.com>
> > > > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > >
> > > > > This looks like a reasonable update to support platforms that have both
> > > > > graphical consoles and serial consoles.
> > > > >
> > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > Konstantin Aladyshev
> > > > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > > > To: devel@edk2.groups.io
> > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > > > Aladyshev <aladyshev22@gmail.com>
> > > > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > >
> > > > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > > > the
> > > > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > > > well, so it would be possible to select the text mode that would take
> > > > > > all the available screen space.
> > > > > >
> > > > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > > > ---
> > > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > > > 1 file changed, 3 insertions(+)
> > > > > >
> > > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > index e2d779c783..6b7b970516 100644
> > > > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > > > mTerminalConsoleModeData[] = {
> > > > > > { 80, 25 },
> > > > > >
> > > > > > { 80, 50 },
> > > > > >
> > > > > > { 100, 31 },
> > > > > >
> > > > > > + { 128, 40 },
> > > > > >
> > > > > > + { 160, 42 },
> > > > > >
> > > > > > + { 240, 56 },
> > > > > >
> > > > > > //
> > > > > >
> > > > > > // New modes can be added here.
> > > > > >
> > > > > > //
> > > > > >
> > > > > > --
> > > > > > 2.25.1
> > > > > >
> > > > > >
> > > > > >
> > > > > > -=-=-=-=-=-=
> > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > View/Reply Online (#93935):
> > > > > https://edk2.groups.io/g/devel/message/93935
> > > > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > [michael.d.kinney@intel.com]
> > > > > > -=-=-=-=-=-=
> > > > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > >
> >
> >
> >
> >
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94114): https://edk2.groups.io/g/devel/message/94114
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
If the goal is to support a mix of graphical and serial consoles both with high text resolutions, then your patch is appropriate.
It looks like you are observing some unexpected limitations in the OVMF console management.
Perhaps the OVMF maintainers can provide some advice on how to configure OVMF with serial consoles disabled.
Are you using QEMU to run OVMF? Are you including UARTs in QEMU launch?
Mike
> -----Original Message-----
> From: Konstantin Aladyshev <aladyshev22@gmail.com>
> Sent: Thursday, September 22, 2022 1:23 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> <ray.ni@intel.com>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> > Is the issue that you are trying to solve supporting larger serial console modes or just wanting to use the largest possible
> text mode at a given graphics resolution?
>
> I spotted this issue when I wanted to use the largest possible text
> mode in UEFI shell at high graphics resolutions. Is it not enough
> reason for this commit?
>
> > The other way to get full screen text on the graphical console is to disable the serial console in the Boot Maintenance
> Manager.
>
> Can you please elaborate this? I've tried to disable UART in "Boot
> Maintenance Manager -> Console Options" like this:
> ```
> Console Input Device Select:
> PciRoot(0x0)/Pci(0x1,0x0)/Acpi(PNP0303,0x0) [X]
> PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
>
> Console Output Device Select:
> PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
>
> Console Standard Error Device Select:
> PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> ```
> But after reset, the console is 80x25 and all UART options are checked again.
>
> Also there is a menu "Boot Maintenance Manager -> Console Options ->
> Console Output Mode Select" that has "Set Console Output Mode", and it
> is always "80x25" without any options for me.
>
> Best regards,
> Konstantin Aladyshev
>
> On Wed, Sep 21, 2022 at 7:28 PM Kinney, Michael D
> <michael.d.kinney@intel.com> wrote:
> >
> > The other way to get full screen text on graphical console is to disable the serial console
> > in the Boot Maintenance Manager.
> >
> > If the serial console is disabled then ConSplitterDxe will only see the text modes supported
> > by GraphicsConsoleDxe which will include these higher text modes.
> >
> > Is the issue that you are trying to solve supporting larger serial console modes or just
> > wanting to use the largest possible text mode at a given graphics resolution?
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> > > Sent: Wednesday, September 21, 2022 1:24 AM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > > <ray.ni@intel.com>
> > > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > >
> > > I have tested it in the following way:
> > > - launch OVMF in Linux and connect with vnc
> > > - change resolution via the form browser (Device Manager -> OVMF
> > > Platform Configuration) from the default 640x480 to the 1920x1080
> > > - "reset"
> > > - after the reset the system will boot to the 1920x1080 resolution and
> > > text mode 80x25
> > > - check "mode" command output
> > >
> > > Before the commit this command displays only these modes:
> > > ```
> > > Shell> mode
> > > Available modes for console output device.
> > > Col 80 Row 25 *
> > > Col 80 Row 50
> > > Col 100 Row 31
> > > ```
> > > But with this commit the high resolution modes are present in the output:
> > > ```
> > > Shell> mode
> > > Available modes for console output device.
> > > Col 80 Row 25 *
> > > Col 80 Row 50
> > > Col 100 Row 31
> > > Col 128 Row 40
> > > Col 160 Row 42
> > > Col 240 Row 56
> > > ```
> > > And in my case setting the appropriate "mode 240 56" at runtime works just fine:
> > > ```
> > > Shell> mode 240 56
> > > ```
> > > After that the text output starts using the full screen.
> > >
> > > Best regards,
> > > Konstantin Aladyshev
> > >
> > > On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
> > > <michael.d.kinney@intel.com> wrote:
> > > >
> > > > Tera Term works.
> > > >
> > > > Mike
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > > > > Sent: Tuesday, September 20, 2022 6:26 PM
> > > > > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > > > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > >
> > > > > Konstantin:
> > > > > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > > > > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> > > > >
> > > > > Thanks
> > > > > Liming
> > > > > > -----邮件原件-----
> > > > > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > > > 发送时间: 2022年9月20日 5:02
> > > > > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > > > > <michael.d.kinney@intel.com>
> > > > > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > > > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > >
> > > > > > This looks like a reasonable update to support platforms that have both
> > > > > > graphical consoles and serial consoles.
> > > > > >
> > > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > > > > >
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > > Konstantin Aladyshev
> > > > > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > > > > To: devel@edk2.groups.io
> > > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > > > > Aladyshev <aladyshev22@gmail.com>
> > > > > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > >
> > > > > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > > > > the
> > > > > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > > > > well, so it would be possible to select the text mode that would take
> > > > > > > all the available screen space.
> > > > > > >
> > > > > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > > > > ---
> > > > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > > > > 1 file changed, 3 insertions(+)
> > > > > > >
> > > > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > index e2d779c783..6b7b970516 100644
> > > > > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > > > > mTerminalConsoleModeData[] = {
> > > > > > > { 80, 25 },
> > > > > > >
> > > > > > > { 80, 50 },
> > > > > > >
> > > > > > > { 100, 31 },
> > > > > > >
> > > > > > > + { 128, 40 },
> > > > > > >
> > > > > > > + { 160, 42 },
> > > > > > >
> > > > > > > + { 240, 56 },
> > > > > > >
> > > > > > > //
> > > > > > >
> > > > > > > // New modes can be added here.
> > > > > > >
> > > > > > > //
> > > > > > >
> > > > > > > --
> > > > > > > 2.25.1
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > -=-=-=-=-=-=
> > > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > > View/Reply Online (#93935):
> > > > > > https://edk2.groups.io/g/devel/message/93935
> > > > > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > > [michael.d.kinney@intel.com]
> > > > > > > -=-=-=-=-=-=
> > > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > >
> > >
> > >
> > >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94121): https://edk2.groups.io/g/devel/message/94121
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Yes, I use QEMU to run OVMF in Linux (actually in WSL). Here is a
command that I use for launch:
```
qemu-system-x86_64 \
-drive if=pflash,format=raw,file=Build/OvmfX64/RELEASE_GCC5/FV/OVMF.fd \
-drive format=raw,file=fat:rw:~/UEFI_disk
-vnc :1
```
So nothing explicit about the UARTs.
Best regards,
Konstantin Aladyshev
On Thu, Sep 22, 2022 at 7:12 PM Kinney, Michael D
<michael.d.kinney@intel.com> wrote:
>
> If the goal is to support a mix of graphical and serial consoles both with high text resolutions, then your patch is appropriate.
>
> It looks like you are observing some unexpected limitations in the OVMF console management.
>
> Perhaps the OVMF maintainers can provide some advice on how to configure OVMF with serial consoles disabled.
>
> Are you using QEMU to run OVMF? Are you including UARTs in QEMU launch?
>
> Mike
>
> > -----Original Message-----
> > From: Konstantin Aladyshev <aladyshev22@gmail.com>
> > Sent: Thursday, September 22, 2022 1:23 AM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > <ray.ni@intel.com>
> > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> >
> > > Is the issue that you are trying to solve supporting larger serial console modes or just wanting to use the largest possible
> > text mode at a given graphics resolution?
> >
> > I spotted this issue when I wanted to use the largest possible text
> > mode in UEFI shell at high graphics resolutions. Is it not enough
> > reason for this commit?
> >
> > > The other way to get full screen text on the graphical console is to disable the serial console in the Boot Maintenance
> > Manager.
> >
> > Can you please elaborate this? I've tried to disable UART in "Boot
> > Maintenance Manager -> Console Options" like this:
> > ```
> > Console Input Device Select:
> > PciRoot(0x0)/Pci(0x1,0x0)/Acpi(PNP0303,0x0) [X]
> > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> >
> > Console Output Device Select:
> > PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> >
> > Console Standard Error Device Select:
> > PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> > ```
> > But after reset, the console is 80x25 and all UART options are checked again.
> >
> > Also there is a menu "Boot Maintenance Manager -> Console Options ->
> > Console Output Mode Select" that has "Set Console Output Mode", and it
> > is always "80x25" without any options for me.
> >
> > Best regards,
> > Konstantin Aladyshev
> >
> > On Wed, Sep 21, 2022 at 7:28 PM Kinney, Michael D
> > <michael.d.kinney@intel.com> wrote:
> > >
> > > The other way to get full screen text on graphical console is to disable the serial console
> > > in the Boot Maintenance Manager.
> > >
> > > If the serial console is disabled then ConSplitterDxe will only see the text modes supported
> > > by GraphicsConsoleDxe which will include these higher text modes.
> > >
> > > Is the issue that you are trying to solve supporting larger serial console modes or just
> > > wanting to use the largest possible text mode at a given graphics resolution?
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> > > > Sent: Wednesday, September 21, 2022 1:24 AM
> > > > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > > > <ray.ni@intel.com>
> > > > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > >
> > > > I have tested it in the following way:
> > > > - launch OVMF in Linux and connect with vnc
> > > > - change resolution via the form browser (Device Manager -> OVMF
> > > > Platform Configuration) from the default 640x480 to the 1920x1080
> > > > - "reset"
> > > > - after the reset the system will boot to the 1920x1080 resolution and
> > > > text mode 80x25
> > > > - check "mode" command output
> > > >
> > > > Before the commit this command displays only these modes:
> > > > ```
> > > > Shell> mode
> > > > Available modes for console output device.
> > > > Col 80 Row 25 *
> > > > Col 80 Row 50
> > > > Col 100 Row 31
> > > > ```
> > > > But with this commit the high resolution modes are present in the output:
> > > > ```
> > > > Shell> mode
> > > > Available modes for console output device.
> > > > Col 80 Row 25 *
> > > > Col 80 Row 50
> > > > Col 100 Row 31
> > > > Col 128 Row 40
> > > > Col 160 Row 42
> > > > Col 240 Row 56
> > > > ```
> > > > And in my case setting the appropriate "mode 240 56" at runtime works just fine:
> > > > ```
> > > > Shell> mode 240 56
> > > > ```
> > > > After that the text output starts using the full screen.
> > > >
> > > > Best regards,
> > > > Konstantin Aladyshev
> > > >
> > > > On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
> > > > <michael.d.kinney@intel.com> wrote:
> > > > >
> > > > > Tera Term works.
> > > > >
> > > > > Mike
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > > > > > Sent: Tuesday, September 20, 2022 6:26 PM
> > > > > > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > > > > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > >
> > > > > > Konstantin:
> > > > > > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > > > > > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> > > > > >
> > > > > > Thanks
> > > > > > Liming
> > > > > > > -----邮件原件-----
> > > > > > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > > > > 发送时间: 2022年9月20日 5:02
> > > > > > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > > > > > <michael.d.kinney@intel.com>
> > > > > > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > > > > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > >
> > > > > > > This looks like a reasonable update to support platforms that have both
> > > > > > > graphical consoles and serial consoles.
> > > > > > >
> > > > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > > > > > >
> > > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > > > Konstantin Aladyshev
> > > > > > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > > > > > To: devel@edk2.groups.io
> > > > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > > > > > Aladyshev <aladyshev22@gmail.com>
> > > > > > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > > >
> > > > > > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > > > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > > > > > the
> > > > > > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > > > > > well, so it would be possible to select the text mode that would take
> > > > > > > > all the available screen space.
> > > > > > > >
> > > > > > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > > > > > ---
> > > > > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > > > > > 1 file changed, 3 insertions(+)
> > > > > > > >
> > > > > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > index e2d779c783..6b7b970516 100644
> > > > > > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > > > > > mTerminalConsoleModeData[] = {
> > > > > > > > { 80, 25 },
> > > > > > > >
> > > > > > > > { 80, 50 },
> > > > > > > >
> > > > > > > > { 100, 31 },
> > > > > > > >
> > > > > > > > + { 128, 40 },
> > > > > > > >
> > > > > > > > + { 160, 42 },
> > > > > > > >
> > > > > > > > + { 240, 56 },
> > > > > > > >
> > > > > > > > //
> > > > > > > >
> > > > > > > > // New modes can be added here.
> > > > > > > >
> > > > > > > > //
> > > > > > > >
> > > > > > > > --
> > > > > > > > 2.25.1
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > -=-=-=-=-=-=
> > > > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > > > View/Reply Online (#93935):
> > > > > > > https://edk2.groups.io/g/devel/message/93935
> > > > > > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > > > [michael.d.kinney@intel.com]
> > > > > > > > -=-=-=-=-=-=
> > > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > >
> > > >
> > >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94123): https://edk2.groups.io/g/devel/message/94123
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
On Thu, Sep 22, 2022 at 07:49:39PM +0300, Konstantin Aladyshev wrote: > Yes, I use QEMU to run OVMF in Linux (actually in WSL). Here is a > command that I use for launch: > ``` > qemu-system-x86_64 \ > -drive if=pflash,format=raw,file=Build/OvmfX64/RELEASE_GCC5/FV/OVMF.fd \ > -drive format=raw,file=fat:rw:~/UEFI_disk > -vnc :1 > ``` > So nothing explicit about the UARTs. qemu adds a bunch of default devices for convenience. vga, nic, cdrom, serial port. You can use '-nodefaults' to turn off all of them or '-serial none' to turn off the serial port only. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94207): https://edk2.groups.io/g/devel/message/94207 Mute This Topic: https://groups.io/mt/93818606/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
You can add the following to disable all UARTs
-serial none
That gives me a graphics only console.
There is something strange with the Boot Maintenance Manager for OVMF.
The console settings are not preserved across a reset. They are
preserved for the current boot. If you go to the shell after console
changes, then only the currently configured consoles are active.
Mike
> -----Original Message-----
> From: Konstantin Aladyshev <aladyshev22@gmail.com>
> Sent: Thursday, September 22, 2022 9:50 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> <ray.ni@intel.com>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
>
> Yes, I use QEMU to run OVMF in Linux (actually in WSL). Here is a
> command that I use for launch:
> ```
> qemu-system-x86_64 \
> -drive if=pflash,format=raw,file=Build/OvmfX64/RELEASE_GCC5/FV/OVMF.fd \
> -drive format=raw,file=fat:rw:~/UEFI_disk
> -vnc :1
> ```
> So nothing explicit about the UARTs.
>
> Best regards,
> Konstantin Aladyshev
>
> On Thu, Sep 22, 2022 at 7:12 PM Kinney, Michael D
> <michael.d.kinney@intel.com> wrote:
> >
> > If the goal is to support a mix of graphical and serial consoles both with high text resolutions, then your patch is
> appropriate.
> >
> > It looks like you are observing some unexpected limitations in the OVMF console management.
> >
> > Perhaps the OVMF maintainers can provide some advice on how to configure OVMF with serial consoles disabled.
> >
> > Are you using QEMU to run OVMF? Are you including UARTs in QEMU launch?
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > Sent: Thursday, September 22, 2022 1:23 AM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > > <ray.ni@intel.com>
> > > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > >
> > > > Is the issue that you are trying to solve supporting larger serial console modes or just wanting to use the largest
> possible
> > > text mode at a given graphics resolution?
> > >
> > > I spotted this issue when I wanted to use the largest possible text
> > > mode in UEFI shell at high graphics resolutions. Is it not enough
> > > reason for this commit?
> > >
> > > > The other way to get full screen text on the graphical console is to disable the serial console in the Boot Maintenance
> > > Manager.
> > >
> > > Can you please elaborate this? I've tried to disable UART in "Boot
> > > Maintenance Manager -> Console Options" like this:
> > > ```
> > > Console Input Device Select:
> > > PciRoot(0x0)/Pci(0x1,0x0)/Acpi(PNP0303,0x0) [X]
> > > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> > >
> > > Console Output Device Select:
> > > PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> > > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> > >
> > > Console Standard Error Device Select:
> > > PciRoot(0x0)/Pci(0x2,0x0)/AcpiAdr(0x80010100) [X]
> > > PciRoot(0x0)/Pci(0x1,0x0)/Serial(0x0)/Uart(115200,8,N,1) [ ]
> > > ```
> > > But after reset, the console is 80x25 and all UART options are checked again.
> > >
> > > Also there is a menu "Boot Maintenance Manager -> Console Options ->
> > > Console Output Mode Select" that has "Set Console Output Mode", and it
> > > is always "80x25" without any options for me.
> > >
> > > Best regards,
> > > Konstantin Aladyshev
> > >
> > > On Wed, Sep 21, 2022 at 7:28 PM Kinney, Michael D
> > > <michael.d.kinney@intel.com> wrote:
> > > >
> > > > The other way to get full screen text on graphical console is to disable the serial console
> > > > in the Boot Maintenance Manager.
> > > >
> > > > If the serial console is disabled then ConSplitterDxe will only see the text modes supported
> > > > by GraphicsConsoleDxe which will include these higher text modes.
> > > >
> > > > Is the issue that you are trying to solve supporting larger serial console modes or just
> > > > wanting to use the largest possible text mode at a given graphics resolution?
> > > >
> > > > Mike
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Konstantin Aladyshev
> > > > > Sent: Wednesday, September 21, 2022 1:24 AM
> > > > > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > > Cc: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>; Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray
> > > > > <ray.ni@intel.com>
> > > > > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > >
> > > > > I have tested it in the following way:
> > > > > - launch OVMF in Linux and connect with vnc
> > > > > - change resolution via the form browser (Device Manager -> OVMF
> > > > > Platform Configuration) from the default 640x480 to the 1920x1080
> > > > > - "reset"
> > > > > - after the reset the system will boot to the 1920x1080 resolution and
> > > > > text mode 80x25
> > > > > - check "mode" command output
> > > > >
> > > > > Before the commit this command displays only these modes:
> > > > > ```
> > > > > Shell> mode
> > > > > Available modes for console output device.
> > > > > Col 80 Row 25 *
> > > > > Col 80 Row 50
> > > > > Col 100 Row 31
> > > > > ```
> > > > > But with this commit the high resolution modes are present in the output:
> > > > > ```
> > > > > Shell> mode
> > > > > Available modes for console output device.
> > > > > Col 80 Row 25 *
> > > > > Col 80 Row 50
> > > > > Col 100 Row 31
> > > > > Col 128 Row 40
> > > > > Col 160 Row 42
> > > > > Col 240 Row 56
> > > > > ```
> > > > > And in my case setting the appropriate "mode 240 56" at runtime works just fine:
> > > > > ```
> > > > > Shell> mode 240 56
> > > > > ```
> > > > > After that the text output starts using the full screen.
> > > > >
> > > > > Best regards,
> > > > > Konstantin Aladyshev
> > > > >
> > > > > On Wed, Sep 21, 2022 at 4:46 AM Kinney, Michael D
> > > > > <michael.d.kinney@intel.com> wrote:
> > > > > >
> > > > > > Tera Term works.
> > > > > >
> > > > > > Mike
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of gaoliming via groups.io
> > > > > > > Sent: Tuesday, September 20, 2022 6:26 PM
> > > > > > > To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io; aladyshev22@gmail.com
> > > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > > > > > Subject: 回复: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > >
> > > > > > > Konstantin:
> > > > > > > Do you try the terminal software (such as Putty) to display the text mode 160, 42 or 240, 56?
> > > > > > > I try Putty in my desktop. It can't display 240 * 56 in one screen.
> > > > > > >
> > > > > > > Thanks
> > > > > > > Liming
> > > > > > > > -----邮件原件-----
> > > > > > > > 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > > > > > 发送时间: 2022年9月20日 5:02
> > > > > > > > 收件人: devel@edk2.groups.io; aladyshev22@gmail.com; Kinney, Michael D
> > > > > > > > <michael.d.kinney@intel.com>
> > > > > > > > 抄送: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>
> > > > > > > > 主题: RE: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > > >
> > > > > > > > This looks like a reasonable update to support platforms that have both
> > > > > > > > graphical consoles and serial consoles.
> > > > > > > >
> > > > > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> > > > > > > >
> > > > > > > >
> > > > > > > > > -----Original Message-----
> > > > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > > > > Konstantin Aladyshev
> > > > > > > > > Sent: Monday, September 19, 2022 1:57 AM
> > > > > > > > > To: devel@edk2.groups.io
> > > > > > > > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Gao, Liming
> > > > > > > > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com>; Konstantin
> > > > > > > > > Aladyshev <aladyshev22@gmail.com>
> > > > > > > > > Subject: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
> > > > > > > > >
> > > > > > > > > The commit b807174fecacf4c9f8400cab4d6fb3f580284021
> > > > > > > > > ("MdeModulePkg/GraphicsConsoleDxe: add modes") has added modes for
> > > > > > > > the
> > > > > > > > > high display resolutions. Support these modes in the TerminalDxe as
> > > > > > > > > well, so it would be possible to select the text mode that would take
> > > > > > > > > all the available screen space.
> > > > > > > > >
> > > > > > > > > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > > > > > > > > ---
> > > > > > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++
> > > > > > > > > 1 file changed, 3 insertions(+)
> > > > > > > > >
> > > > > > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > > index e2d779c783..6b7b970516 100644
> > > > > > > > > --- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > > +++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
> > > > > > > > > @@ -115,6 +115,9 @@ TERMINAL_CONSOLE_MODE_DATA
> > > > > > > > mTerminalConsoleModeData[] = {
> > > > > > > > > { 80, 25 },
> > > > > > > > >
> > > > > > > > > { 80, 50 },
> > > > > > > > >
> > > > > > > > > { 100, 31 },
> > > > > > > > >
> > > > > > > > > + { 128, 40 },
> > > > > > > > >
> > > > > > > > > + { 160, 42 },
> > > > > > > > >
> > > > > > > > > + { 240, 56 },
> > > > > > > > >
> > > > > > > > > //
> > > > > > > > >
> > > > > > > > > // New modes can be added here.
> > > > > > > > >
> > > > > > > > > //
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > 2.25.1
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > -=-=-=-=-=-=
> > > > > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > > > > View/Reply Online (#93935):
> > > > > > > > https://edk2.groups.io/g/devel/message/93935
> > > > > > > > > Mute This Topic: https://groups.io/mt/93777396/1643496
> > > > > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > > > > [michael.d.kinney@intel.com]
> > > > > > > > > -=-=-=-=-=-=
> > > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94127): https://edk2.groups.io/g/devel/message/94127
Mute This Topic: https://groups.io/mt/93818606/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.