On Feb 17, 2007, at 5:41 PM, Pawel Jakub Dawidek wrote: > Hi. > > I noticed that when non-64bit variable is given as a second > argument to > atomic_add_64() function, the result is invalid. > > I found few places where such situation occurs. I wonder how this got > unnoticed with ztest, which fails on me within a few seconds (after I > started to use Solaris atomic operations) on assertions. Maybe this > only doesn't work when compiled with gcc? Not sure, but most of the > time > 64bit variables are used properly. > > Anyway, the patch is here: > > http://people.freebsd.org/~pjd/opensolaris/10.patch
thanks for the code diffs Pawel, i filed the following to cover this: 6526874 ARC is using ints instead of uint64s in some places eric
