From: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
To: igt-dev@lists.freedesktop.org
Cc: adam.miszczak@linux.intel.com, kamil.konieczny@linux.intel.com,
lukasz.laguna@intel.com, zbigniew.kempczynski@intel.com,
Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
Subject: [PATCH i-g-t 0/3] gem_wsim: fix batch premature completion
Date: Fri, 31 Jul 2026 16:18:03 +0200 [thread overview]
Message-ID: <20260731141806.890207-1-marcin.bernatowicz@linux.intel.com> (raw)
Fixes observed verification failure under looped runs:
gem_wsim -w "1.RCS.5000.0.0,1.BCS.5000.0.0,1.CCS.5000.0.1,1.VCS.5000.0.0"
-V -r 4000
CRITICAL: Failed assertion:
w->duration.requested_ticks <= ~w->xe.data->spin.ticks_delta
CRITICAL: error: 96000 > 2
V1 attempted to fix the issue by initializing ticks_delta to ~0u in
xe_spin and resetting it from the host before each fixed-duration
submission in gem_wsim.
V2 fix (suggested by Zbigniew): write sentinel 0 to ticks_delta and use
MI_SEMAPHORE_WAIT to confirm it's in memory before entering the loop.
This is a self-contained GPU-side fix - no host-side reset per submission
needed. Also switches from inverted ticks_delta (STOREINV + ~ctx_ticks)
to direct elapsed (STORE + MAD_LT_IDD + ctx_ticks) for readability.
V1 -> V2:
- Replace host-side ticks_delta reset with GPU-side SDW+SEMAPHORE_WAIT
- Switch from inverted to direct elapsed ticks logic
- Remove ineffective pad loop between SRM and COND_BBE
- Add xe_sriov_scheduling.c fix for the same ticks_delta check
Marcin Bernatowicz (3):
lib/xe/xe_spin: fix premature batch exit on BO resubmission
benchmarks/gem_wsim: fix ticks_delta assertion for non-inverted logic
tests/xe_sriov_scheduling: fix ticks_delta check for non-inverted
logic
benchmarks/gem_wsim.c | 2 +-
lib/xe/xe_spin.c | 31 +++++++++++++++++--------------
tests/intel/xe_sriov_scheduling.c | 2 +-
3 files changed, 19 insertions(+), 16 deletions(-)
--
2.43.0
next reply other threads:[~2026-07-31 14:18 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-31 14:18 Marcin Bernatowicz [this message]
2026-07-31 14:18 ` [PATCH i-g-t 1/3] lib/xe/xe_spin: fix premature batch exit on BO resubmission Marcin Bernatowicz
2026-07-31 15:52 ` Zbigniew Kempczyński
2026-07-31 14:18 ` [PATCH i-g-t 2/3] benchmarks/gem_wsim: fix ticks_delta assertion for non-inverted logic Marcin Bernatowicz
2026-07-31 15:53 ` Zbigniew Kempczyński
2026-07-31 14:18 ` [PATCH i-g-t 3/3] tests/xe_sriov_scheduling: fix ticks_delta check " Marcin Bernatowicz
2026-07-31 15:54 ` Zbigniew Kempczyński
2026-07-31 14:53 ` ✓ Xe.CI.BAT: success for gem_wsim: fix batch premature completion (rev2) Patchwork
2026-07-31 15:25 ` ✓ i915.CI.BAT: " Patchwork
2026-07-31 16:37 ` ✓ Xe.CI.FULL: " Patchwork
2026-07-31 18:34 ` ✓ i915.CI.Full: " Patchwork
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=20260731141806.890207-1-marcin.bernatowicz@linux.intel.com \
--to=marcin.bernatowicz@linux.intel.com \
--cc=adam.miszczak@linux.intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=kamil.konieczny@linux.intel.com \
--cc=lukasz.laguna@intel.com \
--cc=zbigniew.kempczynski@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.