[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <40d3399a-b260-4ffa-a503-40c61468a189@amd.com>
Date: Wed, 13 Sep 2023 10:36:50 -0400
From: Yazen Ghannam <yazen.ghannam@....com>
To: Borislav Petkov <bp@...en8.de>, Elliott Mitchell <ehem+xen@....com>
Cc: yazen.ghannam@....com, smita.koralahallichannabasappa@....com,
linux-edac@...r.kernel.org, linux-kernel@...r.kernel.org,
x86@...nel.org, xen-devel@...ts.xenproject.org, rric@...nel.org,
james.morse@....com, tony.luck@...el.com
Subject: Re: [PATCH] Revert "EDAC/mce_amd: Do not load edac_mce_amd module on
guests"
On 9/7/23 11:59 PM, Borislav Petkov wrote:
> On Thu, Sep 07, 2023 at 08:08:00PM -0700, Elliott Mitchell wrote:
>> This reverts commit 767f4b620edadac579c9b8b6660761d4285fa6f9.
>>
>> There are at least 3 valid reasons why a VM may see MCE events/registers.
>
> Hmm, so they all read like a bunch of handwaving to me, with those
> probable hypothetical "may" formulations.
>
> How about we cut to the chase and you explain what exactly is the
> concrete issue you're encountering and trying to solve?
Also, please note that the EDAC modules don't handle MCE events
directly. They act on information passed from the MCE subsystem.
Furthermore, there are other EDAC modules that have the same !hypervisor
check, so why change only this one?
Thanks,
Yazen
Powered by blists - more mailing lists