Em Tue, 16 Sep 2014 12:09:01 +0300 Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> escreveu: > Hi Mauro, > > On Monday 15 September 2014 21:10:55 Mauro Carvalho Chehab wrote: > > This reverts commit 9241650d62f79a3da01f1d5e8ebd195083330b75. > > > > The commit 9241650d62f7 was meant to solve an issue with Gstreamer > > version 0.10 with libv4l 1.2, where a fixup patch for DQBUF exposed > > a bad behavior ag Gstreamer. > > That's not correct. The patch was created to solve an issue observed with the > Gstreamer 0.10 v4l2src element accessing the video device directly, *without* > libv4l. Ok. From the discussions we took yesterday on the thread, I got the wrong impression from Nicolas comments that this happens only with gst < 1.4 and libv4l >= 1.2. > > The V4L2 specification documents poll() as follows. > > "When the application did not call VIDIOC_QBUF or VIDIOC_STREAMON yet the > poll() function succeeds, but sets the POLLERR flag in the revents field." > > The vb2 poll implementation didn't conform with that, as it returned POLLERR > when the buffer list was empty due to a transient buffer underrun, even if > both VIDIOC_STREAMON and VIDIOC_QBUF have been called. > > The commit thus brought the vb2 poll implementation in line with the > specification. If we really want to revert it to its broken behaviour, then it > would be fair to explain this in the revert message, Ok, I'll rewrite the text. We likely want to fix the documentation too, in order to reflect the way it is. > and I want to know how > you propose fixing this properly, as the revert really causes issues for > userspace. This patch simply broke all VBI applications. So, it should be reverted. >From what you're saying, using Gst 0.10 with a kernel before 3.16 and VB2 was always broken, right? And with VB1, is it also broken? If so, then this is a Gst 0.10 bug, and the fix should be a patch for it, or a recommendation to upgrade to a newer version without such bug. If, otherwise, it works with VB1, then we need to patch VB2 to have exactly the same behavior as VB1 with that regards, as VBI works with VB1. Regards, Mauro. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
