[openib-general] Fix some suspicious ppc64 code in dapl
James Lentini
jlentini at netapp.com
Mon Jun 5 06:38:43 PDT 2006
> Index: dapl/udapl/linux/dapl_osd.h
> ===================================================================
> --- dapl/udapl/linux/dapl_osd.h (revision 7621)
> +++ dapl/udapl/linux/dapl_osd.h (working copy)
> @@ -238,14 +238,13 @@
> #endif /* __ia64__ */
> #elif defined(__PPC64__)
> __asm__ __volatile__ (
> - EIEIO_ON_SMP
> -"1: lwarx %0,0,%2 # __cmpxchg_u64\n\
> - cmpd 0,%0,%3\n\
> +" lwsync\n\
> +1: lwarx %0,0,%2 # __cmpxchg_u32\n\
> + cmpw 0,%0,%3\n\
> bne- 2f\n\
> stwcx. %4,0,%2\n\
> - bne- 1b"
> - ISYNC_ON_SMP
> - "\n\
> + bne- 1b\n\
> + isync\n\
> 2:"
> : "=&r" (current_value), "=m" (*v)
> : "r" (v), "r" (match_value), "r" (new_value), "m" (*v)
Thank you Anton. Could you replying with a signed off by line? I'll
properly attribute this fix to you in the commit log.
More information about the general
mailing list