Re: [PATCH] drm/i915/selftests: Upgrade printing test/subtest name to pr_info

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 05/03/2019 14:47, Michał Winiarski wrote:
> We're using pr_debug for things that we don't really want to see in the
> CI log, but we may find useful during test development.
> Let's upgrade the test name printer - we do want to see those in CI log.
> 
> Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>   drivers/gpu/drm/i915/selftests/i915_selftest.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/selftests/i915_selftest.c b/drivers/gpu/drm/i915/selftests/i915_selftest.c
> index 10ef0e636a24..b18eaefef798 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_selftest.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_selftest.c
> @@ -133,7 +133,7 @@ static int __run_selftests(const char *name,
>   		if (signal_pending(current))
>   			return -EINTR;
>   
> -		pr_debug(DRIVER_NAME ": Running %s\n", st->name);
> +		pr_info(DRIVER_NAME ": Running %s\n", st->name);
>   		if (data)
>   			err = st->live(data);
>   		else
> @@ -255,7 +255,7 @@ int __i915_subtests(const char *caller,
>   		if (!apply_subtest_filter(caller, st->name))
>   			continue;
>   
> -		pr_debug(DRIVER_NAME ": Running %s/%s\n", caller, st->name);
> +		pr_info(DRIVER_NAME ": Running %s/%s\n", caller, st->name);
>   		GEM_TRACE("Running %s/%s\n", caller, st->name);
>   
>   		err = st->func(data);
> 

We are gathering momentum! :) 
(20181120172839.25013-1-tvrtko.ursulin@linux.intel.com)

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx