[igt-dev] [PATCH i-g-t] runner: Including E2BIG error to whitelist

Chris Wilson chris at chris-wilson.co.uk
Mon Mar 18 21:14:54 UTC 2019


Quoting Sujaritha Sundaresan (2019-03-18 20:47:01)
> Adding E2BIG(7) to the whitelist to prevent it from flagging
> up in CI and polluting the dmesg with warnings.
> 
> Signed-off-by: Sujaritha Sundaresan <sujaritha.sundaresan at intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler at intel.com>
> Cc: Petri Latvala <petri.latvala at intel.com>
> ---
>  runner/resultgen.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/runner/resultgen.c b/runner/resultgen.c
> index 32b59d59..9bf1c406 100644
> --- a/runner/resultgen.c
> +++ b/runner/resultgen.c
> @@ -490,6 +490,10 @@ static const char igt_dmesg_whitelist[] =
>          * message.
>          */
>         "i915: probe of [0-9:.]+ failed with error -25" _
> +       /* Fault injection subtest terminates with E2BIG after the
> +        * test ends, which produces this message.
> +        */
> +       "i915: probe of [0-9:.]+ failed with error -7" _

For fault injection it is legal for it to return any error code, but
only fault injection. This needs to be a per-test whitelist, preferably
supplied by the test itself. Could we have a control link back to the
runner if we don't want igt to run itself?
-Chris


More information about the igt-dev mailing list