[Intel-gfx] [PATCH v2] drm/i915: Add RPM references in the *_get_hw_state functions

Daniel Vetter daniel at ffwll.ch
Tue Jan 5 05:54:31 PST 2016


On Mon, Jan 4, 2016 at 4:05 PM, Ville Syrjälä
<ville.syrjala at linux.intel.com> wrote:
> I think what we really need here is some kind of
> intel_display_power_get_unless_zero() thing. We need to make sure not
> only that the rpm reference is held when reading out the state, but also
> that the relevant power well(s) remain enabled while we're reading out
> the state.

Yeah, we need to check whether a given piece of hw is enabled or not.
All the get_hw_state funcs do that, using
intel_display_power_is_enabled. But it looks like
intel_ddi_get_hw_state partially gets this wrong. That would also
address the same backtrace in module unload.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the Intel-gfx mailing list