[Bug 73047] New: radeon_pm_info should be in sysfs instead of debugfs

List overview All Threads
Download

newer

older

[Bug 73060] New: RV630 HDMI audio...

[Bug 72710] New: rv635: resume...

bugzilla-daemon＠freedesktop.org

26 Dec 2013 26 Dec '13

2:25 p.m.

https://bugs.freedesktop.org/show_bug.cgi?id=73047

Priority: medium Bug ID: 73047 Assignee: dri-devel@lists.freedesktop.org Summary: radeon_pm_info should be in sysfs instead of debugfs Severity: normal Classification: Unclassified OS: All Reporter: aaalmosss@gmail.com Hardware: Other Status: NEW Version: XOrg CVS Component: DRM/Radeon Product: DRI

Debugfs is only accessible for root, and it's not advised to use it for other than debugging (as its name suggests).

-- You are receiving this mail because: You are the assignee for the bug.

Attachments:

attachment.html (text/html — 1.9 KB)

Show replies by date

bugzilla-daemon＠freedesktop.org

26 Dec 26 Dec

5:58 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #1 from Alex Deucher agd5f@yahoo.com --- radeon_pm_info is for debugging.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

6:01 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #2 from almos aaalmosss@gmail.com --- OK, then how can I get information about the clock speeds of the GPU when not debugging?

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

2 Jan 2 Jan

3:22 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #3 from almos aaalmosss@gmail.com --- I guess the same thing applies to vgaswitcheroo. I'm not a kernel developer, but I'm quite sure these are not just for debugging.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

4:37 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

Christian König deathsimple@vodafone.de changed:

--- Comment #4 from Christian König deathsimple@vodafone.de --- We had the discussion a couple of times now. The power management infos radeon_pm_info provides are definitely only for debugging.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

3 Jan 3 Jan

11:29 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

almos aaalmosss@gmail.com changed:

--- Comment #5 from almos aaalmosss@gmail.com --- (In reply to comment #4)

...

We had the discussion a couple of times now. The power management infos radeon_pm_info provides are definitely only for debugging.

Are the minutes of those discussions available somewhere? I'm curious about the arguments.

BTW have you considered moving /sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq to debugfs for consistency's sake? How about removing lm-sensors support? These are definitely for debugging only.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

5 Jan 5 Jan

3:21 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #6 from Christian König deathsimple@vodafone.de --- (In reply to comment #5)

...

Are the minutes of those discussions available somewhere? I'm curious about the arguments.

radeon_pm_info is not the real hardware state, but rather the state the kernel driver thinks the hardware is in.

So it's actually not very useful to an end user, except if you want to debug the different states the kernel driver can be into (UVD/performance/battery etc...)

I agree that we need a proper sysfs interface to make certain PM functions and informations available to userspace, but it's definitely not what's inside radeon_pm_info.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

26 Sep 26 Sep

10:14 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #7 from Alexandre Demers alexandre.f.demers@gmail.com --- I'm about to close this bug since, as Christian said, there should be a way to expose some PM functions to the user, but not through radeon_pm_info.

But before, I'd like to know if there is something coming this way to expose these functionality?

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

29 Sep 29 Sep

10:41 a.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #8 from Marek Olšák maraeo@gmail.com --- I'd like the RADEON_INFO ioctl to be able to return the clocks.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

11:06 a.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #9 from Christian König deathsimple@vodafone.de --- (In reply to comment #8)

...

I'd like the RADEON_INFO ioctl to be able to return the clocks.

That sounds like a pretty good idea to me and we should actually query the hardware for the clocks and return that instead of the software state.

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

12:22 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #10 from Marek Olšák maraeo@gmail.com --- I'd like to have RADEON_INFO support for these: - engine and memory clocks - temperatures - status regs (for radeontop without root and Gallium HUD)

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

1 Oct 1 Oct

3:47 p.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

--- Comment #11 from Alex Deucher agd5f@yahoo.com --- Just send out a patch set to implement this: http://lists.freedesktop.org/archives/dri-devel/2014-October/069410.html

-- You are receiving this mail because: You are the assignee for the bug.

bugzilla-daemon＠freedesktop.org

19 Nov 19 Nov

8:41 a.m.

New subject: [Bug 73047] radeon_pm_info should be in sysfs instead of debugfs

https://bugs.freedesktop.org/show_bug.cgi?id=73047

Martin Peres martin.peres@free.fr changed:

--- Comment #12 from Martin Peres martin.peres@free.fr --- -- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/drm/amd/issues/420.

-- You are receiving this mail because: You are the assignee for the bug.

1979

Age (days ago)

4133

Last active (days ago)

dri-devel@lists.freedesktop.org

12 comments

1 participants

tags (0)

participants (1)

bugzilla-daemon＠freedesktop.org