[igt-dev] ✗ Fi.CI.IGT: failure for minor improvements to the kms_cursor_crc doc and some comments cleanup (rev2)

[Melissa Wen]

On 07/16, Arkadiusz Hiler wrote:
> On Wed, Jul 15, 2020 at 10:04:06PM +0000, Patchwork wrote:
> > == Series Details ==
> > 
> > Series: minor improvements to the kms_cursor_crc doc and some comments cleanup (rev2)
> > URL   : https://patchwork.freedesktop.org/series/78814/
> > State : failure
> > 
> > == Summary ==
> > 
> > CI Bug Log - changes from CI_DRM_8751_full -> IGTPW_4766_full
> > ====================================================
> > 
> > Summary
> > -------
> > 
> >   **FAILURE**
> > 
> >   Serious unknown changes coming with IGTPW_4766_full absolutely need to be
> >   verified manually.
> >   
> >   If you think the reported changes have nothing to do with the changes
> >   introduced in IGTPW_4766_full, please notify your bug team 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_4766/index.html
> > 
> > Possible new issues
> > -------------------
> > 
> >   Here are the unknown changes that may have been introduced in IGTPW_4766_full:
> > 
> > ### IGT changes ###
> > 
> > #### Possible regressions ####
> > 
> >   * igt at kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
> >     - shard-tglb:         [PASS][1] -> [INCOMPLETE][2]
> >    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8751/shard-tglb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
> >    [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4766/shard-tglb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
> 
> Hey Lakshmi,
> 
> this is a false positive. No need to re-report though.
> 
> 
> @Melissa, your patches are
> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>
> and merged.
> 
> I've also done some formatting changes:
>  1. indented multiline descriptions so the opening quotes align
>  2. got rid of "\", as they are not needed for multilne strings and we
>     don't use it elsewhere
> 
> This is to mimic the style used in other tests, e.g.:
> https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/blob/master/tests/kms_hdmi_inject.c#L209

Got it! I'll pay more attention to that.

Thanks :)
> 
> Thanks for your contributions!
> 
> -- 
> Cheers,
> Arek