public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Zhichao" <zhichao.gao@intel.com>
To: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>,
	Pete Batard <pete@akeo.ie>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Wang, Jian J" <jian.j.wang@intel.com>,
	"Wu, Hao A" <hao.a.wu@intel.com>, "Ni, Ray" <ray.ni@intel.com>,
	Ard Biesheuvel <Ard.Biesheuvel@arm.com>,
	"Laszlo Ersek" <lersek@redhat.com>
Subject: Re: [edk2-devel] [edk2-platform][PATCH v2 3/3] MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY
Date: Fri, 25 Dec 2020 01:21:43 +0000	[thread overview]
Message-ID: <MWHPR11MB164732189291ACA2EC859DC8F6DC0@MWHPR11MB1647.namprd11.prod.outlook.com> (raw)
In-Reply-To: <DB7PR08MB32603DFD75912373F439F62590DD0@DB7PR08MB3260.eurprd08.prod.outlook.com>

Hi Ray,

Do you have other concerns about this change?

Thanks,
Zhichao

> -----Original Message-----
> From: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
> Sent: Friday, December 25, 2020 12:34 AM
> To: Gao, Zhichao <zhichao.gao@intel.com>; Pete Batard <pete@akeo.ie>;
> devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Ni,
> Ray <ray.ni@intel.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>; Samer El-
> Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Laszlo Ersek
> <lersek@redhat.com>
> Subject: RE: [edk2-devel] [edk2-platform][PATCH v2 3/3]
> MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY
> 
> Thanks Zhichao and Pete
> 
> Now that this series ( https://edk2.groups.io/g/devel/message/69264 ) is
> reviewed, can we get it pushed please?
> 
> Many thanks,
> --Samer
> 
> > -----Original Message-----
> > From: Gao, Zhichao <zhichao.gao@intel.com>
> > Sent: Wednesday, December 23, 2020 8:12 PM
> > To: Pete Batard <pete@akeo.ie>; devel@edk2.groups.io; Samer El-Haj-
> > Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> > <hao.a.wu@intel.com>; Ni, Ray <ray.ni@intel.com>; Ard Biesheuvel
> > <Ard.Biesheuvel@arm.com>
> > Subject: RE: [edk2-devel] [edk2-platform][PATCH v2 3/3]
> > MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY
> >
> > Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>
> >
> > Thanks,
> > Zhichao
> >
> > > -----Original Message-----
> > > From: Pete Batard <pete@akeo.ie>
> > > Sent: Thursday, December 24, 2020 12:35 AM
> > > To: devel@edk2.groups.io; samer.el-haj-mahmoud@arm.com
> > > Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A
> > > <hao.a.wu@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Ni, Ray
> > > <ray.ni@intel.com>; Ard Biesheuvel <Ard.Biesheuvel@arm.com>
> > > Subject: Re: [edk2-devel] [edk2-platform][PATCH v2 3/3]
> > > MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY
> > >
> > > On 2020.12.20 13:53, Samer El-Haj-Mahmoud wrote:
> > > > ConSplitter is using EFI_LIGHTGRAY foreground color for ConOut and
> > > > EFI_MAGENTA for StdErr consoles. This is impacting the DebugLib
> > > > output on that same serial console (e.g. DebugLibSerialPort) after
> > > > gEfiStandardErrorDeviceGuid is installed on that port. The impact
> > > > also extends to Linux serial console output in OVMF because it
> > > > inherits the color setting from the firmware.
> > > >
> > > > This is inconsistent and annoying, with MAGENTA being barely
> > > > legible on a black background.
> > > >
> > > > Let's change StdErr default color to LIGHTGRAY for consistency and
> > > > readability. This results in the same color being used for all
> > > > consoles sharing the same serial port (ConOut, StdErr, DebugLib,
> > > > OS
> > console).
> > > > Platforms wishing to distinguish the colors of consoles can do so
> > > > in their own Platform BDS initialization.
> > > >
> > > > Cc: Jian J Wang <jian.j.wang@intel.com>
> > > > Cc: Hao A Wu <hao.a.wu@intel.com>
> > > > Cc: Zhichao Gao <zhichao.gao@intel.com>
> > > > Cc: Ray Ni <ray.ni@intel.com>
> > > > Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>
> > > > Signed-off-by: Samer El-Haj-Mahmoud <Samer.El-Haj-
> > Mahmoud@arm.com>
> > > > ---
> > > >   MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c | 2 +-
> > > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git
> > > > a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
> > > > b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
> > > > index b090de288517..e8cd4ce120a0 100644
> > > > --- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
> > > > +++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
> > > > @@ -1476,7 +1476,7 @@ ConSplitterStdErrDriverBindingStart (
> > > >     // their MaxMode and QueryData should be the intersection of both.
> > > >
> > > >     //
> > > >
> > > >     Status = ConSplitterTextOutAddDevice (&mStdErr, TextOut, NULL,
> > > > NULL);
> > > >
> > > > -  ConSplitterTextOutSetAttribute (&mStdErr.TextOut, EFI_TEXT_ATTR
> > > > (EFI_MAGENTA, EFI_BLACK));
> > > >
> > > > +  ConSplitterTextOutSetAttribute (&mStdErr.TextOut, EFI_TEXT_ATTR
> > > > + (EFI_LIGHTGRAY, EFI_BLACK));
> > > >
> > > >
> > > >
> > > >     return Status;
> > > >
> > > >   }
> > > >
> > >
> > > Reviewed-by: Pete Batard <pete@akeo.ie>
> > > Tested-by: Pete Batard <pete@akeo.ie> (On an RPi 4 platform where
> > > this was another annoyance)
> 
> IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended recipient,
> please notify the sender immediately and do not disclose the contents to any
> other person, use it for any purpose, or store or copy the information in any
> medium. Thank you.

  reply	other threads:[~2020-12-25  1:21 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-20 13:53 [edk2-platform][PATCH v2 0/3] MdeModulePkg/GraphicsConsole : Console cleanup Samer El-Haj-Mahmoud
2020-12-20 13:53 ` [edk2-platform][PATCH v2 1/3] MdeModulePkg/GraphicsConsoleDxe: Change default CursorVisible to FALSE Samer El-Haj-Mahmoud
2020-12-23 16:34   ` Pete Batard
2020-12-20 13:53 ` [edk2-platform][PATCH v2 2/3] MdeModulePkg/Graphics: Fix spelling mistakes Samer El-Haj-Mahmoud
2020-12-20 13:53 ` [edk2-platform][PATCH v2 3/3] MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY Samer El-Haj-Mahmoud
2020-12-23 16:35   ` [edk2-devel] " Pete Batard
2020-12-24  1:11     ` Gao, Zhichao
2020-12-24 16:33       ` Samer El-Haj-Mahmoud
2020-12-25  1:21         ` Gao, Zhichao [this message]
2020-12-25  2:50           ` Ni, Ray
2020-12-30 18:10             ` Samer El-Haj-Mahmoud
2021-01-05 14:25               ` Pete Batard
2021-01-06  5:28 ` [edk2-devel] [edk2-platform][PATCH v2 0/3] MdeModulePkg/GraphicsConsole : Console cleanup Wu, Hao A

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=MWHPR11MB164732189291ACA2EC859DC8F6DC0@MWHPR11MB1647.namprd11.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox