Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Sebastian Huber
On 20/05/2021 22:33, Gedare Bloom wrote: +static void _Thread_Consider_life_change_request( Minor, I'd prefer "_Thread_Try_life_change_request(" Yes, this is better, thanks for the suggestion. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.

Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Gedare Bloom
On Thu, May 20, 2021 at 9:18 AM Sebastian Huber wrote: > > The _Thread_Cancel() (in contrast to _Thread_Restart() which used a > similar code block) may have produced ready threads with an active timer > in case the thread to cancel had its thread life protection enabled. The > problem was this co

Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Sebastian Huber
On 20/05/2021 17:35, Joel Sherrill wrote: On Thu, May 20, 2021, 10:33 AM Sebastian Huber > wrote: On 20/05/2021 17:30, Joel Sherrill wrote: > Is there a test case for this situation? Not in the current test suite. I write currently the

Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Joel Sherrill
On Thu, May 20, 2021, 10:33 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > On 20/05/2021 17:30, Joel Sherrill wrote: > > Is there a test case for this situation? > > Not in the current test suite. I write currently the specification for > rtems_task_restart() and the validation

Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Sebastian Huber
On 20/05/2021 17:30, Joel Sherrill wrote: Is there a test case for this situation? Not in the current test suite. I write currently the specification for rtems_task_restart() and the validation tests. My plan is to start with the integration of the new validation test suites after they are f

Re: [PATCH] score: Fix _Thread_Cancel()

2021-05-20 Thread Joel Sherrill
Is there a test case for this situation? On Thu, May 20, 2021 at 10:18 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > The _Thread_Cancel() (in contrast to _Thread_Restart() which used a > similar code block) may have produced ready threads with an active timer > in case the thr