pkarashchenko commented on PR #6414: URL: https://github.com/apache/incubator-nuttx/pull/6414#issuecomment-1153861541
@davids5 done. Anyway I was trying to address another part in https://github.com/apache/incubator-nuttx/pull/6376 I think we need to revert libc related changes anyway and then think about better solution. I found out that currently we have even bigger gap since in case of FLAT build the `_SEM_` macro interface is used also for dynamic memory allocation, so `malloc` and `free` become cancellation points. I do not have a solid solution for that, but think that we should not rock the boat with changes like https://github.com/apache/incubator-nuttx/pull/6320 introducing even more inconsistency. IMO we need to revert another part of the changes (that I removed from this PR) and spend some time on a design review so then it will be possible to come up with a solid solution. -- 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
