Received-SPF: pass (google.com: domain of linux-kernel+bounces-253-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: "Luck, Tony" <tony.luck@intel.com>
To: Daniel Ferguson <danielf@os.amperecomputing.com>, "rafael@kernel.org"
	<rafael@kernel.org>, "lenb@kernel.org" <lenb@kernel.org>,
	"james.morse@arm.com" <james.morse@arm.com>, "bp@alien8.de" <bp@alien8.de>,
	"pabeni@redhat.com" <pabeni@redhat.com>, "linmiaohe@huawei.com"
	<linmiaohe@huawei.com>, "rostedt@goodmis.org" <rostedt@goodmis.org>,
	"leoyang.li@nxp.com" <leoyang.li@nxp.com>, "luoshengwei@huawei.com"
	<luoshengwei@huawei.com>
CC: "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH RESEND v3 1/1] RAS: Report ARM processor information to
 userspace
Thread-Topic: [PATCH RESEND v3 1/1] RAS: Report ARM processor information to
 userspace
Thread-Index: AQHaLuSgz1v/s4k9/k+lw9lC4B057LCpbbIA
Date: Thu, 14 Dec 2023 23:36:12 +0000
Message-ID: <SJ1PR11MB60835CC6392CF269E296B82DFC8CA@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <20231214232330.306526-1-danielf@os.amperecomputing.com>
 <20231214232330.306526-2-danielf@os.amperecomputing.com>
In-Reply-To: <20231214232330.306526-2-danielf@os.amperecomputing.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?r3GK27EUfVkqqgbcBx0AuErt5JyCpTIaj5WbWP4NWl6QzBu89MwdWjGRQXK4?=
 =?us-ascii?Q?aMOJcgBtqiRlrlVuXF+vruUrvwI45BVdmMCkt5KP+jBokaKDXtOZQs4VgrE8?=
 =?us-ascii?Q?NfFy8NlzoQ6Ft4HhE9jjh2AsV7NWu51RwA/c35euX6tpn8qrUcC6l+qXUBus?=
 =?us-ascii?Q?xkkjjLIyNdyzpDNA69l6UCmaaOEcIlR5vn77LKpKRr38xZ5VQ8OFNkKzAGeE?=
 =?us-ascii?Q?m2TEEQtx02KGwyXmg7johhmPkPQ5PDuzPnVJEsStYXwNWcJjLKv5EbkMs/gG?=
 =?us-ascii?Q?qw4zkYX+DAAOnH5bEKxyP75tYUElhrFYcQIc0zUo37cWgsJU4h61ZeB8vdwT?=
 =?us-ascii?Q?xCp9VLF0U4jY4ToZMADUZ5CBLb//GdF9Pt7vUVtcudcFUUZReFoDv2zMSkZu?=
 =?us-ascii?Q?y+xoAxh7X8HIMCfYwNHjM2iapyVHnOPErXyBoV44x86lUrNEIz30yaFBVYF+?=
 =?us-ascii?Q?DdrjUn8eWtBwBXG/+p/Y1EExKj7jpy8c2T19CiaNIrUQEhCWcMBRX67XdEsl?=
 =?us-ascii?Q?nukJZUBILz+ISZt4iCHnlro84L1VROqxu1qybqS/0BHuTz3DUt0NRTN0IR7O?=
 =?us-ascii?Q?X/P8jHBtRubt4L/u5PtJYWIC1JEls4RoHN1GlL1w5t3jaXFIJoiggeUiiSdn?=
 =?us-ascii?Q?2W9EdkZA5c1KrupUuok0nf0IAi6gyrkEBkC6vZYSDqb/Xqy7THyp2+Bo3TKz?=
 =?us-ascii?Q?CZpiGab0hV9LS6snrIZTaVoThVLq27gRQ8JRQG07pZfK1rRthsOYS2AeMW1y?=
 =?us-ascii?Q?GaC71reyUwMa6mxjcCYOGv+elSaFAv/KWAk6K/khdZvEcbBnDW9WQCFjPXtz?=
 =?us-ascii?Q?y6MzcgcKr5PMBfV90rB/N27LjNns5Ioe0hySiJfcJGO5AZfyR6puMzGIxeht?=
 =?us-ascii?Q?vtRfZ2Lh+6oPhEdemktFHG3ETvQZji2DDqpjAJe5yf7m+15NVjCxAF33ZVqV?=
 =?us-ascii?Q?XoVij5vuY1BsBcO3U8/ou4E7R7YUtcWKADiDyx5yHuwUL5o31xWvB8vj+aC2?=
 =?us-ascii?Q?aWrBKXlYdT6wzNfFHSE02FoaM+nryOgD/Z50cVXon0m7N86LI0XpuPPaVvNF?=
 =?us-ascii?Q?iKGhjaw3krZ9UReGFYZmw6Su7aYHyvPWtbRfOxTZBAC8S5ohpErRXMYDiCo/?=
 =?us-ascii?Q?JRiJlNyrRKvaLBBAFRQIa49nA5nOfDXZyAyhyTbWuXVNg6BoBEk5B/Ml/LuJ?=
 =?us-ascii?Q?O/B6re7sJlnYOCfH+YPG0tlcOAYvFPcou6agwsAwbNsi9AZqPsRrneNthhKs?=
 =?us-ascii?Q?g7Fof6a5A2YhupxxNlk6Jzj1BvBCQcgxV0Guf/ylMHHVLrxaAYQD3OLhHbzH?=
 =?us-ascii?Q?FJ2E3iT8ra0XPFV0+6W5vb/ODqphEY8wPsdjpbZ7F2ktRSb+XFqMhgjNXCZO?=
 =?us-ascii?Q?dzKoiBEGHGdYNkrKuowgoXVD2xEvKaZTsAGgLj8SghuRqQMI+jZsYOcT4/8i?=
 =?us-ascii?Q?ukhfVsUDe53hLqMTozDv0eoOAu927I8a8jpnEOQvaTMElGg02bemELlXTyjv?=
 =?us-ascii?Q?StWS33r8n1/DZEpJTJvetCgGCB6c2gVJyvoOSQDtSlREV533htPVE4/m0YEy?=
 =?us-ascii?Q?pATIIng+ZeLhrpaw2GA7Hbr4aUoVTKO/cAG/sioP?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
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: 5735c1f9-778f-4571-ebe5-08dbfcfd7570
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Dec 2023 23:36:12.9963
 (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: tL3zYXzmA2wtJn3c4s6r1AVZukxh/jDcwx9N2jWcRnrderRlzHDEZA9IZc09jrJmz4nhncmowKXX2oEgtJhe2Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB5036
X-OriginatorOrg: intel.com

>  drivers/acpi/apei/ghes.c |  3 +--
>  drivers/ras/ras.c        | 46 ++++++++++++++++++++++++++++++++++++--
>  include/linux/ras.h      | 15 +++++++++++--
>  include/ras/ras_event.h  | 48 +++++++++++++++++++++++++++++++++++-----
>  4 files changed, 101 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/ras/ras.c b/drivers/ras/ras.c
> index 95540ea8dd9d..2a7f424d59b9 100644
> --- a/drivers/ras/ras.c
> +++ b/drivers/ras/ras.c
> @@ -21,9 +21,51 @@ void log_non_standard_event(const guid_t *sec_type, co=
nst guid_t *fru_id,
>       trace_non_standard_event(sec_type, fru_id, fru_text, sev, err, len)=
;
>  }
>
> -void log_arm_hw_error(struct cper_sec_proc_arm *err)
> +void log_arm_hw_error(struct cper_sec_proc_arm *err, const u8 sev)
>  {

What GIT tree is this patch based on? I don't see this log_arm_hw_error()
function in drivers/ras/ras.c upstream, or in intel-next.

Should ARM specific code like this be in a file with "arm" in its name? Or =
at least
be inside some #ifdef CONFIG_ARM.

-Tony