From: "Aleksandr V. Piskunov" <aleksandr.v.piskunov@gmail.com>
To: "Aleksandr V. Piskunov" <aleksandr.v.piskunov@gmail.com>
Cc: Antti Palosaari <crope@iki.fi>,
Markus Rechberger <mrechberger@gmail.com>,
Devin Heitmueller <dheitmueller@kernellabs.com>,
Clinton Meyer <clintonmeyer22@gmail.com>,
Linux Media <linux-media@vger.kernel.org>
Subject: Re: LinuxTV firmware blocks all wireless connections / traffic
Date: Thu, 10 Sep 2009 22:39:16 +0300 [thread overview]
Message-ID: <20090910193916.GA4923@moon> (raw)
In-Reply-To: <20090910171631.GA4423@moon>
On Thu, Sep 10, 2009 at 08:16:31PM +0300, Aleksandr V. Piskunov wrote:
> On Thu, Sep 10, 2009 at 05:48:22PM +0300, Antti Palosaari wrote:
> > On 09/10/2009 04:47 PM, Antti Palosaari wrote:
> >> Aleksandr V. Piskunov wrote:
> >>> On Thu, Sep 10, 2009 at 04:12:15PM +0300, Antti Palosaari wrote:
> >>>> Aleksandr V. Piskunov wrote:
> >>>>>> Here is a test case:
> >>>>>> Two DVB-T USB adapters, dvb_usb_af9015 and dvb_usb_af9015.
> >>>>>> Different tuners,
> >>>>> Err, make it: dvb_usb_af9015 and dvb_usb_ce6230
> >>>> Those both uses currently too small bulk urbs, only 512 bytes. I have
> >>>> asked suitable bulk urb size for ~20mbit/sec usb2.0 stream, but
> >>>> no-one have answered yet (search ml back week or two). I think will
> >>>> increase those to the 8k to reduce load.
> >>>>
> >>>
> >>> Nice, I'm ready to test if such change helps.
> >>
> >> OK, I will make test version in couple of hours.
> >
> > Here it is, USB2.0 URB is now about 16k both af9015 and ce6230 devices.
> > Now powertop shows only about 220 wakeups on my computer for the both
> > sticks.
> > Please test and tell what powertop says:
> > http://linuxtv.org/hg/~anttip/urb_size/
> >
> > I wonder if we can decide what URB size DVB USB drivers should follow
> > and even add new module param for overriding driver default.
>
> Thanks, Antti!
>
> Tested your branch on affected system.
>
> Load definitely went down, from ~7000 wakeups to ~250 for each tuner
> according to powertop.
> Both tuners still working ok if not used simultaneously or if used the
> same time on different USB controllers.
>
> Bad news are that original problem still persists: putting both tuners
> on same USB controller and zapping simultaneously corrupts stream.
> Interesting observation: no matter in what sequence tuners are connected
> (i.e. become adapter0 or adapter1), af9015 stream always gets heavily
> distorted, visually mplayer picture becomes like 80% corrupted with
> random color blocks and pixels, sound becomes a mess. At the same time
> ce6230 gets slight corruption, a few discolored blocks at the time and
> sound hickups.
>
> Anyway, will try to do a few more tests:
> 1) Two usb flash drives on same controller calculating md5sum of
> big .iso file, to check if it is/isn't dvb-usb problem.
> 2) Will see if same issue persists on another PC with same motherboard
> (slightly different revision) to rule out hardware issues. If I manage
> to wire antenna there, that is...
Ok, two USB flash drives on same controller, no problem when bulk reading
from both at the same time, no speed drops, no corruption.
Now if I plug ce6230 tuner, zap to channel and then start reading from
flash drive:
* slightly corrupted TS stream
* flash drive read getting starved on bandwidth, speed drops from 10 MB/s
to ~7 MB/s
If I plug af9015 tuner, zap and read from flash
* heavy corruption of TS stream
* flash drive read speed drops from 10 MB/s to 2(!) MB/s
Now I don't really know the USB protocol under-the-hood details, all the
different types of bandwidth, reservation and so on. But shouldn't one
480 Mbit/sec controller handle rather large number of digital tuners, each
pushing 20-25 Mbit/sec max, even considering all the overhead?
next prev parent reply other threads:[~2009-09-10 19:39 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-09 21:43 LinuxTV firmware blocks all wireless connections / traffic Clinton Meyer
2009-09-09 21:59 ` Devin Heitmueller
2009-09-10 9:14 ` Aleksandr V. Piskunov
2009-09-10 10:58 ` Markus Rechberger
2009-09-10 12:45 ` Aleksandr V. Piskunov
2009-09-10 12:48 ` Aleksandr V. Piskunov
2009-09-10 13:12 ` Antti Palosaari
2009-09-10 13:41 ` Aleksandr V. Piskunov
2009-09-10 13:47 ` Antti Palosaari
2009-09-10 14:48 ` Antti Palosaari
2009-09-10 15:26 ` Devin Heitmueller
2009-09-10 15:55 ` Antti Palosaari
2009-09-10 16:12 ` Devin Heitmueller
2009-09-10 16:48 ` Antti Palosaari
2009-09-10 17:17 ` Devin Heitmueller
2009-09-10 20:29 ` Antti Palosaari
2009-09-10 20:45 ` Devin Heitmueller
2009-09-10 17:16 ` Aleksandr V. Piskunov
2009-09-10 19:39 ` Aleksandr V. Piskunov [this message]
2009-09-11 14:38 ` Antti Palosaari
2009-09-11 17:50 ` Aleksandr V. Piskunov
2009-09-11 18:01 ` Devin Heitmueller
2009-09-11 19:47 ` Antti Palosaari
2009-09-12 15:46 ` CityK
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=20090910193916.GA4923@moon \
--to=aleksandr.v.piskunov@gmail.com \
--cc=clintonmeyer22@gmail.com \
--cc=crope@iki.fi \
--cc=dheitmueller@kernellabs.com \
--cc=linux-media@vger.kernel.org \
--cc=mrechberger@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox