From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f44.google.com (mail-wr1-f44.google.com [209.85.221.44]) by mx.groups.io with SMTP id smtpd.web10.5957.1609856759887510750 for ; Tue, 05 Jan 2021 06:26:00 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@akeo-ie.20150623.gappssmtp.com header.s=20150623 header.b=IGY+LjeJ; spf=pass (domain: akeo.ie, ip: 209.85.221.44, mailfrom: pete@akeo.ie) Received: by mail-wr1-f44.google.com with SMTP id w5so36321173wrm.11 for ; Tue, 05 Jan 2021 06:25:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=akeo-ie.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=MV0pZnjsTnOWgmNghAusHCKUEbCPPncxtFQOmw8atGM=; b=IGY+LjeJhHWBW2rY8wsgqXMam7f07mtpcV6Z9k5UdyduKQ7/1BqQVtOya3JahLr8zm lq6YdFeV7khrpOK058NKOZmQefacAoMvpa3VEBXAFdzUR6Vp6zIqhtNLuJqitE1wM/fe k+O5bixF/ECPpEkTwx7Jxye343SZ5R05MXgwdBiuREGFmEGEgvTIajYO/GLKKOYwBoqh /HBRWSJoJ0DiESlRnw4XBUaOS4KNC9V1+OAgzVOJu1cYNxaknXyP+T0kI5XTpgv6VS5c Eg5qp1mtX7R8TbFNoEmykP+kOtdDjIZzH43kKJZ15t6vSPOns+eJvlhOAWOM1y/h68Gs nk/g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=MV0pZnjsTnOWgmNghAusHCKUEbCPPncxtFQOmw8atGM=; b=NhC5aP6eKkPYAO9ToYanOL6+EqIq+19NaPgYfjM4IIl9FuHxgH/+qPAExQdz1IBQIh 4w0BRoR7B6XxKAGjr/VQAoZPqVAJnVyJOXcC52Y0Sf2ODRIcJNupc83E3k5Dk4FcBcLL TUPvNpCp+bYORgvHPwx9YSYpQq3zhvIrdY8Rb/Dd9bxcF2dgQQIDh5Fh7B1258cYfMKe QmUKerjqsId6jqYOZ1vH1SqkHOmJxZ4GLc/y6ABte5Qv7Pyitadx7jWPcO8X8Wqqx/rf 3rOk8fMpI6QSqisqB9+2Ld7XubkCLRGp11/VQZ0Pk4P1gHINv4yam9Sem0pn5YrCOKpF ytBQ== X-Gm-Message-State: AOAM532+wRRH/KREoGLD0oF0JRlzR/c0Z/olGC/qdy0YQTE65G9uQQDt UoBkrEH5iFJXhfklSzXgMq752w== X-Google-Smtp-Source: ABdhPJyeh5PmFbJLPe5fmNq4meiTWEUBDQWz+TSeWXFOgIzcXIxmLAKc7dYWVuPiT8h8K9gYqXuoRQ== X-Received: by 2002:adf:d84e:: with SMTP id k14mr84942905wrl.104.1609856758396; Tue, 05 Jan 2021 06:25:58 -0800 (PST) Return-Path: Received: from [10.0.0.122] ([84.203.63.11]) by smtp.googlemail.com with ESMTPSA id w189sm4453364wmg.31.2021.01.05.06.25.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 05 Jan 2021 06:25:57 -0800 (PST) Subject: Re: [edk2-devel] [edk2-platform][PATCH v2 3/3] MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY To: Samer El-Haj-Mahmoud , "Ni, Ray" , "Gao, Zhichao" , "devel@edk2.groups.io" Cc: "Wang, Jian J" , "Wu, Hao A" , Ard Biesheuvel , Laszlo Ersek References: <20201220135355.12814-1-Samer.El-Haj-Mahmoud@arm.com> <20201220135355.12814-4-Samer.El-Haj-Mahmoud@arm.com> <87f526a4-1328-7af3-e540-e3cf6f36cab0@akeo.ie> From: "Pete Batard" Message-ID: Date: Tue, 5 Jan 2021 14:25:56 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Hi All, Just going to second Samer that some of us are eagerly waiting on the integration of this series, as, along with other changes we have lined up, it will help make the Raspberry Pi UEFI platform look more professional. We would therefore appreciate if this could be integrated soon. Thanks, /Pete On 2020.12.30 18:10, Samer El-Haj-Mahmoud wrote: > Thanks Ray. Could you or one of the maintainers please help merge the series? > >> -----Original Message----- >> From: Ni, Ray >> Sent: Thursday, December 24, 2020 9:50 PM >> To: Gao, Zhichao ; Samer El-Haj-Mahmoud >> ; Pete Batard ; >> devel@edk2.groups.io >> Cc: Wang, Jian J ; Wu, Hao A >> ; Ard Biesheuvel ; Laszlo >> Ersek >> Subject: RE: [edk2-devel] [edk2-platform][PATCH v2 3/3] >> MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY >> >> I don't have other better ideas to resolve the issue. So, I don't have >> concerns. >> >>> -----Original Message----- >>> From: Gao, Zhichao >>> Sent: Friday, December 25, 2020 9:22 AM >>> To: Samer El-Haj-Mahmoud ; Pete >> Batard >>> ; devel@edk2.groups.io >>> Cc: Wang, Jian J ; Wu, Hao A >>> ; Ni, Ray ; Ard Biesheuvel >>> ; Laszlo Ersek >>> Subject: RE: [edk2-devel] [edk2-platform][PATCH v2 3/3] >>> MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY >>> >>> Hi Ray, >>> >>> Do you have other concerns about this change? >>> >>> Thanks, >>> Zhichao >>> >>>> -----Original Message----- >>>> From: Samer El-Haj-Mahmoud >>>> Sent: Friday, December 25, 2020 12:34 AM >>>> To: Gao, Zhichao ; Pete Batard >>>> ; devel@edk2.groups.io >>>> Cc: Wang, Jian J ; Wu, Hao A >>>> ; Ni, Ray ; Ard Biesheuvel >>>> ; Samer El- Haj-Mahmoud >>>> ; Laszlo Ersek >> >>>> 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 >>>>> Sent: Wednesday, December 23, 2020 8:12 PM >>>>> To: Pete Batard ; devel@edk2.groups.io; Samer >>>>> El-Haj- Mahmoud >>>>> Cc: Wang, Jian J ; Wu, Hao A >>>>> ; Ni, Ray ; Ard Biesheuvel >>>>> >>>>> Subject: RE: [edk2-devel] [edk2-platform][PATCH v2 3/3] >>>>> MdeModulePkg/ConSplitter: Change StdErr color to EFI_LIGHTGRAY >>>>> >>>>> Reviewed-by: Zhichao Gao >>>>> >>>>> Thanks, >>>>> Zhichao >>>>> >>>>>> -----Original Message----- >>>>>> From: Pete Batard >>>>>> Sent: Thursday, December 24, 2020 12:35 AM >>>>>> To: devel@edk2.groups.io; samer.el-haj-mahmoud@arm.com >>>>>> Cc: Wang, Jian J ; Wu, Hao A >>>>>> ; Gao, Zhichao ; Ni, >>>>>> Ray ; Ard Biesheuvel >>>>>> 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 >>>>>>> Cc: Hao A Wu >>>>>>> Cc: Zhichao Gao >>>>>>> Cc: Ray Ni >>>>>>> Cc: Ard Biesheuvel >>>>>>> Signed-off-by: Samer El-Haj-Mahmoud >>>> 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/ConSplitte >>>>>>> +++ r.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 >>>>>> Tested-by: Pete Batard (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. > 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. >