Re[6]: [Gc] [PATCH 2/2]: Hexagon atomic ops

Ivan Maidanski ivmai at mail.ru
Tue Nov 29 22:56:53 PST 2011


Hi Linas,

It turned out that ao_t_is_int.h included from hexagon.h required prior generalization (at least to define AO_or and some AO_..._read/write primitives), so I've put "include generalize.h" into hexagon.h.

Please test both "release" and "master" branches on Hexagon again.

Regards.

17 11 2011, 10:27 Ivan Maidanski <ivmai at mail.ru>:
> Hi,
> 
> Good. Thanks.
> 
> 16 11 2011, 21:40 Linas Vepstas <linasvepstas at gmail.com>:
> > Hi,
> >
> > On 16 November 2011 10:33, Ivan Maidanski <ivmai at mail.ru> wrote:
> > > Hi Linas,
> > >
> > > Based on your implementation of CAS for Hexagon, I've created another CAS primitive equivalent to GCC __sync_val_compare_and_swap one (see AO_fetch_compare_and_swap() in https://github.com/ivmai/libatomic_ops/blob/ao_val_cas/src/atomic_ops/sysdeps/gcc/hexagon.h).
> > > But I have no opportunity to test the code.
> > >
> > > Could you please test that code ("ao_val_cas" branch) for Hexagon?
> > >
> > > git clone git://github.com/ivmai/libatomic_ops.git -b ao_val_cas
> > > cd libatomic_ops; autoreconf -vif
> > > ./configure; make check
> >
> > Looks good to me; the last few lines are:
> >
> > PASS: test_stack
> > Performing 1000 reversals of 1000 element lists in 10 threads
> > Testing AO_malloc/AO_free
> > Succeeded
> > PASS: test_malloc
> > ==================
> > All 4 tests passed
> > ==================
> >
> > --linas
> >
> > _______________________________________________
> > Gc mailing list
> > Gc at linux.hpl.hp.com
> > http://www.hpl.hp.com/hosted/linux/mail-archives/gc/
> >
> 



More information about the Gc mailing list