On 13/05/17 00:30, Chris Johns wrote:
On 12/5/17 8:24 pm, Sebastian Huber wrote:
Mark tests that require review due to
CONFIGURE_DISABLE_SMP_CONFIGURATION.
If a test fails it fails. I feel the test results need to correctly
express the failures and we should not masks them in the way. I do not
agree with changes that suppress a failure.

I do not see SMP as any special over other reasons a test fails and I
hope we do not start adding them. We exclude tests from being built
because they do not fit into the target resources and that is fixed,
bounded.

There are differences in non-SMP vs. SMP mode, e.g. disable thread preemption and interrupt level task modes are not supported. The tests are perfectly fine, but they use these explicit non-SMP features. You wanted more tests to run in the SMP mode if SMP is enabled. Now only 36 tests are left over. They should be fixed to not use these features, but someone has to look at each test individually for this. This could be as easy as this:

https://git.rtems.org/rtems/commit/?id=00d982080cf1e630fea9c6e8b3a4e7a5be501781

I cannot fix everything instantly. I have to prioritize my work. There are a lot of other things to do and is this really the most important stuff right now before the RTEMS 4.12 release?

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.hu...@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to