Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Sylvain Munaut
Hi, To me it look like all it's missing is a basic_block::delete_head_blocking with a timeout. Then the pdu_to_tagged_stream_impl::calculate_output_stream_length could just use that with a reasonably small timeout (like 100ms or so) so that it waits on a message but if none comes, it still

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Johnathan Corgan
On 01/15/2015 12:59 AM, Sylvain Munaut wrote: To me it look like all it's missing is a basic_block::delete_head_blocking with a timeout. Then the pdu_to_tagged_stream_impl::calculate_output_stream_length could just use that with a reasonably small timeout (like 100ms or so) so that it waits

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Sylvain Munaut
Hi, Yes, this looks like a pretty big issue to me too. 100% CPU spinning is just not acceptable, ever. To me it look like all it's missing is a basic_block::delete_head_blocking with a timeout. Then the pdu_to_tagged_stream_impl::calculate_output_stream_length could just use that with a

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Marcus Müller
Stupid question: at least in the old version of the code, the current message was popped from the queue using delete_head_nowait(). Why don't we just delete_head_blocking() and -- block? Greetings, Marcus On 01/15/2015 01:59 PM, Sylvain Munaut wrote: Just a warning: pdu_to_tagged_stream saw

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Sylvain Munaut
Just a warning: pdu_to_tagged_stream saw significant change in the middle of last december; so make sure you're all talking about the same version :) Just tried with the last version and the issue is definitely there. I'm not entirely sure what motivated the fact of making it a TSB source,

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Sylvain Munaut
On Thu, Jan 15, 2015 at 2:01 PM, Marcus Müller marcus.muel...@ettus.com wrote: Stupid question: at least in the old version of the code, the current message was popped from the queue using delete_head_nowait(). Why don't we just delete_head_blocking() and -- block? I think this would prevent

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Martin Braun
Some background: It was made a TSB block because TSB source blocks weren't possible until recently, and that's an example use-case. Doesn't solve the spinning issue. On 01/15/2015 02:26 PM, Sylvain Munaut wrote: On Thu, Jan 15, 2015 at 2:01 PM, Marcus Müller marcus.muel...@ettus.com wrote:

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-15 Thread Marcus Müller
Hi, On 01/15/2015 02:26 PM, Sylvain Munaut wrote: Why don't we just delete_head_blocking() and -- block? I think this would prevent the flow graph from stopping because it would block there indefinitely. Yes, a friendly cohabitant of this lab just told me to rtfm and also consider it; I do

[Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-14 Thread Achilleas Anastasopoulos
Hi all, I have the following problem that is been bugging me for quite some time now, and I would like to solicit your help. I made a hier block in GRC (called test_pdu_to_tag) with: pad_source---pdu_to_tagged_stream--pad_sink (I also made the pad_source optional and the pad_sink required)

Re: [Discuss-gnuradio] PDU_to_tagged_stream consumes 100% CPU even though it is throttled!

2015-01-14 Thread Bastian Bloessl
Hi, On 01/14/2015 11:29 PM, Achilleas Anastasopoulos wrote: I have the following problem that is been bugging me for quite some time now, and I would like to solicit your help. I made a hier block in GRC (called test_pdu_to_tag) with: pad_source---pdu_to_tagged_stream--pad_sink (I also made