Re: [igt-dev] [PATCH i-g-t] xe_live_ktest: remove wa from the testlist

[Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>]

On Thu, 10 Aug 2023 06:23:50 +0200
Mauro Carvalho Chehab <mauro.chehab@linux.intel.com> wrote:

> On Wed, 9 Aug 2023 09:00:28 -0600
> Lucas De Marchi <lucas.demarchi@intel.com> wrote:
> 
> > On Wed, Aug 09, 2023 at 04:48:45PM +0200, Mauro Carvalho Chehab wrote:  
> > >From: Mauro Carvalho Chehab <mchehab@kernel.org>
> > >
> > >The workaround selftest is not a live test. It is meant to run
> > >via kunit.py, as part of the driver-independent validation.    
> > 
> > s/driver-independent/device-independent/
> > 
> >   
> > >
> > >So, remove it from IGT.
> > >
> > >Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > >Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> > >---
> > > tests/xe/xe_live_ktest.c | 1 -
> > > 1 file changed, 1 deletion(-)
> > >
> > >diff --git a/tests/xe/xe_live_ktest.c b/tests/xe/xe_live_ktest.c
> > >index f0cbf4f623bc..53cc8c186215 100644
> > >--- a/tests/xe/xe_live_ktest.c
> > >+++ b/tests/xe/xe_live_ktest.c
> > >@@ -40,7 +40,6 @@ static const struct kunit_tests live_tests[] = {
> > > 	{ "xe_migrate_test",	"migrate" },
> > > 	{ "xe_pci_test",	"pci" },
> > > 	{ "xe_rtp_test",	"rtp" },    
> > 
> > this on too, but could be a separate patch  
> 
> Do you mean xe_rtp_test? I'll send a separate patch for it.

Hmm... from the logs, both xe_pci ana xe_rtp are device-independent tests:

[08:16:01] =================== xe_pci (2 subtests) ====================
[08:16:01] [PASSED] xe_gmdid_graphics_ip
[08:16:01] [PASSED] xe_gmdid_media_ip
[08:16:01] ===================== [PASSED] xe_pci ======================
[08:16:01] ==================== xe_rtp (1 subtest) ====================
[08:16:01] ================== xe_rtp_process_tests  ===================
[08:16:01] [PASSED] coalesce-same-reg
[08:16:01] [PASSED] no-match-no-add
[08:16:01] [PASSED] no-match-no-add-multiple-rules
[08:16:01] [PASSED] two-regs-two-entries
[08:16:01] [PASSED] clr-one-set-other
[08:16:01] [PASSED] set-field
[08:16:01] [PASSED] conflict-duplicate
[08:16:01] [PASSED] conflict-not-disjoint
[08:16:01] [PASSED] conflict-reg-type
[08:16:01] ============== [PASSED] xe_rtp_process_tests ===============
[08:16:01] ===================== [PASSED] xe_rtp ======================

I'll place both altogether at the same patch.

Regards,
Mauro