On 19/12/2018 16:19, Chris Wilson wrote:
Quoting Lionel Landwerlin (2018-12-19 14:37:46)
static void free_oa_config(struct drm_i915_private *dev_priv,
@@ -1852,6 +1854,13 @@ static void gen7_oa_enable(struct i915_perf_stream *stream)
*/
gen7_init_oa_buffer(dev_priv);
+ if (stream->oa_interrupt_monitor) {
+ spin_lock(&dev_priv->irq_lock);
Doesn't look like you are already in interrupt context, so spin_lock_irq()
required for dev_priv->irq_lock.
-Chris
Thanks a lot Chris, will fix.
Any comment on the atomic counter? Does that look like a bad design/idea?
Cheers,
-
Lionel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
