Thomas Hellström authored 3 months ago and Kamil Konieczny committed 3 months ago

Threads may compete over VRAM resources, and since there is no
serialization in TTM, threaded evict tests may have a high
possibility of failing.

Disable such tests until we have a working exhaustive eviction
implementation in place.

v2:
- Disable also xe_evict@evict-cm-threads-small in BAT.
(Kamil).

Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: <akshata.jahagirdar@intel.com>
Reviewed-by: Nirmoy Das <nirmoy.das@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Reviewed-by: Akshata Jahagirdar <akshata.jahagirdar@intel.com>

Remove xe_bo_shrink_kunit which takes a while to run on various
machines. There is no good way to make it faster.

Also add it to the blocklist so this doesn't executed by non-BAT
test.

v2: Add it to blocklist(Matt)

Cc: Matthew Auld <matthew.auld@intel.com>
Suggested-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Link: drm/xe/kernel#2998


Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Jakub197 authored 4 months ago and Marcin Bernatowicz committed 4 months ago

Add SR-IOV test to exercise Function Level Reset (FLR) on VF1 in BAT
testing.
Test being added:
- igt@xe_sriov_flr@flr-vf1-clear

Signed-off-by: Jakub Kolakowski <jakub1.kolakowski@intel.com>
Cc: Adam Miszczak <adam.miszczak@intel.com>
Cc: Marcin Bernatowicz <marcin.bernatowicz@intel.com>
Cc: Lukasz Laguna <lukasz.laguna@intel.com>
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>

Matthew Auld authored 5 months ago and Kamil Konieczny committed 5 months ago

export-dma-buf-once was split into a few different subtests, but we
missed updating the test list here.

Closes: drm/xe/kernel#3061


Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Nirmoy Das <nirmoy.das@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Jakub197 authored 8 months ago and Marcin Bernatowicz committed 8 months ago

Add SR-IOV tests to xe-fast-feedback testlist so that they will be
executed in BAT runs. Added tests will exercise:
- enabling maximum number of VFs without autoprobe
- enabling one VF with autoprobe

Signed-off-by: Jakub Kolakowski <jakub1.kolakowski@intel.com>
Cc: Adam Miszczak <adam.miszczak@intel.com>
Cc: Marcin Bernatowicz <marcin.bernatowicz@intel.com>
Cc: Lukasz Laguna <lukasz.laguna@intel.com>
Cc: Michal Wajdeczko <Michal.Wajdeczko@intel.com>
Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>

Add few fbdev tests to XE BAT.

Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Acked-by: Karthik B S <karthik.b.s@intel.com>

Francois Dugast authored 11 months ago and Kamil Konieczny committed 11 months ago

The commit ca460793 ("drm/xe/uapi: Remove unused flags") is partially
reverted. At the time, flags not used by user space were removed during
cleanup. As a consequence, tests depending on those flags were also
removed. Some of those flags are now needed by user space, so bring them
back and the removed tests as well.

Align with kernel commit ("drm/xe/uapi: Restore flags \
VM_BIND_FLAG_READONLY and VM_BIND_FLAG_IMMEDIATE").

v2: Fix missing period in kernel doc (Ashutosh Dixit)

Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Acked-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Himanshu Girotra authored 11 months ago and Kamil Konieczny committed 11 months ago

Addback xe_evict and xe_exec_threads to fast feedback testlist to bring
back the coverage of BAT and reorder testlist so that kms tests run
before core tests to encounter problem caused by "6.9.0-rc1-xe"
Regression.

v2: move evict, threads tests at the end(Rama)

Cc: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Signed-off-by: Himanshu Girotra <himanshu.girotra@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Bhanuprakash Modem authored 1 year ago and Kamil Konieczny committed 1 year ago

Regession with "6.9.0-rc1-xe" is preventing to run KMS tests
in BAT. Hence drop "xe_exec_threads" & "xe_evict" from XE BAT
until we fix the regression.

V2:
 - Drop xe_evict tests

Issue: drm/xe/kernel#1044


Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

This aligns with kernel commit ("drm/xe/uapi: Remove unused flags"). As
a consequence, some tests which dependend on the removed flags are also
removed. Also, ensure that the removed flag values are no longer
accepted to prevent mismatch.

v2:
- Rebase after removal of persistent exec_queues (Francois Dugast)
- Update xe-fast-feedback.testlist (Rodrigo Vivi)

Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

The test suite name in the kernel modules contain an xe_ prefix.
Upcoming changes for filtering will make the match more strict,
so rename the subtests to match the test suite name. dmabuf subtest also
needs an extra underscore to match the kernel side.

v2: s/xe_dmabuf/xe_dma_buf/ to really match test suite name (Janusz)

References: https://lore.kernel.org/igt-dev/3267150.N7aMVyhfb1@jkrzyszt-mobl2.ger.corp.intel.com/


Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>

Himanshu Girotra authored 1 year ago and Kamil Konieczny committed 1 year ago

Add xe_vm large subtest to fast feedback to enhance our premerge testing

Signed-off-by: Girotra Himanshu <himanshu.girotra@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>

Nirmoy Das authored 1 year ago and Rahul Janga committed 1 year ago

Add this basic functionality test the fast feedback list.

Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Ensure we have some coverage in BAT for gem_create MBZ fields.

v2: (Kamil)
  - Keep it locally sorted

Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Cc: Pallavi Mishra <pallavi.mishra@intel.com>
Cc: Francois Dugast <francois.dugast@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

kms_psr subtest names have been changed resulting in skips on CI bat
testlist. Update the test names accordingly.

Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/9806



Signed-off-by: Karthik B S <karthik.b.s@intel.com>
Reviewed-by: Jeevan B <jeevan.b@intel.com>

Aligns with kernel commit ("drm/xe: Create a xe_gt_freq component for raw management and sysfs")

v2: Fix copyright year (Kamil)

Cc: Sujaritha Sundaresan <sujaritha.sundaresan@intel.com>
Cc: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
Cc: Riana Tauro <riana.tauro@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Drop i915 specific tests from XE BAT.

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

Add the various pat_index, coh_mode and cpu_caching related tests to
BAT.

Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Pallavi Mishra <pallavi.mishra@intel.com>
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

Add xe_waitfence@engine to xe-fast-feedback.
Test is simple check of waitfence with provided
engine class instance.

Signed-off-by: Daniel Mrzyglod <daniel.t.mrzyglod@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Previously since there were only two categories
psr and psr2, there was no psr_ prefix, updated
testlist with prefix

Cc: Jouni Högander <jouni.hogander@intel.com>
Cc: Animesh Manna <animesh.manna@intel.com>
Cc: Arun R Murthy <arun.r.murthy@intel.com>
Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

Platforms like PVC don't use flatccs compression so on them exercise
eviction only. Rename all subtests and remove "-ccs-" part in it.

Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>

Zbigniew Kempczyński authored 1 year ago and Matthew Auld committed 1 year ago

There are two subtests which should be run in bat.
First is simple check of eviction, second exercises driver in multiple
processes to check does any race influences on object consistency.

Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Katarzyna Dec <katarzyna.dec@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>

a-intel authored 1 year ago and Kamil Konieczny committed 1 year ago

Move xe_exec_fault_mode tests to the end of xe fast-feedback
testlist to unblock execution.

Signed-off-by: Aditya Chauhan <aditya.chauhan@intel.com>
Acked-by: Katarzyna Piecielska <katarzyna.piecielska@intel.com>

Rodrigo Vivi authored 1 year ago and Francois Dugast committed 1 year ago

Align with commit ("drm/xe/uapi: Rename gts to gt_list")

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Francois Dugast <francois.dugast@intel.com>

Align with commit ("drm/xe/uapi: Remove MMIO ioctl")

Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

Swati2 Sharma authored 1 year ago and Kamil Konieczny committed 1 year ago

Add dsc-basic as part of ffb.

Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>

Add 4-tile basic tiling tests to ffb.

Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>

Its safer to run suspend tests under resume runs. Drop
only suspend test from the ffb.

Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Acked-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>

Add basic audio test to i915/XE fast feedback testlist.

Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>

Add basic tiling tests to XE fast feedback testlist.

V2: - Drop i915 specific tests

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Kunal Joshi <kunal1.joshi@intel.com>

Add kms_frontbuffer_tracking tests back to xe fast-feedback testlist and
drop them from xe blocklist.

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Kunal Joshi <kunal1.joshi@intel.com>

In BAT, chamelium setups are not frequently deployed. To enhance
overall execution efficiency, it's recommended to remove chamelium
tests from fast-feedback/xe-fast-feedback and create a dedicated
fast-feedback-chamelium-only/xe-fast-feedback-chamelium-only
testlists for executing chamelium tests.

v2: -update xe fast-feedback testlist (Kunal)
v3: -update commit message (Kamil)
    -sort test names (Kamil)

Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
Reviewed-by: Kunal Joshi <kunal1.joshi@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Mauro Carvalho Chehab authored 1 year ago and Kamil Konieczny committed 1 year ago

Those are executed at build time, and were removed from
xe_live_ktest. So, drop them from the testlist.

Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Daniel Mrzygłód authored 1 year ago and Kamil Konieczny committed 1 year ago

This patch revert testplan change for igt@xe_query
where query-engines which is a proper HW engine test.

This patch fix the line which propably occured by mistake.
-igt@xe_query@query-engines
+igt@xe_query@query-execqueues

Fixes: 481fe148 ("xe: Rename engine to exec_queue")

Signed-off-by: Daniel Mrzyglod <daniel.t.mrzyglod@intel.com>
Cc: francois.dugast@intel.com
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>

Katarzyna Piecielska authored 1 year ago and Kamil Konieczny committed 1 year ago

We received recommendation for KMD team to reduce xe_exec_threads
tests to have only:
igt@xe_exec_threads@threads-basic
igt@xe_exec_threads@threads-mixed-basic
igt@xe_exec_threads@threads-mixed-shared-vm-basic
igt@xe_exec_threads@threads-mixed-fd-basic
NEW igt@xe_exec_threads@threads-mixed-userptr-invalidate
NEW igt@xe_exec_threads@threads-mixed-shared-vm-userptr-invalidate-race

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Gandi Ramadevi <ramadevi.gandi@intel.com>
Cc: Aditya Chauhan <aditya.chauhan@intel.com>
Cc: Himanshu Girotra <himanshu.girotra@intel.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Matthew Brost <matthew.brost@intel.com>
Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Add sysfs_scheduler and sysfs_defaults to BAT.
This validates the basic sysfs scheduler parameters.

Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com>
Acked-by: Tejas Upadhyay <tejas.upadhyay@intel.com>

Engine was inappropriately used to refer to execution queues and
it also created some confusion with hardware engines. Update to
the kernel header where this was fixed and also update variable
names in tests.

The xe_drm.h uapi header is now in sync with drm-xe-next
commit ("drm/xe: Rename engine to exec_queue").

Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

Riana Tauro authored 1 year ago and Anshuman Gupta committed 1 year ago

Add the new test gt-c6-on-idle to BAT.
This test validates if gt is in c6 on idle.

Signed-off-by: Riana Tauro <riana.tauro@intel.com>
Acked-by: Badal Nilawar <badal.nilawar@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>

Katarzyna Piecielska authored 1 year ago and Kamil Konieczny committed 1 year ago

We have reached point where we need to reduce BAT testlist.
First step of cleaning up fast-feedback is covering:
 * remove 'race' testcases
 * reduce xe_exec_balancer, xe_exec_basic, xe_spin_batch test cases

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
Cc: Sai Gowtham Ch <sai.gowtham.ch@intel.com>
Cc: Priyanka Dandamudi <priyanka.dandamudi@intel.com>
Cc: Gandi Ramadevi <ramadevi.gandi@intel.com>
Cc: Aditya Chauhan <aditya.chauhan@intel.com>
Cc: Himanshu Girotra <himanshu.girotra@intel.com>
Reviewed-by: Mauro Carvalho Chehab <mchehab@kernel.org>

Riana Tauro authored 1 year ago and Anshuman Gupta committed 1 year ago

A new test was accidently added to BAT.
Remove the test as its not present.

Fixes: 96f077d7 (xe-fast-feedback.testlist: Remove rc6 tests from
BAT)
Signed-off-by: Riana Tauro <riana.tauro@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>