[ Split from pull #43 ] Add ioctl functions #44

Danysan1 · 2017-01-07T18:05:47Z

See radeon_ioctl.h for all the details of the ioctl functions.
Also, fix gcc 4.6 build in radeon_profile.h and radeon_profile.cpp.

Danysan1 · 2017-01-17T16:28:40Z

@marazmista the code is complete.
Ioctls allow to get most GPU informations without root. They also allow to read some GPU registers, including the status register, allowing to get the GPU usage.
Ioctls require libdrm headers (libdrm on Arch, libdrm-devel on Fedora, libdrm-dev on Debian).
Here is the feature matrix for each driver and Linux version:

	Radeon (<3.15)	Radeon (>=3.15)	Radeon (>=4.1)	Amdgpu (<4.10)	Amdgpu (>=4.10)
Temperature	❌	❌	✔️	❌	❌
Core clock	❌	❌	✔️	❌	✔️
Memory clock	❌	❌	✔️	❌	✔️
Max core clock	Need root	✔️	✔️	✔️	✔️
VRAM size	❌	❌	❌	✔️	✔️
VRAM usage	Need root	✔️	✔️	✔️	✔️
GPU usage	❌	❌	✔️	✔️	✔️

I have tested all the functions on radeon driver.

PS: I added NO_IOCTL to the release defines because until we use the ioctls to effectively retrieve data there's no reason to add extra dependencies.

EDIT: Updated table

Before: radeon_ioctl.h + radeon_ioctl.cpp After: ioctlHandler.h + ioctlHandler.cpp + ioctl_radeon.cpp + ioctl_amdgpu.cpp

Danysan1 · 2017-02-06T17:11:38Z

Updated table:

| Driver Kernel | Radeon 3.7<=ver<3.15 | Radeon 3.15<=ver<4.1 | Radeon >=4.1 | Amdgpu 4.2<=ver<4.10 |
Amdgpu >=4.10 |
|---|:---:|:---:|:---:|:---:|:---:|
Temperature | ❌ | ❌ | ✔️ | ❌ | ❌ |
Core clock | ❌ | ❌ | ✔️ | ❌ | ✔️ |
Memory clock | ❌ | ❌ | ✔️ | ❌ | ✔️ |
Max core clock | Need root | ✔️ | ✔️ | ✔️ | ✔️ |
Max mem clock | ❌ | ❌ | ❌ | ✔️ | ✔️ |
Memory size | Need root | ✔️ | ✔️ | ✔️ | ✔️ |
Memory usage | Need root | ✔️ | ✔️ | ✔️ | ✔️ |
GPU usage | ❌ | ❌ | ✔️ | ✔️ | ✔️ |
Driver name | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ |

I tested the functions on radeon driver.

Danysan1 · 2017-05-27T07:25:00Z

Updated table:

| Driver Kernel | Radeon >=3.7 | Radeon >=3.15 | Radeon >=4.1 | Amdgpu >=4.2 |
Amdgpu >=4.10 | Amdgpu >= 4.12 |
|---|:---:|:---:|:---:|:---:|:---:|:---:|
Temperature | ❌ | ❌ | ✔️ | ❌ | ❌ | ✔️ |
Core clock | ❌ | ❌ | ✔️ | ❌ | ✔️ | ✔️ |
Memory clock | ❌ | ❌ | ✔️ | ❌ | ✔️ | ✔️ |
Max core clock | Root | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ |
Max mem clock | ❌ | ❌ | ❌ | ✔️ | ✔️ | ✔️ |
Memory size | Root | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ |
Memory usage | Root | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ |
GPU usage | ❌ | ❌ | ✔️ | ✔️ | ✔️ | ✔️ |
Driver name | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ | ✔️ |

Danysan1 · 2017-06-16T06:38:58Z

Closing because this is already being merged through branch rp-dev

This was referenced Jan 7, 2017

[ Split from pull #43 ] Add configure, make install, Flatpak support #45

Closed

configure, make install, AppStream, Flatpak, ioctl #43

Closed

Add IOCTLs

66205c4

Danysan1 force-pushed the add_ioctl branch from 0069812 to 66205c4 Compare January 9, 2017 21:17

Move ioctl code to ioctlHandler class

d82c25a

Danysan1 force-pushed the add_ioctl branch from 4baa5da to d82c25a Compare January 10, 2017 14:30

Improve documentation

4e37a5f

Danysan1 force-pushed the add_ioctl branch from dab33f4 to 4e37a5f Compare January 10, 2017 18:02

Improve Doxyfile

ea42299

Danysan1 force-pushed the add_ioctl branch from ce79329 to ea42299 Compare January 10, 2017 20:08

Danysan1 added 7 commits January 10, 2017 21:31

Fix doc

e82c57e

Add non-root drm authentication

d632020

Fix ioctl sizeof(data)

2e2b078

Fix sharedMem managment

b5ceb91

Improve doc

2485e91

Minor fixes

834ffb1

Allow non-root ioctl

40ff29c

Danysan1 force-pushed the add_ioctl branch from 77926d0 to 40ff29c Compare January 12, 2017 18:52

Danysan1 added 9 commits January 12, 2017 20:18

Fix release build

9012971

Improve comments

af998ae

Check if headers are available

eac25dc

Update README

0c15184

Check NO_AMDGPU_IOCTL

f80c90a

Add AMDGPU_INFO_VCE_CLOCK_TABLE

0b25925

Use libdrm instead of kernel-headers, simplify NO_IOCTL check

a469e10

Auto detect driver name

d0aca31

Cleanup

b402891

Danysan1 force-pushed the add_ioctl branch from 4c00211 to b402891 Compare January 17, 2017 12:29

Danysan1 added 2 commits January 17, 2017 16:54

Check libdrm headers only if NO_IOCTL is not defined

4d3df98

Add AMDGPU_INFO_DEV_INFO

2d50800

Danysan1 added 4 commits January 18, 2017 10:08

Check if RADEON_INFO_CURRENT_GPU_TEMP is available before using it

c8e86b9

Improve Doxyfile (add .qch documentation)

066f204

ioctl cleanup (create openPath())

57c3db9

Refactoring: separe ioctl radeon and amdgpu code

e61eb4f

Before: radeon_ioctl.h + radeon_ioctl.cpp After: ioctlHandler.h + ioctlHandler.cpp + ioctl_radeon.cpp + ioctl_amdgpu.cpp

Danysan1 force-pushed the add_ioctl branch from 131b4ef to e61eb4f Compare January 23, 2017 09:05

Danysan1 added 10 commits January 23, 2017 14:00

Improve documentation and header

91bd78d

Cleanup of includes

7fd862f

Fix getMemoryClock(), getCoreClock() in amdgpuIoctlHandler

b55fa3c

Add getClocks()

c7876f0

Minor cleanup

822cfc0

Backward support for gcc 4.6

9b9531f

Cleanup, Use DRM_RADEON_GEM_INFO in radeonIoctlHandler::getVramSize()

1cb5c17

Add ioctlHandler::getVramUsagePercentage()

56083e2

Add getMaxMemoryClock() and getMaxClocks()

f46b160

Fix valgrind warnings, cleanup

b150a49

Danysan1 force-pushed the add_ioctl branch from 225aa4d to b150a49 Compare February 5, 2017 18:01

Danysan1 added 2 commits February 6, 2017 10:42

Improve ioctlHandler::getGpuUsage()

e37f376

Cleanup

ebb6c42

Danysan1 added 6 commits February 13, 2017 20:29

Improve documentation

10bacef

Improve Doxyfile

401afec

Add ioctlHandler::isCardActive(), improve documentation

e7beb10

Improved ioctl loading

1772cfe

dXorg::getClocks and getTemperature can now use ioctls

eea3ce4

Implement amdgpuIoctlHandler::getTemperature()

7c59b45

bugz8unny69 mentioned this pull request May 27, 2017

Don't read amdgpu_pm_info anymore #54

Closed

Danysan1 closed this Jun 16, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[ Split from pull #43 ] Add ioctl functions #44

[ Split from pull #43 ] Add ioctl functions #44

Danysan1 commented Jan 7, 2017

Danysan1 commented Jan 17, 2017 •

edited

Loading

Danysan1 commented Feb 6, 2017

Danysan1 commented May 27, 2017

Danysan1 commented Jun 16, 2017 •

edited

Loading

[ Split from pull #43 ] Add ioctl functions #44

[ Split from pull #43 ] Add ioctl functions #44

Conversation

Danysan1 commented Jan 7, 2017

Danysan1 commented Jan 17, 2017 • edited Loading

Danysan1 commented Feb 6, 2017

Danysan1 commented May 27, 2017

Danysan1 commented Jun 16, 2017 • edited Loading

Danysan1 commented Jan 17, 2017 •

edited

Loading

Danysan1 commented Jun 16, 2017 •

edited

Loading