> -----Original Message----- > From: [email protected] [mailto:linux-crypto- > [email protected]] On Behalf Of Li Yang > Sent: Tuesday, July 17, 2012 11:37 AM > To: Liu Qiang-B32616 > Cc: [email protected]; [email protected]; Ira W. > Snyder; Vinod Koul; [email protected]; Dan Williams; > [email protected] > Subject: Re: [PATCH v3 3/4] fsl-dma: change release process of dma > descriptor for supporting async_tx > > On Mon, Jul 16, 2012 at 12:08 PM, Qiang Liu <[email protected]> > wrote: > > Fix the potential risk when enable config NET_DMA and ASYNC_TX. > > Async_tx is lack of support in current release process of dma > > descriptor, all descriptors will be released whatever is acked or > > no-acked by async_tx, so there is a potential race condition when dma > > engine is uesd by others clients (e.g. when enable NET_DMA to offload > TCP). > > > > In our case, 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 affects the process of raid > > operations due to ack_tx is not checked in fsl dma. The no-acked > > descriptor is freed which is submitted just now, as a dependent tx, > > this freed descriptor trigger > > BUG_ON(async_tx_test_ack(depend_tx)) in async_tx_submit(). > > > > Cc: Dan Williams <[email protected]> > > Cc: Vinod Koul <[email protected]> > > Cc: Li Yang <[email protected]> > > Cc: Ira W. Snyder <[email protected]> > > Signed-off-by: Qiang Liu <[email protected]> > > Also separate the function ordering change and real code change into > different patches when you work on the next patch set. Accept, I will spilt it up in v4. Thanks.
> > - Leo > -- > To unsubscribe from this list: send the line "unsubscribe linux-crypto" > in the body of a message to [email protected] More majordomo info > at http://vger.kernel.org/majordomo-info.html
