lgtm

Reviewed-by: Sean V Kelley <[email protected]>

On Mon, Feb 2, 2015 at 12:16 AM, Xiang, Haihao <[email protected]> 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 <[email protected]>
> Tested-by: Zhao Yakui <[email protected]>
> ---
>  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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/libva



-- 
Sean V. Kelley <[email protected]>
Open Source Technology Center / SSG
Intel Corp.
_______________________________________________
Libva mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libva

Reply via email to