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