From: Raimonds Cicans <ray@apollo.lv>
To: Hans Verkuil <hverkuil@xs4all.nl>,
linux-media <linux-media@vger.kernel.org>,
gtmkramer@xs4all.nl
Subject: Re: [PATCH] cx23885/vb2 regression: please test this patch
Date: Wed, 14 Jan 2015 08:16:02 +0200 [thread overview]
Message-ID: <54B609A2.2040503@apollo.lv> (raw)
In-Reply-To: <54B55C23.1070409@apollo.lv>
On 13.01.2015 19:55, Raimonds Cicans wrote:
> On 13.01.2015 16:01, 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.
>
> With this patch I am unable to get any error except first
> (AMD-Vi: Event logged [IO_PAGE_FAULT...).
> But I am not convinced, because before patch I get
> first error much often and earlier than almost any other error,
> so it may be just "bad luck" and other errors do not
> appear because first error appear earlier.
>
I noticed that if I "initialize" card with commands:
/usr/bin/dvb-fe-tool -a 4 -d DVBS
/usr/bin/dvb-fe-tool -a 5 -d DVBS
and then run load which starts on first front-end and
only after some time on second, then I receive first
error much later or do not receive at all (for example
I was able to run VDR whole night without problems)
Raimonds Cicans
next prev parent reply other threads:[~2015-01-14 6:16 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
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 [this message]
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=54B609A2.2040503@apollo.lv \
--to=ray@apollo.lv \
--cc=gtmkramer@xs4all.nl \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
/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.