Csound Csound-dev Csound-tekno Search About

[Cs-dev] Spinlocks

Date2012-10-02 23:56
FromMichael Gogins
Subject[Cs-dev] Spinlocks
I've committed some changes to remove macros regarding spinlocks in
the Csound 5 branch. Please let me know if there are problems
compiling or running.

These changes enable the spinlocks used with OpenMP to run without
"hanging" Csound, which is what happened before.

Not using macros means that if spinlocks are not used, empty functions
will be called in some places. I believe the compiler will remove
these as the return values are not used.

Regards,
Mike

-- 
Michael Gogins
Irreducible Productions
http://www.michael-gogins.com
Michael dot Gogins at gmail dot com

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-05 15:04
FromSteven Yi
SubjectRe: [Cs-dev] Spinlocks
Hi Michael,

I had to re-enable "--includeall" for the SWIG wrapper environment in
SConstruct, as the Java generated wrapper was no longer compatible
with the previous API (committed to GIT).  Did you intend for that
change?

Thanks!
steven

On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
 wrote:
> I've committed some changes to remove macros regarding spinlocks in
> the Csound 5 branch. Please let me know if there are problems
> compiling or running.
>
> These changes enable the spinlocks used with OpenMP to run without
> "hanging" Csound, which is what happened before.
>
> Not using macros means that if spinlocks are not used, empty functions
> will be called in some places. I believe the compiler will remove
> these as the return values are not used.
>
> Regards,
> Mike
>
> --
> Michael Gogins
> Irreducible Productions
> http://www.michael-gogins.com
> Michael dot Gogins at gmail dot com
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-05 15:16
FromMichael Gogins
SubjectRe: [Cs-dev] Spinlocks
Yes, I had to do that to enable building on Linux. What platform are
you talking about? If it's OS X, can you make your change only for OS
X?

On Fri, Oct 5, 2012 at 10:04 AM, Steven Yi  wrote:
> Hi Michael,
>
> I had to re-enable "--includeall" for the SWIG wrapper environment in
> SConstruct, as the Java generated wrapper was no longer compatible
> with the previous API (committed to GIT).  Did you intend for that
> change?
>
> Thanks!
> steven
>
> On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
>  wrote:
>> I've committed some changes to remove macros regarding spinlocks in
>> the Csound 5 branch. Please let me know if there are problems
>> compiling or running.
>>
>> These changes enable the spinlocks used with OpenMP to run without
>> "hanging" Csound, which is what happened before.
>>
>> Not using macros means that if spinlocks are not used, empty functions
>> will be called in some places. I believe the compiler will remove
>> these as the return values are not used.
>>
>> Regards,
>> Mike
>>
>> --
>> Michael Gogins
>> Irreducible Productions
>> http://www.michael-gogins.com
>> Michael dot Gogins at gmail dot com
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel



-- 
Michael Gogins
Irreducible Productions
http://www.michael-gogins.com
Michael dot Gogins at gmail dot com

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-05 15:24
FromSteven Yi
SubjectRe: [Cs-dev] Spinlocks
I can, but doesn't changing --includeall change the generated API?
It's worked for a long time with it, what errors are you getting on
Linux?

On Fri, Oct 5, 2012 at 3:16 PM, Michael Gogins  wrote:
> Yes, I had to do that to enable building on Linux. What platform are
> you talking about? If it's OS X, can you make your change only for OS
> X?
>
> On Fri, Oct 5, 2012 at 10:04 AM, Steven Yi  wrote:
>> Hi Michael,
>>
>> I had to re-enable "--includeall" for the SWIG wrapper environment in
>> SConstruct, as the Java generated wrapper was no longer compatible
>> with the previous API (committed to GIT).  Did you intend for that
>> change?
>>
>> Thanks!
>> steven
>>
>> On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
>>  wrote:
>>> I've committed some changes to remove macros regarding spinlocks in
>>> the Csound 5 branch. Please let me know if there are problems
>>> compiling or running.
>>>
>>> These changes enable the spinlocks used with OpenMP to run without
>>> "hanging" Csound, which is what happened before.
>>>
>>> Not using macros means that if spinlocks are not used, empty functions
>>> will be called in some places. I believe the compiler will remove
>>> these as the return values are not used.
>>>
>>> Regards,
>>> Mike
>>>
>>> --
>>> Michael Gogins
>>> Irreducible Productions
>>> http://www.michael-gogins.com
>>> Michael dot Gogins at gmail dot com
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
>
> --
> Michael Gogins
> Irreducible Productions
> http://www.michael-gogins.com
> Michael dot Gogins at gmail dot com
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-05 15:33
FromSteven Yi
SubjectRe: [Cs-dev] Spinlocks
Note, I just pulled from GIT and tested with the re-added --includeall
and it built fine here.  Using Ubuntu 12.04.

On Fri, Oct 5, 2012 at 3:16 PM, Michael Gogins  wrote:
> Yes, I had to do that to enable building on Linux. What platform are
> you talking about? If it's OS X, can you make your change only for OS
> X?
>
> On Fri, Oct 5, 2012 at 10:04 AM, Steven Yi  wrote:
>> Hi Michael,
>>
>> I had to re-enable "--includeall" for the SWIG wrapper environment in
>> SConstruct, as the Java generated wrapper was no longer compatible
>> with the previous API (committed to GIT).  Did you intend for that
>> change?
>>
>> Thanks!
>> steven
>>
>> On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
>>  wrote:
>>> I've committed some changes to remove macros regarding spinlocks in
>>> the Csound 5 branch. Please let me know if there are problems
>>> compiling or running.
>>>
>>> These changes enable the spinlocks used with OpenMP to run without
>>> "hanging" Csound, which is what happened before.
>>>
>>> Not using macros means that if spinlocks are not used, empty functions
>>> will be called in some places. I believe the compiler will remove
>>> these as the return values are not used.
>>>
>>> Regards,
>>> Mike
>>>
>>> --
>>> Michael Gogins
>>> Irreducible Productions
>>> http://www.michael-gogins.com
>>> Michael dot Gogins at gmail dot com
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
>
> --
> Michael Gogins
> Irreducible Productions
> http://www.michael-gogins.com
> Michael dot Gogins at gmail dot com
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-05 16:17
FromMichael Gogins
SubjectRe: [Cs-dev] Spinlocks
Leave it as you made it, I will re-test myself.

Thanks,
Mike

On Fri, Oct 5, 2012 at 10:33 AM, Steven Yi  wrote:
> Note, I just pulled from GIT and tested with the re-added --includeall
> and it built fine here.  Using Ubuntu 12.04.
>
> On Fri, Oct 5, 2012 at 3:16 PM, Michael Gogins  wrote:
>> Yes, I had to do that to enable building on Linux. What platform are
>> you talking about? If it's OS X, can you make your change only for OS
>> X?
>>
>> On Fri, Oct 5, 2012 at 10:04 AM, Steven Yi  wrote:
>>> Hi Michael,
>>>
>>> I had to re-enable "--includeall" for the SWIG wrapper environment in
>>> SConstruct, as the Java generated wrapper was no longer compatible
>>> with the previous API (committed to GIT).  Did you intend for that
>>> change?
>>>
>>> Thanks!
>>> steven
>>>
>>> On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
>>>  wrote:
>>>> I've committed some changes to remove macros regarding spinlocks in
>>>> the Csound 5 branch. Please let me know if there are problems
>>>> compiling or running.
>>>>
>>>> These changes enable the spinlocks used with OpenMP to run without
>>>> "hanging" Csound, which is what happened before.
>>>>
>>>> Not using macros means that if spinlocks are not used, empty functions
>>>> will be called in some places. I believe the compiler will remove
>>>> these as the return values are not used.
>>>>
>>>> Regards,
>>>> Mike
>>>>
>>>> --
>>>> Michael Gogins
>>>> Irreducible Productions
>>>> http://www.michael-gogins.com
>>>> Michael dot Gogins at gmail dot com
>>>>
>>>> ------------------------------------------------------------------------------
>>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>>> Deploy New Relic app performance management and know exactly
>>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>>> http://p.sf.net/sfu/newrelic-dev2dev
>>>> _______________________________________________
>>>> Csound-devel mailing list
>>>> Csound-devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>>
>>
>> --
>> Michael Gogins
>> Irreducible Productions
>> http://www.michael-gogins.com
>> Michael dot Gogins at gmail dot com
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel



-- 
Michael Gogins
Irreducible Productions
http://www.michael-gogins.com
Michael dot Gogins at gmail dot com

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-10 07:30
Fromjpff@cs.bath.ac.uk
SubjectRe: [Cs-dev] Spinlocks
Not read all the mail about this but whatever changes were made sure give
a lot of warnings!


> Leave it as you made it, I will re-test myself.
>
> Thanks,
> Mike
>
> On Fri, Oct 5, 2012 at 10:33 AM, Steven Yi  wrote:
>> Note, I just pulled from GIT and tested with the re-added --includeall
>> and it built fine here.  Using Ubuntu 12.04.
>>
>> On Fri, Oct 5, 2012 at 3:16 PM, Michael Gogins
>>  wrote:
>>> Yes, I had to do that to enable building on Linux. What platform are
>>> you talking about? If it's OS X, can you make your change only for OS
>>> X?
>>>
>>> On Fri, Oct 5, 2012 at 10:04 AM, Steven Yi  wrote:
>>>> Hi Michael,
>>>>
>>>> I had to re-enable "--includeall" for the SWIG wrapper environment in
>>>> SConstruct, as the Java generated wrapper was no longer compatible
>>>> with the previous API (committed to GIT).  Did you intend for that
>>>> change?
>>>>
>>>> Thanks!
>>>> steven
>>>>
>>>> On Tue, Oct 2, 2012 at 11:56 PM, Michael Gogins
>>>>  wrote:
>>>>> I've committed some changes to remove macros regarding spinlocks in
>>>>> the Csound 5 branch. Please let me know if there are problems
>>>>> compiling or running.
>>>>>
>>>>> These changes enable the spinlocks used with OpenMP to run without
>>>>> "hanging" Csound, which is what happened before.
>>>>>
>>>>> Not using macros means that if spinlocks are not used, empty
>>>>> functions
>>>>> will be called in some places. I believe the compiler will remove
>>>>> these as the return values are not used.
>>>>>
>>>>> Regards,
>>>>> Mike
>>>>>
>>>>> --
>>>>> Michael Gogins
>>>>> Irreducible Productions
>>>>> http://www.michael-gogins.com
>>>>> Michael dot Gogins at gmail dot com
>>>>>
>>>>> ------------------------------------------------------------------------------
>>>>> Don't let slow site performance ruin your business. Deploy New Relic
>>>>> APM
>>>>> Deploy New Relic app performance management and know exactly
>>>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>>>> http://p.sf.net/sfu/newrelic-dev2dev
>>>>> _______________________________________________
>>>>> Csound-devel mailing list
>>>>> Csound-devel@lists.sourceforge.net
>>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>>
>>>> ------------------------------------------------------------------------------
>>>> Don't let slow site performance ruin your business. Deploy New Relic
>>>> APM
>>>> Deploy New Relic app performance management and know exactly
>>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>>> http://p.sf.net/sfu/newrelic-dev2dev
>>>> _______________________________________________
>>>> Csound-devel mailing list
>>>> Csound-devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>>
>>>
>>>
>>> --
>>> Michael Gogins
>>> Irreducible Productions
>>> http://www.michael-gogins.com
>>> Michael dot Gogins at gmail dot com
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic
>>> APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Csound-devel mailing list
>>> Csound-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Csound-devel mailing list
>> Csound-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
>
> --
> Michael Gogins
> Irreducible Productions
> http://www.michael-gogins.com
> Michael dot Gogins at gmail dot com
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel
>
>
>



------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-12 09:43
Fromjpff@cs.bath.ac.uk
SubjectRe: [Cs-dev] Spinlocks
> I've committed some changes to remove macros regarding spinlocks in
> the Csound 5 branch. Please let me know if there are problems
> compiling or running.
>
> These changes enable the spinlocks used with OpenMP to run without
> "hanging" Csound, which is what happened before.
>

Why?

> Not using macros means that if spinlocks are not used, empty functions
> will be called in some places. I believe the compiler will remove
> these as the return values are not used.

How can it know there are not side effects?



------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net

Date2012-10-12 14:58
FromMichael Gogins
SubjectRe: [Cs-dev] Spinlocks
On my first build for OpenMP, the macros were empty (locked nothing).
When I enabled the macros, they produced a deadlock. Because the
macros were scattered, hard to read, and impossible to debug, I
removed them. I observed no side effects. If any problems do show up,
they will be much easier to debug with the functions than with the
previous macros.

This again shows why people who write industrial strength code
strongly advise against using macros.

Regards,
Mike

On Fri, Oct 12, 2012 at 4:43 AM,   wrote:
>> I've committed some changes to remove macros regarding spinlocks in
>> the Csound 5 branch. Please let me know if there are problems
>> compiling or running.
>>
>> These changes enable the spinlocks used with OpenMP to run without
>> "hanging" Csound, which is what happened before.
>>
>
> Why?
>
>> Not using macros means that if spinlocks are not used, empty functions
>> will be called in some places. I believe the compiler will remove
>> these as the return values are not used.
>
> How can it know there are not side effects?
>
>
>
> ------------------------------------------------------------------------------
> Don't let slow site performance ruin your business. Deploy New Relic APM
> Deploy New Relic app performance management and know exactly
> what is happening inside your Ruby, Python, PHP, Java, and .NET app
> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
> http://p.sf.net/sfu/newrelic-dev2dev
> _______________________________________________
> Csound-devel mailing list
> Csound-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/csound-devel



-- 
Michael Gogins
Irreducible Productions
http://www.michael-gogins.com
Michael dot Gogins at gmail dot com

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Csound-devel mailing list
Csound-devel@lists.sourceforge.net