Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-30 Thread Alexandre Courbot
On Tue, May 30, 2017 at 5:30 PM, Sylwester Nawrocki wrote: > Hi, > > On 05/29/2017 09:08 PM, Jacek Anaszewski wrote: >> >> This patch seems to have lost somehow. Could you help merging it? > > > It's not lost, it has been on my todo queue. I have applied it now. Awesome, thanks!

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-30 Thread Sylwester Nawrocki
Hi, On 05/29/2017 09:08 PM, Jacek Anaszewski wrote: This patch seems to have lost somehow. Could you help merging it? It's not lost, it has been on my todo queue. I have applied it now. -- Thanks, Sylwester

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-29 Thread Jacek Anaszewski
Hi Mauro, This patch seems to have lost somehow. Could you help merging it? Thanks, Jacek Anaszewski On 05/29/2017 09:29 AM, Alexandre Courbot wrote: > Hi everyone, > > On Thu, Apr 27, 2017 at 4:35 AM, Jacek Anaszewski > wrote: >> On 04/26/2017 04:54 AM, Alexandre Courbot wrote: >>> On Wed, Ap

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-29 Thread Alexandre Courbot
Hi everyone, On Thu, Apr 27, 2017 at 4:35 AM, Jacek Anaszewski wrote: > On 04/26/2017 04:54 AM, Alexandre Courbot wrote: >> On Wed, Apr 26, 2017 at 4:15 AM, Jacek Anaszewski >> wrote: >>> Hi Alexandre, >>> >>> Thanks for the patch. >>> >>> On 04/25/2017 08:19 AM, Alexandre Courbot wrote: v4

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-04-26 Thread Jacek Anaszewski
On 04/26/2017 04:54 AM, Alexandre Courbot wrote: > On Wed, Apr 26, 2017 at 4:15 AM, Jacek Anaszewski > wrote: >> Hi Alexandre, >> >> Thanks for the patch. >> >> On 04/25/2017 08:19 AM, Alexandre Courbot wrote: >>> v4l2_m2m_job_finish(), which is called from the interrupt handler with >>> slock acq

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-04-25 Thread Alexandre Courbot
On Wed, Apr 26, 2017 at 4:15 AM, Jacek Anaszewski wrote: > Hi Alexandre, > > Thanks for the patch. > > On 04/25/2017 08:19 AM, Alexandre Courbot wrote: >> v4l2_m2m_job_finish(), which is called from the interrupt handler with >> slock acquired, can call the device_run() hook immediately if another

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-04-25 Thread Jacek Anaszewski
Hi Alexandre, Thanks for the patch. On 04/25/2017 08:19 AM, Alexandre Courbot wrote: > v4l2_m2m_job_finish(), which is called from the interrupt handler with > slock acquired, can call the device_run() hook immediately if another > context was in the queue. This hook also acquires slock, resultin

[PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-04-24 Thread Alexandre Courbot
v4l2_m2m_job_finish(), which is called from the interrupt handler with slock acquired, can call the device_run() hook immediately if another context was in the queue. This hook also acquires slock, resulting in a deadlock for this scenario. Fix this by releasing slock right before calling v4l2_m2m