[Bug 770657] New: basetransform: Don't disable pool in set_allocation if it is not replaced

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=770657

            Bug ID: 770657
           Summary: basetransform: Don't disable pool in set_allocation if
                    it is not replaced
    Classification: Platform
           Product: GStreamer
           Version: 1.9.1
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: p.zabel at pengutronix.de
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Created attachment 334542
  --> https://bugzilla.gnome.org/attachment.cgi?id=334542&action=edit
basetransform: Don't disable pool in set_allocation if it is not replaced

If the pool and allocator don't change, skip disabling the pool.

If the "videotestsrc ! v4l2transform ! tee ! queue ! fakesink" pipeline
is running and a new "queue ! fakesink" is added to the tee, caps
negotiation in reconfigure results in the same caps, but setcaps still
calls do_bufferpool, which calls set_allocation with the same pool
and subsequently stops the v4l2 capture pool, issuing a VIDIOC_STREAMOFF
to the kernel driver. This unnecessary stream stop can be avoided if
disabling an unchanged pool is skipped in set_allocation.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.