Re: [PATCH] cx23885/vb2 regression: please test this patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2015-01-16 at 15:58 +0100, Hans Verkuil wrote:
> On 01/15/2015 05:32 PM, Jurgen Kramer wrote:
> > Hi Hans,
> > 
> > On Tue, 2015-01-13 at 17:59 +0100, Jurgen Kramer wrote:
> >> Hi Hans,
> >> On Tue, 2015-01-13 at 15:01 +0100, Hans Verkuil wrote:
> >>> Hi Raimonds, Jurgen,
> >>>
> >>> Can you both test this patch? It should (I hope) solve the problems you
> >>> both had with the cx23885 driver.
> >>>
> >>> This patch fixes a race condition in the vb2_thread that occurs when
> >>> the thread is stopped. The crucial fix is calling kthread_stop much
> >>> earlier in vb2_thread_stop(). But I also made the vb2_thread more
> >>> robust.
> >>
> >> Thanks. Will test your patch and report back.
> > I have tested the patch, unfortunately results are not positive.
> > With the patch MythTV has issues with the tuners. Live TV no longer
> > works and eventually mythbackend hangs. Reverting the patch brings
> > everything back to a working state.
> 
> Which kernel version are you using? Do you use media_build to install
> the drivers or do you use a git repository? Do you see any kernel messages?
I am on 3.17.7 (F20 x86-64) with current media_build (git://linuxtv.org/media_build.git)
There were no kernel messages indicating failure, same goes for
mythbackend.

> Do you get problems with e.g. mplayer or vlc or another GUI tool as well?
I have only tested it with MythTV. I'll give a w_scan and mplayer a try to see if that combo gives working TV output.

Regards,
Jurgen

--
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



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux