[Libva] [libva][libva-intel-driver PATCH] Disable upper bound check for decoding on BDW+

[Sean V Kelley]

lgtm

Reviewed-by: Sean V Kelley <sean.v.kelley at intel.com>

On Mon, Feb 2, 2015 at 12:16 AM, Xiang, Haihao <haihao.xiang at intel.com> wrote:
> We saw pixel corruption in stress test with the wrong up-bound address.
> For simplicity, disable upper bound check in the driver.
>
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> Tested-by: Zhao Yakui <yakui.zhao at intel.com>
> ---
>  src/gen8_mfd.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gen8_mfd.c b/src/gen8_mfd.c
> index 5a802f5..a5cedd6 100644
> --- a/src/gen8_mfd.c
> +++ b/src/gen8_mfd.c
> @@ -282,7 +282,7 @@ gen8_mfd_ind_obj_base_addr_state(VADriverContextP ctx,
>      OUT_BCS_BATCH(batch, 0);
>      OUT_BCS_BATCH(batch, 0);
>         /* Upper bound 4-5 */
> -    OUT_BCS_BATCH(batch, 0x80000000); /* must set, up to 2G */
> +    OUT_BCS_BATCH(batch, 0);
>      OUT_BCS_BATCH(batch, 0);
>
>         /* MFX indirect MV 6-10 */
> @@ -2493,7 +2493,7 @@ gen8_jpeg_wa_ind_obj_base_addr_state(VADriverContextP ctx,
>                    gen7_mfd_context->jpeg_wa_slice_data_bo,
>                    I915_GEM_DOMAIN_INSTRUCTION, 0,
>                    0);
> -    OUT_BCS_BATCH(batch, 0x80000000); /* must set, up to 2G */
> +    OUT_BCS_BATCH(batch, 0);
>      OUT_BCS_BATCH(batch, 0); /* ignore for VLD mode */
>      OUT_BCS_BATCH(batch, 0);
>      OUT_BCS_BATCH(batch, 0); /* ignore for VLD mode */
> --
> 1.9.1
>
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva



-- 
Sean V. Kelley <sean.v.kelley at intel.com>
Open Source Technology Center / SSG
Intel Corp.