[Intel-gfx] [PATCH 12/17] drm/i915: Arm cmd parser with aliasng ppgtt only
Mika Kuoppala
mika.kuoppala at linux.intel.com
Fri Apr 24 05:57:36 PDT 2015
Daniel Vetter <daniel.vetter at ffwll.ch> writes:
> With the binding regression from the original full ppgtt patches
> fixed we can throw the switch. Yay!
>
> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> ---
> drivers/gpu/drm/i915/i915_gem_execbuffer.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> index f005f3151147..819f2b2317ff 100644
> --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> @@ -1557,12 +1557,8 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
> * dispatch_execbuffer implementations. We specifically
> * don't want that set when the command parser is
> * enabled.
> - *
> - * FIXME: with aliasing ppgtt, buffers that should only
> - * be in ggtt still end up in the aliasing ppgtt. remove
> - * this check when that is fixed.
> */
> - if (USES_FULL_PPGTT(dev))
> + if (USES_PPGTT(dev))
> dispatch_flags |= I915_DISPATCH_SECURE;
>
You can remove the check for USES_PPGTT here, like in patch
that Chris posted. The i915_needs_cmd_parser() checks that
for us.
Reviewed-by: Mika Kuoppala <mika.kuoppala at intel.com>
> exec_start = 0;
> --
> 2.1.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
More information about the Intel-gfx
mailing list