From patchwork Mon Feb 7 23:10:14 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alison Schofield X-Patchwork-Id: 12737944 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9669D2F2C for ; Mon, 7 Feb 2022 23:06:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1644275174; x=1675811174; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=JFS3oCQeAlST4R4Lawyyy7QjftNOyORmMpZIXJ9X6d4=; b=FcPvB0WgzyDg/VCT6oPEIWlO0WWr/JB9tbfuthtVUQ/8pf8ucTrt+f+k MRsg1EbJl8gB9DbSKjL9sPqR9/m4ta6XWXvhC7yu9saiWf3l9cIyYSL2K JYzxvi28LAboeyZm7qVXW4j5xxGOASf8ZYdUvCILvAueYiU90UvoixNNY ZIg3aGlXsIypb6FG0uB3GXGY6/Hej7lehxYUd1tHvT2G2viH201QQGEqe fNQRJQ6cczSkzUTjp5Q6NLNArpV2qK/wlhB5BRyQj/kHc+UWPypXCqe2g OXo57lHakn4OApJTL4NcfQ/8490gSeAySGz9LfaMJzxVQKRUFQuATDyv9 Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10251"; a="236232051" X-IronPort-AV: E=Sophos;i="5.88,351,1635231600"; d="scan'208";a="236232051" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 15:06:14 -0800 X-IronPort-AV: E=Sophos;i="5.88,351,1635231600"; d="scan'208";a="481742150" Received: from alison-desk.jf.intel.com (HELO localhost) ([10.54.74.41]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 15:06:14 -0800 From: alison.schofield@intel.com To: Ben Widawsky , Dan Williams , Ira Weiny , Vishal Verma Cc: Alison Schofield , nvdimm@lists.linux.dev, linux-cxl@vger.kernel.org Subject: [ndctl PATCH v4 0/6] Add partitioning support for CXL memdevs Date: Mon, 7 Feb 2022 15:10:14 -0800 Message-Id: X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: nvdimm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: Alison Schofield Users may want to view and change partition layouts of CXL memory devices that support partitioning. Provide userspace access to the device partitioning capabilities as defined in the CXL 2.0 specification. The first 4 patches add accessors for all the information needed to formulate a new partition layout. This info is accessible via the libcxl API and a new option in the cxl list command: "partition_info":{ "active_volatile_bytes":273535729664, "active_persistent_bytes":0, "next_volatile_bytes":268435456, "next_persistent_bytes":273267294208, "total_bytes":273535729664, "volatile_only_bytes":0, "persistent_only_bytes":0, "partition_alignment_bytes":268435456 } Patch 5 introduces the libcxl interfaces for the SET_PARTITION_INFO mailbox command and Patch 6 adds the new CXL command: Synopsis: cxl set-partition [..] [] -t, --type= 'pmem' or 'volatile' (Default: 'pmem') -s, --size= size in bytes (Default: all partitionable capacity) -a, --align allow alignment correction -v, --verbose turn on debug The CXL command does not offer the IMMEDIATE mode option defined in the CXL 2.0 spec because the CXL kernel driver does not support immediate mode yet. Since userspace could use the libcxl API to send a SET_PARTITION_INFO mailbox command with immediate mode selected, a kernel patch that rejects such requests is in review for the CXL driver. Changes in v4: (from Dan's review) - cxl set-partition command: add type (pmem | volatile), add defaults for type and size, and add an align option. - Replace macros with return statements with functions. - Add cxl_set_partition_set_mode() to the libcxl API. - Add API documentation to Documentation/cxl/lib/libcxl.txt. - Use log_err/info mechanism. - Display memdev JSON info upon completion of set-partition command. - Remove unneeded casts when accessing command payloads. - Name changes - like drop _info suffix, use _size instead of _bytes. Changes in v3: - Back out the API name change to the partition align accessor. The API was already released in v72. - Man page: collapse into one .txt file. - Man page: add to Documentation/meson.build Changes in v2: - Rebase onto https://github.com/pmem/ndctl.git djbw/meson. - Clarify that capacities are reported in bytes. (Ira) This led to renaming accessors like *_capacity to *_bytes and a spattering of changes in the man pages and commit messages. - Add missing cxl_cmd_unref() when creating json objects. (Ira) - Change the cxl list option to: -I --partition (Dan) - Use OPT_STRING for the --volatile_size parameter (Dan, Vishal) - Drop extra _get_ in accessor names. (Vishal) - Add an accessor for the SET_PARTITION_INFO mbox cmd flag. - Display usage_with_options if size parameter is missing. - Drop the OPT64 define patch. Use OPT_STRING instead. Alison Schofield (6): libcxl: add GET_PARTITION_INFO mailbox command and accessors libcxl: add accessors for capacity fields of the IDENTIFY command libcxl: return the partition alignment field in bytes cxl: add memdev partition information to cxl-list libcxl: add interfaces for SET_PARTITION_INFO mailbox command cxl: add command set-partition Documentation/cxl/cxl-list.txt | 23 +++ Documentation/cxl/cxl-set-partition.txt | 60 ++++++++ Documentation/cxl/lib/libcxl.txt | 5 + Documentation/cxl/meson.build | 1 + cxl/builtin.h | 1 + cxl/cxl.c | 1 + cxl/filter.c | 2 + cxl/filter.h | 1 + cxl/json.c | 113 ++++++++++++++ cxl/lib/libcxl.c | 123 ++++++++++++++- cxl/lib/libcxl.sym | 10 ++ cxl/lib/private.h | 18 +++ cxl/libcxl.h | 18 +++ cxl/list.c | 2 + cxl/memdev.c | 196 ++++++++++++++++++++++++ util/json.h | 1 + util/size.h | 1 + 17 files changed, 575 insertions(+), 1 deletion(-) create mode 100644 Documentation/cxl/cxl-set-partition.txt