Re: [PATCH 3/4] media i.MX27 camera: improve discard buffer handling.

2012-01-30 Thread javier Martin
Hi Guennadi, On 27 January 2012 19:02, Guennadi Liakhovetski wrote: > (removing bar...@tkos.co.il - it bounces) > > On Thu, 26 Jan 2012, javier Martin wrote: > >> Hi Guennadi, >> >> On 25 January 2012 13:12, Guennadi Liakhovetski >> wrote: >> > On Fri, 20 Jan 2012, Javier Martin wrote: >> > >>

Re: [PATCH 3/4] media i.MX27 camera: improve discard buffer handling.

2012-01-27 Thread Guennadi Liakhovetski
(removing bar...@tkos.co.il - it bounces) On Thu, 26 Jan 2012, javier Martin wrote: > Hi Guennadi, > > On 25 January 2012 13:12, Guennadi Liakhovetski wrote: > > On Fri, 20 Jan 2012, Javier Martin wrote: > > > >> The way discard buffer was previously handled lead > >> to possible races that mad

Re: [PATCH 3/4] media i.MX27 camera: improve discard buffer handling.

2012-01-26 Thread javier Martin
Hi Guennadi, On 25 January 2012 13:12, Guennadi Liakhovetski wrote: > On Fri, 20 Jan 2012, Javier Martin wrote: > >> The way discard buffer was previously handled lead >> to possible races that made a buffer that was not >> yet ready to be overwritten by new video data. This >> is easily detected

Re: [PATCH 3/4] media i.MX27 camera: improve discard buffer handling.

2012-01-25 Thread Guennadi Liakhovetski
On Fri, 20 Jan 2012, Javier Martin wrote: > The way discard buffer was previously handled lead > to possible races that made a buffer that was not > yet ready to be overwritten by new video data. This > is easily detected at 25fps just adding "#define DEBUG" > to enable the "memset" check and seei

[PATCH 3/4] media i.MX27 camera: improve discard buffer handling.

2012-01-20 Thread Javier Martin
The way discard buffer was previously handled lead to possible races that made a buffer that was not yet ready to be overwritten by new video data. This is easily detected at 25fps just adding "#define DEBUG" to enable the "memset" check and seeing how the image is corrupted. A new "discard" queue