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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 682 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1990 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 216 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 991 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 269 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1041 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 310 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1082 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 643 bytes --]

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

Christian König <deathsimple@vodafone.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |WONTFIX

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

[-- Attachment #1.2: Type: text/html, Size: 2176 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 915 bytes --]

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

almos <aaalmosss@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|WONTFIX                     |---

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

[-- Attachment #1.2: Type: text/html, Size: 2417 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 783 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1683 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 448 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1260 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 257 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1074 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 445 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1342 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 346 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1164 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 302 bytes --]

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.

[-- Attachment #1.2: Type: text/html, Size: 1185 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

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

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 805 bytes --]

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

Martin Peres <martin.peres@free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |MOVED
             Status|REOPENED                    |RESOLVED

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

[-- Attachment #1.2: Type: text/html, Size: 2382 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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