From patchwork Tue Apr 30 09:37:46 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 163098 Delivered-To: patch@linaro.org Received: by 2002:a92:7e86:0:0:0:0:0 with SMTP id q6csp2989854ill; Tue, 30 Apr 2019 02:37:55 -0700 (PDT) X-Google-Smtp-Source: APXvYqx1FTaGz4lCVBTZGZJ9mz2uo/UwMsnvamkG1OcvmyMsEYADQjd8pvASfelUbMw/BmwT/PQ2 X-Received: by 2002:a63:c10c:: with SMTP id w12mr19010988pgf.39.1556617075188; Tue, 30 Apr 2019 02:37:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1556617075; cv=none; d=google.com; s=arc-20160816; b=S8XpxyOllXbDIJqf3MjfUKU4P4iBw66wKPjby5W3gRhHFhQjzB4TLSbGWGnqK0Du9G V6suJfza8Q9zepBjUEMGoBGBtsifhcGqNmEFLKKq3K30Gn4ja9W460mEsVTCGAgg54pt 07lyIOl7wdqGB96adc9BkvZybjOQnyYPZnY9zT1+halGDFyU/k40pfI3NTE1wd5In7MM 6fHt+qVKEcau00BegKi50NwHCwBYzpW8EebzXcz7ZI5le0Z4Ytdv/tXgtdOWcjZK07f4 W4RLr9L7CV6YCyqziVkR2cURNHnBe9nA07lP4TpmK+t3MtAJ3Bf0REchwjRMZWv5LTog KV8w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:cc:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:mime-version:message-id:date:subject:to:from :delivered-to; bh=FhLu9Lv2pWGwVb7uw8qiLd9YGmLCDT/s9zxPfllhLrY=; b=pfGeCKWV3fqlIrNIUjxaMRb2JC7zPW9nace3f9pdP5fYkfsvAS5GqKUaWeWlyw0tno 9hmWDoCeeXC17Rp4g5o322gHLnWBQpJbRgQ/Gov2vKMVol5f8OF3BWnwvydME0NErLfw j1DcDpvopHJSO0Cie/HOxx61x6DmSPzfQHK6xz9awnp45Og8wQ7TvDlJhMB+geu9rrWb /S2axoiVogDV2JqDffyqFDSLKiERZEcF6iXySzMSaiq9qOevqdBaIw43KWbs8HJcAD/G IcMvfE+YKKp7I2nRksRWplRsRW3OfV+VDKUcc6uc+ZU+DNDldx0mTitM+9lPdQBwgncc 78ZQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 2610:10:20:722:a800:ff:fe36:1795 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from gabe.freedesktop.org (gabe.freedesktop.org. [2610:10:20:722:a800:ff:fe36:1795]) by mx.google.com with ESMTPS id cm19si7544251plb.202.2019.04.30.02.37.54 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 30 Apr 2019 02:37:55 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 2610:10:20:722:a800:ff:fe36:1795 as permitted sender) client-ip=2610:10:20:722:a800:ff:fe36:1795; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 2610:10:20:722:a800:ff:fe36:1795 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E86F788FA4; Tue, 30 Apr 2019 09:37:53 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lj1-x244.google.com (mail-lj1-x244.google.com [IPv6:2a00:1450:4864:20::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id 12B1488FA4 for ; Tue, 30 Apr 2019 09:37:53 +0000 (UTC) Received: by mail-lj1-x244.google.com with SMTP id k8so12114283lja.8 for ; Tue, 30 Apr 2019 02:37:52 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Ug09hZ4eYYnkShE0Xn5KTs+gkypuHyMBpZHpCmxzczc=; b=QDR+iDmIjmMlT551p233MX9+aovkUW1I4VhfLGRoZCOo+Zks7o7YyY6T8WTAvbBlZM TCSROY8jHSY7rEgqPYT7pbcQWZCdNtr287iKtRqo/dV/Q4t4J4QvJlz2FwA+ACwtlTWv iE2HzUfzWO84dEa9Ncf07NyNNUs7AD2NMbSm4vIctivX3XpCBln22E0WEzqbctfR/ZAu RDiNwZxgqxFXZe/X1QZDT7KTns+Tnxqj2lF3UtsxdFUMFv9IuZn+fDNcY0iBgTGD1hHl FKq03c4KYQiS99ystlcjOfd/nzuLdzPsL5o94akcZIA7Xaap79UU92RNlypylvs3P9AB pzHQ== X-Gm-Message-State: APjAAAU+n3SKu8EAL0+m5l3fWyUnc7lfXV9oaadJoN+RtWtHiIha2TyO M0MyeuuGnUFit1VWhrTlchSuJ/RtIAk= X-Received: by 2002:a2e:9c0a:: with SMTP id s10mr18458421lji.162.1556617071239; Tue, 30 Apr 2019 02:37:51 -0700 (PDT) Received: from genomnajs.ideon.se ([85.235.10.227]) by smtp.gmail.com with ESMTPSA id y23sm7677210lfy.31.2019.04.30.02.37.49 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 30 Apr 2019 02:37:50 -0700 (PDT) From: Linus Walleij To: dri-devel@lists.freedesktop.org, Daniel Vetter , Sam Ravnborg Subject: [PATCH] drm/atomic-helper: Bump vblank timeout to 100 ms Date: Tue, 30 Apr 2019 11:37:46 +0200 Message-Id: <20190430093746.26485-1-linus.walleij@linaro.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Ug09hZ4eYYnkShE0Xn5KTs+gkypuHyMBpZHpCmxzczc=; b=yvCyASVrVDX4wNmvcOAJniMdPJ1vM118ZLkqRahj7vNECWCt2q1aRNYxxF8pdBIIMI kEI9V5nPuTjHt+sRSzqAYNV2k390/ON8PSgkQZWDlvVbwT1Jv0R6zJ15Pdj2x3mmwB/0 LBBaPfLuJ/PFNYa/JLaPR2+H3ml7HJfdw67nXXuXnDpAL6v9x4rhuPFMpLut+H2FsAMd OSagQT4ZpyG02Lk+x3m9xqIUl0lHXCLXy3K651sHfsbfPtLswUuRqaOQmPSXS22M8pH1 3bPggsmMv0Ya0b/Kz114X1AR792OeqignQHcNwb5WXT25P0swE11fl2spDBYoVCK2J/J ZC/A== X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Sean Paul Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" The 50 ms default timeout waiting for vblanks is too small for the first vblank from the ST-Ericsson MCDE display controller over DSI. Presumably this is because the DSI display is command-mode only and the state machine will take some time setting up its state for the first display update, and we hit a timeout. 100 ms makes it pass without problems. Signed-off-by: Linus Walleij Reviewed-by: Ville Syrjälä Reviewed-by: Daniel Vetter --- After a quite prolonged hunting for the cause of missed vblanks in the MCDE driver I finally realized it timed out because it was simply taking some time on the first vblank. 50 ms makes sense on 60Hz monitors for sure, but an embedded DSI state machine can be slow, as it turns out. Tell me if this should be a per-driver variable and I will think of something. --- drivers/gpu/drm/drm_atomic_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index 40ac19848034..f0aa7b195d79 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c @@ -1424,7 +1424,7 @@ drm_atomic_helper_wait_for_vblanks(struct drm_device *dev, ret = wait_event_timeout(dev->vblank[i].queue, old_state->crtcs[i].last_vblank_count != drm_crtc_vblank_count(crtc), - msecs_to_jiffies(50)); + msecs_to_jiffies(100)); WARN(!ret, "[CRTC:%d:%s] vblank wait timed out\n", crtc->base.id, crtc->name);