Experimental Physics and
Industrial Control System

mdavidsaver via Core-talk <core-talk at aps.anl.gov> · Tue, 02 Feb 2021 19:06:31 -0000

Ok, maybe time to take a(nother) step back and ask if epicsMutexPriorityInversionTest should be run by default.

> testOk1( hiPriStalledTimeUs < 1000 );

This will almost certainly fail randomly with the CI builders since they don't have a PREEMPT_RT kernel, and are anyway timeshare VMs.

I don't see a need for Base to test whether PTHREAD_PRIO_INHERIT functions correctly.  This seems like testing malloc().  The critical question which a user will have is whether PTHREAD_PRIO_INHERIT is being set.  This seems more like asking whether OS priorities are being set at all.  This is complicated since, unlike priority or mlock(), I don't know of a straightforward way to verify this externally.  (I would resort to strace, which is complicated to interpret)

The question of OS priorities is answered by epicsThreadShowAll().  So maybe it would be sufficient for epicsMutexShowAll() to print a line showing if PTHREAD_PRIO_INHERIT is being set?
-- 
https://code.launchpad.net/~dirk.zimoch/epics-base/+git/epics-base/+merge/394327
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System