From: Hans Verkuil <hverkuil@xs4all.nl>
To: Jurgen Kramer <gtmkramer@xs4all.nl>
Cc: linux-media <linux-media@vger.kernel.org>, ray@apollo.lv
Subject: Re: [PATCH] cx23885/vb2 regression: please test this patch
Date: Fri, 16 Jan 2015 15:58:37 +0100 [thread overview]
Message-ID: <54B9271D.8010703@xs4all.nl> (raw)
In-Reply-To: <1421339578.2355.2.camel@xs4all.nl>
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?
Do you get problems with e.g. mplayer or vlc or another GUI tool as well?
This doesn't really make sense to me why this would fail.
Regards,
Hans
>
> Regards,
> Jurgen
>
>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2015-01-16 14:58 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-13 14:01 [PATCH] cx23885/vb2 regression: please test this patch Hans Verkuil
2015-01-13 16:59 ` Jurgen Kramer
2015-01-15 16:32 ` Jurgen Kramer
2015-01-16 14:58 ` Hans Verkuil [this message]
2015-01-16 16:48 ` Jurgen Kramer
2015-01-18 8:54 ` Jurgen Kramer
2015-01-18 10:40 ` Hans Verkuil
2015-01-18 12:35 ` Jurgen Kramer
2015-01-13 17:55 ` Raimonds Cicans
2015-01-14 6:16 ` Raimonds Cicans
2015-01-16 14:54 ` Hans Verkuil
2015-01-16 16:20 ` Raimonds Cicans
2015-01-16 16:48 ` Luis Alves
2015-01-16 17:38 ` Hans Verkuil
2015-01-16 17:40 ` Hans Verkuil
2015-01-16 19:05 ` Raimonds Cicans
2015-01-17 11:09 ` Hans Verkuil
2015-01-17 12:02 ` Raimonds Cicans
2015-01-17 12:30 ` Tycho Lürsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54B9271D.8010703@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=gtmkramer@xs4all.nl \
--cc=linux-media@vger.kernel.org \
--cc=ray@apollo.lv \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.