From patchwork Thu Sep 23 14:57:49 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Cristian Marussi X-Patchwork-Id: 12513139 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3F40CC433F5 for ; Thu, 23 Sep 2021 15:01:56 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 0A3C560EB4 for ; Thu, 23 Sep 2021 15:01:56 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 0A3C560EB4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:List-Subscribe:List-Help: List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:Subject:Cc:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=Mcl8sRz+a+EbjiPPiWlMkeoHa1nTiwRoOtINO0g0E6c=; b=YGQeel9bBwKfSN pEph7e5KOXSlfZfqq4a0uZmLkDkbVna9OZIlMfUZ8NPAcBub8N0/XLwbHDF/5qeYF/L+usoJyn8hq WlQ8lG4JRBpZUdf/CO47elNUHqdac8fW2xceJlSwnh3z6F6+ZBuN/Edr4jbckTIwCJSDdhdOCVJPj GlIuS8OmQWJc+l0cuVJFEgt0BjtMHEm4c8b6Ny8a4kqzw9WpJShBBQab1D2NtLD5FBhjyg1ToDG3I 6IxrSlNwTUwwwZTWzlnIxj4oPBnbO+USvFYw9Yzn7R7q14+WtCOsmzGCV/gF1U10iLri7JZZknhyT fn2VGCPZq86IrA/TZqrA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTQBR-00Bw16-6g; Thu, 23 Sep 2021 14:58:45 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTQBE-00Bvx6-6w for linux-arm-kernel@lists.infradead.org; Thu, 23 Sep 2021 14:58:34 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C5259D6E; Thu, 23 Sep 2021 07:58:26 -0700 (PDT) Received: from e120937-lin.home (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DB23A3F718; Thu, 23 Sep 2021 07:58:24 -0700 (PDT) From: Cristian Marussi To: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: sudeep.holla@arm.com, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, peter.hilber@opensynergy.com, igor.skalkin@opensynergy.com, cristian.marussi@arm.com Subject: [PATCH v5 0/13] Introduce atomic support for SCMI transports Date: Thu, 23 Sep 2021 15:57:49 +0100 Message-Id: <20210923145802.50938-1-cristian.marussi@arm.com> X-Mailer: git-send-email 2.17.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210923_075832_403726_39A1DD54 X-CRM114-Status: GOOD ( 20.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi all, This series mainly aims to introduce atomic support for transports that can support it. At first in [02/13], as a closely related addition, it is introduced a common way for a transport to signal to the SCMI core that it does not offer completion interrupts, so that the usual polling behaviour based on .poll_done() will be required: this can be done enabling statically a global polling behaviour for the whole transport with flag scmi_desc.force_polling OR dynamically enabling at runtime such polling behaviour on a per-channel basis with scmi_chan_info.no_completion_irq, typically during .chan_setup(). The usual per-command polling selection behaviour based on hdr.poll_completion is preserved as before. In [03/13] a Kconfig is added to enable at build-time force_polling on SCMI Mailbox transport only. (VirtIO does not support polling, while it does not make sense to support global force_polling on SMC given the presence of sync_cmds_atomic_replies flag) Then in [04/13], a transport that supports atomic operations on its TX path can now declare itself as .atomic_capable and, if the platform has been also configured to use such atomic operation mode, the SCMI core will refrain itself too from sleeping on the correspondent RX-path even when using completion IRQs. In [07/13] a simple method is introduced so that an SCMI driver can easily query the core to check if the currently used transport is configured to behave in an atomic manner: in this way, interested SCMI driver users, like Clock framework [08/13], can optionally support atomic operations when operating on an atomically configured transport. In [09/13] virtio transport is declared atomic and a new Kconfig is added to be able to choose at build time if that specific platform should use the atomic operation mode or not; such new option is default N, so VirtIO will continue to operate in non-atomic mode unless otherwise specified. Finally there are 4 patches related to SMC transport. At first [10/13] ports SMC to use the common core completions when completion interrupt is available or otherwise revert to use common core polling mechanism above introduced: this avoids the improper call of scmi_rx_callback directly in smc_send_message. Then in [11/13] SMC is converted to be .atomic_capable by substituting the mutexes with busy-waiting to keep the channel 'locked' ONLY IF the SMC transport is configured to operate in atomic mode. (CONFIG_ARM_SCMI_TRANSPORT_SMC_ATOMIC_ENABLE=y) With [12/13] I introduce a flag to allow a transport to signal to the core that upon return of a .send_message() the requested command execution can be assumed by the core to have been fully completed by the platform, so that the response payload (if any) can be immediately fetched without the need to poll the channel. Finally in [13/13] I enable the above flag for SMC transport. Atomic support has been minimally tested against the virtio transport, while polling has been tested with mailbox transports. The series is based on sudeep/for-next/scmi[1] on top of: commit 1cd73200dad2 ("firmware: arm_scmi: Remove __exit annotation") Given I'm still gathering feedback on this, I still not have CCed any maintainer out of SCMI subsystem. Any feedback welcome. Thanks, Cristian [1]: https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scmi --- V4 --> V5 - removed RFCs tags - added scmi_desc.atomic_enabled flags and a few Kconfig options to set atomic mode for SMC and VirtIO transports. Default disabled. - added Kconfig option to enable forced polling as a whole on the Mailbox transport - removed .poll_done callback from SMC transport since no real polling is needed once sync_cmds_atomic_replies is set - made atomic_capable changes on SMC transport dependent on Kconfig CONFIG_ARM_SCMI_TRANSPORT_SMC_ATOMIC_ENABLE: so no change and no busy waiting if atomic mode is NOT enabled in Kconfig. - made const: force_polling/atomic_capable/atomic_enabled/sync_cmds_atomic_replies V3 --> V4 - rebased on linux-next/master next-20210824 - renamed .needs_polling to .no_completion_irq - added .sync_cmds_atomic_replies - make SMC use .sync_cmd_atomic_replies V2 --> v3 - rebased on SCMI VirtIO V6 which in turn is based on v5.14-rc1 Cristian Marussi (13): firmware: arm_scmi: Perform earlier cinfo lookup call in do_xfer firmware: arm_scmi: Add configurable polling mode for transports firmware: arm_scmi: Add forced polling support to mailbox transport firmware: arm_scmi: Add support for atomic transports include: trace: Add new scmi_xfer_response_wait event firmware: arm_scmi: Use new trace event scmi_xfer_response_wait firmware: arm_scmi: Add is_transport_atomic() handle method clk: scmi: Support atomic enable/disable API firmware: arm_scmi: Add atomic mode support to virtio transport firmware: arm_scmi: Make smc transport use common completions firmware: arm_scmi: Add atomic mode support to smc transport firmware: arm_scmi: Add sync_cmds_atomic_replies transport flag firmware: arm_scmi: Make smc support atomic sync commands replies drivers/clk/clk-scmi.c | 44 ++++- drivers/firmware/arm_scmi/Kconfig | 38 +++++ drivers/firmware/arm_scmi/common.h | 25 +++ drivers/firmware/arm_scmi/driver.c | 251 +++++++++++++++++++++++----- drivers/firmware/arm_scmi/mailbox.c | 1 + drivers/firmware/arm_scmi/smc.c | 114 ++++++++++--- drivers/firmware/arm_scmi/virtio.c | 2 + include/linux/scmi_protocol.h | 8 + include/trace/events/scmi.h | 28 ++++ 9 files changed, 432 insertions(+), 79 deletions(-)