From: Dan Williams <dan.j.williams@intel.com>
To: Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Dan Williams <dan.j.williams@intel.com>
Cc: <linux-cxl@vger.kernel.org>,
Sreenivas Bagalkote <sreenivas.bagalkote@broadcom.com>,
Brett Henning <brett.henning@broadcom.com>,
Harold Johnson <harold.johnson@broadcom.com>,
Sumanesh Samanta <sumanesh.samanta@broadcom.com>,
<linux-kernel@vger.kernel.org>,
Davidlohr Bueso <dave@stgolabs.net>,
"Dave Jiang" <dave.jiang@intel.com>,
Alison Schofield <alison.schofield@intel.com>,
Vishal Verma <vishal.l.verma@intel.com>,
Ira Weiny <ira.weiny@intel.com>, <linuxarm@huawei.com>,
<linux-api@vger.kernel.org>,
Lorenzo Pieralisi <lpieralisi@kernel.org>,
"Natu, Mahesh" <mahesh.natu@intel.com>
Subject: Re: RFC: Restricting userspace interfaces for CXL fabric management
Date: Thu, 25 Apr 2024 09:18:53 -0700 [thread overview]
Message-ID: <662a826dbeeff_b6e02943c@dwillia2-mobl3.amr.corp.intel.com.notmuch> (raw)
In-Reply-To: <20240425123344.000001a9@Huawei.com>
Jonathan Cameron wrote:
[..]
> > Also, the assertion that these kernels will be built with
> > CONFIG_SECURITY_LOCKDOWN_LSM=n and likely CONFIG_STRICT_DEVMEM=n, then
> > the entire user-mode driver ABI is available for use. CXL commands are
> > simple polled mmio, does Linux really benefit from carrying drivers in
> > the kernel that the kernel itself does not care about?
>
> Sure we could it in userspace... It's bad engineering, limits the design
> to polling only and uses a bunch of interfaces we put a lot of effort into
> telling people not to use except for debug.
>
> I really don't see the advantage in pushing a project/group of projects
> all of which are picking the upstream kernel up directly, to do a dirty
> hack. We loose all the advantages of a proper well maintained kernel
> driver purely on the argument that one use model is not the same as
> this one. Sensible security lockdown requirements is fine (along
> with all the other kernel features that must be disable for that
> to work), making open kernel development on for a large Linux
> market harder is not.
The minimum requirement for justifying an in kernel driver is that
something else in the kernel consumes that facility. So, again, I want
to get back to specifics what else in the kernel is going to leverage
the Switch CCI mailbox?
The generic-Type-3-device mailbox has an in kernel driver because the
kernel has need to send mailbox commands internally and it is
fundamental to RAS and provisioning flows that the kernel have this
coordination. What are the motivations for an in-band Switch CCI command
submission path?
It could be the case that you have a self-evident example in mind that I
have thus far failed to realize.
next prev parent reply other threads:[~2024-04-25 16:19 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-21 17:44 RFC: Restricting userspace interfaces for CXL fabric management Jonathan Cameron
2024-03-21 21:41 ` Sreenivas Bagalkote
2024-03-22 9:32 ` Jonathan Cameron
2024-03-22 13:24 ` Sreenivas Bagalkote
2024-04-01 16:51 ` Sreenivas Bagalkote
2024-04-06 0:04 ` Dan Williams
2024-04-10 11:45 ` Jonathan Cameron
2024-04-15 20:09 ` Sreenivas Bagalkote
2024-04-23 22:44 ` Sreenivas Bagalkote
2024-04-23 23:24 ` Greg KH
2024-04-24 0:07 ` Dan Williams
2024-04-25 11:33 ` Jonathan Cameron
2024-04-25 16:18 ` Dan Williams [this message]
2024-04-25 17:26 ` Jonathan Cameron
2024-04-25 19:25 ` Dan Williams
2024-04-26 8:45 ` Jonathan Cameron
2024-04-26 16:16 ` Dan Williams
2024-04-26 16:53 ` Jonathan Cameron
2024-04-26 19:25 ` Harold Johnson
2024-04-27 11:12 ` Greg KH
2024-04-27 16:22 ` Dan Williams
2024-04-28 4:25 ` Sirius
2024-04-29 12:18 ` Jonathan Cameron
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=662a826dbeeff_b6e02943c@dwillia2-mobl3.amr.corp.intel.com.notmuch \
--to=dan.j.williams@intel.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=alison.schofield@intel.com \
--cc=brett.henning@broadcom.com \
--cc=dave.jiang@intel.com \
--cc=dave@stgolabs.net \
--cc=harold.johnson@broadcom.com \
--cc=ira.weiny@intel.com \
--cc=linux-api@vger.kernel.org \
--cc=linux-cxl@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=lpieralisi@kernel.org \
--cc=mahesh.natu@intel.com \
--cc=sreenivas.bagalkote@broadcom.com \
--cc=sumanesh.samanta@broadcom.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;
as well as URLs for NNTP newsgroup(s).