[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-05 Thread Lauri Kasanen
On Mon, 4 Jun 2012 13:05:46 -0400 Jerome Glisse wrote: > My dream here is to talk with the gnome folks to have them make some > kind GPU module we could write and that would show in control center. > I just need to corner some of my gnome coworker to work something out. > So if you were using nou

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-05 Thread Lauri Kasanen
On Mon, 4 Jun 2012 13:05:46 -0400 Jerome Glisse wrote: > My dream here is to talk with the gnome folks to have them make some > kind GPU module we could write and that would show in control center. > I just need to corner some of my gnome coworker to work something out. > So if you were using nou

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
On 04.06.2012 21:54, Daniel Vetter wrote: > > In i915-land we're trying to make things Just Work. If needed we can > expose (generation/platform-specific) tunables in sysfs. But on snb and > later the combination of rc6+gpu turbo (mostly handled all by hw) is > rather ok, so I don't see anything go

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
On 04.06.2012 20:29, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 2:18 PM, Jerome Glisse wrote: >>> >>> Yeah, I get your point as a kernel dev, but I pitty the userspace dev that >>> will need to figure out how to use all these ioctls and configuration >>> options. >> My point there is that we do

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Daniel Vetter
On Mon, Jun 04, 2012 at 02:18:00PM -0400, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: > > Answers inlined. > > > > Le 04/06/2012 19:19, Jerome Glisse a ?crit : > > > >> > >> My point is that there is no way for power management to find an API > >> that fits all GPU.

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Answers inlined. Le 04/06/2012 19:19, Jerome Glisse a ?crit : > > My point is that there is no way for power management to find an API > that fits all GPU. If i were to do it now, i would have one ioctl > version for r3xx, one for r5xx, one for r6xx/r7xx, one for r8xx, one > for r9xx, ... yes ther

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Lauri Kasanen
On Mon, 04 Jun 2012 18:18:10 +0200 Christian K?nig wrote: > > My experience is that things that are true today for GPU, are not > > tomorrow. Yes there will still be clock/voltage, but there could be > > new complete different things, like shutting down block. > > > > I am not even mentioning thin

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 17:18, Jerome Glisse a ?crit : > > My experience is that things that are true today for GPU, are not > tomorrow. Yes there will still be clock/voltage, but there could be > new complete different things, like shutting down block. IMO, this isn't something the user should ever be aware

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Christian König
On 04.06.2012 17:18, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 11:02 AM, Martin Peres wrote: >> Le 04/06/2012 16:31, Jerome Glisse a ?crit : >> >>> I don't think sysfs is the way to go, i am pretty sure that power >>> management will change drasticly again in the future especialy btw >>> discr

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 16:31, Jerome Glisse a ?crit : > > I don't think sysfs is the way to go, i am pretty sure that power > management will change drasticly again in the future especialy btw > discret and integrated GPU. I would rather have hardware specific > ioctl. > > Cheers, > Jerome Any particular id

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 13:30, Christian K?nig a ?crit : > On 04.06.2012 10:44, Lauri Kasanen wrote: >> So the issue is the location of the info, not the format. I'd be more >> than happy to split it into six files (default_core_clock, >> current_core_clock...) that each offer just a kHz number, just like

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 2:18 PM, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: >> Answers inlined. >> >> Le 04/06/2012 19:19, Jerome Glisse a ?crit : >> >>> >>> My point is that there is no way for power management to find an API >>> that fits all GPU. If i were to do

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: > Answers inlined. > > Le 04/06/2012 19:19, Jerome Glisse a ?crit : > >> >> My point is that there is no way for power management to find an API >> that fits all GPU. If i were to do it now, i would have one ioctl >> version for r3xx, one for r5x

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
On 04.06.2012 21:54, Daniel Vetter wrote: In i915-land we're trying to make things Just Work. If needed we can expose (generation/platform-specific) tunables in sysfs. But on snb and later the combination of rc6+gpu turbo (mostly handled all by hw) is rather ok, so I don't see anything going abo

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
On 04.06.2012 20:29, Jerome Glisse wrote: On Mon, Jun 4, 2012 at 2:18 PM, Jerome Glisse wrote: Yeah, I get your point as a kernel dev, but I pitty the userspace dev that will need to figure out how to use all these ioctls and configuration options. My point there is that we do the userspace b

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Christian König
On 04.06.2012 10:44, Lauri Kasanen wrote: > On Sun, 03 Jun 2012 12:54:30 +0200 > Christian K?nig wrote: > >>> This moves the pm_info file from debugfs to next to the other two power >>> files. >>> >>> Requested by several users at Phoronix. >>> >>> PS: Please CC me. Also please be gentle, it's my

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 1:01 PM, Martin Peres wrote: > Le 04/06/2012 17:18, Jerome Glisse a ?crit : > >> >> My experience is that things that are true today for GPU, are not >> tomorrow. Yes there will still be clock/voltage, but there could be >> new complete different things, like shutting down b

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 12:29 PM, Lauri Kasanen wrote: > On Mon, 04 Jun 2012 18:18:10 +0200 > Christian K?nig wrote: >> > My experience is that things that are true today for GPU, are not >> > tomorrow. Yes there will still be clock/voltage, but there could be >> > new complete different things, l

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Daniel Vetter
On Mon, Jun 04, 2012 at 02:18:00PM -0400, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: > > Answers inlined. > > > > Le 04/06/2012 19:19, Jerome Glisse a écrit : > > > >> > >> My point is that there is no way for power management to find an API > >> that fits all GPU.

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Lauri Kasanen
On Sun, 03 Jun 2012 12:54:30 +0200 Christian K?nig wrote: > > This moves the pm_info file from debugfs to next to the other two power > > files. > > > > Requested by several users at Phoronix. > > > > PS: Please CC me. Also please be gentle, it's my first step in kernel-land > > ;) > Hui? What

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 2:18 PM, Jerome Glisse wrote: > On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: >> Answers inlined. >> >> Le 04/06/2012 19:19, Jerome Glisse a écrit : >> >>> >>> My point is that there is no way for power management to find an API >>> that fits all GPU. If i were to do

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 1:54 PM, Martin Peres wrote: > Answers inlined. > > Le 04/06/2012 19:19, Jerome Glisse a écrit : > >> >> My point is that there is no way for power management to find an API >> that fits all GPU. If i were to do it now, i would have one ioctl >> version for r3xx, one for r5x

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 11:02 AM, Martin Peres wrote: > Le 04/06/2012 16:31, Jerome Glisse a ?crit : > >> >> I don't think sysfs is the way to go, i am pretty sure that power >> management will change drasticly again in the future especialy btw >> discret and integrated GPU. I would rather have har

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Answers inlined. Le 04/06/2012 19:19, Jerome Glisse a écrit : My point is that there is no way for power management to find an API that fits all GPU. If i were to do it now, i would have one ioctl version for r3xx, one for r5xx, one for r6xx/r7xx, one for r8xx, one for r9xx, ... yes there would

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 8:40 AM, Martin Peres wrote: > Le 04/06/2012 13:30, Christian K?nig a ?crit : >> >> On 04.06.2012 10:44, Lauri Kasanen wrote: >>> >>> So the issue is the location of the info, not the format. I'd be more >>> than happy to split it into six files (default_core_clock, >>> curr

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 1:01 PM, Martin Peres wrote: > Le 04/06/2012 17:18, Jerome Glisse a écrit : > >> >> My experience is that things that are true today for GPU, are not >> tomorrow. Yes there will still be clock/voltage, but there could be >> new complete different things, like shutting down b

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 12:29 PM, Lauri Kasanen wrote: > On Mon, 04 Jun 2012 18:18:10 +0200 > Christian König wrote: >> > My experience is that things that are true today for GPU, are not >> > tomorrow. Yes there will still be clock/voltage, but there could be >> > new complete different things, l

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 17:18, Jerome Glisse a écrit : My experience is that things that are true today for GPU, are not tomorrow. Yes there will still be clock/voltage, but there could be new complete different things, like shutting down block. IMO, this isn't something the user should ever be aware of.

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Lauri Kasanen
On Mon, 04 Jun 2012 18:18:10 +0200 Christian König wrote: > > My experience is that things that are true today for GPU, are not > > tomorrow. Yes there will still be clock/voltage, but there could be > > new complete different things, like shutting down block. > > > > I am not even mentioning thin

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Christian König
On 04.06.2012 17:18, Jerome Glisse wrote: On Mon, Jun 4, 2012 at 11:02 AM, Martin Peres wrote: Le 04/06/2012 16:31, Jerome Glisse a écrit : I don't think sysfs is the way to go, i am pretty sure that power management will change drasticly again in the future especialy btw discret and integrat

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Alex Deucher
On Mon, Jun 4, 2012 at 7:30 AM, Christian K?nig wrote: > On 04.06.2012 10:44, Lauri Kasanen wrote: >> >> On Sun, 03 Jun 2012 12:54:30 +0200 >> Christian K?nig ?wrote: >> This moves the pm_info file from debugfs to next to the other two power files. Requested by several users a

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 11:02 AM, Martin Peres wrote: > Le 04/06/2012 16:31, Jerome Glisse a écrit : > >> >> I don't think sysfs is the way to go, i am pretty sure that power >> management will change drasticly again in the future especialy btw >> discret and integrated GPU. I would rather have har

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 16:31, Jerome Glisse a écrit : I don't think sysfs is the way to go, i am pretty sure that power management will change drasticly again in the future especialy btw discret and integrated GPU. I would rather have hardware specific ioctl. Cheers, Jerome Any particular idea of what

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Jerome Glisse
On Mon, Jun 4, 2012 at 8:40 AM, Martin Peres wrote: > Le 04/06/2012 13:30, Christian König a écrit : >> >> On 04.06.2012 10:44, Lauri Kasanen wrote: >>> >>> So the issue is the location of the info, not the format. I'd be more >>> than happy to split it into six files (default_core_clock, >>> curr

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Martin Peres
Le 04/06/2012 13:30, Christian König a écrit : On 04.06.2012 10:44, Lauri Kasanen wrote: So the issue is the location of the info, not the format. I'd be more than happy to split it into six files (default_core_clock, current_core_clock...) that each offer just a kHz number, just like the cpuf

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Alex Deucher
On Mon, Jun 4, 2012 at 7:30 AM, Christian König wrote: > On 04.06.2012 10:44, Lauri Kasanen wrote: >> >> On Sun, 03 Jun 2012 12:54:30 +0200 >> Christian König  wrote: >> This moves the pm_info file from debugfs to next to the other two power files. Requested by several users at

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Christian König
On 04.06.2012 10:44, Lauri Kasanen wrote: On Sun, 03 Jun 2012 12:54:30 +0200 Christian König wrote: This moves the pm_info file from debugfs to next to the other two power files. Requested by several users at Phoronix. PS: Please CC me. Also please be gentle, it's my first step in kernel-lan

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-04 Thread Lauri Kasanen
On Sun, 03 Jun 2012 12:54:30 +0200 Christian König wrote: > > This moves the pm_info file from debugfs to next to the other two power > > files. > > > > Requested by several users at Phoronix. > > > > PS: Please CC me. Also please be gentle, it's my first step in kernel-land > > ;) > Hui? What

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-03 Thread Christian König
On 02.06.2012 18:08, Lauri Kasanen wrote: > Hi all > > This moves the pm_info file from debugfs to next to the other two power files. > > Requested by several users at Phoronix. > > PS: Please CC me. Also please be gentle, it's my first step in kernel-land ;) > > Signed-off-by: Lauri Kasanen Hui? W

Re: [PATCH] radeon: Make PM info available to all, not just debug users

2012-06-03 Thread Christian König
On 02.06.2012 18:08, Lauri Kasanen wrote: Hi all This moves the pm_info file from debugfs to next to the other two power files. Requested by several users at Phoronix. PS: Please CC me. Also please be gentle, it's my first step in kernel-land ;) Signed-off-by: Lauri Kasanen Hui? What should

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-02 Thread Lauri Kasanen
Hi all This moves the pm_info file from debugfs to next to the other two power files. Requested by several users at Phoronix. PS: Please CC me. Also please be gentle, it's my first step in kernel-land ;) Signed-off-by: Lauri Kasanen --- drivers/gpu/drm/radeon/radeon_pm.c | 86 ++

[PATCH] radeon: Make PM info available to all, not just debug users

2012-06-02 Thread Lauri Kasanen
Hi all This moves the pm_info file from debugfs to next to the other two power files. Requested by several users at Phoronix. PS: Please CC me. Also please be gentle, it's my first step in kernel-land ;) Signed-off-by: Lauri Kasanen --- drivers/gpu/drm/radeon/radeon_pm.c | 86 ++