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 3574BCEACD5 for ; Tue, 1 Oct 2024 15:47:23 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id BD103680021; Tue, 1 Oct 2024 11:47:22 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id B5A6B280068; Tue, 1 Oct 2024 11:47:22 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9AC2F680021; Tue, 1 Oct 2024 11:47:22 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 74A13280068 for ; Tue, 1 Oct 2024 11:47:22 -0400 (EDT) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 199C980A09 for ; Tue, 1 Oct 2024 15:47:22 +0000 (UTC) X-FDA: 82625462724.06.6C087A1 Received: from mail-pl1-f178.google.com (mail-pl1-f178.google.com [209.85.214.178]) by imf10.hostedemail.com (Postfix) with ESMTP id 0F451C0015 for ; Tue, 1 Oct 2024 15:47:19 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=Pp9vvlUk; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf10.hostedemail.com: domain of nifan.cxl@gmail.com designates 209.85.214.178 as permitted sender) smtp.mailfrom=nifan.cxl@gmail.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1727797536; a=rsa-sha256; cv=none; b=QlsFyFI0e3T/rMhfkNPBNeMPM4U3S6WMj26ygnrz/aEtrPNt6TPvNdlDCuWpYRAs3il4wl CP5NYXv6lc758Kv5FVbUVMBYufYe4ONy5NITN6SntXIvKesXKWcSzBfNsG9jH69M6VV0ry oN4LdIll7RHTV+ZQ/cSFyrzePmlNDXU= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=Pp9vvlUk; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf10.hostedemail.com: domain of nifan.cxl@gmail.com designates 209.85.214.178 as permitted sender) smtp.mailfrom=nifan.cxl@gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1727797536; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=YdCLMn5ZpQ+7rFvDfl2k8rSsKmnsDj7+j9oBZJ1kwpA=; b=aDkkodTd9DAx6rgRbMcQekBM2sLBb0ZYia/U9pSMNHEwpS5zHaGSyDq2UzWQGhsqYUolpf 3MCV00mLf/HAWHBbKItnkccAqMNWj4oZL9HXTPCw3I1GJrTO2fTDXeoYSladNe/cN0bbGM F5uVs6VGp/fLKloYQBIApdyiaPT5E10= Received: by mail-pl1-f178.google.com with SMTP id d9443c01a7336-20b6458ee37so38206095ad.1 for ; Tue, 01 Oct 2024 08:47:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1727797638; x=1728402438; darn=kvack.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:from:to:cc:subject:date:message-id:reply-to; bh=YdCLMn5ZpQ+7rFvDfl2k8rSsKmnsDj7+j9oBZJ1kwpA=; b=Pp9vvlUkW3ZfB925FoF6Sz5bNWhE7X9aS/EUw1aEdvT0tYZEKpW4KJHx5OYO448e7B 13eUfc0IsaDT6xjT2osTJCofPWjyT2t4Oq8YaubutWWOR4/DnqdR/LkrD3NN0OvOVyBL mv4H2j/q/NKhbnudwrIwYOEnw0EMO+do3sX/UYKEvOHn7RoQyXTGqjU7oWEKSOB8VXZz fzfMiGAk1eElrQ0HRCYpfBud8Xb+2OE/osxt+V3UKRgZH6qVfTvG0VsBXtB9U/sUgtQ0 bhaWdaRC5kpCthMfSnNgg2r9qdwGtovpacd1oiOs4CHviEQo8aY7v7tS/UpTH/e1R4w5 5q+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727797639; x=1728402439; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=YdCLMn5ZpQ+7rFvDfl2k8rSsKmnsDj7+j9oBZJ1kwpA=; b=uVQMJn5xRMArZz+HPHifGJT0DWaVwq7i4FoDiVRH+odv8DkVOsundFzhVslQBZeOgi Ce0jaPdKxMVjK7CH3Tu6TvJxxPvGb6g6fdKt4L7cTqRy83AxIdDWC5f4lgozhnEvgVgQ Py3bGsofnUvhNy1JL5uafRCD/+GDdTcGP2ywZ2jOmONumYF2BjEOwZo0+iVS5Ddu2XqD sM+ioszm2Bt+SzQSSRCzb28IxVNh78Zn/w70u760bdGqr8FkfGeRjqC14R81+npTMcdK RfRQH/q3b67crNyTt5SNzFVIBLORetPyeoVczXgojIJ5yhLizHmM7fNecblg8YtHiNSP CJpg== X-Forwarded-Encrypted: i=1; AJvYcCXEDZ9za0eLOTvV9SVMx/ShB44iiQoz2jxwUNIKpFs706F+dvWu9XcdVgkJO+lNKlhDORL6vY4e1Q==@kvack.org X-Gm-Message-State: AOJu0YykF4RCXY0DpRy6mZipHYDVGgkkYDbsKDTF3mATytRCFhpAJXgP ygS3f46SlnLVYg6RFTBgQFBg3GT2g5zz9XPBlTqSuNK0/f70U5sl X-Google-Smtp-Source: AGHT+IEMbtm052TafSeVHJR05rc0f2Z5POx5ZKXoEIgIOO5suOxCV5fraMKul8fONcGiPBM+C8SmjA== X-Received: by 2002:a17:902:fb8e:b0:20b:b479:94ca with SMTP id d9443c01a7336-20bc59f080emr1053455ad.23.1727797638346; Tue, 01 Oct 2024 08:47:18 -0700 (PDT) Received: from fan ([2601:646:8f03:9fee:7dd:d82b:9686:b02a]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-20b37da22basm71127265ad.100.2024.10.01.08.47.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 01 Oct 2024 08:47:17 -0700 (PDT) From: Fan Ni X-Google-Original-From: Fan Ni Date: Tue, 1 Oct 2024 08:47:07 -0700 To: shiju.jose@huawei.com Cc: linux-edac@vger.kernel.org, linux-cxl@vger.kernel.org, linux-acpi@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, 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, dave.jiang@intel.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 Subject: Re: [PATCH v12 13/17] ACPI:RAS2: Add ACPI RAS2 driver Message-ID: References: <20240911090447.751-1-shiju.jose@huawei.com> <20240911090447.751-14-shiju.jose@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240911090447.751-14-shiju.jose@huawei.com> X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 0F451C0015 X-Stat-Signature: bjcc4u8xtzni77mi66pryaj749utm7qy X-Rspam-User: X-HE-Tag: 1727797639-496081 X-HE-Meta: U2FsdGVkX1/2UXYCxrWEhwd/XfNMBcLsQCSu3STL/RNyV0rCoCEIgEpARNAy/YLEZT57crzVlAf/qya6iIpZNXlmb8ajEV7JmEPznpVgMWg+EpQu1KRFTgEjlLyg52fyfbDyMkVC773tm9i7nl7GNK+ezTFlQWYiJ4YESzWpNbJSKsWsGVWrwJOvlGf6ZstzQZsGP4corGgyyFfD97XSq1F+HqtzQui6s41Y8iPurqNia0sI/LGrh27FVgod7IO+Yl5KfY6OPKH/uJeQqNVoTaNrZkFeXYk+PFbl3F1E2HE9c5I6jfPSiDB3X+RqmwRpGtWML4IXR5TgQELhpzs42Jc98/UCj/dR84w0UuZOoW4sTRQmjhV4cxdWZYrtt/Lwnq9c1NsyVG+xLc+xAXPgDW9EMWkKQKoqOdMVzmaBYy5+3wD0GojNppQTtVRTiHaismGLwMu/svkCZPtT4q2s7GFT3SlLH24FFxti5OfuGMPWztnsYo4zeDarh+k9aBai9kAEJTxVU4Yj8bEQdwgNhbVeQPPIeWEpujJoTZ22PMLWNaxhtp3jGAMHS9PYZHB3kNH6xU5/gTTz9st4VFEiT9hmuFnalTOPX1DzytPmPpoJPFnKZMQxnH3suHdAdHj2czgDuqT545X49KYREqiE4c81A2gztrc19RbYZxcQ6dEh6CKj5wzAVW3W2oAsur+NKeRCZ7LyYmM52mPa7mBLL7CqX09q+ac6Q6ppqAfF2YUj2fJrIdL5OSmsO9SMLwbWJrswM39aGp4nysS650xhOZaFJFF1VXt8AEi5GuyMuH/aFzKHFAfamVUX51yswb6uPIHjQbzeK3dKbg2eKkLt70BcVarXw9baSMdn0CbcFvlUt3hIe5PFRkVPRMymnN2GSF3XpveLDvPNz7Lo8WpY81CCJoTswK1FLUC+/l6pxkA/C+V/jzm2TxbyLDSH3lpf5Iz4xyVJTdbRd2Wpf+5 4pYmlqVc k2HCa3lK0aKR00S6LUICTiNKD2gzP9Czj6Iv/gm89GhTXL8jCv0EwMgz6rsRrtoBsJBpKvECmieF+u1VNBM2jPavx4/eenWAL0+ZY/qhdBke3PSXIArvs0XUn8rPpm87wvZUO4A/2OGxW4ovXQauWSy+e2lfPQ58JCXueoc1nof0p7oPufRUnGTfZhFME89T76jRRmy0jAPOC01FYYmvq0+vC9clIIFz/4GMJmUB6M8V052r1tsNLFfwK83I3ptSCNnhppDvvwLtn8w6pQ7Oxwhpf1a5t17zXmJw00P3fs0CdEXu/fEV03ATWo75ydPpN0+oHrLQZazrY3zvsw1VhxHYc5E+lc3BDDd41dPv7cQsEHj5V8Gx36Qgg4XanyTM9ObxaYFarFzUTHDVrpOiEQjVIoMou6nlgrvR8ll4pYjFuFdZ7fhRzpU8LizIWNhd0CzXnpTPyLjfkoGlgS+3m8gygK3s86ptqNNQhqh3IGKzoJkZXk4WZ+lBOuFOiowAcsiXF+XYzKNud/24Awk61AdFgVUc4fZqvK0nU8Z0QbvOGFM/0dHREZdHhcNopc+Yh4hoeroLK/Gr2Y8380Tor5lv0yA== 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 Wed, Sep 11, 2024 at 10:04:42AM +0100, shiju.jose@huawei.com wrote: > From: Shiju Jose > > Add support for ACPI RAS2 feature table (RAS2) defined in the > ACPI 6.5 Specification, section 5.2.21. > Driver contains RAS2 Init, which extracts the RAS2 table and driver > adds platform device for each memory features which binds to the s/features/feature/ Fan > RAS2 memory driver. > > Driver uses PCC mailbox to communicate with the ACPI HW and the > driver adds OSPM interfaces to send RAS2 commands. > > Co-developed-by: A Somasundaram > Signed-off-by: A Somasundaram > Co-developed-by: Jonathan Cameron > Signed-off-by: Jonathan Cameron > Signed-off-by: Shiju Jose > --- > drivers/acpi/Kconfig | 10 + > drivers/acpi/Makefile | 1 + > drivers/acpi/ras2.c | 391 +++++++++++++++++++++++++++++++++++++++ > include/acpi/ras2_acpi.h | 60 ++++++ > 4 files changed, 462 insertions(+) > create mode 100755 drivers/acpi/ras2.c > create mode 100644 include/acpi/ras2_acpi.h > > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > index e3a7c2aedd5f..482080f1f0c5 100644 > --- a/drivers/acpi/Kconfig > +++ b/drivers/acpi/Kconfig > @@ -284,6 +284,16 @@ config ACPI_CPPC_LIB > If your platform does not support CPPC in firmware, > leave this option disabled. > > +config ACPI_RAS2 > + bool "ACPI RAS2 driver" > + select MAILBOX > + select PCC > + help > + The driver adds support for ACPI RAS2 feature table(extracts RAS2 > + table from OS system table) and OSPM interfaces to send RAS2 > + commands via PCC mailbox subspace. Driver adds platform device for > + the RAS2 memory features which binds to the RAS2 memory driver. > + > config ACPI_PROCESSOR > tristate "Processor" > depends on X86 || ARM64 || LOONGARCH || RISCV > diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile > index 61ca4afe83dc..84e2a2519bae 100644 > --- a/drivers/acpi/Makefile > +++ b/drivers/acpi/Makefile > @@ -100,6 +100,7 @@ obj-$(CONFIG_ACPI_EC_DEBUGFS) += ec_sys.o > obj-$(CONFIG_ACPI_BGRT) += bgrt.o > obj-$(CONFIG_ACPI_CPPC_LIB) += cppc_acpi.o > obj-$(CONFIG_ACPI_SPCR_TABLE) += spcr.o > +obj-$(CONFIG_ACPI_RAS2) += ras2.o > obj-$(CONFIG_ACPI_DEBUGGER_USER) += acpi_dbg.o > obj-$(CONFIG_ACPI_PPTT) += pptt.o > obj-$(CONFIG_ACPI_PFRUT) += pfr_update.o pfr_telemetry.o > diff --git a/drivers/acpi/ras2.c b/drivers/acpi/ras2.c > new file mode 100755 > index 000000000000..5daf1510d19e > --- /dev/null > +++ b/drivers/acpi/ras2.c > @@ -0,0 +1,391 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Implementation of ACPI RAS2 driver. > + * > + * Copyright (c) 2024 HiSilicon Limited. > + * > + * Support for RAS2 - ACPI 6.5 Specification, section 5.2.21 > + * > + * Driver contains ACPI RAS2 init, which extracts the ACPI RAS2 table and > + * get the PCC channel subspace for communicating with the ACPI compliant > + * HW platform which supports ACPI RAS2. Driver adds platform devices > + * for each RAS2 memory feature which binds to the memory ACPI RAS2 driver. > + */ > + > +#define pr_fmt(fmt) "ACPI RAS2: " fmt > + > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* > + * Arbitrary Retries for PCC commands because the > + * remote processor could be much slower to reply. > + */ > +#define RAS2_NUM_RETRIES 600 > + > +#define RAS2_FEATURE_TYPE_MEMORY 0x00 > + > +/* global variables for the RAS2 PCC subspaces */ > +static DEFINE_MUTEX(ras2_pcc_subspace_lock); > +static LIST_HEAD(ras2_pcc_subspaces); > + > +static int ras2_report_cap_error(u32 cap_status) > +{ > + switch (cap_status) { > + case ACPI_RAS2_NOT_VALID: > + case ACPI_RAS2_NOT_SUPPORTED: > + return -EPERM; > + case ACPI_RAS2_BUSY: > + return -EBUSY; > + case ACPI_RAS2_FAILED: > + case ACPI_RAS2_ABORTED: > + case ACPI_RAS2_INVALID_DATA: > + return -EINVAL; > + default: /* 0 or other, Success */ > + return 0; > + } > +} > + > +static int ras2_check_pcc_chan(struct ras2_pcc_subspace *pcc_subspace) > +{ > + struct acpi_ras2_shared_memory __iomem *generic_comm_base = pcc_subspace->pcc_comm_addr; > + ktime_t next_deadline = ktime_add(ktime_get(), pcc_subspace->deadline); > + u32 cap_status; > + u16 status; > + u32 ret; > + > + while (!ktime_after(ktime_get(), next_deadline)) { > + /* > + * As per ACPI spec, the PCC space will be initialized by > + * platform and should have set the command completion bit when > + * PCC can be used by OSPM > + */ > + status = readw_relaxed(&generic_comm_base->status); > + if (status & RAS2_PCC_CMD_ERROR) { > + cap_status = readw_relaxed(&generic_comm_base->set_capabilities_status); > + ret = ras2_report_cap_error(cap_status); > + > + status &= ~RAS2_PCC_CMD_ERROR; > + writew_relaxed(status, &generic_comm_base->status); > + return ret; > + } > + if (status & RAS2_PCC_CMD_COMPLETE) > + return 0; > + /* > + * Reducing the bus traffic in case this loop takes longer than > + * a few retries. > + */ > + msleep(10); > + } > + > + return -EIO; > +} > + > +/** > + * ras2_send_pcc_cmd() - Send RAS2 command via PCC channel > + * @ras2_ctx: pointer to the RAS2 context structure > + * @cmd: command to send > + * > + * Returns: 0 on success, an error otherwise > + */ > +int ras2_send_pcc_cmd(struct ras2_scrub_ctx *ras2_ctx, u16 cmd) > +{ > + struct ras2_pcc_subspace *pcc_subspace = ras2_ctx->pcc_subspace; > + struct acpi_ras2_shared_memory *generic_comm_base = pcc_subspace->pcc_comm_addr; > + static ktime_t last_cmd_cmpl_time, last_mpar_reset; > + struct mbox_chan *pcc_channel; > + unsigned int time_delta; > + static int mpar_count; > + int ret; > + > + guard(mutex)(&ras2_pcc_subspace_lock); > + ret = ras2_check_pcc_chan(pcc_subspace); > + if (ret < 0) > + return ret; > + pcc_channel = pcc_subspace->pcc_chan->mchan; > + > + /* > + * Handle the Minimum Request Turnaround Time(MRTT) > + * "The minimum amount of time that OSPM must wait after the completion > + * of a command before issuing the next command, in microseconds" > + */ > + if (pcc_subspace->pcc_mrtt) { > + time_delta = ktime_us_delta(ktime_get(), last_cmd_cmpl_time); > + if (pcc_subspace->pcc_mrtt > time_delta) > + udelay(pcc_subspace->pcc_mrtt - time_delta); > + } > + > + /* > + * Handle the non-zero Maximum Periodic Access Rate(MPAR) > + * "The maximum number of periodic requests that the subspace channel can > + * support, reported in commands per minute. 0 indicates no limitation." > + * > + * This parameter should be ideally zero or large enough so that it can > + * handle maximum number of requests that all the cores in the system can > + * collectively generate. If it is not, we will follow the spec and just > + * not send the request to the platform after hitting the MPAR limit in > + * any 60s window > + */ > + if (pcc_subspace->pcc_mpar) { > + if (mpar_count == 0) { > + time_delta = ktime_ms_delta(ktime_get(), last_mpar_reset); > + if (time_delta < 60 * MSEC_PER_SEC) { > + dev_dbg(ras2_ctx->dev, > + "PCC cmd not sent due to MPAR limit"); > + return -EIO; > + } > + last_mpar_reset = ktime_get(); > + mpar_count = pcc_subspace->pcc_mpar; > + } > + mpar_count--; > + } > + > + /* Write to the shared comm region. */ > + writew_relaxed(cmd, &generic_comm_base->command); > + > + /* Flip CMD COMPLETE bit */ > + writew_relaxed(0, &generic_comm_base->status); > + > + /* Ring doorbell */ > + ret = mbox_send_message(pcc_channel, &cmd); > + if (ret < 0) { > + dev_err(ras2_ctx->dev, > + "Err sending PCC mbox message. cmd:%d, ret:%d\n", > + cmd, ret); > + return ret; > + } > + > + /* > + * If Minimum Request Turnaround Time is non-zero, we need > + * to record the completion time of both READ and WRITE > + * command for proper handling of MRTT, so we need to check > + * for pcc_mrtt in addition to CMD_READ > + */ > + if (cmd == RAS2_PCC_CMD_EXEC || pcc_subspace->pcc_mrtt) { > + ret = ras2_check_pcc_chan(pcc_subspace); > + if (pcc_subspace->pcc_mrtt) > + last_cmd_cmpl_time = ktime_get(); > + } > + > + if (pcc_channel->mbox->txdone_irq) > + mbox_chan_txdone(pcc_channel, ret); > + else > + mbox_client_txdone(pcc_channel, ret); > + > + return ret >= 0 ? 0 : ret; > +} > +EXPORT_SYMBOL_GPL(ras2_send_pcc_cmd); > + > +static int ras2_register_pcc_channel(struct device *dev, struct ras2_scrub_ctx *ras2_ctx, > + int pcc_subspace_id) > +{ > + struct acpi_pcct_hw_reduced *ras2_ss; > + struct mbox_client *ras2_mbox_cl; > + struct pcc_mbox_chan *pcc_chan; > + struct ras2_pcc_subspace *pcc_subspace; > + > + if (pcc_subspace_id < 0) > + return -EINVAL; > + > + mutex_lock(&ras2_pcc_subspace_lock); > + list_for_each_entry(pcc_subspace, &ras2_pcc_subspaces, elem) { > + if (pcc_subspace->pcc_subspace_id == pcc_subspace_id) { > + ras2_ctx->pcc_subspace = pcc_subspace; > + pcc_subspace->ref_count++; > + mutex_unlock(&ras2_pcc_subspace_lock); > + return 0; > + } > + } > + mutex_unlock(&ras2_pcc_subspace_lock); > + > + pcc_subspace = kcalloc(1, sizeof(*pcc_subspace), GFP_KERNEL); > + if (!pcc_subspace) > + return -ENOMEM; > + pcc_subspace->pcc_subspace_id = pcc_subspace_id; > + ras2_mbox_cl = &pcc_subspace->mbox_client; > + ras2_mbox_cl->dev = dev; > + ras2_mbox_cl->knows_txdone = true; > + > + pcc_chan = pcc_mbox_request_channel(ras2_mbox_cl, pcc_subspace_id); > + if (IS_ERR(pcc_chan)) { > + kfree(pcc_subspace); > + return PTR_ERR(pcc_chan); > + } > + pcc_subspace->pcc_chan = pcc_chan; > + ras2_ss = pcc_chan->mchan->con_priv; > + pcc_subspace->comm_base_addr = ras2_ss->base_address; > + > + /* > + * ras2_ss->latency is just a Nominal value. In reality > + * the remote processor could be much slower to reply. > + * So add an arbitrary amount of wait on top of Nominal. > + */ > + pcc_subspace->deadline = ns_to_ktime(RAS2_NUM_RETRIES * ras2_ss->latency * > + NSEC_PER_USEC); > + pcc_subspace->pcc_mrtt = ras2_ss->min_turnaround_time; > + pcc_subspace->pcc_mpar = ras2_ss->max_access_rate; > + pcc_subspace->pcc_comm_addr = acpi_os_ioremap(pcc_subspace->comm_base_addr, > + ras2_ss->length); > + /* Set flag so that we dont come here for each CPU. */ > + pcc_subspace->pcc_channel_acquired = true; > + > + mutex_lock(&ras2_pcc_subspace_lock); > + list_add(&pcc_subspace->elem, &ras2_pcc_subspaces); > + pcc_subspace->ref_count++; > + mutex_unlock(&ras2_pcc_subspace_lock); > + ras2_ctx->pcc_subspace = pcc_subspace; > + > + return 0; > +} > + > +static void ras2_unregister_pcc_channel(void *ctx) > +{ > + struct ras2_scrub_ctx *ras2_ctx = ctx; > + struct ras2_pcc_subspace *pcc_subspace = ras2_ctx->pcc_subspace; > + > + if (!pcc_subspace || !pcc_subspace->pcc_chan) > + return; > + > + guard(mutex)(&ras2_pcc_subspace_lock); > + if (pcc_subspace->ref_count > 0) > + pcc_subspace->ref_count--; > + if (!pcc_subspace->ref_count) { > + list_del(&pcc_subspace->elem); > + pcc_mbox_free_channel(pcc_subspace->pcc_chan); > + kfree(pcc_subspace); > + } > +} > + > +/** > + * devm_ras2_register_pcc_channel() - Register RAS2 PCC channel > + * @dev: pointer to the RAS2 device > + * @ras2_ctx: pointer to the RAS2 context structure > + * @pcc_subspace_id: identifier of the RAS2 PCC channel. > + * > + * Returns: 0 on success, an error otherwise > + */ > +int devm_ras2_register_pcc_channel(struct device *dev, struct ras2_scrub_ctx *ras2_ctx, > + int pcc_subspace_id) > +{ > + int ret; > + > + ret = ras2_register_pcc_channel(dev, ras2_ctx, pcc_subspace_id); > + if (ret) > + return ret; > + > + return devm_add_action_or_reset(dev, ras2_unregister_pcc_channel, ras2_ctx); > +} > +EXPORT_SYMBOL_NS_GPL(devm_ras2_register_pcc_channel, ACPI_RAS2); > + > +static struct platform_device *ras2_add_platform_device(char *name, int channel) > +{ > + int ret; > + struct platform_device *pdev __free(platform_device_put) = > + platform_device_alloc(name, PLATFORM_DEVID_AUTO); > + if (!pdev) > + return ERR_PTR(-ENOMEM); > + > + ret = platform_device_add_data(pdev, &channel, sizeof(channel)); > + if (ret) > + return ERR_PTR(ret); > + > + ret = platform_device_add(pdev); > + if (ret) > + return ERR_PTR(ret); > + > + return_ptr(pdev); > +} > + > +static int __init ras2_acpi_init(void) > +{ > + struct acpi_table_header *pAcpiTable = NULL; > + struct acpi_ras2_pcc_desc *pcc_desc_list; > + struct acpi_table_ras2 *pRas2Table; > + struct platform_device *pdev; > + int pcc_subspace_id; > + acpi_size ras2_size; > + acpi_status status; > + u8 count = 0, i; > + int ret; > + > + status = acpi_get_table("RAS2", 0, &pAcpiTable); > + if (ACPI_FAILURE(status) || !pAcpiTable) { > + pr_err("ACPI RAS2 driver failed to initialize, get table failed\n"); > + return -EINVAL; > + } > + > + ras2_size = pAcpiTable->length; > + if (ras2_size < sizeof(struct acpi_table_ras2)) { > + pr_err("ACPI RAS2 table present but broken (too short #1)\n"); > + ret = -EINVAL; > + goto free_ras2_table; > + } > + > + pRas2Table = (struct acpi_table_ras2 *)pAcpiTable; > + if (pRas2Table->num_pcc_descs <= 0) { > + pr_err("ACPI RAS2 table does not contain PCC descriptors\n"); > + ret = -EINVAL; > + goto free_ras2_table; > + } > + > + struct platform_device **pdev_list __free(kfree) = > + kcalloc(pRas2Table->num_pcc_descs, sizeof(*pdev_list), > + GFP_KERNEL); > + if (!pdev_list) { > + ret = -ENOMEM; > + goto free_ras2_table; > + } > + > + pcc_desc_list = (struct acpi_ras2_pcc_desc *)(pRas2Table + 1); > + /* Double scan for the case of only one actual controller */ > + pcc_subspace_id = -1; > + count = 0; > + for (i = 0; i < pRas2Table->num_pcc_descs; i++, pcc_desc_list++) { > + if (pcc_desc_list->feature_type != RAS2_FEATURE_TYPE_MEMORY) > + continue; > + if (pcc_subspace_id == -1) { > + pcc_subspace_id = pcc_desc_list->channel_id; > + count++; > + } > + if (pcc_desc_list->channel_id != pcc_subspace_id) > + count++; > + } > + if (count == 1) { > + pdev = ras2_add_platform_device("acpi_ras2", pcc_subspace_id); > + if (!pdev) { > + ret = -ENODEV; > + goto free_ras2_pdev; > + } > + pdev_list[0] = pdev; > + return 0; > + } > + > + count = 0; > + for (i = 0; i < pRas2Table->num_pcc_descs; i++, pcc_desc_list++) { > + if (pcc_desc_list->feature_type != RAS2_FEATURE_TYPE_MEMORY) > + continue; > + pcc_subspace_id = pcc_desc_list->channel_id; > + /* Add the platform device and bind ACPI RAS2 memory driver */ > + pdev = ras2_add_platform_device("acpi_ras2", pcc_subspace_id); > + if (!pdev) > + goto free_ras2_pdev; > + pdev_list[count++] = pdev; > + } > + > + acpi_put_table(pAcpiTable); > + return 0; > + > +free_ras2_pdev: > + for (i = count; i >= 0; i++) > + platform_device_put(pdev_list[i]); > + > +free_ras2_table: > + acpi_put_table(pAcpiTable); > + > + return ret; > +} > +late_initcall(ras2_acpi_init) > diff --git a/include/acpi/ras2_acpi.h b/include/acpi/ras2_acpi.h > new file mode 100644 > index 000000000000..edfca253d88a > --- /dev/null > +++ b/include/acpi/ras2_acpi.h > @@ -0,0 +1,60 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +/* > + * RAS2 ACPI driver header file > + * > + * (C) Copyright 2014, 2015 Hewlett-Packard Enterprises > + * > + * Copyright (c) 2024 HiSilicon Limited > + */ > + > +#ifndef _RAS2_ACPI_H > +#define _RAS2_ACPI_H > + > +#include > +#include > +#include > +#include > + > +#define RAS2_PCC_CMD_COMPLETE BIT(0) > +#define RAS2_PCC_CMD_ERROR BIT(2) > + > +/* RAS2 specific PCC commands */ > +#define RAS2_PCC_CMD_EXEC 0x01 > + > +struct device; > + > +/* Data structures for PCC communication and RAS2 table */ > +struct pcc_mbox_chan; > + > +struct ras2_pcc_subspace { > + int pcc_subspace_id; > + struct mbox_client mbox_client; > + struct pcc_mbox_chan *pcc_chan; > + struct acpi_ras2_shared_memory __iomem *pcc_comm_addr; > + u64 comm_base_addr; > + bool pcc_channel_acquired; > + ktime_t deadline; > + unsigned int pcc_mpar; > + unsigned int pcc_mrtt; > + struct list_head elem; > + u16 ref_count; > +}; > + > +struct ras2_scrub_ctx { > + struct device *dev; > + struct ras2_pcc_subspace *pcc_subspace; > + int id; > + u8 instance; > + struct device *scrub_dev; > + bool bg; > + u64 base, size; > + u8 scrub_cycle_hrs, min_scrub_cycle, max_scrub_cycle; > + /* Lock to provide mutually exclusive access to PCC channel */ > + struct mutex lock; > +}; > + > +int ras2_send_pcc_cmd(struct ras2_scrub_ctx *ras2_ctx, u16 cmd); > +int devm_ras2_register_pcc_channel(struct device *dev, struct ras2_scrub_ctx *ras2_ctx, > + int pcc_subspace_id); > + > +#endif /* _RAS2_ACPI_H */ > -- > 2.34.1 > -- Fan Ni