[PATCH v3 0/5] Exception Extensions

Kinsey Moore kinsey.moore at oarcorp.com
Thu Oct 28 16:04:57 UTC 2021


On 10/28/2021 00:51, Sebastian Huber wrote:
> On 27/10/2021 23:44, Kinsey Moore wrote:
>> This patch set is a rework of the previous Exception Management API
>> patch set based on feedback from the community. From the last revision:
>> * Everything has been moved out of the RTEMS Classic API namespace
>> * The concept of exception classes have been removed in favor of a
>>    direct CPU_Exception_frame to signal mapping function
>> * The added tests have been reworked to accomodate the above changes
>
> Thanks, the patch set looks good overall. I just have some minor 
> comments.
>
>>
>> It was not possible to defer the call to raise() to a post-switch action
>> since the post-switch action list is already locked and thus cannot be
>> relocked by calls underneath raise() to add the signal handler call to
>> the list.
>
> You can release the thread state lock in a post-switch action, see 
> _Signal_Action_handler() for an example. You can keep the code as is 
> if you want. I personally would be a bit nervous if someone calls the 
> highly complex raise() function in exception context.
>
Will do.


Thanks,

Kinsey



More information about the devel mailing list