Received-SPF: pass (google.com: domain of linux-kernel+bounces-5659-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Tue, 19 Dec 2023 09:17:10 -0800
From: Ira Weiny <ira.weiny@intel.com>
To: Ira Weiny <ira.weiny@intel.com>, Dan Williams <dan.j.williams@intel.com>,
	Smita Koralahalli <Smita.KoralahalliChannabasappa@amd.com>, Jonathan Cameron
	<jonathan.cameron@huawei.com>, Shiju Jose <shiju.jose@huawei.com>
CC: Yazen Ghannam <yazen.ghannam@amd.com>, Davidlohr Bueso
	<dave@stgolabs.net>, Dave Jiang <dave.jiang@intel.com>, Alison Schofield
	<alison.schofield@intel.com>, Vishal Verma <vishal.l.verma@intel.com>, "Ard
 Biesheuvel" <ardb@kernel.org>, <linux-efi@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <linux-cxl@vger.kernel.org>, Bjorn Helgaas
	<bhelgaas@google.com>
Subject: Re: [PATCH v4 7/7] cxl/memdev: Register for and process CPER events
Message-ID: <6581d01645c01_277bd294c8@iweiny-mobl.notmuch>
References: <20231215-cxl-cper-v4-0-01b6dab44fcd@intel.com>
 <20231215-cxl-cper-v4-7-01b6dab44fcd@intel.com>
 <4cb5c275-566c-9414-7088-1e91378a409a@amd.com>
 <6580b21723b2c_269bd294f8@dwillia2-mobl3.amr.corp.intel.com.notmuch>
 <6581cba663770_24f6b2942b@iweiny-mobl.notmuch>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <6581cba663770_24f6b2942b@iweiny-mobl.notmuch>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?eHQHmcC2Re96x6np5Ly6tGbepV4VZoTWfGWvjMkiucaoXDNuPCPbiLl78xrg?=
 =?us-ascii?Q?IvJlmAtje5iBDe83bN51o4dUbbiY6b5jGSNVVAcFfsXoKtiMe+kvMU1mW0r0?=
 =?us-ascii?Q?CT/LPQbr1VTtfU2sNdyUxPIaJmEU4AWuPkdlw9GdgmZ9pJ1M2XSpcRdbG6Ec?=
 =?us-ascii?Q?GZsAS40R6KoLn8i01MmAiZTKyvdI4Ge+TIj37EC5XZWGwXYx5SHZ4PYKJDY0?=
 =?us-ascii?Q?CSTcQvTQ3b92M57ar4Ai+rpgqJYdvS+vaxqDRaAvBFjCvrijbwQSXusDhAX/?=
 =?us-ascii?Q?CIwtz9uBPZKkD3Mamuruao93QHlMFR8wXTjHX4+iTPCR9aJXWyTl37fKvbCf?=
 =?us-ascii?Q?Sa38p4Wo1cFlONR6sDWrtV1laCkwL0IUqjl9NjV+xOLvbvTRCk7SQXc0EIg+?=
 =?us-ascii?Q?07EoCFRkiN5NBRaq7kMjQTp9EJjBLFmpbLoobsy3gZoyZrqs6Ob0qd8MMLIK?=
 =?us-ascii?Q?dUBxK0svYbUw4fdwt4Rx2f0N/8NAMlJavZG2h7iDGUyx8dInP48zea04RdM7?=
 =?us-ascii?Q?rd2FLlO/oBgjgebWEo0CsX2yDAYZZyon6hTsI1jtskxxZ05qYiP5ImknjiRS?=
 =?us-ascii?Q?C7gnCNPw7p7sNUyvdv92Gk2wtWK67HJI1xK7sGCVP6G9q/oORmHsJ+UMfOwV?=
 =?us-ascii?Q?ghLbXNWeUv0fGD/HGwmS+6kDU2DVIat3H9dNnhsYiYSPPabaDq1vcBZ7iR8Z?=
 =?us-ascii?Q?lJKYFuUag4igEo7KWHoJZWkL2ELU8rjouNQpywJtsRycJoiIkBtq09iRqOvt?=
 =?us-ascii?Q?b3FaqHLqh+JK2w3zZwWralHK8O55M/5GPS5TBzU+4SHew3E2RDIYNGoOGjT2?=
 =?us-ascii?Q?Ttd+tmz/ZndIPx74Z9AQaYnFBEjPsNRyLBTe2LmDPXZHMNqH7baFiu5ZBb8T?=
 =?us-ascii?Q?0jJFSBHtQKobs5GE34TNmN4QfRhCiDQ0dnu1piGyKxAJvkQ6vdy/tQoNVSJB?=
 =?us-ascii?Q?JK5o0BywTdiL8P5ai9VmncqlfyD/dNM/WPc05Yci/Dwoar4W44rKXurjLHaw?=
 =?us-ascii?Q?MpAA2Xju+Tag543eyt90pQ6bX9jRg8pWkdPy+VOQZwS5DMR8I9OeCnaYNaSg?=
 =?us-ascii?Q?YbkOMsI3t4+jWsp+eGzA2NsoRzl5sTXlTscSAWy/3RXkzOBHzgiFnlxiVQMK?=
 =?us-ascii?Q?Of0aJlUDEpSoNdB7xVQK3r0kVDbQyxkAiDm7T0xExVxcQmliZ3VB7/ljKF4K?=
 =?us-ascii?Q?r9F6sF2iXF0b1EuMRUc1U/EWig18tAMZMrjtxYD+lCI5H6qtmObrszFEYfqj?=
 =?us-ascii?Q?8MERK6B1qBixI84BkYFvBM48EMX9sygl1WAm2zExeNXLDBfdb7qU/98RSa4w?=
 =?us-ascii?Q?KTvrz5OnFeUwgWuTZiW+TQi2+a+Sj5ItkY7wkWv0UBxMrIMzvRt77j+dvaj0?=
 =?us-ascii?Q?e3gjGJkrAdYj3jECDinCTYOa8RbjwUOR8ASO5IqgcGF8KPz27M6rdY1PKzAR?=
 =?us-ascii?Q?AjiDIbHkJOIyiN3s09yoh+IaGDpVT/9dkpQMitvhgFhasW2jfGZKs9Nz60+f?=
 =?us-ascii?Q?DbWsFr1SHfnloPa6I+owzNSn+nfik7YFwx7CH/sgqUrZTqjs3IvU8ulF99HT?=
 =?us-ascii?Q?g5EhvPIRg3nhOCaDhXfhy/v+qnWb1x24GicXXNv08rVpziM/zS6tikNCkx7q?=
 =?us-ascii?Q?JgjoVDs+Hwl7rGhiJDKaj0BsjrwdrzGv8qJXJt1y5VLN?=
X-MS-Exchange-CrossTenant-Network-Message-Id: de8c4204-c434-4176-a5a8-08dc00b657d5
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2023 17:17:13.8480
 (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: cyME1Ju9Q084duNmHlyumgJmeHluqWdwTpTrOduNe49UO60/xUUDs64nGGO8BPvCUewHNtg7lhtseYkpOz0fmQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV2PR11MB5976
X-OriginatorOrg: intel.com

Ira Weiny wrote:
> Dan Williams wrote:
> > Smita Koralahalli wrote:
> > > On 12/15/2023 3:26 PM, Ira Weiny wrote:

[snip]

> > > I remember Dan pointing out to me this when I sent decoding for protocol 
> > > errors and its still pending on me for protocol errors.
> > 
> > Good point, so I think the responsibility to trace CXL events should
> > belong to ghes_do_proc() and ghes_print_estatus() can just ignore CXL
> > events.
> > 
> > Notice how ghes_proc() sometimes skips ghes_print_estatus(), but
> > uncoditionally emits a trace event in ghes_do_proc()? To me that means
> > that the cper_estatus_print() inside ghes_print_estatus() can just defer
> > to the ghes code to do the hookup to the trace code.
> > 
> > For example, ras_userspace_consumers() was introduced to skip emitting
> > events to the kernel log when the trace event might be handled. My
> > assumption is that was for historical reasons, but since CXL events are
> > new, just never emit them to the kernel log and always require the trace
> > path.
> > 
> > I am open to other thoughts here, but it seems like ghes_do_proc() is
> > where the callback needs to be triggered.
> 
> I see.
> 
> Ok.  I'll create a pre-patch which moves the protocol error first then
> I'll put the events in the ghes_do_proc() well.
> 

Apologies.  I really wanted to make this work a pre-cursor patch but I
see that there is not a trace point for the protocol errors yet.  So as
not to slow the progress of this work I'm going to skip moving the
protocol stuff right now.

Also, as part of this work I think moving the CXL specific defines into
the common linux/cper.h is appropriate at this time.

Unless I hear otherwise I'm going to land the event stuff in that common
header and we can move the protocol error defines later.

Thanks again for all the testing,
Ira