From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
"Gao, Liming" <gaoliming@byosoft.com.cn>,
"Liu, Zhiguang" <zhiguang.liu@intel.com>,
"Ni, Ray" <ray.ni@intel.com>
Subject: Re: [PATCH v2] MdePkg/AcpiXX.h: Update Error Severity type for Generic Error Status Block
Date: Tue, 15 Mar 2022 06:08:27 +0000 [thread overview]
Message-ID: <DM6PR11MB402502EC2BEE422203C08C2BCA109@DM6PR11MB4025.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20220308054316.2501-1-hao.a.wu@intel.com>
Hello,
Any feedback for the patch? Thanks in advance.
Best Regards,
Hao Wu
> -----Original Message-----
> From: Wu, Hao A <hao.a.wu@intel.com>
> Sent: Tuesday, March 8, 2022 1:43 PM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Liu,
> Zhiguang <zhiguang.liu@intel.com>; Ni, Ray <ray.ni@intel.com>
> Subject: [PATCH v2] MdePkg/AcpiXX.h: Update Error Severity type for Generic
> Error Status Block
>
> Starting from ACPI Specification Version 5.1 Errata B, the term
> 'Correctable' is no longer being used as an error severity of the
> reported error in Chapter 18 APEI.
>
> This commit will
> a) For Acpi40.h & Acpi50.h
> Add new macro EFI_ACPI_X_X_ERROR_SEVERITY_RECOVERABLE, since both the
> terms 'Correctable' and 'Recoverable' are used to denote the same error
> severity.
>
> b) Header files starting from Acpi51.h to Acpi64.h
> Add new macro EFI_ACPI_X_X_ERROR_SEVERITY_RECOVERABLE.
> Keeps the origin EFI_ACPI_X_X_ERROR_SEVERITY_CORRECTABLE for
> compatibility
> consideration, but add comments to mark it as deprecated and should no
> longer be used.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> ---
> MdePkg/Include/IndustryStandard/Acpi40.h | 3 ++-
> MdePkg/Include/IndustryStandard/Acpi50.h | 3 ++-
> MdePkg/Include/IndustryStandard/Acpi51.h | 10 ++++++++--
> MdePkg/Include/IndustryStandard/Acpi60.h | 10 ++++++++--
> MdePkg/Include/IndustryStandard/Acpi61.h | 10 ++++++++--
> MdePkg/Include/IndustryStandard/Acpi62.h | 10 ++++++++--
> MdePkg/Include/IndustryStandard/Acpi63.h | 10 ++++++++--
> MdePkg/Include/IndustryStandard/Acpi64.h | 10 ++++++++--
> 8 files changed, 52 insertions(+), 14 deletions(-)
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi40.h
> b/MdePkg/Include/IndustryStandard/Acpi40.h
> index 862113dff9..a2da09346f 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi40.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi40.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 4.0 definitions from the ACPI Specification Revision 4.0a April 5, 2010
>
>
>
> - Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2010 - 2022, Intel Corporation. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
>
>
>
> @@ -690,6 +690,7 @@ typedef struct {
> // Boot Error Severity types
>
> //
>
> #define EFI_ACPI_4_0_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_4_0_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_4_0_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_4_0_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_4_0_ERROR_SEVERITY_NONE 0x03
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi50.h
> b/MdePkg/Include/IndustryStandard/Acpi50.h
> index be8f85f577..76706aa640 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi50.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi50.h
> @@ -2,7 +2,7 @@
> ACPI 5.0 definitions from the ACPI Specification Revision 5.0a November 13,
> 2013.
>
>
>
> Copyright (c) 2014 Hewlett-Packard Development Company, L.P.<BR>
>
> - Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2011 - 2022, Intel Corporation. All rights reserved.<BR>
>
> Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
>
> @@ -1361,6 +1361,7 @@ typedef struct {
> // Boot Error Severity types
>
> //
>
> #define EFI_ACPI_5_0_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_5_0_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_5_0_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_5_0_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_5_0_ERROR_SEVERITY_NONE 0x03
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi51.h
> b/MdePkg/Include/IndustryStandard/Acpi51.h
> index d8ee3ef8f2..01ef544c3a 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi51.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi51.h
> @@ -2,7 +2,7 @@
> ACPI 5.1 definitions from the ACPI Specification Revision 5.1 Errata B January,
> 2016.
>
>
>
> Copyright (c) 2014 Hewlett-Packard Development Company, L.P.<BR>
>
> - Copyright (c) 2014 - 2018, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2014 - 2022, Intel Corporation. All rights reserved.<BR>
>
> (C) Copyright 2015 Hewlett Packard Enterprise Development LP<BR>
>
> Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1376,10 +1376,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_5_1_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_5_1_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_5_1_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_5_1_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_5_1_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_5_1_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi60.h
> b/MdePkg/Include/IndustryStandard/Acpi60.h
> index f4ab016d70..5ac3be6ad7 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi60.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi60.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 6.0 definitions from the ACPI Specification Revision 6.0 Errata A January,
> 2016.
>
>
>
> - Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2015 - 2022, Intel Corporation. All rights reserved.<BR>
>
> (C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP<BR>
>
> Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1560,10 +1560,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_6_0_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_6_0_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_6_0_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_6_0_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_6_0_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_6_0_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi61.h
> b/MdePkg/Include/IndustryStandard/Acpi61.h
> index 5ab31e7520..7af67d832b 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi61.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi61.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 6.1 definitions from the ACPI Specification Revision 6.1 January, 2016.
>
>
>
> - Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2016 - 2022, Intel Corporation. All rights reserved.<BR>
>
> (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
>
> Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1566,10 +1566,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_6_1_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_6_1_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_6_1_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_6_1_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_6_1_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_6_1_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi62.h
> b/MdePkg/Include/IndustryStandard/Acpi62.h
> index 0ede23c716..313db63044 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi62.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi62.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 6.2 definitions from the ACPI Specification Revision 6.2 May, 2017.
>
>
>
> - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2017 - 2022, Intel Corporation. All rights reserved.<BR>
>
> Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
>
> @@ -1738,10 +1738,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_6_2_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_6_2_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_6_2_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_6_2_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_6_2_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_6_2_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi63.h
> b/MdePkg/Include/IndustryStandard/Acpi63.h
> index e4d5825946..b1e9d5db5b 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi63.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi63.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 6.3 definitions from the ACPI Specification Revision 6.3 Jan, 2019.
>
>
>
> - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2017 - 2022, Intel Corporation. All rights reserved.<BR>
>
> Copyright (c) 2019 - 2020, ARM Ltd. All rights reserved.<BR>
>
>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1702,10 +1702,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_6_3_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_6_3_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_6_3_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_6_3_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_6_3_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_6_3_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> diff --git a/MdePkg/Include/IndustryStandard/Acpi64.h
> b/MdePkg/Include/IndustryStandard/Acpi64.h
> index a79b7f2eaa..232697f228 100644
> --- a/MdePkg/Include/IndustryStandard/Acpi64.h
> +++ b/MdePkg/Include/IndustryStandard/Acpi64.h
> @@ -1,7 +1,7 @@
> /** @file
>
> ACPI 6.4 definitions from the ACPI Specification Revision 6.4 Jan, 2021.
>
>
>
> - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
>
> + Copyright (c) 2017 - 2022, Intel Corporation. All rights reserved.<BR>
>
> Copyright (c) 2019 - 2021, ARM Ltd. All rights reserved.<BR>
>
>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -1783,10 +1783,16 @@ typedef struct {
> //
>
> // Boot Error Severity types
>
> //
>
> -#define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTABLE 0x00
>
> +#define EFI_ACPI_6_4_ERROR_SEVERITY_RECOVERABLE 0x00
>
> #define EFI_ACPI_6_4_ERROR_SEVERITY_FATAL 0x01
>
> #define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTED 0x02
>
> #define EFI_ACPI_6_4_ERROR_SEVERITY_NONE 0x03
>
> +//
>
> +// The term 'Correctable' is no longer being used as an error severity of the
>
> +// reported error since ACPI Specification Version 5.1 Errata B.
>
> +// The below macro is considered as deprecated and should no longer be used.
>
> +//
>
> +#define EFI_ACPI_6_4_ERROR_SEVERITY_CORRECTABLE 0x00
>
>
>
> ///
>
> /// Generic Error Data Entry Definition
>
> --
> 2.27.0.windows.1
next prev parent reply other threads:[~2022-03-15 6:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-08 5:43 [PATCH v2] MdePkg/AcpiXX.h: Update Error Severity type for Generic Error Status Block Wu, Hao A
2022-03-15 6:08 ` Wu, Hao A [this message]
2022-03-18 1:15 ` 回复: " gaoliming
2022-03-21 1:28 ` [edk2-devel] " 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=DM6PR11MB402502EC2BEE422203C08C2BCA109@DM6PR11MB4025.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