Re: [PATCH 00/32] dmaengine: cleanups...

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

 



On Sat, Jul 16, 2016 at 07:38:38PM +0200, Robert Jarzmik wrote:
> Vinod Koul <vinod.koul@xxxxxxxxx> writes:
> 
> > On Tue, Jul 05, 2016 at 08:24:06PM +0530, Vinod Koul wrote:
> >> This series attempts to do few cleanup for drivers. My aim was to fix irq
> >> cleanup but on the way found few more things to be fixed as well.
> >>  o explicitly freeup irq: Many drivers nowadays use devm variant for
> >>    requesting irq, which is fine from cleanup POV, but on remove you leave irq
> >>    enabled and able to fire. This can lead to nasty races, so better idea is
> >>    to explicitly freeup.
> >>  o same situation is with tasklets. They need to be killed before we exit,
> >>    so that has been fixed too
> >>  o while at it noticed few drivers still set .owner, removed them as well.
> >>  o also fixed few sparse complaints
> >
> > Merging this now with the ACKs and changes as discussed
> 
> Hi Vinod,
> 
> I thought that we agreed that commit "dmaengine: pxa_dma: implement
> device_synchronize" superseeded "dmaengine: pxa_dm: explicitly freeup irq" which
> wasn't necessary anymore.

sync does help but it won't help you when you channel is free and hw decides
to send you an interrupt for no reason... while you are removing.

The irq will trigger, probably your tasklet as well and while you are
figuring this out some resources are removed. So I do prefer disabling or
freeing irqs explicitly before we do anything else...

-- 
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe dmaengine" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux