Quoting Jakub Bartmiński (2018-07-18 12:54:10)
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c 
> b/drivers/gpu/drm/i915/intel_lrc.c
> index 35d37af0cb9a..7a3c8921f688 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -1363,8 +1363,8 @@ static int __context_pin(struct i915_gem_context *ctx, 
> struct i915_vma *vma)
>         }
>  
>         flags = PIN_GLOBAL | PIN_HIGH;
> -       if (ctx->ggtt_offset_bias)
> -               flags |= PIN_OFFSET_BIAS | ctx->ggtt_offset_bias;
> +       if (ctx->i915->ggtt.pin_bias)
> +               flags |= PIN_OFFSET_BIAS | ctx->i915->ggtt.pin_bias;

static inline u32 i915_ggtt_pin_bias(struct i915_vma *vma)
{
        return i915_vm_to_ggtt(vma->vm)->pin_bias;
}

To include | PIN_OFFSET_BIAS or not? Probably not. Retcon that earlier
in the series.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to