All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Frank Schäfer" <fschaefer.oss@googlemail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: em28xx vb2 warnings
Date: Tue, 29 Jul 2014 19:21:42 +0200	[thread overview]
Message-ID: <53D7D826.2020703@googlemail.com> (raw)
In-Reply-To: <53D66BFD.6020809@xs4all.nl>


Am 28.07.2014 um 17:27 schrieb Hans Verkuil:
...
> OK, I looked at it: the problem is in get_next_buf() and finish_field_prepare_next().
> In get_next_buf() the driver gets a buffer from the active list and deletes it from
> that list. In finish_field_prepare_next() that buffer is given back to vb2 via
> finish_buffer().
>
> But if you stop streaming and em28xx_stop_streaming() is called, then that buffer that
> is being processed isn't part of the active list anymore and so it is never given back.
>
> em28xx_stop_streaming() should give that buffer back as well, and that will keep
> everything in balance. The easiest solution seems to be to move the list_del() call
> from get_next_buf() to finish_buffer(). It seemed to work in a quick test, but I
> haven't looked at vbi support or corner cases. I leave that to you :-)

Ok, thank you so far Hans !
I will see what I can do.

Regards,
Frank



      reply	other threads:[~2014-07-29 17:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-25 16:20 em28xx vb2 warnings Frank Schäfer
2014-07-25 19:07 ` Hans Verkuil
2014-07-27 21:17   ` Frank Schäfer
2014-07-27 21:19     ` Hans Verkuil
2014-07-28 15:27 ` Hans Verkuil
2014-07-29 17:21   ` Frank Schäfer [this message]

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=53D7D826.2020703@googlemail.com \
    --to=fschaefer.oss@googlemail.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.