I put it for 1.7.5 just for completeness - I agree that not many people will care, but we should reward your hard work!
Thanks Ralph On Jan 23, 2014, at 2:06 PM, Paul Hargrove <phhargr...@lbl.gov> wrote: > On Thu, Jan 23, 2014 at 1:16 PM, Paul Hargrove <phhargr...@lbl.gov> wrote: > [snip] > I will retest ASAP and report with, I hope, an attachment to fix both > IA64.asm and ia64/atomic.h > [snip] > > Eureka!! > > With the bogus cast removed in both places, I can now run ring_c on > linux/ia64. > The attached patch is against trunk, but applies cleanly to v1.7. > I fact, since the code has been incorrect for a long time it applies cleanly > to v1.6 too. > > FWIW: > The code was broken by r3448, which apparently fixed some warnings but also > added the incorrect narrowing casts to the 64-bit cmpset code. So, IA64 > *did* work prior to April 2010. > > Given the timeline, this can't possibly be a regression in the 1.7 series. > Additionally, with Sylvesrte Ledru having given up on ia64 nobody may care at > all. > So, CMR to 1.7.4 vs .5 seems like a potentially moot point. > > -Paul > > > -- > Paul H. Hargrove phhargr...@lbl.gov > Future Technologies Group > Computer and Data Sciences Department Tel: +1-510-495-2352 > Lawrence Berkeley National Laboratory Fax: +1-510-486-6900 > <ia64-cmpset_64.patch>_______________________________________________ > devel mailing list > de...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/devel
