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 98BADC4332F for ; Thu, 15 Dec 2022 03:05:57 +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=Ykn4KPiwze88wU6OZPaaBCbAtSaaE8daj68SKReA59A=; b=VCfo+i2GeF499W iyAjFNVDv0eE1vcZc4KykyqwRvyTIbNGkcvp4H81Nol6CLMVFX6HzzaK8uSMzYZlzGpPz4ssUvtnM atrYM8o6lnFFdD+7kbE7G+MhHX7qoiEuO2s/RmpFHHE4myyYy5deYxFrg32wx2EosprT5dbCROGId UzsbDOmNaPgJfbm9Y3Y1/b0ockMFWMFpXcyaD9XTCxIGPj3lUBim51WjLpedyfBEIV+nY2enYIxZl TSiazLMNL0cZHR9dwxje7+espVpjc4JGjX8AlvHQh8sNVndjz5NbNcKhDN7YnEHbzRbx9YfS7+1Q/ HW2PmZ/X2EbmeMo2R7VQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p5eY7-005rJT-L3; Thu, 15 Dec 2022 03:04:43 +0000 Received: from mail-pl1-x62b.google.com ([2607:f8b0:4864:20::62b]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p5eY3-005rHK-LU for linux-arm-kernel@lists.infradead.org; Thu, 15 Dec 2022 03:04:41 +0000 Received: by mail-pl1-x62b.google.com with SMTP id a9so5447252pld.7 for ; Wed, 14 Dec 2022 19:04:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=K3IDb4rreGLvgd+2ug69AbMNttQGcSlWv6zwqNmBMEs=; b=WO+jqkME9JS3LPl3YEwJ9fsDiX7H+yavqwaNSvSMYi4QNdrrpXYeINXgw4DAvI1RT4 DHE3EJg9ncOQk6Fm2cy4nRE/s3tqXBfXZUOTn5VW2Be9HM1ZXdrpZIezWYkoCTBHgVsH +O2wt7wDzP5I+bVZRyXDMXsLRcLafoo2XMK8M= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=K3IDb4rreGLvgd+2ug69AbMNttQGcSlWv6zwqNmBMEs=; b=RSRMu8AUipNdC9IUzlW79TKaJ8z8+qsuFvu20bLhtR87OoOC12v7n72NQBL1HIGs5H FQoyeghodWNC/0GlbJ0wPjJU4DLjBhhKrXtXx8wZIuaI4+101Y7T4wvdgl8Bcm6kVHop zJO1TLgOf+Rs5OXD7nixKi0ctWHPIO+5+OTYy1kkxn2My5Xy9Ai+LN2bUUeDF4zx4Gy6 HIqOTV/IJognDa30hHlKOqop3SIIw6rbv8OB8xxZ/LP/mHWW+Resg2o5rPXADTMtTpUY yi45OqhFBvRey3wHyFGyV+Ckg40xNT8FswaehBS+L4ueGrBbcenX44FPNP91z7Dm3Crc OSKA== X-Gm-Message-State: ANoB5pnlzsbwoetbQ7nJIVnUQtzBl7OZcphRe3NKE60D9BdVgl+vr+34 kOdZGKlROjkP4D8lGyeE3Fy6QQ== X-Google-Smtp-Source: AA0mqf7wKc6XzZX1oNXGMf8tcflozHngQGwuO9UrUVF9WZ/jsDYALIyxEzz5shOz3mkM1vRpJcglSA== X-Received: by 2002:a17:90a:3e03:b0:219:2aa9:2c7c with SMTP id j3-20020a17090a3e0300b002192aa92c7cmr27847877pjc.0.1671073477529; Wed, 14 Dec 2022 19:04:37 -0800 (PST) Received: from google.com ([2620:15c:9d:2:8edc:d4ff:fe53:350d]) by smtp.gmail.com with ESMTPSA id s18-20020a17090a5d1200b00218f830c63esm2063785pji.1.2022.12.14.19.04.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Dec 2022 19:04:36 -0800 (PST) Date: Wed, 14 Dec 2022 19:04:34 -0800 From: Brian Norris To: Mark Brown Cc: kernelci-results@groups.io, bot@kernelci.org, Sean Paul , Douglas Anderson , gtucker@collabora.com, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec Subject: Re: renesas/master bisection: igt-kms-rockchip.kms_vblank.pipe-A-wait-forked on rk3399-gru-kevin Message-ID: References: <6398848e.170a0220.f8e8e.d44f@mx.google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221214_190439_776139_00F14C24 X-CRM114-Status: GOOD ( 25.36 ) 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 Tue, Dec 13, 2022 at 04:51:11PM +0000, Mark Brown wrote: > On Tue, Dec 13, 2022 at 05:56:30AM -0800, KernelCI bot wrote: > > The KernelCI bisection bot found regressions in at least two KMS tests > in the Renesas tree on rk3399-gru-kevin just after the Renesas tree > merged up mainline: > > igt-kms-rockchip.kms_vblank.pipe-A-wait-forked > igt-kms-rockchip.kms_vblank.pipe-A-query-busy > > which it bisected to ca871659ec16 ("drm/bridge: analogix_dp: Support > PSR-exit to disable transition"). I'm not *100%* sure I trust the > bisection but it sure is suspicous that two separate bisects for related > issues landed on the same commit. > > Below is the full report for the bisect for the first test, the bisect > for the latter looks identical. It's got links to full logs for the > test run and a Reported-by for the bot - I do see some backtraces from > userspace in the output, the first is: I think the backtraces are just because IGT calls assert(). > | IGT-Version: 1.26-gf8a4a0b (aarch64) (Linux: 6.1.0 aarch64) > | <14>[ 35.444448] [IGT] drm_read: starting subtest short-buffer-wakeup > | Starting subtest: short-buffer-wakeup > | > | (| drm_read:350) CRITICAL: Test assertion failure function generate_event, file ../tests/drm_read.c:65: > | (drm_read:350) CRITICAL: <14>[ 36.155642] [IGT] drm_read: exiting, ret=98 > | Failed assertion: kmstest_get_vblank(fd, pipe, DRM_VBLANK_EVENT) > | > | (drm_read:350) CRITICAL: Last errno: 22, Invalid argument > | Stack trace: > | > | #0 ../lib/igt_core.c:1933 __igt_fail_assert() > | #1 [+0xd5362770] > | #2 [+0xd536193c] > | #3 [__libc_start_main+0xe8] > | #4 [+0xd5361974] > | #5 [[ 36.162851] Console: switching to colour frame buffer device 300x100>+0xd5361974] > | Subtest short-buffer-wakeup failed. > > Unfortunately we don't have current results from mainline or -next. Thanks for the notification. I'm running: 6e516faf0431 drm/panfrost: Job should reference MMU not file_priv which is allegedly a "good" kernel. And running this: ### First kill my display manager, etc. ## Then: IGT_FORCE_DRIVER=rockchip /usr/libexec/igt-gpu-tools/kms_vblank --run-subtest pipe-A-wait-forked Gives the appended log [1]. If I'm looking right, it seems like it's failing the same way as the "regression." I also tested this: # the 5.10.x backport, and its parent: dbe04e874d4fbd56be64fdfcb29410241b6ad08a^ -- i.e.: 61297ee0c329 Input: bcm5974 - set missing URB_NO_TRANSFER_DMA_MAP urb flag and saw the same failures, and I also see the failures I was trying to avoid in this series (see e54a4424925a ("drm/atomic: Force bridge self-refresh-exit on CRTC switch")). Perhaps that's because of the "First kill my display manager, etc." -- because that step means we might end up switching CRTCs (VOPs) when the test starts, and triggering the bug. I'll look some more, but this might be a difference of test setup, such that my setup has the issue before and after that commit, but your setup sees a regression. Small tangent: It's possible this is similar to stuff I had to debug a while back in this space: atomictest: Disable CRTCs before test https://chromium-review.googlesource.com/c/chromiumos/platform/drm-tests/+/3309960 null_platform_test: Disable CRTCs first https://chromium-review.googlesource.com/c/chromiumos/platform/drm-tests/+/3315478 In that case, there was actually an underlying kernel regression due to: 846c7dfc1193 drm/atomic: Try to preserve the crtc enabled state in drm_atomic_remove_fb, v2. But our tests were broken too (assuming an initial state that wasn't guaranteed), so we just fixed the tests. Anyway, I'll confirm when I get some fresh eyes and can try a few more things (like neutering the ChromeOS display framework for my tests). Brian [1] IGT-Version: 1.26-gf8a4a0b5 (arm) (Linux: 5.18.0-rc6+ aarch64) Starting subtest: pipe-A-wait-forked Beginning pipe-A-wait-forked on pipe A, connector eDP-1 (kms_vblank:2532) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2535) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2531) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2532) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2534) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2535) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2531) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2536) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2532) CRITICAL: Last errno: 22, Invalid argument (kms_vblank:2534) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2536) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2532) CRITICAL: error: -22 != 0 (kms_vblank:2531) CRITICAL: Last errno: 22, Invalid argument (kms_vblank:2535) CRITICAL: Last errno: 22, Invalid argument (kms_vblank:2531) CRITICAL: error: -22 != 0 (kms_vblank:2534) CRITICAL: Last errno: 22, Invalid argument (kms_vblank:2536) CRITICAL: Last errno: 22, Invalid argument (kms_vblank:2535) CRITICAL: error: -22 != 0 (kms_vblank:2534) CRITICAL: error: -22 != 0 (kms_vblank:2536) CRITICAL: error: -22 != 0 (kms_vblank:2533) CRITICAL: Test assertion failure function vblank_wait, file ../igt-gpu-tools-1.25/tests/kms_vblank.c:319: (kms_vblank:2533) CRITICAL: Failed assertion: wait_vblank(fd, &vbl) == 0 (kms_vblank:2533) CRITICAL: Last errno: 22, Invalid argument Stack trace: Stack trace: (kms_vblank:2533) CRITICAL: error: -22 != 0 Stack trace: Stack trace: Stack trace: Stack trace: child 1 failed with exit status 98 Subtest pipe-A-wait-forked failed. **** DEBUG **** (kms_vblank:2526) igt_kms-DEBUG: display: eDP-1: set_pipe(A) (kms_vblank:2526) igt_kms-DEBUG: display: eDP-1: Selecting pipe A (kms_vblank:2526) igt_fb-DEBUG: igt_create_fb_with_bo_size(width=2400, height=1600, format=XR24(0x34325258), modifier=0x0, size=0) (kms_vblank:2526) igt_fb-DEBUG: igt_create_fb_with_bo_size(handle=1, pitch=9600) (kms_vblank:2526) ioctl_wrappers-DEBUG: Test requirement passed: igt_has_fb_modifiers(fd) (kms_vblank:2526) igt_fb-DEBUG: Test requirement passed: cairo_surface_status(fb->cairo_surface) == CAIRO_STATUS_SUCCESS (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: A.0: plane_set_fb(56) (kms_vblank:2526) igt_kms-DEBUG: display: A.0: plane_set_size (2400x1600) (kms_vblank:2526) igt_kms-DEBUG: display: A.0: fb_set_position(0,0) (kms_vblank:2526) igt_kms-DEBUG: display: A.0: fb_set_size(2400x1600) (kms_vblank:2526) igt_kms-DEBUG: display: commit { (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe A, plane 0 (kms_vblank:2526) igt_kms-DEBUG: display: eDP-1: SetCrtc pipe A, fb 56, src (0, 0), mode 2400x1600 (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe A, plane 1 (kms_vblank:2526) igt_kms-DEBUG: display: SetCursor pipe A, disabling (kms_vblank:2526) igt_kms-DEBUG: display: MoveCursor pipe A, (0, 0) (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe B, plane 0 (kms_vblank:2526) igt_kms-DEBUG: display: SetCrtc pipe B, disabling (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe B, plane 1 (kms_vblank:2526) igt_kms-DEBUG: display: SetPlane pipe B, plane 1, disabling (kms_vblank:2526) igt_kms-DEBUG: display: SetProp plane B.1 "rotation" to 0x1/1 (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe B, plane 2 (kms_vblank:2526) igt_kms-DEBUG: display: SetPlane pipe B, plane 2, disabling (kms_vblank:2526) igt_kms-DEBUG: display: SetProp plane B.2 "rotation" to 0x1/1 (kms_vblank:2526) igt_kms-DEBUG: Test requirement passed: plane_idx >= 0 && plane_idx < pipe->n_planes (kms_vblank:2526) igt_kms-DEBUG: display: Fixing up initial rotation pipe B, plane 3 (kms_vblank:2526) igt_kms-DEBUG: display: SetCursor pipe B, disabling (kms_vblank:2526) igt_kms-DEBUG: display: MoveCursor pipe B, (0, 0) (kms_vblank:2526) igt_kms-DEBUG: display: } (kms_vblank:2526) igt_debugfs-DEBUG: Opening debugfs directory '/sys/kernel/debug/dri/1' (kms_vblank:2526) INFO: Beginning pipe-A-wait-forked on pipe A, connector eDP-1 (kms_vblank:2526) DEBUG: Spawning 6 threads **** END **** Subtest pipe-A-wait-forked: FAIL (0.938s) Test requirement not met in function igt_require_pipe, file ../igt-gpu-tools-1.25/lib/igt_kms.c:2360: Test requirement: !(pipe >= display->n_pipes || !display->pipes[pipe].enabled) Pipe C does not exist or not enabled Test requirement not met in function igt_require_pipe, file ../igt-gpu-tools-1.25/lib/igt_kms.c:2360: Test requirement: !(pipe >= display->n_pipes || !display->pipes[pipe].enabled) Pipe D does not exist or not enabled Test requirement not met in function igt_require_pipe, file ../igt-gpu-tools-1.25/lib/igt_kms.c:2360: Test requirement: !(pipe >= display->n_pipes || !display->pipes[pipe].enabled) Pipe E does not exist or not enabled Test requirement not met in function igt_require_pipe, file ../igt-gpu-tools-1.25/lib/igt_kms.c:2360: Test requirement: !(pipe >= display->n_pipes || !display->pipes[pipe].enabled) Pipe F does not exist or not enabled _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel