Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-10 Thread Sebastian Huber
Hello, I added a ticket for this: https://devel.rtems.org/ticket/4074 ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-09 Thread Sebastian Huber
On 09/09/2020 00:25, Joel Sherrill wrote:   This has gotten to be a long-ish side discussion. I havecomments on the patch and will reply to the original email for that. On Thu, Sep 3, 2020 at 10:17 PM Chris Johns > wrote: On 3/9/20 10:08 pm, Sebastian Huber wrote

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-09 Thread Sebastian Huber
On 09/09/2020 10:07, Chris Johns wrote: On 9/9/20 4:49 pm, Sebastian Huber wrote: On 09/09/2020 07:56, Chris Johns wrote: What about a test for this API that has a 100K char TLS array and all the other task values as defaults? If the space allocated and reported in the link map is not just over

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-09 Thread Sebastian Huber
On 09/09/2020 00:43, Joel Sherrill wrote: On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber > wrote: In contrast to rtems_task_create() this function creates a task with a user-provided task storage area.  The new create function uses a co

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-09 Thread Chris Johns
On 9/9/20 4:49 pm, Sebastian Huber wrote: > On 09/09/2020 07:56, Chris Johns wrote: >> What about a test for this API that has a 100K char TLS array and all the >> other >> task values as defaults? If the space allocated and reported in the link map >> is >> not just over 100K there is a problem.

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Sebastian Huber
On 09/09/2020 07:56, Chris Johns wrote: On 9/9/20 3:14 pm, Sebastian Huber wrote: On 09/09/2020 03:37, Chris Johns wrote: On 9/9/20 8:43 am, Joel Sherrill wrote: On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber mailto:sebastian.hu...@embedded-brains.de>> wrote: +/** + * @brief Retur

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Chris Johns
On 9/9/20 3:14 pm, Sebastian Huber wrote: > On 09/09/2020 03:37, Chris Johns wrote: >> On 9/9/20 8:43 am, Joel Sherrill wrote: >>> On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber >>> >> > >>> wrote: >>> +/** >>> + * @brief Returns the recommended

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Sebastian Huber
On 09/09/2020 03:37, Chris Johns wrote: On 9/9/20 8:43 am, Joel Sherrill wrote: On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber mailto:sebastian.hu...@embedded-brains.de>> wrote: +/** + * @brief Returns the recommended task storage area size for the specified size + *   and task

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Chris Johns
On 9/9/20 8:43 am, Joel Sherrill wrote: > On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber > > > wrote: > +/** > + * @brief Returns the recommended task storage area size for the > specified size > + *   and task attributes. > + * > +

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Joel Sherrill
On Wed, Sep 2, 2020 at 11:09 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > In contrast to rtems_task_create() this function creates a task with a > user-provided task storage area. The new create function uses a > configuration structure instead of individual parameters. > > A

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-08 Thread Joel Sherrill
This has gotten to be a long-ish side discussion. I havecomments on the patch and will reply to the original email for that. On Thu, Sep 3, 2020 at 10:17 PM Chris Johns wrote: > On 3/9/20 10:08 pm, Sebastian Huber wrote: > > > > On 03/09/2020 04:08, Chris Johns wrote: > >> On 3/9/20 2:09 am, S

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-07 Thread Sebastian Huber
Hello Joel, On 04/09/2020 05:17, Chris Johns wrote: On 3/9/20 10:08 pm, Sebastian Huber wrote: On 03/09/2020 04:08, Chris Johns wrote: On 3/9/20 2:09 am, Sebastian Huber wrote: In contrast to rtems_task_create() this function creates a task with a user-provided task storage area.  The new cre

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-03 Thread Chris Johns
On 3/9/20 10:08 pm, Sebastian Huber wrote: > > On 03/09/2020 04:08, Chris Johns wrote: >> On 3/9/20 2:09 am, Sebastian Huber wrote: >>> In contrast to rtems_task_create() this function creates a task with a >>> user-provided task storage area.  The new create function uses a >>> configuration stru

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-03 Thread Sebastian Huber
On 03/09/2020 04:08, Chris Johns wrote: On 3/9/20 2:09 am, Sebastian Huber wrote: In contrast to rtems_task_create() this function creates a task with a user-provided task storage area. The new create function uses a configuration structure instead of individual parameters. Add RTEMS_TASK_ST

Re: [PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-02 Thread Chris Johns
On 3/9/20 2:09 am, Sebastian Huber wrote: > In contrast to rtems_task_create() this function creates a task with a > user-provided task storage area. The new create function uses a > configuration structure instead of individual parameters. > > Add RTEMS_TASK_STORAGE_ALIGNMENT to define the recom

[PATCH v2] rtems: Add rtems_task_create_from_config()

2020-09-02 Thread Sebastian Huber
In contrast to rtems_task_create() this function creates a task with a user-provided task storage area. The new create function uses a configuration structure instead of individual parameters. Add RTEMS_TASK_STORAGE_ALIGNMENT to define the recommended alignment of a task storage area. Add RTEMS_