* 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
* [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 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.