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 http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx