On Mon, Jul 9, 2012 at 10:59 PM, Qiang Liu <qiang.liu@xxxxxxxxxxxxx> wrote: > - delete attribute of DMA_INTERRUPT because fsl-dma doesn't support > this function, exception will be thrown if talitos is used to compute xor > at the same time; > - change the release process of dma descriptor for avoiding exception when > enable config NET_DMA, release dma descriptor from 1st to last second, the > last descriptor which is reserved in current descriptor register may not be > completed, race condition will be raised if free current descriptor; > - use spin_lock_bh to instead of spin_lock_irqsave for improving performance; > > A race condition which is raised when use both of talitos and dmaengine to > offload xor is because napi scheduler will sync all pending requests in dma > channels, it will affect the process of raid operations. The descriptor is > freed which is submitted just now, but async_tx must check whether this depend > tx descriptor is acked, there are poison contents in the invalid address, > then BUG_ON() is thrown, so this descriptor will be freed in the next time. > > Cc: Dan Williams <dan.j.williams@xxxxxxxxx> > Cc: Vinod Koul <vinod.koul@xxxxxxxxx> > Cc: Li Yang <leoli@xxxxxxxxxxxxx> > Signed-off-by: Qiang Liu <qiang.liu@xxxxxxxxxxxxx> > --- >From the description this sounds like 3 or 4 patches. Can you split it up? -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
