| Message ID | 20170616073915.5027-8-gustavo@padovan.org (mailing list archive) |
|---|---|
| State | Changes Requested, archived |
| Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from <linux-media-owner@vger.kernel.org>) id 1dLlrM-0005p6-Se; Fri, 16 Jun 2017 07:40:00 +0000 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.89/mailfrontend-8) with esmtp id 1dLlrL-0004vQ-jA; Fri, 16 Jun 2017 09:40:00 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752687AbdFPHj5 (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 16 Jun 2017 03:39:57 -0400 Received: from mail-pg0-f66.google.com ([74.125.83.66]:33854 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752779AbdFPHj4 (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 16 Jun 2017 03:39:56 -0400 Received: by mail-pg0-f66.google.com with SMTP id j186so4551180pge.1 for <linux-media@vger.kernel.org>; Fri, 16 Jun 2017 00:39:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=LlOSlFUmkEFkbBD17Ys6pGkx+XVaW8kmjQncLsHw418=; b=TrmtgvgtRYnJOK8MNYlR8ym6OGhgL7SwAhmUQuN86BFe+Ns/vJLPiflNVXAiqMJHwQ 9u2j8he+iOfMK7V2W90puh6JpYPmtfm8ExGicKPOPlwJU6wPEf2Z0uOLws005NrmbpeH NlEew7z0SOa9ZcSGYEj1A24ToN8GIghh8MXTCh3QeUDixguFJjKkmkdMYlQWcKN1ulL6 Do9XbYEybWQcxTEqLhQPOZI/rwHTI0bxobrCpAfSU6nmPSoYZ5/5xk1/cNCFMatAtFhx IfBR/rDDQus3/BMXn2yILLNVKblkc7+oSsp1TkSP/Sam4yNBCbESUsiT0KByPQr/dNGf 0QeA== X-Gm-Message-State: AKS2vOw/fo6t0p9Zuvvrc0pBVlIxGrtI7wycxQUDX5S0shmjOFXdJJjd e0Rl3HX7ESRqgOUszHE= X-Received: by 10.101.91.7 with SMTP id y7mr9739242pgq.226.1497598780934; Fri, 16 Jun 2017 00:39:40 -0700 (PDT) Received: from jade.nodan1.kt.home.ne.jp (202-72-64-244.koalanet.ne.jp. [202.72.64.244]) by smtp.gmail.com with ESMTPSA id a2sm2760568pfj.8.2017.06.16.00.39.38 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 16 Jun 2017 00:39:40 -0700 (PDT) From: Gustavo Padovan <gustavo@padovan.org> To: linux-media@vger.kernel.org Cc: Hans Verkuil <hverkuil@xs4all.nl>, Javier Martinez Canillas <javier@osg.samsung.com>, Mauro Carvalho Chehab <mchehab@osg.samsung.com>, Shuah Khan <shuahkh@osg.samsung.com>, Gustavo Padovan <gustavo.padovan@collabora.com> Subject: [PATCH 07/12] [media] v4l: add support to BUF_QUEUED event Date: Fri, 16 Jun 2017 16:39:10 +0900 Message-Id: <20170616073915.5027-8-gustavo@padovan.org> X-Mailer: git-send-email 2.9.4 In-Reply-To: <20170616073915.5027-1-gustavo@padovan.org> References: <20170616073915.5027-1-gustavo@padovan.org> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2017.6.16.73017 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_2000_2999 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, IN_REP_TO 0, LEGITIMATE_SIGNS 0, MSG_THREAD 0, MULTIPLE_REAL_RCPTS 0, NO_URI_HTTPS 0, REFERENCES 0, __ANY_URI 0, __CC_NAME 0, __CC_NAME_DIFF_FROM_ACC 0, __CC_REAL_NAMES 0, __HAS_CC_HDR 0, __HAS_FROM 0, __HAS_LIST_ID 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MIME_TEXT_P 0, __MIME_TEXT_P1 0, __MULTIPLE_RCPTS_CC_X2 0, __NO_HTML_TAG_RAW 0, __REFERENCES 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' |
Commit Message
Gustavo F. Padovan
June 16, 2017, 7:39 a.m. UTC
From: Gustavo Padovan <gustavo.padovan@collabora.com> Implement the needed pieces to let userspace subscribe for V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the DQEVENT ioctl. Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> --- drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ 2 files changed, 20 insertions(+), 1 deletion(-)
Comments
Em Fri, 16 Jun 2017 16:39:10 +0900 Gustavo Padovan <gustavo@padovan.org> escreveu: > From: Gustavo Padovan <gustavo.padovan@collabora.com> > > Implement the needed pieces to let userspace subscribe for > V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the > DQEVENT ioctl. > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> > --- > drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- > drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ > 2 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c > index 5aed7bd..f55b5da 100644 > --- a/drivers/media/v4l2-core/v4l2-ctrls.c > +++ b/drivers/media/v4l2-core/v4l2-ctrls.c > @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); > int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, > const struct v4l2_event_subscription *sub) > { > - if (sub->type == V4L2_EVENT_CTRL) > + switch (sub->type) { > + case V4L2_EVENT_CTRL: > return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); > + case V4L2_EVENT_BUF_QUEUED: > + return v4l2_event_subscribe(fh, sub, 0, NULL); > + } > return -EINVAL; > } > EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 29aa9d4..00d9c35 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -25,6 +25,7 @@ > #include <linux/kthread.h> > > #include <media/videobuf2-core.h> > +#include <media/v4l2-event.h> > #include <media/v4l2-mc.h> > > #include <trace/events/vb2.h> > @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) > return ret; > } > > +static void vb2_buffer_queued_event(struct vb2_buffer *vb) > +{ > + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); > + struct v4l2_event event; > + > + memset(&event, 0, sizeof(event)); > + event.type = V4L2_EVENT_BUF_QUEUED; > + event.u.buf_queued.index = vb->index; > + > + v4l2_event_queue(vdev, &event); > +} > + It doesn't sound right to add a V4L2 event to VB2 core. The hole point of splitting the core from V4L2 specific stuff is to allow VB2 to be used by non-V4L2 APIs[1]. Please move this to videobuf2-v4l2. [1] The split happened as part of a patchset meant to make the DVB core to use VB2 and provide DMA APIs to it. Unfortunately, the developer that worked on this project moved to some other project. The final patch was not applied yet. I have it on my patchwork queue. I intend to test and apply it sometime this year. > /** > * __enqueue_in_driver() - enqueue a vb2_buffer in driver for processing > */ > @@ -1234,6 +1247,8 @@ static void __enqueue_in_driver(struct vb2_buffer *vb) > trace_vb2_buf_queue(q, vb); > > call_void_vb_qop(vb, buf_queue, vb); > + > + vb2_buffer_queued_event(vb); > } > > static int __buf_prepare(struct vb2_buffer *vb, const void *pb)
Hi Mauro, 2017-06-30 Mauro Carvalho Chehab <mchehab@osg.samsung.com>: > Em Fri, 16 Jun 2017 16:39:10 +0900 > Gustavo Padovan <gustavo@padovan.org> escreveu: > > > From: Gustavo Padovan <gustavo.padovan@collabora.com> > > > > Implement the needed pieces to let userspace subscribe for > > V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the > > DQEVENT ioctl. > > > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> > > --- > > drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- > > drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ > > 2 files changed, 20 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c > > index 5aed7bd..f55b5da 100644 > > --- a/drivers/media/v4l2-core/v4l2-ctrls.c > > +++ b/drivers/media/v4l2-core/v4l2-ctrls.c > > @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); > > int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, > > const struct v4l2_event_subscription *sub) > > { > > - if (sub->type == V4L2_EVENT_CTRL) > > + switch (sub->type) { > > + case V4L2_EVENT_CTRL: > > return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); > > + case V4L2_EVENT_BUF_QUEUED: > > + return v4l2_event_subscribe(fh, sub, 0, NULL); > > + } > > return -EINVAL; > > } > > EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); > > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > > index 29aa9d4..00d9c35 100644 > > --- a/drivers/media/v4l2-core/videobuf2-core.c > > +++ b/drivers/media/v4l2-core/videobuf2-core.c > > @@ -25,6 +25,7 @@ > > #include <linux/kthread.h> > > > > #include <media/videobuf2-core.h> > > +#include <media/v4l2-event.h> > > #include <media/v4l2-mc.h> > > > > #include <trace/events/vb2.h> > > @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) > > return ret; > > } > > > > +static void vb2_buffer_queued_event(struct vb2_buffer *vb) > > +{ > > + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); > > + struct v4l2_event event; > > + > > + memset(&event, 0, sizeof(event)); > > + event.type = V4L2_EVENT_BUF_QUEUED; > > + event.u.buf_queued.index = vb->index; > > + > > + v4l2_event_queue(vdev, &event); > > +} > > + > > It doesn't sound right to add a V4L2 event to VB2 core. The hole point > of splitting the core from V4L2 specific stuff is to allow VB2 to be > used by non-V4L2 APIs[1]. Please move this to videobuf2-v4l2. Yes, that makes sense. My lack of understanding of v4l2 core didn't me allow realize that. I'll move it to videobuf2-v4l2. Gustavo
On 06/16/17 09:39, Gustavo Padovan wrote: > From: Gustavo Padovan <gustavo.padovan@collabora.com> > > Implement the needed pieces to let userspace subscribe for > V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the > DQEVENT ioctl. > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> > --- > drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- > drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ > 2 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c > index 5aed7bd..f55b5da 100644 > --- a/drivers/media/v4l2-core/v4l2-ctrls.c > +++ b/drivers/media/v4l2-core/v4l2-ctrls.c > @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); > int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, > const struct v4l2_event_subscription *sub) > { > - if (sub->type == V4L2_EVENT_CTRL) > + switch (sub->type) { > + case V4L2_EVENT_CTRL: > return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); > + case V4L2_EVENT_BUF_QUEUED: > + return v4l2_event_subscribe(fh, sub, 0, NULL); This is dangerous. The '0' argument will only allocate room for a single BUF_QUEUED event. So if two such events are triggered without the application reading the first event, then the first event will be lost. I recommend VIDEO_MAX_FRAME instead. I.e. have room for up to the maximum number of buffers. > + } > return -EINVAL; > } > EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); > diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > index 29aa9d4..00d9c35 100644 > --- a/drivers/media/v4l2-core/videobuf2-core.c > +++ b/drivers/media/v4l2-core/videobuf2-core.c > @@ -25,6 +25,7 @@ > #include <linux/kthread.h> > > #include <media/videobuf2-core.h> > +#include <media/v4l2-event.h> > #include <media/v4l2-mc.h> > > #include <trace/events/vb2.h> > @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) > return ret; > } > > +static void vb2_buffer_queued_event(struct vb2_buffer *vb) > +{ > + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); > + struct v4l2_event event; > + > + memset(&event, 0, sizeof(event)); > + event.type = V4L2_EVENT_BUF_QUEUED; > + event.u.buf_queued.index = vb->index; > + > + v4l2_event_queue(vdev, &event); > +} > + > /** > * __enqueue_in_driver() - enqueue a vb2_buffer in driver for processing > */ > @@ -1234,6 +1247,8 @@ static void __enqueue_in_driver(struct vb2_buffer *vb) > trace_vb2_buf_queue(q, vb); > > call_void_vb_qop(vb, buf_queue, vb); > + > + vb2_buffer_queued_event(vb); > } > > static int __buf_prepare(struct vb2_buffer *vb, const void *pb) > Regards, Hans
On 06/30/17 14:04, Mauro Carvalho Chehab wrote: > Em Fri, 16 Jun 2017 16:39:10 +0900 > Gustavo Padovan <gustavo@padovan.org> escreveu: > >> From: Gustavo Padovan <gustavo.padovan@collabora.com> >> >> Implement the needed pieces to let userspace subscribe for >> V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the >> DQEVENT ioctl. >> >> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> >> --- >> drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- >> drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ >> 2 files changed, 20 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c >> index 5aed7bd..f55b5da 100644 >> --- a/drivers/media/v4l2-core/v4l2-ctrls.c >> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c >> @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); >> int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, >> const struct v4l2_event_subscription *sub) >> { >> - if (sub->type == V4L2_EVENT_CTRL) >> + switch (sub->type) { >> + case V4L2_EVENT_CTRL: >> return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); >> + case V4L2_EVENT_BUF_QUEUED: >> + return v4l2_event_subscribe(fh, sub, 0, NULL); >> + } >> return -EINVAL; >> } >> EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); >> diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c >> index 29aa9d4..00d9c35 100644 >> --- a/drivers/media/v4l2-core/videobuf2-core.c >> +++ b/drivers/media/v4l2-core/videobuf2-core.c >> @@ -25,6 +25,7 @@ >> #include <linux/kthread.h> >> >> #include <media/videobuf2-core.h> >> +#include <media/v4l2-event.h> >> #include <media/v4l2-mc.h> >> >> #include <trace/events/vb2.h> >> @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) >> return ret; >> } >> >> +static void vb2_buffer_queued_event(struct vb2_buffer *vb) >> +{ >> + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); >> + struct v4l2_event event; >> + >> + memset(&event, 0, sizeof(event)); >> + event.type = V4L2_EVENT_BUF_QUEUED; >> + event.u.buf_queued.index = vb->index; >> + >> + v4l2_event_queue(vdev, &event); >> +} >> + > > It doesn't sound right to add a V4L2 event to VB2 core. The hole point > of splitting the core from V4L2 specific stuff is to allow VB2 to be > used by non-V4L2 APIs[1]. Please move this to videobuf2-v4l2. Good point. So this should be a callback to the higher level. One thing I was wondering about: v4l2_event_queue sends the event to all open filehandles of the video node that subscribed to this event. Is that what we want? Or should we use v4l2_event_queue_fh to only send it to the vb2 queue owner? I don't know what is best. I think it is OK to send it to anyone that is interested. If nothing else it will help debugging. Regards, Hans > > [1] The split happened as part of a patchset meant to make the DVB > core to use VB2 and provide DMA APIs to it. Unfortunately, the > developer that worked on this project moved to some other project. > The final patch was not applied yet. I have it on my patchwork > queue. I intend to test and apply it sometime this year. > > > >> /** >> * __enqueue_in_driver() - enqueue a vb2_buffer in driver for processing >> */ >> @@ -1234,6 +1247,8 @@ static void __enqueue_in_driver(struct vb2_buffer *vb) >> trace_vb2_buf_queue(q, vb); >> >> call_void_vb_qop(vb, buf_queue, vb); >> + >> + vb2_buffer_queued_event(vb); >> } >> >> static int __buf_prepare(struct vb2_buffer *vb, const void *pb) > >
2017-07-06 Hans Verkuil <hverkuil@xs4all.nl>: > On 06/30/17 14:04, Mauro Carvalho Chehab wrote: > > Em Fri, 16 Jun 2017 16:39:10 +0900 > > Gustavo Padovan <gustavo@padovan.org> escreveu: > > > >> From: Gustavo Padovan <gustavo.padovan@collabora.com> > >> > >> Implement the needed pieces to let userspace subscribe for > >> V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the > >> DQEVENT ioctl. > >> > >> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com> > >> --- > >> drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- > >> drivers/media/v4l2-core/videobuf2-core.c | 15 +++++++++++++++ > >> 2 files changed, 20 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c > >> index 5aed7bd..f55b5da 100644 > >> --- a/drivers/media/v4l2-core/v4l2-ctrls.c > >> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c > >> @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); > >> int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, > >> const struct v4l2_event_subscription *sub) > >> { > >> - if (sub->type == V4L2_EVENT_CTRL) > >> + switch (sub->type) { > >> + case V4L2_EVENT_CTRL: > >> return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); > >> + case V4L2_EVENT_BUF_QUEUED: > >> + return v4l2_event_subscribe(fh, sub, 0, NULL); > >> + } > >> return -EINVAL; > >> } > >> EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); > >> diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c > >> index 29aa9d4..00d9c35 100644 > >> --- a/drivers/media/v4l2-core/videobuf2-core.c > >> +++ b/drivers/media/v4l2-core/videobuf2-core.c > >> @@ -25,6 +25,7 @@ > >> #include <linux/kthread.h> > >> > >> #include <media/videobuf2-core.h> > >> +#include <media/v4l2-event.h> > >> #include <media/v4l2-mc.h> > >> > >> #include <trace/events/vb2.h> > >> @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) > >> return ret; > >> } > >> > >> +static void vb2_buffer_queued_event(struct vb2_buffer *vb) > >> +{ > >> + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); > >> + struct v4l2_event event; > >> + > >> + memset(&event, 0, sizeof(event)); > >> + event.type = V4L2_EVENT_BUF_QUEUED; > >> + event.u.buf_queued.index = vb->index; > >> + > >> + v4l2_event_queue(vdev, &event); > >> +} > >> + > > > > It doesn't sound right to add a V4L2 event to VB2 core. The hole point > > of splitting the core from V4L2 specific stuff is to allow VB2 to be > > used by non-V4L2 APIs[1]. Please move this to videobuf2-v4l2. > > Good point. So this should be a callback to the higher level. > > One thing I was wondering about: v4l2_event_queue sends the event to all > open filehandles of the video node that subscribed to this event. Is that > what we want? Or should we use v4l2_event_queue_fh to only send it to the > vb2 queue owner? I don't know what is best. I think it is OK to send it > to anyone that is interested. If nothing else it will help debugging. I don't have any preference here, I'll keep it as is - sending events to anyone - if no one objects. Gustavo
diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index 5aed7bd..f55b5da 100644 --- a/drivers/media/v4l2-core/v4l2-ctrls.c +++ b/drivers/media/v4l2-core/v4l2-ctrls.c @@ -3435,8 +3435,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, const struct v4l2_event_subscription *sub) { - if (sub->type == V4L2_EVENT_CTRL) + switch (sub->type) { + case V4L2_EVENT_CTRL: return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); + case V4L2_EVENT_BUF_QUEUED: + return v4l2_event_subscribe(fh, sub, 0, NULL); + } return -EINVAL; } EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 29aa9d4..00d9c35 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c @@ -25,6 +25,7 @@ #include <linux/kthread.h> #include <media/videobuf2-core.h> +#include <media/v4l2-event.h> #include <media/v4l2-mc.h> #include <trace/events/vb2.h> @@ -1221,6 +1222,18 @@ static int __prepare_dmabuf(struct vb2_buffer *vb, const void *pb) return ret; } +static void vb2_buffer_queued_event(struct vb2_buffer *vb) +{ + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); + struct v4l2_event event; + + memset(&event, 0, sizeof(event)); + event.type = V4L2_EVENT_BUF_QUEUED; + event.u.buf_queued.index = vb->index; + + v4l2_event_queue(vdev, &event); +} + /** * __enqueue_in_driver() - enqueue a vb2_buffer in driver for processing */ @@ -1234,6 +1247,8 @@ static void __enqueue_in_driver(struct vb2_buffer *vb) trace_vb2_buf_queue(q, vb); call_void_vb_qop(vb, buf_queue, vb); + + vb2_buffer_queued_event(vb); } static int __buf_prepare(struct vb2_buffer *vb, const void *pb)