[Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

Keith Packard keithp at keithp.com
Tue Jul 26 08:23:13 PDT 2011


On Tue, 26 Jul 2011 09:24:39 +0200, Daniel Vetter <daniel at ffwll.ch> wrote:
> Two things I've noticed:

> - Why not dev->mode_config.mutex?

You're right, of course. I noticed that just after posting that version
and updated it; the updated version is on my drm-intel-fixes branch
already (having been reviewed by Jesse).

> - And a nitpick: Why the dev_priv->dev indirection, when dev is
> already lying around?

All nicely cleaned up by using &mode_config->mutex instead :-)

Thanks for looking it over!

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110726/1b607231/attachment-0001.pgp>


More information about the dri-devel mailing list