* [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest
@ 2024-01-22 2:45 Bommu Krishnaiah
2024-01-22 2:45 ` [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence Bommu Krishnaiah
` (4 more replies)
0 siblings, 5 replies; 9+ messages in thread
From: Bommu Krishnaiah @ 2024-01-22 2:45 UTC (permalink / raw)
To: igt-dev; +Cc: Bommu Krishnaiah, Rodrigo Vivi
tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence
engine and instance members are removed from drm_xe_wait_user_fence,
hence invalid_engine subtest is invalid and removing it.
kernel implementation:
https://patchwork.freedesktop.org/series/127365/
failure link: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1067
tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait
drm_xe_sync flags are declared wrongly and drm_xe_sync data is not used
in this test, hence removing drm_xe_sync which is unused.
failure link: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1081
tests/xe_waitfence: Removed duplicate code in waitfence
Removed duplicate code in xe_waitfence@waitfence.
Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Bommu Krishnaiah (3):
tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence
tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait
tests/xe_waitfence: Removed duplicate code in waitfence
lib/xe/xe_ioctl.c | 35 -------------------------------
lib/xe/xe_ioctl.h | 2 --
tests/intel/xe_waitfence.c | 42 ++------------------------------------
3 files changed, 2 insertions(+), 77 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah @ 2024-01-22 2:45 ` Bommu Krishnaiah 2024-01-22 20:53 ` Rodrigo Vivi 2024-01-22 2:45 ` [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait Bommu Krishnaiah ` (3 subsequent siblings) 4 siblings, 1 reply; 9+ messages in thread From: Bommu Krishnaiah @ 2024-01-22 2:45 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah, Rodrigo Vivi engine and instance members are removed from drm_xe_wait_user_fence, hence invalid_engine subtest is invalid and removing it. kernel implementation: https://patchwork.freedesktop.org/series/127365/ Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> --- tests/intel/xe_waitfence.c | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c index 7ba20764c..dfdb97fb5 100644 --- a/tests/intel/xe_waitfence.c +++ b/tests/intel/xe_waitfence.c @@ -151,9 +151,6 @@ waitfence(int fd, enum waittype wt) * SUBTEST: invalid-ops * Description: Check query with invalid ops returns expected error code * - * SUBTEST: invalid-engine - * Description: Check query with invalid engine info returns expected error code - * * SUBTEST: exec_queue-reset-wait * Description: Don’t wait till timeout on user fence when exec_queue reset is detected and return return proper error */ @@ -206,30 +203,6 @@ invalid_ops(int fd) do_ioctl_err(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait, EINVAL); } -static void -invalid_engine(int fd) -{ - uint32_t bo; - - struct drm_xe_wait_user_fence wait = { - .addr = to_user_pointer(&wait_fence), - .op = DRM_XE_UFENCE_WAIT_OP_EQ, - .flags = 0, - .value = 1, - .mask = DRM_XE_UFENCE_WAIT_MASK_U64, - .timeout = -1, - .exec_queue_id = 0, - }; - - uint32_t vm = xe_vm_create(fd, 0, 0); - - bo = xe_bo_create(fd, vm, 0x40000, vram_if_possible(fd, 0), 0); - - do_bind(fd, vm, bo, 0, 0x200000, 0x40000, 1); - - do_ioctl_err(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait, EFAULT); -} - static void exec_queue_reset_wait(int fd) { @@ -329,9 +302,6 @@ igt_main igt_subtest("invalid-ops") invalid_ops(fd); - igt_subtest("invalid-engine") - invalid_engine(fd); - igt_subtest("exec_queue-reset-wait") exec_queue_reset_wait(fd); -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence 2024-01-22 2:45 ` [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence Bommu Krishnaiah @ 2024-01-22 20:53 ` Rodrigo Vivi 0 siblings, 0 replies; 9+ messages in thread From: Rodrigo Vivi @ 2024-01-22 20:53 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev On Mon, Jan 22, 2024 at 08:15:41AM +0530, Bommu Krishnaiah wrote: > engine and instance members are removed from drm_xe_wait_user_fence, > hence invalid_engine subtest is invalid and removing it. > > kernel implementation: > https://patchwork.freedesktop.org/series/127365/ > > Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > --- > tests/intel/xe_waitfence.c | 30 ------------------------------ > 1 file changed, 30 deletions(-) > > diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c > index 7ba20764c..dfdb97fb5 100644 > --- a/tests/intel/xe_waitfence.c > +++ b/tests/intel/xe_waitfence.c > @@ -151,9 +151,6 @@ waitfence(int fd, enum waittype wt) > * SUBTEST: invalid-ops > * Description: Check query with invalid ops returns expected error code > * > - * SUBTEST: invalid-engine > - * Description: Check query with invalid engine info returns expected error code > - * > * SUBTEST: exec_queue-reset-wait > * Description: Don’t wait till timeout on user fence when exec_queue reset is detected and return return proper error > */ > @@ -206,30 +203,6 @@ invalid_ops(int fd) > do_ioctl_err(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait, EINVAL); > } > > -static void > -invalid_engine(int fd) > -{ > - uint32_t bo; > - > - struct drm_xe_wait_user_fence wait = { > - .addr = to_user_pointer(&wait_fence), > - .op = DRM_XE_UFENCE_WAIT_OP_EQ, > - .flags = 0, > - .value = 1, > - .mask = DRM_XE_UFENCE_WAIT_MASK_U64, > - .timeout = -1, > - .exec_queue_id = 0, > - }; > - > - uint32_t vm = xe_vm_create(fd, 0, 0); > - > - bo = xe_bo_create(fd, vm, 0x40000, vram_if_possible(fd, 0), 0); > - > - do_bind(fd, vm, bo, 0, 0x200000, 0x40000, 1); > - > - do_ioctl_err(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait, EFAULT); > -} > - > static void > exec_queue_reset_wait(int fd) > { > @@ -329,9 +302,6 @@ igt_main > igt_subtest("invalid-ops") > invalid_ops(fd); > > - igt_subtest("invalid-engine") > - invalid_engine(fd); > - > igt_subtest("exec_queue-reset-wait") > exec_queue_reset_wait(fd); > > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah 2024-01-22 2:45 ` [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence Bommu Krishnaiah @ 2024-01-22 2:45 ` Bommu Krishnaiah 2024-01-22 20:53 ` Rodrigo Vivi 2024-01-22 2:45 ` [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence Bommu Krishnaiah ` (2 subsequent siblings) 4 siblings, 1 reply; 9+ messages in thread From: Bommu Krishnaiah @ 2024-01-22 2:45 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah, Rodrigo Vivi drm_xe_sync flags are declared wrongly and drm_xe_sync data is not used in this test, hence removing drm_xe_sync which is unused. Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> --- tests/intel/xe_waitfence.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c index dfdb97fb5..e6354b3ff 100644 --- a/tests/intel/xe_waitfence.c +++ b/tests/intel/xe_waitfence.c @@ -221,16 +221,8 @@ exec_queue_reset_wait(int fd) uint32_t data; } *data; -#define USER_FENCE_VALUE 0xdeadbeefdeadbeefull - struct drm_xe_sync sync[1] = { - { .flags = DRM_XE_SYNC_TYPE_USER_FENCE | DRM_XE_SYNC_FLAG_SIGNAL, - .timeline_value = USER_FENCE_VALUE }, - }; - struct drm_xe_exec exec = { .num_batch_buffer = 1, - .num_syncs = 1, - .syncs = to_user_pointer(sync), }; uint32_t vm = xe_vm_create(fd, 0, 0); -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait 2024-01-22 2:45 ` [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait Bommu Krishnaiah @ 2024-01-22 20:53 ` Rodrigo Vivi 0 siblings, 0 replies; 9+ messages in thread From: Rodrigo Vivi @ 2024-01-22 20:53 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev On Mon, Jan 22, 2024 at 08:15:42AM +0530, Bommu Krishnaiah wrote: > drm_xe_sync flags are declared wrongly and drm_xe_sync data is not used > in this test, hence removing drm_xe_sync which is unused. > > Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > --- > tests/intel/xe_waitfence.c | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c > index dfdb97fb5..e6354b3ff 100644 > --- a/tests/intel/xe_waitfence.c > +++ b/tests/intel/xe_waitfence.c > @@ -221,16 +221,8 @@ exec_queue_reset_wait(int fd) > uint32_t data; > } *data; > > -#define USER_FENCE_VALUE 0xdeadbeefdeadbeefull > - struct drm_xe_sync sync[1] = { > - { .flags = DRM_XE_SYNC_TYPE_USER_FENCE | DRM_XE_SYNC_FLAG_SIGNAL, > - .timeline_value = USER_FENCE_VALUE }, > - }; > - > struct drm_xe_exec exec = { > .num_batch_buffer = 1, > - .num_syncs = 1, > - .syncs = to_user_pointer(sync), > }; > > uint32_t vm = xe_vm_create(fd, 0, 0); > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah 2024-01-22 2:45 ` [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence Bommu Krishnaiah 2024-01-22 2:45 ` [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait Bommu Krishnaiah @ 2024-01-22 2:45 ` Bommu Krishnaiah 2024-01-22 20:54 ` Rodrigo Vivi 2024-01-22 3:13 ` ✓ CI.xeBAT: success for tests/xe_waitfence: removed invalid_engine subtest (rev4) Patchwork 2024-01-22 3:33 ` ✗ Fi.CI.BAT: failure " Patchwork 4 siblings, 1 reply; 9+ messages in thread From: Bommu Krishnaiah @ 2024-01-22 2:45 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah, Rodrigo Vivi Removed duplicate code in xe_waitfence@waitfence Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> --- lib/xe/xe_ioctl.c | 35 ----------------------------------- lib/xe/xe_ioctl.h | 2 -- tests/intel/xe_waitfence.c | 4 ++-- 3 files changed, 2 insertions(+), 39 deletions(-) diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c index 39605a019..49c5d359e 100644 --- a/lib/xe/xe_ioctl.c +++ b/lib/xe/xe_ioctl.c @@ -523,41 +523,6 @@ int64_t xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, return timeout; } -/** - * xe_wait_ufence_abstime: - * @fd: xe device fd - * @addr: address of value to compare - * @value: expected value (equal) in @address - * @exec_queue: exec_queue id - * @timeout: absolute time when wait expire - * @flag: wait flag - * - * Function compares @value with memory pointed by @addr until they are equal. - * Asserts that ioctl returned without error. - * - * Returns elapsed time in nanoseconds if user fence was signalled. - */ -int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, - uint32_t exec_queue, int64_t timeout, - uint16_t flag) -{ - struct drm_xe_wait_user_fence wait = { - .addr = to_user_pointer(addr), - .op = DRM_XE_UFENCE_WAIT_OP_EQ, - .flags = flag, - .value = value, - .mask = DRM_XE_UFENCE_WAIT_MASK_U64, - .timeout = timeout, - .exec_queue_id = exec_queue, - }; - struct timespec ts; - - igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait), 0); - igt_assert_eq(clock_gettime(CLOCK_MONOTONIC, &ts), 0); - - return ts.tv_sec * 1e9 + ts.tv_nsec; -} - void xe_force_gt_reset(int fd, int gt) { char reset_string[128]; diff --git a/lib/xe/xe_ioctl.h b/lib/xe/xe_ioctl.h index 8a92073b0..03932561d 100644 --- a/lib/xe/xe_ioctl.h +++ b/lib/xe/xe_ioctl.h @@ -90,8 +90,6 @@ int __xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, uint32_t exec_queue, int64_t *timeout); int64_t xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, uint32_t exec_queue, int64_t timeout); -int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, uint32_t - exec_queue, int64_t timeout, uint16_t flag); void xe_force_gt_reset(int fd, int gt); #endif /* XE_IOCTL_H */ diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c index e6354b3ff..6ed9d56f8 100644 --- a/tests/intel/xe_waitfence.c +++ b/tests/intel/xe_waitfence.c @@ -36,7 +36,7 @@ static void do_bind(int fd, uint32_t vm, uint32_t bo, uint64_t offset, xe_vm_bind_async(fd, vm, 0, bo, offset, addr, size, sync, 1); } -static int64_t wait_with_eci_abstime(int fd, uint64_t *addr, uint64_t value, +static int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, uint32_t exec_queue, int64_t timeout, uint16_t flag) { @@ -117,7 +117,7 @@ waitfence(int fd, enum waittype wt) clock_gettime(CLOCK_MONOTONIC, &ts); current = ts.tv_sec * 1e9 + ts.tv_nsec; timeout = current + MS_TO_NS(10); - signalled = wait_with_eci_abstime(fd, &wait_fence, 7, + signalled = xe_wait_ufence_abstime(fd, &wait_fence, 7, exec_queue, timeout, DRM_XE_UFENCE_WAIT_FLAG_ABSTIME); igt_debug("wait type: ENGINE ABSTIME - timeout: %" PRId64 -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence 2024-01-22 2:45 ` [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence Bommu Krishnaiah @ 2024-01-22 20:54 ` Rodrigo Vivi 0 siblings, 0 replies; 9+ messages in thread From: Rodrigo Vivi @ 2024-01-22 20:54 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev On Mon, Jan 22, 2024 at 08:15:43AM +0530, Bommu Krishnaiah wrote: > Removed duplicate code in xe_waitfence@waitfence > > Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > --- > lib/xe/xe_ioctl.c | 35 ----------------------------------- > lib/xe/xe_ioctl.h | 2 -- > tests/intel/xe_waitfence.c | 4 ++-- > 3 files changed, 2 insertions(+), 39 deletions(-) > > diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c > index 39605a019..49c5d359e 100644 > --- a/lib/xe/xe_ioctl.c > +++ b/lib/xe/xe_ioctl.c > @@ -523,41 +523,6 @@ int64_t xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, > return timeout; > } > > -/** > - * xe_wait_ufence_abstime: > - * @fd: xe device fd > - * @addr: address of value to compare > - * @value: expected value (equal) in @address > - * @exec_queue: exec_queue id > - * @timeout: absolute time when wait expire > - * @flag: wait flag > - * > - * Function compares @value with memory pointed by @addr until they are equal. > - * Asserts that ioctl returned without error. > - * > - * Returns elapsed time in nanoseconds if user fence was signalled. > - */ > -int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, > - uint32_t exec_queue, int64_t timeout, > - uint16_t flag) > -{ > - struct drm_xe_wait_user_fence wait = { > - .addr = to_user_pointer(addr), > - .op = DRM_XE_UFENCE_WAIT_OP_EQ, > - .flags = flag, > - .value = value, > - .mask = DRM_XE_UFENCE_WAIT_MASK_U64, > - .timeout = timeout, > - .exec_queue_id = exec_queue, > - }; > - struct timespec ts; > - > - igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_XE_WAIT_USER_FENCE, &wait), 0); > - igt_assert_eq(clock_gettime(CLOCK_MONOTONIC, &ts), 0); > - > - return ts.tv_sec * 1e9 + ts.tv_nsec; > -} > - > void xe_force_gt_reset(int fd, int gt) > { > char reset_string[128]; > diff --git a/lib/xe/xe_ioctl.h b/lib/xe/xe_ioctl.h > index 8a92073b0..03932561d 100644 > --- a/lib/xe/xe_ioctl.h > +++ b/lib/xe/xe_ioctl.h > @@ -90,8 +90,6 @@ int __xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, > uint32_t exec_queue, int64_t *timeout); > int64_t xe_wait_ufence(int fd, uint64_t *addr, uint64_t value, > uint32_t exec_queue, int64_t timeout); > -int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, uint32_t > - exec_queue, int64_t timeout, uint16_t flag); > void xe_force_gt_reset(int fd, int gt); > > #endif /* XE_IOCTL_H */ > diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c > index e6354b3ff..6ed9d56f8 100644 > --- a/tests/intel/xe_waitfence.c > +++ b/tests/intel/xe_waitfence.c > @@ -36,7 +36,7 @@ static void do_bind(int fd, uint32_t vm, uint32_t bo, uint64_t offset, > xe_vm_bind_async(fd, vm, 0, bo, offset, addr, size, sync, 1); > } > > -static int64_t wait_with_eci_abstime(int fd, uint64_t *addr, uint64_t value, > +static int64_t xe_wait_ufence_abstime(int fd, uint64_t *addr, uint64_t value, > uint32_t exec_queue, int64_t timeout, > uint16_t flag) No need for the 'xe_' prefix addition in a local static function. with that changed, Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > { > @@ -117,7 +117,7 @@ waitfence(int fd, enum waittype wt) > clock_gettime(CLOCK_MONOTONIC, &ts); > current = ts.tv_sec * 1e9 + ts.tv_nsec; > timeout = current + MS_TO_NS(10); > - signalled = wait_with_eci_abstime(fd, &wait_fence, 7, > + signalled = xe_wait_ufence_abstime(fd, &wait_fence, 7, > exec_queue, timeout, > DRM_XE_UFENCE_WAIT_FLAG_ABSTIME); > igt_debug("wait type: ENGINE ABSTIME - timeout: %" PRId64 > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* ✓ CI.xeBAT: success for tests/xe_waitfence: removed invalid_engine subtest (rev4) 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah ` (2 preceding siblings ...) 2024-01-22 2:45 ` [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence Bommu Krishnaiah @ 2024-01-22 3:13 ` Patchwork 2024-01-22 3:33 ` ✗ Fi.CI.BAT: failure " Patchwork 4 siblings, 0 replies; 9+ messages in thread From: Patchwork @ 2024-01-22 3:13 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev [-- Attachment #1: Type: text/plain, Size: 986 bytes --] == Series Details == Series: tests/xe_waitfence: removed invalid_engine subtest (rev4) URL : https://patchwork.freedesktop.org/series/127926/ State : success == Summary == CI Bug Log - changes from XEIGT_7683_BAT -> XEIGTPW_10567_BAT ==================================================== Summary ------- **SUCCESS** No regressions found. Participating hosts (4 -> 4) ------------------------------ No changes in participating hosts Changes ------- No changes found Build changes ------------- * IGT: IGT_7683 -> IGTPW_10567 * Linux: xe-644-238e8655c184b7cf16731690b59da560641a07ad -> xe-656-b239ffd479309ebbf0bd530ef632c8ef3ee78d7a IGTPW_10567: 10567 IGT_7683: 7683 xe-644-238e8655c184b7cf16731690b59da560641a07ad: 238e8655c184b7cf16731690b59da560641a07ad xe-656-b239ffd479309ebbf0bd530ef632c8ef3ee78d7a: b239ffd479309ebbf0bd530ef632c8ef3ee78d7a == Logs == For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_10567/index.html [-- Attachment #2: Type: text/html, Size: 1545 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* ✗ Fi.CI.BAT: failure for tests/xe_waitfence: removed invalid_engine subtest (rev4) 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah ` (3 preceding siblings ...) 2024-01-22 3:13 ` ✓ CI.xeBAT: success for tests/xe_waitfence: removed invalid_engine subtest (rev4) Patchwork @ 2024-01-22 3:33 ` Patchwork 4 siblings, 0 replies; 9+ messages in thread From: Patchwork @ 2024-01-22 3:33 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev [-- Attachment #1: Type: text/plain, Size: 3607 bytes --] == Series Details == Series: tests/xe_waitfence: removed invalid_engine subtest (rev4) URL : https://patchwork.freedesktop.org/series/127926/ State : failure == Summary == CI Bug Log - changes from CI_DRM_14149 -> IGTPW_10567 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_10567 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_10567, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/index.html Participating hosts (38 -> 35) ------------------------------ Additional (1): fi-pnv-d510 Missing (4): bat-mtlp-8 bat-rpls-2 bat-atsm-1 fi-snb-2520m Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_10567: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live@reset: - bat-adlm-1: [PASS][1] -> [INCOMPLETE][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14149/bat-adlm-1/igt@i915_selftest@live@reset.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/bat-adlm-1/igt@i915_selftest@live@reset.html Known issues ------------ Here are the changes found in IGTPW_10567 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@core_hotunplug@unbind-rebind: - bat-adlp-6: [PASS][3] -> [DMESG-WARN][4] ([i915#1982]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14149/bat-adlp-6/igt@core_hotunplug@unbind-rebind.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/bat-adlp-6/igt@core_hotunplug@unbind-rebind.html * igt@gem_exec_suspend@basic-s0@lmem0: - bat-dg2-8: [PASS][5] -> [INCOMPLETE][6] ([i915#9275]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14149/bat-dg2-8/igt@gem_exec_suspend@basic-s0@lmem0.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/bat-dg2-8/igt@gem_exec_suspend@basic-s0@lmem0.html * igt@gem_exec_suspend@basic-s0@smem: - bat-dg2-9: [PASS][7] -> [INCOMPLETE][8] ([i915#9275]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14149/bat-dg2-9/igt@gem_exec_suspend@basic-s0@smem.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/bat-dg2-9/igt@gem_exec_suspend@basic-s0@smem.html * igt@gem_lmem_swapping@basic: - fi-pnv-d510: NOTRUN -> [SKIP][9] ([fdo#109271]) +31 other tests skip [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/fi-pnv-d510/igt@gem_lmem_swapping@basic.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591 [i915#9275]: https://gitlab.freedesktop.org/drm/intel/issues/9275 Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_7683 -> IGTPW_10567 CI-20190529: 20190529 CI_DRM_14149: b239ffd479309ebbf0bd530ef632c8ef3ee78d7a @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_10567: 10567 IGT_7683: 7683 Testlist changes ---------------- -igt@xe_waitfence@invalid-engine == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_10567/index.html [-- Attachment #2: Type: text/html, Size: 4341 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-01-22 20:55 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-01-22 2:45 [PATCH v4 0/3] tests/xe_waitfence: removed invalid_engine subtest Bommu Krishnaiah 2024-01-22 2:45 ` [PATCH v4 1/3] tests/xe_waitfence: remove invalid_engine subtest from xe_waitfence Bommu Krishnaiah 2024-01-22 20:53 ` Rodrigo Vivi 2024-01-22 2:45 ` [PATCH v4 2/3] tests/xe_waitfence: Removed drm_xe_sync data in exec_queue_reset_wait Bommu Krishnaiah 2024-01-22 20:53 ` Rodrigo Vivi 2024-01-22 2:45 ` [PATCH v4 3/3] tests/xe_waitfence: Removed duplicate code in waitfence Bommu Krishnaiah 2024-01-22 20:54 ` Rodrigo Vivi 2024-01-22 3:13 ` ✓ CI.xeBAT: success for tests/xe_waitfence: removed invalid_engine subtest (rev4) Patchwork 2024-01-22 3:33 ` ✗ Fi.CI.BAT: failure " Patchwork
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox