On Mon April 1 2013 16:23:51 Hans de Goede wrote: > Hi, > > On 04/01/2013 12:19 PM, Hans Verkuil wrote: > > Hi Hans, > > > > On Sun March 31 2013 14:48:01 Hans de Goede wrote: > >> Hi, > >> > >> On 03/30/2013 10:47 AM, Hans Verkuil wrote: > >>> This patch for xawtv3 releases the buffer if it can't be displayed because > >>> the resolution of the current format is larger than the size of the > >>> window. > >>> > >>> This will happen if the hardware cannot scale down to the initially quite > >>> small xawtv window. For example the au0828 driver has a fixed size of > >>> 720x480, > >>> so it will not display anything until the window is large enough for that > >>> resolution. > >>> > >>> The problem is that xawtv never releases (== calls QBUF) the buffer in > >>> that > >>> case, and it will of course run out of buffers and stall. The only way to > >>> kill it is to issue a 'kill -9' since ctrl-C won't work either. > >>> > >>> By releasing the buffer xawtv at least remains responsive and a picture > >>> will > >>> appear after resizing the window. Ideally of course xawtv should resize > >>> itself > >>> to the minimum supported resolution, but that's left as an exercise for > >>> the > >>> reader... > >>> > >>> Hans, the xawtv issues I reported off-list are all caused by this bug and > >>> by > >>> by the scaling bug introduced recently in em28xx. They had nothing to do > >>> with > >>> the alsa streaming, that was a red herring. > >> > >> Thanks for the debugging and for the patch. I've pushed the patch to > >> xawtv3.git. I've a 2 patch follow up set which should fix the issue with > >> being > >> able to resize the window to a too small size. > >> > >> I'll send this patch set right after this mail, can you test it with the > >> au0828 > >> please? > > > > I've tested it and it is not yet working. I've tracked it down to > > video_gd_configure > > where it calls ng_ratio_fixup() which changes the cur_tv_width of 736 to > > 640. The > > height remains the same at 480. > > Thanks for testing and for figuring out where the problem lies. I've attached > a > second version of the second patch, can you give that a try please?
This is now working for au0828, but now vivi is broken... That worked fine with your previous patch. I'm getting: $ xawtv This is xawtv-3.102, running on Linux/x86_64 (3.9.0-rc1-tschai) ioctl: VIDIOC_QUERYMENU(id=134217731;index=2;name="Menu Item 1";reserved=0): Invalid argument vid-open-auto: using grabber/webcam device /dev/video0 libv4l2: error setting pixformat: Device or resource busy ioctl: VIDIOC_S_FMT(type=VIDEO_CAPTURE;fmt.pix.width=384;fmt.pix.height=288;fmt.pix.pixelformat=0x34524742 [BGR4];fmt.pix.field=INTERLACED;fmt.pix.bytesperline=1536;fmt.pix.sizeimage=442368;fmt.pix.colorspace=SRGB;fmt.pix.priv=0): Device or resource busy Note that the QUERYMENU error is harmless, although it would be nice if xawtv would understand menu controls with 'holes' in the menu list. The 'Device or resource busy' errors are new and I didn't have them in your previous version. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html