[Intel-gfx] [PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-16 Thread Sean Paul
On Tue, Dec 13, 2016 at 6:08 PM, Daniel Vetter wrote: > The requirements for connector_list locking are a bit tricky: > - We need to be able to jump over zombie conectors (i.e. with refcount > == 0, but not yet removed from the list). If instead we require that > there's no zombies on the lis

[Intel-gfx] [PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-14 Thread Jani Nikula
On Wed, 14 Dec 2016, Daniel Vetter wrote: > On Wed, Dec 14, 2016 at 01:22:15PM +0200, Jani Nikula wrote: >> On Wed, 14 Dec 2016, Daniel Vetter wrote: >> > +/** >> > + * drm_for_each_connector_iter - connector_list iterator macro >> > + * @connector: struct &drm_connector pointer used as cursor >>

[Intel-gfx] [PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-14 Thread Daniel Vetter
On Wed, Dec 14, 2016 at 01:22:15PM +0200, Jani Nikula wrote: > On Wed, 14 Dec 2016, Daniel Vetter wrote: > > +/** > > + * drm_for_each_connector_iter - connector_list iterator macro > > + * @connector: struct &drm_connector pointer used as cursor > > + * @iter: struct &drm_connector_list_iter > >

[Intel-gfx] [PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-14 Thread Jani Nikula
On Wed, 14 Dec 2016, Daniel Vetter wrote: > The requirements for connector_list locking are a bit tricky: > - We need to be able to jump over zombie conectors (i.e. with refcount > == 0, but not yet removed from the list). If instead we require that > there's no zombies on the list then the fi

[PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-14 Thread Chris Wilson
On Wed, Dec 14, 2016 at 12:08:06AM +0100, Daniel Vetter wrote: > The requirements for connector_list locking are a bit tricky: > - We need to be able to jump over zombie conectors (i.e. with refcount > == 0, but not yet removed from the list). If instead we require that > there's no zombies on

[PATCH 05/13] drm: locking&new iterators for connector_list

2016-12-14 Thread Daniel Vetter
The requirements for connector_list locking are a bit tricky: - We need to be able to jump over zombie conectors (i.e. with refcount == 0, but not yet removed from the list). If instead we require that there's no zombies on the list then the final kref_put must happen under the list protectio