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 15330C3DA4A for ; Tue, 20 Aug 2024 22:46:10 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 801ED6B0085; Tue, 20 Aug 2024 18:46:10 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 7D8B46B0088; Tue, 20 Aug 2024 18:46:10 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6C7276B0089; Tue, 20 Aug 2024 18:46:10 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 4C4216B0085 for ; Tue, 20 Aug 2024 18:46:10 -0400 (EDT) Received: from smtpin14.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id EAB41A7D32 for ; Tue, 20 Aug 2024 22:46:09 +0000 (UTC) X-FDA: 82474108458.14.FF85875 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.10]) by imf17.hostedemail.com (Postfix) with ESMTP id 9D35B40003 for ; Tue, 20 Aug 2024 22:46:06 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=fd3ZPHtD; spf=pass (imf17.hostedemail.com: domain of dave.jiang@intel.com designates 198.175.65.10 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=1724193888; 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=PvKN1jEHOWoRiiBC+QhJvIh/zjUIOvLJvkSkhOA+QrQ=; b=kGbMX7Mj7GL9zCY0NwaaeyPf48kHR+iivnhvth86uJV0PDP5LusoOpRqdhVHUyZSYpHGo4 IOb7on0gZJe/RcSd5pUAQ0/ATsCWpIsjKjxPQmm5a4N1mSeVkgWiCkiD5z3YC8cLEFHOuC 50MCM33h/QrlIuGujM2Ih+ecis/x+p0= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1724193888; a=rsa-sha256; cv=none; b=HZdZnpBdPtJ9Fg9zeeuJopwXVK+idMDeYA0BFbpkp7QEBCeqNTMNv3G6hvhSS4B4SrIfDm U6v/BepZqddNERjNzEUp55UBBnTMRu34oehlMTeOPey5OTlZDPqwvYrfvFpWme1X+GvE6P 5TT7x88sFJgttZAdFlz80MO1M40ZodY= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=fd3ZPHtD; spf=pass (imf17.hostedemail.com: domain of dave.jiang@intel.com designates 198.175.65.10 as permitted sender) smtp.mailfrom=dave.jiang@intel.com; dmarc=pass (policy=none) header.from=intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1724193965; x=1755729965; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=F3PA9yHyrbjgiUFut2vhgib1ni2rF84nJs2tKUq25NM=; b=fd3ZPHtDSSji+bUul0/e5G/bqh//owYtAAjYlnX9TOpfhk7FEDrcWYIb kLC04nzRFIfckZ6fEUWsrBLv0tgNBANqqCJ5UvA4as/CaH4YkuVYCMI66 ZfpZxkMwzQioC2CJIFlpK7Tj72/XpdDtJxCf849NHTa2STaEeTEdrM6Ir XW8tMDrtr4xFE9mI+mMG4VKWbqQ6n40YyFYUrnsQQfHwao3Dd3UnX8dmy 9tvYx0qaU3oQSm0LSoiLWhTo8sTunVX7b0hC8jrD3c241HCRhL6iyf3YW vr7O2wO1mtaBqsO66wb3kUQD/nKCzZ3TLFRhZNwGr9Pv2awGHlhImyKbt w==; X-CSE-ConnectionGUID: +vJvxI/0T0CJ80LC4eNfDA== X-CSE-MsgGUID: VDfR4Q4tQ36g5wXUVWAb4A== X-IronPort-AV: E=McAfee;i="6700,10204,11170"; a="39984490" X-IronPort-AV: E=Sophos;i="6.10,163,1719903600"; d="scan'208";a="39984490" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by orvoesa102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Aug 2024 15:46:03 -0700 X-CSE-ConnectionGUID: GQ3XMDjfQcC8bIXChJYsxg== X-CSE-MsgGUID: Oy9LSQUsQmiUWU5Tpk9pAQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.10,163,1719903600"; d="scan'208";a="65576969" Received: from cdpresto-mobl2.amr.corp.intel.com.amr.corp.intel.com (HELO [10.125.108.88]) ([10.125.108.88]) by fmviesa004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Aug 2024 15:46:01 -0700 Message-ID: <8b78d617-26f6-45a2-87d7-d3b66f5e7277@intel.com> Date: Tue, 20 Aug 2024 15:46:00 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v11 04/14] cxl/mbox: Add GET_SUPPORTED_FEATURES mailbox command 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, 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, mike.malvestuto@intel.com, gthelen@google.com, wschwartz@amperecomputing.com, dferguson@amperecomputing.com, wbs@os.amperecomputing.com, nifan.cxl@gmail.com, jgroves@micron.com, vsalve@micron.com, tanxiaofei@huawei.com, prime.zeng@hisilicon.com, roberto.sassu@huawei.com, kangkang.shen@futurewei.com, wanghuiqiang@huawei.com, linuxarm@huawei.com References: <20240816164238.1902-1-shiju.jose@huawei.com> <20240816164238.1902-5-shiju.jose@huawei.com> Content-Language: en-US From: Dave Jiang In-Reply-To: <20240816164238.1902-5-shiju.jose@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 9D35B40003 X-Stat-Signature: 1ssk91aioabi8ctdp7q9e5ki5u8prmto X-HE-Tag: 1724193966-145133 X-HE-Meta: U2FsdGVkX1/9B3oAohxDXaQgRLoDZklQBUmGNRiBYnfScuSOHMn3gboCM8k75XZrfboyibAaXlhq28UnaIqapN4WMJDyHYs+r5pSifw3oLBwhOmRje8yYfZE3L+k21P/B1iqtcWUMfAJeeFjvMSK3IRc6lxN27pkakqPPWlKdBzG+BVnTbwC4oxJLsF+lwHGOuypdTk+HQaYimHvDMFoxQffWvc7B9c8VkM+BmxmUNfdU1JooLG2ehfvmKX7SqMUBUSoDoiPfAfYJ0IWTAOJuQvRB01zD5EYUixFkDsVxb3nY3jIR6F+upmInKBMq23Y82K2V2WScffIhbx6X3MXdVNhfW1HwqncH+bwebFFa2Z0HI1toEuce5cAGguuAGpvGExGmowrfiAEWpvlCAPZYPG9AUEDZlLy9oFGB3kQG5t9FNhociqgRzFooi9rnMHlVJA2yEFDoT4VrfnT/XQbnaoY/Qv5yXvylA0QQkC4yTKjo2r14//YtlMj1LS/RMa+zHY0nhSMmMwkXVIzmETgnUftLjY8uQNDS2ncCGTTC81jh79V313vqHYdyzKDI5mKPFAaSHk0iWcPHsb4XsWZdNeDeKXVIDa0LYu7v6y8XiVgmxRehHBH3X+/4YHKMu43UYhyrSWRM88GXqxDQHFVPoASYtnxORP+L9SyTyhsipkBMxb0JDt89/ILVsqyi2Z6QnwYpxDsKs95/O5NfBuFK5anbYrWQk7TJznM59whihIGdpVaZ6eoKBESjAzQFw0rdfOOX8lUodebRLs8ok3Lofo7QqW2AKNBKxWqHtzCOw6/8XSDwsob/knrrgJwvOe1JMA86UIf48l7dGUnRriivrV64dez9CjCNxdegAQj0a996wYSx4GsxCQglSRGAlIZ6ceI+AWces5noCNLbX1gi+yBI1d05cGnCPN8t41Q152N+SEqC8uafp/CTXdtd1W4RRN21Lsn8Ze3GN1hQCl g5B5Ti7C 3UrvwDpR8nFbsCRlWhddv3bptVFGUkLIxg1MamRa0P+DFauIXVRpTWOoVJMhwh5alEfPvHNnhXLztcCWLMpc9zLs0su3SZK1ocOHGixBlXobyVyvizRe7TKrbNMer3r7JzHjlB8Md6G2tEdoOqzqp34HrW3w4buCxCMTPfTDlPZX8SicR6ZN/TtB0HCrcLmXQYBr/4d/mMfN5XoAU+5arNkN/spnJQteB6rgokW1rAcDhBuUwjcqDg3NvNeewpkcaDOB2m4lMidCTEPrxqZY+TMSPOLMzwYeCtbI6woK5dvPb8w5t5EcnzaACotunw/2AQbTa 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 8/16/24 9:42 AM, shiju.jose@huawei.com wrote: > From: Shiju Jose > > Add support for GET_SUPPORTED_FEATURES mailbox command. > > CXL spec 3.1 section 8.2.9.6 describes optional device specific features. > CXL devices support features with changeable attributes. > > CXL spec 3.1 section 8.2.9.6.1 describes Get Supported features command. > Get Supported Features retrieves the list of supported device specific > features. The settings of a feature can be retrieved using Get Feature > and optionally modified using Set Feature. > > Signed-off-by: Shiju Jose > --- > drivers/cxl/core/mbox.c | 68 +++++++++++++++++++++++++++++++++++++++++ > drivers/cxl/cxlmem.h | 63 ++++++++++++++++++++++++++++++++++++++ > 2 files changed, 131 insertions(+) > > diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c > index 2626f3fff201..760fa3e1075f 100644 > --- a/drivers/cxl/core/mbox.c > +++ b/drivers/cxl/core/mbox.c > @@ -1324,6 +1324,74 @@ int cxl_set_timestamp(struct cxl_memdev_state *mds) > } > EXPORT_SYMBOL_NS_GPL(cxl_set_timestamp, CXL); > > +int cxl_get_supported_features(struct cxl_memdev_state *mds, > + u32 count, u16 start_index, > + struct cxl_mbox_get_supp_feats_out *feats_out) > +{ > + struct cxl_mbox_get_supp_feats_in pi; > + struct cxl_mbox_cmd mbox_cmd; > + int rc; > + > + pi.count = cpu_to_le32(count); > + pi.start_index = cpu_to_le16(start_index); > + > + mbox_cmd = (struct cxl_mbox_cmd) { > + .opcode = CXL_MBOX_OP_GET_SUPPORTED_FEATURES, > + .size_in = sizeof(pi), > + .payload_in = &pi, > + .size_out = count, > + .payload_out = feats_out, > + .min_out = sizeof(*feats_out), > + }; > + rc = cxl_internal_send_cmd(mds, &mbox_cmd); > + if (rc < 0) > + return rc; > + > + return 0; > +} > +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_mbox_supp_feat_entry *feat_entry_out) Hi Shiju, thoughts on storing all the supported features meta info from GET_SUPPORTED_FEATURES command in the cxl core driver during device enumeration and the CXL EDAC code can just query the core instead of reading from the device? Just looking forward to supporting other read/set features operations in the future and avoiding going through this entire read and find routine for each feature. DJ > +{ > + struct cxl_mbox_supp_feat_entry *feat_entry; > + int feat_index, feats_out_size; > + int nentries, count; > + int ret; > + > + feat_index = 0; > + feats_out_size = sizeof(struct cxl_mbox_get_supp_feats_out) + > + sizeof(struct cxl_mbox_supp_feat_entry); > + struct cxl_mbox_get_supp_feats_out *feats_out __free(kfree) = > + kmalloc(feats_out_size, GFP_KERNEL); > + if (!feats_out) > + return -ENOMEM; > + > + while (true) { > + memset(feats_out, 0, feats_out_size); > + ret = cxl_get_supported_features(mds, feats_out_size, > + feat_index, feats_out); > + if (ret) > + return ret; > + > + nentries = feats_out->nr_entries; > + if (!nentries) > + return -EOPNOTSUPP; > + > + /* Check CXL memdev supports the feature */ > + feat_entry = feats_out->feat_entries; > + for (count = 0; count < nentries; count++, feat_entry++) { > + if (uuid_equal(&feat_entry->uuid, feat_uuid)) { > + memcpy(feat_entry_out, feat_entry, > + sizeof(*feat_entry_out)); > + return 0; > + } > + } > + feat_index += nentries; > + } > +} > +EXPORT_SYMBOL_NS_GPL(cxl_get_supported_feature_entry, CXL); > + > int cxl_mem_get_poison(struct cxl_memdev *cxlmd, u64 offset, u64 len, > struct cxl_region *cxlr) > { > diff --git a/drivers/cxl/cxlmem.h b/drivers/cxl/cxlmem.h > index af8169ccdbc0..9939c771f642 100644 > --- a/drivers/cxl/cxlmem.h > +++ b/drivers/cxl/cxlmem.h > @@ -531,6 +531,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, > @@ -700,6 +701,63 @@ struct cxl_mbox_set_timestamp_in { > > } __packed; > > +/* > + * Get Supported Features CXL 3.1 Spec 8.2.9.6.1 > + */ > + > +/* > + * Get Supported Features input payload > + * CXL rev 3.1 section 8.2.9.6.1 Table 8-95 > + */ > +struct cxl_mbox_get_supp_feats_in { > + __le32 count; > + __le16 start_index; > + u8 rsvd[2]; > +} __packed; > + > +/* > + * Get Supported Features Supported Feature Entry > + * CXL rev 3.1 section 8.2.9.6.1 Table 8-97 > + */ > +/* 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_mbox_supp_feat_entry { > + uuid_t uuid; > + __le16 index; > + __le16 get_size; > + __le16 set_size; > + __le32 attr_flags; > + u8 get_version; > + u8 set_version; > + __le16 set_effects; > + u8 rsvd[18]; > +} __packed; > + > +/* > + * Get Supported Features output payload > + * CXL rev 3.1 section 8.2.9.6.1 Table 8-96 > + */ > +struct cxl_mbox_get_supp_feats_out { > + __le16 nr_entries; > + __le16 nr_supported; > + u8 rsvd[4]; > + struct cxl_mbox_supp_feat_entry feat_entries[]; > +} __packed; > + > /* Get Poison List CXL 3.0 Spec 8.2.9.8.4.1 */ > struct cxl_mbox_poison_in { > __le64 offset; > @@ -831,6 +889,11 @@ void cxl_event_trace_record(const struct cxl_memdev *cxlmd, > enum cxl_event_type event_type, > const uuid_t *uuid, union cxl_event *evt); > int cxl_set_timestamp(struct cxl_memdev_state *mds); > +int cxl_get_supported_features(struct cxl_memdev_state *mds, > + u32 count, u16 start_index, > + struct cxl_mbox_get_supp_feats_out *feats_out); > +int cxl_get_supported_feature_entry(struct cxl_memdev_state *mds, const uuid_t *feat_uuid, > + struct cxl_mbox_supp_feat_entry *feat_entry_out); > int cxl_poison_state_init(struct cxl_memdev_state *mds); > int cxl_mem_get_poison(struct cxl_memdev *cxlmd, u64 offset, u64 len, > struct cxl_region *cxlr);