Re: [PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid

Chris Wilson
2016-06-02T22:40:04Z

On Thu, Jun 02, 2016 at 11:57:02PM +0200, Daniel Vetter wrote:
> drm_encoder_find is an idr lookup. That should be plenty fast,
> especially for modeset code. Usually what's too expensive even for
> modeset code is linear list walks. But Chris just submitted patches to
> convert most of them into simple lookups.

For the idr_find, I'm tempted to replace the mutex with a rwlock. It
helps pathological cases, but in reality there are more crucial
locks around the hw that limit concurrency. ;-)
-Chris

--
Chris Wilson, Intel Open Source Technology Centre