You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/10/28 03:51:19 UTC
[GitHub] [incubator-nuttx] Gary-Hobson opened a new pull request, #7457: Check for memory leaks in tasks
Gary-Hobson opened a new pull request, #7457:
URL: https://github.com/apache/incubator-nuttx/pull/7457
## Summary
Print all unreleased memory blocks when the task exits
## Impact
## Testing
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7457: Check for memory leaks in tasks
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged PR #7457:
URL: https://github.com/apache/incubator-nuttx/pull/7457
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] hartmannathan commented on a diff in pull request #7457: Check for memory leaks in tasks
Posted by GitBox <gi...@apache.org>.
hartmannathan commented on code in PR #7457:
URL: https://github.com/apache/incubator-nuttx/pull/7457#discussion_r1008005942
##########
sched/Kconfig:
##########
@@ -595,6 +595,12 @@ config SCHED_WAITPID
compliant) and will enable the waitid() and wait() interfaces as
well.
+config SCHED_DUMP_LEAK
+ bool "Enable catch task memory leak"
+ default n
+ ---help---
+ Catch memory leaks when the task exits.
Review Comment:
Improve the description? Suggestion:
```suggestion
When this option is enabled, the task's outstanding memory allocations
are printed using syslog. This helps catch any memory allocated by the
task that remains unreleased when the task exits.
```
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7457: Check for memory leaks in tasks
Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7457:
URL: https://github.com/apache/incubator-nuttx/pull/7457#discussion_r1008009180
##########
sched/Kconfig:
##########
@@ -595,6 +595,12 @@ config SCHED_WAITPID
compliant) and will enable the waitid() and wait() interfaces as
well.
+config SCHED_DUMP_LEAK
+ bool "Enable catch task memory leak"
+ default n
+ ---help---
+ Catch memory leaks when the task exits.
Review Comment:
Done~
--
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-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org