Please pull powerpc.git merge branch

Paul Mackerras paulus at samba.org
Mon Dec 31 12:40:40 EST 2007


Geoff Levand writes:

> This broke Cell builds.
> 
> arch/powerpc/platforms/cell/spufs/sched.c:200: multiple definition of `.notify_spus_active'
> arch/powerpc/platforms/cell/spu_syscalls.c:149: first defined here
> 
> 
> commit aed3a8c9bb1a8623a618232087c5ff62718e3b9a
> Author: Bob Nelson <rrnelson at linux.vnet.ibm.com>
> Date:   Sat Dec 15 01:27:30 2007 +1100
> 
>     [POWERPC] Oprofile: Remove dependency on spufs module
> 
> 
> Looking at the code, it seems just when CONFIG_SPU_FS=y, not when =m.

Indeed.  Clearly, several people have failed to review this patch
properly. :(

It looks to me that the definition of notify_spus_active under #ifndef
MODULE in spufs/sched.c is bogus and should just be removed
completely.  Also, I don't see any need for do_notify_spus_active to
be exported.

I propose the patch below.  Arnd, Jeremy, Bob, any opinions?

Paul.
---

diff --git a/arch/powerpc/platforms/cell/spufs/sched.c b/arch/powerpc/platforms/cell/spufs/sched.c
index 6e2a45e..9ad53e6 100644
--- a/arch/powerpc/platforms/cell/spufs/sched.c
+++ b/arch/powerpc/platforms/cell/spufs/sched.c
@@ -192,15 +192,6 @@ void do_notify_spus_active(void)
 		mutex_unlock(&cbe_spu_info[node].list_mutex);
 	}
 }
-EXPORT_SYMBOL_GPL(do_notify_spus_active);
-
-#ifndef MODULE
-void notify_spus_active(void)
-{
-	do_notify_spus_active();
-}
-EXPORT_SYMBOL_GPL(notify_spus_active);
-#endif
 
 /**
  * spu_bind_context - bind spu context to physical spu



More information about the Linuxppc-dev mailing list