On Feb 8, 2021, at 15:23, Christian König christian.koenig@amd.com wrote:
Am 08.02.21 um 21:21 schrieb Zack Rusin:
On Feb 8, 2021, at 08:35, Christian König christian.koenig@amd.com wrote:
Hi Zack,
ok we figured out how to do this correctly.
Basically using the pdev->kobj instead of the drm->primary->kdev->kobj pointer worked quite well.
I've just send the latest patches to the mailing list. If you don't have any objections I will commit that tomorrow with your and Daniels rb.
That sounds good. That moves the entries under /sys/devices/pciBUS/ID/memory_accounting , yes?
Yes correct. I really hope nobody relied on the old location or otherwise I need to add some kind of symlink workaround or such.
I’m not aware of anyone depending on that location. I’ll think about it this week and in the worst case I’ll just disable it for the next release until we can spend some time creating an actual structure to our sysfs layout that we can maintain long term but that’s not something you need to worry about. Thank you.
z