[Nouveau] [PATCH] drm: serialize access to debugs_nodes.list

Marcin Slusarz marcin.slusarz at gmail.com
Mon Oct 31 10:00:00 PDT 2011


On Mon, Oct 31, 2011 at 09:06:57AM +0100, Daniel Vetter wrote:
> This is just ugly, you're adding a mutex to every drm_info_node, but only
> use the one embedded into the minor. On a quick grep we're only ever using
> the list in there, so I suggest to
> - replace minor->debugfs_node.list with minor->debugfs_list and kill
>   ->debugfs_node
> - add the mutex as minor->debugfs_lock
> That way it's clear what's going on.

Yes, you are right. I don't know what the heck I was thinking.

> Also, you've forgotten to add the locking to i915/i915_debugfs.c

Yeah. Will fix it too.

Marcin


More information about the Nouveau mailing list