All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Jiang <dave.jiang@intel.com>
To: Alison Schofield <alison.schofield@intel.com>
Cc: linux-cxl@vger.kernel.org, dave@stgolabs.net, jic23@kernel.org,
	vishal.l.verma@intel.com, ira.weiny@intel.com, djbw@kernel.org,
	Davidlohr Bueso <dave@stgolabs.net>
Subject: Re: [PATCH 0/7] cxl: Add CXL type2 accelerator support for cxl_test
Date: Wed, 6 May 2026 07:59:42 -0700	[thread overview]
Message-ID: <13c78882-d3df-474c-8232-6b22ef1b44ec@intel.com> (raw)
In-Reply-To: <afrBq3AIDrAkTTWR@aschofie-mobl2.lan>



On 5/5/26 9:20 PM, Alison Schofield wrote:
> On Wed, Apr 22, 2026 at 04:02:30PM -0700, Dave Jiang wrote:
>> The series is based on v7.0-rc7 with Alejandro's type2 series [1] and
>> with Dan's follow on series [2].
>>
>> Series adds a 'type2_test' module parameter where it setup a mock type2
>> hierarchy with a mock type2 accelerator device directly under a root
>> port that has an auto region setup.
>>
>> It also includes a CXL fix that was encountered when running tests with
>> the new code.
>>
>> [
>>   {
>>     "memdevs":[
>>       {
>>         "memdev":"mem0",
>>         "ram_size":536870912,
>>         "ram_qos_class":42,
>>         "host":"cxl_type2_accel.0",
>>         "poison_injectable":false
>>       }
>>     ]
>>   },
>>   {
>>     "regions":[
>>       {
>>         "region":"region0",
>>         "resource":70300293136384,
>>         "size":536870912,
>>         "type":"ram",
>>         "interleave_ways":1,
>>         "interleave_granularity":4096,
>>         "decode_state":"commit"
>>       }
>>     ]
>>   }
>> ]
>>
>>     "root decoders":[
>>       {
>>         "decoder":"decoder0.0",
>>         "resource":70300293136384,
>>         "size":1073741824,
>>         "interleave_ways":1,
>>         "accelmem_capable":true,
> 
> FWIW - above was a pleasant surprise. We've had accelmem_capable plumbed in
> cxl list since 2022 !
> 
> That's a segway to a sanity check: is there anything else we want to
> display or do in the cxl-cli tooling that is special for Type 2 ?

Maybe back invalidate attributes? I don't remember if that's already there or if Davidlohr is adding that.

DJ

> 
> 
> 
>>         "qos_class":42,
>>         "nr_targets":1
>>       },
>>
>> [1]: https://lore.kernel.org/linux-cxl/20260330143827.1278677-1-alejandro.lucero-palau@amd.com/T/#t
>> [2]: https://lore.kernel.org/linux-cxl/20260403210050.1058650-1-dan.j.williams@intel.com/T/#t
>>
>>
>> Dave Jiang (7):
>>   cxl/test: Refactor mock_init_hdm_decoder() to prep for type2 decoder
>>   cxl/test: Add type2 support for mock CFMWS0
>>   cxl/test: Refactor platform device enumerations
>>   cxl/test: Add hierarchy enumeration support for type2 device
>>   cxl/test: Fixup hdm init for auto region to support type2
>>   cxl/test: Add cxl_test accelerator driver
>>   cxl: Fix double unregistration of CXL regions for type2 devices
>>
>>  drivers/cxl/core/region.c      |   3 +-
>>  tools/testing/cxl/test/Kbuild  |   2 +
>>  tools/testing/cxl/test/accel.c |  71 ++++
>>  tools/testing/cxl/test/cxl.c   | 717 ++++++++++++++++++++++++---------
>>  4 files changed, 605 insertions(+), 188 deletions(-)
>>  create mode 100644 tools/testing/cxl/test/accel.c
>>
>>
>> base-commit: 6fa897f2d9a917c4bd7b911b230fcfad05a952c2
>> -- 
>> 2.53.0
>>


      reply	other threads:[~2026-05-06 14:59 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-22 23:02 [PATCH 0/7] cxl: Add CXL type2 accelerator support for cxl_test Dave Jiang
2026-04-22 23:02 ` [PATCH 1/7] cxl/test: Refactor mock_init_hdm_decoder() to prep for type2 decoder Dave Jiang
2026-05-06  4:31   ` Alison Schofield
2026-05-07 23:35     ` Dave Jiang
2026-04-22 23:02 ` [PATCH 2/7] cxl/test: Add type2 support for mock CFMWS0 Dave Jiang
2026-05-06  4:38   ` Alison Schofield
2026-05-07 23:36     ` Dave Jiang
2026-04-22 23:02 ` [PATCH 3/7] cxl/test: Refactor platform device enumerations Dave Jiang
2026-05-06  4:45   ` Alison Schofield
2026-05-07 23:36     ` Dave Jiang
2026-04-22 23:02 ` [PATCH 4/7] cxl/test: Add hierarchy enumeration support for type2 device Dave Jiang
2026-05-06  5:05   ` Alison Schofield
2026-05-07 23:37     ` Dave Jiang
2026-04-22 23:02 ` [PATCH 5/7] cxl/test: Fixup hdm init for auto region to support type2 Dave Jiang
2026-05-06  5:07   ` Alison Schofield
2026-04-22 23:02 ` [PATCH 6/7] cxl/test: Add cxl_test accelerator driver Dave Jiang
2026-05-06  5:19   ` Alison Schofield
2026-05-07 23:41     ` Dave Jiang
2026-04-22 23:02 ` [PATCH 7/7] cxl: Fix double unregistration of CXL regions for type2 devices Dave Jiang
2026-04-23  7:10   ` Alejandro Lucero Palau
2026-04-23 14:36     ` Dave Jiang
2026-04-29 23:45   ` Dan Williams (nvidia)
2026-04-23  7:16 ` [PATCH 0/7] cxl: Add CXL type2 accelerator support for cxl_test Alejandro Lucero Palau
2026-05-06  4:20 ` Alison Schofield
2026-05-06 14:59   ` Dave Jiang [this message]

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=13c78882-d3df-474c-8232-6b22ef1b44ec@intel.com \
    --to=dave.jiang@intel.com \
    --cc=alison.schofield@intel.com \
    --cc=dave@stgolabs.net \
    --cc=djbw@kernel.org \
    --cc=ira.weiny@intel.com \
    --cc=jic23@kernel.org \
    --cc=linux-cxl@vger.kernel.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.