Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     Yazen Ghannam <yazen.ghannam@amd.com>,
        Borislav Petkov <bp@alien8.de>
CC:     "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "x86@kernel.org" <x86@kernel.org>,
        "Smita.KoralahalliChannabasappa@amd.com" 
        <Smita.KoralahalliChannabasappa@amd.com>
Subject: RE: [PATCH 1/3] x86/MCE, EDAC/mce_amd: Add support for new
 MCA_SYND{1,2} registers
Thread-Topic: [PATCH 1/3] x86/MCE, EDAC/mce_amd: Add support for new
 MCA_SYND{1,2} registers
Thread-Index: AQHYjHDb+eGrTrYjV0q4Ak2q6+c136159KGAgApwm4CAAFH9AIAXBOsAgABMvtA=
Date:   Tue, 2 Aug 2022 16:58:45 +0000
Message-ID: <SJ1PR11MB60835796448A4E04ACC1B968FC9D9@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <20220418174440.334336-1-yazen.ghannam@amd.com>
 <20220418174440.334336-2-yazen.ghannam@amd.com> <Yr2CpuL+JHWblJMD@zn.tnic>
 <YsxefXQDCiJ1zxLG@yaz-fattaah> <YtUgb2Y/H/Wq9yIn@zn.tnic>
 <YtVlNrW58cFmksln@zn.tnic> <YukW/IltcCRwvSM4@yaz-fattaah>
In-Reply-To: <YukW/IltcCRwvSM4@yaz-fattaah>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.6.500.17
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?8DzMQoLhYXgaMFrB8XXd1LqvbeLZg8zJXLnB8sT7UQp/Wrhdeqv9euYQLGxV?=
 =?us-ascii?Q?XH4oQfQqB/AGZg+GPXrxRkVliiUTtqbhtL2N5lFt9bjMA1c8wbFs3us466B7?=
 =?us-ascii?Q?2P47ZMdMMbitpwi2u19/WSjUxXSteNbsE3ecBfaEJPCnqJ3M9W+faBjnH4IW?=
 =?us-ascii?Q?ItrLeh0BdurB8NlirDQ9ucpQDNi/IZsMz4K7Hh2dEeUzEI7fR/yFuDCg8W9I?=
 =?us-ascii?Q?6CeiVk6k+6fZpplTTISNuS0JMeSYmaXzmkA7B48/j/cshsmGlwf0tvQ3weL8?=
 =?us-ascii?Q?sXdeb3nuSplpqvd1g6dkEkZ1SMeC1nzY5riqt/VjOsdEeTOEfnhqii+Fvrm0?=
 =?us-ascii?Q?idz7BVHz9sLL/MWoeoytToOdFmPpTHb6C5VUG2FYJw3flNtfSm8o6TXAHRC1?=
 =?us-ascii?Q?AgCMx2r572arr6RMJRzpdpJnL3TE/zkL2fZgGmbK4XMihCQevX9w3bPeYnY9?=
 =?us-ascii?Q?/Ck+te1+BVXXCL6z+0KygDLPDeHUc87VIwS/n+P4+yhzFSTeBdbsaJRgkXZ+?=
 =?us-ascii?Q?r964mxBdlcLR7rCoszlDB2/VGjJgKGTFRpojTTsxwSZhf3E3kwfTpNo58cED?=
 =?us-ascii?Q?ayzpPbB747RetzAcIIAA/iQ81TKUM5lZEqeufUAp2+bdcdEiNQiVnsBtz7Ch?=
 =?us-ascii?Q?ZADBTSiYbFV8thQ5Iv2pwHT+knQ6XYzqM25WA71MsQK+MNP2jc8RrH4QLHNT?=
 =?us-ascii?Q?kgzO5e3dpHRtCcjA9FSN71RCXfj6iPRIfO2BYljPfaUhYa8FRnQor8ZVPyAj?=
 =?us-ascii?Q?vsyKiuxh0W6SGRX6/comWTBGuNCQtoIF2Bzhep+zwldRVQLysu3PCmLsM3sN?=
 =?us-ascii?Q?G8s4kgsauC73ZNZaooxrkboqYPvoMbdfmmLQbfKXknc68bgF+DBnni/gwM5Q?=
 =?us-ascii?Q?2ECUVByaydpjYpVmJalAzMcoyhDFdWHbcgrWbPNmdap5Ol7Tr9gIzV4IK1+z?=
 =?us-ascii?Q?X6nREql3Ivp+RaMggeAPJHaHiZMdl8uOB+nD/H2a/Xem3UFmzyA1cOKTDP6t?=
 =?us-ascii?Q?PNUsgTZCt1Px+f9AterZztMojnzK/L3DXvn1vm7dt6yqOo/WCAxDUa/8kpvb?=
 =?us-ascii?Q?kbXqzMnswuf8yGgchThXeuc3DglgWpd21FWfmX5JoOTCJ87htl20FRtjiZ8x?=
 =?us-ascii?Q?L3Io4cX9MGgJNeI0jvU+TqRGjsTgv9q8fUvksovnK2LF5Jeu0Gq5aGrLfCln?=
 =?us-ascii?Q?yMWXltySvdLcz9vF3Y2wRB5TfoCEfP525hYHoaio7m7q/MRMt+pve5xOVY+U?=
 =?us-ascii?Q?r4MDdzYnE3jSPZ1A2HIFgD8kNdDKdzfIibJYbHNmVrnjG5xONSsZTuQPZtO9?=
 =?us-ascii?Q?wMO+s2N3cvhv9lof2VrWSgaPNYKIydMfB6fTl5/3qi10vV2ckSTFLK6hYe/M?=
 =?us-ascii?Q?FQSiC76UxdzTpsn9wnoJJm7KykSsD6qFxY8Q6hc17xn3BqHDux/VLG8PYP1o?=
 =?us-ascii?Q?rj5fT8fZco79FSLW4yufR1ekgvOCvzXxgb8taQFetZumcGypJkojVar+XoVI?=
 =?us-ascii?Q?dnJc7RoZwHfB4rqDzJ6DyHQ2iYgwbgDetBExjl3SyVlcwtyZ1V/Nf4u5/7Nx?=
 =?us-ascii?Q?hBkOrWzdLXg814lOqY600h8XI5/3MUzshZcg5GSu?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6083.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e3ac0648-333c-4c88-5c47-08da74a84311
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Aug 2022 16:58:45.4416
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: tENzKSfvgAJA5xxcbEDcEepqhmtqdF1YtByYPPZOjNyVTV0C78MJKAO/KwKiQsOw1pFvy62QCcr0+BpNeI/Gig==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4382
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> I ask because struct mce is UAPI. But I think this is just for /dev/mcelo=
g,
> and this has been deprecated for a while. So on a related note, should
> /dev/mcelog be removed and struct mce moved out of UAPI? Then changes to
> struct mce won't affect user space, and we can just consider the mce trac=
e
> event when reporting to user space.

Even though deprecated, mcelog still has users, so I don't think it should =
go away
yet.

-Tony