Re: [RFC] drm/i915/gt: create a subfolder for GT debugfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 8/3/19 1:27 AM, Chris Wilson wrote:
Quoting Daniele Ceraolo Spurio (2019-08-03 02:09:44)
The idea is to better organize our debugfs entries by moving the
GT-related ones into their own subfolder, similarly to what we did for
the kernel code.

As an example, this patch moves the GuC/HuC and user_forcewake debugfs,
but a few more files can be moved if we decide to go this way (engine
status, gt_pm etc). Test updates will also be required to look for the
file in the correct place.

Quick question, why stop at intel_gt_debugfs.c? Do we not want to pull the
[gh]uc closer to the source i.e. gt/uc/intel_uc_debugfs.c?

That works for me. I first wanted to see if the idea of using a subfolder for GT would fly before doing any proper refactoring. I picked the uC files as an example in this RFC just because I have been touching that area recently ;)


I'd also recommend for any future debugfs, we split it between the
debugfs hook and the pretty printer, and spit out yaml :)

You mean having a pretty printer that takes a key:value pair? Sounds reasonable, but I wouldn't mix it with moving the files around.

Daniele

-Chris

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux