On Thu, Aug 14, 2014 at 4:11 PM, panpan liu <panpan1....@samsung.com> wrote: > If num_users returns 1 or more than 1, that means we are not the > only user of the plane's memory. > > Signed-off-by: panpan liu <panpan1....@samsung.com>
NACK. Please read the function documentation and how it is used. If the number of users is 1, we are the only user and should be able to free the queue. This will make us unable to do so. > --- > drivers/media/v4l2-core/videobuf2-core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c > b/drivers/media/v4l2-core/videobuf2-core.c > index c359006..d3a4b8f 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -625,7 +625,7 @@ static bool __buffer_in_use(struct vb2_queue *q, struct > vb2_buffer *vb) > * case anyway. If num_users() returns more than 1, > * we are not the only user of the plane's memory. > */ > - if (mem_priv && call_memop(vb, num_users, mem_priv) > 1) > + if (mem_priv && call_memop(vb, num_users, mem_priv) >= 1) > return true; > } > return false; > -- > 1.7.9.5 -- Thanks, Pawel Osciak -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
