Ouss4 commented on a change in pull request #4782:
URL: https://github.com/apache/incubator-nuttx/pull/4782#discussion_r745640174
##########
File path: mm/mm_heap/mm_free.c
##########
@@ -86,6 +86,10 @@ void mm_free(FAR struct mm_heap_s *heap, FAR void *mem)
kasan_poison(mem, mm_malloc_size(mem));
+ /* Must not in IRQ */
+
+ DEBUGASSERT(!up_interrupt_context());
Review comment:
In the case we are in an interrupt context, this will nullify the
changes we have in this PR for `mm_takesemaphore`.
This check seems to be already implemented by the usual
`DEBUGVERIFY(mm_takesemaphore)`.
--
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
