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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 22081C001B0 for ; Wed, 9 Aug 2023 11:23:53 +0000 (UTC) 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=li2JT505v4aEcgKftX54afHqNS6d2VMlrZKdhLvrnLs=; b=HhhjVkFFcw6NoK MBZ8+xr5UvJA3s2TK/WqrwQBXE+G6z1e42kkg5hWBn4XDNZptZblbLcYzZFAfNezvdCh8N9YzP8UU XxH2DJFHURs91y2DRcMxfTu2pAIun3SIKvRM1wipCBbWi7haZ/P63o622LFGnFI/voyuinXl/LKT1 45np04WiTP6+inxTtIeQAcVD50mgyXVMspmZhWoGw7V+hiIQURCjQFSEifr6DEyw3JBTZw1HX276j QqLEOLaA4BkJshn0h+Rog8dUVHb4SfAlz4C359jLfifKhShvg9PnR6TLW2nZ1bY7WW1Ar9TB8H2ed tMwE70vPjrSjXE1ODVkg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qThHj-004lb4-1i; Wed, 09 Aug 2023 11:23:27 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qThHg-004laE-38 for linux-arm-kernel@lists.infradead.org; Wed, 09 Aug 2023 11:23:26 +0000 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 7223863220; Wed, 9 Aug 2023 11:23:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BB94EC433C9; Wed, 9 Aug 2023 11:23:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1691580203; bh=/Jo698VUwWViOfRpDRaOF+geLJTlE2ZHuvKqfJHymbg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cBVsEVdhjXnSoLGT3ZxfxQMdwIpTlHwYq31bkpyXuEt3FKgzMn8pbVUhgqStXya3/ lcw+9dGGF+1Q2BZm2cJN1Bdpt/evjLRT4YVvAK3z0rqt5GvfgApsICmtIccmpgYJ3j YqiB4ahG1XIRUzI46dqIU1Ak6syGoPR0U9TFSJI8QUej/PICfsZ+svckYc07Q5qvJB a2g6nLd6G6ySr9vZQwcBIWNY54wilqzqqTl7QJVOXMAHcjGDfZSH6Oq95MsYvjZe/K YVFsSCCj4vJVlaU8xQoWXU+92Gt7AaZzj61uWagT0lOfsBREbBYGUSD1iX1/Rumki7 MjGIxfMUma2yw== Date: Wed, 9 Aug 2023 12:23:18 +0100 From: Will Deacon To: zhurui Cc: Robin Murphy , Nicolin Chen , linux-arm-kernel@lists.infradead.org, iommu@lists.linux.dev, linux-kernel@vger.kernel.org, Joerg Roedel , Lu Baolu , Jason Gunthorpe , Yicong Yang , Tomas Krcka , Jean-Philippe Brucker Subject: Re: [PATCH v2 1/1] iommu/arm-smmu-v3: Fix error case of range command Message-ID: <20230809112317.GA3830@willie-the-truck> References: <20230801085504.GA26130@willie-the-truck> <27c895b8-1fb0-be88-8bc3-878d754684c8@huawei.com> <20230804165225.GF30679@willie-the-truck> <015b4573-9d74-451b-8028-a1050ade7019@huawei.com> <661a7bb5-99e1-de16-d860-0cd17f7a0470@arm.com> <20230808162409.GB2890@willie-the-truck> <80ead8ee-4dbe-7b3c-44f5-944073a2a39d@arm.com> <412886be-644a-5b46-9bfa-1c9a358f9a5d@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <412886be-644a-5b46-9bfa-1c9a358f9a5d@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-20230809_042325_084518_614AEBD2 X-CRM114-Status: GOOD ( 33.69 ) 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 Wed, Aug 09, 2023 at 05:22:06PM +0800, zhurui wrote: > On 2023/8/9 0:43, Robin Murphy wrote: > > On 08/08/2023 5:24 pm, Will Deacon wrote: > >> On Mon, Aug 07, 2023 at 08:20:45PM +0100, Robin Murphy wrote: > >>> Yeah, I'd rather not downgrade to a non-range invalidate since that > >>> complicates the reasoning for the errata affecting those. If the size of the > >>> invalidation is equal to TG then it can only represent a single last-level > >>> page, i.e. TTL=3, thus if it does warrant handling here then indeed > >>> rearranging to base the condition on num_pages as well ought to suffice. > >>> However, this is all still begging the question of where and why we're doing > >>> a *non-leaf* invalidation that isn't aligned to the size of a table, because > >>> that in itself doesn't make a whole heap of sense - my hunch is that that > >>> wants figuring out and could probably be fixed at the source. > >> > >> Isn't that described above because we're using CMDQ_TLBI_RANGE_NUM_MAX > >> to break up the range into separate commands? > > > > Not really, because if we're doing a genuine non-leaf invalidation of a > > table then it should be a block-aligned range that ought to fit in a > > single command and should certainly never involve a single-granule > > remainder. If we're doing non-leaf invalidations of things that > > logically don't need to be non-leaf, making them leaf would be the even > > better option. > > > > I agree with Robin that if the caller is doing a genuine non-leaf invalidation > of a table, it should not involve a single-granule tlbi. It seems that the > caller only filter the block size, but not the address aligned or not maybe. There's only one caller though, right? That's the io_pgtable_tlb_flush_walk() call in io-pgtable-arm.c which shouldn't trigger this problem. Do you have a backtrace for the case when we generate the illegal command? > >> Do you mind if I queue the patch as-is for now? I don't think the driver > >> should be emitting illegal commands, and v2 of the patch does seem like > >> the obvious thing to do. > > > > TBH I'd rather you just drop my patch if it's proven problematic, and > > I'll take another crack at it soon. The potential problems we introduce > > by using non-range invalidates on errata-affected MMU-700 revisions are > > worse than the almost-entirely-theoretical one I was trying to address. > > > > If you all agree to roll back the problematic code, is the first patch be OK? > Should I need to add some more descriptions to clarify this? I can just go and revert Robin's original patch, but I'd like to see your backtrace first so that we understand how this is occurring. Thanks, Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel