Re: [linux-dvb] [BUG] changeset 9029 (http://linuxtv.org/hg/v4l-dvb/rev/aa3e5cc1d833)

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

 



On Mon, 2009-02-16 at 14:15 -0500, Steven Toth wrote:
> Steven Toth wrote:
> >> Hartmut, Oliver and Trent: Thanks for helping with this issue. I've just
> >> reverted the changeset. We still need a fix at dm1105, au0828-dvb and 
> >> maybe
> >> other drivers that call the filtering routines inside IRQ's.
> > 
> > Fix the demux, add a worker thread and allow drivers to call it directly.
> > 
> > I'm not a big fan of videobuf_dvb or having each driver do it's own 
> > thing as an alternative.
> > 
> > Fixing the demux... Would this require and extra buffer copy? probably, 
> > but it's a trade-off between  the amount of spent during code management 
> > on a driver by driver basis vs wrestling with videobuf_dvb and all of 
> > problems highlighted on the ML over the last 2 years.
> > 
> > demux->register_driver()
> > demux->deliver_payload()
> > demux->unregister_driver()
> > 
> > Then deprecate sw_filter....N() methods.
> > 
> > That would simplify drivers significantly, at the expense of another 
> > buffer copy while deliver-payload() clones the buffer into its internal 
> > state to be more timely.
> 
> I meant to add...
> 
> The cx18 and a few other smaller drivers (flexcop?) dvb drivers also call 
> directly. cx23885/cx88 does not.

cx18 calls it from it's own worker thread.  To keep up with the rate at
which the CX23418 firmware hands over buffers (and times out the cpu2epu
mailbox!) during a dual analog DTV capture, there's no avoiding having a
worker thread in the cx18 driver.

Regards,
Andy


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