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 2021/07/02 09:46:11 UTC
[GitHub] [incubator-nuttx-apps] GUIDINGLI opened a new pull request #793: Update testing/mm testcase, add stress test
GUIDINGLI opened a new pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793
## Summary
Update testing/mm testcase, add stress test
## 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-apps] xiaoxiang781216 closed pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 closed pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793
--
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-apps] xiaoxiang781216 commented on pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#issuecomment-876223480
Let's close this PR.
--
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-apps] acassis commented on a change in pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
acassis commented on a change in pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#discussion_r663133267
##########
File path: testing/mm/Kconfig
##########
@@ -20,7 +20,7 @@ config TESTING_MM_PROGNAME
config TESTING_MM_PRIORITY
int "Task priority"
- default 100
+ default 250
Review comment:
Why do you need to increase the priority? Does it fail when using priority 100?
--
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-apps] xiaoxiang781216 commented on a change in pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#discussion_r663354059
##########
File path: testing/mm/Kconfig
##########
@@ -20,7 +20,7 @@ config TESTING_MM_PROGNAME
config TESTING_MM_PRIORITY
int "Task priority"
- default 100
+ default 250
Review comment:
Yes, but it's rare:).
--
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-apps] xiaoxiang781216 commented on a change in pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#discussion_r663324363
##########
File path: testing/mm/Kconfig
##########
@@ -20,7 +20,7 @@ config TESTING_MM_PROGNAME
config TESTING_MM_PRIORITY
int "Task priority"
- default 100
+ default 250
Review comment:
We run memtest in the background. The high prirority could avoid the bad high prority task block memtest.
--
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-apps] acassis commented on a change in pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
acassis commented on a change in pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#discussion_r663350544
##########
File path: testing/mm/Kconfig
##########
@@ -20,7 +20,7 @@ config TESTING_MM_PROGNAME
config TESTING_MM_PRIORITY
int "Task priority"
- default 100
+ default 250
Review comment:
Ok! BTW it still be possible to have higher priority tasks (251-255)
--
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-apps] xiaoxiang781216 commented on pull request #793: Update testing/mm testcase, add stress test
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #793:
URL: https://github.com/apache/incubator-nuttx-apps/pull/793#issuecomment-873354126
Fix here: https://github.com/apache/incubator-nuttx/pull/4051
--
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