* [igt-dev] [PATCH 0/2] tests/kms_writeback: Test Cleanup
@ 2022-08-25 21:06 Nidhi Gupta
2022-08-25 21:06 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Nidhi Gupta @ 2022-08-25 21:06 UTC (permalink / raw)
To: igt-dev; +Cc: Nidhi Gupta
Nidhi Gupta (2):
tests/kms_writeback: Convert tests to dynamic
tests/kms_writeback: Test Cleanup
tests/kms_writeback.c | 65 ++++++++++++++++++++++++++++++++-----------
1 file changed, 49 insertions(+), 16 deletions(-)
--
2.36.0
^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-08-25 21:06 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
@ 2022-08-25 21:06 ` Nidhi Gupta
2022-08-26 18:15 ` Jessica Zhang
2022-08-25 21:06 ` [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
2022-08-25 21:36 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
2 siblings, 1 reply; 10+ messages in thread
From: Nidhi Gupta @ 2022-08-25 21:06 UTC (permalink / raw)
To: igt-dev; +Cc: Nidhi Gupta
Convert the existing subtests to dynamic subtests.
Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
---
tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
1 file changed, 41 insertions(+), 16 deletions(-)
diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
index 9d134585..2508b1e1 100644
--- a/tests/kms_writeback.c
+++ b/tests/kms_writeback.c
@@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
uint32_t fb_id;
bool ptr_valid;
int32_t *out_fence_ptr;
+ const char *name;
} invalid_tests[] = {
{
/* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
.fb_id = 0,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "WRITEBACK_OUT_FENCE_PTR-set",
},
{
/* Invalid output buffer. */
.fb_id = invalid_fb->fb_id,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "Invalid-output-buffer",
},
{
/* Invalid WRITEBACK_OUT_FENCE_PTR. */
.fb_id = valid_fb->fb_id,
.ptr_valid = false,
.out_fence_ptr = (int32_t *)0x8,
+ .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
},
};
for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
- ret = do_writeback_test(output, invalid_tests[i].fb_id,
- invalid_tests[i].out_fence_ptr,
- invalid_tests[i].ptr_valid);
- igt_assert(ret != 0);
+ igt_dynamic_f("%s\n", invalid_tests[i].name) {
+ ret = do_writeback_test(output, invalid_tests[i].fb_id,
+ invalid_tests[i].out_fence_ptr,
+ invalid_tests[i].ptr_valid);
+ igt_assert(ret != 0);
+ }
}
}
@@ -247,18 +253,37 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
{
int ret;
+ struct {
+ const char *name;
+ uint32_t fb_id;
+ int i,ret1;
+ } fb_id_tests[] = {
+ {
+ .name = "Invalid-object",
+ .fb_id = output->id,
+ .ret1 = -EINVAL,
+ },
- /* Invalid object for WRITEBACK_FB_ID */
- ret = do_writeback_test(output, output->id, NULL, false);
- igt_assert(ret == -EINVAL);
+ {
+ .name = "Zero-WRITEBACK_FB_ID",
+ .fb_id = 0,
+ .ret1 = 0,
+ },
+
+ {
+ .name = "Valid-output-buffer",
+ .fb_id = valid_fb->fb_id,
+ .ret1 = 0,
+ },
+ }
+ for (i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
+ igt_dynamic_f("%s\n", fb_id_tests[i].name) {
+ do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
+ igt_assert(ret == fb_id_tests[i].ret1);
+ }
+ }
- /* Zero WRITEBACK_FB_ID */
- ret = do_writeback_test(output, 0, NULL, false);
- igt_assert(ret == 0);
- /* Valid output buffer */
- ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
- igt_assert(ret == 0);
}
static void fill_fb(igt_fb_t *fb, uint32_t pixel)
@@ -553,7 +578,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
igt_describe("Writeback has a couple of parameters linked together"
"(output framebuffer and fence); this test goes through"
"the combination of possible bad options");
- igt_subtest("writeback-invalid-parameters") {
+ igt_subtest_with_dynamic("writeback-invalid-parameters") {
igt_fb_t invalid_output_fb;
igt_skip_on(data.dump_check || data.list_modes);
@@ -570,7 +595,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
}
igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
- igt_subtest("writeback-fb-id") {
+ igt_subtest_with_dynamic("writeback-fb-id") {
igt_fb_t output_fb;
igt_skip_on(data.dump_check || data.list_modes);
@@ -586,7 +611,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
}
igt_describe("Check writeback output with CRC validation");
- igt_subtest("writeback-check-output") {
+ igt_subtest_with_dynamic("writeback-check-output") {
igt_fb_t output_fb;
igt_skip_on(data.dump_check || data.list_modes);
--
2.36.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup
2022-08-25 21:06 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
2022-08-25 21:06 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
@ 2022-08-25 21:06 ` Nidhi Gupta
2022-08-26 18:49 ` Jessica Zhang
2022-08-25 21:36 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
2 siblings, 1 reply; 10+ messages in thread
From: Nidhi Gupta @ 2022-08-25 21:06 UTC (permalink / raw)
To: igt-dev; +Cc: Nidhi Gupta
Sanitize the system state before starting the subtest.
Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
---
tests/kms_writeback.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
index 2508b1e1..0c6c64b3 100644
--- a/tests/kms_writeback.c
+++ b/tests/kms_writeback.c
@@ -169,6 +169,7 @@ static void detach_crtc(igt_display_t *display, igt_output_t *output)
if (get_writeback_fb_id(output) == 0)
return;
+ igt_display_fini(&display);
igt_output_set_pipe(output, PIPE_NONE);
igt_display_commit2(display, COMMIT_ATOMIC);
}
@@ -580,6 +581,9 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
"the combination of possible bad options");
igt_subtest_with_dynamic("writeback-invalid-parameters") {
igt_fb_t invalid_output_fb;
+ igt_display_reset(&display);
+ igt_display_commit(&display);
+
igt_skip_on(data.dump_check || data.list_modes);
fb_id = igt_create_fb(display.drm_fd, mode.hdisplay / 2,
@@ -597,6 +601,8 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
igt_subtest_with_dynamic("writeback-fb-id") {
igt_fb_t output_fb;
+ igt_display_reset(&display);
+ igt_display_commit(&display);
igt_skip_on(data.dump_check || data.list_modes);
fb_id = igt_create_fb(display.drm_fd, mode.hdisplay, mode.vdisplay,
@@ -613,6 +619,8 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
igt_describe("Check writeback output with CRC validation");
igt_subtest_with_dynamic("writeback-check-output") {
igt_fb_t output_fb;
+ igt_display_reset(&display);
+ igt_display_commit(&display);
igt_skip_on(data.dump_check || data.list_modes);
fb_id = igt_create_fb(display.drm_fd, mode.hdisplay, mode.vdisplay,
--
2.36.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [igt-dev] ✗ Fi.CI.BUILD: failure for tests/kms_writeback: Test Cleanup
2022-08-25 21:06 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
2022-08-25 21:06 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
2022-08-25 21:06 ` [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
@ 2022-08-25 21:36 ` Patchwork
2 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-08-25 21:36 UTC (permalink / raw)
To: Nidhi Gupta; +Cc: igt-dev
== Series Details ==
Series: tests/kms_writeback: Test Cleanup
URL : https://patchwork.freedesktop.org/series/107764/
State : failure
== Summary ==
IGT patchset build failed on latest successful build
1298b5f0e1b3e010657ffba41d2e775fab028e08 igt: Allow overriding the commit hash in the version string
[334/663] Linking target tests/kms_plane_alpha_blend
[335/663] Linking target tests/kms_plane_lowres
[336/663] Linking target tests/kms_plane_multiple
[337/663] Linking target tests/kms_plane_scaling
[338/663] Linking target tests/kms_prop_blob
[339/663] Linking target tests/kms_prime
[340/663] Linking target tests/kms_rotation_crc
[341/663] Linking target tests/kms_properties
[342/663] Linking target tests/kms_rmfb
[343/663] Linking target tests/kms_scaling_modes
[344/663] Linking target tests/kms_selftest
[345/663] Linking target tests/kms_sequence
[346/663] Compiling C object tests/kms_writeback.p/kms_writeback.c.o
FAILED: tests/kms_writeback.p/kms_writeback.c.o
ccache cc -Itests/kms_writeback.p -Itests -I../tests -I../include/drm-uapi -I../include/linux-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/freetype2 -I/usr/include/libpng16 -I/usr/include/libpng12 -I/usr/include/libdrm -I/usr/include/libdrm/nouveau -I/usr/include/x86_64-linux-gnu -I/usr/include/alsa -I/usr/include -I/home/cidrm/kernel_headers/include -fdiagnostics-color=always -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/kms_writeback.p/kms_writeback.c.o -MF tests/kms_writeback.p/kms_writeback.c.o.d -o tests/kms_writeback.p/kms_writeback.c.o -c ../tests/kms_writeback.c
../tests/kms_writeback.c: In function ‘detach_crtc’:
../tests/kms_writeback.c:172:19: warning: passing argument 1 of ‘igt_display_fini’ from incompatible pointer type [-Wincompatible-pointer-types]
igt_display_fini(&display);
^~~~~~~~
In file included from ../lib/igt.h:38,
from ../tests/kms_writeback.c:31:
../lib/igt_kms.h:454:38: note: expected ‘igt_display_t *’ {aka ‘struct igt_display *’} but argument is of type ‘igt_display_t **’ {aka ‘struct igt_display **’}
void igt_display_fini(igt_display_t *display);
~~~~~~~~~~~~~~~^~~~~~~
../tests/kms_writeback.c: In function ‘writeback_fb_id’:
../tests/kms_writeback.c:280:2: error: expected ‘,’ or ‘;’ before ‘for’
for (i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
^~~
../tests/kms_writeback.c:280:14: error: ‘i’ undeclared (first use in this function)
for (i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
^
../tests/kms_writeback.c:280:14: note: each undeclared identifier is reported only once for each function it appears in
../tests/kms_writeback.c:280:46: error: expected ‘;’ before ‘)’ token
for (i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
^
;
../tests/kms_writeback.c:280:46: error: expected statement before ‘)’ token
In file included from ../lib/drmtest.h:39,
from ../lib/igt.h:27,
from ../tests/kms_writeback.c:31:
../lib/igt_core.h:350:16: warning: left-hand operand of comma expression has no effect [-Wunused-value]
format), \
^
../lib/igt_core.h:372:2: note: in expansion of macro ‘__igt_dynamic_f’
__igt_dynamic_f(igt_unique(__tmpchar), f)
^~~~~~~~~~~~~~~
../tests/kms_writeback.c:281:3: note: in expansion of macro ‘igt_dynamic_f’
igt_dynamic_f("%s\n", fb_id_tests[i].name) {
^~~~~~~~~~~~~
ninja: build stopped: subcommand failed.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-08-25 21:06 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
@ 2022-08-26 18:15 ` Jessica Zhang
0 siblings, 0 replies; 10+ messages in thread
From: Jessica Zhang @ 2022-08-26 18:15 UTC (permalink / raw)
To: Nidhi Gupta; +Cc: igt-dev
Hi Nidhi,
On 8/25/2022 2:06 PM, Nidhi Gupta wrote:
> Convert the existing subtests to dynamic subtests.
>
> Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
> ---
> tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
> 1 file changed, 41 insertions(+), 16 deletions(-)
>
> diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
> index 9d134585..2508b1e1 100644
> --- a/tests/kms_writeback.c
> +++ b/tests/kms_writeback.c
> @@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
> uint32_t fb_id;
> bool ptr_valid;
> int32_t *out_fence_ptr;
> + const char *name;
> } invalid_tests[] = {
> {
> /* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
> .fb_id = 0,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "WRITEBACK_OUT_FENCE_PTR-set",
> },
> {
> /* Invalid output buffer. */
> .fb_id = invalid_fb->fb_id,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "Invalid-output-buffer",
> },
> {
> /* Invalid WRITEBACK_OUT_FENCE_PTR. */
> .fb_id = valid_fb->fb_id,
> .ptr_valid = false,
> .out_fence_ptr = (int32_t *)0x8,
> + .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
> },
> };
>
> for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
> - ret = do_writeback_test(output, invalid_tests[i].fb_id,
> - invalid_tests[i].out_fence_ptr,
> - invalid_tests[i].ptr_valid);
> - igt_assert(ret != 0);
> + igt_dynamic_f("%s\n", invalid_tests[i].name) {
From `valid_name_for_subtest()`: "check the subtest name only contains
a-z, A-Z, 0-9, '-' and '_'"[1].
Please remove the newline character from dynamic subtest name.
[1]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/lib/igt_core.c#L1276
> + ret = do_writeback_test(output, invalid_tests[i].fb_id,
> + invalid_tests[i].out_fence_ptr,
> + invalid_tests[i].ptr_valid);
> + igt_assert(ret != 0);
> + }
> }
> }
>
> @@ -247,18 +253,37 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
> {
>
> int ret;
> + struct {
> + const char *name;
Please fix whitespace inconsistency.
> + uint32_t fb_id;
> + int i,ret1;
`ret1` is a little vague for a variable name. Renaming it to something
like `expected_ret` would improve readability.
In addition, please move the `int i` declaration to outside of the struct.
> + } fb_id_tests[] = {
> + {
> + .name = "Invalid-object",
> + .fb_id = output->id,
> + .ret1 = -EINVAL, > + },
>
> - /* Invalid object for WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, output->id, NULL, false);
> - igt_assert(ret == -EINVAL);
> + {
> + .name = "Zero-WRITEBACK_FB_ID",
> + .fb_id = 0,
> + .ret1 = 0,
> + },
> +
> + {
> + .name = "Valid-output-buffer",
> + .fb_id = valid_fb->fb_id,
> + .ret1 = 0,
> + },
> + }
Missing semicolon.
> + for (i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
> + igt_dynamic_f("%s\n", fb_id_tests[i].name) {
Same as above -- please remove newline character.
> + do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
> + igt_assert(ret == fb_id_tests[i].ret1);
Please initialize `ret` before comparing it.
Thanks,
Jessica Zhang
> + }
> + }
>
> - /* Zero WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, 0, NULL, false);
> - igt_assert(ret == 0);
>
> - /* Valid output buffer */
> - ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
> - igt_assert(ret == 0);
> }
>
> static void fill_fb(igt_fb_t *fb, uint32_t pixel)
> @@ -553,7 +578,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> igt_describe("Writeback has a couple of parameters linked together"
> "(output framebuffer and fence); this test goes through"
> "the combination of possible bad options");
> - igt_subtest("writeback-invalid-parameters") {
> + igt_subtest_with_dynamic("writeback-invalid-parameters") {
> igt_fb_t invalid_output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> @@ -570,7 +595,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> }
>
> igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
> - igt_subtest("writeback-fb-id") {
> + igt_subtest_with_dynamic("writeback-fb-id") {
> igt_fb_t output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> @@ -586,7 +611,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> }
>
> igt_describe("Check writeback output with CRC validation");
> - igt_subtest("writeback-check-output") {
> + igt_subtest_with_dynamic("writeback-check-output") {
> igt_fb_t output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> --
> 2.36.0
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup
2022-08-25 21:06 ` [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
@ 2022-08-26 18:49 ` Jessica Zhang
0 siblings, 0 replies; 10+ messages in thread
From: Jessica Zhang @ 2022-08-26 18:49 UTC (permalink / raw)
To: Nidhi Gupta, igt-dev
Hi Nidhi,
On 8/25/2022 2:06 PM, Nidhi Gupta wrote:
> Sanitize the system state before starting the subtest.
>
> Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
> ---
> tests/kms_writeback.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
> index 2508b1e1..0c6c64b3 100644
> --- a/tests/kms_writeback.c
> +++ b/tests/kms_writeback.c
> @@ -169,6 +169,7 @@ static void detach_crtc(igt_display_t *display, igt_output_t *output)
> if (get_writeback_fb_id(output) == 0)
> return;
>
> + igt_display_fini(&display);
igt_display_fini() frees the memory for various members of the
igt_display_t struct [1]. Since there's already a call to
igt_display_fini() in the same fixture that detach_crtc() is called [2],
adding an extra call here will cause a double free.
[1]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/lib/igt_kms.c#L2752
[2]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/tests/kms_writeback.c#L607
> igt_output_set_pipe(output, PIPE_NONE);
> igt_display_commit2(display, COMMIT_ATOMIC);
> }
> @@ -580,6 +581,9 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> "the combination of possible bad options");
> igt_subtest_with_dynamic("writeback-invalid-parameters") {
> igt_fb_t invalid_output_fb;
> + igt_display_reset(&display);
> + igt_display_commit(&display);
I have a doubt about this:
There's a lot of set up going on in the initial igt_fixture here [3],
specifically within kms_writeback_get_output() [4], where the output
pipe is set.
If you just reset the display then do a commit, the output->pending pipe
will be set to NONE within igt_output_reset() [5]. This will cause the
commit to detach the CRTC (in a similar manner to what detach_crtc()
does [6]).
If you don't redo the setup from the initial igt_fixture after resetting
the display, the commits within do_writeback_test() will fail since
there won't be a CRTC attached to the connector.
Thanks,
Jessica Zhang
[3]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/tests/kms_writeback.c#L491
[4]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/tests/kms_writeback.c#L108
[5]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/lib/igt_kms.c#L2180
[6]
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/tests/kms_writeback.c#L167
> +
>
> igt_skip_on(data.dump_check || data.list_modes);
> fb_id = igt_create_fb(display.drm_fd, mode.hdisplay / 2,
> @@ -597,6 +601,8 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
> igt_subtest_with_dynamic("writeback-fb-id") {
> igt_fb_t output_fb;
> + igt_display_reset(&display);
> + igt_display_commit(&display);
>
> igt_skip_on(data.dump_check || data.list_modes);
> fb_id = igt_create_fb(display.drm_fd, mode.hdisplay, mode.vdisplay,
> @@ -613,6 +619,8 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> igt_describe("Check writeback output with CRC validation");
> igt_subtest_with_dynamic("writeback-check-output") {
> igt_fb_t output_fb;
> + igt_display_reset(&display);
> + igt_display_commit(&display);
>
> igt_skip_on(data.dump_check || data.list_modes);
> fb_id = igt_create_fb(display.drm_fd, mode.hdisplay, mode.vdisplay,
> --
> 2.36.0
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-09-22 0:58 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
@ 2022-09-22 10:18 ` Modem, Bhanuprakash
0 siblings, 0 replies; 10+ messages in thread
From: Modem, Bhanuprakash @ 2022-09-22 10:18 UTC (permalink / raw)
To: Nidhi Gupta, igt-dev
On Thu-22-09-2022 06:28 am, Nidhi Gupta wrote:
> Convert the existing subtests to dynamic subtests.
>
> Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
> ---
> tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
> 1 file changed, 42 insertions(+), 15 deletions(-)
>
> diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
> index 9d134585..3781fa34 100644
> --- a/tests/kms_writeback.c
> +++ b/tests/kms_writeback.c
> @@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
> uint32_t fb_id;
> bool ptr_valid;
> int32_t *out_fence_ptr;
> + const char *name;
> } invalid_tests[] = {
> {
> /* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
> .fb_id = 0,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "WRITEBACK_OUT_FENCE_PTR-set",
Please avoid mixed case, also use "-" instead of "_".
This comment is applicable to all places.
> },
> {
> /* Invalid output buffer. */
> .fb_id = invalid_fb->fb_id,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "Invalid-output-buffer",
> },
> {
> /* Invalid WRITEBACK_OUT_FENCE_PTR. */
> .fb_id = valid_fb->fb_id,
> .ptr_valid = false,
> .out_fence_ptr = (int32_t *)0x8,
> + .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
> },
> };
>
> for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
> - ret = do_writeback_test(output, invalid_tests[i].fb_id,
> - invalid_tests[i].out_fence_ptr,
> - invalid_tests[i].ptr_valid);
> - igt_assert(ret != 0);
> + igt_dynamic_f("%s", invalid_tests[i].name) {
> + ret = do_writeback_test(output, invalid_tests[i].fb_id,
> + invalid_tests[i].out_fence_ptr,
> + invalid_tests[i].ptr_valid);
> + igt_assert(ret != 0);
> + }
> }
> }
>
> @@ -247,18 +253,39 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
> {
>
> int ret;
> + struct {
> + const char *name;
> + uint32_t fb_id;
> + int i, expected_ret;
> + } fb_id_tests[] = {
>
> - /* Invalid object for WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, output->id, NULL, false);
> - igt_assert(ret == -EINVAL);
> + {
> + .name = "Invalid-object",
> + .fb_id = output->id,
> + .expected_ret = -EINVAL,
> + },
> +
Newlines are not required.
> + {
> + .name = "Zero-WRITEBACK_FB_ID",
> + .fb_id = 0,
> + .expected_ret = 0,
> + },
> +
> + {
> + .name = "Valid-output-buffer",
> + .fb_id = valid_fb->fb_id,
> + .expected_ret = 0,
> + },
> + };
> +
> + for (int i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
> + igt_dynamic_f("%s", fb_id_tests[i].name) {
> + ret = do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
> + igt_assert(ret == fb_id_tests[i].expected_ret);
> + }
> + }
>
> - /* Zero WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, 0, NULL, false);
> - igt_assert(ret == 0);
>
> - /* Valid output buffer */
> - ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
> - igt_assert(ret == 0);
> }
>
> static void fill_fb(igt_fb_t *fb, uint32_t pixel)
> @@ -553,7 +580,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> igt_describe("Writeback has a couple of parameters linked together"
> "(output framebuffer and fence); this test goes through"
> "the combination of possible bad options");
> - igt_subtest("writeback-invalid-parameters") {
> + igt_subtest_with_dynamic("writeback-invalid-parameters") {
> igt_fb_t invalid_output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> @@ -570,7 +597,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> }
>
> igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
> - igt_subtest("writeback-fb-id") {
> + igt_subtest_with_dynamic("writeback-fb-id") {
> igt_fb_t output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-09-22 0:58 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test cleanup Nidhi Gupta
@ 2022-09-22 0:58 ` Nidhi Gupta
2022-09-22 10:18 ` Modem, Bhanuprakash
0 siblings, 1 reply; 10+ messages in thread
From: Nidhi Gupta @ 2022-09-22 0:58 UTC (permalink / raw)
To: igt-dev; +Cc: Nidhi Gupta
Convert the existing subtests to dynamic subtests.
Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
---
tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
1 file changed, 42 insertions(+), 15 deletions(-)
diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
index 9d134585..3781fa34 100644
--- a/tests/kms_writeback.c
+++ b/tests/kms_writeback.c
@@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
uint32_t fb_id;
bool ptr_valid;
int32_t *out_fence_ptr;
+ const char *name;
} invalid_tests[] = {
{
/* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
.fb_id = 0,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "WRITEBACK_OUT_FENCE_PTR-set",
},
{
/* Invalid output buffer. */
.fb_id = invalid_fb->fb_id,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "Invalid-output-buffer",
},
{
/* Invalid WRITEBACK_OUT_FENCE_PTR. */
.fb_id = valid_fb->fb_id,
.ptr_valid = false,
.out_fence_ptr = (int32_t *)0x8,
+ .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
},
};
for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
- ret = do_writeback_test(output, invalid_tests[i].fb_id,
- invalid_tests[i].out_fence_ptr,
- invalid_tests[i].ptr_valid);
- igt_assert(ret != 0);
+ igt_dynamic_f("%s", invalid_tests[i].name) {
+ ret = do_writeback_test(output, invalid_tests[i].fb_id,
+ invalid_tests[i].out_fence_ptr,
+ invalid_tests[i].ptr_valid);
+ igt_assert(ret != 0);
+ }
}
}
@@ -247,18 +253,39 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
{
int ret;
+ struct {
+ const char *name;
+ uint32_t fb_id;
+ int i, expected_ret;
+ } fb_id_tests[] = {
- /* Invalid object for WRITEBACK_FB_ID */
- ret = do_writeback_test(output, output->id, NULL, false);
- igt_assert(ret == -EINVAL);
+ {
+ .name = "Invalid-object",
+ .fb_id = output->id,
+ .expected_ret = -EINVAL,
+ },
+
+ {
+ .name = "Zero-WRITEBACK_FB_ID",
+ .fb_id = 0,
+ .expected_ret = 0,
+ },
+
+ {
+ .name = "Valid-output-buffer",
+ .fb_id = valid_fb->fb_id,
+ .expected_ret = 0,
+ },
+ };
+
+ for (int i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
+ igt_dynamic_f("%s", fb_id_tests[i].name) {
+ ret = do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
+ igt_assert(ret == fb_id_tests[i].expected_ret);
+ }
+ }
- /* Zero WRITEBACK_FB_ID */
- ret = do_writeback_test(output, 0, NULL, false);
- igt_assert(ret == 0);
- /* Valid output buffer */
- ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
- igt_assert(ret == 0);
}
static void fill_fb(igt_fb_t *fb, uint32_t pixel)
@@ -553,7 +580,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
igt_describe("Writeback has a couple of parameters linked together"
"(output framebuffer and fence); this test goes through"
"the combination of possible bad options");
- igt_subtest("writeback-invalid-parameters") {
+ igt_subtest_with_dynamic("writeback-invalid-parameters") {
igt_fb_t invalid_output_fb;
igt_skip_on(data.dump_check || data.list_modes);
@@ -570,7 +597,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
}
igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
- igt_subtest("writeback-fb-id") {
+ igt_subtest_with_dynamic("writeback-fb-id") {
igt_fb_t output_fb;
igt_skip_on(data.dump_check || data.list_modes);
--
2.36.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-09-08 8:48 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
@ 2022-09-08 18:03 ` Jessica Zhang
0 siblings, 0 replies; 10+ messages in thread
From: Jessica Zhang @ 2022-09-08 18:03 UTC (permalink / raw)
To: Nidhi Gupta, igt-dev
Hi Nidhi,
On 9/8/2022 1:48 AM, Nidhi Gupta wrote:
> Convert the existing subtests to dynamic subtests.
>
> Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
> ---
> tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
> 1 file changed, 42 insertions(+), 15 deletions(-)
>
> diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
> index 9d134585..2c5421ce 100644
> --- a/tests/kms_writeback.c
> +++ b/tests/kms_writeback.c
> @@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
> uint32_t fb_id;
> bool ptr_valid;
> int32_t *out_fence_ptr;
> + const char *name;
> } invalid_tests[] = {
> {
> /* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
> .fb_id = 0,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "WRITEBACK_OUT_FENCE_PTR-set",
> },
> {
> /* Invalid output buffer. */
> .fb_id = invalid_fb->fb_id,
> .ptr_valid = true,
> .out_fence_ptr = &out_fence,
> + .name = "Invalid-output-buffer",
> },
> {
> /* Invalid WRITEBACK_OUT_FENCE_PTR. */
> .fb_id = valid_fb->fb_id,
> .ptr_valid = false,
> .out_fence_ptr = (int32_t *)0x8,
> + .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
> },
> };
>
> for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
> - ret = do_writeback_test(output, invalid_tests[i].fb_id,
> - invalid_tests[i].out_fence_ptr,
> - invalid_tests[i].ptr_valid);
> - igt_assert(ret != 0);
> + igt_dynamic_f("%s", invalid_tests[i].name) {
> + ret = do_writeback_test(output, invalid_tests[i].fb_id,
> + invalid_tests[i].out_fence_ptr,
> + invalid_tests[i].ptr_valid);
> + igt_assert(ret != 0);
> + }
> }
> }
>
> @@ -247,18 +253,39 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
> {
>
> int ret;
> + struct {
> + const char *name;
Please fix the whitespace inconsistency here -- I believe it might be
caused by this line using spaces when it should be tabs instead.
> + uint32_t fb_id;
> + int i, expected_ret;
Any reason to keep the `int i` declaration here? I'm not seeing it being
used elsewhere.
Thanks,
Jessica Zhang
> + } fb_id_tests[] = {
>
> - /* Invalid object for WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, output->id, NULL, false);
> - igt_assert(ret == -EINVAL);
> + {
> + .name = "Invalid-object",
> + .fb_id = output->id,
> + .expected_ret = -EINVAL,
> + },
> +
> + {
> + .name = "Zero-WRITEBACK_FB_ID",
> + .fb_id = 0,
> + .expected_ret = 0,
> + },
> +
> + {
> + .name = "Valid-output-buffer",
> + .fb_id = valid_fb->fb_id,
> + .expected_ret = 0,
> + },
> + };
> +
> + for (int i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
> + igt_dynamic_f("%s", fb_id_tests[i].name) {
> + ret = do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
> + igt_assert(ret == fb_id_tests[i].expected_ret);
> + }
> + }
>
> - /* Zero WRITEBACK_FB_ID */
> - ret = do_writeback_test(output, 0, NULL, false);
> - igt_assert(ret == 0);
>
> - /* Valid output buffer */
> - ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
> - igt_assert(ret == 0);
> }
>
> static void fill_fb(igt_fb_t *fb, uint32_t pixel)
> @@ -553,7 +580,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> igt_describe("Writeback has a couple of parameters linked together"
> "(output framebuffer and fence); this test goes through"
> "the combination of possible bad options");
> - igt_subtest("writeback-invalid-parameters") {
> + igt_subtest_with_dynamic("writeback-invalid-parameters") {
> igt_fb_t invalid_output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> @@ -570,7 +597,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
> }
>
> igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
> - igt_subtest("writeback-fb-id") {
> + igt_subtest_with_dynamic("writeback-fb-id") {
> igt_fb_t output_fb;
>
> igt_skip_on(data.dump_check || data.list_modes);
> --
> 2.36.0
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic
2022-09-08 8:48 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test cleanup Nidhi Gupta
@ 2022-09-08 8:48 ` Nidhi Gupta
2022-09-08 18:03 ` Jessica Zhang
0 siblings, 1 reply; 10+ messages in thread
From: Nidhi Gupta @ 2022-09-08 8:48 UTC (permalink / raw)
To: igt-dev; +Cc: Nidhi Gupta
Convert the existing subtests to dynamic subtests.
Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
---
tests/kms_writeback.c | 57 +++++++++++++++++++++++++++++++------------
1 file changed, 42 insertions(+), 15 deletions(-)
diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c
index 9d134585..2c5421ce 100644
--- a/tests/kms_writeback.c
+++ b/tests/kms_writeback.c
@@ -214,32 +214,38 @@ static void test_invalid_parameters(igt_output_t *output, igt_fb_t *valid_fb, ig
uint32_t fb_id;
bool ptr_valid;
int32_t *out_fence_ptr;
+ const char *name;
} invalid_tests[] = {
{
/* No output buffer, but the WRITEBACK_OUT_FENCE_PTR set. */
.fb_id = 0,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "WRITEBACK_OUT_FENCE_PTR-set",
},
{
/* Invalid output buffer. */
.fb_id = invalid_fb->fb_id,
.ptr_valid = true,
.out_fence_ptr = &out_fence,
+ .name = "Invalid-output-buffer",
},
{
/* Invalid WRITEBACK_OUT_FENCE_PTR. */
.fb_id = valid_fb->fb_id,
.ptr_valid = false,
.out_fence_ptr = (int32_t *)0x8,
+ .name = "Invalid-WRITEBACK_OUT_FENCE_PTR",
},
};
for (i = 0; i < ARRAY_SIZE(invalid_tests); i++) {
- ret = do_writeback_test(output, invalid_tests[i].fb_id,
- invalid_tests[i].out_fence_ptr,
- invalid_tests[i].ptr_valid);
- igt_assert(ret != 0);
+ igt_dynamic_f("%s", invalid_tests[i].name) {
+ ret = do_writeback_test(output, invalid_tests[i].fb_id,
+ invalid_tests[i].out_fence_ptr,
+ invalid_tests[i].ptr_valid);
+ igt_assert(ret != 0);
+ }
}
}
@@ -247,18 +253,39 @@ static void writeback_fb_id(igt_output_t *output, igt_fb_t *valid_fb, igt_fb_t *
{
int ret;
+ struct {
+ const char *name;
+ uint32_t fb_id;
+ int i, expected_ret;
+ } fb_id_tests[] = {
- /* Invalid object for WRITEBACK_FB_ID */
- ret = do_writeback_test(output, output->id, NULL, false);
- igt_assert(ret == -EINVAL);
+ {
+ .name = "Invalid-object",
+ .fb_id = output->id,
+ .expected_ret = -EINVAL,
+ },
+
+ {
+ .name = "Zero-WRITEBACK_FB_ID",
+ .fb_id = 0,
+ .expected_ret = 0,
+ },
+
+ {
+ .name = "Valid-output-buffer",
+ .fb_id = valid_fb->fb_id,
+ .expected_ret = 0,
+ },
+ };
+
+ for (int i = 0; i < ARRAY_SIZE(fb_id_tests); i++) {
+ igt_dynamic_f("%s", fb_id_tests[i].name) {
+ ret = do_writeback_test(output, fb_id_tests[i].fb_id, NULL, false);
+ igt_assert(ret == fb_id_tests[i].expected_ret);
+ }
+ }
- /* Zero WRITEBACK_FB_ID */
- ret = do_writeback_test(output, 0, NULL, false);
- igt_assert(ret == 0);
- /* Valid output buffer */
- ret = do_writeback_test(output, valid_fb->fb_id, NULL, false);
- igt_assert(ret == 0);
}
static void fill_fb(igt_fb_t *fb, uint32_t pixel)
@@ -553,7 +580,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
igt_describe("Writeback has a couple of parameters linked together"
"(output framebuffer and fence); this test goes through"
"the combination of possible bad options");
- igt_subtest("writeback-invalid-parameters") {
+ igt_subtest_with_dynamic("writeback-invalid-parameters") {
igt_fb_t invalid_output_fb;
igt_skip_on(data.dump_check || data.list_modes);
@@ -570,7 +597,7 @@ igt_main_args("b:c:dl", long_options, help_str, opt_handler, NULL)
}
igt_describe("Validate WRITEBACK_FB_ID with valid and invalid options");
- igt_subtest("writeback-fb-id") {
+ igt_subtest_with_dynamic("writeback-fb-id") {
igt_fb_t output_fb;
igt_skip_on(data.dump_check || data.list_modes);
--
2.36.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-09-22 10:18 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-25 21:06 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
2022-08-25 21:06 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
2022-08-26 18:15 ` Jessica Zhang
2022-08-25 21:06 ` [igt-dev] [PATCH 2/2] tests/kms_writeback: Test Cleanup Nidhi Gupta
2022-08-26 18:49 ` Jessica Zhang
2022-08-25 21:36 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork
2022-09-08 8:48 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test cleanup Nidhi Gupta
2022-09-08 8:48 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
2022-09-08 18:03 ` Jessica Zhang
2022-09-22 0:58 [igt-dev] [PATCH 0/2] tests/kms_writeback: Test cleanup Nidhi Gupta
2022-09-22 0:58 ` [igt-dev] [PATCH 1/2] tests/kms_writeback: Convert tests to dynamic Nidhi Gupta
2022-09-22 10:18 ` Modem, Bhanuprakash
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.