[PATCH] ppc64: SMU partitions & device-tree

Benjamin Herrenschmidt benh at kernel.crashing.org
Fri Sep 23 08:43:54 EST 2005


On Thu, 2005-09-22 at 10:29 -0500, Nathan Lynch wrote:

> 
> I think this is abusing devtree_lock, which is meant to protect
> traversal of the device tree and not the contents of nodes.  True,
> this serializes prom_add_property itself, and will prevent corruption
> of the node's property list.  But I think I'd rather the SMU code take
> care of this itself, i.e. use its own mutex to serialize the addition
> of properties to /smu.

It does, but I felt like prom_add_property() should be made a bit safer.
It's definitely an abuse of the devtree_lock, but knowing how rarely
prom_add_property() is to be called, I don't think it's that much of a
problem, what do you think ?

Ben.





More information about the Linuxppc64-dev mailing list