Linux CXL
 help / color / mirror / Atom feed
From: Dan Williams <dan.j.williams@intel.com>
To: Jonathan Cameron <Jonathan.Cameron@huawei.com>,
	Dan Williams <dan.j.williams@intel.com>
Cc: Shiyang Ruan <ruansy.fnst@fujitsu.com>,
	<linux-cxl@vger.kernel.org>, <dave.jiang@intel.com>,
	<vishal.l.verma@intel.com>
Subject: Re: Some thoughts and questions about CXL & MCE
Date: Mon, 8 Jan 2024 13:14:06 -0800	[thread overview]
Message-ID: <659c659e938cf_127da22943b@dwillia2-xfh.jf.intel.com.notmuch> (raw)
In-Reply-To: <20240108123749.0000027f@Huawei.com>

Jonathan Cameron wrote:
[..]
> One other wrinkle I'm working through is the control of CPER vs normal reporting.
> Current thought is we do what ACPI allows and start in firmware first, until the
> _OSC call.  If that requests native handling we go back to what we currently
> support (native only emulation).
> 
> However, there isn't a convenient way to mess with what Linux asks for which we'd
> want to make it easy to test the handling once the driver stack is up.
> 
> I'm not sure anyone would be keen on a pci_aer=no-ask,cxl-mem-error=no-ask type
> kernel boot parameter to instruct the kernel to never ask for control.

I just expressed a similar lament to someone else asking about this, and
claimed that is up to the BIOS to say "no", not for Linux to skip
asking. It turns out that the Linux pci=noear knob predated _OSC:

   7ece14175376 PCI/AER: Remove aerdriver.forceload kernel parameter

..., so there was legacy to carry forward. Otherwise, in a post _OSC
world it's the OS responsibility to ask and the firmware responsibility
to optionally say, "no".

> I also don't much like a qemu parameter which basically says 'report aer as
> broken so the OS can't grab it'. Anyhow those are details.
> 
> Ah well. Getting the _OSC handshake to save what was negotiated on qemu side was
> fiddly but I got that working on Friday so I have all the pieces for protocol errors
> done (ARM only for now - I need to look at notifications in ACPI on x86 + enable
> HEST in general on qemu-x86).  Will post an RFC for ARM shortly.
> 
> Bare metal will be burried in bios config most likely.
> 
> Jonathan
> 
> 
> > 
> > > I don't fully understand the CXL spec yet (it's difficult for me), so 
> > > the above ideas may be immature, but I really want to figure out how we 
> > > can make CXL & MCE work.  I'd really appreciate it if you could help me 
> > > on this!
> > > 
> > > [1] 
> > > https://lore.kernel.org/linux-cxl/20231220-cxl-cper-v5-0-1bb8a4ca2c7a@intel.com/T/#u  
> > 
> > Following this work from Smita and Ira is the right path.
> 



  reply	other threads:[~2024-01-08 21:14 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-22 16:33 Some thoughts and questions about CXL & MCE Shiyang Ruan
2024-01-02 17:29 ` Jonathan Cameron
2024-01-03 16:45 ` Dan Williams
2024-01-08 12:37   ` Jonathan Cameron
2024-01-08 21:14     ` Dan Williams [this message]
2024-01-09 16:18       ` Jonathan Cameron
2024-01-09 19:59         ` Dan Williams

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=659c659e938cf_127da22943b@dwillia2-xfh.jf.intel.com.notmuch \
    --to=dan.j.williams@intel.com \
    --cc=Jonathan.Cameron@huawei.com \
    --cc=dave.jiang@intel.com \
    --cc=linux-cxl@vger.kernel.org \
    --cc=ruansy.fnst@fujitsu.com \
    --cc=vishal.l.verma@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox