All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.