> -----Original Message----- > From: Sebastian von Ohr <vonohr@xxxxxxxxxxx> > Sent: Tuesday, April 7, 2020 1:22 PM > To: Radhey Shyam Pandey <radheys@xxxxxxxxxx>; Vinod Koul > <vkoul@xxxxxxxxxx>; Appana Durga Kedareswara Rao > <appanad@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx> > Cc: dmaengine@xxxxxxxxxxxxxxx > Subject: RE: [PATCH] dmaengine: xilinx_dma: Add missing check for empty > list > > > -----Original Message----- > > From: Radhey Shyam Pandey [mailto:radheys@xxxxxxxxxx] > > Sent: Friday, March 6, 2020 2:57 PM > > To: Vinod Koul <vkoul@xxxxxxxxxx>; Sebastian von Ohr > > <vonohr@xxxxxxxxxxx>; Appana Durga Kedareswara Rao > > <appanad@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx> > > Cc: dmaengine@xxxxxxxxxxxxxxx > > Subject: RE: [PATCH] dmaengine: xilinx_dma: Add missing check for > > empty list > > > > Sure, we will test it. Changes look fine. Though had a question in > > mind, for a generic fix to this problem, should we make locking > > mandatory for all cookie helper functions? Or is there any limitation? > > Any progress on the testing? If you need help reproducing the issue please > let me know. Thanks for reminding me. Somehow I missed it. You mentioned in one of earlier thread that this bug is introduced it using dma_sync_wait to wait for DMA completion. So to reproduce the issue in xilinx axidma test client I have to replace issue_pending with sync_wait API?
