Re: [[PATCH v3]] videobuf2: Add missing lock held on vb2_fop_relase

2013-11-02 Thread Ricardo Ribalda Delgado
Hello Sylwester Thanks for your comments. There is a new patch: v4! :) On Fri, Nov 1, 2013 at 11:36 PM, Sylwester Nawrocki sylvester.nawro...@gmail.com wrote: Hi Ricardo, On 10/31/2013 09:54 PM, Ricardo Ribalda Delgado wrote: From: Ricardo Ribaldaricardo.riba...@gmail.com vb2_fop_relase

Re: [[PATCH v3]] videobuf2: Add missing lock held on vb2_fop_relase

2013-11-01 Thread Sylwester Nawrocki
Hi Ricardo, On 10/31/2013 09:54 PM, Ricardo Ribalda Delgado wrote: From: Ricardo Ribaldaricardo.riba...@gmail.com vb2_fop_relase does not held the lock although it is modifying the queue-owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications

[[PATCH v3]] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-31 Thread Ricardo Ribalda Delgado
From: Ricardo Ribalda ricardo.riba...@gmail.com vb2_fop_relase does not held the lock although it is modifying the queue-owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications are accessing the video device via read/write API: [ 308.297741]