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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id D99E3C76188 for ; Wed, 5 Apr 2023 07:58:47 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 0848D10E85D; Wed, 5 Apr 2023 07:58:42 +0000 (UTC) Received: from mail-ed1-x52b.google.com (mail-ed1-x52b.google.com [IPv6:2a00:1450:4864:20::52b]) by gabe.freedesktop.org (Postfix) with ESMTPS id 90B7310E857 for ; Wed, 5 Apr 2023 07:58:39 +0000 (UTC) Received: by mail-ed1-x52b.google.com with SMTP id 4fb4d7f45d1cf-4fd1f2a0f82so4580a12.1 for ; Wed, 05 Apr 2023 00:58:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; t=1680681518; x=1683273518; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=7SBSkYNegGJEqfKYfRSv37qruEG8RJ8bRHf7fXfzSzY=; b=YsXVlBitEhbmbuzcKAXoVGfue6vdXaT4ovDEVFJcG9/mTl+Xo8ptJSsQCqqzN/Z0Ib DUWmxqn4QHjOGV4gFBOnpkvrehfKm3eXSV9nBNJaFSuZ9l4j0zx1cXVlHfY/mlUP9PVY WXhe3Ry2PQS4fOT9NpTG0tCeq04xbjrzKnEHA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680681518; x=1683273518; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=7SBSkYNegGJEqfKYfRSv37qruEG8RJ8bRHf7fXfzSzY=; b=ZirRVHxU51tUZ1f2vZ3IE8itEEvw1vS+fpZCrPDVEyb4hAgaFw10xb/+hq/2BvfVwo nw2cECTJAzgx2Gw6/iRs+eVirZeG+uMmTI+jlSyyKGvbiDDsjhqVKlrvUzsnF4j+DoO7 IXIUnmdtKwZb2rAHn2geEfR/8R24MLbyM101DSGcH3DTnxz/w2X2/GDAQYIUxEMpOs3v /ilXwbeHJI8GsH84Q0ef8oZ6fgdWIPkLVn0gApdqZupDQpfPzGeW4aPYbFnf3wySGj9t L+52SJZvVX7TccxYVzO2xIw+8bwEsjwEMdnIX5wIB0PdfHab5cpKN0bGvm5g4lHv7Vpw VTWQ== X-Gm-Message-State: AAQBX9fApjzCa8AynUm8dS/WSlmsAZLwo3iZeHgtWh5LEgud2vVXVanG GaPxy+WNAJ0dcr0ZDATt0hvknA== X-Google-Smtp-Source: AKy350a+V0Xbn15uUVJ56qff4Fj2aX205PUFyA2dU6ANZVL0/hNnQUg+zdTRIt/BZUwVsUX+BWvDiw== X-Received: by 2002:a17:906:1d5:b0:949:148d:82bc with SMTP id 21-20020a17090601d500b00949148d82bcmr1100490ejj.2.1680681517810; Wed, 05 Apr 2023 00:58:37 -0700 (PDT) Received: from phenom.ffwll.local (212-51-149-33.fiber7.init7.net. [212.51.149.33]) by smtp.gmail.com with ESMTPSA id r19-20020a50c013000000b004fd29e87535sm6856022edb.14.2023.04.05.00.58.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 05 Apr 2023 00:58:37 -0700 (PDT) Date: Wed, 5 Apr 2023 09:58:35 +0200 From: Daniel Vetter To: Dmitry Baryshkov Message-ID: Mail-Followup-To: Dmitry Baryshkov , Rob Clark , dri-devel@lists.freedesktop.org, Rob Clark , Christian =?iso-8859-1?Q?K=F6nig?= , intel-gfx@lists.freedesktop.org, open list , Luben Tuikov , Thomas Zimmermann , Rodrigo Vivi , Matt Turner , freedreno@lists.freedesktop.org References: <20230308155322.344664-1-robdclark@gmail.com> <20230308155322.344664-12-robdclark@gmail.com> <198c42ad-73de-9e9d-23d4-f36cb6840b5d@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <198c42ad-73de-9e9d-23d4-f36cb6840b5d@linaro.org> X-Operating-System: Linux phenom 6.1.0-7-amd64 Subject: Re: [Intel-gfx] [PATCH v10 11/15] drm/atomic-helper: Set fence deadline for vblank 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: Rob Clark , Christian =?iso-8859-1?Q?K=F6nig?= , intel-gfx@lists.freedesktop.org, open list , dri-devel@lists.freedesktop.org, Luben Tuikov , Thomas Zimmermann , Rodrigo Vivi , Matt Turner , freedreno@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Wed, Apr 05, 2023 at 12:53:29AM +0300, Dmitry Baryshkov wrote: > On 04/04/2023 22:16, Daniel Vetter wrote: > > On Tue, Apr 04, 2023 at 08:22:05PM +0300, Dmitry Baryshkov wrote: > > > On 08/03/2023 17:53, Rob Clark wrote: > > > > From: Rob Clark > > > > > > > > For an atomic commit updating a single CRTC (ie. a pageflip) calculate > > > > the next vblank time, and inform the fence(s) of that deadline. > > > > > > > > v2: Comment typo fix (danvet) > > > > v3: If there are multiple CRTCs, consider the time of the soonest vblank > > > > > > > > Signed-off-by: Rob Clark > > > > Reviewed-by: Daniel Vetter > > > > Signed-off-by: Rob Clark > > > > --- > > > > drivers/gpu/drm/drm_atomic_helper.c | 37 +++++++++++++++++++++++++++++ > > > > 1 file changed, 37 insertions(+) > > > > > > As I started playing with hotplug on RB5 (sm8250, DSI-HDMI bridge), I found > > > that this patch introduces the following backtrace on HDMI hotplug. Is there > > > anything that I can do to debug/fix the issue? The warning seems harmless, > > > but it would be probably be good to still fix it. With addresses decoded: > > > > Bit a shot in the dark, but does the below help? > > This indeed seems to fix the issue. I'm not sure about the possible side > effects, but, if you were to send the patch: > > Tested-by: Dmitry Baryshkov Thanks for the quick feedback, I already discussed this with Rob on irc yesterday (and landed his more throughrough version of the drm_vblank.c fix to drm-misc-next). I'll polish the drm_atomic_helper.c part asap and will send it out. Would be great if you can then retest to make sure all the pieces still work together for your case. -Daniel > > > > > > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c > > index f21b5a74176c..6640d80d84f3 100644 > > --- a/drivers/gpu/drm/drm_atomic_helper.c > > +++ b/drivers/gpu/drm/drm_atomic_helper.c > > @@ -1528,6 +1528,9 @@ static void set_fence_deadline(struct drm_device *dev, > > for_each_new_crtc_in_state (state, crtc, new_crtc_state, i) { > > ktime_t v; > > + if (drm_atomic_crtc_needs_modeset(new_crtc_state)) > > + continue; > > + > > if (drm_crtc_next_vblank_start(crtc, &v)) > > continue; > > diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c > > index 78a8c51a4abf..7ae38e8e27e8 100644 > > --- a/drivers/gpu/drm/drm_vblank.c > > +++ b/drivers/gpu/drm/drm_vblank.c > > @@ -1001,6 +1001,9 @@ int drm_crtc_next_vblank_start(struct drm_crtc *crtc, ktime_t *vblanktime) > > struct drm_display_mode *mode = &vblank->hwmode; > > u64 vblank_start; > > + if (!drm_dev_has_vblank(crtc->dev)) > > + return -EINVAL; > > + > > if (!vblank->framedur_ns || !vblank->linedur_ns) > > return -EINVAL; > > > > > > > > [ 31.151348] ------------[ cut here ]------------ > > > [ 31.157043] msm_dpu ae01000.display-controller: > > > drm_WARN_ON_ONCE(drm_drv_uses_atomic_modeset(dev)) > > > [ 31.157177] WARNING: CPU: 0 PID: 13 at drivers/gpu/drm/drm_vblank.c:728 > > > drm_crtc_vblank_helper_get_vblank_timestamp_internal > > > (drivers/gpu/drm/drm_vblank.c:728) > > > [ 31.180629] Modules linked in: > > > [ 31.184106] CPU: 0 PID: 13 Comm: kworker/0:1 Not tainted > > > 6.3.0-rc2-00008-gd39e48ca80c0 #542 > > > [ 31.193358] Hardware name: Qualcomm Technologies, Inc. Robotics RB5 (DT) > > > [ 31.200796] Workqueue: events lt9611uxc_hpd_work > > > [ 31.205990] pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS > > > BTYPE=--) > > > [ 31.213722] pc : drm_crtc_vblank_helper_get_vblank_timestamp_internal > > > (drivers/gpu/drm/drm_vblank.c:728) > > > [ 31.222032] lr : drm_crtc_vblank_helper_get_vblank_timestamp_internal > > > (drivers/gpu/drm/drm_vblank.c:728) > > > [ 31.230341] sp : ffff8000080bb8d0 > > > [ 31.234061] x29: ffff8000080bb900 x28: 0000000000000038 x27: > > > ffff61a7956b8d60 > > > [ 31.242051] x26: 0000000000000000 x25: 0000000000000000 x24: > > > ffff8000080bb9c4 > > > [ 31.250038] x23: 0000000000000001 x22: ffffbf0033b94ef0 x21: > > > ffff61a7957901d0 > > > [ 31.258029] x20: ffff61a795710000 x19: ffff61a78128b000 x18: > > > fffffffffffec278 > > > [ 31.266014] x17: 0040000000000465 x16: 0000000000000020 x15: > > > 0000000000000060 > > > [ 31.274001] x14: 0000000000000001 x13: ffffbf00354550e0 x12: > > > 0000000000000825 > > > [ 31.281989] x11: 00000000000002b7 x10: ffffbf00354b1208 x9 : > > > ffffbf00354550e0 > > > [ 31.289976] x8 : 00000000ffffefff x7 : ffffbf00354ad0e0 x6 : > > > 00000000000002b7 > > > [ 31.297963] x5 : ffff61a8feebbe48 x4 : 40000000fffff2b7 x3 : > > > ffffa2a8c9f64000 > > > [ 31.305947] x2 : 0000000000000000 x1 : 0000000000000000 x0 : > > > ffff61a780283100 > > > [ 31.313934] Call trace: > > > [ 31.316719] drm_crtc_vblank_helper_get_vblank_timestamp_internal > > > (drivers/gpu/drm/drm_vblank.c:728) > > > [ 31.324646] drm_crtc_vblank_helper_get_vblank_timestamp > > > (drivers/gpu/drm/drm_vblank.c:843) > > > [ 31.331528] drm_crtc_get_last_vbltimestamp > > > (drivers/gpu/drm/drm_vblank.c:884) > > > [ 31.337170] drm_crtc_next_vblank_start > > > (drivers/gpu/drm/drm_vblank.c:1006) > > > [ 31.342430] drm_atomic_helper_wait_for_fences > > > (drivers/gpu/drm/drm_atomic_helper.c:1531 > > > drivers/gpu/drm/drm_atomic_helper.c:1578) > > > [ 31.348561] drm_atomic_helper_commit > > > (drivers/gpu/drm/drm_atomic_helper.c:2007) > > > [ 31.353724] drm_atomic_commit (drivers/gpu/drm/drm_atomic.c:1444) > > > [ 31.358127] drm_client_modeset_commit_atomic > > > (drivers/gpu/drm/drm_client_modeset.c:1045) > > > [ 31.364146] drm_client_modeset_commit_locked > > > (drivers/gpu/drm/drm_client_modeset.c:1148) > > > [ 31.370071] drm_client_modeset_commit > > > (drivers/gpu/drm/drm_client_modeset.c:1174) > > > [ 31.375233] drm_fb_helper_set_par (drivers/gpu/drm/drm_fb_helper.c:254 > > > drivers/gpu/drm/drm_fb_helper.c:229 drivers/gpu/drm/drm_fb_helper.c:1644) > > > [ 31.380108] drm_fb_helper_hotplug_event > > > (drivers/gpu/drm/drm_fb_helper.c:2302 (discriminator 4)) > > > [ 31.385456] drm_fb_helper_output_poll_changed > > > (drivers/gpu/drm/drm_fb_helper.c:2331) > > > [ 31.391376] drm_kms_helper_hotplug_event > > > (drivers/gpu/drm/drm_probe_helper.c:697) > > > [ 31.396825] drm_bridge_connector_hpd_cb > > > (drivers/gpu/drm/drm_bridge_connector.c:129) > > > [ 31.402175] drm_bridge_hpd_notify (drivers/gpu/drm/drm_bridge.c:1315) > > > [ 31.406954] lt9611uxc_hpd_work > > > (drivers/gpu/drm/bridge/lontium-lt9611uxc.c:185) > > > [ 31.411450] process_one_work (kernel/workqueue.c:2395) > > > [ 31.415949] worker_thread (include/linux/list.h:292 > > > kernel/workqueue.c:2538) > > > [ 31.426843] kthread (kernel/kthread.c:376) > > > [ 31.437182] ret_from_fork (arch/arm64/kernel/entry.S:871) > > > [ 31.447828] irq event stamp: 44642 > > > [ 31.458284] hardirqs last enabled at (44641): __up_console_sem > > > (arch/arm64/include/asm/irqflags.h:182 (discriminator 1) > > > arch/arm64/include/asm/irqflags.h:202 (discriminator 1) > > > kernel/printk/printk.c:345 (discriminator 1)) > > > [ 31.474540] hardirqs last disabled at (44642): el1_dbg > > > (arch/arm64/kernel/entry-common.c:335 arch/arm64/kernel/entry-common.c:406) > > > [ 31.489882] softirqs last enabled at (42912): _stext > > > (arch/arm64/include/asm/current.h:19 arch/arm64/include/asm/preempt.h:13 > > > kernel/softirq.c:415 kernel/softirq.c:600) > > > [ 31.505256] softirqs last disabled at (42907): ____do_softirq > > > (arch/arm64/kernel/irq.c:81) > > > [ 31.521139] ---[ end trace 0000000000000000 ]--- > > > > > > > > > > > > -- > > > With best wishes > > > Dmitry > > > > > > > -- > With best wishes > Dmitry > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch