From: Daniel Vetter <daniel@ffwll.ch>
To: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>,
Rodrigo Vivi <rodrigo.vivi@intel.com>
Subject: Re: [PATCH 1/4] drm/i915: Increase PSR Idle Frame to 2.
Date: Mon, 15 Sep 2014 11:08:40 +0200 [thread overview]
Message-ID: <20140915090840.GM4740@phenom.ffwll.local> (raw)
In-Reply-To: <CABVU7+vUqzFBSAnNs_Ox1MU=SwON+MCA5fEgD_wB2_S35gyP-Q@mail.gmail.com>
On Fri, Sep 12, 2014 at 05:29:44PM -0700, Rodrigo Vivi wrote:
> Please ignore this full series here.
>
> I have a better one that let PSR on HSW in a better stage with only 1 idle
> frame and without changing the 100ms. Actually if we are brave we can
> reduce this to 24 or less. The new work is currently under
> http://cgit.freedesktop.org/~vivijim/drm-intel/log/?h=psr-3.18
> However I'm not going to send yet because on BDW it got works. It seems BDW
> doesn't like to get PSR enabled immediatelly. So I'm justs sendint new
> series after I'm confortable that PSR is in a better stage for both HSW and
> BDW.
How does this interact with the improved psr crc capture code? And if we
aim for perfect we should be able to get the delay for the reenable for
down to 0. 24ms is still about 1.5 frames ...
But sounding much better indeed now, nice work!
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
prev parent reply other threads:[~2014-09-15 9:08 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-04 2:49 [PATCH 1/4] drm/i915: Increase PSR Idle Frame to 2 Rodrigo Vivi
2014-09-04 2:49 ` [PATCH 2/4] drm/i915: Remove PSR HW tracking Rodrigo Vivi
[not found] ` <20140904080643.GC4193@intel.com>
2014-09-04 19:53 ` Rodrigo Vivi
2014-09-05 9:14 ` Ville Syrjälä
2014-09-04 2:49 ` [PATCH 3/4] drm/i915: Move PSR w/as to enable source Rodrigo Vivi
2014-09-04 2:49 ` [PATCH 4/4] drm/i915: Rename psr setup vsc function and set it always Rodrigo Vivi
[not found] ` <20140904075516.GB4193@intel.com>
2014-09-04 18:18 ` [PATCH 1/4] drm/i915: Increase PSR Idle Frame to 2 Rodrigo Vivi
[not found] ` <20140904092916.GF15520@phenom.ffwll.local>
[not found] ` <20140904100427.GD4193@intel.com>
[not found] ` <20140904101819.GE4193@intel.com>
2014-09-04 11:04 ` Daniel Vetter
2014-09-04 18:26 ` Rodrigo Vivi
2014-09-04 18:22 ` Rodrigo Vivi
[not found] ` <87wq9jg62e.fsf@intel.com>
2014-09-04 18:18 ` Rodrigo Vivi
[not found] ` <20140904092253.GD15520@phenom.ffwll.local>
2014-09-04 18:20 ` Rodrigo Vivi
2014-09-04 19:05 ` Daniel Vetter
2014-09-04 19:16 ` Rodrigo Vivi
2014-09-04 20:20 ` Daniel Vetter
2014-09-05 0:40 ` Rodrigo Vivi
2014-09-05 8:37 ` Daniel Vetter
2014-09-13 0:29 ` Rodrigo Vivi
2014-09-15 9:08 ` Daniel Vetter [this message]
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=20140915090840.GM4740@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@gmail.com \
--cc=rodrigo.vivi@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