* [PATCH] dmaengine: pxa_dma: fix initial list move
@ 2015-09-21 9:06 Robert Jarzmik
2015-09-24 17:28 ` Vinod Koul
0 siblings, 1 reply; 4+ messages in thread
From: Robert Jarzmik @ 2015-09-21 9:06 UTC (permalink / raw)
To: Daniel Mack, Haojian Zhuang, Robert Jarzmik, Vinod Koul
Cc: linux-arm-kernel, dmaengine, linux-kernel, Arnd Bergmann,
Olof Johansson, Kevin Hilman
Since the commit to have an allocated list of virtual descriptors was
reverted, the pxa_dma driver is broken, as it assumes the descriptor is
placed on the allocated list upon allocation.
Fix the issue in pxa_dma by making an allocated virtual descriptor a
singleton.
Fixes: 8c8fe97b2b8a ("Revert "dmaengine: virt-dma: don't always free descriptor upon completion"")
Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
---
Vinod, this is my fallback plan as I'm getting no answer on
https://lkml.org/lkml/2015/9/6/112. If I get no answer before v4.3-rc3,
I'll put that into the pxa tree.
---
drivers/dma/pxa_dma.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/dma/pxa_dma.c b/drivers/dma/pxa_dma.c
index 9d3458e2a366..53c05ca514f1 100644
--- a/drivers/dma/pxa_dma.c
+++ b/drivers/dma/pxa_dma.c
@@ -891,6 +891,7 @@ pxad_tx_prep(struct virt_dma_chan *vc, struct virt_dma_desc *vd,
struct dma_async_tx_descriptor *tx;
struct pxad_chan *chan = container_of(vc, struct pxad_chan, vc);
+ INIT_LIST_HEAD(&vd->node);
tx = vchan_tx_prep(vc, vd, tx_flags);
tx->tx_submit = pxad_tx_submit;
dev_dbg(&chan->vc.chan.dev->device,
--
2.1.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dmaengine: pxa_dma: fix initial list move
2015-09-21 9:06 [PATCH] dmaengine: pxa_dma: fix initial list move Robert Jarzmik
@ 2015-09-24 17:28 ` Vinod Koul
2015-09-24 19:39 ` Robert Jarzmik
0 siblings, 1 reply; 4+ messages in thread
From: Vinod Koul @ 2015-09-24 17:28 UTC (permalink / raw)
To: Robert Jarzmik
Cc: Daniel Mack, Haojian Zhuang, linux-arm-kernel, dmaengine,
linux-kernel, Arnd Bergmann, Olof Johansson, Kevin Hilman
On Mon, Sep 21, 2015 at 11:06:32AM +0200, Robert Jarzmik wrote:
> Since the commit to have an allocated list of virtual descriptors was
> reverted, the pxa_dma driver is broken, as it assumes the descriptor is
> placed on the allocated list upon allocation.
>
> Fix the issue in pxa_dma by making an allocated virtual descriptor a
> singleton.
This seems okay, simpler and safer. Do you want me to apply this instead
while detailed approach can be reviewed and discussed
--
~Vinod
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dmaengine: pxa_dma: fix initial list move
2015-09-24 17:28 ` Vinod Koul
@ 2015-09-24 19:39 ` Robert Jarzmik
2015-09-25 1:51 ` Vinod Koul
0 siblings, 1 reply; 4+ messages in thread
From: Robert Jarzmik @ 2015-09-24 19:39 UTC (permalink / raw)
To: Vinod Koul
Cc: Daniel Mack, Haojian Zhuang, linux-arm-kernel, dmaengine,
linux-kernel, Arnd Bergmann, Olof Johansson, Kevin Hilman
Vinod Koul <vinod.koul@intel.com> writes:
> On Mon, Sep 21, 2015 at 11:06:32AM +0200, Robert Jarzmik wrote:
>> Since the commit to have an allocated list of virtual descriptors was
>> reverted, the pxa_dma driver is broken, as it assumes the descriptor is
>> placed on the allocated list upon allocation.
>>
>> Fix the issue in pxa_dma by making an allocated virtual descriptor a
>> singleton.
>
> This seems okay, simpler and safer. Do you want me to apply this instead
> while detailed approach can be reviewed and discussed
Yes, please. Having the guarantee this gets into the next or next after next -rc
will enable me to do my pull request, and repair pxa.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dmaengine: pxa_dma: fix initial list move
2015-09-24 19:39 ` Robert Jarzmik
@ 2015-09-25 1:51 ` Vinod Koul
0 siblings, 0 replies; 4+ messages in thread
From: Vinod Koul @ 2015-09-25 1:51 UTC (permalink / raw)
To: Robert Jarzmik
Cc: Daniel Mack, Haojian Zhuang, linux-arm-kernel, dmaengine,
linux-kernel, Arnd Bergmann, Olof Johansson, Kevin Hilman
On Thu, Sep 24, 2015 at 09:39:32PM +0200, Robert Jarzmik wrote:
> Vinod Koul <vinod.koul@intel.com> writes:
>
> > On Mon, Sep 21, 2015 at 11:06:32AM +0200, Robert Jarzmik wrote:
> >> Since the commit to have an allocated list of virtual descriptors was
> >> reverted, the pxa_dma driver is broken, as it assumes the descriptor is
> >> placed on the allocated list upon allocation.
> >>
> >> Fix the issue in pxa_dma by making an allocated virtual descriptor a
> >> singleton.
> >
> > This seems okay, simpler and safer. Do you want me to apply this instead
> > while detailed approach can be reviewed and discussed
> Yes, please. Having the guarantee this gets into the next or next after next -rc
> will enable me to do my pull request, and repair pxa.
Applied, thanks
--
~Vinod
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-09-25 1:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-21 9:06 [PATCH] dmaengine: pxa_dma: fix initial list move Robert Jarzmik
2015-09-24 17:28 ` Vinod Koul
2015-09-24 19:39 ` Robert Jarzmik
2015-09-25 1:51 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).