Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
@ 2023-08-16 16:17 Erik Kurzinger
  2023-08-16 18:23 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Erik Kurzinger @ 2023-08-16 16:17 UTC (permalink / raw)
  To: igt-dev

The syncobj_timeline test expects waiting for an unsubmitted fence with
the WAIT_AVAILABLE flag set to fail with EINVAL. While this matches the
behavior of current kernels, that behavior is incorrect and will be
fixed by
https://lists.freedesktop.org/archives/dri-devel/2023-August/418710.html

With that fix, the WAIT_AVAILABLE flag will have the intended semantics
of waiting for an unsubmitted fence to be submitted instead of failing.

Note that this will cause the test to fail with kernels that lack the
fix, but such failures will reflect a genuine defect.

Signed-off-by: Erik Kurzinger <ekurzinger@nvidia.com>
---
 tests/syncobj_timeline.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/syncobj_timeline.c b/tests/syncobj_timeline.c
index b4e1d093a..0422bca51 100644
--- a/tests/syncobj_timeline.c
+++ b/tests/syncobj_timeline.c
@@ -1782,7 +1782,7 @@ igt_main
 						WAIT_SIGNALED)) != 1)
 			continue;
 
-		if ((flags & WAIT_UNSUBMITTED) && !(flags & WAIT_FOR_SUBMIT))
+		if ((flags & WAIT_UNSUBMITTED) && !((flags & WAIT_FOR_SUBMIT) || (flags & WAIT_AVAILABLE)))
 			err = -EINVAL;
 		else if (!(flags & WAIT_SIGNALED) && !((flags & WAIT_SUBMITTED) && (flags & WAIT_AVAILABLE)))
 			err = -ETIME;
@@ -1851,7 +1851,7 @@ igt_main
 			continue;
 
 		err = 0;
-		if ((flags & WAIT_UNSUBMITTED) && !(flags & WAIT_FOR_SUBMIT)) {
+		if ((flags & WAIT_UNSUBMITTED) && !((flags & WAIT_FOR_SUBMIT) || (flags & WAIT_AVAILABLE))) {
 			err = -EINVAL;
 		} else if (flags & WAIT_ALL) {
 			if (flags & (WAIT_UNSUBMITTED | WAIT_SUBMITTED))
-- 
2.41.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [igt-dev] ✗ Fi.CI.BUILD: failure for syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
  2023-08-16 16:17 [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE Erik Kurzinger
@ 2023-08-16 18:23 ` Patchwork
  2023-08-16 18:27 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
  2023-08-21 10:26 ` [igt-dev] [PATCH] " Zbigniew Kempczyński
  2 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2023-08-16 18:23 UTC (permalink / raw)
  To: Erik Kurzinger; +Cc: igt-dev

== Series Details ==

Series: syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
URL   : https://patchwork.freedesktop.org/series/122525/
State : failure

== Summary ==

IGT patchset build failed on latest successful build
152bb04fd1297075b5d0b1c4487dac8e9a70d070 tests/amdgpu: fix cs nop test when no GFX IP

Tail of build.log:
Warning: igt@xe_exercise_blt@fast-copy-emit Category documentation is missing
Warning: igt@xe_media_fill@media-fill Functionality documentation is missing
Warning: igt@xe_media_fill@media-fill Sub-category documentation is missing
Warning: igt@xe_media_fill@media-fill Category documentation is missing
Warning: igt@xe_pm@vram-d3cold-threshold Functionality documentation is missing
Warning: igt@xe_sysfs_scheduler@job_timeout_ms-invalid Category documentation is missing
Warning: igt@xe_sysfs_scheduler@job_timeout_ms-min-max Category documentation is missing
Warning: igt@xe_sysfs_scheduler@job_timeout_ms-nonprivileged-user Category documentation is missing
Warning: igt@xe_sysfs_scheduler@preempt_timeout_us-invalid Category documentation is missing
Warning: igt@xe_sysfs_scheduler@preempt_timeout_us-min-max Category documentation is missing
Warning: igt@xe_sysfs_scheduler@preempt_timeout_us-nonprivileged-user Category documentation is missing
Warning: igt@xe_sysfs_scheduler@timeslice_duration_us-invalid Category documentation is missing
Warning: igt@xe_sysfs_scheduler@timeslice_duration_us-min-max Category documentation is missing
Warning: igt@xe_sysfs_scheduler@timeslice_duration_us-nonprivileged-user Category documentation is missing
Warning: igt@xe_sysfs_tile@physical_vram_size_bytes Category documentation is missing
Warning: igt@xe_sysfs_tile@physical_vram_size_bytes Run type documentation is missing
Warning: igt@xe_uevent@fake_reset_uevent_listener Category documentation is missing
Warning: igt@xe_uevent@fake_reset_uevent_listener Run type documentation is missing
Warning: igt@xe_vm@mmap-style-bind-all Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-either-side-full Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-either-side-partial Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-either-side-partial-hammer Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-either-side-partial-large-page-hammer Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-either-side-partial-split-page-hammer Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-end Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-front Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-many-all Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-many-either-side-partial Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-many-either-side-partial-hammer Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-one-partial Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-userptr-all Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-userptr-either-side-full Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-userptr-either-side-partial Functionality documentation is missing
Warning: igt@xe_vm@mmap-style-bind-userptr-one-partial Functionality documentation is missing
[1617/1622] Generating i915_tests.rst with a custom command.
FAILED: docs/testplan/i915_tests.rst 
/usr/src/igt-gpu-tools/scripts/igt_doc.py --config /usr/src/igt-gpu-tools/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /opt/igt/build
Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
[1618/1622] Generating kms_tests.rst with a custom command.
[1619/1622] Generating xe_tests.html with a custom command.
ninja: build stopped: subcommand failed.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [igt-dev] ✗ GitLab.Pipeline: warning for syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
  2023-08-16 16:17 [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE Erik Kurzinger
  2023-08-16 18:23 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
@ 2023-08-16 18:27 ` Patchwork
  2023-08-21 10:26 ` [igt-dev] [PATCH] " Zbigniew Kempczyński
  2 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2023-08-16 18:27 UTC (permalink / raw)
  To: Erik Kurzinger; +Cc: igt-dev

== Series Details ==

Series: syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
URL   : https://patchwork.freedesktop.org/series/122525/
State : warning

== Summary ==

Pipeline status: FAILED.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/963088 for the overview.

build:tests-debian-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531572):
  [4/5] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210329:step_script
  section_start:1692210329:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210329:cleanup_file_variables
  ERROR: Job failed: exit code 1
  

build:tests-fedora has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531567):
  [4/5] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210342:step_script
  section_start:1692210342:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210343:cleanup_file_variables
  ERROR: Job failed: exit code 1
  

build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531571):
  [2/4] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210360:step_script
  section_start:1692210360:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210362:cleanup_file_variables
  ERROR: Job failed: exit code 1
  

build:tests-fedora-no-libdrm-nouveau has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531570):
  [3/5] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210335:step_script
  section_start:1692210335:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210335:cleanup_file_variables
  ERROR: Job failed: exit code 1
  

build:tests-fedora-no-libunwind has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531568):
  [3/4] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210338:step_script
  section_start:1692210338:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210339:cleanup_file_variables
  ERROR: Job failed: exit code 1
  

build:tests-fedora-oldest-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/47531569):
  [3/5] Generating i915_tests.rst with a custom command.
  FAILED: docs/testplan/i915_tests.rst 
  /builds/gfx-ci/igt-ci-tags/scripts/igt_doc.py --config /builds/gfx-ci/igt-ci-tags/tests/i915/i915_test_config.json --rest docs/testplan/i915_tests.rst --check-testlist --igt-build-path /builds/gfx-ci/igt-ci-tags/build
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-all-available-unsubmitted-submitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@etime-multi-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-all-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@etime-single-wait-available-unsubmitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-signaled
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted
  Warning: Missing documentation for igt@syncobj_timeline@multi-wait-available-unsubmitted-submitted-signaled
  ninja: build stopped: subcommand failed.
  section_end:1692210329:step_script
  section_start:1692210329:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1692210330:cleanup_file_variables
  ERROR: Job failed: exit code 1

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/963088

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE
  2023-08-16 16:17 [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE Erik Kurzinger
  2023-08-16 18:23 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
  2023-08-16 18:27 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
@ 2023-08-21 10:26 ` Zbigniew Kempczyński
  2 siblings, 0 replies; 4+ messages in thread
From: Zbigniew Kempczyński @ 2023-08-21 10:26 UTC (permalink / raw)
  To: Erik Kurzinger; +Cc: igt-dev

On Wed, Aug 16, 2023 at 09:17:56AM -0700, Erik Kurzinger wrote:
> The syncobj_timeline test expects waiting for an unsubmitted fence with
> the WAIT_AVAILABLE flag set to fail with EINVAL. While this matches the
> behavior of current kernels, that behavior is incorrect and will be
> fixed by
> https://lists.freedesktop.org/archives/dri-devel/2023-August/418710.html
> 
> With that fix, the WAIT_AVAILABLE flag will have the intended semantics
> of waiting for an unsubmitted fence to be submitted instead of failing.
> 
> Note that this will cause the test to fail with kernels that lack the
> fix, but such failures will reflect a genuine defect.
> 
> Signed-off-by: Erik Kurzinger <ekurzinger@nvidia.com>
> ---
>  tests/syncobj_timeline.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/syncobj_timeline.c b/tests/syncobj_timeline.c
> index b4e1d093a..0422bca51 100644
> --- a/tests/syncobj_timeline.c
> +++ b/tests/syncobj_timeline.c
> @@ -1782,7 +1782,7 @@ igt_main
>  						WAIT_SIGNALED)) != 1)
>  			continue;
>  
> -		if ((flags & WAIT_UNSUBMITTED) && !(flags & WAIT_FOR_SUBMIT))
> +		if ((flags & WAIT_UNSUBMITTED) && !((flags & WAIT_FOR_SUBMIT) || (flags & WAIT_AVAILABLE)))
>  			err = -EINVAL;
>  		else if (!(flags & WAIT_SIGNALED) && !((flags & WAIT_SUBMITTED) && (flags & WAIT_AVAILABLE)))
>  			err = -ETIME;
> @@ -1851,7 +1851,7 @@ igt_main
>  			continue;
>  
>  		err = 0;
> -		if ((flags & WAIT_UNSUBMITTED) && !(flags & WAIT_FOR_SUBMIT)) {
> +		if ((flags & WAIT_UNSUBMITTED) && !((flags & WAIT_FOR_SUBMIT) || (flags & WAIT_AVAILABLE))) {
>  			err = -EINVAL;
>  		} else if (flags & WAIT_ALL) {
>  			if (flags & (WAIT_UNSUBMITTED | WAIT_SUBMITTED))
> -- 
> 2.41.0
> 
>

Right, reading your example sent to dri-devel WAIT_AVAILABLE definitely
should wait until reaching requested point of time instead returning
EINVAL immediately if there's no fence submitted before.

Merging this before kernel change will land is ok for me, at least
bug will be filled. But please resend this series one more time as
buildbot didn't pass build step. You may add my rb:

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
--
Zbigniew

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-08-21 10:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-16 16:17 [igt-dev] [PATCH] syncobj_timeline: don't expect EINVAL for WAIT_UNSUBMITTED | WAIT_AVAILABLE Erik Kurzinger
2023-08-16 18:23 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
2023-08-16 18:27 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
2023-08-21 10:26 ` [igt-dev] [PATCH] " Zbigniew Kempczyński

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox