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=-8.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 A36BBC43461 for ; Thu, 17 Sep 2020 11:02:35 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 41A402074B for ; Thu, 17 Sep 2020 11:02:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="px/pygYG" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 41A402074B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com 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=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=5Fe7CyM9VwMetdCAS5INS86YpCFM4OVixCtuqX5Lszw=; b=px/pygYGcceUQ4YaSKW4xTVfs iHXArISaQlujkkSD5WJk9StTn18tM6LtwlpF8krkxaPHk7uRZ3X16pFbCDGnQxQ70Av4CPd9d4ydi kyblWdiIrv/BEq4xU4ocYF5TlZbmYcTy21mT1z3aQptikBTWkSffFBcwkute6Vb59/7+1gq7ybOVf EteeiA32uF5RQJc97xEwYYKxh0BKWTIAKXhsRDAa2tIcNedKyFmyRZnPOJYO3+2RGnpK88m7rMEJ0 +wRN5lJ47sDNaIuqY5oI3/QWNaqTqpyPQk7LfQ3ht/qPZ1ooMQmAfYvNIwshTi1aZWt2eLhbcwMdd 3w79uNogw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kIrf1-0002os-Cv; Thu, 17 Sep 2020 11:01:07 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kIrex-0002o2-HN; Thu, 17 Sep 2020 11:01:04 +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 5E5E130E; Thu, 17 Sep 2020 04:01:02 -0700 (PDT) Received: from [192.168.1.79] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 4E8523F68F; Thu, 17 Sep 2020 04:01:00 -0700 (PDT) Subject: Re: [PATCH 0/3] drm: panfrost: Coherency support To: Tomeu Vizoso , Rob Herring , Alyssa Rosenzweig References: <20200916170409.GA2543@kevin> <0518e067-3b9a-8fe3-619f-8fce4c6c9677@collabora.com> From: Steven Price Message-ID: <51a06e78-7b80-c65d-67c4-e026e729e505@arm.com> Date: Thu, 17 Sep 2020 12:00:58 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <0518e067-3b9a-8fe3-619f-8fce4c6c9677@collabora.com> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200917_070103_645279_C1F7447B X-CRM114-Status: GOOD ( 18.09 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Neil Armstrong , Robin Murphy , dri-devel , Linux IOMMU , Kevin Hilman , "open list:ARM/Amlogic Meson..." , Will Deacon , linux-arm-kernel , Jerome Brunet Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 17/09/2020 11:51, Tomeu Vizoso wrote: > On 9/17/20 12:38 PM, Steven Price wrote: >> On 16/09/2020 18:46, Rob Herring wrote: >>> On Wed, Sep 16, 2020 at 11:04 AM Alyssa Rosenzweig >>> wrote: >>>> >>>>> So I get a performance regression with the dma-coherent approach, >>>>> even if it's >>>>> clearly the cleaner. >>>> >>>> That's bizarre -- this should really be the faster of the two. >>> >>> Coherency may not be free. CortexA9 had something like 4x slower >>> memcpy if SMP was enabled as an example. I don't know if there's >>> anything going on like that specifically here. If there's never any >>> CPU accesses mixed in with kmscube, then there would be no benefit to >>> coherency. >> >> The DDK blob has the ability to mark only certain areas of memory as >> coherent for performance reasons. For simple things like kmscube I >> would expect that it's basically write-only from the CPU and almost >> all memory the GPU touches isn't touched by the CPU. I.e. coherency >> isn't helping and the coherency traffic is probably expensive. Whether >> the complexity is worth it for "real" content I don't know - it may >> just be silly benchmarks that benefit. > > Or maybe it's only a problem for applications that do silly things? I > don't think kmscube was ever optimized for performance. Well doing silly things is almost the definition of a benchmark ;) A lot of the mobile graphics benchmarks suffer from not being very intelligent in how they render (e.g. many have meshes that are far too detailed so the triangles are smaller than the pixels). Of course there are also applications that get things wrong too. Steve _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel