EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <20192020  2021  2022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <20192020  2021  2022  2023  2024 
<== Date ==> <== Thread ==>

Subject: Re: [Merge] ~epics-core/epics-base/+git/Com:thread-join into epics-base:7.0
From: mdavidsaver via Core-talk <[email protected]>
To: mdavidsaver <[email protected]>
Date: Tue, 18 Jun 2019 01:38:23 -0000
> But there is a cure: https://plugins.jenkins.io/build-timeout

I agree an overall timeout would be a better way to solve the Bothering Andrew part of this problem.

If Jenkins can't be made to work, then maybe makeTestfile.pl could be taught about SIGALRM.

> If it's too hard to implement a timeout on Posix I would accept that as a reason for not adding it

The standard pthread_join() doesn't have a timeout.  A quick search finds pthread_tryjoin_np(), which is a glibc extension.

> that epicsThreadJoin() should return a status

What status?  How would a caller react?

Part of my reasoning for not having a return value is that by the time a join is performed, there often isn't anything sensible to be done.

eg. I regard epicsMutexLock() returning as a mis-feature.  There is nothing sensible which user code can do with a epicsMutexLockError.  So why offer up the option?  Seems an invitation to ignore a serious error.

-- 
https://code.launchpad.net/~epics-core/epics-base/+git/Com/+merge/361379
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.

Navigate by Date:
Prev: Re: [Merge] ~epics-core/epics-base/+git/Com:thread-join into epics-base:7.0 Martin Konrad via Core-talk
Next: Jenkins build is back to stable : epics-pva2pva-linux32 #119 APS Jenkins via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <20192020  2021  2022  2023  2024 
Navigate by Thread:
Prev: Re: [Merge] ~epics-core/epics-base/+git/Com:thread-join into epics-base:7.0 Martin Konrad via Core-talk
Next: Re: [Merge] ~epics-core/epics-base/+git/Com:thread-join into epics-base:7.0 Andrew Johnson via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <20192020  2021  2022  2023  2024 
ANJ, 18 Jun 2019 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·