Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-29 Thread Manoj Gudi
There is no equivalent instruction of pause in ARM that hints the following loop is a spin-lock loop; However I just mapped pause into NOP and its not throwing an error (yet) :D Here's the diff: diff --git a/Include/Theron/Detail/Threading/Utils.h b/Include/Theron/Detail/Threading/Utils.h index

Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-29 Thread Philip Balister
On 07/29/2013 09:08 AM, Manoj Gudi wrote: There is no equivalent instruction of pause in ARM that hints the following loop is a spin-lock loop; However I just mapped pause into NOP and its not throwing an error (yet) :D Wny not only insert the asm for x86 and no code for all other archs. This

Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-29 Thread Manoj Gudi
That sounds good. From: manojgudi manoj.p.g...@gmail.com Date: Mon, 29 Jul 2013 17:00:35 +0530 Subject: [PATCH] fixed ASM --- Include/Theron/Detail/Threading/Utils.h |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Include/Theron/Detail/Threading/Utils.h

Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-29 Thread Josh Blum
On 07/29/2013 07:31 AM, Manoj Gudi wrote: That sounds good. I second that. Just ifdef the pause for the fix. For a little extra information here. The thread pools in Theron can have different yield strategies. GRAS happens to use by default the condition variable strategy. So the code in

[Discuss-gnuradio] GRAS build error on armv7

2013-07-26 Thread Manoj Gudi
We've been successful in building dependencies for gnuradio on armv7 platform, however while building GRAS, we got this error: *Scanning dependencies of target gras [ 13%] Building CXX object lib/CMakeFiles/gras.dir/__/Theron/Theron/Receiver.cpp.o [ 13%] Generating GrExtras_Ops.pyc [ 13%]

Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-26 Thread Josh Blum
On 07/26/2013 07:57 AM, Manoj Gudi wrote: We've been successful in building dependencies for gnuradio on armv7 platform, however while building GRAS, we got this error: This line is probably the culprit.

Re: [Discuss-gnuradio] GRAS build error on armv7

2013-07-26 Thread Manoj Gudi
Yes I'll work on it and send you a patch with conditional preprocessors. On Sat, Jul 27, 2013 at 3:06 AM, Ashton Mason a...@ashtonmason.net wrote: Hi guys Yes that line is no doubt the culprit; The 'pause' is intended to help prevent a spinning thread from burning a core that could be used