Dan Williams <[email protected]> writes: > [ add Jeff, Michal, and Adam ]
[ adding Bryan Gurney, who is helping out with RHEL packaging ] > Hey ndctl distro maintainers, > > Just wanted to highlight this new tool submission for your > consideration. The goal here is to have a Linux native provisioning > tool that covers the basics of the functionality that is outside of > the ACPI specification, and reduce the need for ipmctl outside of > exceptional device-specific debug scenarios. Recall that the ACPI NFIT > communicates the static region configuration to the OS, but changing > that configuration is a device-specific protocol plus a reboot. Until > the arrival of pcdctl, region provisioning required ipmctl. It's great to see progress on this, thanks! Shipping another utility as part of the ndctl package is fine with me, though I'm not sure why we wouldn't just make this an ndctl sub-command. From a user's perspective, these are all operations on or about nvdimms. ipmctl didn't have separate utilities for provisioning goals and namespace configuration, for example. > I will note that CXL moves the region configuration into the base CXL > specification so the ndctl project will pick up a "cxl-cli" tool for > that purpose. In general, the ndctl project is open to carrying > support for persistent memory devices with open specifications. In > this case the provisioning specification for devices formerly driven > by ipmctl was opened up and provided here: Is there a meaningful difference to the user? Can you show some examples of how configuration would be different between cxl-attached pmem and memory-bus attached pmem? > https://cdrdv2.intel.com/v1/dl/getContent/634430 > > Please comment on its suitability for shipping in distros alongside > the ndctl tool. It's completely fine to ship more tools with ndctl. I would like a better overall picture of configuration from the admin's perspective. At first glance, I think we're adding unneeded complexity. Cheers, Jeff p.s. I don't find the name 'pdctl' particularly endearing. If we do stick with a separate utility, I'd suggest coming up with a more descriptive name. > > On Thu, Jul 8, 2021 at 11:38 AM James Anandraj > <[email protected]> wrote: >> >> From: James Sushanth Anandraj <[email protected]> >> >> The Intel Optane Persistent Memory OS provisioning specification >> describes how to support basic provisioning for Intel Optane >> persistent memory 100 and 200 series for use in different >> operating modes using OS software. >> >> This patch set introduces a new utility pcdctl that implements >> basic provisioning as described in the provisioning specification >> document at https://cdrdv2.intel.com/v1/dl/getContent/634430 . >> >> The pcdctl utility provides enumeration and region reconfiguration >> commands for "nvdimm" subsystem devices (Non-volatile Memory). This >> is implemented as a separate tool rather than as a feature of ndctl as >> the steps for provisioning are specific to Intel Optane devices and >> are as follows. >> 1..Generate a new region configuration request using this utility. >> 2. Reset the platform. >> 3. Use this utility to list the status of operation. >> >> James Sushanth Anandraj (4): >> Documentation/pcdctl: Add documentation for pcdctl tool and commands >> pcdctl/list: Add pcdctl-list command to enumerate 'nvdimm' devices >> pcdctl/reconfigure: Add pcdctl-reconfigure-region command >> pcdctl/reconfigure: Add support for pmem and iso-pmem modes >> >> Documentation/pcdctl/Makefile.am | 59 + >> .../pcdctl/asciidoctor-extensions.rb | 30 + >> Documentation/pcdctl/pcdctl-list.txt | 56 + >> .../pcdctl/pcdctl-reconfigure-region.txt | 50 + >> Documentation/pcdctl/pcdctl.txt | 40 + >> Documentation/pcdctl/theory-of-operation.txt | 28 + >> Makefile.am | 4 +- >> configure.ac | 2 + >> pcdctl/Makefile.am | 18 + >> pcdctl/builtin.h | 9 + >> pcdctl/list.c | 114 ++ >> pcdctl/list.h | 11 + >> pcdctl/pcat.c | 59 + >> pcdctl/pcat.h | 13 + >> pcdctl/pcd.h | 381 +++++ >> pcdctl/pcdctl.c | 88 + >> pcdctl/reconfigure.c | 1458 +++++++++++++++++ >> pcdctl/reconfigure.h | 12 + >> util/main.h | 1 + >> 19 files changed, 2431 insertions(+), 2 deletions(-) >> create mode 100644 Documentation/pcdctl/Makefile.am >> create mode 100644 Documentation/pcdctl/asciidoctor-extensions.rb >> create mode 100644 Documentation/pcdctl/pcdctl-list.txt >> create mode 100644 Documentation/pcdctl/pcdctl-reconfigure-region.txt >> create mode 100644 Documentation/pcdctl/pcdctl.txt >> create mode 100644 Documentation/pcdctl/theory-of-operation.txt >> create mode 100644 pcdctl/Makefile.am >> create mode 100644 pcdctl/builtin.h >> create mode 100644 pcdctl/list.c >> create mode 100644 pcdctl/list.h >> create mode 100644 pcdctl/pcat.c >> create mode 100644 pcdctl/pcat.h >> create mode 100644 pcdctl/pcd.h >> create mode 100644 pcdctl/pcdctl.c >> create mode 100644 pcdctl/reconfigure.c >> create mode 100644 pcdctl/reconfigure.h >> >> -- >> 2.20.1 >> >>
