[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

> I'd rather not have a sysfs entry. I'd prefer a mesa env var or drirc > option to force the CS flag. Yeah, that actually sounds like a good idea. I think we could add a new dword to the flags chunk for this, or use the priority field which is unused for UVD on those older chipsets anyway. If

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Am 15.08.2014 um 17:32 schrieb Grigori Goronzy: > On 15.08.2014 17:26, Alex Deucher wrote: >> On Fri, Aug 15, 2014 at 11:20 AM, Grigori Goronzy >> wrote: >>> On 15.08.2014 16:11, Christian K?nig wrote: Hi Marco, the problem with an CS ioctl flag is that we sometimes don't know how

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On 15.08.2014 18:43, Marco Benatto wrote: > Looking at radeon_uvd source code and AFAIK it's currently disable > since commit dca5086a90c9ec64f4e0de801a659508202b0640 >

Fwd: Re: [PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

For reference, I forgot to send to you all. Forwarded Message Subject: Re: [PATCH] drm/radeon: Adding UVD handle basis fps estimation v2 Date: Fri, 15 Aug 2014 18:40:02 +0200 From: Grigori Goronzy <g...@chown.ath.cx> To: Alex Deucher On 15.08.2014 17:45, Alex Deucher

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On 15.08.2014 17:26, Alex Deucher wrote: > On Fri, Aug 15, 2014 at 11:20 AM, Grigori Goronzy > wrote: >> On 15.08.2014 16:11, Christian K?nig wrote: >>> Hi Marco, >>> >>> the problem with an CS ioctl flag is that we sometimes don't know how >>> much SCLK/MCLK boost is needed, for example when we

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On 15.08.2014 16:11, Christian K?nig wrote: > Hi Marco, > > the problem with an CS ioctl flag is that we sometimes don't know how > much SCLK/MCLK boost is needed, for example when we do post processing > in the player using OpenGL and UVD decoding with VDPAU. In this case > VDPAU don't has the

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Hi Marco, the problem with an CS ioctl flag is that we sometimes don't know how much SCLK/MCLK boost is needed, for example when we do post processing in the player using OpenGL and UVD decoding with VDPAU. In this case VDPAU don't has the slightest idea how high SCLK/MCLK must be and so

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Ok, I'll start the changes this night (BRT) taking 3.18-wip as basis. In case of any doubt I'll return to you guys. Thanks for the help On Fri, Aug 15, 2014 at 2:33 PM, Christian K?nig wrote: > I'd rather not have a sysfs entry. I'd prefer a mesa env var or drirc >> option to force the CS

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Grigori, Alex and Christian are you ok if I merge ioctl flag idea with sysfs idea? We let the system decide the state using the hint provided by CS ioctl flag but if performance is not good as expected or DPM table is not sane we still will have a fallback way o override this decision. On Fri,

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Looking at radeon_uvd source code and AFAIK it's currently disable since commit dca5086a90c9ec64f4e0de801a659508202b0640 Please, correct me if I'm

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Fri, Aug 15, 2014 at 1:03 PM, Marco Benatto wrote: > Grigori, Alex and Christian > > are you ok if I merge ioctl flag idea with sysfs idea? > > We let the system decide the state using the hint provided by CS ioctl flag > but if performance is not good as expected > or DPM table is not sane we

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Fri, Aug 15, 2014 at 12:46 PM, Grigori Goronzy wrote: > For reference, I forgot to send to you all. > > Forwarded Message > Subject: Re: [PATCH] drm/radeon: Adding UVD handle basis fps estimation v2 > Date: Fri, 15 Aug 2014 18:40:02 +0200 > From: Grigori G

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Hi Christian, got it. I'll start to add a sysfs entry here called uvd_power_level where we'll be able to change UVD performance profile, ok? I'll need your help to define the power profiles and some one to test it on r600-SI as I don't have anyone around here. Thank you. On Fri, Aug 15, 2014

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Fri, Aug 15, 2014 at 10:11 AM, Christian K?nig wrote: > Hi Marco, > > the problem with an CS ioctl flag is that we sometimes don't know how much > SCLK/MCLK boost is needed, for example when we do post processing in the > player using OpenGL and UVD decoding with VDPAU. In this case VDPAU

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Fri, Aug 15, 2014 at 11:20 AM, Grigori Goronzy wrote: > On 15.08.2014 16:11, Christian K?nig wrote: >> Hi Marco, >> >> the problem with an CS ioctl flag is that we sometimes don't know how >> much SCLK/MCLK boost is needed, for example when we do post processing >> in the player using OpenGL

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Hi guys, to make a long story short every time I watch a movie my laptop start to heat up because we always select the standard UVD power profile without actually measuring if that is necessary. Marco came up with a patch that seems to reliable measure the fps send down to the kernel and so

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Hey all, I also had a talk with Alex yesterday about post-processing issues when using dynamic UVD profiles and a chamge on CS ioctl including a flag to let user mode driver tell to the kernel which performance requirement it wants for post processing. A commom point for both discussion is to

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Am 11.08.2014 um 16:52 schrieb Alex Deucher: > On Mon, Aug 11, 2014 at 5:08 AM, Christian K?nig > wrote: >> Am 07.08.2014 um 21:43 schrieb Alex Deucher: >> >>> On Thu, Aug 7, 2014 at 11:32 AM, Christian K?nig >>> wrote: Am 07.08.2014 um 16:32 schrieb Alex Deucher: > On Thu, Aug 7,

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Tue, Aug 12, 2014 at 6:00 AM, Christian K?nig wrote: > Am 11.08.2014 um 16:52 schrieb Alex Deucher: > >> On Mon, Aug 11, 2014 at 5:08 AM, Christian K?nig >> wrote: >>> >>> Am 07.08.2014 um 21:43 schrieb Alex Deucher: >>> On Thu, Aug 7, 2014 at 11:32 AM, Christian K?nig wrote: >

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Am 07.08.2014 um 21:43 schrieb Alex Deucher: > On Thu, Aug 7, 2014 at 11:32 AM, Christian K?nig > wrote: >> Am 07.08.2014 um 16:32 schrieb Alex Deucher: >> >>> On Thu, Aug 7, 2014 at 7:33 AM, Christian K?nig >>> wrote: From: Marco A Benatto Adding a Frames Per Second estimation

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Mon, Aug 11, 2014 at 5:08 AM, Christian K?nig wrote: > Am 07.08.2014 um 21:43 schrieb Alex Deucher: > >> On Thu, Aug 7, 2014 at 11:32 AM, Christian K?nig >> wrote: >>> >>> Am 07.08.2014 um 16:32 schrieb Alex Deucher: >>> On Thu, Aug 7, 2014 at 7:33 AM, Christian K?nig wrote:

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

Am 07.08.2014 um 16:32 schrieb Alex Deucher: > On Thu, Aug 7, 2014 at 7:33 AM, Christian K?nig > wrote: >> From: Marco A Benatto >> >> Adding a Frames Per Second estimation logic on UVD handles >> when it has being used. This estimation is per handle basis >> and will help on DPM profile

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Thu, Aug 7, 2014 at 11:32 AM, Christian K?nig wrote: > Am 07.08.2014 um 16:32 schrieb Alex Deucher: > >> On Thu, Aug 7, 2014 at 7:33 AM, Christian K?nig >> wrote: >>> >>> From: Marco A Benatto >>> >>> Adding a Frames Per Second estimation logic on UVD handles >>> when it has being used. This

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

From: Marco A Benatto Adding a Frames Per Second estimation logic on UVD handles when it has being used. This estimation is per handle basis and will help on DPM profile calculation. v2 (chk): fix timestamp type, move functions around and cleanup code a

[PATCH] drm/radeon: Adding UVD handle basis fps estimation v2

On Thu, Aug 7, 2014 at 7:33 AM, Christian K?nig wrote: > From: Marco A Benatto > > Adding a Frames Per Second estimation logic on UVD handles > when it has being used. This estimation is per handle basis > and will help on DPM profile calculation. > > v2 (chk): fix timestamp type, move