Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-05-15 Thread Niklas Söderlund
Hi Vinod, On 2017-05-14 17:31:36 +0530, Vinod Koul wrote: > On Fri, May 12, 2017 at 02:49:38PM +0200, Niklas Söderlund wrote: > > On 2017-04-07 14:33:47 +0300, Laurent Pinchart wrote: > > > Hi Geert, > > > > > > On Wednesday 05 Apr 2017 12:40:11 Geert Uytterhoeven wrote: > > > > On Wed, Apr 5, 20

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-05-14 Thread Vinod Koul
On Fri, May 12, 2017 at 02:49:38PM +0200, Niklas Söderlund wrote: > On 2017-04-07 14:33:47 +0300, Laurent Pinchart wrote: > > Hi Geert, > > > > On Wednesday 05 Apr 2017 12:40:11 Geert Uytterhoeven wrote: > > > On Wed, Apr 5, 2017 at 11:14 AM, Niklas Söderlund wrote: > > > > On 2017-04-05 08:55:31

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-05-12 Thread Niklas Söderlund
On 2017-04-07 14:33:47 +0300, Laurent Pinchart wrote: > Hi Geert, > > On Wednesday 05 Apr 2017 12:40:11 Geert Uytterhoeven wrote: > > On Wed, Apr 5, 2017 at 11:14 AM, Niklas Söderlund wrote: > > > On 2017-04-05 08:55:31 +0530, Vinod Koul wrote: > > >> On Thu, Mar 30, 2017 at 09:38:39AM +0200, Nikl

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-04-07 Thread Laurent Pinchart
Hi Geert, On Wednesday 05 Apr 2017 12:40:11 Geert Uytterhoeven wrote: > On Wed, Apr 5, 2017 at 11:14 AM, Niklas Söderlund wrote: > > On 2017-04-05 08:55:31 +0530, Vinod Koul wrote: > >> On Thu, Mar 30, 2017 at 09:38:39AM +0200, Niklas Söderlund wrote: > >>> On 2017-03-29 15:30:42 +0200, Niklas Söd

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-04-05 Thread Geert Uytterhoeven
Hi Niklas, (CC Laurent) On Wed, Apr 5, 2017 at 11:14 AM, Niklas Söderlund wrote: > On 2017-04-05 08:55:31 +0530, Vinod Koul wrote: >> On Thu, Mar 30, 2017 at 09:38:39AM +0200, Niklas Söderlund wrote: >> > On 2017-03-29 15:30:42 +0200, Niklas Söderlund wrote: >> > > On 2017-03-29 14:31:33 +0200,

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-04-05 Thread Niklas Söderlund
Hi Vinod, On 2017-04-05 08:55:31 +0530, Vinod Koul wrote: > On Thu, Mar 30, 2017 at 09:38:39AM +0200, Niklas Söderlund wrote: > > Hi Geert, > > > > On 2017-03-29 15:30:42 +0200, Niklas Söderlund wrote: > > > Hi Geert, > > > > > > On 2017-03-29 14:31:33 +0200, Geert Uytterhoeven wrote: > > > > Hi

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-04-04 Thread Vinod Koul
On Thu, Mar 30, 2017 at 09:38:39AM +0200, Niklas Söderlund wrote: > Hi Geert, > > On 2017-03-29 15:30:42 +0200, Niklas Söderlund wrote: > > Hi Geert, > > > > On 2017-03-29 14:31:33 +0200, Geert Uytterhoeven wrote: > > > Hi Niklas, > > > > > > On Wed, Mar 29, 2017 at 12:40 AM, Niklas Söderlund >

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-03-30 Thread Niklas Söderlund
Hi Geert, On 2017-03-29 15:30:42 +0200, Niklas Söderlund wrote: > Hi Geert, > > On 2017-03-29 14:31:33 +0200, Geert Uytterhoeven wrote: > > Hi Niklas, > > > > On Wed, Mar 29, 2017 at 12:40 AM, Niklas Söderlund > > wrote: > > > This fixes a race condition where the channel resources could be fre

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-03-29 Thread Niklas Söderlund
Hi Geert, On 2017-03-29 14:31:33 +0200, Geert Uytterhoeven wrote: > Hi Niklas, > > On Wed, Mar 29, 2017 at 12:40 AM, Niklas Söderlund > wrote: > > This fixes a race condition where the channel resources could be freed > > before the ISR had finished running resulting in a NULL pointer > > refere

Re: [PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-03-29 Thread Geert Uytterhoeven
Hi Niklas, On Wed, Mar 29, 2017 at 12:40 AM, Niklas Söderlund wrote: > This fixes a race condition where the channel resources could be freed > before the ISR had finished running resulting in a NULL pointer > reference from the ISR. > > [ 167.148934] Unable to handle kernel NULL pointer derefer

[PATCH 3/3] dmaengine: rcar-dmac: wait for ISR to finish before freeing resources

2017-03-28 Thread Niklas Söderlund
This fixes a race condition where the channel resources could be freed before the ISR had finished running resulting in a NULL pointer reference from the ISR. [ 167.148934] Unable to handle kernel NULL pointer dereference at virtual address [ 167.157051] pgd = 80003c641000 [ 167.1