Re: [PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-11-10 Thread Guennadi Liakhovetski
On Mon, 10 Nov 2014, Guennadi Liakhovetski wrote: > Hi Laurent, > > On Thu, 2 Oct 2014, Laurent Pinchart wrote: > > > Hi Guennadi, > > > > Ping ? > > Sorry again for a delay, and unfortunately my eventual reply won't be very > helpful: we've modified our user-space in a way, that that path is

Re: [PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-11-10 Thread Guennadi Liakhovetski
Hi Laurent, On Thu, 2 Oct 2014, Laurent Pinchart wrote: > Hi Guennadi, > > Ping ? Sorry again for a delay, and unfortunately my eventual reply won't be very helpful: we've modified our user-space in a way, that that path isn't triggered anymore, so, I cannot easily verify your patch. In any c

Re: [PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-11-07 Thread Guennadi Liakhovetski
Hi Laurent, On Thu, 2 Oct 2014, Laurent Pinchart wrote: > Hi Guennadi, > > Ping ? Uhm, yes, that's bad. 1 month and a week. Sorry. I'll try to test this on Monday. Thanks Guennadi > On Wednesday 01 October 2014 00:42:51 Laurent Pinchart wrote: > > Commit e93e7fd9f5a3fffec7792dbcc4c3574653eff

Re: [PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-10-02 Thread Laurent Pinchart
Hi Guennadi, Ping ? On Wednesday 01 October 2014 00:42:51 Laurent Pinchart wrote: > Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow > using larger buffers") reworked the buffer size sanity check at buffer > completion time to use the frame size instead of the allocated buf

Re: [PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-09-30 Thread Laurent Pinchart
On Wednesday 01 October 2014 00:42:51 Laurent Pinchart wrote: > Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow > using larger buffers") reworked the buffer size sanity check at buffer > completion time to use the frame size instead of the allocated buffer > size. However, i

[PATCH] v4l: uvcvideo: Fix buffer completion size check

2014-09-30 Thread Laurent Pinchart
Commit e93e7fd9f5a3fffec7792dbcc4c3574653effda7 ("v4l2: uvcvideo: Allow using larger buffers") reworked the buffer size sanity check at buffer completion time to use the frame size instead of the allocated buffer size. However, it introduced two bugs in doing so: - it assigned the allocated buffer