Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 31 Jan 2001 20:18:04 -0500
From:      "Matthew Emmerton" <matt@gsicomp.on.ca>
To:        "John Baldwin" <jhb@osd.bsdi.com>
Cc:        <freebsd-hackers@FreeBSD.org>, "John Baldwin" <jhb@FreeBSD.org>
Subject:   Re: Atomic bit operations
Message-ID:  <010501c08bec$d48c96a0$1200a8c0@gsicomp.on.ca>
References:  <XFMail.010131160650.jhb@osd.bsdi.com>

next in thread | previous in thread | raw e-mail | index | archive | help
> On 01-Feb-01 Matthew Emmerton wrote:
> >> On 31-Jan-01 Matthew Emmerton wrote:
> >> > Hi all,
> >> >
> >> > I've taken a look around for an implementation of atomic bit
operations
> > in
> >> > FreeBSD (similar to Linux' asm/bitopt.h, which include clear_bit()
and
> >> > test_and_set_bit()) but haven't found any.  The only thing I've found
> > are
> >> > the atomic clear/set/add/sub routines in machine/atomic.h.
> >> >
> >> > Do we have an implementation of atomic bit operations, and if we
don't,
> >> > would we like some?
> >>
> >> Erm, atomic_set() sets's bits, and atomic_clear() clear's bits.
Anything
> > else
> >> you might need can be done with atomic_cmpset() anyways.
> >
> > Where are these functions implemented?  /usr/include/machine/atomic.h
just
> > has atomic_set_XXX and clear_XXX primitives, which work on
char/short/longs,
> > but not individual bits.
>
> 1) You need to look in -current.

I only recently moved to 4-stable, so I'm not going to jump to -current
quite yet :)

> 2) atomic_set_int(&my_int, 4); sets bit _2_ in the integer variable
my_int.
> Make sense?  You can't address individual bits on a machine. :-P
>
> > I presume that I could wrap the char operations with something that
takes
> > 0x01 and bit-shifts it appropriately so that atomic_set/clear could be
used.
>
> Hmm, so you mean the API is atomic_set(&foo, x) implies foo |= 1 << x?
> That means you can't set more than 1 bit atomically, which is a bit
> limiting.

Yes, but it's the API used by some code I recently inherited.  (Recall that
this API is Linux's asm/bitops.h)  I realize that the existing
atomic_xxx_xxx functions are much more flexible than bit-based ones, but
wouldn't it make sense to have the full complement of functions available?

> > However, certain other primitives are missing, such as an atomic
> > test-and-set operation.  Under the current scheme this would have to be
done
> > by two independent operations, which is not useful when atomicitiy is
> > required.
>
> do {
>    x = my_int;
> while (atomic_cmpset_int(&my_int, x, x | foo) == 0);
> if (x & foo)
>    foo_was_already_set;
> else
>    foo_was_not_already_set;

I had based my assumptions on what I saw in 4-stable, which doesn't have
atomic_cmpset.  I'll have to take a peek at 4-current or wait until some
features trickle down to 4-stable.

Thanks,

--
Matt Emmerton



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?010501c08bec$d48c96a0$1200a8c0>