From: Paulo Zanoni <przanoni@gmail.com>
To: intel-gfx@lists.freedesktop.org
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Subject: [PATCH 0/4] Remove useless vblank waits on Haswell
Date: Thu, 19 Sep 2013 17:07:25 -0300 [thread overview]
Message-ID: <1379621249-1816-1-git-send-email-przanoni@gmail.com> (raw)
From: Paulo Zanoni <paulo.r.zanoni@intel.com>
Hi
This series gets rid of those messages saying that the vblank wait timed out on
my Haswell machine. It saves 2 timeouts per pipe (one at mode_set, one at
crtc_enable), so it's 100ms we save per modeset.
The big idea exercised by this patch is that the pipe doesn't really start
running after we enable PIPECONF, so we had some useless waits. This idea
probably also applies to ILK/SNB/IVB and perhaps we could try to save some
vblanks on those Gens too.
I tested these patches on top of the 4 patches that fix the Haswell underruns +
the 2 IPS patches I just sent.
Thanks,
Paulo
Paulo Zanoni (4):
drm/i915: WARN in case PIPECONF is already enabled
drm/i915: don't intel_wait_for_vblank inside intel_enable_pipe
drm/i915: remove useless vblank wait form haswell_crtc_enable
drm/i915: skip useless vblank wait on Haswell audio sequence
drivers/gpu/drm/i915/intel_display.c | 22 +++++++++++++++-------
1 file changed, 15 insertions(+), 7 deletions(-)
--
1.8.3.1
next reply other threads:[~2013-09-19 20:07 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-19 20:07 Paulo Zanoni [this message]
2013-09-19 20:07 ` [PATCH 1/4] drm/i915: WARN in case PIPECONF is already enabled Paulo Zanoni
2013-09-20 6:36 ` Ville Syrjälä
2013-09-20 19:51 ` Paulo Zanoni
2013-09-19 20:07 ` [PATCH 2/4] drm/i915: don't intel_wait_for_vblank inside intel_enable_pipe Paulo Zanoni
2013-09-19 20:07 ` [PATCH 3/4] drm/i915: remove useless vblank wait form haswell_crtc_enable Paulo Zanoni
2013-09-19 20:07 ` [PATCH 4/4] drm/i915: skip useless vblank wait on Haswell audio sequence Paulo Zanoni
2013-09-19 20:28 ` Chris Wilson
2013-09-20 8:17 ` Daniel Vetter
2013-09-20 19:22 ` Paulo Zanoni
2013-09-22 10:37 ` Lin, Mengdong
2013-10-10 18:33 ` Paulo Zanoni
2013-10-10 19:07 ` Daniel Vetter
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=1379621249-1816-1-git-send-email-przanoni@gmail.com \
--to=przanoni@gmail.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=paulo.r.zanoni@intel.com \
/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