Review Request: Add spinlocks lock type, based on GCC intrisincs

[Michael Pyne]

On Monday, August 27, 2012 09:17:43 Thiago Macieira wrote:
> On domingo, 26 de agosto de 2012 19.09.15, Vadim Zhukov wrote:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://git.reviewboard.kde.org/r/106224/
> > -----------------------------------------------------------
> > 
> > Review request for kdelibs and Michael Pyne.
> > 
> > 
> > Description
> > -------
> > 
> > Add simple spin locking mechanism.
> > Written by Michael Pyne as from
> > https://bugs.kde.org/attachment.cgi?id=73282 , with some tweaking by me.
> > 
> > 
> > This addresses bug 305023.
> > 
> >     http://bugs.kde.org/show_bug.cgi?id=305023
> 
> Please use the Qt atomic types. Until GCC 4.7, they generate better code.

I agree, the reason it wasn't that way initially is mentioned in the 
discussion on the bug (but basically because you can't simply put 
QBasicAtomicInt in the union used to store the different lock types that are 
possible).

I have an idea of what to do to fix that but haven't had time to code it up.

Regards,
 - Michael Pyne
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120827/a948c2fe/attachment.sig>