Stutter can lead to a fair bit of instability so an option to disable/enable to debug is quite important.
I don't have a strong preference on how this option should look. ppfeaturemask should work well. Harry On 2018-06-26 10:26 AM, Deucher, Alexander wrote: > I was going to say the same thing. I think it would be better to add it to > the ppfeature mask for consistency. > > > Alex > > ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ > *From:* amd-gfx <[email protected]> on behalf of Zhu, Rex > <[email protected]> > *Sent:* Tuesday, June 26, 2018 7:10:52 AM > *To:* Michel Dänzer > *Cc:* [email protected] > *Subject:* Re: [PATCH 1/2] drm/amdgpu: Add module parameter stutter_mode > > Evev when enabled this feature by default,we still need a way to disable it > easily for triage issue maybe related to stutter mode. > > as this feature is related to power saving. so we may add a bit mask in > parameter ppfeaturemask. > > Best Regards > Rex > > 获取 Outlook for Android <https://aka.ms/ghei36> > > ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ > *From:* amd-gfx <[email protected]> on behalf of Michel > Dänzer <[email protected]> > *Sent:* Tuesday, June 26, 2018 6:38:14 PM > *To:* Zhu, Rex > *Cc:* Zhu, Rex; [email protected] > *Subject:* Re: [PATCH 1/2] drm/amdgpu: Add module parameter stutter_mode > > On 2018-06-26 12:23 PM, rex zhu wrote: >> Add new parameter to control stutter mode enable/disable. >> >> Enable stutter mode can save power in low DRAM use cases >> including but not limited to productivity application use, >> web browsing, and video playback. > > We already have too many module parameters. Do we really need yet > another one for this? Can't stutter mode just automatically be enabled > when appropriate? > > > -- > Earthling Michel Dänzer | http://www.amd.com > Libre software enthusiast | Mesa and X developer > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx > > > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx > _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
