Skip to content
Snippets Groups Projects
  1. Jun 05, 2013
  2. Jun 04, 2013
  3. Jun 03, 2013
    • Michael Olbrich's avatar
      v4l2: rework sink buffer refcounting · 496995a7
      Michael Olbrich authored
      This is a followup patch for #700781, which is not quite correct.
      The buffer handling is quite complicated here.
      The original code intended to the the following:
      
      - gst_v4l2_buffer_pool_process() calls QBUF and adds the buffer to the
        local list.
      - The sink calls gst_buffer_unref() which returns the buffer to the pool
        but not the 'free list'.
      - Some time later DQBUF returns the buffer and
        gst_v4l2_buffer_pool_release_buffer() puts in on the 'free list'.
      
      If the buffer must be copied then (parent_class)->acquire_buffer() is
      called directly to keep the buffer in the pool.
      
      This has two problems:
      1. If gst_v4l2_buffer_pool_release_buffer() is called before the buffer is
         returned to the pool, then the buffer is put on the 'free list' twice.
         This can happen if a reference to the buffer is kept outside the sink,
         of if DQBUF returns the buffer, that was just queued with QBUF.
      2. If buffers are copied, then all buffers are in the pool at all times. As
         a result gst_v4l2_buffer_pool_stop() and gst_v4l2_buffer_pool_dqbuf()
         can access pool->buffers at the same time, which can lead to memory
         corruption.
      
      The patch for #700781 fixes those problems, but with the side effect that
      there are always buffers outside the pool (because they are queued) and
      the pool is never stopped.
      This patch fixes this by releasing the reference to the buffer after
      handling it (to avoid problem 2.) so it can be returned to the pool.
      gst_v4l2_buffer_pool_release_buffer() is only called if the buffer is
      already in the pool (to avoid problem 1.).
      
      Fixes https://bugzilla.gnome.org/show_bug.cgi?id=701375
      496995a7
  4. Jun 02, 2013
  5. Jun 01, 2013
  6. May 31, 2013
  7. May 30, 2013
  8. May 29, 2013
  9. May 28, 2013
  10. May 26, 2013
  11. May 25, 2013
Loading