[Intel-gfx] [PATCH 2/2] drm/mm: Convert to drm_printer

Chris Wilson chris at chris-wilson.co.uk
Wed Dec 28 17:01:17 UTC 2016


On Wed, Dec 28, 2016 at 05:42:10PM +0100, Daniel Vetter wrote:
>  static int ttm_pl_vram = TTM_PL_VRAM;
> diff --git a/drivers/gpu/drm/selftests/test-drm_mm.c b/drivers/gpu/drm/selftests/test-drm_mm.c
> index 2ce92f4dcfc7..598fde492863 100644
> --- a/drivers/gpu/drm/selftests/test-drm_mm.c
> +++ b/drivers/gpu/drm/selftests/test-drm_mm.c
> @@ -249,8 +249,10 @@ static int igt_init(void *ignored)
>  	}
>  
>  out:
> -	if (ret)
> -		drm_mm_debug_table(&mm, __func__);
> +	if (ret) {
> +		struct drm_printer p = drm_debug_printer(__func__);
> +		drm_mm_print_table(&p, &mm);
> +	}

drm_mm_print_table(&drm_debug_printer(__func__), &mm);

But that says the parameters are reversed. Either this should be

	drm_print_mm(&printer, &mm);

or
	drm_mm_print(&mm, &printer);

Then here we would want a 

#define show_mm(mm) drm_mm_print((mm), &drm_debug_printer(__func__))

for similarity with show_scan(), show_holes(), with the exception of
igt_debug where we still ultimately want to redirect the output to a
stringbuf.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list