Re: [PATCH i-g-t] tests/kms_rmfb: Fix cursor fb size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Tested OK, also NOTE that this fixes bug 97975

/Marta

> -----Original Message-----
> From: ville.syrjala@xxxxxxxxxxxxxxx [mailto:ville.syrjala@xxxxxxxxxxxxxxx]
> Sent: Monday, June 19, 2017 3:29 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Lofstedt, Marta <marta.lofstedt@xxxxxxxxx>
> Subject: [PATCH i-g-t] tests/kms_rmfb: Fix cursor fb size
> 
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> 
> Don't use the display mode size to determine the cursor fb size. That will lead
> to a framebuffer with a stride that the cursor plane won't support. Just use
> the max cursor size as the fb size instead.
> 
> This started to trip up after i915 started to actually use the cursor fb stride in
> commit 1e1bb8710e60 ("drm/i915: Use fb->pitches[0] in cursor code")
> 
> Cc: Marta Löfstedt <marta.lofstedt@xxxxxxxxx>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97975
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
>  tests/kms_rmfb.c | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c index
> 5753d74c9411..9f1f564e54aa 100644
> --- a/tests/kms_rmfb.c
> +++ b/tests/kms_rmfb.c
> @@ -66,16 +66,11 @@ test_rmfb(struct rmfb_data *data, igt_output_t
> *output, enum pipe pipe, bool reo
>  	igt_create_fb(data->drm_fd, mode->hdisplay, mode-
> >vdisplay,
>  		      DRM_FORMAT_XRGB8888,
> LOCAL_DRM_FORMAT_MOD_NONE, &fb);
> 
> -	igt_create_fb(data->drm_fd, mode->hdisplay, mode-
> >vdisplay,
> -		      DRM_FORMAT_ARGB8888,
> LOCAL_DRM_FORMAT_MOD_NONE, &argb_fb);
> -
>  	do_or_die(drmGetCap(data->drm_fd,
> DRM_CAP_CURSOR_WIDTH, &cursor_width));
> -	if (cursor_width > mode->hdisplay)
> -		cursor_width = mode->hdisplay;
> -
>  	do_or_die(drmGetCap(data->drm_fd,
> DRM_CAP_CURSOR_HEIGHT, &cursor_height));
> -	if (cursor_height > mode->vdisplay)
> -		cursor_height = mode->vdisplay;
> +
> +	igt_create_fb(data->drm_fd, cursor_width, cursor_height,
> +		      DRM_FORMAT_ARGB8888,
> LOCAL_DRM_FORMAT_MOD_NONE, &argb_fb);
> 
>  	/*
>  	 * Make sure these buffers are suited for display use
> --
> 2.13.0

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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux