xiaoxiang781216 commented on pull request #5070: URL: https://github.com/apache/incubator-nuttx/pull/5070#issuecomment-1000980875
> Drivers that use the semaphores for signaling should not be involved with priority inheritance (when enabled). That use case was assumed to be the non default. > > It seams to me this change can be done with adding an Kconfig option for the default and leave it as it was. Kconfig is good to disable/enable the optional feature, but it isn't a good solution here. Do you want sem_t change it's behavior by an option? And all place which call sem_init/nxsem_init need check the option and call nxsem_set_protocol with the different value. > But allow the default to be changed if someone is using Linux code semantics. First, it isn't Linux semantics. Second, my demo show that isn't a minor issue because the program will crash the system. -- 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
