From patchwork Tue Jul 18 23:30:40 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Brown X-Patchwork-Id: 122300 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c923:0:b0:3e4:2afc:c1 with SMTP id j3csp2084243vqt; Tue, 18 Jul 2023 16:49:49 -0700 (PDT) X-Google-Smtp-Source: APBJJlHtnyAjQN+0fgZ61QDwwsgk+pyFrTu8nUsip3rHpwLYWycRS+wZRM7vxp0ivr3c2aXpE7Qx X-Received: by 2002:a17:902:ce8f:b0:1b8:6cae:3570 with SMTP id f15-20020a170902ce8f00b001b86cae3570mr1330573plg.11.1689724188853; Tue, 18 Jul 2023 16:49:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689724188; cv=none; d=google.com; s=arc-20160816; b=rAm/IkHcjrkDBe8HvCyZu/ZCoU59g8oLcx7tXqc3LS9ns9jv4HD3gpVVlMJG90nIAc tzrO2O9RQC891E+ge4rqiV4zXDFYZ0GLusWfnofKVCblX0lEHgGMgrPMSEYxTQeO3mSe QVxYohIlgJJjoMx6lEGO8gvcMzDSt6EZx+L5nWJXUU7LUS+kZrWhuTKEmf+VWWt2tiCj pP8Xahesn2U/iLEDEI/auwsKbV+3+lAFNh3AEbnmclGV9+CwUMfrk9zUNhrp73LWBGif mMdbPCG9JqAOtwy7Is4HGFxTAClQACKKz2GEwOzntVdrCBPtvmP0WAppnxUiIZKqo8UI Srog== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:message-id:content-transfer-encoding :mime-version:subject:date:from:dkim-signature; bh=6IhW66+oxExfgnOTpGZ0bD6MeuTieIjab81MxGb6/QU=; fh=HSv56OuHqjp72jYTvnGZRWNMJuHQtBnV+GIs5C3gX/c=; b=hDtkk+N/N5rK1TsU0tGDPo3rRovDDrCsF9ZvZ0f1OkPJWcWR05cgYCCCFMtBugDWny qCR0B9ScPFgdeXQancskC9NtKO3+DsN+V/a9lJscCBkcA/Qj3Nm0XDMMjVIGJJkpSLdd JShUX94YaBCw4IKXv6woItu4vNOC0g01HL2vlNAFy1lMa+lVVM+rlA3mKjyj1pXTlu9D GXV9q4o1S1pu8/qqBsrglMCxUEpDY1ndoYEXz59PCK9nz+2GB6doXheNoRauE0kdzA/t +EKXSicWLSB+Ftj7OopLIS6AsIenlLtkiWd40yUlFXFnIi5tSao7sgHu621SrpvRaoAg XxAQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=qWcFdOs2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id lo5-20020a170903434500b001b670382e83si2225606plb.327.2023.07.18.16.49.34; Tue, 18 Jul 2023 16:49:48 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=qWcFdOs2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229989AbjGRXat (ORCPT + 99 others); Tue, 18 Jul 2023 19:30:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44052 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229480AbjGRXar (ORCPT ); Tue, 18 Jul 2023 19:30:47 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 39DD4EC for ; Tue, 18 Jul 2023 16:30:46 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id CBB5561519 for ; Tue, 18 Jul 2023 23:30:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7C230C433C7; Tue, 18 Jul 2023 23:30:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689723045; bh=q9GUBT/U895/mZRmm4QeBGzDtN92EtDp9NkceL0+W74=; h=From:Date:Subject:To:Cc:From; b=qWcFdOs23hRao8/SHNWaBQiebTmzsqCouudyLSypT5lUwCA+P5YaMSDu2HCP9yGKA XBOzlxNuBLnacjpIusYSZay7hABDXaCNZDg7T2sbyMl2/EaWkgLm2xAZ8xECYYUcPs ioR47n3S50W1DxPvoNg9zcg6zd6OpvHpmNUvGcRdstrbqQQufcIR75Jxwlc13xtJpf iF1TnN2dZjRfRxJa+P4hlKDcYxVtnMdibIUW1rqL07eQBI0yLAzFKLTohGRmnB7iN8 Bqb3OduiiNl7XpupnomNWUUuyrk8TxTucgOtDLW+FblA9gTkbP+aOYTsSuXln/CbZo 8rviutdZr2Jzw== From: Mark Brown Date: Wed, 19 Jul 2023 00:30:40 +0100 Subject: [PATCH] regcache: Push async I/O request down into the rbtree cache MIME-Version: 1.0 Message-Id: <20230719-regcache-async-rbtree-v1-1-b03d30cf1daf@kernel.org> X-B4-Tracking: v=1; b=H4sIAJ8gt2QC/x3MSwqDQBBF0a1IjVPQH1DJVsRBp3xqTdpQLRIR9 57G4Rnce1GBKQq9m4sMhxbdcoV/NSRrygtYp2oKLkTX+Z4NiyRZwamcWdg+uwE8iSDE6KLvWqr t1zDr7/kO433/Aa9Yt6xnAAAA To: Charles Keepax Cc: linux-kernel@vger.kernel.org, Mark Brown X-Mailer: b4 0.13-dev-099c9 X-Developer-Signature: v=1; a=openpgp-sha256; l=3005; i=broonie@kernel.org; h=from:subject:message-id; bh=q9GUBT/U895/mZRmm4QeBGzDtN92EtDp9NkceL0+W74=; b=owEBbQGS/pANAwAKASTWi3JdVIfQAcsmYgBktyCjVsce16IUnaWxanMU4iqhQ9VFPsZa1QdWghB4 c/S1QDyJATMEAAEKAB0WIQSt5miqZ1cYtZ/in+ok1otyXVSH0AUCZLcgowAKCRAk1otyXVSH0INCB/ 4lP0rYuM661Z7mFNGrowyB9BlXUiqEug2BvnU8mvicdVMkmYq0Cg/mocvnSis+kXfuglTEK3HrwHSs LTO0PyNcdDZE5/CEiBQzxzC93g9QPjcQrXhLAvVYut6zkwji+ElyUQJ5ennsc/D+W7Jd1r02tTGr2e G7wSsgATIE53D5RnDulpYHj+nk3BUTBXFPiLTkEOK6wBH+XqPY+X0yy2NWKwY7tX0BHEQ5XOAtnZtN Zj8F6u73Nll1ZwgbqP2EHRyriewf6Q51Dzolj/LMfHlW6LC/NZfbZdYaqv/2i4id9Fh6bK4NiOFmWO GyITukamBG0zcJ+l/BL8l43GZiGDl6 X-Developer-Key: i=broonie@kernel.org; a=openpgp; fpr=3F2568AAC26998F9E813A1C5C3F436CA30F5D8EB X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771804231003691682 X-GMAIL-MSGID: 1771804231003691682 Currently the regcache core unconditionally enables async I/O for all cache types, causing problems for the maple tree cache which dynamically allocates the buffers used to write registers to the device since async requires the buffers to be kept around until the I/O has been completed. This use of async I/O is mainly for the rbtree cache which stores data in a format directly usable for regmap_raw_write(), though there is a special case for single register writes which would also have allowed it to be used with the flat cache. It is a bit of a landmine for other caches since it implicitly converts sync operations to async, and with modern hardware it is not clear that async I/O is actually a performance win as shown by the performance work David Jander did with SPI. In multi core systems the cost of managing concurrency ends up swamping the performance benefit and almost all modern systems are multi core. Address this by pushing the enablement of async I/O down into the rbtree cache where it is actively used, avoiding surprises for other cache implementations. Reported-by: Charles Keepax Fixes: bfa0b38c1483 ("regmap: maple: Implement block sync for the maple tree cache") Signed-off-by: Mark Brown Reviewed-by: Charles Keepax Tested-by: Charles Keepax --- drivers/base/regmap/regcache-rbtree.c | 4 ++++ drivers/base/regmap/regcache.c | 3 --- 2 files changed, 4 insertions(+), 3 deletions(-) --- base-commit: fdf0eaf11452d72945af31804e2a1048ee1b574c change-id: 20230718-regcache-async-rbtree-dcce23303176 Best regards, diff --git a/drivers/base/regmap/regcache-rbtree.c b/drivers/base/regmap/regcache-rbtree.c index fabf87058d80..584bcc55f56e 100644 --- a/drivers/base/regmap/regcache-rbtree.c +++ b/drivers/base/regmap/regcache-rbtree.c @@ -471,6 +471,8 @@ static int regcache_rbtree_sync(struct regmap *map, unsigned int min, unsigned int start, end; int ret; + map->async = true; + rbtree_ctx = map->cache; for (node = rb_first(&rbtree_ctx->root); node; node = rb_next(node)) { rbnode = rb_entry(node, struct regcache_rbtree_node, node); @@ -499,6 +501,8 @@ static int regcache_rbtree_sync(struct regmap *map, unsigned int min, return ret; } + map->async = false; + return regmap_async_complete(map); } diff --git a/drivers/base/regmap/regcache.c b/drivers/base/regmap/regcache.c index 28bc3ae9458a..7d3e47436056 100644 --- a/drivers/base/regmap/regcache.c +++ b/drivers/base/regmap/regcache.c @@ -368,8 +368,6 @@ int regcache_sync(struct regmap *map) if (!map->cache_dirty) goto out; - map->async = true; - /* Apply any patch first */ map->cache_bypass = true; for (i = 0; i < map->patch_regs; i++) { @@ -392,7 +390,6 @@ int regcache_sync(struct regmap *map) out: /* Restore the bypass state */ - map->async = false; map->cache_bypass = bypass; map->no_sync_defaults = false; map->unlock(map->lock_arg);