Re: [PATCH libdrm 0/4] libdrm: add new GPU sensor interface for AMDGPU

[Samuel Pitoiset]

On 02/14/2017 10:43 AM, Christian König wrote:

Am 14.02.2017 um 02:44 schrieb Emil Velikov:

On 14 February 2017 at 00:08, Samuel Pitoiset
 wrote:

Hi,

This series implements a new GPU sensor interface which will allow to
query the current temperature and the shader/memory clocks via the
GALLIUM_HUD. This adds the same functionality as the Radeon driver.

The first patch of the series syncs amdgpu_drm.h with kernel headers.

Branch is here:
https://cgit.freedesktop.org/~hakzsam/drm/log/?h=si_clocks_temp

Please review.
Thanks!

Samuel Pitoiset (4):
   amdgpu: sync amdgpu_drm.h with the kernel

Samuel,

Please read through drm/include/drm/README in particular "When and how
to update these files".
JFYI multiple people are trying to get various UABI in, so do keep an
eye open for fireworks.


   amdgpu: add new GPU sensor related interface
   amdgpu: allow to query GPU sensor related information

You'll need to add the new interface(s) to the symbols test. `make
check' must pass.


   Bump version for 2.4.76 release


Skim through RELEASING for details on how to roll one.

Pardon, if I'm flocking up a dead horse here.


Please keep an eye open on this Emil.

Additionally to what said before you didn't used upstream, but rather
the branch including the DC code (with the freesync IOCTL).

We are probably not getting this upstream like that, so pushing it into
libdrm is clearly not a good idea.


Thanks for your feedbacks. I will first wait for the kernel interface to 
be merged before making a new series.




Regards,
Christian.



Thanks
Emil
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel




___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/4] libdrm: add new GPU sensor interface for AMDGPU

[Christian König]

Am 14.02.2017 um 02:44 schrieb Emil Velikov:

On 14 February 2017 at 00:08, Samuel Pitoiset  wrote:

Hi,

This series implements a new GPU sensor interface which will allow to
query the current temperature and the shader/memory clocks via the
GALLIUM_HUD. This adds the same functionality as the Radeon driver.

The first patch of the series syncs amdgpu_drm.h with kernel headers.

Branch is here:
https://cgit.freedesktop.org/~hakzsam/drm/log/?h=si_clocks_temp

Please review.
Thanks!

Samuel Pitoiset (4):
   amdgpu: sync amdgpu_drm.h with the kernel

Samuel,

Please read through drm/include/drm/README in particular "When and how
to update these files".
JFYI multiple people are trying to get various UABI in, so do keep an
eye open for fireworks.


   amdgpu: add new GPU sensor related interface
   amdgpu: allow to query GPU sensor related information

You'll need to add the new interface(s) to the symbols test. `make
check' must pass.


   Bump version for 2.4.76 release


Skim through RELEASING for details on how to roll one.

Pardon, if I'm flocking up a dead horse here.


Please keep an eye open on this Emil.

Additionally to what said before you didn't used upstream, but rather 
the branch including the DC code (with the freesync IOCTL).


We are probably not getting this upstream like that, so pushing it into 
libdrm is clearly not a good idea.


Regards,
Christian.



Thanks
Emil
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel



___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/4] libdrm: add new GPU sensor interface for AMDGPU

[Emil Velikov]

On 14 February 2017 at 00:08, Samuel Pitoiset  wrote:
> Hi,
>
> This series implements a new GPU sensor interface which will allow to
> query the current temperature and the shader/memory clocks via the
> GALLIUM_HUD. This adds the same functionality as the Radeon driver.
>
> The first patch of the series syncs amdgpu_drm.h with kernel headers.
>
> Branch is here:
> https://cgit.freedesktop.org/~hakzsam/drm/log/?h=si_clocks_temp
>
> Please review.
> Thanks!
>
> Samuel Pitoiset (4):
>   amdgpu: sync amdgpu_drm.h with the kernel
Samuel,

Please read through drm/include/drm/README in particular "When and how
to update these files".
JFYI multiple people are trying to get various UABI in, so do keep an
eye open for fireworks.

>   amdgpu: add new GPU sensor related interface
>   amdgpu: allow to query GPU sensor related information
You'll need to add the new interface(s) to the symbols test. `make
check' must pass.

>   Bump version for 2.4.76 release
>
Skim through RELEASING for details on how to roll one.

Pardon, if I'm flocking up a dead horse here.

Thanks
Emil
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH libdrm 0/4] libdrm: add new GPU sensor interface for AMDGPU

[Samuel Pitoiset]

Hi,

This series implements a new GPU sensor interface which will allow to
query the current temperature and the shader/memory clocks via the
GALLIUM_HUD. This adds the same functionality as the Radeon driver.

The first patch of the series syncs amdgpu_drm.h with kernel headers.

Branch is here:
https://cgit.freedesktop.org/~hakzsam/drm/log/?h=si_clocks_temp

Please review.
Thanks!

Samuel Pitoiset (4):
  amdgpu: sync amdgpu_drm.h with the kernel
  amdgpu: add new GPU sensor related interface
  amdgpu: allow to query GPU sensor related information
  Bump version for 2.4.76 release

 amdgpu/amdgpu.h  |  19 +++
 amdgpu/amdgpu_gpu_info.c |  15 ++
 configure.ac |   2 +-
 include/drm/amdgpu_drm.h | 135 +++
 4 files changed, 170 insertions(+), 1 deletion(-)

-- 
2.11.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel