From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id EBE736E51B for ; Tue, 6 Jul 2021 16:22:11 +0000 (UTC) From: "Vudum, Lakshminarayana" Date: Tue, 6 Jul 2021 16:21:11 +0000 Message-ID: References: <20210628131435.1070725-1-Anson.Jacob@amd.com> <20210706124524.nr4t3qwoyu5kodfm@outlook.office365.com> <136ed2af-a0f1-ebb6-23a9-927a5985efaf@gmail.com> In-Reply-To: <136ed2af-a0f1-ebb6-23a9-927a5985efaf@gmail.com> Content-Language: en-US MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH i-g-t V2] tests/kms_plane_multiple: Start continuous CRC capture after commit List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: "juhapekka.heikkila@gmail.com" , Rodrigo Siqueira , "Latvala, Petri" , Arkadiusz Hiler Cc: "igt-dev@lists.freedesktop.org" , Anson Jacob , Victor Lu List-ID: I filed a new bug for the failure and re-reported the results. https://gitlab.freedesktop.org/drm/intel/-/issues/3737 Thanks, Lakshmi. -----Original Message----- From: Juha-Pekka Heikkila Sent: Tuesday, July 6, 2021 6:48 AM To: Rodrigo Siqueira ; Latvala, Petri ; Arkadiusz Hiler Cc: igt-dev@lists.freedesktop.org; Anson Jacob ; Victor Lu ; Vudum, Lakshminarayana Subject: Re: [igt-dev] [PATCH i-g-t V2] tests/kms_plane_multiple: Start continuous CRC capture after commit On 6.7.2021 15.45, Rodrigo Siqueira wrote: > On 06/28, Anson Jacob wrote: >> From: Victor Lu >> >> atomic-pipe-a-tiling-none fails on amdgpu(Raven) because it fails to >> read the CRC when there is no CRC generated (zeroed framebuffer). >> Move igt_pipe_crc_start to be called after the first nonzero FB commit. >> >> Signed-off-by: Victor Lu >> Acked-by: Anson Jacob >> Cc: Petri Latvala >> Cc: Rodrigo Siqueira >> Cc: Harry Wentland >> Cc: Mark Yacoub >> --- >> tests/kms_plane_multiple.c | 7 +++++-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c >> index 93c6ea7d269d..4424b864b2d6 100644 >> --- a/tests/kms_plane_multiple.c >> +++ b/tests/kms_plane_multiple.c >> @@ -288,6 +288,7 @@ test_plane_position_with_output(data_t *data, enum pipe pipe, >> igt_plane_t *plane; >> int i; >> int err, c = 0; >> + int crc_enabled = 0; >> int iterations = opt.iterations < 1 ? 1 : opt.iterations; >> bool loop_forever; >> char info[256]; >> @@ -327,14 +328,16 @@ test_plane_position_with_output(data_t *data, enum pipe pipe, >> igt_output_name(output), kmstest_pipe_name(pipe), c, >> info, opt.seed); >> >> - igt_pipe_crc_start(data->pipe_crc); >> - >> i = 0; >> while (i < iterations || loop_forever) { >> /* randomize planes and set up the holes */ >> prepare_planes(data, pipe, &blue, tiling, c, output); >> >> igt_display_commit2(&data->display, COMMIT_ATOMIC); >> + if (!crc_enabled) { >> + igt_pipe_crc_start(data->pipe_crc); >> + crc_enabled = 1; >> + } >> >> igt_pipe_crc_get_current(data->display.drm_fd, data->pipe_crc, >> &crc); >> >> -- >> 2.25.1 >> > > Hi Petri and Hiler, > > I was looking at the failure log for this patch [1], and I did not > find any reference to a failure related to kms_plane_multiple, which > makes me think this is a false positive. In this case, is it worth > resubmitting the patch for CI? > > 1. https://patchwork.freedesktop.org/series/81857/ > Hi Rodrigo, In cases like this where your change was not responsible for ci reported failure, ie. false positive, you can report it to Lakshminarayana Vudum (CC'd). After reporting to Lakshmi there's no need for rerun of the series. /Juha-Pekka _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev