Experimental Physics and
Industrial Control System

I think a smaller diff would help move the process faster, as would more targeted changes.

Personally, I'm not sure I'll accept a global pthread_mutexattr_t.  I think this should become a local in epicsMutexOsdCreate() (or equivalent helper).

As a general rule, I prefer locality for various reasons.  Modern processors with deep caches generally reward memory locality.  Mostly though, it is a case of readability.  I find it easier to understand code which doesn't reference globals.

-- 
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.

References:

[Merge] ~dirk.zimoch/epics-base:epicsMutexPriorityInheritance into epics-base:7.0 Dirk Zimoch via Core-talk

Navigate by Date:

Prev: Re: [Merge] ~dirk.zimoch/epics-base:epicsMutexPriorityInheritance into epics-base:7.0 Dirk Zimoch via Core-talk

Next: epics-7.0 - Build # 272 - Fixed! APS Jenkins via Core-talk

Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <2020>  2021  2022  2023  2024 

Navigate by Thread:

Prev: Re: [Merge] ~dirk.zimoch/epics-base:epicsMutexPriorityInheritance into epics-base:7.0 Dirk Zimoch via Core-talk

Next: Re: [Merge] ~dirk.zimoch/epics-base:epicsMutexPriorityInheritance into epics-base:7.0 Dirk Zimoch via Core-talk

Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <2020>  2021  2022  2023  2024