From: Mark Yacoub <markyacoub@chromium.org>
To: seanpaul@chromium.org, abhinavk@codeaurora.org,
robdclark@chromium.org, linux-arm-msm@vger.kernel.org
Cc: Mark Yacoub <markyacoub@google.com>,
Mark Yacoub <markyacoub@chromium.org>
Subject: [PATCH] drm/msm: Read frame_count and line_count even when disabled.
Date: Wed, 11 Aug 2021 14:08:44 -0400 [thread overview]
Message-ID: <20210811180844.2130484-1-markyacoub@chromium.org> (raw)
From: Mark Yacoub <markyacoub@google.com>
[why]
Reading frame count register used to get the vblank counter, which calls
dpu_encoder_phys to get the frame count. Even when it's disabled, the
vblank counter (through frame count) should return a valid value for the
count. An invalid value of 0, when compared to vblank->last (in
drm_vblank.c::drm_update_vblank_count()) returns an invalid number that
throws off the vblank counter for the lifetime of the process.
Rationale:
In drm_vblank.c::drm_update_vblank_count(), the new diff is calculated
through:
diff = (cur_vblank - vblank->last) & max_vblank_count;
cur_vblank comes from: cur_vblank = __get_vblank_counter(dev, pipe);
When the value is 0, diff results in a negative number (a very large
number as it's unsigned), which inflates the vblank count when the diff
is added to the current vblank->count.
[How]
Read frame_count register whether interface timing engine is enabled or
not.
Fixes: IGT:kms_flip::modeset-vs-vblank-race-interruptible
Tested on ChromeOS Trogdor(msm)
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
---
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c | 9 ++-------
drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h | 2 +-
2 files changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
index 116e2b5b1a90f..c436d901629f3 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c
@@ -266,13 +266,8 @@ static void dpu_hw_intf_get_status(
s->is_en = DPU_REG_READ(c, INTF_TIMING_ENGINE_EN);
s->is_prog_fetch_en = !!(DPU_REG_READ(c, INTF_CONFIG) & BIT(31));
- if (s->is_en) {
- s->frame_count = DPU_REG_READ(c, INTF_FRAME_COUNT);
- s->line_count = DPU_REG_READ(c, INTF_LINE_COUNT);
- } else {
- s->line_count = 0;
- s->frame_count = 0;
- }
+ s->frame_count = DPU_REG_READ(c, INTF_FRAME_COUNT);
+ s->line_count = DPU_REG_READ(c, INTF_LINE_COUNT);
}
static u32 dpu_hw_intf_get_line_count(struct dpu_hw_intf *intf)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h
index 3568be80dab51..877ff48bfef04 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h
@@ -41,7 +41,7 @@ struct intf_prog_fetch {
struct intf_status {
u8 is_en; /* interface timing engine is enabled or not */
u8 is_prog_fetch_en; /* interface prog fetch counter is enabled or not */
- u32 frame_count; /* frame count since timing engine enabled */
+ u32 frame_count; /* frame count since timing engine first enabled */
u32 line_count; /* current line count including blanking */
};
--
2.33.0.rc1.237.g0d66db33f3-goog
next reply other threads:[~2021-08-11 18:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-11 18:08 Mark Yacoub [this message]
2021-08-11 18:43 ` [PATCH] drm/msm: Read frame_count and line_count even when disabled Rob Clark
2021-08-11 23:11 ` [Freedreno] " abhinavk
2021-08-11 23:28 ` Rob Clark
2021-08-11 23:36 ` abhinavk
2021-08-12 20:16 ` Mark Yacoub
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210811180844.2130484-1-markyacoub@chromium.org \
--to=markyacoub@chromium.org \
--cc=abhinavk@codeaurora.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=markyacoub@google.com \
--cc=robdclark@chromium.org \
--cc=seanpaul@chromium.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).