Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Chris Johns
On 28/09/2016 18:38, Pavel Pisa wrote: And even rtems_* API should include mutex operations. It is clear and it is not necessary to complicate documentation then what is allowed and what is not with each semaphore flags combination. I am fine with adding a mutex API to the Classic API. The curr

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Chris Johns
On 28/09/2016 19:20, Pavel Pisa wrote: On Wednesday 28 of September 2016 11:06:19 Sebastian Huber wrote: In case the name "self-contained" is confusing for this purpose, then I am happy to replace it with a better alternative. No, I like self-contained, I have no problem with that. It describ

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Pavel Pisa
Hello Sebastian, On Wednesday 28 of September 2016 11:06:19 Sebastian Huber wrote: > On 28/09/16 10:47, Sebastian Huber wrote: > > On 28/09/16 10:38, Pavel Pisa wrote: > >> Hello Sebastian and Gedare, > >> > >> I cannot hold myself to not express my opinion there. > >> > >> On Wednesday 28 of Sept

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Sebastian Huber
On 28/09/16 10:47, Sebastian Huber wrote: On 28/09/16 10:38, Pavel Pisa wrote: Hello Sebastian and Gedare, I cannot hold myself to not express my opinion there. On Wednesday 28 of September 2016 07:52:51 Sebastian Huber wrote: On 27/09/16 16:59, Gedare Bloom wrote: A mostly unrelated questio

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Sebastian Huber
On 28/09/16 10:38, Pavel Pisa wrote: Hello Sebastian and Gedare, I cannot hold myself to not express my opinion there. On Wednesday 28 of September 2016 07:52:51 Sebastian Huber wrote: On 27/09/16 16:59, Gedare Bloom wrote: A mostly unrelated question: why do we have two different _Semaphore_

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-28 Thread Pavel Pisa
Hello Sebastian and Gedare, I cannot hold myself to not express my opinion there. On Wednesday 28 of September 2016 07:52:51 Sebastian Huber wrote: > On 27/09/16 16:59, Gedare Bloom wrote: > > A mostly unrelated question: why do we have two different > > _Semaphore_Get functions, one static in sc

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-27 Thread Sebastian Huber
On 27/09/16 16:59, Gedare Bloom wrote: A mostly unrelated question: why do we have two different _Semaphore_Get functions, one static in score/src/semaphore.c and the other inlined from semimpl.h? Yes, this is a bit confusing. One is part of the Classic API, the other is for the self-contained

Re: [PATCH] score: Unify CORE mutex seize/surrender

2016-09-27 Thread Gedare Bloom
OK. A mostly unrelated question: why do we have two different _Semaphore_Get functions, one static in score/src/semaphore.c and the other inlined from semimpl.h? On Tue, Sep 27, 2016 at 9:25 AM, Sebastian Huber wrote: > Use the Thread_Control::resource_count for the no protocol mutexes. > Merge t

[PATCH] score: Unify CORE mutex seize/surrender

2016-09-27 Thread Sebastian Huber
Use the Thread_Control::resource_count for the no protocol mutexes. Merge the no protocol and priority inherit CORE mutex seize/surrender operations. --- cpukit/libnetworking/rtems/rtems_glue.c | 2 + cpukit/posix/src/mutexlocksupp.c | 3 +- cpukit/posix/src/mutexunlock.