Commit 15d21db8 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Refactor golden render state emission to unconfuse gcc

GCC was inlining the init and setup functions, but was getting itself
confused into thinking that variables could be used uninitialised. If we
do the inline for gcc, it is happy! As a bonus we shrink the code.

v2: A couple of minor tweaks from Joonas
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1469432687-22756-29-git-send-email-chris@chris-wilson.co.ukReviewed-by: default avatarJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1470174640-18242-20-git-send-email-chris@chris-wilson.co.uk
parent e40f9ee6
...@@ -32,15 +32,14 @@ struct render_state { ...@@ -32,15 +32,14 @@ struct render_state {
const struct intel_renderstate_rodata *rodata; const struct intel_renderstate_rodata *rodata;
struct drm_i915_gem_object *obj; struct drm_i915_gem_object *obj;
u64 ggtt_offset; u64 ggtt_offset;
int gen;
u32 aux_batch_size; u32 aux_batch_size;
u32 aux_batch_offset; u32 aux_batch_offset;
}; };
static const struct intel_renderstate_rodata * static const struct intel_renderstate_rodata *
render_state_get_rodata(const int gen) render_state_get_rodata(const struct drm_i915_gem_request *req)
{ {
switch (gen) { switch (INTEL_GEN(req->i915)) {
case 6: case 6:
return &gen6_null_state; return &gen6_null_state;
case 7: case 7:
...@@ -54,36 +53,6 @@ render_state_get_rodata(const int gen) ...@@ -54,36 +53,6 @@ render_state_get_rodata(const int gen)
return NULL; return NULL;
} }
static int render_state_init(struct render_state *so,
struct drm_i915_private *dev_priv)
{
int ret;
so->gen = INTEL_GEN(dev_priv);
so->ggtt_offset = 0; /* keep gcc quiet */
so->rodata = render_state_get_rodata(so->gen);
if (so->rodata == NULL)
return 0;
if (so->rodata->batch_items * 4 > 4096)
return -EINVAL;
so->obj = i915_gem_object_create(&dev_priv->drm, 4096);
if (IS_ERR(so->obj))
return PTR_ERR(so->obj);
ret = i915_gem_obj_ggtt_pin(so->obj, 4096, 0);
if (ret)
goto free_gem;
so->ggtt_offset = i915_gem_obj_ggtt_offset(so->obj);
return 0;
free_gem:
i915_gem_object_put(so->obj);
return ret;
}
/* /*
* Macro to add commands to auxiliary batch. * Macro to add commands to auxiliary batch.
* This macro only checks for page overflow before inserting the commands, * This macro only checks for page overflow before inserting the commands,
...@@ -106,6 +75,7 @@ static int render_state_setup(struct render_state *so) ...@@ -106,6 +75,7 @@ static int render_state_setup(struct render_state *so)
{ {
struct drm_device *dev = so->obj->base.dev; struct drm_device *dev = so->obj->base.dev;
const struct intel_renderstate_rodata *rodata = so->rodata; const struct intel_renderstate_rodata *rodata = so->rodata;
const bool has_64bit_reloc = INTEL_GEN(dev) >= 8;
unsigned int i = 0, reloc_index = 0; unsigned int i = 0, reloc_index = 0;
struct page *page; struct page *page;
u32 *d; u32 *d;
...@@ -124,7 +94,7 @@ static int render_state_setup(struct render_state *so) ...@@ -124,7 +94,7 @@ static int render_state_setup(struct render_state *so)
if (i * 4 == rodata->reloc[reloc_index]) { if (i * 4 == rodata->reloc[reloc_index]) {
u64 r = s + so->ggtt_offset; u64 r = s + so->ggtt_offset;
s = lower_32_bits(r); s = lower_32_bits(r);
if (so->gen >= 8) { if (has_64bit_reloc) {
if (i + 1 >= rodata->batch_items || if (i + 1 >= rodata->batch_items ||
rodata->batch[i + 1] != 0) { rodata->batch[i + 1] != 0) {
ret = -EINVAL; ret = -EINVAL;
...@@ -202,53 +172,40 @@ static int render_state_setup(struct render_state *so) ...@@ -202,53 +172,40 @@ static int render_state_setup(struct render_state *so)
#undef OUT_BATCH #undef OUT_BATCH
static void render_state_fini(struct render_state *so) int i915_gem_render_state_init(struct drm_i915_gem_request *req)
{
i915_gem_object_ggtt_unpin(so->obj);
i915_gem_object_put(so->obj);
}
static int render_state_prepare(struct intel_engine_cs *engine,
struct render_state *so)
{ {
struct render_state so;
int ret; int ret;
if (WARN_ON(engine->id != RCS)) if (WARN_ON(req->engine->id != RCS))
return -ENOENT; return -ENOENT;
ret = render_state_init(so, engine->i915); so.rodata = render_state_get_rodata(req);
if (ret) if (!so.rodata)
return ret;
if (so->rodata == NULL)
return 0; return 0;
ret = render_state_setup(so); if (so.rodata->batch_items * 4 > 4096)
if (ret) { return -EINVAL;
render_state_fini(so);
return ret;
}
return 0;
}
int i915_gem_render_state_init(struct drm_i915_gem_request *req) so.obj = i915_gem_object_create(&req->i915->drm, 4096);
{ if (IS_ERR(so.obj))
struct render_state so; return PTR_ERR(so.obj);
int ret;
ret = render_state_prepare(req->engine, &so); ret = i915_gem_obj_ggtt_pin(so.obj, 4096, 0);
if (ret) if (ret)
return ret; goto err_obj;
if (so.rodata == NULL) so.ggtt_offset = i915_gem_obj_ggtt_offset(so.obj);
return 0;
ret = render_state_setup(&so);
if (ret)
goto err_unpin;
ret = req->engine->emit_bb_start(req, so.ggtt_offset, ret = req->engine->emit_bb_start(req, so.ggtt_offset,
so.rodata->batch_items * 4, so.rodata->batch_items * 4,
I915_DISPATCH_SECURE); I915_DISPATCH_SECURE);
if (ret) if (ret)
goto out; goto err_unpin;
if (so.aux_batch_size > 8) { if (so.aux_batch_size > 8) {
ret = req->engine->emit_bb_start(req, ret = req->engine->emit_bb_start(req,
...@@ -257,11 +214,13 @@ int i915_gem_render_state_init(struct drm_i915_gem_request *req) ...@@ -257,11 +214,13 @@ int i915_gem_render_state_init(struct drm_i915_gem_request *req)
so.aux_batch_size, so.aux_batch_size,
I915_DISPATCH_SECURE); I915_DISPATCH_SECURE);
if (ret) if (ret)
goto out; goto err_unpin;
} }
i915_vma_move_to_active(i915_gem_obj_to_ggtt(so.obj), req); i915_vma_move_to_active(i915_gem_obj_to_ggtt(so.obj), req);
out: err_unpin:
render_state_fini(&so); i915_gem_object_ggtt_unpin(so.obj);
err_obj:
i915_gem_object_put(so.obj);
return ret; return ret;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment