From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6DA4DD59F7C for ; Wed, 6 Nov 2024 23:35:00 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id CE4F46B0088; Wed, 6 Nov 2024 18:34:59 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id C94E26B008C; Wed, 6 Nov 2024 18:34:59 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B35356B0092; Wed, 6 Nov 2024 18:34:59 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 944496B0088 for ; Wed, 6 Nov 2024 18:34:59 -0500 (EST) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 10A0BC0217 for ; Wed, 6 Nov 2024 23:34:59 +0000 (UTC) X-FDA: 82757276322.19.C408A75 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by imf16.hostedemail.com (Postfix) with ESMTP id 01FAF18000F for ; Wed, 6 Nov 2024 23:34:20 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=drTYMG5Z; spf=pass (imf16.hostedemail.com: domain of dave.jiang@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=dave.jiang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1730935930; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=V5I/fNCw1d9RnkmGKh3Pd2GvkoBd5cVcUPP1y4aBx0Y=; b=fk29VRFRk5ksZ9XIns1mmgjFIZXQGomm0HEmh5lYWh3U/kH5QgAXv+DP/NqzcltW+dmrAW d8+foiGI6BTlIrpJViktdatFrVkaS6mPC2dURptNWY1TBh4bKWemoKjwxI+KoKNSiLIl84 EXnzvM9TLKCdXoq8r0KJ0t88bsULBcE= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=drTYMG5Z; spf=pass (imf16.hostedemail.com: domain of dave.jiang@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=dave.jiang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1730935930; a=rsa-sha256; cv=none; b=8mS2DkJic1vjdGH3RBVbUzLSoGXSkmHNjyl7GZQiHIKzi6vuY51TCUMwkecraxAV3iuXUN SpApJUzBS6ARijz2xQySMzW+tOPORiRqjfqW33UYUoQ6lJKxaa69j14FOwlbWH49Rjw/Sn 5ybx/Xb9b0aJypXLa43qIZxnkYuOipc= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1730936096; x=1762472096; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=eSWnXMt84uuSKv4n1E/4P/zRZgnj3xbM9RKn/bEzlmg=; b=drTYMG5Z2ILnDMxe1r8LfZ++/MtdPHAfwLEtdbX+hH6P9jrXZfhc9byE PZTXVfIojNapq41ac6D+QLAVhbjjWFn93dYniHFDA85H5dc+T44p0SwTV QXmA75lNiYr5vTGG0MOlQRGfN56VwVQDaNSfz4leNTrA3IWinkSvMRZ5K QaLpkIP3enleV4Do0CCUDpLfdaDTPx8oIZ0Om6oS0rudfaLMpXoFZWQPs XKAVHiiuCUSMhGJ8jJtWaGXoZBR2aanbJEPuuXN7iszrLTlhqBG0eQPEh grrYSzRlysg4rsPUcL3TGgZI5ABOIha5+ltdx/QoPTHUG2L7/sBDFNIjj g==; X-CSE-ConnectionGUID: zrUq7a6lQAunReX/vrXvDA== X-CSE-MsgGUID: 0POB0tm6TuO/hMXnYc6cTA== X-IronPort-AV: E=McAfee;i="6700,10204,11222"; a="53317605" X-IronPort-AV: E=Sophos;i="6.11,199,1725346800"; d="scan'208";a="53317605" Received: from fmviesa008.fm.intel.com ([10.60.135.148]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Nov 2024 15:34:54 -0800 X-CSE-ConnectionGUID: NF0vpozUQTikJA2BFFTzYg== X-CSE-MsgGUID: o1lNPTweSA64xESvsANXgw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,264,1725346800"; d="scan'208";a="84912598" Received: from ehanks-mobl1.amr.corp.intel.com (HELO [10.125.110.92]) ([10.125.110.92]) by fmviesa008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Nov 2024 15:34:51 -0800 Message-ID: <49228a3c-23b0-4aac-9c7f-e605045e3c8f@intel.com> Date: Wed, 6 Nov 2024 16:34:49 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v15 04/15] cxl: Add Get Supported Features command for kernel usage To: shiju.jose@huawei.com, linux-edac@vger.kernel.org, linux-cxl@vger.kernel.org, linux-acpi@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Cc: bp@alien8.de, tony.luck@intel.com, rafael@kernel.org, lenb@kernel.org, mchehab@kernel.org, dan.j.williams@intel.com, dave@stgolabs.net, jonathan.cameron@huawei.com, gregkh@linuxfoundation.org, sudeep.holla@arm.com, jassisinghbrar@gmail.com, alison.schofield@intel.com, vishal.l.verma@intel.com, ira.weiny@intel.com, david@redhat.com, Vilas.Sridharan@amd.com, leo.duran@amd.com, Yazen.Ghannam@amd.com, rientjes@google.com, jiaqiyan@google.com, Jon.Grimm@amd.com, dave.hansen@linux.intel.com, naoya.horiguchi@nec.com, james.morse@arm.com, jthoughton@google.com, somasundaram.a@hpe.com, erdemaktas@google.com, pgonda@google.com, duenwen@google.com, gthelen@google.com, wschwartz@amperecomputing.com, dferguson@amperecomputing.com, wbs@os.amperecomputing.com, nifan.cxl@gmail.com, tanxiaofei@huawei.com, prime.zeng@hisilicon.com, roberto.sassu@huawei.com, kangkang.shen@futurewei.com, wanghuiqiang@huawei.com, linuxarm@huawei.com References: <20241101091735.1465-1-shiju.jose@huawei.com> <20241101091735.1465-5-shiju.jose@huawei.com> Content-Language: en-US From: Dave Jiang In-Reply-To: <20241101091735.1465-5-shiju.jose@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 01FAF18000F X-Stat-Signature: hcreyugm1dxiiznw1ji1znkgac51qmfm X-Rspam-User: X-HE-Tag: 1730936060-251191 X-HE-Meta: U2FsdGVkX1/Yk2Vw9Nrz6NHmFjhhlFaJ2Dpdml6mSPiNMCTTlPny4DSOa3y44K/bTpVfmWKYhI94+f1H1n2+7sjHjnu83ZtTIbk1V7xDufRjlQrjrQwVuOoKFPCfzsIUKiWgXewc2tQZQWTDNcnxSHIuGnkpAe8h25gkdQMt9lrAIg8bjdKYMDBuArf/6Xr6HHAEoA1l3c4guy5dxanvRBFlj3G/Qu6yMTSskGbji9WidUnenLJk53NTncTNBi8jv6fXwfWQyrMywlv3O5MbbeRsbtwsu8hfxiPGZm1dHY2fSbcyb7xp380M2IX4EK+cX/en+kHf8+1CrRKyNiwD+pYsOyhT8DxJhAz2vaBC2/nwg8cARMAy/lIe9KV/j0R9iln+EQot9gg909yXk/qRlnM4uVlf35O8sBnD1RTAU+ZUKk+O84D/yWjVc4AouEd7t1BkeVdI1Y4bxbqHxvfm8VLDyiqMEIKhEFNx+P7miUBBilU9Kmup1ozNo/F/4Oenvf4Am/7hEM5lnyZNBP8uOgY3z4g8BWyA4o1l27yiYRaRq/uE+xKTkbaeO+TaQTDsG8pARnlXX5e53IpuZqDO8yh5N6bCxmbLIeSGoUYBYrmS1Y1NKH3GBWG+jYgMuqw79peiFWxZq23JY+5SHMOI4JujG/tJljfUfwntOQgZy4oVOT2dIY0MwHdn4QkJblhjzXYqcVMq4opyzESOgbwpJjIVZtYqi2cnytHa16O8NdTK33jZsPzINGny8Cli6XerSojCkvjmRk2lvtdK2ac+UWFevawmshz2WeVqI/uI9nnt1QvNlyo7gshZkhCOoV3XkfUrgd/FiDIKz0HSuyltmozyMwFw4pALUUUObhXUzrBEdLkbf4+HQh5vy/XyFv+NitL5mv14iPAHoG4vhJahQwmSrPUNKGoegdIpihrIvFUj6s5TdTr2yHpvz0pUr33rVK6So4IcQaZPhl6mekT 2ah4pxVr saB3slD0bt8XwKQeqYtW3zo+SS9Dde8A+KDIvzF4J/QEdZY4GfX+gb1sObXY2mRcsCfRVIU+XU95mewZc4yrOouC4XuBghUbD36maFkSU8pa48ABY9ZjUiKQYgSp3OVjvDY5YMThHUMV50o9Y5cs1ISYE6lfJKhwiahUWQ5lIfnqE/yrtvFTzT9xNe0CqVOuA45/nPfK4sC0tmbTTsT08yf7WDiubd2aNCeI3+W6F0ub4F+OMSaGWp4B8BlGZsg3h7n2TzRBOETOFqMTDMgKUoyaH4Cdo3W0AHfczhDTY+fTuYoUev0RBdmhPQQ0XPDaY/rA+ X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 11/1/24 2:17 AM, shiju.jose@huawei.com wrote: > From: Dave Jiang > > CXL spec r3.1 8.2.9.6.1 Get Supported Features (Opcode 0500h) > The command retrieve the list of supported device-specific features > (identified by UUID) and general information about each Feature. > > The driver will retrieve the feature entries in order to make checks and > provide information for the Get Feature and Set Feature command. One of > the main piece of information retrieved are the effects a Set Feature > command would have for a particular feature. > > Reviewed-by: Jonathan Cameron > Signed-off-by: Dave Jiang > Co-developed-by: Shiju Jose > Signed-off-by: Shiju Jose Found couple endien conversion changes below: > --- > drivers/cxl/core/mbox.c | 175 +++++++++++++++++++++++++++++++++++ > drivers/cxl/cxlmem.h | 47 ++++++++++ > drivers/cxl/pci.c | 4 + > include/cxl/mailbox.h | 4 + > include/uapi/linux/cxl_mem.h | 1 + > 5 files changed, 231 insertions(+) > > diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c > index 5175138c4fb7..5045960e3bfe 100644 > --- a/drivers/cxl/core/mbox.c > +++ b/drivers/cxl/core/mbox.c > @@ -67,6 +67,7 @@ static struct cxl_mem_command cxl_mem_commands[CXL_MEM_COMMAND_ID_MAX] = { > CXL_CMD(SET_SHUTDOWN_STATE, 0x1, 0, 0), > CXL_CMD(GET_SCAN_MEDIA_CAPS, 0x10, 0x4, 0), > CXL_CMD(GET_TIMESTAMP, 0, 0x8, 0), > + CXL_CMD(GET_SUPPORTED_FEATURES, 0x8, CXL_VARIABLE_PAYLOAD, 0), > }; > > /* > @@ -795,6 +796,180 @@ static const uuid_t log_uuid[] = { > [VENDOR_DEBUG_UUID] = DEFINE_CXL_VENDOR_DEBUG_UUID, > }; > > +static void cxl_free_features(void *features) > +{ > + kvfree(features); > +} > + > +static int cxl_get_supported_features_count(struct cxl_dev_state *cxlds) > +{ > + struct cxl_mailbox *cxl_mbox = &cxlds->cxl_mbox; > + struct cxl_mbox_get_sup_feats_out mbox_out; > + struct cxl_mbox_get_sup_feats_in mbox_in; > + struct cxl_mbox_cmd mbox_cmd; > + int rc; > + > + memset(&mbox_in, 0, sizeof(mbox_in)); > + mbox_in.count = sizeof(mbox_out); mbox_in.count = cpu_to_le32(sizeof(mbox_out)); > + memset(&mbox_out, 0, sizeof(mbox_out)); > + mbox_cmd = (struct cxl_mbox_cmd) { > + .opcode = CXL_MBOX_OP_GET_SUPPORTED_FEATURES, > + .size_in = sizeof(mbox_in), > + .payload_in = &mbox_in, > + .size_out = sizeof(mbox_out), > + .payload_out = &mbox_out, > + .min_out = sizeof(mbox_out), > + }; > + rc = cxl_internal_send_cmd(cxl_mbox, &mbox_cmd); > + if (rc < 0) > + return rc; > + > + cxl_mbox->num_features = le16_to_cpu(mbox_out.supported_feats); > + if (!cxl_mbox->num_features) > + return -ENOENT; > + > + return 0; > +} > + > +int cxl_get_supported_features(struct cxl_memdev_state *mds) > +{ > + int remain_feats, max_size, max_feats, start, rc; > + struct cxl_dev_state *cxlds = &mds->cxlds; > + struct cxl_mailbox *cxl_mbox = &cxlds->cxl_mbox; > + int feat_size = sizeof(struct cxl_feat_entry); > + struct cxl_mbox_get_sup_feats_out *mbox_out; > + struct cxl_mbox_get_sup_feats_in mbox_in; > + int hdr_size = sizeof(*mbox_out); > + struct cxl_mbox_cmd mbox_cmd; > + struct cxl_mem_command *cmd; > + void *ptr; > + > + /* Get supported features is optional, need to check */ > + cmd = cxl_mem_find_command(CXL_MBOX_OP_GET_SUPPORTED_FEATURES); > + if (!cmd) > + return -EOPNOTSUPP; > + if (!test_bit(cmd->info.id, mds->enabled_cmds)) > + return -EOPNOTSUPP; > + > + rc = cxl_get_supported_features_count(cxlds); > + if (rc) > + return rc; > + > + struct cxl_feat_entry *entries __free(kvfree) = > + kvmalloc(cxl_mbox->num_features * feat_size, GFP_KERNEL); > + > + if (!entries) > + return -ENOMEM; > + > + cxl_mbox->entries = no_free_ptr(entries); > + rc = devm_add_action_or_reset(cxl_mbox->host, cxl_free_features, > + cxl_mbox->entries); > + if (rc) > + return rc; > + > + max_size = cxl_mbox->payload_size - hdr_size; > + /* max feat entries that can fit in mailbox max payload size */ > + max_feats = max_size / feat_size; > + ptr = &cxl_mbox->entries[0]; > + > + mbox_out = kvmalloc(cxl_mbox->payload_size, GFP_KERNEL); > + if (!mbox_out) > + return -ENOMEM; > + > + start = 0; > + remain_feats = cxl_mbox->num_features; > + do { > + int retrieved, alloc_size, copy_feats; + int num_entries; > + > + if (remain_feats > max_feats) { > + alloc_size = sizeof(*mbox_out) + max_feats * feat_size; > + remain_feats = remain_feats - max_feats; > + copy_feats = max_feats; > + } else { > + alloc_size = sizeof(*mbox_out) + remain_feats * feat_size; > + copy_feats = remain_feats; > + remain_feats = 0; > + } > + > + memset(&mbox_in, 0, sizeof(mbox_in)); > + mbox_in.count = alloc_size; > + mbox_in.start_idx = start; mbox_in.count = cpu_to_le32(alloc_size); mbox_in.start_idx = cpu_to_le16(start); > + memset(mbox_out, 0, alloc_size); > + mbox_cmd = (struct cxl_mbox_cmd) { > + .opcode = CXL_MBOX_OP_GET_SUPPORTED_FEATURES, > + .size_in = sizeof(mbox_in), > + .payload_in = &mbox_in, > + .size_out = alloc_size, > + .payload_out = mbox_out, > + .min_out = hdr_size, > + }; > + rc = cxl_internal_send_cmd(cxl_mbox, &mbox_cmd); > + if (rc < 0) > + goto err; > + if (mbox_cmd.size_out <= hdr_size) { > + rc = -ENXIO; > + goto err; > + } > + > + /* > + * Make sure retrieved out buffer is multiple of feature > + * entries. > + */ > + retrieved = mbox_cmd.size_out - hdr_size; > + if (retrieved % feat_size) { > + rc = -ENXIO; > + goto err; > + } > + + num_entries = le16_to_cpu(mbox_out->num_entries); > + /* > + * If the reported output entries * defined entry size != > + * retrieved output bytes, then the output package is incorrect. > + */ > + if (mbox_out->num_entries * feat_size != retrieved) { if (num_entries * feat_size != retrieved) { > + rc = -ENXIO; > + goto err; > + } > + > + memcpy(ptr, mbox_out->ents, retrieved); > + ptr += retrieved; > + /* > + * If the number of output entries is less than expected, add the > + * remaining entries to the next batch. > + */ > + remain_feats += copy_feats - mbox_out->num_entries; > + start += mbox_out->num_entries; remain_feat += copy_feats - num_entries; start += num_entries; > + } while (remain_feats); > + > + kfree(mbox_out); > + return 0; > + > +err: > + kfree(mbox_out); > + cxl_mbox->num_features = 0; > + return rc; > +} > +EXPORT_SYMBOL_NS_GPL(cxl_get_supported_features, CXL); > + > +int cxl_get_supported_feature_entry(struct cxl_memdev_state *mds, const uuid_t *feat_uuid, > + struct cxl_feat_entry *feat_entry_out) > +{ > + struct cxl_dev_state *cxlds = &mds->cxlds; > + struct cxl_feat_entry *feat_entry; > + int count; > + > + /* Check CXL dev supports the feature */ > + feat_entry = &cxlds->cxl_mbox.entries[0]; > + for (count = 0; count < cxlds->cxl_mbox.num_features; count++, feat_entry++) { > + if (uuid_equal(&feat_entry->uuid, feat_uuid)) { > + memcpy(feat_entry_out, feat_entry, sizeof(*feat_entry_out)); > + return 0; > + } > + } > + > + return -EOPNOTSUPP; > +} > +EXPORT_SYMBOL_NS_GPL(cxl_get_supported_feature_entry, CXL); > + > /** > * cxl_enumerate_cmds() - Enumerate commands for a device. > * @mds: The driver data for the operation > diff --git a/drivers/cxl/cxlmem.h b/drivers/cxl/cxlmem.h > index 2a25d1957ddb..f88b10188632 100644 > --- a/drivers/cxl/cxlmem.h > +++ b/drivers/cxl/cxlmem.h > @@ -530,6 +530,7 @@ enum cxl_opcode { > CXL_MBOX_OP_GET_LOG_CAPS = 0x0402, > CXL_MBOX_OP_CLEAR_LOG = 0x0403, > CXL_MBOX_OP_GET_SUP_LOG_SUBLIST = 0x0405, > + CXL_MBOX_OP_GET_SUPPORTED_FEATURES = 0x0500, > CXL_MBOX_OP_IDENTIFY = 0x4000, > CXL_MBOX_OP_GET_PARTITION_INFO = 0x4100, > CXL_MBOX_OP_SET_PARTITION_INFO = 0x4101, > @@ -813,6 +814,48 @@ enum { > CXL_PMEM_SEC_PASS_USER, > }; > > +/* Get Supported Features (0x500h) CXL r3.1 8.2.9.6.1 */ > +struct cxl_mbox_get_sup_feats_in { > + __le32 count; > + __le16 start_idx; > + u8 reserved[2]; > +} __packed; > + > +/* Supported Feature Entry : Payload out attribute flags */ > +#define CXL_FEAT_ENTRY_FLAG_CHANGABLE BIT(0) > +#define CXL_FEAT_ENTRY_FLAG_DEEPEST_RESET_PERSISTENCE_MASK GENMASK(3, 1) > +#define CXL_FEAT_ENTRY_FLAG_PERSIST_ACROSS_FIRMWARE_UPDATE BIT(4) > +#define CXL_FEAT_ENTRY_FLAG_SUPPORT_DEFAULT_SELECTION BIT(5) > +#define CXL_FEAT_ENTRY_FLAG_SUPPORT_SAVED_SELECTION BIT(6) > + > +enum cxl_feat_attr_value_persistence { > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_NONE, > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_CXL_RESET, > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_HOT_RESET, > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_WARM_RESET, > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_COLD_RESET, > + CXL_FEAT_ATTR_VALUE_PERSISTENCE_MAX > +}; > + > +struct cxl_feat_entry { > + uuid_t uuid; > + __le16 id; > + __le16 get_feat_size; > + __le16 set_feat_size; > + __le32 attr_flags; > + u8 get_feat_ver; > + u8 set_feat_ver; > + __le16 set_effects; > + u8 reserved[18]; > +} __packed; > + > +struct cxl_mbox_get_sup_feats_out { > + __le16 num_entries; > + __le16 supported_feats; > + u8 reserved[4]; > + struct cxl_feat_entry ents[] __counted_by_le(supported_feats); > +} __packed; > + > int cxl_internal_send_cmd(struct cxl_mailbox *cxl_mbox, > struct cxl_mbox_cmd *cmd); > int cxl_dev_state_identify(struct cxl_memdev_state *mds); > @@ -872,4 +915,8 @@ struct cxl_hdm { > struct seq_file; > struct dentry *cxl_debugfs_create_dir(const char *dir); > void cxl_dpa_debug(struct seq_file *file, struct cxl_dev_state *cxlds); > + > +int cxl_get_supported_features(struct cxl_memdev_state *mds); > +int cxl_get_supported_feature_entry(struct cxl_memdev_state *mds, const uuid_t *feat_uuid, > + struct cxl_feat_entry *feat_entry_out); > #endif /* __CXL_MEM_H__ */ > diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c > index 188412d45e0d..5c2926eec3c3 100644 > --- a/drivers/cxl/pci.c > +++ b/drivers/cxl/pci.c > @@ -887,6 +887,10 @@ static int cxl_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) > if (rc) > return rc; > > + rc = cxl_get_supported_features(mds); > + if (rc) > + dev_dbg(&pdev->dev, "No features enumerated.\n"); > + > rc = cxl_set_timestamp(mds); > if (rc) > return rc; > diff --git a/include/cxl/mailbox.h b/include/cxl/mailbox.h > index bacd111e75f1..cc66afec3473 100644 > --- a/include/cxl/mailbox.h > +++ b/include/cxl/mailbox.h > @@ -14,6 +14,8 @@ struct cxl_mbox_cmd; > * @mbox_mutex: mutex protects device mailbox and firmware > * @mbox_wait: rcuwait for mailbox > * @mbox_send: @dev specific transport for transmitting mailbox commands > + * @num_features: number of supported features > + * @entries: list of supported feature entries. > */ > struct cxl_mailbox { > struct device *host; > @@ -21,6 +23,8 @@ struct cxl_mailbox { > struct mutex mbox_mutex; /* lock to protect mailbox context */ > struct rcuwait mbox_wait; > int (*mbox_send)(struct cxl_mailbox *cxl_mbox, struct cxl_mbox_cmd *cmd); > + int num_features; > + struct cxl_feat_entry *entries; > }; > > int cxl_mailbox_init(struct cxl_mailbox *cxl_mbox, struct device *host); > diff --git a/include/uapi/linux/cxl_mem.h b/include/uapi/linux/cxl_mem.h > index c6c0fe27495d..bd2535962f70 100644 > --- a/include/uapi/linux/cxl_mem.h > +++ b/include/uapi/linux/cxl_mem.h > @@ -50,6 +50,7 @@ > ___C(GET_LOG_CAPS, "Get Log Capabilities"), \ > ___C(CLEAR_LOG, "Clear Log"), \ > ___C(GET_SUP_LOG_SUBLIST, "Get Supported Logs Sub-List"), \ > + ___C(GET_SUPPORTED_FEATURES, "Get Supported Features"), \ > ___C(MAX, "invalid / last command") > > #define ___C(a, b) CXL_MEM_COMMAND_ID_##a