Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp1109038pxk; Fri, 2 Oct 2020 00:54:46 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzypXu/bsTUMj7Us+/CEDuG36PF88M/4HXRBEp2gmIiyeKsnRzCXtzoeTjKLaI+gCiFgpIi X-Received: by 2002:a17:906:2c04:: with SMTP id e4mr991271ejh.147.1601625286610; Fri, 02 Oct 2020 00:54:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1601625286; cv=none; d=google.com; s=arc-20160816; b=K89cMzQajm3u/jIRNYc05djU17Otxa9YNUA7yHXfagXy5uB/2YOSiGar8Woiq+j8Pd 4H3kx4k6V2WRX750M3Ux+9FDFC2ulbw4ErGLeaq+suGpFlPSg0cUWBE3Gv020e78wP6F bLmR0JENIlRkCIdNUrTIvkl4S3i7720QVQADHCC1RAwEpQsm5YGgQH8VISjUcioArkgH pPfs+F5ZxcqfBCZDyP+ZcI6xd2wbFzXEX1rChOcKKVZonRKZWAkK5lBVO6TxpJo3I/1h uEUVeGevcFAp1Mv+c60AdizKveIVhh+mYpVGXWlaHMrl/v8zfcupZoyOPSEhzV0MukRt dqZw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=XW2oAnf1nP/NaPgcm7Rs78JJ4Tv+nx++28EAZGgca0o=; b=b8ZFGPrkyV9O3cZWUiE5mueXIdiklvGXabsniezgRrIqJTtiaC9UpiKFk08cgzIdxy xeUxBvJl0X4Nd2/F+290WVInHkxEMl9G9eWRvfjUmXuMq8sgs5+nnvC9QLKCjrLPgUY/ ohWvBvKa9bPLh7tItN+BCtF+9WSNRO2cV/Oov14IiSxeyPzreOHTfbQnk8Hii08CHC/W lqw87HY3+/LU48Qmw1SUPkg1QO+8eFxocoYUrJg+DgOwyHe1ATrXtB7mkVs7VYbwpaux rVUooKfyVYZTyCm++dgUDiQjs72HwDdPEF8clWvIYZe1pUl2IdaqVoOrsutH0NqyeJUP NPJw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=casper.20170209 header.b=vZsuh1A0; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ds1si514026ejc.279.2020.10.02.00.54.24; Fri, 02 Oct 2020 00:54:46 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=casper.20170209 header.b=vZsuh1A0; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726096AbgJBHx1 (ORCPT + 99 others); Fri, 2 Oct 2020 03:53:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50134 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725968AbgJBHx0 (ORCPT ); Fri, 2 Oct 2020 03:53:26 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 55D35C0613D0; Fri, 2 Oct 2020 00:53:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=XW2oAnf1nP/NaPgcm7Rs78JJ4Tv+nx++28EAZGgca0o=; b=vZsuh1A0vOzF5XcYtf6FYQay+e cua8JGWDRiTZa/D8624BynsoLOs/bvqatzl4naq0J369e214Oaf5sH0wx/Nb3X2SK0jqCztZ0hCD/ 5RW6GA0cIxijQz3ZOnMPv3Jv4hoQSn8roOZS7utCiy9Sut8vnv/cxwN5lPyrRvTXoD5x+fNMPqSAL pFZ+xDE8Bi/RehqwInBhmbRzYKcFau4Xw+n6DgE6p9NUFgDgqqAKEhFLhodx+5f1x8OfK87uoxf1i XE215FY5GXBqsAK9mtaCcz3DrRngE/sA0niWrYcGOfjl+C+7RXeGRh2TbmXsIQb8FNMU+wIkpXStT T75cYj0Q==; Received: from hch by casper.infradead.org with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1kOFsX-0002Vm-DQ; Fri, 02 Oct 2020 07:53:21 +0000 Date: Fri, 2 Oct 2020 08:53:21 +0100 From: Christoph Hellwig To: Jonathan Marek Cc: freedreno@lists.freedesktop.org, Rob Clark , Sean Paul , David Airlie , Daniel Vetter , "open list:DRM DRIVER FOR MSM ADRENO GPU" , "open list:DRM DRIVER FOR MSM ADRENO GPU" , open list Subject: Re: [PATCH 2/3] drm/msm: add DRM_MSM_GEM_SYNC_CACHE for non-coherent cache maintenance Message-ID: <20201002075321.GA7547@infradead.org> References: <20201001002709.21361-1-jonathan@marek.ca> <20201001002709.21361-3-jonathan@marek.ca> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201001002709.21361-3-jonathan@marek.ca> X-SRS-Rewrite: SMTP reverse-path rewritten from by casper.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > @@ -8,6 +8,7 @@ > #include > #include > #include > +#include NAK, dma-noncoherent.h is not for driver use. And will in fact go away in 5.10. > > #include > > @@ -808,6 +809,20 @@ int msm_gem_cpu_fini(struct drm_gem_object *obj) > return 0; > } > > +void msm_gem_sync_cache(struct drm_gem_object *obj, uint32_t flags, > + size_t range_start, size_t range_end) > +{ > + struct msm_gem_object *msm_obj = to_msm_bo(obj); > + > + /* TODO: sync only the required range, and don't invalidate on clean */ > + > + if (flags & MSM_GEM_SYNC_CACHE_CLEAN) > + sync_for_device(msm_obj); > + > + if (flags & MSM_GEM_SYNC_CACHE_INVALIDATE) > + sync_for_cpu(msm_obj); And make to these ones as well. They are complete abuses of the DMA API, and while we had to live with that for now to not cause regressions they absoutely must not be exposed in a userspace ABI like this.