Hi Mauro,
On Tue, Apr 10, 2018 at 08:04:09AM -0300, Mauro Carvalho Chehab wrote:
> Em Mon, 9 Apr 2018 16:20:03 +0200
> Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
>
> > From: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> >
> > Add media_request_find() to find a request based on the file
> > descriptor.
> >
> > The caller has to call media_request_put() for the returned
> > request since this function increments the refcount.
> >
> > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> > ---
> > drivers/media/media-request.c | 47 +++++++++++++++++++++++++++++++++++++++++++
> > include/media/media-request.h | 10 +++++++++
> > 2 files changed, 57 insertions(+)
> >
> > diff --git a/drivers/media/media-request.c b/drivers/media/media-request.c
> > index 27739ff7cb09..02b620c81de5 100644
> > --- a/drivers/media/media-request.c
> > +++ b/drivers/media/media-request.c
> > @@ -207,6 +207,53 @@ static const struct file_operations request_fops = {
> > .release = media_request_close,
> > };
> >
> > +/**
> > + * media_request_find - Find a request based on the file descriptor
> > + * @mdev: The media device
> > + * @request: The request file handle
>
> request_id.
I believe you meant "request_fd".
>
> > + *
> > + * Find and return the request associated with the given file descriptor, or
> > + * an error if no such request exists.
> > + *
> > + * When the function returns a request it increases its reference count. The
> > + * caller is responsible for releasing the reference by calling
> > + * media_request_put() on the request.
>
> IMHO, this can only be called with mutex held. Please add such note
> here.
Why?
The kernel already handles this for file descriptors, so if we get a struct
file reference here, we do already have a reference to the media request
--- struct file always does, this reference is only released when the file
is released.
>
> > + */
> > +struct media_request *
> > +media_request_find(struct media_device *mdev, int request_fd)
> > +{
> > + struct file *filp;
> > + struct media_request *req;
> > +
> > + if (!mdev || !mdev->ops || !mdev->ops->req_queue)
> > + return ERR_PTR(-EPERM);
> > +
> > + filp = fget(request_fd);
> > + if (!filp)
> > + return ERR_PTR(-ENOENT);
> > +
> > + if (filp->f_op != &request_fops)
> > + goto err_fput;
> > + req = filp->private_data;
> > + media_request_get(req);
> > +
> > + if (req->mdev != mdev)
> > + goto err_kref_put;
> > +
> > + fput(filp);
> > +
> > + return req;
> > +
> > +err_kref_put:
> > + media_request_put(req);
> > +
> > +err_fput:
> > + fput(filp);
> > +
> > + return ERR_PTR(-ENOENT);
> > +}
> > +EXPORT_SYMBOL_GPL(media_request_find);
> > +
> > int media_request_alloc(struct media_device *mdev,
> > struct media_request_alloc *alloc)
> > {
> > diff --git a/include/media/media-request.h b/include/media/media-request.h
> > index 082c3cae04ac..033697d493cd 100644
> > --- a/include/media/media-request.h
> > +++ b/include/media/media-request.h
> > @@ -59,6 +59,9 @@ static inline void media_request_get(struct media_request *req)
> > void media_request_put(struct media_request *req);
> > void media_request_cancel(struct media_request *req);
> >
> > +struct media_request *
> > +media_request_find(struct media_device *mdev, int request_fd);
> > +
> > int media_request_alloc(struct media_device *mdev,
> > struct media_request_alloc *alloc);
> > #else
> > @@ -74,6 +77,12 @@ static inline void media_request_cancel(struct media_request *req)
> > {
> > }
> >
> > +static inline struct media_request *
> > +media_request_find(struct media_device *mdev, int request_fd)
> > +{
> > + return ERR_PTR(-ENOENT);
> > +}
> > +
> > #endif
> >
> > struct media_request_object_ops {
> > @@ -173,6 +182,7 @@ static inline void media_request_object_unbind(struct media_request_object *obj)
> > static inline void media_request_object_complete(struct media_request_object *obj)
> > {
> > }
> > +
> > #endif
> >
> > #endif
>
>
>
> Thanks,
> Mauro
--
Sakari Ailus
e-mail: sakari.ailus@xxxxxx
