[Piglit] [PATCH] egl_ext_device_drm: don't fail the test if open("/dev/dri/cardX") errors

Emil Velikov emil.l.velikov at gmail.com
Tue Oct 9 12:35:39 UTC 2018

On Fri, 5 Oct 2018 at 19:13, Dylan Baker <dylan at pnwbakers.com> wrote:
> In other cases like this we return skip instead of warn.
Can you be more specific about those "other cases like these"? It's
the core part of your argument so w/o a quote/reference it's
impossible to make a decision.

> I think that it should
> be skip because:
> 1) warn doesn't have any real meaning
Not a usual black-and-white one, but it does.
WARN is something that should be addressed (looked at), but isn't
considered an blocking issue.

> 2) we couldn't actually run the test so there isn't a result
There is no clear "one test" here, but a collection of small subtests
- more or less one for each piglit_report_result.
You don't want to skip the whole thing, because a section is no applicable.


More information about the Piglit mailing list