Re: [igt-dev] [PATCH i-g-t] runner: Adapt to device-based DRM logging changes

[Jani Nikula <jani.nikula@intel.com>]

On Tue, 02 Jun 2020, Petri Latvala <petri.latvala@intel.com> wrote:
> DRM logging in the kernel has switched to device based logging,
> changing the pattern of logging from
>
> [drm:function_name] Message
>
> to
>
> i915 0000:00:02.0: [drm] Message
>
> Change the pattern we use with --piglit-style-dmesg to match the new
> style logging, catching all warnings that contain [drm]. We're no
> longer result-wise exactly piglit compatible but then again that ship
> sailed long time ago with the change of introducing dmesg-level
> parameter. Piglit compatibility result-wise is not a goal regardless.
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> Cc: Jani Nikula <jani.nikula@intel.com>

Reviewed-by: Jani Nikula <jani.nikula@intel.com>


> ---
>
> For the record: This won't be merged without making sure cibuglog
> filtering is up to date.
>
>  runner/resultgen.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/runner/resultgen.c b/runner/resultgen.c
> index 17147a8f..add4aad5 100644
> --- a/runner/resultgen.c
> +++ b/runner/resultgen.c
> @@ -808,7 +808,7 @@ static const char igt_dmesg_whitelist[] =
>  #undef _
>  
>  static const char igt_piglit_style_dmesg_blacklist[] =
> -	"(\\[drm:|drm_|intel_|i915_)";
> +	"(\\[drm:|drm_|intel_|i915_|\\[drm\\])";
>  
>  static bool init_regex_whitelist(struct settings* settings, GRegex **re)
>  {

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev