[PATCH 10/12] cpukit/sapi: Resources for bdbuf SMP workaround
Chris Johns
chrisj at rtems.org
Tue May 27 23:39:04 UTC 2014
On 28/05/2014 12:48 am, Ralf Kirchner wrote:
> Enabling and disabling preemption as done for single core in bdbuf will not work for SMP.
Correct. Do the bdbuf tests currently fail on SMP ?
> Thus as a temporary workaround
If this is a workaround what do you see is the proper fix ?
> use POSIX mutexes and POSIX condition variables for SMP instead of the combination of semaphores and preemption handling used for single core.
I question this being conditional on SMP. Why not make it the same for
both builds of RTEMS ? I have no problem with POSIX condition variables
being used as the current method is a poor hacked version (done by me)
and should be removed. Is doing a suitable "proper fix" ?
I would rather see us have a single method used for this code and it be
tested and run as much as possible than see the code fragment in this way.
> These will be allocated automatically.
On the topic of automatic maybe POSIX defaults to on and if disabled
this code is not built. I do not think we need more POSIX enable work
arounds.
Chris
More information about the devel
mailing list