Ira Weiny wrote: > Dan Williams wrote: > > Initial tests with the CXL CPER implementation identified that error > > reports were being duplicated in the log and the trace event [1]. Then > > it was discovered that the notification handler took sleeping locks > > while the GHES event handling runs in spin_lock_irqsave() context [2] > > > > Given multiple bugs to fix and how late it is in the development cycle, > > remove the CXL hookup for now and try again during the next merge > > window. > > > > Cc: Ard Biesheuvel <ardb@xxxxxxxxxx> > > Cc: Rafael J. Wysocki <rafael@xxxxxxxxxx> > > Cc: Ira Weiny <ira.weiny@xxxxxxxxx> > > Cc: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Link: http://lore.kernel.org/r/20240108165855.00002f5a@xxxxxxxxxx [1] > > Closes: http://lore.kernel.org/r/b963c490-2c13-4b79-bbe7-34c6568423c7@moroto.mountain [2] > > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> > > Dan should we add the following hunk to remove that dead code for now? A dead definition, not dead logic. I think it is ok to stick around.