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 X-Spam-Level: X-Spam-Status: No, score=-15.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 CC05EC2B9F4 for ; Tue, 22 Jun 2021 12:18:40 +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 9E4D6611CE for ; Tue, 22 Jun 2021 12:18:40 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9E4D6611CE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@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:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=f84uf1zQVhQfqiz1R3BhkoeNYZFXgYmO0SvFpXVno4A=; b=GPd3pF0JhVy1Qx Bw6hzYEEEvK4k1leuDdwAkVXChnzOWopZgb5bBmpm/0CzUaSfnhKZHXe4CF9sZDOtAg54kQHMNXUR eprJivBtZLdA6sEHp6I84YrOLqRffrKLxKqIBb+PIIJNtvmaS82esA6ndGGyfBdZBeO6q4+tevFez WHMBvTpF8teeLRRvVIqsCkofHteTbEOgzE2dQ3JLMjKYSLpQoQsG77VAlTchxXetmQkfuUDubPeOt CVlpjrI3ndtsORM5NoDKsd453R7rkldakPnhGMMgo/BrgBFoo1OT78tXte+F2ISbFpzs0iNrU/k9y gwoq9E+Lu1UlRNh3nLAA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lvfKb-00763r-4X; Tue, 22 Jun 2021 12:16:41 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lvfKW-00761v-U4 for linux-arm-kernel@lists.infradead.org; Tue, 22 Jun 2021 12:16:38 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 8105B60C41; Tue, 22 Jun 2021 12:16:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1624364196; bh=mNyb/gADrHsQToqWpHLZJITn6vzAF50b6ulNT3tHuRg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Zz/3delKwa2AtnTGh5TDwEROCk8xxj1lB762qpjwekZSo/UXEOPMzz5clP6/QCTCi JRQR8Hs6AdHN0W4/24bSGIjQ4EtjQDwte+vKOSVgYkN408R4H+SL50vk9qlwsC3kK/ PzU4/fc+V6IGIqpkikNYnDW0d8+tX663J6DqyD8kMCejYy5diLi2aKqldM2/5CTtAO CiPTVHzBk8TyDMPV8X6/UQx3GKEA63iPXB2EoNjxfiaEQWvcAY8EhvxH1roHTjeb8j Fys3Al/tnBF1HeX6LuSIJCcbM0i/oSNOBC04ASdfX7OR4h9x1O5d1wo0mIpoxBfVUx sd0xCo8GyO47Q== Date: Tue, 22 Jun 2021 13:16:31 +0100 From: Will Deacon To: Guangbin Huang Cc: davem@davemloft.net, kuba@kernel.org, catalin.marinas@arm.com, maz@kernel.org, mark.rutland@arm.com, dbrazdil@google.com, qperret@google.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, lipeng321@huawei.com, peterz@infradead.org Subject: Re: [PATCH net-next 1/3] arm64: barrier: add DGH macros to control memory accesses merging Message-ID: <20210622121630.GC30757@willie-the-truck> References: <1624360271-17525-1-git-send-email-huangguangbin2@huawei.com> <1624360271-17525-2-git-send-email-huangguangbin2@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1624360271-17525-2-git-send-email-huangguangbin2@huawei.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210622_051637_077957_67DAA6AC X-CRM114-Status: GOOD ( 20.47 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Jun 22, 2021 at 07:11:09PM +0800, Guangbin Huang wrote: > From: Xiongfeng Wang > > DGH prohibits merging memory accesses with Normal-NC or Device-GRE > attributes before the hint instruction with any memory accesses > appearing after the hint instruction. Provide macros to expose it to the > arch code. Hmm. The architecture states: | DGH is a hint instruction. A DGH instruction is not expected to be | performance optimal to merge memory accesses with Normal Non-cacheable | or Device-GRE attributes appearing in program order before the hint | instruction with any memory accesses appearing after the hint instruction | into a single memory transaction on an interconnect. which doesn't make a whole lot of sense to me, in all honesty. > Signed-off-by: Xiongfeng Wang > Signed-off-by: Cheng Jian > Signed-off-by: Yufeng Mo > --- > arch/arm64/include/asm/assembler.h | 7 +++++++ > arch/arm64/include/asm/barrier.h | 1 + > 2 files changed, 8 insertions(+) > > diff --git a/arch/arm64/include/asm/assembler.h b/arch/arm64/include/asm/assembler.h > index 8418c1bd8f04..d723899328bd 100644 > --- a/arch/arm64/include/asm/assembler.h > +++ b/arch/arm64/include/asm/assembler.h > @@ -90,6 +90,13 @@ > .endm > > /* > + * Data gathering hint > + */ > + .macro dgh > + hint #6 > + .endm > + > +/* > * RAS Error Synchronization barrier > */ > .macro esb > diff --git a/arch/arm64/include/asm/barrier.h b/arch/arm64/include/asm/barrier.h > index 451e11e5fd23..02e1735706d2 100644 > --- a/arch/arm64/include/asm/barrier.h > +++ b/arch/arm64/include/asm/barrier.h > @@ -22,6 +22,7 @@ > #define dmb(opt) asm volatile("dmb " #opt : : : "memory") > #define dsb(opt) asm volatile("dsb " #opt : : : "memory") > > +#define dgh() asm volatile("hint #6" : : : "memory") Although I'm fine with this in arm64, I don't think this is the interface which drivers should be using. Instead, once we know what this instruction is supposed to do, we should look at exposing it as part of the I/O barriers and providing a NOP implementation for other architectures. That way, drivers can use it without having to have the #ifdef CONFIG_ARM64 stuff that you have in the later patches here. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel