Linux CXL
 help / color / mirror / Atom feed
From: Dan Williams <dan.j.williams@intel.com>
To: <sthanneeru.opensrc@micron.com>, <linux-cxl@vger.kernel.org>
Cc: <Jonathan.Cameron@huawei.com>, <dan.j.williams@intel.com>,
	<john@jagalactic.com>, <emirakhur@micron.com>,
	<ajayjoshi@micron.com>, <Ravis.OpenSrc@micron.com>,
	<sthanneeru@micron.com>
Subject: RE: [PATCH v3 0/2] Add log related mailbox commands
Date: Tue, 26 Mar 2024 16:37:18 -0700	[thread overview]
Message-ID: <66035c2e8ba17_770232948b@dwillia2-xfh.jf.intel.com.notmuch> (raw)
In-Reply-To: <20240313071218.729-1-sthanneeru.opensrc@micron.com>

sthanneeru.opensrc@ wrote:
> From: Srinivasulu Thanneeru <sthanneeru.opensrc@micron.com>
> 
> Add support to expose following mailbox commands to userspace
> for clearing and populating the Vendor debug log in certain
> scenarios, allowing for the aggregation of results over time.
> 
> 1. CXL r3.1 8.2.9.5.3 Get Log Capabilities.
> 2. CXL r3.1 8.2.9.5.4 Clear Log commands.
> 3. CXL r3.1 8.2.9.5.6 Get Supported Logs Sub-List.
> 
> ---
> Changes in v3:
> - 'Component State Dump log' has several caveats for ioctl()
>    not being a suitable ABI as pointed in v2.(Dan Williams)
> - Remove Component State Dump from Clear log filter.
> - Implement a seperate patch(yet to do) to address issues as pointed in v2.

Circling back to this question... I had overlooked the fact that in v3.1
the "Request Abort Background Operation" command was added (8.2.9.1.5).
With that the kernel can safely support background commands with
indefinite residency. So as long as the device supports that command and
advertises that Log populate requests can be cancelled then we can build
a facility to cancel any user-submitted background commands when a
kernel internal need for the background command slot arises.

  parent reply	other threads:[~2024-03-26 23:37 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-13  7:12 [PATCH v3 0/2] Add log related mailbox commands sthanneeru.opensrc
2024-03-13  7:12 ` [PATCH v3 1/2] cxl/mbox: Add Get Log Capabilities and Get Supported Logs Sub-List commands sthanneeru.opensrc
2024-04-26 16:02   ` Jonathan Cameron
2024-03-13  7:12 ` [PATCH v3 2/2] cxl/mbox: Add Clear Log mailbox command sthanneeru.opensrc
2024-04-26 16:05   ` Jonathan Cameron
2024-03-26 23:37 ` Dan Williams [this message]
2024-04-01  6:03   ` [EXT] RE: [PATCH v3 0/2] Add log related mailbox commands Srinivasulu Opensrc
2024-04-05 17:31     ` 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=66035c2e8ba17_770232948b@dwillia2-xfh.jf.intel.com.notmuch \
    --to=dan.j.williams@intel.com \
    --cc=Jonathan.Cameron@huawei.com \
    --cc=Ravis.OpenSrc@micron.com \
    --cc=ajayjoshi@micron.com \
    --cc=emirakhur@micron.com \
    --cc=john@jagalactic.com \
    --cc=linux-cxl@vger.kernel.org \
    --cc=sthanneeru.opensrc@micron.com \
    --cc=sthanneeru@micron.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