Re: [PATCH 0/3] Hack to test DRRS, independent of front buffer tracking

Daniel Vetter
2015-03-24T10:51:23Z

On Tue, Mar 24, 2015 at 03:17:35PM +0530, Ramalingam C wrote:
>
> On Tuesday 24 March 2015 03:05 PM, Daniel Vetter wrote:
> >On Mon, Mar 23, 2015 at 11:14:41PM +0530, Ramalingam C wrote:
> >>These patches removes the dependancy of DRRS with front buffer tracking.
> >>This will help us to test the DRRS and the IGT app for DRRS.
> >>
> >>As we have some functional issues with front buffer tracking, We are
> >>adding a debugfs, so that IGT can notify the content change to DRRS.
> >>Once the front buffer tracking is fully functional these patches are not
> >>required.
> >Nope, we need to track down the bugs in frontbuffer tracking and use the
> >existing interface to kick updates to the kernel. Yes I know this is hard,
> >but we've been struggling with frontbuffer tracking issues with fbc, psr
> >and drrs for years, adding more hacks will not magically resolve them.
> >-Daniel
> Daniel, I completely I agree with you. These patch series is not meant to
> be reviewed or merged.
> Intention of these patch creation is to enable the DRRS so that we develop
> the DRRS test cases.
> With these test cases it will be easy to narrow down on the missing fb
> status update call.

Ah, sounds excellent and makes sense. Usually I mention this in the cover
letter to avoid ppl reviewing something that I don't intend to get merged.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation