Re: [PATCH 1/2] drm/i915: mask RPS IRQs properly when disabling RPS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2014-12-01 14:34 GMT-02:00 Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>:
> On Mon, Dec 01, 2014 at 02:29:11PM -0200, Paulo Zanoni wrote:
>> 2014-11-20 19:01 GMT-02:00 Imre Deak <imre.deak@xxxxxxxxx>:
>> > Atm, igt/gem_reset_stats can trigger the recently added WARN on
>> > left-over PM_IIR bits in gen6_enable_rps_interrupts(). There are two
>> > reasons for this:
>> > 1. we call intel_enable_gt_powersave() without a preceeding
>> >    intel_disable_gt_powersave()
>> > 2. gen6_disable_rps_interrupts() doesn't mask interrupts in PM_IMR
>>
>> We don't do this, but we mask stuff through GEN6_PMINTRMSK. Shouldn't
>> this be enough to prevent IIR from changing?
>>
>> Chris?
>
> It should. We should be doing both really, use PM_IMR to treat
> IMR/IIR/IER consistently with other interrupts, and use the special
> PMINTRMASK as part of rps power tuning.

In that case: Reviewed-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>

But one thing makes me wonder: we disable IER on
gen6_disable_rps_interrupts but never seem to enable it again (except
for the usual pre/post/uninstall functions)... I know it is not a
problem introduced by this patch, but shouldn't this be a problem too?

I also wonder if there's a way to greatly simplify all this RPS
interrupt handling...

> -Chris
>
> --
> Chris Wilson, Intel Open Source Technology Centre



-- 
Paulo Zanoni
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux