[Libva] [intel-driver][PATCH] VPP: i965_vpp_clear_surface() is still used for CSC on BDW

Zhao, Yakui yakui.zhao at intel.com
Mon May 26 01:06:50 PDT 2014


On Sun, 2014-05-25 at 20:36 -0600, Xiang, Haihao wrote:
> From: "Xiang, Haihao" <haihao.xiang at intel.com>
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=79065
> 
> The regression is caused by commit 42258e1
> 
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>

Add: Reviewed-by: Zhao, Yakui <yakui.zhao at intel.com>

Thanks.
   Yakui
> ---
>  src/i965_post_processing.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/i965_post_processing.c b/src/i965_post_processing.c
> index 230694f..d21f73d 100755
> --- a/src/i965_post_processing.c
> +++ b/src/i965_post_processing.c
> @@ -4803,13 +4803,13 @@ i965_vpp_clear_surface(VADriverContextP ctx,
>      br13 |= BR13_8;
>      br13 |= pitch;
>  
> -    if (IS_GEN6(i965->intel.device_info) ||
> -        IS_GEN7(i965->intel.device_info)) {
> -        intel_batchbuffer_start_atomic_blt(batch, 48);
> -        BEGIN_BLT_BATCH(batch, 12);
> -    } else {
> +    if (IS_IRONLAKE(i965->intel.device_info)) {
>          intel_batchbuffer_start_atomic(batch, 48);
>          BEGIN_BATCH(batch, 12);
> +    } else {
> +        /* Will double-check the command if the new chipset is added */
> +        intel_batchbuffer_start_atomic_blt(batch, 48);
> +        BEGIN_BLT_BATCH(batch, 12);
>      }
>  
>      region_width = obj_surface->width;




More information about the Libva mailing list