Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
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 > Sent: Thursday, September 22, 2022 9:50 AM > To: Kinney, Michael D > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, Jian > J ; Ni, Ray > > 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 > 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 > > > Sent: Thursday, September 22, 2022 1:23 AM > > > To: Kinney, Michael D > > > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, > > > Jian J ; Ni, Ray > > > > > > 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 > > > 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 > > > >
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 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 > > Sent: Thursday, September 22, 2022 1:23 AM > > To: Kinney, Michael D > > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, > > Jian J ; Ni, Ray > > > > 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 > > 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 On Behalf Of > > > > Konstantin Aladyshev > > > > Sent: Wednesday, September 21, 2022 1:24 AM > > > > To: Kinney, Michael D > > > > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, > > > > Jian J ; Ni, Ray > > > > > > > > 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: > > > > ``` > > > > S
Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
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 > Sent: Thursday, September 22, 2022 1:23 AM > To: Kinney, Michael D > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, Jian > J ; Ni, Ray > > 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 > 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 On Behalf Of Konstantin > > > Aladyshev > > > Sent: Wednesday, September 21, 2022 1:24 AM > > > To: Kinney, Michael D > > > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, > > > Jian J ; Ni, Ray > > > > > > 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. > > > Col80 Row25 * > > > Col80 Row50 > > > Col 100 Row31 > > > ``` > > > But with this commit the high resolution modes are present in the output: > > > ``` > > > Shell> mode > > > Available modes for console output device. > > > Col80 Row25 * > > > Col80 Row50 > > > Col 100 Row31 > > > Col 128 Row40 > > > Col 160 Row42 > > > Col 240 Row56 > > > ``` > > > And in my case setting the appropriate "mode 240 56" at runtime works > > > just fine: > > > ``` > > > Shell> mode 240 56 > > > ``` > > > After that t
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 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 On Behalf Of Konstantin > > Aladyshev > > Sent: Wednesday, September 21, 2022 1:24 AM > > To: Kinney, Michael D > > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, > > Jian J ; Ni, Ray > > > > 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. > > Col80 Row25 * > > Col80 Row50 > > Col 100 Row31 > > ``` > > But with this commit the high resolution modes are present in the output: > > ``` > > Shell> mode > > Available modes for console output device. > > Col80 Row25 * > > Col80 Row50 > > Col 100 Row31 > > Col 128 Row40 > > Col 160 Row42 > > Col 240 Row56 > > ``` > > 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 > > wrote: > > > > > > Tera Term works. > > > > > > Mike > > > > > > > -Original Message- > > > > From: devel@edk2.groups.io On Behalf Of > > > > gaoliming via groups.io > > > > Sent: Tuesday, September 20, 2022 6:26 PM > > > > To: Kinney, Michael D ; > > > > devel@edk2.groups.io; aladyshe...@gmail.com > > > > Cc: Wang, Jian J ; Ni, Ray > > > > 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 > >
Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
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 On Behalf Of Konstantin > Aladyshev > Sent: Wednesday, September 21, 2022 1:24 AM > To: Kinney, Michael D > Cc: devel@edk2.groups.io; Gao, Liming ; Wang, Jian > J ; Ni, Ray > > 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. > Col80 Row25 * > Col80 Row50 > Col 100 Row31 > ``` > But with this commit the high resolution modes are present in the output: > ``` > Shell> mode > Available modes for console output device. > Col80 Row25 * > Col80 Row50 > Col 100 Row31 > Col 128 Row40 > Col 160 Row42 > Col 240 Row56 > ``` > 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 > wrote: > > > > Tera Term works. > > > > Mike > > > > > -Original Message- > > > From: devel@edk2.groups.io On Behalf Of gaoliming > > > via groups.io > > > Sent: Tuesday, September 20, 2022 6:26 PM > > > To: Kinney, Michael D ; devel@edk2.groups.io; > > > aladyshe...@gmail.com > > > Cc: Wang, Jian J ; Ni, Ray > > > 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 > > > > 发送时间: 2022年9月20日 5:02 > > > > 收件人: devel@edk2.groups.io; aladyshe...@gmail.com; Kinney, Michael D > > > > > > > > 抄送: Wang, Jian J ; Gao, Liming > > > > ; Ni, Ray > > > > 主题: 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 > > > > > > > > > > > > > -Original Message- > > > > > From: 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 ; Gao, Liming > > > > ; Ni, Ray ; Konstantin > > > > > Aladyshev > > > > > 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 > > > > > --- > > > > > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 3 +++ > > > > > 1 file changed, 3 insertions(+) > > > > > > > > > > diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c > > > > b/MdeModulePkg/Un
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. Col80 Row25 * Col80 Row50 Col 100 Row31 ``` But with this commit the high resolution modes are present in the output: ``` Shell> mode Available modes for console output device. Col80 Row25 * Col80 Row50 Col 100 Row31 Col 128 Row40 Col 160 Row42 Col 240 Row56 ``` 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 wrote: > > Tera Term works. > > Mike > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of gaoliming > > via groups.io > > Sent: Tuesday, September 20, 2022 6:26 PM > > To: Kinney, Michael D ; devel@edk2.groups.io; > > aladyshe...@gmail.com > > Cc: Wang, Jian J ; Ni, Ray > > 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 > > > 发送时间: 2022年9月20日 5:02 > > > 收件人: devel@edk2.groups.io; aladyshe...@gmail.com; Kinney, Michael D > > > > > > 抄送: Wang, Jian J ; Gao, Liming > > > ; Ni, Ray > > > 主题: 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 > > > > > > > > > > -----Original Message- > > > > From: 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 ; Gao, Liming > > > ; Ni, Ray ; Konstantin > > > > Aladyshev > > > > 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 > > > > --- > > > > 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+ow...@edk2.groups.io > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub > > > [michael.d.kin...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] MdeModulePkg/TerminalDxe: add modes
Tera Term works. Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of gaoliming via > groups.io > Sent: Tuesday, September 20, 2022 6:26 PM > To: Kinney, Michael D ; devel@edk2.groups.io; > aladyshe...@gmail.com > Cc: Wang, Jian J ; Ni, Ray > 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 > > 发送时间: 2022年9月20日 5:02 > > 收件人: devel@edk2.groups.io; aladyshe...@gmail.com; Kinney, Michael D > > > > 抄送: Wang, Jian J ; Gao, Liming > > ; Ni, Ray > > 主题: 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 > > > > > > > -Original Message- > > > From: 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 ; Gao, Liming > > ; Ni, Ray ; Konstantin > > > Aladyshev > > > 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 > > > --- > > > 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+ow...@edk2.groups.io > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub > > [michael.d.kin...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
回复: [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 > 发送时间: 2022年9月20日 5:02 > 收件人: devel@edk2.groups.io; aladyshe...@gmail.com; Kinney, Michael D > > 抄送: Wang, Jian J ; Gao, Liming > ; Ni, Ray > 主题: 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 > > > > -Original Message- > > From: 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 ; Gao, Liming > ; Ni, Ray ; Konstantin > > Aladyshev > > 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 > > --- > > 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+ow...@edk2.groups.io > > Unsubscribe: https://edk2.groups.io/g/devel/unsub > [michael.d.kin...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.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 > -Original Message- > From: 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 ; Gao, Liming > ; Ni, Ray ; Konstantin > Aladyshev > 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 > --- > 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+ow...@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub [michael.d.kin...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[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 --- 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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-