[pulseaudio-discuss] [PATCH] flist: Avoid the ABA problem

Tanu Kaskinen tanuk at iki.fi
Tue Mar 6 23:44:30 PST 2012


On Sun, 2012-03-04 at 22:56 +0200, Jyri Sarha wrote:
> On Sun,  4 Mar 2012 06:07:37 +0100, David Henningsson
> <david.henningsson at canonical.com> wrote:
> > Our flist implementation suffers from the ABA problem
> > (see http://en.wikipedia.org/wiki/ABA_problem), causing PulseAudio
> > to crash very rarely, usually inside memblock operations.
> > By turning stored pointers into stored table indices, we have some
> > extra bits that we can use to store tag bits, which is a known
> > workaround for the ABA problem.
> > 
> > Buglink: https://bugs.launchpad.net/bugs/924416
> > Signed-off-by: David Henningsson <david.henningsson at canonical.com>
> <code removed>
> 
> After reviewing the patch I have to admit it looks cleaner than my
> version. As far as I can see it should make the ABA problem highly
> improbable. At least the patch provides a big improvement over the
> current implementation.

I've pushed David's patch now. Thanks to both of you!

-- 
Tanu



More information about the pulseaudio-discuss mailing list