Re: [Intel-gfx] [PATCH 01/14] drm/i915: Cleanup gt powerstate from gem

2018-08-13 Thread Mika Kuoppala
Chris Wilson writes: > Since the gt powerstate is allocated by i915_gem_init, clean it from > i915_gem_fini for symmetry and to correct the imbalance on error. > Was first confused about the allocation as it is embedded. But vlv really does allocate an underlying power context object. And the

[Intel-gfx] [PATCH 01/14] drm/i915: Cleanup gt powerstate from gem

2018-08-12 Thread Chris Wilson
Since the gt powerstate is allocated by i915_gem_init, clean it from i915_gem_fini for symmetry and to correct the imbalance on error. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem.c | 3 +++ drivers/gpu/drm/i915/intel_display.c | 4 2 files changed, 3 insertions(+), 4