Re: [PATCH i-g-t] tests/intel/intel_sysfs_debugfs: Remove xe-gt

[Peter Senna Tschudin <peter.senna@linux.intel.com>]

Hi Kamil,

On 4/10/2025 8:11 PM, Kamil Konieczny wrote:
> Hi Peter,
> On 2025-04-10 at 11:03:15 +0200, Peter Senna Tschudin wrote:
>> The intel_sysfs_debugfs test now includes functionality to read all Xe
>> debugfs files, making the older xe-gt test redundant. Additionally,
>> xe-gt causes issues when testing Virtual Functions (VFs) in SR-IOV
>> setups, as some of the debugfs files it expects are not present for VFs.
>>
>> Rather than extending the overlapping and problematic xe-gt, this commit
>> removes it entirely.
> 
> I agree on removing reading part but not on existence checks.
> Imho start with small refactor:
> 
> - remove readings
> - change existing checks so they will print _all_ missing
>   debugfs nodes and only after that test will fail if there
>   were misses

But that is the issue. For printing _all_ missing debugfs nodes
we need first to have a list of nodes that is complete and up to
date. Looking at git blame it is not clear that the current list
had the intention of being complete. Do you know why are we
testing for these few specific files?

[...]

Peter