Linux ACPI
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <M.Chehab@huawei.com>
To: Jonathan Cameron <jic23@kernel.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: "uswg@uefi.org" <uswg@uefi.org>,
	"taoxiaofei@huawei.com" <taoxiaofei@huawei.com>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
	Dong Wei <Dong.Wei@arm.com>,
	Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>,
	Jose Marinho <jose.marinho@arm.com>,
	wanghuiqiang <wanghuiqiang@huawei.com>
Subject: RE: [RESEND] troubles with ARM processor type changes on UEFI 2.11
Date: Mon, 22 Jun 2026 09:36:53 +0000	[thread overview]
Message-ID: <d685e5a9d5a2470c8a30cfd96e2f81d5@huawei.com> (raw)
In-Reply-To: <20260622094233.1088c8bf@jic23-huawei>

On Mon, 22 Jun 2026 09:42:33 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> On Mon, 22 Jun 2026 10:25:18 +0200
> Mauro Carvalho Chehab <mchehab@kernel.org> wrote:
> 
> > (as I got no answer, I'm resending it, c/c some ARM people as well)
> > 
> > Hi,
> > 
> > I maintain ACPI HEST support on QEMU and I'm one of the Linux kernel reviewers
> > for the RAS subsystem. I'm also the author and maintainer of Linux rasdaemon,
> > responsible to handle RAS reports (including from GHES) on Linux.
> > 
> > There is an incompatible change at UEFI 2.11 spec at Section N.2.4.4.1,
> > which, according with UEFI 2.11 changelog, it is related to this issue:
> > 
> > 	2462 - Fix the Type mnemonic description in the ARM Processor Error Information Structure
> > 
> > This is the second time the specs changed the meaning of the type field
> > inside the ARM processor error info:
> > 
> > - Before UEFI 2.9A, the spec has a list of values for error type. 
> >   From the specs, type, at offset 4 was described as:
> >     
> >             - Cache error
> >             - TLB Error
> >             - Bus Error
> >             - Micro-architectural Error
> >             All other values are reserved
> >     
> >     Yet, there was no information about how this would be encoded.
> >     
> > - UEFI 2.9A defined the actual expected values as:
> >     
> >             - Bit 1 - Cache Error
> >             - Bit 2 - TLB Error
> >             - Bit 3 - Bus Error
> >             - Bit 4 - Micro-architectural Error
> >             All other values are reserved
> > 
> >     However, even being an incompatible change, the spec didn't change
> >     version field at Section N.2.4.4.1.
> > 
> >    Linux, QEMU, rasdaemon and likely several BIOS implementations
> >    were updated accordingly.
> > 
> > - UEFI 2.11 changed the definition to:
> > 
> >             - Bit 0 - Cache Error
> >             - Bit 1 - TLB Error
> >             - Bit 2 - Bus Error
> >             - Bit 3 - Micro-architectural Error
> >             All other values are reserved
> > 
> >   Again, another incompatible change and without changing the version
> >   field at Section N.2.4.4.1.
> > 
> > With the current mess, it is impossible for the OSPM (and RAS apps) to
> > properly map ARM processor error types, as for instance, type=2 can
> > mean:
> > 
> > 	- Bus error (up to UEFI 2.9)
> > 	- Cache error (UEFI 2.9A and UEFI 2.10)
> > 	- TLB error (UEFI 2.11)
> > 
> > Please issue an errata incrementing version number, to allow OSPM
> > QEMU and rasdaemon to properly interpret this field.
> >   
> FWIW I fully agree that an errata is needed. (Though, even though ambiguous,
> changing meaning should probably have been done by deprecating that field
> and adding a new one... too late now!) 
> 
> It seems a bit odd to do it this way but Mauro, you should be able
> to file a code first proposal at:
> https://github.com/tianocore/edk2/issues?q=is%3Aissue%20state%3Aopen%20label%3Atype%3Acode-first

Thank you to point it to me. I opted to open this as a bug:
	https://github.com/tianocore/edk2/issues/12708

As the root cause for such regression was another bug:
	https://github.com/tianocore/edk2/issues/10524
	

> 
> Make sure it has a code first label.  I think you also need
> someone to take it through the committee though (I'm sadly not
> in a position to do so at the moment).
> 
> Samer or Jose (+CC), can you deal with this one?
> 
> Thanks,
> 
> Jonathan
> 
> > Thanks,
> > Maur  


-- 
Thanks,
Mauro

      reply	other threads:[~2026-06-22  9:36 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-22  8:25 [RESEND] troubles with ARM processor type changes on UEFI 2.11 Mauro Carvalho Chehab
2026-06-22  8:42 ` Jonathan Cameron
2026-06-22  9:36   ` Mauro Carvalho Chehab [this message]

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-all 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=d685e5a9d5a2470c8a30cfd96e2f81d5@huawei.com \
    --to=m.chehab@huawei.com \
    --cc=Dong.Wei@arm.com \
    --cc=Samer.El-Haj-Mahmoud@arm.com \
    --cc=jic23@kernel.org \
    --cc=jose.marinho@arm.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=qemu-arm@nongnu.org \
    --cc=taoxiaofei@huawei.com \
    --cc=uswg@uefi.org \
    --cc=wanghuiqiang@huawei.com \
    /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