From: "gaoliming" <gaoliming@byosoft.com.cn>
To: "'Wu, Hao A'" <hao.a.wu@intel.com>, <devel@edk2.groups.io>,
"'Kinney, Michael D'" <michael.d.kinney@intel.com>,
"'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
Date: Fri, 18 Mar 2022 09:15:48 +0800 [thread overview]
Message-ID: <018201d83a65$b3ce4fa0$1b6aeee0$@byosoft.com.cn> (raw)
In-Reply-To: <DM6PR11MB402502EC2BEE422203C08C2BCA109@DM6PR11MB4025.namprd11.prod.outlook.com>
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
> -----邮件原件-----
> 发件人: Wu, Hao A <hao.a.wu@intel.com>
> 发送时间: 2022年3月15日 14:08
> 收件人: 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>
> 主题: RE: [PATCH v2] MdePkg/AcpiXX.h: Update Error Severity type for
> Generic Error Status Block
>
> 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-18 1:15 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
2022-03-18 1:15 ` gaoliming [this message]
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='018201d83a65$b3ce4fa0$1b6aeee0$@byosoft.com.cn' \
--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