[PATCH] drm/i915/gvt: Remove logging when checking surface base address

Zhang, Xiaolin xiaolin.zhang at intel.com
Wed Apr 24 01:51:37 UTC 2019


On 04/24/2019 09:39 AM, Zhang, Tina wrote:
> The vgpu's plane surface base address register could contain invalidated
> values. Only validated values can be used by GVT-g display model.
> Invalidated values should be ignored w/o causing any noise.
>
> Cc: Xiaolin Zhang <xiaolin.zhang at intel.com>
> Signed-off-by: Tina Zhang <tina.zhang at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/fb_decoder.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gvt/fb_decoder.c b/drivers/gpu/drm/i915/gvt/fb_decoder.c
> index 65e8473..3564bc4 100644
> --- a/drivers/gpu/drm/i915/gvt/fb_decoder.c
> +++ b/drivers/gpu/drm/i915/gvt/fb_decoder.c
> @@ -245,7 +245,7 @@ int intel_vgpu_decode_primary_plane(struct intel_vgpu *vgpu,
>  	plane->hw_format = fmt;
>  
>  	plane->base = vgpu_vreg_t(vgpu, DSPSURF(pipe)) & I915_GTT_PAGE_MASK;
> -	if (!intel_gvt_ggtt_validate_range(vgpu, plane->base, 0))
> +	if (!plane->base || !vgpu_gmadr_is_valid(vgpu, plane->base))
>  		return  -EINVAL;
I think we still fire error message in second check for non zero base
case to indicate there is something wrong.
>
>  	plane->base_gpa = intel_vgpu_gma_to_gpa(vgpu->gtt.ggtt_mm, plane->base);
> @@ -368,7 +368,7 @@ int intel_vgpu_decode_cursor_plane(struct intel_vgpu *vgpu,
>  			alpha_plane, alpha_force);
>  
>  	plane->base = vgpu_vreg_t(vgpu, CURBASE(pipe)) & I915_GTT_PAGE_MASK;
> -	if (!intel_gvt_ggtt_validate_range(vgpu, plane->base, 0))
> +	if (!plane->base || !vgpu_gmadr_is_valid(vgpu, plane->base))
>  		return  -EINVAL;
>  
>  	plane->base_gpa = intel_vgpu_gma_to_gpa(vgpu->gtt.ggtt_mm, plane->base);
> @@ -472,7 +472,7 @@ int intel_vgpu_decode_sprite_plane(struct intel_vgpu *vgpu,
>  	plane->drm_format = drm_format;
>  
>  	plane->base = vgpu_vreg_t(vgpu, SPRSURF(pipe)) & I915_GTT_PAGE_MASK;
> -	if (!intel_gvt_ggtt_validate_range(vgpu, plane->base, 0))
> +	if (!plane->base || !vgpu_gmadr_is_valid(vgpu, plane->base))
>  		return  -EINVAL;
>  
>  	plane->base_gpa = intel_vgpu_gma_to_gpa(vgpu->gtt.ggtt_mm, plane->base);




More information about the intel-gvt-dev mailing list