debug enabled.. wkspace under reserved Re: change log for rtems (2011-08-21)

Joel Sherrill joel.sherrill at OARcorp.com
Mon Aug 22 13:28:23 UTC 2011


On 08/22/2011 02:17 AM, Sebastian Huber wrote:
> On 08/21/2011 10:10 PM, rtems-vc at rtems.org wrote:
>> diff -u rtems/cpukit/score/include/rtems/score/heap.h:1.45 rtems/cpukit/score/include/rtems/score/heap.h:1.46
>> --- rtems/cpukit/score/include/rtems/score/heap.h:1.45	Thu Feb 17 08:17:09 2011
>> +++ rtems/cpukit/score/include/rtems/score/heap.h	Sun Aug 21 14:51:41 2011
>> @@ -177,7 +177,9 @@
>>      } Heap_Protection_block_end;
>>
>>      #define HEAP_PROTECTION_HEADER_SIZE \
>> -    (sizeof(Heap_Protection_block_begin) + sizeof(Heap_Protection_block_end))
>> +    (sizeof(Heap_Protection_block_begin) + \
>> +     CPU_ALIGNMENT + \
>> +     sizeof(Heap_Protection_block_end))
>>    #endif
>>
>>    /**
> I doubt that this change is correct.  What was the reason for this?  This
> changes also the HEAP_BLOCK_HEADER_SIZE which is used for very fundamental
> calculations within the heap code.  The CPU_ALIGNMENT value seems to be a bit
> arbitrary here, since the heap uses the page size for alignment.
>

All I know is that turning on debug results in under estimating
the amount of workspace required.  This is the only factor
that changes as best I can tell.  So it appears to be wrong
somehow.

Any other ideas?

-- 
Joel Sherrill, Ph.D.             Director of Research&  Development
joel.sherrill at OARcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
    Support Available             (256) 722-9985





More information about the vc mailing list