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

[Sujaritha <sujaritha.sundaresan@intel.com>]

On 3/18/19 2:14 PM, Chris Wilson wrote:
> 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@intel.com>
>> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
>> Cc: Petri Latvala <petri.latvala@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

It makes sense to have the whitelist be specific to tests.

-Sujaritha

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