[PATCH i-g-t v2 4/5] intel-ci: rename i915 testlists

[Kamil Konieczny]

Hi Sharma,,
On 2025-07-07 at 18:01:32 +0530, Sharma, Swati2 wrote:
> Hi Kamil,
> 
> On 07-07-2025 03:11 pm, Kamil Konieczny wrote:
> > Follow Xe nameing and add i915 prefix to its testlists. In
> > fast-feedback use 'i915-' and in normal testlist 'i915.'.
> Even for fast-feedback, can't we have i915.fast-feedback? Is there some
> restriction?

Good points, using 'i915.' would be better.

> This might create confusion, when to use . and when to use -.
> ex: xe-sriov-vf.blocklist.txt
> Lets follow one approach either . or -

Same here, we should migrate this also from 'xe-' to 'xe.'

> 
> Also, we had idea of creating 2 folders : testlists and blocklists.
> Do you think we should have those directories?

Hmm, this is already complicated, let's go with unification first.
Also it could complicate inter-gpu checks.

Regards,
Kamil

> > 
> > Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> > Reviewed-by: Karthik B S <karthik.b.s at intel.com>
> > ---
> >   tests/intel-ci/README                                       | 6 +++---
> >   ....testlist => i915-fast-feedback-chamelium-only.testlist} | 0
> >   .../{fast-feedback.testlist => i915-fast-feedback.testlist} | 0
> >   ...chamelium-only.testlist => i915.chamelium-only.testlist} | 0
> >   tests/intel-ci/meson.build                                  | 6 +++---
> >   tests/intel/i915_test_config.json                           | 2 +-
> >   tests/intel/kms_test_config.json                            | 6 +++---
> >   7 files changed, 10 insertions(+), 10 deletions(-)
> >   rename tests/intel-ci/{fast-feedback-chamelium-only.testlist => i915-fast-feedback-chamelium-only.testlist} (100%)
> >   rename tests/intel-ci/{fast-feedback.testlist => i915-fast-feedback.testlist} (100%)
> >   rename tests/intel-ci/{chamelium-only.testlist => i915.chamelium-only.testlist} (100%)
> > 
> > diff --git a/tests/intel-ci/README b/tests/intel-ci/README
> > index 5ef2ee2d1..102cdb1fa 100644
> > --- a/tests/intel-ci/README
> > +++ b/tests/intel-ci/README
> > @@ -17,9 +17,9 @@ Changing the test lists should only happen with approval from the CI
> >   people. If you make a new test that should be tested by CI regularly,
> >   ask a maintainer (see MAINTAINERS).
> > -======================
> > -fast-feedback.testlist
> > -======================
> > +===========================
> > +i915.fast-feedback.testlist
> > +===========================
> >   Fast-feedback contains tests that roughly tests a wide coverage of
> >   features in a short time. If the tests in fast-feedback report no
> > diff --git a/tests/intel-ci/fast-feedback-chamelium-only.testlist b/tests/intel-ci/i915-fast-feedback-chamelium-only.testlist
> > similarity index 100%
> > rename from tests/intel-ci/fast-feedback-chamelium-only.testlist
> > rename to tests/intel-ci/i915-fast-feedback-chamelium-only.testlist
> > diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/i915-fast-feedback.testlist
> > similarity index 100%
> > rename from tests/intel-ci/fast-feedback.testlist
> > rename to tests/intel-ci/i915-fast-feedback.testlist
> > diff --git a/tests/intel-ci/chamelium-only.testlist b/tests/intel-ci/i915.chamelium-only.testlist
> > similarity index 100%
> > rename from tests/intel-ci/chamelium-only.testlist
> > rename to tests/intel-ci/i915.chamelium-only.testlist
> > diff --git a/tests/intel-ci/meson.build b/tests/intel-ci/meson.build
> > index d18170402..17d3102bd 100644
> > --- a/tests/intel-ci/meson.build
> > +++ b/tests/intel-ci/meson.build
> > @@ -4,9 +4,9 @@ intelci_files = [
> >     'README',
> >     'i915.blocklist.txt',
> >     'i915.blocklist-pre-merge.txt',
> > -  'chamelium-only.testlist',
> > -  'fast-feedback.testlist',
> > -  'fast-feedback-chamelium-only.testlist',
> > +  'i915.chamelium-only.testlist',
> > +  'i915-fast-feedback.testlist',
> > +  'i915-fast-feedback-chamelium-only.testlist',
> >     'meta.testlist',
> >     'xe.blocklist.txt',
> >     'xe.bmg.core.blocklist.txt',
> > diff --git a/tests/intel/i915_test_config.json b/tests/intel/i915_test_config.json
> > index da871f493..1a75d52e5 100644
> > --- a/tests/intel/i915_test_config.json
> > +++ b/tests/intel/i915_test_config.json
> > @@ -25,7 +25,7 @@
> >                       "exclude-type": "regex-ignorecase",
> >                       "append-value-if-not-excluded": "i915 FULL",
> >                       "include": [
> > -                        { "i915 BAT": "../intel-ci/fast-feedback.testlist" }
> > +                        { "i915 BAT": "../intel-ci/i915-fast-feedback.testlist" }
> >                       ],
> >                       "exclude": [
> >                           { "all": "../intel-ci/i915.blocklist.txt" }
> > diff --git a/tests/intel/kms_test_config.json b/tests/intel/kms_test_config.json
> > index fc8766279..886870783 100644
> > --- a/tests/intel/kms_test_config.json
> > +++ b/tests/intel/kms_test_config.json
> > @@ -13,9 +13,9 @@
> >                       "append-value-if-not-excluded": "Xe FULL, i915 FULL",
> >                       "exclude-type": "regex-ignorecase",
> >                       "include": [
> > -                        { "i915 BAT": "../intel-ci/fast-feedback.testlist" },
> > -                        { "i915 BAT chamelium": "../intel-ci/fast-feedback-chamelium-only.testlist" },
> > -                        { "i915 chamelium": "../intel-ci/chamelium-only.testlist" },
> > +                        { "i915 BAT": "../intel-ci/i915-fast-feedback.testlist" },
> > +                        { "i915 BAT chamelium": "../intel-ci/i915-fast-feedback-chamelium-only.testlist" },
> > +                        { "i915 chamelium": "../intel-ci/i915.chamelium-only.testlist" },
> >                           { "Xe BAT": "../intel-ci/xe-fast-feedback.testlist" },
> >                           { "Xe BAT chamelium": "../intel-ci/xe-fast-feedback-chamelium-only.testlist" }
>