* [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles
@ 2017-08-14 22:29 Michael D Kinney
2017-08-15 1:00 ` Ni, Ruiyu
2017-08-15 6:26 ` Wang, Sunny (HPS SW)
0 siblings, 2 replies; 3+ messages in thread
From: Michael D Kinney @ 2017-08-14 22:29 UTC (permalink / raw)
To: edk2-devel; +Cc: Ruiyu Ni, Hao Wu, Michael Kinney
Add STD_ERROR flag to all output consoles that the Nt32
platform supports so all messages sent to the standard
error console device(s) are visible by default.
The Boot Maintenance Manager can be used to manually disable
standard error output to specific console devices.
UEFI Applications and UEFI Drivers are recommended to be
built with DEBUG() and ASSERT() messages sent to the standard
error device using MdePkg/Library/UefiDebugLibStdErr. Prior
to this change, a user would have to use the Boot Maintenance
Manager to configure a standard error console device to make
these messages visible.
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Hao Wu <hao.a.wu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
index e92f377bd1..e4b400b4a3 100644
--- a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
+++ b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
@@ -2,7 +2,7 @@
Defined the platform specific device path which will be filled to
ConIn/ConOut variables.
-Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License
which accompanies this distribution. The full text of the license may be found at
@@ -137,19 +137,19 @@ NT_ISA_SERIAL_DEVICE_PATH gNtSerialDevicePath1 = {
PLATFORM_CONSOLE_CONNECT_ENTRY gPlatformConsole[] = {
{
(EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath0,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath1,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath0,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath1,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
NULL,
--
2.13.1.windows.2
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles
2017-08-14 22:29 [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles Michael D Kinney
@ 2017-08-15 1:00 ` Ni, Ruiyu
2017-08-15 6:26 ` Wang, Sunny (HPS SW)
1 sibling, 0 replies; 3+ messages in thread
From: Ni, Ruiyu @ 2017-08-15 1:00 UTC (permalink / raw)
To: Kinney, Michael D, edk2-devel@lists.01.org; +Cc: Wu, Hao A
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
Thanks/Ray
> -----Original Message-----
> From: Kinney, Michael D
> Sent: Tuesday, August 15, 2017 6:30 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu <ruiyu.ni@intel.com>; Wu, Hao A <hao.a.wu@intel.com>;
> Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all
> consoles
>
> Add STD_ERROR flag to all output consoles that the Nt32 platform supports
> so all messages sent to the standard error console device(s) are visible by
> default.
>
> The Boot Maintenance Manager can be used to manually disable standard
> error output to specific console devices.
>
> UEFI Applications and UEFI Drivers are recommended to be built with
> DEBUG() and ASSERT() messages sent to the standard error device using
> MdePkg/Library/UefiDebugLibStdErr. Prior to this change, a user would have
> to use the Boot Maintenance Manager to configure a standard error console
> device to make these messages visible.
>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
> ---
> Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
> b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
> index e92f377bd1..e4b400b4a3 100644
> --- a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
> +++ b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
> @@ -2,7 +2,7 @@
> Defined the platform specific device path which will be filled to
> ConIn/ConOut variables.
>
> -Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.<BR>
> This program and the accompanying materials
> are licensed and made available under the terms and conditions of the BSD
> License
> which accompanies this distribution. The full text of the license may be
> found at
> @@ -137,19 +137,19 @@ NT_ISA_SERIAL_DEVICE_PATH
> gNtSerialDevicePath1 = {
> PLATFORM_CONSOLE_CONNECT_ENTRY gPlatformConsole[] = {
> {
> (EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath0,
> - (CONSOLE_OUT | CONSOLE_IN)
> + (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
> },
> {
> (EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath1,
> - (CONSOLE_OUT | CONSOLE_IN)
> + (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
> },
> {
> (EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath0,
> - (CONSOLE_OUT | CONSOLE_IN)
> + (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
> },
> {
> (EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath1,
> - (CONSOLE_OUT | CONSOLE_IN)
> + (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
> },
> {
> NULL,
> --
> 2.13.1.windows.2
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles
2017-08-14 22:29 [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles Michael D Kinney
2017-08-15 1:00 ` Ni, Ruiyu
@ 2017-08-15 6:26 ` Wang, Sunny (HPS SW)
1 sibling, 0 replies; 3+ messages in thread
From: Wang, Sunny (HPS SW) @ 2017-08-15 6:26 UTC (permalink / raw)
To: Michael D Kinney, edk2-devel@lists.01.org
Cc: Ruiyu Ni, Hao Wu, Wang, Sunny (HPS SW)
Reviewed and tested it. Looks good to me. Thanks for addressing my offline comment, Michael. :)
Reviewed-by: Sunny Wang <sunnywang@hpe.com>
Regards,
Sunny Wang
-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Michael D Kinney
Sent: Tuesday, August 15, 2017 6:30 AM
To: edk2-devel@lists.01.org
Cc: Ruiyu Ni <ruiyu.ni@intel.com>; Hao Wu <hao.a.wu@intel.com>; Michael Kinney <michael.d.kinney@intel.com>
Subject: [edk2] [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles
Add STD_ERROR flag to all output consoles that the Nt32 platform supports so all messages sent to the standard error console device(s) are visible by default.
The Boot Maintenance Manager can be used to manually disable standard error output to specific console devices.
UEFI Applications and UEFI Drivers are recommended to be built with DEBUG() and ASSERT() messages sent to the standard error device using MdePkg/Library/UefiDebugLibStdErr. Prior to this change, a user would have to use the Boot Maintenance Manager to configure a standard error console device to make these messages visible.
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Hao Wu <hao.a.wu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
index e92f377bd1..e4b400b4a3 100644
--- a/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
+++ b/Nt32Pkg/Library/PlatformBootManagerLib/PlatformData.c
@@ -2,7 +2,7 @@
Defined the platform specific device path which will be filled to
ConIn/ConOut variables.
-Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2015 - 2017, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License
which accompanies this distribution. The full text of the license may be found at
@@ -137,19 +137,19 @@ NT_ISA_SERIAL_DEVICE_PATH gNtSerialDevicePath1 = {
PLATFORM_CONSOLE_CONNECT_ENTRY gPlatformConsole[] = {
{
(EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath0,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gNtSerialDevicePath1,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath0,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
(EFI_DEVICE_PATH_PROTOCOL *) &gGopDevicePath1,
- (CONSOLE_OUT | CONSOLE_IN)
+ (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)
},
{
NULL,
--
2.13.1.windows.2
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-15 6:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-14 22:29 [Patch] Nt32/PlatformBootManagerLib: Enable STD_ERROR on all consoles Michael D Kinney
2017-08-15 1:00 ` Ni, Ruiyu
2017-08-15 6:26 ` Wang, Sunny (HPS SW)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox