On Tue, Sep 02, 2014 at 04:17:34PM +0300, Ville Syrjälä wrote: > On Tue, Sep 02, 2014 at 01:45:29PM +0100, Chris Wilson wrote: > > On Tue, Sep 02, 2014 at 03:12:17PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > > > If the GPU frequency isn't going to change don't spam dmesg with > > > debug messages about it. > > > > I'd be more worried about why. gen6_rps_idle()? > > Hmm. Good question. I suppose the PMINTRMSK frobbing should prevent > it from doing that. rps_idle should not be involved since it tries to > set the max freq all the time when running some benchmark. I'll try > to investigate a bit more... gen6_rps_boost() is one culprit at least. So yeah I think we want this patch, or maybe even make it DRM_DEBUG() to shut it up more? Or maybe just kill it since we don't print this stuff for other platforms either? -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
