Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Vinod Koul]

On Thu, Sep 24, 2015 at 09:39:32PM +0200, Robert Jarzmik wrote:
> Vinod Koul  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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Robert Jarzmik]

Vinod Koul  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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Vinod Koul]

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Vinod Koul]

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Robert Jarzmik]

Vinod Koul  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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] dmaengine: pxa_dma: fix initial list move

[Vinod Koul]

On Thu, Sep 24, 2015 at 09:39:32PM +0200, Robert Jarzmik wrote:
> Vinod Koul  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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] dmaengine: pxa_dma: fix initial list move

[Robert Jarzmik]

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 
---
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(>node);
tx = vchan_tx_prep(vc, vd, tx_flags);
tx->tx_submit = pxad_tx_submit;
dev_dbg(>vc.chan.dev->device,
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] dmaengine: pxa_dma: fix initial list move

[Robert Jarzmik]

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 
---
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(>node);
tx = vchan_tx_prep(vc, vd, tx_flags);
tx->tx_submit = pxad_tx_submit;
dev_dbg(>vc.chan.dev->device,
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/