All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lucas De Marchi <lucas.demarchi@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: thomas.hellstrom@linux.intel.com,
	Lucas De Marchi <lucas.demarchi@intel.com>,
	petri.latvala@intel.com, nirmoy.das@intel.com
Subject: [igt-dev] [PATCH i-g-t] tests/intel-ci/fast-feedback: Move gem_lmem_swapping
Date: Tue,  5 Apr 2022 14:19:13 -0700	[thread overview]
Message-ID: <20220405211913.2728176-1-lucas.demarchi@intel.com> (raw)

Since commit 37ec3b8ffd2c ("tests/gem_lmem_swapping: limit lmem to 4G")
we load the driver with limited 4GB of lmem for test execution. However
this means we also have to 1) reload the driver with that param set and
2) unload it after we are done. However tests/intel-ci/fast-feedback.testlist
has a dedicated section for tests that rely on module (un|re)loading so
not to impact other tests.

Move all gem_lmem_swapping to that section, after the coment:

	# All tests that do module unloading and reloading are executed last.
	# They will sometimes reveal issues of earlier tests leaving the
	# driver in a broken state that is not otherwise noticed in that test.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index c1014558..e24d2fa4 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -34,10 +34,6 @@ igt@gem_flink_basic@double-flink
 igt@gem_flink_basic@flink-lifetime
 igt@gem_huc_copy@huc-copy
 igt@gem_linear_blits@basic
-igt@gem_lmem_swapping@basic
-igt@gem_lmem_swapping@parallel-random-engines
-igt@gem_lmem_swapping@random-engines
-igt@gem_lmem_swapping@verify-random
 igt@gem_mmap@basic
 igt@gem_mmap_gtt@basic
 igt@gem_render_linear_blits@basic
@@ -170,6 +166,10 @@ igt@vgem_basic@sysfs
 # They will sometimes reveal issues of earlier tests leaving the
 # driver in a broken state that is not otherwise noticed in that test.
 
+igt@gem_lmem_swapping@basic
+igt@gem_lmem_swapping@parallel-random-engines
+igt@gem_lmem_swapping@random-engines
+igt@gem_lmem_swapping@verify-random
 igt@core_hotunplug@unbind-rebind
 igt@vgem_basic@unload
 igt@i915_module_load@reload
-- 
2.35.1

             reply	other threads:[~2022-04-05 21:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-05 21:19 Lucas De Marchi [this message]
2022-04-05 22:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/intel-ci/fast-feedback: Move gem_lmem_swapping Patchwork
2022-04-06  7:36 ` [igt-dev] [PATCH i-g-t] " Petri Latvala
2022-04-06 14:40   ` Lucas De Marchi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220405211913.2728176-1-lucas.demarchi@intel.com \
    --to=lucas.demarchi@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=nirmoy.das@intel.com \
    --cc=petri.latvala@intel.com \
    --cc=thomas.hellstrom@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.