summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Widawsky <benjamin.widawsky@intel.com>2014-04-08 09:20:05 -0700
committerBen Widawsky <benjamin.widawsky@intel.com>2014-06-20 16:51:42 -0700
commit9b3be5ea067a20addb862845f4e84cad9f0b256b (patch)
tree3995ecb01042c17fcc369fb5a9784466c4872c99
parent90435699ceb382f283f92fc0623e3be4ab93751d (diff)
drm/i915: Extract PPGTT param from pagedir alloc
Now that we don't need to trace num_pd_pages, we may as well kill all need for the PPGTT structure in the alloc_pagedirs. This is very useful for when we move to 48b addressing, and the PDP isn't the root of the page table structure. The param is replaced with drm_device, which is an unavoidable wart throughout the series. (in other words, not extra flagrant). Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
-rw-r--r--drivers/gpu/drm/i915/i915_gem_gtt.c15
1 files changed, 7 insertions, 8 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 690d9fab4ef5..65d89d8bb6c6 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -608,10 +608,9 @@ unwind_out:
static int gen8_ppgtt_alloc_pagedirs(struct i915_pagedirpo *pdp,
uint64_t start,
- uint64_t length)
+ uint64_t length,
+ struct drm_device *dev)
{
- struct i915_hw_ppgtt *ppgtt =
- container_of(pdp, struct i915_hw_ppgtt, pdp);
struct i915_pagedir *unused;
uint64_t temp;
uint32_t pdpe;
@@ -621,8 +620,8 @@ static int gen8_ppgtt_alloc_pagedirs(struct i915_pagedirpo *pdp,
gen8_for_each_pdpe(unused, pdp, start, length, temp, pdpe) {
BUG_ON(unused);
- pdp->pagedirs[pdpe] = alloc_pd_single(ppgtt->base.dev);
- if (IS_ERR(ppgtt->pdp.pagedirs[pdpe]))
+ pdp->pagedirs[pdpe] = alloc_pd_single(dev);
+ if (IS_ERR(pdp->pagedirs[pdpe]))
goto unwind_out;
}
@@ -630,8 +629,7 @@ static int gen8_ppgtt_alloc_pagedirs(struct i915_pagedirpo *pdp,
unwind_out:
while (pdpe--)
- free_pd_single(ppgtt->pdp.pagedirs[pdpe],
- ppgtt->base.dev);
+ free_pd_single(pdp->pagedirs[pdpe], dev);
return -ENOMEM;
}
@@ -645,7 +643,8 @@ static int gen8_ppgtt_alloc(struct i915_hw_ppgtt *ppgtt,
uint32_t pdpe;
int ret;
- ret = gen8_ppgtt_alloc_pagedirs(&ppgtt->pdp, start, length);
+ ret = gen8_ppgtt_alloc_pagedirs(&ppgtt->pdp, start, length,
+ ppgtt->base.dev);
if (ret)
return ret;