Commit dd72bde0 authored by Mika Kuoppala's avatar Mika Kuoppala Committed by Daniel Vetter

drm/i915: Do kunmap if renderstate parsing fails

Kunmap the renderstate page on error path.
Reviewed-by: default avatarArun Siluvery <arun.siluvery@linux.intel.com>
Signed-off-by: default avatarMika Kuoppala <mika.kuoppala@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 2617268f
...@@ -96,8 +96,10 @@ static int render_state_setup(struct render_state *so) ...@@ -96,8 +96,10 @@ static int render_state_setup(struct render_state *so)
s = lower_32_bits(r); s = lower_32_bits(r);
if (so->gen >= 8) { if (so->gen >= 8) {
if (i + 1 >= rodata->batch_items || if (i + 1 >= rodata->batch_items ||
rodata->batch[i + 1] != 0) rodata->batch[i + 1] != 0) {
return -EINVAL; ret = -EINVAL;
goto err_out;
}
d[i++] = s; d[i++] = s;
s = upper_32_bits(r); s = upper_32_bits(r);
...@@ -120,6 +122,10 @@ static int render_state_setup(struct render_state *so) ...@@ -120,6 +122,10 @@ static int render_state_setup(struct render_state *so)
} }
return 0; return 0;
err_out:
kunmap(page);
return ret;
} }
void i915_gem_render_state_fini(struct render_state *so) void i915_gem_render_state_fini(struct render_state *so)
......
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