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 0A07BC636D3 for ; Mon, 6 Feb 2023 15:46:39 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 80A4D6B0073; Mon, 6 Feb 2023 10:46:38 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 7BA2F6B0074; Mon, 6 Feb 2023 10:46:38 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 681AE6B0075; Mon, 6 Feb 2023 10:46:38 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 594EA6B0073 for ; Mon, 6 Feb 2023 10:46:38 -0500 (EST) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 24EB540982 for ; Mon, 6 Feb 2023 15:46:38 +0000 (UTC) X-FDA: 80437294476.13.CC743FF Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by imf25.hostedemail.com (Postfix) with ESMTP id 54702A001F for ; Mon, 6 Feb 2023 15:46:35 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf25.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1675698396; 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; bh=dd/hTh+LhRTXXAkcJ2qoiu1td4JpmyiwsA/OZoX+lik=; b=k8UhWyOuwtUOjbCMwrCrNaacDdXnhc+TpkRq80GyBL7O5CZHPf+sttPEpZFeZBhPi2t6gG Z57W6Ida/kwOL4ZIz7XenG9IT5Gn3W/vNz33wEtXVwUkVMa+ZyPJYp5+tknQ6v7YsDcQF6 TCE/3EGNS5y1TNbmObAWtc+fx/nqr64= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf25.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1675698396; a=rsa-sha256; cv=none; b=xmsXoKZ28obGaE44jZ8vP2pXGzJPm7xZ50ytPXipNRVVhgds5AvPC9oX3sommNFiefR/K0 Kwt2sohJ97CbjNTXQ7QZHdeyOzpASvqOz8ypjLTdYBCn+KqOmbDCdGadFXFmhmGXZsnWqY EKDdAUBPHZE8Uz+R1iFbIItea8rnCr8= Received: from lhrpeml500005.china.huawei.com (unknown [172.18.147.206]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4P9Vqy5t0Vz6J7gF; Mon, 6 Feb 2023 23:42:06 +0800 (CST) Received: from localhost (10.81.207.58) by lhrpeml500005.china.huawei.com (7.191.163.240) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.17; Mon, 6 Feb 2023 15:46:29 +0000 Date: Mon, 6 Feb 2023 15:46:27 +0000 From: Jonathan Cameron To: Dan Williams CC: , , , Subject: Re: [PATCH 02/18] cxl/region: Add a mode attribute for regions Message-ID: <20230206154627.00000a1b@Huawei.com> In-Reply-To: <167564536041.847146.11330354943211409793.stgit@dwillia2-xfh.jf.intel.com> References: <167564534874.847146.5222419648551436750.stgit@dwillia2-xfh.jf.intel.com> <167564536041.847146.11330354943211409793.stgit@dwillia2-xfh.jf.intel.com> Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 4.1.0 (GTK 3.24.33; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.81.207.58] X-ClientProxiedBy: lhrpeml100001.china.huawei.com (7.191.160.183) To lhrpeml500005.china.huawei.com (7.191.163.240) X-CFilter-Loop: Reflected X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 54702A001F X-Stat-Signature: u6eetps9fsj1rr7kos4bttfyayij9bxn X-HE-Tag: 1675698395-600405 X-HE-Meta: U2FsdGVkX18W07KbpWm7KYQU6P8lURATsbRLMMai/YNHvwVzLu930314w+WxJhiT19mnZAIvB+DsZMAWV6oXgrg3XdAj4yuXOCr4/F315dUVHDLIbJbbuf25YC2ny65nVbu/D9N6v/uU6vazqu9M2VNOTNeAV0ipay+X3Rw+3djL9dkrMk7LaBGKav1rFb8k4EOU8lXNg/h5VQZin2s2QyWfGoFkQJD3oryFDQy1V2kHNSBLtLN9NQmCPzSeGG94To512hC3xgLP50E8Eu5K+86ARk0zEuBh7mH5zDuDRuKDRAx/LytlFPYwgQelWvWiOOGZsUY7F0R4slY4Q2Fm0uhEEAEznfGcQzUAUqok7rKqSfTpZeFHMnHhvMwJnrELirXyMxVHYWOKYeFsfgwHsCOrVaEjCqVqUQlgsLHGHTcGcCOm8cyCw5AhQFrOW1ckVsgFo+aoL3bT/PZjPAZs1XLa6+V5Yc3GNh2XsiNXyng5tr9aivfmu1jAmtDSvFbcyFrMKetp+SD4c22r57AQ0tUfqT1Yd3oHbSD7F2qRKhnJts2mNpRS7na+gQybQEp+TTXfly76hrGr8LV4Ge29lG5aU2dzG5/l9UfZhlRgePMYukI5oqbAGRZB47mCkJWrKbfXSwjrMQbNrWnMgUvI8apgYpDEPoBoHsSNYQQQrY8zA1WkfQrNPOA+f1IC7h/wRQ/P/zfR1MNT7PiQfcIEs60BJAQSJzPTF3P64im07KH+4NQurpASwVrv6GDFer7w4EGPq3QqigPi/m82FxnQKR/kRnGLzHh5CrJvADDiZgVSo3oqKSz5cFTbooiQ1SA89CXu1NLnD98X3qqddT7Vx0jjgRxkf1Ser+LZoAkNaQVibC9bfYvRDQZ68QDhJgfXHEyeP1z3EvlkvM4ESBpuax0OhO/yJAKmwNa4AAb+AcKCbKXqwtITcLugareSXsa67G9zrySsYZdWOEWKCfu qIgsYuO5 tazTrY8mFQ6G46UMTUsLqbA5cXATmLyNNXeTj+luixyx2FYE0puqNbOYfPcl5ohmo2DEl0f9ZcFt+Gg3/pviAAFch0Lto7UEbQ6u5QsrqmS2s9TgIfjgFRmDaFV3OaIUt28PtHN9y7AUFGj3r9y948nGGmZSU8XUTGTyNX7JbQFLDvJ5JCE/crpwCz6l/LzgqakQyRvgHwLfA0JGNedHWp6rH/fD49neSvNlNUflmoyeM0H4JtSXrso0R6h1v/TTCs2O/iKtDUH4bPjkJKy/evFnnpS/SBp7UcDOkK9s1A/q38Bj1suFaDhhCfR0EzDguxwBiMHyM3/jHi7SpKFrtTN0HQ97/E1PV0my8 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: On Sun, 05 Feb 2023 17:02:40 -0800 Dan Williams wrote: > In preparation for a new region type, "ram" regions, add a mode > attribute to clarify the mode of the decoders that can be added to a > region. Share the internals of mode_show() (for decoders) with the > region case. > > Signed-off-by: Dan Williams I guess the DEAD decoder cases is an odd enough path that it is fine to not worry that it is presented as mixed (none might be more appropriate as you can't put any memory behind it). Other than that this is straight forwards exposure of internal data that already existed... Reviewed-by: Jonathan Cameron > --- > Documentation/ABI/testing/sysfs-bus-cxl | 11 +++++++++++ > drivers/cxl/core/port.c | 12 +----------- > drivers/cxl/core/region.c | 10 ++++++++++ > drivers/cxl/cxl.h | 14 ++++++++++++++ > 4 files changed, 36 insertions(+), 11 deletions(-) > > diff --git a/Documentation/ABI/testing/sysfs-bus-cxl b/Documentation/ABI/testing/sysfs-bus-cxl > index 5be032313e29..058b0c45001f 100644 > --- a/Documentation/ABI/testing/sysfs-bus-cxl > +++ b/Documentation/ABI/testing/sysfs-bus-cxl > @@ -358,6 +358,17 @@ Description: > results in the same address being allocated. > > > +What: /sys/bus/cxl/devices/regionZ/mode > +Date: January, 2023 > +KernelVersion: v6.3 > +Contact: linux-cxl@vger.kernel.org > +Description: > + (RO) The mode of a region is established at region creation time > + and dictates the mode of the endpoint decoder that comprise the > + region. For more details on the possible modes see > + /sys/bus/cxl/devices/decoderX.Y/mode > + > + > What: /sys/bus/cxl/devices/regionZ/resource > Date: May, 2022 > KernelVersion: v6.0 > diff --git a/drivers/cxl/core/port.c b/drivers/cxl/core/port.c > index 410c036c09fa..8566451cb22f 100644 > --- a/drivers/cxl/core/port.c > +++ b/drivers/cxl/core/port.c > @@ -180,17 +180,7 @@ static ssize_t mode_show(struct device *dev, struct device_attribute *attr, > { > struct cxl_endpoint_decoder *cxled = to_cxl_endpoint_decoder(dev); > > - switch (cxled->mode) { > - case CXL_DECODER_RAM: > - return sysfs_emit(buf, "ram\n"); > - case CXL_DECODER_PMEM: > - return sysfs_emit(buf, "pmem\n"); > - case CXL_DECODER_NONE: > - return sysfs_emit(buf, "none\n"); > - case CXL_DECODER_MIXED: > - default: > - return sysfs_emit(buf, "mixed\n"); > - } > + return sysfs_emit(buf, "%s\n", cxl_decoder_mode_name(cxled->mode)); > } > > static ssize_t mode_store(struct device *dev, struct device_attribute *attr, > diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c > index 60828d01972a..17d2d0c12725 100644 > --- a/drivers/cxl/core/region.c > +++ b/drivers/cxl/core/region.c > @@ -458,6 +458,15 @@ static ssize_t resource_show(struct device *dev, struct device_attribute *attr, > } > static DEVICE_ATTR_RO(resource); > > +static ssize_t mode_show(struct device *dev, struct device_attribute *attr, > + char *buf) > +{ > + struct cxl_region *cxlr = to_cxl_region(dev); > + > + return sysfs_emit(buf, "%s\n", cxl_decoder_mode_name(cxlr->mode)); > +} > +static DEVICE_ATTR_RO(mode); > + > static int alloc_hpa(struct cxl_region *cxlr, resource_size_t size) > { > struct cxl_root_decoder *cxlrd = to_cxl_root_decoder(cxlr->dev.parent); > @@ -585,6 +594,7 @@ static struct attribute *cxl_region_attrs[] = { > &dev_attr_interleave_granularity.attr, > &dev_attr_resource.attr, > &dev_attr_size.attr, > + &dev_attr_mode.attr, > NULL, > }; > > diff --git a/drivers/cxl/cxl.h b/drivers/cxl/cxl.h > index aa3af3bb73b2..ca76879af1de 100644 > --- a/drivers/cxl/cxl.h > +++ b/drivers/cxl/cxl.h > @@ -320,6 +320,20 @@ enum cxl_decoder_mode { > CXL_DECODER_DEAD, > }; > > +static inline const char *cxl_decoder_mode_name(enum cxl_decoder_mode mode) > +{ > + static const char * const names[] = { > + [CXL_DECODER_NONE] = "none", > + [CXL_DECODER_RAM] = "ram", > + [CXL_DECODER_PMEM] = "pmem", > + [CXL_DECODER_MIXED] = "mixed", > + }; > + > + if (mode >= CXL_DECODER_NONE && mode <= CXL_DECODER_MIXED) > + return names[mode]; > + return "mixed"; > +} > + > /** > * struct cxl_endpoint_decoder - Endpoint / SPA to DPA decoder > * @cxld: base cxl_decoder_object >