On Tue, 2009-12-22 at 08:45 -0600, Nathan Fontenot wrote: > The recently introduced cpu_hotplug_driver_lock used to serialize > cpu hotplug operations, namely for the pseries platform, causes a build > issue for other platforms. The base cpu hotplug code attempts > to take this lock, but it may not be needed for all platforms. This patch > moves the lock/unlock routines to be part of the ppc_md structure > so that platforms needing the lock can take it. This also makes the > previous cpu_hotplug_driver_lock, defined in pseries code, pseries specific. > > The past failure without this patch was seen when building pmac and may > be present in other platform builds. The error is included below for > reference. > > drivers/built-in.o: In function `.store_online': > cpu.c:(.ref.text+0xf5c): undefined reference to `.cpu_hotplug_driver_lock' > cpu.c:(.ref.text+0xfc8): undefined reference to `.cpu_hotplug_driver_unlock' > make: *** [.tmp_vmlinux1] Error 1
Why does the pmac code /not/ need a lock? And would it be harmless if it was locked too? If so, you could just make the mutex available to all powerpc code, and rename it, and then we wouldn't need all this jiggery pokery just to take & release a lock. cheers
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
