All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH] [PATCH i-g-t] i915/gem_pipe_control_store_loop: added description for test case
@ 2020-06-30 23:23 ranjeet1.kumar
  2020-07-06  8:18 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
  2020-07-06  8:53 ` [igt-dev] [PATCH] [PATCH i-g-t] " Chris Wilson
  0 siblings, 2 replies; 3+ messages in thread
From: ranjeet1.kumar @ 2020-06-30 23:23 UTC (permalink / raw)
  To: igt-dev, ranjeet1.kumar

From: Ranjeet kumar <ranjeet1.kumar@intel.com>

Added test description for subtest.

Cc: Melkaveri, Arjun <Arjun.Melkaveri@intel.com>
Signed-off-by: Ranjeet kumar <ranjeet1.kumar@intel.com>
---
 tests/i915/gem_pipe_control_store_loop.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/i915/gem_pipe_control_store_loop.c b/tests/i915/gem_pipe_control_store_loop.c
index e44e9fb53..a9ff66895 100644
--- a/tests/i915/gem_pipe_control_store_loop.c
+++ b/tests/i915/gem_pipe_control_store_loop.c
@@ -181,9 +181,12 @@ igt_main
 		igt_assert(batch);
 	}
 
+	igt_describe("Creates new buffer each time, emit commands and send it to ring"
+		     " for execution.")
 	igt_subtest("fresh-buffer")
 		store_pipe_control_loop(false, 2);
 
+	igt_describe("Emit commands in reused buffer and send it to ring for execution.")
 	igt_subtest("reused-buffer")
 		store_pipe_control_loop(true, 2);
 
-- 
2.26.2

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* [igt-dev] ✗ Fi.CI.BUILD: failure for i915/gem_pipe_control_store_loop: added description for test case
  2020-06-30 23:23 [igt-dev] [PATCH] [PATCH i-g-t] i915/gem_pipe_control_store_loop: added description for test case ranjeet1.kumar
@ 2020-07-06  8:18 ` Patchwork
  2020-07-06  8:53 ` [igt-dev] [PATCH] [PATCH i-g-t] " Chris Wilson
  1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2020-07-06  8:18 UTC (permalink / raw)
  To: ranjeet1.kumar; +Cc: igt-dev

== Series Details ==

Series: i915/gem_pipe_control_store_loop: added description for test case
URL   : https://patchwork.freedesktop.org/series/79143/
State : failure

== Summary ==

IGT patchset build failed on latest successful build
75bcaf7a609d184305d5f6ac9da38d9297ed22ef tests/api_intel_bb: Add intel_bb API test

[381/569] Linking target tests/gem_exec_parallel.
[382/569] Linking target tests/gen7_exec_parse.
[383/569] Linking target tests/gen9_exec_parse.
[384/569] Linking target tests/gem_exec_reloc.
[385/569] Linking target tests/gem_exec_store.
[386/569] Linking target tests/gem_exec_schedule.
[387/569] Linking target tests/gem_exec_suspend.
[388/569] Linking target tests/gem_exec_whisper.
[389/569] Linking target tests/gem_fd_exhaustion.
[390/569] Linking target tests/gem_fence_thrash.
[391/569] Linking target tests/gem_flink_race.
[392/569] Linking target tests/gem_fenced_exec_thrash.
[393/569] Linking target tests/gem_fence_upload.
[394/569] Linking target tests/gem_flink_basic.
[395/569] Linking target tests/gem_gtt_hog.
[396/569] Linking target tests/gem_gpgpu_fill.
[397/569] Linking target tests/gem_gtt_cpu_tlb.
[398/569] Linking target tests/gem_lut_handle.
[399/569] Linking target tests/gem_gtt_speed.
[400/569] Linking target tests/gem_linear_blits.
[401/569] Linking target tests/gem_mmap_wc.
[402/569] Linking target tests/gem_madvise.
[403/569] Linking target tests/gem_media_fill.
[404/569] Linking target tests/gem_media_vme.
[405/569] Linking target tests/gem_mmap.
[406/569] Linking target tests/gem_mmap_gtt.
[407/569] Compiling C object 'tests/tests@@gem_pipe_control_store_loop@exe/i915_gem_pipe_control_store_loop.c.o'.
FAILED: tests/tests@@gem_pipe_control_store_loop@exe/i915_gem_pipe_control_store_loop.c.o 
ccache cc -Itests/tests@@gem_pipe_control_store_loop@exe -Itests -I../tests -I../include/drm-uapi -Ilib -I../lib -I../lib/stubs/syscalls -I. -I../ -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/libpng16 -I/usr/include/freetype2 -I/usr/include/libpng12 -I/usr/include/libdrm -I/usr/include/x86_64-linux-gnu -I/usr/include/alsa -I/usr/include -I/usr/include/libdrm/nouveau -I/home/cidrm/kernel_headers/include -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -O2 -g -D_GNU_SOURCE -include config.h -D_FORTIFY_SOURCE=2 -Wbad-function-cast -Wdeclaration-after-statement -Wformat=2 -Wimplicit-fallthrough=0 -Wlogical-op -Wmissing-declarations -Wmissing-format-attribute -Wmissing-noreturn -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wpointer-arith -Wredundant-decls -Wshadow -Wstrict-prototypes -Wuninitialized -Wunused -Wno-clobbered -Wno-maybe-uninitialized -Wno-missing-field-initializers -Wno-pointer-arith -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-unused-result -Werror=address -Werror=array-bounds -Werror=implicit -Werror=init-self -Werror=int-to-pointer-cast -Werror=main -Werror=missing-braces -Werror=nonnull -Werror=pointer-to-int-cast -Werror=return-type -Werror=sequence-point -Werror=trigraphs -Werror=write-strings -fno-builtin-malloc -fno-builtin-calloc -fcommon -pthread  -MD -MQ 'tests/tests@@gem_pipe_control_store_loop@exe/i915_gem_pipe_control_store_loop.c.o' -MF 'tests/tests@@gem_pipe_control_store_loop@exe/i915_gem_pipe_control_store_loop.c.o.d' -o 'tests/tests@@gem_pipe_control_store_loop@exe/i915_gem_pipe_control_store_loop.c.o' -c ../tests/i915/gem_pipe_control_store_loop.c
In file included from ../lib/drmtest.h:39:0,
                 from ../lib/igt.h:27,
                 from ../tests/i915/gem_pipe_control_store_loop.c:45:
../tests/i915/gem_pipe_control_store_loop.c: In function ‘__real_main162’:
../lib/igt_core.h:214:27: error: expected ‘;’ before ‘for’
 #define igt_subtest(name) for (; __igt_run_subtest((name), __FILE__, __LINE__) && \
                           ^
../tests/i915/gem_pipe_control_store_loop.c:188:2: note: in expansion of macro ‘igt_subtest’
  igt_subtest("fresh-buffer")
  ^~~~~~~~~~~
../lib/igt_core.h:214:27: error: expected ‘;’ before ‘for’
 #define igt_subtest(name) for (; __igt_run_subtest((name), __FILE__, __LINE__) && \
                           ^
../tests/i915/gem_pipe_control_store_loop.c:192:2: note: in expansion of macro ‘igt_subtest’
  igt_subtest("reused-buffer")
  ^~~~~~~~~~~
At top level:
../tests/i915/gem_pipe_control_store_loop.c:67:1: warning: ‘store_pipe_control_loop’ defined but not used [-Wunused-function]
 store_pipe_control_loop(bool preuse_buffer, int timeout)
 ^~~~~~~~~~~~~~~~~~~~~~~
ninja: build stopped: subcommand failed.

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [igt-dev] [PATCH] [PATCH i-g-t] i915/gem_pipe_control_store_loop: added description for test case
  2020-06-30 23:23 [igt-dev] [PATCH] [PATCH i-g-t] i915/gem_pipe_control_store_loop: added description for test case ranjeet1.kumar
  2020-07-06  8:18 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
@ 2020-07-06  8:53 ` Chris Wilson
  1 sibling, 0 replies; 3+ messages in thread
From: Chris Wilson @ 2020-07-06  8:53 UTC (permalink / raw)
  To: igt-dev, ranjeet1.kumar

Quoting ranjeet1.kumar@intel.com (2020-07-01 00:23:37)
> From: Ranjeet kumar <ranjeet1.kumar@intel.com>
> 
> Added test description for subtest.
> 
> Cc: Melkaveri, Arjun <Arjun.Melkaveri@intel.com>
> Signed-off-by: Ranjeet kumar <ranjeet1.kumar@intel.com>
> ---
>  tests/i915/gem_pipe_control_store_loop.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tests/i915/gem_pipe_control_store_loop.c b/tests/i915/gem_pipe_control_store_loop.c
> index e44e9fb53..a9ff66895 100644
> --- a/tests/i915/gem_pipe_control_store_loop.c
> +++ b/tests/i915/gem_pipe_control_store_loop.c
> @@ -181,9 +181,12 @@ igt_main
>                 igt_assert(batch);
>         }
>  
> +       igt_describe("Creates new buffer each time, emit commands and send it to ring"
> +                    " for execution.")

i++; /* post increments i by one, returns the old value */

>         igt_subtest("fresh-buffer")
>                 store_pipe_control_loop(false, 2);

That is less valuable than a link to the code.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-07-06  8:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-30 23:23 [igt-dev] [PATCH] [PATCH i-g-t] i915/gem_pipe_control_store_loop: added description for test case ranjeet1.kumar
2020-07-06  8:18 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
2020-07-06  8:53 ` [igt-dev] [PATCH] [PATCH i-g-t] " Chris Wilson

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.