From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lb2-smtp-cloud9.xs4all.net ([194.109.24.26]:32878 "EHLO lb2-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965664AbeEXJ21 (ORCPT ); Thu, 24 May 2018 05:28:27 -0400 Subject: Re: [PATCHv13 05/28] media-request: add media_request_object_find To: Sakari Ailus Cc: linux-media@vger.kernel.org, Hans Verkuil References: <20180503145318.128315-1-hverkuil@xs4all.nl> <20180503145318.128315-6-hverkuil@xs4all.nl> <20180504124307.sddriagirmig4yf4@valkosipuli.retiisi.org.uk> From: Hans Verkuil Message-ID: <330a4251-dce2-73a3-34a1-129809fd271d@xs4all.nl> Date: Thu, 24 May 2018 11:28:24 +0200 MIME-Version: 1.0 In-Reply-To: <20180504124307.sddriagirmig4yf4@valkosipuli.retiisi.org.uk> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 04/05/18 14:43, Sakari Ailus wrote: > On Thu, May 03, 2018 at 04:52:55PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Add media_request_object_find to find a request object inside a >> request based on ops and/or priv values. >> >> Objects of the same type (vb2 buffer, control handler) will have >> the same ops value. And objects that refer to the same 'parent' >> object (e.g. the v4l2_ctrl_handler that has the current driver >> state) will have the same priv value. >> >> The caller has to call media_request_object_put() for the returned >> object since this function increments the refcount. >> >> Signed-off-by: Hans Verkuil >> --- >> drivers/media/media-request.c | 25 +++++++++++++++++++++++++ >> include/media/media-request.h | 24 ++++++++++++++++++++++++ >> 2 files changed, 49 insertions(+) >> >> diff --git a/drivers/media/media-request.c b/drivers/media/media-request.c >> index edc1c3af1959..c7e11e816e27 100644 >> --- a/drivers/media/media-request.c >> +++ b/drivers/media/media-request.c >> @@ -322,6 +322,31 @@ static void media_request_object_release(struct kref *kref) >> obj->ops->release(obj); >> } >> >> +struct media_request_object * >> +media_request_object_find(struct media_request *req, >> + const struct media_request_object_ops *ops, >> + void *priv) >> +{ >> + struct media_request_object *obj; >> + struct media_request_object *found = NULL; >> + unsigned long flags; >> + >> + if (WARN_ON(!ops || !priv)) >> + return NULL; >> + >> + spin_lock_irqsave(&req->lock, flags); >> + list_for_each_entry(obj, &req->objects, list) { >> + if (obj->ops == ops && obj->priv == priv) { >> + media_request_object_get(obj); >> + found = obj; >> + break; >> + } >> + } >> + spin_unlock_irqrestore(&req->lock, flags); >> + return found; >> +} >> +EXPORT_SYMBOL_GPL(media_request_object_find); >> + >> void media_request_object_put(struct media_request_object *obj) >> { >> kref_put(&obj->kref, media_request_object_release); >> diff --git a/include/media/media-request.h b/include/media/media-request.h >> index 997e096d7128..5367b4a2f91c 100644 >> --- a/include/media/media-request.h >> +++ b/include/media/media-request.h >> @@ -196,6 +196,22 @@ static inline void media_request_object_get(struct media_request_object *obj) >> */ >> void media_request_object_put(struct media_request_object *obj); >> >> +/** >> + * media_request_object_find - Find an object in a request >> + * >> + * @ops: Find an object with this ops value >> + * @priv: Find an object with this priv value >> + * >> + * Both @ops and @priv must be non-NULL. >> + * >> + * Returns NULL if not found or the object pointer. The caller must > > I'd describe the successful case first. I.e. "Returns the object pointer or > NULL it not found". Done. > > Acked-by: Sakari Ailus Thanks! Hans > >> + * call media_request_object_put() once it finished using the object. >> + */ >> +struct media_request_object * >> +media_request_object_find(struct media_request *req, >> + const struct media_request_object_ops *ops, >> + void *priv); >> + >> /** >> * media_request_object_init - Initialise a media request object >> * >> @@ -241,6 +257,14 @@ static inline void media_request_object_put(struct media_request_object *obj) >> { >> } >> >> +static inline struct media_request_object * >> +media_request_object_find(struct media_request *req, >> + const struct media_request_object_ops *ops, >> + void *priv) >> +{ >> + return NULL; >> +} >> + >> static inline void media_request_object_init(struct media_request_object *obj) >> { >> obj->ops = NULL; >