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 <phhargrove@lbl.gov> wrote:

On Thu, Jan 23, 2014 at 1:16 PM, Paul Hargrove <phhargrove@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                          PHHargrove@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
devel@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel