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>
CC:     Borislav Petkov <bp@alien8.de>,
        "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+c136159KGAgApwm4CAAFH9AIAXBOsAgIKw4QCAAEkuuoAABiIggAAIhICAAAIP0IAABBTAgAE7p4CAAAKiUA==
Date:   Tue, 25 Oct 2022 16:46:28 +0000
Message-ID: <SJ1PR11MB608397CDE65551043EE3B44AFC319@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <YtUgb2Y/H/Wq9yIn@zn.tnic> <YtVlNrW58cFmksln@zn.tnic>
 <YukW/IltcCRwvSM4@yaz-fattaah> <Y1a4prRIYNw8GIkm@zn.tnic>
 <Y1a/lCVnlTMk8p75@agluck-desk3.sc.intel.com> <Y1b15vnE/Pd1U4r8@zn.tnic>
 <SJ1PR11MB608390D539CD4B405A195344FC2E9@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <Y1cCU9UqGG7nl8cy@zn.tnic>
 <SJ1PR11MB6083DBDAA90E1E03EC7A9EEAFC2E9@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <SJ1PR11MB6083A794C876D6F44E530CAFFC2E9@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <Y1gQQ8gh1CJf0Tuy@yaz-fattaah>
In-Reply-To: <Y1gQQ8gh1CJf0Tuy@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?1gr2dHgdk8HLXZomOK3+oAY9AqBhBsQR2xddhZr4vE41FrU/9Zw9Q0DhbPRN?=
 =?us-ascii?Q?a5PxVCb2c3Wl2cBAoFRye79F98RjKWUUrY3qc05eCoPQqHSgiTKwzxs2Iji9?=
 =?us-ascii?Q?n5SCXEDI2JkNIwy97Gqb73rRWfoXtOeeDAdX28bzMuQXBDN+r1GlvhlGRu9U?=
 =?us-ascii?Q?oU4Pjan8ZKhQ2CGyQKALxfZeKv+/Ide9h1Kpf9vQEcdSy737jUMVBFkiFV+c?=
 =?us-ascii?Q?/aBpUvv+IRk2+0mpVas5nRvcn1Ai344vwCbK3Vpt6tKdO5FxZ5kbRU+jA8fE?=
 =?us-ascii?Q?h07ocDF7UE4IIL5UPUx+ebDsGAeSE89053nTa7gIH1RgYWpDVrcDsFeYp82+?=
 =?us-ascii?Q?pNM4x2lB5MePesG+n/lkqXnri3/fyEa2TS0Xw8mL20HL1I4W4i4U8VlajJhP?=
 =?us-ascii?Q?vvVXTeOPJtTBLNSuUGBvZ5pGrkhNYX7456lk2hmk/7XnMbmO+f4IAZmyPkVc?=
 =?us-ascii?Q?PkaNFnWPTMicIAia99ryOKPT6zN6ijIOTWCmhLskT27ShrPcIdDr44tq+3OA?=
 =?us-ascii?Q?SaDuQqprSdRdmuhhHcPIXjj+D6q2pCj7AsCxoRZWz+6dk9iI/fLFe8w9ZJcz?=
 =?us-ascii?Q?zgLV8SjhfBKXBQNR+RgGK9aaooJ3o1oVpcdChG7b65tuJkZ10ObMnS5B8wr8?=
 =?us-ascii?Q?YwKXlJQkQ36adbsb4oFcsVj7QUM2ITAuezTXh2mEk7/FqwZepk1s9O+wsRNl?=
 =?us-ascii?Q?0v3LlxdWkZ5sQ8oFdBU7Hm+Pfe3JC2ETBDDS4ekODpMeDKwkbh+avH5yFjTI?=
 =?us-ascii?Q?09seX3Vmb6uSRZJZ+emoj8oS4xbuQOMbr3LR1dnsBQIHHjG4ckf4q5DUMZGq?=
 =?us-ascii?Q?CKxt2Kqo9TN9wBatbeRWiKQ1ugEdFWGbOke6S9mrhkb6qjhHkf2qhvFAldOZ?=
 =?us-ascii?Q?pQ/OV9KZQa3cEHfy7oyCGZpdklym2U0Ir467ShSx/DgNst6W8KayWYVdvVf5?=
 =?us-ascii?Q?+WbViBrBunSpG5UipBAUiox00Y9JyBdzhaa+m1lQ+9sedirtA9t8f6PFQLc1?=
 =?us-ascii?Q?zaqSD/XTjFm+zWmVwXgmmVYnTxHZKprLHU3aEr2sXZc21XeZ6kS2Fce8065q?=
 =?us-ascii?Q?4haEYS/JjjOI86NghbZg7Ejtnf0LYMytqM2A/Ont+TSioszoc86YjiD4tTnb?=
 =?us-ascii?Q?8rY1hrK7T8vyCa+4bUT6/vMjaWo+VXYUXOxMqU3I1aDcKkr3tXoGAz3xaF4M?=
 =?us-ascii?Q?kMucNwVjG/YvTANRrFSZ0tZ7EPJS+tiG0gpMTq2AYybHBRW0Hez5H7ViZHNU?=
 =?us-ascii?Q?Th3qBnAiIXJwU/qYN6ZWrXwE7vPQAOI19tzgfvM2qQlGNMIkfSFRiy7+Arnz?=
 =?us-ascii?Q?fLv/Zq0RqOqzA8X2/+egy2JOz2AW2SrQXHPHmDH3vZE0VNjQcbn53B7m/wt9?=
 =?us-ascii?Q?P473bKJm1AMAcQ8e5dWCW0xE2+Bu2Gaig5mVoxSRmauGyFNliPNo20Fd3dDF?=
 =?us-ascii?Q?PfYZ5byOH6qq5aw3qeM4mN3M5keYEC5wZskBRaNwtsTOg4wFLB9/O6yKDSll?=
 =?us-ascii?Q?fnA1QbfQUW+I6JhomQ0swUAP8ALDQ15xMDHJjrDimbWX408P8/mG3baCGfQK?=
 =?us-ascii?Q?gBlaV56w1IsyacJ6uW0=3D?=
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: f7281006-5a01-42ae-3085-08dab6a8769a
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Oct 2022 16:46:28.6767
 (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: diMahdoQSGx+9HJRwcutSInuq+2M+Gie6kgSyq2egf0jW38ga4h08675wE6Fhzu5TNRpYQ6PYBF4iNXytJVkgQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7145
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        SPF_HELO_NONE,SPF_NONE,URIBL_BLOCKED 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 do like the suggestion from Boris to have a length and vendor data in s=
truct
> mce. This should keep mcelog happy while letting us treat struct mce as a
> semi-internal kernel structure. Also, this avoids having to mess around w=
ith
> all the notifier chain definitions.
>
> I'll start working on an implementation if that's okay with you all. I'll
> include kernel and rasdaemon patches together so we can have context for
> discussion.

Sounds good to me. I'll work on the mcelog changes once you have some
agreed definition for the "struct mce" changes.

-Tony