From: Thomas Richter <richter@rus.uni-stuttgart.de>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: 830GM still woes
Date: Thu, 29 May 2014 16:10:43 +0200 [thread overview]
Message-ID: <53873FE3.8080704@rus.uni-stuttgart.de> (raw)
In-Reply-To: <23914_1400259040_537641E0_23914_9298_1_20140516165034.GT8790@phenom.ffwll.local>
Hi Daniel, hi folks,
still a couple of observations from my side on this. The 1024x786x24
mode here uses a clock of 65MHz (65000kHz), if that is inserted into the
watermark computation, it computes from that a prefetch of 40 entries,
and thus a watermark level of four, which is much much too high. To get
a stable display, the watermark level can be at most eight.
However, if I program the FW_BLC register by hand, I can set the
watermark down to 32, i.e. 15(!) entries, and I still get a wonderful
stable display, scrolling and everything included.
If I compare that with what is required by intel_calculate_wm() to get
the same value, I find that the input is, in some place, off by about a
factor of two. Which could mean:
a) the latency is too high by a factor of two. Even with a latency of
2500ns, I do get a good watermark level and a stable display. The limit
is around 1500ns.
b) the I830_FIFO_LINE_SIZE is off by a factor of two. Is it really 32
bytes? Is it *measured* in bytes? With a line size twice as large, the
result would fit again.
c) Is the I830 FIFO_SIZE really 47? This is the value I get when I debug
i9xx_update_wm(). It seems the code splits the totally available fifo
(unified fifo of the 830GM) of 95 entries approximately in half for pipe
A and pipe B. Is the *unit* correct? Is the fifo size measured in *entries*?
What makes me wonder is that there is really approximately a factor of
two between the *real* limit and the value computed by the code, which
looks to me that at some point a division or multiplication by two is
missing.
Finally, a regression with the 3.15.0 code: I already had the phenomenon
that the boot console is vertically shifted, which is caused by the
pipe-a quirk (without that quirk, the display is correct), but it now
also happens from time to time that the DVO is again not clocked
correctly. The screen then goes dead in the boot console, but as soon as
X starts up, I get again a display. This display is sometimes a bit
broken (flickers, as if the frequency is about 40Hz, not 60Hz) or no
display at all. Switching to the boot console and back to X resolves the
issue. As said, disable the pipe_A quirk and we are good.
A second observation is that the boot console now reports PIPE_A (and
sometimes also) PIPE_B underruns during the bootstrap, only once. The
system recovers from this (if you call a dead boot console as above
"recovery"), so something is likely broken with the quirk. (No news, of
course).
Greetings,
Thomas
next prev parent reply other threads:[~2014-05-29 14:20 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-16 14:02 830GM still woes Thomas Richter
2014-05-16 14:41 ` Chris Wilson
2014-05-16 15:09 ` Daniel Vetter
2014-05-16 16:04 ` Ville Syrjälä
2014-05-16 16:50 ` Daniel Vetter
[not found] ` <23914_1400259040_537641E0_23914_9298_1_20140516165034.GT8790@phenom.ffwll.local>
2014-05-17 10:58 ` Thomas Richter
2014-05-29 14:10 ` Thomas Richter [this message]
2014-05-29 21:19 ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-02 8:27 ` Daniel Vetter
2014-06-02 10:34 ` [PATCH] Align i830 watermark to cache lines Thomas Richter
2014-06-02 10:41 ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-02 15:27 ` Daniel Vetter
2014-06-02 15:38 ` [Patch] Disabling the pipe A quirk for the Fujitsu S6010 Thomas Richter
2014-06-02 15:56 ` Daniel Vetter
2014-06-02 16:52 ` Thomas Richter
2014-06-02 17:39 ` Daniel Vetter
2014-06-02 18:44 ` Thomas Richter
[not found] ` <1027_1401722832_538C97D0_1027_15897_1_20140602152702.GU19050@phenom.ffwll.local>
2014-06-03 14:38 ` Breaking suspend/resume by the Pipe A quirk Thomas Richter
2014-06-03 14:45 ` Daniel Vetter
2014-06-03 15:04 ` Thomas Richter
2014-06-03 15:14 ` Chris Wilson
2014-06-03 15:19 ` Thomas Richter
2014-06-03 15:26 ` Chris Wilson
2014-06-03 15:50 ` Thomas Richter
2014-06-03 16:03 ` Chris Wilson
[not found] ` <17205_1401811442_538DF1F2_17205_16362_1_20140603160352.GA6129@nuc-i3427.alporthouse.com>
2014-06-04 22:43 ` Bug: Pipe A underrun on resolution switching on 830MG Thomas Richter
2014-06-08 21:20 ` Broken suspend/resume on i830 (with debug information) Thomas Richter
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=53873FE3.8080704@rus.uni-stuttgart.de \
--to=richter@rus.uni-stuttgart.de \
--cc=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.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