| Message ID | f34de3fbb419ded00e16fd9c0538313e01cd0b2c.1459950922.git.mchehab@osg.samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Mauro, On Wed, Apr 06, 2016 at 06:55:25AM -0700, Mauro Carvalho Chehab wrote: > Those callbacks are called with the media_device.graph_mutex hold. > > Add a note about that, as the code called by those notifiers should > not be touching in the mutex. > > Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com> > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> > --- > include/media/media-device.h | 3 ++- > include/media/media-entity.h | 3 +++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/include/media/media-device.h b/include/media/media-device.h > index b21ef244ad3e..44563ec17d45 100644 > --- a/include/media/media-device.h > +++ b/include/media/media-device.h > @@ -311,7 +311,8 @@ struct media_entity_notify { > * @enable_source: Enable Source Handler function pointer > * @disable_source: Disable Source Handler function pointer > * > - * @link_notify: Link state change notification callback > + * @link_notify: Link state change notification callback. This callback is > + * Called with the graph_mutex hold. s/Called/called/ s/hold/held/ The indentation could be better, too. > * > * This structure represents an abstract high-level media device. It allows easy > * access to entities and provides basic media device-level support. The > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > index 6dc9e4e8cbd4..0b16ebe36db7 100644 > --- a/include/media/media-entity.h > +++ b/include/media/media-entity.h > @@ -179,6 +179,9 @@ struct media_pad { > * @link_validate: Return whether a link is valid from the entity point of > * view. The media_entity_pipeline_start() function > * validates all links by calling this operation. Optional. > + * > + * Note: Those ioctls should not touch the struct media_device.@graph_mutex > + * field, as they're called with it already hold. How about simply: "these callbacks are called with struct media_device.@graph_mutex mutex held" ? They're not IOCTLs, and the above is more simple, too. > */ > struct media_entity_operations { > int (*link_setup)(struct media_entity *entity,
On 04/06/2016 03:55 PM, Mauro Carvalho Chehab wrote: > Those callbacks are called with the media_device.graph_mutex hold. > > Add a note about that, as the code called by those notifiers should > not be touching in the mutex. > > Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com> > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Acked-by: Hans Verkuil <hans.verkuil@cisco.com> Regards, Hans > --- > include/media/media-device.h | 3 ++- > include/media/media-entity.h | 3 +++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/include/media/media-device.h b/include/media/media-device.h > index b21ef244ad3e..44563ec17d45 100644 > --- a/include/media/media-device.h > +++ b/include/media/media-device.h > @@ -311,7 +311,8 @@ struct media_entity_notify { > * @enable_source: Enable Source Handler function pointer > * @disable_source: Disable Source Handler function pointer > * > - * @link_notify: Link state change notification callback > + * @link_notify: Link state change notification callback. This callback is > + * Called with the graph_mutex hold. > * > * This structure represents an abstract high-level media device. It allows easy > * access to entities and provides basic media device-level support. The > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > index 6dc9e4e8cbd4..0b16ebe36db7 100644 > --- a/include/media/media-entity.h > +++ b/include/media/media-entity.h > @@ -179,6 +179,9 @@ struct media_pad { > * @link_validate: Return whether a link is valid from the entity point of > * view. The media_entity_pipeline_start() function > * validates all links by calling this operation. Optional. > + * > + * Note: Those ioctls should not touch the struct media_device.@graph_mutex > + * field, as they're called with it already hold. > */ > struct media_entity_operations { > int (*link_setup)(struct media_entity *entity, > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/media/media-device.h b/include/media/media-device.h index b21ef244ad3e..44563ec17d45 100644 --- a/include/media/media-device.h +++ b/include/media/media-device.h @@ -311,7 +311,8 @@ struct media_entity_notify { * @enable_source: Enable Source Handler function pointer * @disable_source: Disable Source Handler function pointer * - * @link_notify: Link state change notification callback + * @link_notify: Link state change notification callback. This callback is + * Called with the graph_mutex hold. * * This structure represents an abstract high-level media device. It allows easy * access to entities and provides basic media device-level support. The diff --git a/include/media/media-entity.h b/include/media/media-entity.h index 6dc9e4e8cbd4..0b16ebe36db7 100644 --- a/include/media/media-entity.h +++ b/include/media/media-entity.h @@ -179,6 +179,9 @@ struct media_pad { * @link_validate: Return whether a link is valid from the entity point of * view. The media_entity_pipeline_start() function * validates all links by calling this operation. Optional. + * + * Note: Those ioctls should not touch the struct media_device.@graph_mutex + * field, as they're called with it already hold. */ struct media_entity_operations { int (*link_setup)(struct media_entity *entity,