* [PATCH 0/7] More PSR patches [not found] <1440119053-26535-1-git-send-email-rodrigo.vivi@intel.com> @ 2015-08-21 1:04 ` Rodrigo Vivi 2015-08-21 14:06 ` Zanoni, Paulo R 0 siblings, 1 reply; 3+ messages in thread From: Rodrigo Vivi @ 2015-08-21 1:04 UTC (permalink / raw) To: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 1087 bytes --] This patch series brings stability to PSR on VLV, CHV, HSW and BDW. It fixes issues Matthew Garrett without causing the blank and frozen screens Ivan Mitev was facing. It also move the VLV/CHV workaround of that big delay from re-enable to the first enable after modeset that was the real issue. With this besides the stability this series brings more PSR residency to these platforms. However the enable by default is still out of this series and will come only after an intensive QA. Thanks, Rodrigo. Rodrigo Vivi (7): drm/i915: Remove duplicated dpcd write on hsw_psr_enable_sink. drm/i915: Fix PSR disable sequence on core platforms. drm/i915: PSR: Let's rely more on frontbuffer tracking. drm/i915: PSR: Mask LPSP hw tracking back again. drm/i915: Delay first PSR activation. drm/i915: Remove psr re-activation delay on HSW+. drm/i915: Reduce PSR re-activation time for VLV/CHV. drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_psr.c | 75 +++++++++++++++++++++++++--------------- 2 files changed, 49 insertions(+), 27 deletions(-) -- 2.4.3 [-- Attachment #1.2: Type: text/html, Size: 1329 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/7] More PSR patches 2015-08-21 1:04 ` [PATCH 0/7] More PSR patches Rodrigo Vivi @ 2015-08-21 14:06 ` Zanoni, Paulo R 2015-08-21 16:10 ` Rodrigo Vivi 0 siblings, 1 reply; 3+ messages in thread From: Zanoni, Paulo R @ 2015-08-21 14:06 UTC (permalink / raw) To: intel-gfx@lists.freedesktop.org, rodrigo.vivi@gmail.com Em Sex, 2015-08-21 às 01:04 +0000, Rodrigo Vivi escreveu: > This patch series brings stability to PSR on VLV, CHV, HSW and BDW. > > It fixes issues Matthew Garrett without causing the blank and frozen > screens Ivan Mitev was facing. > > It also move the VLV/CHV workaround of that big delay from re-enable > to the first enable after modeset that was the real issue. > With this besides the stability this series brings more PSR > residency to these platforms. I hate to be "that guy", but: no new IGT tests for anything? At least on my machine, all PSR tests pass without your series, so maybe we could write some new tests. Some commits mention bugs, but they don't even teach us how to reproduce the bugs. > > However the enable by default is still out of this series and will > come > only after an intensive QA. > > Thanks, > Rodrigo. > > Rodrigo Vivi (7): > drm/i915: Remove duplicated dpcd write on hsw_psr_enable_sink. > drm/i915: Fix PSR disable sequence on core platforms. > drm/i915: PSR: Let's rely more on frontbuffer tracking. > drm/i915: PSR: Mask LPSP hw tracking back again. > drm/i915: Delay first PSR activation. > drm/i915: Remove psr re-activation delay on HSW+. > drm/i915: Reduce PSR re-activation time for VLV/CHV. > > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/intel_psr.c | 75 +++++++++++++++++++++++++----- > ---------- > 2 files changed, 49 insertions(+), 27 deletions(-) > > -- > 2.4.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/7] More PSR patches 2015-08-21 14:06 ` Zanoni, Paulo R @ 2015-08-21 16:10 ` Rodrigo Vivi 0 siblings, 0 replies; 3+ messages in thread From: Rodrigo Vivi @ 2015-08-21 16:10 UTC (permalink / raw) To: Zanoni, Paulo R, intel-gfx@lists.freedesktop.org [-- Attachment #1.1: Type: text/plain, Size: 2643 bytes --] On Fri, Aug 21, 2015 at 7:06 AM Zanoni, Paulo R <paulo.r.zanoni@intel.com> wrote: > > Em Sex, 2015-08-21 às 01:04 +0000, Rodrigo Vivi escreveu: > > This patch series brings stability to PSR on VLV, CHV, HSW and BDW. > > > > It fixes issues Matthew Garrett without causing the blank and frozen > > screens Ivan Mitev was facing. > > > > It also move the VLV/CHV workaround of that big delay from re-enable > > to the first enable after modeset that was the real issue. > > With this besides the stability this series brings more PSR > > residency to these platforms. > > I hate to be "that guy", no worries, you are absolutely right. > but: no new IGT tests for anything? Unfortunately no new test. I couldn't reproduce with igt neither of the 2 bugs this series fixes. > At least > on my machine, all PSR tests pass without your series, so maybe we > could write some new tests. > yes, it was passing all ltests and are still passing, but I couldn't reproduce with igt the new cases. > > Some commits mention bugs, but they don't even teach us how to > reproduce the bugs. > 1 is mentioned on the bugzilla, just right a new modeset when psr was running already it get unrecoverable lock screen. On igt the combination simple works, but on the distro flow, any screen off-on will trigger it. 2 is mentioned on the commit but also I coulnd't simulate that on igt, right after modeset after dpms off-on keep moving your mouse fast and you wont receive any screen update while you are moving your mouse... then you stop moving and when you move again the screen is updated... > > > > > However the enable by default is still out of this series and will > > come > > only after an intensive QA. > > > > Thanks, > > Rodrigo. > > > > Rodrigo Vivi (7): > > drm/i915: Remove duplicated dpcd write on hsw_psr_enable_sink. > > drm/i915: Fix PSR disable sequence on core platforms. > > drm/i915: PSR: Let's rely more on frontbuffer tracking. > > drm/i915: PSR: Mask LPSP hw tracking back again. > > drm/i915: Delay first PSR activation. > > drm/i915: Remove psr re-activation delay on HSW+. > > drm/i915: Reduce PSR re-activation time for VLV/CHV. > > > > drivers/gpu/drm/i915/i915_drv.h | 1 + > > drivers/gpu/drm/i915/intel_psr.c | 75 +++++++++++++++++++++++++----- > > ---------- > > 2 files changed, 49 insertions(+), 27 deletions(-) > > > > -- > > 2.4.3 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx [-- Attachment #1.2: Type: text/html, Size: 3904 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-21 16:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1440119053-26535-1-git-send-email-rodrigo.vivi@intel.com>
2015-08-21 1:04 ` [PATCH 0/7] More PSR patches Rodrigo Vivi
2015-08-21 14:06 ` Zanoni, Paulo R
2015-08-21 16:10 ` Rodrigo Vivi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.