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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 37625C433EF for ; Tue, 16 Nov 2021 14:05:20 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 B381961B48 for ; Tue, 16 Nov 2021 14:05:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org B381961B48 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 26EE66EA6F; Tue, 16 Nov 2021 14:05:14 +0000 (UTC) Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by gabe.freedesktop.org (Postfix) with ESMTPS id 90F546F3A2 for ; Tue, 16 Nov 2021 07:57:01 +0000 (UTC) Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mmtKr-0004aV-87; Tue, 16 Nov 2021 08:56:57 +0100 Received: from sha by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1mmtKq-0004Qq-7w; Tue, 16 Nov 2021 08:56:56 +0100 Date: Tue, 16 Nov 2021 08:56:56 +0100 From: Sascha Hauer To: Daniel Vetter Message-ID: <20211116075656.GD6556@pengutronix.de> References: <20210917123513.1106-1-christian.koenig@amd.com> <20210917123513.1106-13-christian.koenig@amd.com> <20211115140353.GC6556@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 08:55:02 up 271 days, 11:18, 130 users, load average: 0.29, 0.23, 0.19 User-Agent: Mutt/1.10.1 (2018-07-13) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: sha@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: intel-gfx@lists.freedesktop.org X-Mailman-Approved-At: Tue, 16 Nov 2021 14:05:08 +0000 Subject: Re: [Intel-gfx] [PATCH 12/26] drm/scheduler: use new iterator in drm_sched_job_add_implicit_dependencies v2 X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linaro-mm-sig@lists.linaro.org, Christian =?iso-8859-15?Q?K=F6nig?= , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Mon, Nov 15, 2021 at 03:08:49PM +0100, Daniel Vetter wrote: > On Mon, Nov 15, 2021 at 03:03:53PM +0100, Sascha Hauer wrote: > > Hi, > > > > On Fri, Sep 17, 2021 at 02:34:59PM +0200, Christian König wrote: > > > Simplifying the code a bit. > > > > > > v2: use dma_resv_for_each_fence > > > > > > Signed-off-by: Christian König > > > --- > > > drivers/gpu/drm/scheduler/sched_main.c | 26 ++++++-------------------- > > > 1 file changed, 6 insertions(+), 20 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/scheduler/sched_main.c b/drivers/gpu/drm/scheduler/sched_main.c > > > index 042c16b5d54a..5bc5f775abe1 100644 > > > --- a/drivers/gpu/drm/scheduler/sched_main.c > > > +++ b/drivers/gpu/drm/scheduler/sched_main.c > > > @@ -699,30 +699,16 @@ int drm_sched_job_add_implicit_dependencies(struct drm_sched_job *job, > > > struct drm_gem_object *obj, > > > bool write) > > > { > > > + struct dma_resv_iter cursor; > > > + struct dma_fence *fence; > > > int ret; > > > - struct dma_fence **fences; > > > - unsigned int i, fence_count; > > > - > > > - if (!write) { > > > - struct dma_fence *fence = dma_resv_get_excl_unlocked(obj->resv); > > > - > > > - return drm_sched_job_add_dependency(job, fence); > > > - } > > > - > > > - ret = dma_resv_get_fences(obj->resv, NULL, &fence_count, &fences); > > > - if (ret || !fence_count) > > > - return ret; > > > > > > - for (i = 0; i < fence_count; i++) { > > > - ret = drm_sched_job_add_dependency(job, fences[i]); > > > + dma_resv_for_each_fence(&cursor, obj->resv, write, fence) { > > > + ret = drm_sched_job_add_dependency(job, fence); > > > if (ret) > > > - break; > > > + return ret; > > > } > > > - > > > - for (; i < fence_count; i++) > > > - dma_fence_put(fences[i]); > > > - kfree(fences); > > > - return ret; > > > + return 0; > > > } > > > EXPORT_SYMBOL(drm_sched_job_add_implicit_dependencies); > > > > > > > This patch lets the panfrost driver explode on v5.16-rc1 with the > > following. I didn't bisect it, but it goes away when I revert this > > patch. I only started weston, nothing more. > > > > Any idea what goes wrong here? > > Should be fixed in 13e9e30cafea1, but Christian pushed it to the wrong > patch so it missed -rc1. I can confirm 13e9e30cafea1 fixes the issue, thanks Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |