[Bug 641204] dvbsrc: Faster tuning logic: Use poll instead of artificial usleep

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon May 26 21:03:13 PDT 2014


https://bugzilla.gnome.org/show_bug.cgi?id=641204
  GStreamer | gst-plugins-bad | git

Reynaldo H. Verdejo Pinochet <reynaldo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #179848|0                           |1
        is obsolete|                            |
 Attachment #179950|0                           |1
        is obsolete|                            |

--- Comment #25 from Reynaldo H. Verdejo Pinochet <reynaldo at opendot.cl> 2014-05-27 01:49:44 UTC ---
Created an attachment (id=277252)
 View: https://bugzilla.gnome.org/attachment.cgi?id=277252
 Review: https://bugzilla.gnome.org/review?bug=641204&attachment=277252

Tuning: s/sleep()/poll + configurable timeouts + extras

Drops sleep logic in favor of configurable timeouts/polling
plus some extra refactors and improvements.

Most of the "extras" made sense on a single change, some
I might had handled separated (like the new signal bits)
but the thing grow organically as I attempted to match
Fabrizio's patch breakdown scheme, my bad.

Patch should apply cleanly over today's master, and has been
tested to work (with ATSC mostly).

Would love some hints/corrections specially on the polling /
timeout bits. I'm unsure as to whether moving the frontend's
poll vars to the GstDvbSrc struct would be desirable, maybe
you would like me to provide the user with a way to cancel
the main gst_poll_wait() ? I thought about pushing that idea
forward but would like to have your take on it first.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.


More information about the gstreamer-bugs mailing list