All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/22] new radeon info queries
@ 2014-10-01 15:38 Alex Deucher
  2014-12-21 20:05 ` Siavash Eliasi
  2015-02-24 22:09 ` Marek Olšák
  0 siblings, 2 replies; 9+ messages in thread
From: Alex Deucher @ 2014-10-01 15:38 UTC (permalink / raw)
  To: dri-devel; +Cc: Alex Deucher

This patch set implements a number of new
radeon info ioctl queries to support additional
profiling parameters for things like the GALLIUM
HUD.  It could also be used by apps like radeontop
to query the status registers, etc. without requiring
root privileges or needing asis specific knowledge
with respect to register BARs.

Alex Deucher (22):
  drm/radeon: add INFO query for GPU temperature
  drm/radeon/dpm: add new callbacks to get the current sclk/mclk
  drm/radeon/rs780: implement get_current_sclk/mclk
  drm/radeon/rv6xx: implement get_current_sclk/mclk
  drm/radeon/rv7xx/eg: implement get_current_sclk/mclk
  drm/radeon/btc: implement get_current_sclk/mclk
  drm/radeon: remove some rv7xx leftovers from btc dpm code
  drm/radeon/ni: implement get_current_sclk/mclk
  drm/radeon/si: implement get_current_sclk/mclk
  drm/radeon/ci: implement get_current_sclk/mclk
  drm/radeon/sumo: implement get_current_sclk/mclk
  drm/radeon/tn: implement get_current_sclk/mclk
  drm/radeon/kv: implement get_current_sclk/mclk
  drm/radeon: add INFO query for current sclk/mclk
  drm/radeon: add new callback for info ioctl register accessor
  drm/radeon: add get_allowed_info_register function for r1xx-r5xx
  drm/radeon: add get_allowed_info_register for r6xx/r7xx
  drm/radeon: add get_allowed_info_register for EG/BTC
  drm/radeon: add get_allowed_info_register for cayman/TN
  drm/radeon: add get_allowed_info_register for SI
  drm/radeon: add get_allowed_info_register for CIK
  drm/radeon: add support for read reg query from radeon info ioctl

 drivers/gpu/drm/radeon/btc_dpm.c     | 55 +++++++++++++++++++++++++++++++-----
 drivers/gpu/drm/radeon/ci_dpm.c      | 14 +++++++++
 drivers/gpu/drm/radeon/cik.c         | 33 ++++++++++++++++++++++
 drivers/gpu/drm/radeon/cikd.h        |  2 ++
 drivers/gpu/drm/radeon/evergreen.c   | 28 ++++++++++++++++++
 drivers/gpu/drm/radeon/evergreend.h  |  1 +
 drivers/gpu/drm/radeon/kv_dpm.c      | 23 +++++++++++++++
 drivers/gpu/drm/radeon/ni.c          | 29 +++++++++++++++++++
 drivers/gpu/drm/radeon/ni_dpm.c      | 36 +++++++++++++++++++++++
 drivers/gpu/drm/radeon/nid.h         |  1 +
 drivers/gpu/drm/radeon/r600.c        | 26 +++++++++++++++++
 drivers/gpu/drm/radeon/radeon.h      |  7 +++++
 drivers/gpu/drm/radeon/radeon_asic.c | 49 ++++++++++++++++++++++++++++++++
 drivers/gpu/drm/radeon/radeon_asic.h | 30 ++++++++++++++++++++
 drivers/gpu/drm/radeon/radeon_kms.c  | 29 +++++++++++++++++++
 drivers/gpu/drm/radeon/rs780_dpm.c   | 22 +++++++++++++++
 drivers/gpu/drm/radeon/rv6xx_dpm.c   | 46 ++++++++++++++++++++++++++++++
 drivers/gpu/drm/radeon/rv770_dpm.c   | 44 +++++++++++++++++++++++++++++
 drivers/gpu/drm/radeon/si.c          | 30 ++++++++++++++++++++
 drivers/gpu/drm/radeon/si_dpm.c      | 36 +++++++++++++++++++++++
 drivers/gpu/drm/radeon/sid.h         |  1 +
 drivers/gpu/drm/radeon/sumo_dpm.c    | 28 ++++++++++++++++++
 drivers/gpu/drm/radeon/trinity_dpm.c | 25 ++++++++++++++++
 include/uapi/drm/radeon_drm.h        |  4 +++
 24 files changed, 592 insertions(+), 7 deletions(-)

-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
@ 2014-10-01 16:30 Alexandre Demers
  2014-10-02  4:50 ` Alex Deucher
  0 siblings, 1 reply; 9+ messages in thread
From: Alexandre Demers @ 2014-10-01 16:30 UTC (permalink / raw)
  To: dri-devel

May I suggest something more to add if available? It would be great to
have the core and memory voltages. As an example of a specific
application where it would have been usefull: it would have been
easier to debug the problem with Cayman when vddci was not correctly
set (it would have been higher or lower than expected or the value
would have been nonsense).

Thanks,
Alexandre Demers

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-10-01 16:30 [PATCH 00/22] new radeon info queries Alexandre Demers
@ 2014-10-02  4:50 ` Alex Deucher
  2014-10-02 12:41   ` Christian König
  0 siblings, 1 reply; 9+ messages in thread
From: Alex Deucher @ 2014-10-02  4:50 UTC (permalink / raw)
  To: Alexandre Demers; +Cc: dri-devel

On Wed, Oct 1, 2014 at 12:30 PM, Alexandre Demers
<alexandre.f.demers@gmail.com> wrote:
> May I suggest something more to add if available? It would be great to
> have the core and memory voltages. As an example of a specific
> application where it would have been usefull: it would have been
> easier to debug the problem with Cayman when vddci was not correctly
> set (it would have been higher or lower than expected or the value
> would have been nonsense).

It wouldn't have helped for that problem since there's no way to query
the actual vddc or vddci or even the clocks.  All you can query is
which performance level is active, the driver has to determine the
clocks and voltages based on that.  It's basically the same info that
is already exposed via debugfs.

Alex

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-10-02  4:50 ` Alex Deucher
@ 2014-10-02 12:41   ` Christian König
  0 siblings, 0 replies; 9+ messages in thread
From: Christian König @ 2014-10-02 12:41 UTC (permalink / raw)
  To: Alex Deucher, Alexandre Demers; +Cc: dri-devel

Am 02.10.2014 um 06:50 schrieb Alex Deucher:
> On Wed, Oct 1, 2014 at 12:30 PM, Alexandre Demers
> <alexandre.f.demers@gmail.com> wrote:
>> May I suggest something more to add if available? It would be great to
>> have the core and memory voltages. As an example of a specific
>> application where it would have been usefull: it would have been
>> easier to debug the problem with Cayman when vddci was not correctly
>> set (it would have been higher or lower than expected or the value
>> would have been nonsense).
> It wouldn't have helped for that problem since there's no way to query
> the actual vddc or vddci or even the clocks.  All you can query is
> which performance level is active, the driver has to determine the
> clocks and voltages based on that.  It's basically the same info that
> is already exposed via debugfs.

I hoped we could expose the PLL test registers as well, this way 
somebody could actually measure at least the clocks instead of just 
calculating them from the PM profile in use.

Christian.

>
> Alex
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-10-01 15:38 Alex Deucher
@ 2014-12-21 20:05 ` Siavash Eliasi
  2014-12-22 17:01   ` Alex Deucher
  2015-02-24 22:09 ` Marek Olšák
  1 sibling, 1 reply; 9+ messages in thread
From: Siavash Eliasi @ 2014-12-21 20:05 UTC (permalink / raw)
  To: dri-devel

Hello, are these patches merged yet? or are they on the queue for Linux 
kernel 3.19?

And how they (shader/memory clocks) should be queried?

Thanks!
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-12-21 20:05 ` Siavash Eliasi
@ 2014-12-22 17:01   ` Alex Deucher
  2014-12-23  4:44     ` Siavash Eliasi
  0 siblings, 1 reply; 9+ messages in thread
From: Alex Deucher @ 2014-12-22 17:01 UTC (permalink / raw)
  To: Siavash Eliasi; +Cc: Maling list - DRI developers

On Sun, Dec 21, 2014 at 3:05 PM, Siavash Eliasi <siavashserver@gmail.com> wrote:
> Hello, are these patches merged yet? or are they on the queue for Linux
> kernel 3.19?
>
> And how they (shader/memory clocks) should be queried?

They are not merged yet.  There were some changes I wanted to make to
them before going upstream and I haven't had a chance to do it yet.

Alex
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-12-22 17:01   ` Alex Deucher
@ 2014-12-23  4:44     ` Siavash Eliasi
  0 siblings, 0 replies; 9+ messages in thread
From: Siavash Eliasi @ 2014-12-23  4:44 UTC (permalink / raw)
  To: Alex Deucher; +Cc: dri-devel


On 12/22/2014 08:31 PM, Alex Deucher wrote:
> On Sun, Dec 21, 2014 at 3:05 PM, Siavash Eliasi <siavashserver@gmail.com> wrote:
>> Hello, are these patches merged yet? or are they on the queue for Linux
>> kernel 3.19?
>>
>> And how they (shader/memory clocks) should be queried?
> They are not merged yet.  There were some changes I wanted to make to
> them before going upstream and I haven't had a chance to do it yet.
>
> Alex

Thank you very much for your great work on the drivers, and wish you 
guys have a happy new year :)
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2014-10-01 15:38 Alex Deucher
  2014-12-21 20:05 ` Siavash Eliasi
@ 2015-02-24 22:09 ` Marek Olšák
  2015-02-24 22:51   ` Alex Deucher
  1 sibling, 1 reply; 9+ messages in thread
From: Marek Olšák @ 2015-02-24 22:09 UTC (permalink / raw)
  To: Alex Deucher; +Cc: Alex Deucher, dri-devel

Tested-by: Marek Olšák <marek.olsak@amd.com>

Tested with: RV670, RV730, Redwood, Cayman, Cape Verde, Bonaire, Hawaii.

I have Mesa patches for the temperature, clocks, and GPU load. All
exposed via the Gallium HUD.

Marek

On Wed, Oct 1, 2014 at 5:38 PM, Alex Deucher <alexdeucher@gmail.com> wrote:
> This patch set implements a number of new
> radeon info ioctl queries to support additional
> profiling parameters for things like the GALLIUM
> HUD.  It could also be used by apps like radeontop
> to query the status registers, etc. without requiring
> root privileges or needing asis specific knowledge
> with respect to register BARs.
>
> Alex Deucher (22):
>   drm/radeon: add INFO query for GPU temperature
>   drm/radeon/dpm: add new callbacks to get the current sclk/mclk
>   drm/radeon/rs780: implement get_current_sclk/mclk
>   drm/radeon/rv6xx: implement get_current_sclk/mclk
>   drm/radeon/rv7xx/eg: implement get_current_sclk/mclk
>   drm/radeon/btc: implement get_current_sclk/mclk
>   drm/radeon: remove some rv7xx leftovers from btc dpm code
>   drm/radeon/ni: implement get_current_sclk/mclk
>   drm/radeon/si: implement get_current_sclk/mclk
>   drm/radeon/ci: implement get_current_sclk/mclk
>   drm/radeon/sumo: implement get_current_sclk/mclk
>   drm/radeon/tn: implement get_current_sclk/mclk
>   drm/radeon/kv: implement get_current_sclk/mclk
>   drm/radeon: add INFO query for current sclk/mclk
>   drm/radeon: add new callback for info ioctl register accessor
>   drm/radeon: add get_allowed_info_register function for r1xx-r5xx
>   drm/radeon: add get_allowed_info_register for r6xx/r7xx
>   drm/radeon: add get_allowed_info_register for EG/BTC
>   drm/radeon: add get_allowed_info_register for cayman/TN
>   drm/radeon: add get_allowed_info_register for SI
>   drm/radeon: add get_allowed_info_register for CIK
>   drm/radeon: add support for read reg query from radeon info ioctl
>
>  drivers/gpu/drm/radeon/btc_dpm.c     | 55 +++++++++++++++++++++++++++++++-----
>  drivers/gpu/drm/radeon/ci_dpm.c      | 14 +++++++++
>  drivers/gpu/drm/radeon/cik.c         | 33 ++++++++++++++++++++++
>  drivers/gpu/drm/radeon/cikd.h        |  2 ++
>  drivers/gpu/drm/radeon/evergreen.c   | 28 ++++++++++++++++++
>  drivers/gpu/drm/radeon/evergreend.h  |  1 +
>  drivers/gpu/drm/radeon/kv_dpm.c      | 23 +++++++++++++++
>  drivers/gpu/drm/radeon/ni.c          | 29 +++++++++++++++++++
>  drivers/gpu/drm/radeon/ni_dpm.c      | 36 +++++++++++++++++++++++
>  drivers/gpu/drm/radeon/nid.h         |  1 +
>  drivers/gpu/drm/radeon/r600.c        | 26 +++++++++++++++++
>  drivers/gpu/drm/radeon/radeon.h      |  7 +++++
>  drivers/gpu/drm/radeon/radeon_asic.c | 49 ++++++++++++++++++++++++++++++++
>  drivers/gpu/drm/radeon/radeon_asic.h | 30 ++++++++++++++++++++
>  drivers/gpu/drm/radeon/radeon_kms.c  | 29 +++++++++++++++++++
>  drivers/gpu/drm/radeon/rs780_dpm.c   | 22 +++++++++++++++
>  drivers/gpu/drm/radeon/rv6xx_dpm.c   | 46 ++++++++++++++++++++++++++++++
>  drivers/gpu/drm/radeon/rv770_dpm.c   | 44 +++++++++++++++++++++++++++++
>  drivers/gpu/drm/radeon/si.c          | 30 ++++++++++++++++++++
>  drivers/gpu/drm/radeon/si_dpm.c      | 36 +++++++++++++++++++++++
>  drivers/gpu/drm/radeon/sid.h         |  1 +
>  drivers/gpu/drm/radeon/sumo_dpm.c    | 28 ++++++++++++++++++
>  drivers/gpu/drm/radeon/trinity_dpm.c | 25 ++++++++++++++++
>  include/uapi/drm/radeon_drm.h        |  4 +++
>  24 files changed, 592 insertions(+), 7 deletions(-)
>
> --
> 1.8.3.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 00/22] new radeon info queries
  2015-02-24 22:09 ` Marek Olšák
@ 2015-02-24 22:51   ` Alex Deucher
  0 siblings, 0 replies; 9+ messages in thread
From: Alex Deucher @ 2015-02-24 22:51 UTC (permalink / raw)
  To: Marek Olšák; +Cc: Alex Deucher, dri-devel

On Tue, Feb 24, 2015 at 5:09 PM, Marek Olšák <maraeo@gmail.com> wrote:
> Tested-by: Marek Olšák <marek.olsak@amd.com>
>
> Tested with: RV670, RV730, Redwood, Cayman, Cape Verde, Bonaire, Hawaii.
>
> I have Mesa patches for the temperature, clocks, and GPU load. All
> exposed via the Gallium HUD.

Thanks!  Pushed to my 4.1 branch.

Alex

>
> Marek
>
> On Wed, Oct 1, 2014 at 5:38 PM, Alex Deucher <alexdeucher@gmail.com> wrote:
>> This patch set implements a number of new
>> radeon info ioctl queries to support additional
>> profiling parameters for things like the GALLIUM
>> HUD.  It could also be used by apps like radeontop
>> to query the status registers, etc. without requiring
>> root privileges or needing asis specific knowledge
>> with respect to register BARs.
>>
>> Alex Deucher (22):
>>   drm/radeon: add INFO query for GPU temperature
>>   drm/radeon/dpm: add new callbacks to get the current sclk/mclk
>>   drm/radeon/rs780: implement get_current_sclk/mclk
>>   drm/radeon/rv6xx: implement get_current_sclk/mclk
>>   drm/radeon/rv7xx/eg: implement get_current_sclk/mclk
>>   drm/radeon/btc: implement get_current_sclk/mclk
>>   drm/radeon: remove some rv7xx leftovers from btc dpm code
>>   drm/radeon/ni: implement get_current_sclk/mclk
>>   drm/radeon/si: implement get_current_sclk/mclk
>>   drm/radeon/ci: implement get_current_sclk/mclk
>>   drm/radeon/sumo: implement get_current_sclk/mclk
>>   drm/radeon/tn: implement get_current_sclk/mclk
>>   drm/radeon/kv: implement get_current_sclk/mclk
>>   drm/radeon: add INFO query for current sclk/mclk
>>   drm/radeon: add new callback for info ioctl register accessor
>>   drm/radeon: add get_allowed_info_register function for r1xx-r5xx
>>   drm/radeon: add get_allowed_info_register for r6xx/r7xx
>>   drm/radeon: add get_allowed_info_register for EG/BTC
>>   drm/radeon: add get_allowed_info_register for cayman/TN
>>   drm/radeon: add get_allowed_info_register for SI
>>   drm/radeon: add get_allowed_info_register for CIK
>>   drm/radeon: add support for read reg query from radeon info ioctl
>>
>>  drivers/gpu/drm/radeon/btc_dpm.c     | 55 +++++++++++++++++++++++++++++++-----
>>  drivers/gpu/drm/radeon/ci_dpm.c      | 14 +++++++++
>>  drivers/gpu/drm/radeon/cik.c         | 33 ++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/cikd.h        |  2 ++
>>  drivers/gpu/drm/radeon/evergreen.c   | 28 ++++++++++++++++++
>>  drivers/gpu/drm/radeon/evergreend.h  |  1 +
>>  drivers/gpu/drm/radeon/kv_dpm.c      | 23 +++++++++++++++
>>  drivers/gpu/drm/radeon/ni.c          | 29 +++++++++++++++++++
>>  drivers/gpu/drm/radeon/ni_dpm.c      | 36 +++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/nid.h         |  1 +
>>  drivers/gpu/drm/radeon/r600.c        | 26 +++++++++++++++++
>>  drivers/gpu/drm/radeon/radeon.h      |  7 +++++
>>  drivers/gpu/drm/radeon/radeon_asic.c | 49 ++++++++++++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/radeon_asic.h | 30 ++++++++++++++++++++
>>  drivers/gpu/drm/radeon/radeon_kms.c  | 29 +++++++++++++++++++
>>  drivers/gpu/drm/radeon/rs780_dpm.c   | 22 +++++++++++++++
>>  drivers/gpu/drm/radeon/rv6xx_dpm.c   | 46 ++++++++++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/rv770_dpm.c   | 44 +++++++++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/si.c          | 30 ++++++++++++++++++++
>>  drivers/gpu/drm/radeon/si_dpm.c      | 36 +++++++++++++++++++++++
>>  drivers/gpu/drm/radeon/sid.h         |  1 +
>>  drivers/gpu/drm/radeon/sumo_dpm.c    | 28 ++++++++++++++++++
>>  drivers/gpu/drm/radeon/trinity_dpm.c | 25 ++++++++++++++++
>>  include/uapi/drm/radeon_drm.h        |  4 +++
>>  24 files changed, 592 insertions(+), 7 deletions(-)
>>
>> --
>> 1.8.3.1
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2015-02-24 22:51 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-01 16:30 [PATCH 00/22] new radeon info queries Alexandre Demers
2014-10-02  4:50 ` Alex Deucher
2014-10-02 12:41   ` Christian König
  -- strict thread matches above, loose matches on Subject: below --
2014-10-01 15:38 Alex Deucher
2014-12-21 20:05 ` Siavash Eliasi
2014-12-22 17:01   ` Alex Deucher
2014-12-23  4:44     ` Siavash Eliasi
2015-02-24 22:09 ` Marek Olšák
2015-02-24 22:51   ` Alex Deucher

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.