diff options
author | Brad Volkin <bradley.d.volkin@intel.com> | 2014-09-18 19:26:26 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-09-19 11:53:58 -0400 |
commit | 9beb0ccb66ec5d011ac13a02d3b1098035195251 (patch) | |
tree | 638ffe74a5200296a9d43ec5281404bc244a2726 | |
parent | 9d533c5a5219ed7e3d52a05309723964be2307b3 (diff) |
drm/i915: Re-enable the command parser when using PPGTT
In commit
commit 896ab1a5d54269b463a24194c2e4a369103b46d8
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Wed Aug 6 15:04:51 2014 +0200
drm/i915: Fix up checks for aliasing ppgtt
it looks like we accidentally inverted the check that the command
parser should only run when the driver enables some form of PPGTT.
Testcase: igt/gem_exec_parse
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Brad Volkin <bradley.d.volkin@intel.com>
[danvet: Also drop the comment right above, all production vlv now
have hw ppgtt enabled.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/i915/i915_cmd_parser.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/gpu/drm/i915/i915_cmd_parser.c b/drivers/gpu/drm/i915/i915_cmd_parser.c index c45856bcc8b9..d40f264bd5f0 100644 --- a/drivers/gpu/drm/i915/i915_cmd_parser.c +++ b/drivers/gpu/drm/i915/i915_cmd_parser.c | |||
@@ -845,12 +845,7 @@ bool i915_needs_cmd_parser(struct intel_engine_cs *ring) | |||
845 | if (!ring->needs_cmd_parser) | 845 | if (!ring->needs_cmd_parser) |
846 | return false; | 846 | return false; |
847 | 847 | ||
848 | /* | 848 | if (!USES_PPGTT(ring->dev)) |
849 | * XXX: VLV is Gen7 and therefore has cmd_tables, but has PPGTT | ||
850 | * disabled. That will cause all of the parser's PPGTT checks to | ||
851 | * fail. For now, disable parsing when PPGTT is off. | ||
852 | */ | ||
853 | if (USES_PPGTT(ring->dev)) | ||
854 | return false; | 849 | return false; |
855 | 850 | ||
856 | return (i915.enable_cmd_parser == 1); | 851 | return (i915.enable_cmd_parser == 1); |