From: Sohil Mehta <sohil.mehta@intel.com>
To: Adrian Schlegel <me@adrianschlegel.com>,
Yazen Ghannam <yazen.ghannam@amd.com>
Cc: <tony.luck@intel.com>, <bp@alien8.de>, <x86@kernel.org>,
<linux-edac@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<kai.huang@intel.com>,
"Edgecombe, Rick P" <rick.p.edgecombe@intel.com>,
"Dave Hansen" <dave.hansen@linux.intel.com>
Subject: Re: [PATCH v3] x86/mce: Do not treat cache hierarchy errors as memory errors
Date: Mon, 19 Oct 2026 13:45:42 -0700 [thread overview]
Message-ID: <7886e7b1-95fb-4338-a7bb-b6bf1da3a634@intel.com> (raw)
In-Reply-To: <DM914GO4A8WW.Q7YN7ISOQJJI@adrianschlegel.com>
On 10/19/2026 11:34 AM, Adrian Schlegel wrote:
> On Mon Oct 19, 2026 at 5:32 PM CEST, Yazen Ghannam wrote:
>> The SDM also mentions "Extended Memory Errors". I guess these were left
>> out before, but should they be included?
>
> I don't think so, since from what I can tell from the Intel SDM (Memory
> Controller and Extended Memory Errors), extended memory errors use the
> same encoding as normal memory errors, but report errors in memory that
> is used as a cache, similar to what cache hierarchy errors do.
>
Yeah, the mismatch between the code and SDM has been a little confusing.
It might be worth adding a code comment or a changelog note about why
Extended Memory errors are not part of the check.
> I also found this thread, where Tony made the same point:
> https://lore.kernel.org/all/SJ1PR11MB60831E048B544C83E845D907FC8DA@SJ1PR11MB6083.namprd11.prod.outlook.com/
>
Adrian just added me to this thread..
But, the TDX issue that Sashiko reported seems valid to me. It may only
affect early TDX platforms. So, adding some TDX folks in case they care
about this change.
https://sashiko.dev/#/patchset/20261012193305.38931-1-me%40adrianschlegel.com
Looking at the comment on top of tdx_dump_mce_info() and commit
70060463cb2b ("x86/mce: Differentiate real hardware #MCs from TDX
erratum ones") could poison consumption be reported with a cache
hierarchy code?
IIUC (based on the MCACOD_DATA example in the commit), the TDX error
line will no longer be printed after this patch:
CPU 147: Machine Check Exception: f Bank 1: bd80000000100134
Machine check: Data load in unrecoverable area of kernel
Machine Check: TDX private memory error. Possible kernel bug.
next prev parent reply other threads:[~2026-10-19 20:45 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-27 15:19 [PATCH] RAS/CEC: Only count corrected DRAM errors on Intel Adrian Schlegel
2026-09-27 18:14 ` Borislav Petkov
2026-09-29 19:01 ` Luck, Tony
2026-10-03 22:20 ` [PATCH v2] x86/mce: Do not treat cache hierarchy errors as memory errors Adrian Schlegel
2026-10-03 23:17 ` Borislav Petkov
2026-10-05 15:30 ` Luck, Tony
2026-10-12 19:33 ` [PATCH v3] " Adrian Schlegel
2026-10-13 0:01 ` Luck, Tony
2026-10-13 1:04 ` Qiuxu (Leo) Zhuo
2026-10-19 15:32 ` Yazen Ghannam
2026-10-19 18:34 ` Adrian Schlegel
2026-10-19 18:42 ` Yazen Ghannam
2026-10-19 20:45 ` Sohil Mehta [this message]
2026-10-19 21:06 ` Luck, Tony
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=7886e7b1-95fb-4338-a7bb-b6bf1da3a634@intel.com \
--to=sohil.mehta@intel.com \
--cc=bp@alien8.de \
--cc=dave.hansen@linux.intel.com \
--cc=kai.huang@intel.com \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=me@adrianschlegel.com \
--cc=rick.p.edgecombe@intel.com \
--cc=tony.luck@intel.com \
--cc=x86@kernel.org \
--cc=yazen.ghannam@amd.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
all inboxes | Powered by JetHome®