mbox series

[ndctl,RFC,0/1] cxl: Script to manage regions during CXL device hotplug remove-reinsert

Message ID 20230807213635.3633907-1-terry.bowman@amd.com
Headers show
Series cxl: Script to manage regions during CXL device hotplug remove-reinsert | expand

Message

Bowman, Terry Aug. 7, 2023, 9:36 p.m. UTC
This patch is a script providing region deletion-creation management
required during hotplug remove-insert. It allows a user to hotplug
remove and reinsert a CXL device without requiring the user to make
additional tool calls to destroy-create the region. The tool will save
the region configuration before removal and recreate the region later
after hotplug reinsert. The script makes the necessary calls to the cxl
cli and daxctl tools to manage the regions.

This may be useful to add to the ndctl tools as a standalone tool or
incoprorate into the existing CXL cli or daxctl tools. I'm interested in
feedback.

Terry Bowman (1):
  cxl: Script to manage regions during CXL device hotplug
    remove-reinsert

Comments

Dan Williams Aug. 8, 2023, 5:45 a.m. UTC | #1
Terry Bowman wrote:
> This patch is a script providing region deletion-creation management
> required during hotplug remove-insert. It allows a user to hotplug
> remove and reinsert a CXL device without requiring the user to make
> additional tool calls to destroy-create the region. The tool will save
> the region configuration before removal and recreate the region later
> after hotplug reinsert. The script makes the necessary calls to the cxl
> cli and daxctl tools to manage the regions.

Nice!

> This may be useful to add to the ndctl tools as a standalone tool or
> incoprorate into the existing CXL cli or daxctl tools. I'm interested in
> feedback.

I think something like this could be carried in a new cxl/contrib/
directory.

I also note that the cxl-cli command harness is stolen from git which
has the ability to have sub-commands defined by scripts, like
'git-send-email', rather than C library calls.

So if there is a desire for scripting language support that could be an
option.