pkarashchenko commented on pull request #5180:
URL: https://github.com/apache/incubator-nuttx/pull/5180#issuecomment-1006553929


   > > > If the usage is fixed as lock(e.g. pthread_mutex_t, mutex_t), it's 
good to enable the priority inheritance by default. If the usage isn't fixed 
like sem_t, we have to disable the priority inheritance by default.
   > > 
   > > 
   > > So we need to violate POSIX compliance by default, right?
   > 
   > We can add an option like pthread_mutex_t: 
https://github.com/apache/incubator-nuttx/blob/master/sched/Kconfig#L644-L657
   
   Yes. But in this example `default PTHREAD_MUTEX_DEFAULT_ROBUST`, so should 
we `default PTHREAD_MUTEX_PROTO_DEFAULT_NONE` or `default 
PTHREAD_MUTEX_PROTO_DEFAULT_INHERIT`?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to