Re: [PATCH] drm/i915: Refine mmio debug flow to avoid bad unlock balance detected.

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

 



Hi Jani,

> >> diff --git a/drivers/gpu/drm/i915/intel_uncore.c b/drivers/gpu/drm/i915/intel_uncore.c
> >> index 796ebfe6c550..9d665978cc43 100644
> >> --- a/drivers/gpu/drm/i915/intel_uncore.c
> >> +++ b/drivers/gpu/drm/i915/intel_uncore.c
> >> @@ -1931,9 +1931,6 @@ unclaimed_reg_debug(struct intel_uncore *uncore,
> >>  		    const bool read,
> >>  		    const bool before)
> >>  {
> >> -	if (likely(!uncore->i915->params.mmio_debug) || !uncore->debug)
> >> -		return;
> >> -
> >
> > this is a very good catch! I'm fine with the change from my side:
> >
> > Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>
> 
> Nope, there's a bug in the patch.
> 
> https://lore.kernel.org/r/87ilb00zot.fsf@xxxxxxxxx

OK, I missed your reply.

Thanks, Jani!

Andi



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux