[igt-dev] [PATCH i-g-t 0/2] IGT XE documentation cleanup
Mauro Carvalho Chehab
mauro.chehab at linux.intel.com
Thu Aug 31 12:30:40 UTC 2023
On Thu, 31 Aug 2023 11:39:34 +0000
Katarzyna Dec <katarzyna.dec at intel.com> wrote:
> Let's add missing documentation fileds that are causing warnings.
> Some tests documentation needed changes to have more accurate naming.
> Let's mark Negative and Stress tests.
> Adding missing 'Run type' and 'Test category' for remaining test.
Kasia,
Besides the changes, could you please also change the
sub-category field and/or Functionalities for xe_mmap.c?
Right now, both are using "mmap" bucket. This is valid for
the documentation itself, but it causes confusion when analyzing
the results later on, as there will be two buckets with the same
name but different meanings.
Perhaps we could use Sub-category == "VMA".
What do you think?
Another issue I noticed in this series is that we have "Blitter"
with the first letter in uppercase. Perhaps there are more cases like
that. Please let's use lowercase on such buckets, as this is less
prone to errors (IMO).
Regards,
Mauro
More information about the igt-dev
mailing list