[PATCH 2/3] score: Add CONFIGURE_INIT_TASK_STORAGE_SIZE

Chris Johns chrisj at rtems.org
Mon Nov 23 20:23:09 UTC 2020


On 23/11/20 7:46 pm, Sebastian Huber wrote:
> On 22/11/2020 23:22, Chris Johns wrote:
>> On 20/11/20 7:31 pm, Sebastian Huber wrote:
>>> In order to better support applications which use the new
>>> rtems_task_construct() directive add the
>>> CONFIGURE_INIT_TASK_STORAGE_SIZE configuration option.  If this option
>>> is specified, then the Classic API initialization task is constructed
>>> with rtems_task_construct().
>> The name CONFIGURE_INIT_TASK_STORAGE_SIZE does not reflect the role described
>> here and is a little ambiguous unless you know the implementation detail.
>>
>> CONFIGURE_INIT_TASK_CONSTRUCT_STORAGE_SIZE ?
> I am not sure. The storage size is a property of the task. How this property is
> achieved is a detail and may change.

Is it a property of a specific instance (Init) of a specific type (construct) of
task?

You have stated:

 "If this option is specified, then the Classic API initialization task
  is constructed with rtems_task_construct()."

This is how I am reading this sentence. Sure in time the task may not be
constructed and that may change but the effect needs to be a "constructed" task
or you would need another variables.

The key issue is the config option triggers a change in the type of Init task
and we should be as clear about that as we can.

Chris


More information about the devel mailing list