Thanks for reporting this Matthew. Fixed in r23576 (https://svn.open-mpi.org/trac/ompi/changeset/23576)

Regards
--Nysal

On Fri, Aug 6, 2010 at 10:38 PM, Matthew Clark <linux.matthew@gmail.com> wrote:
I was looking in my copy of openmpi-1.4.1 opal/asm/base/POWERPC32.asm
and saw the following:

START_FUNC(opal_sys_timer_get_cycles)
       LSYM(15)
       mftbu r0
       mftb r11
       mftbu r2
       cmpw cr7,r2,r0
       bne+ cr7,REFLSYM(14)
       li r4,0
       li r9,0
       or r3,r2,r9
       or r4,r4,r11
       blr
END_FUNC(opal_sys_timer_get_cycles)

I'll readily admit at my lack of ppc assembly smartness, but shouldn't
the loop back at bne+ be to REFLSYM(15) instead of (14)?

Matt
_______________________________________________
users mailing list
users@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/users