Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-27 Thread via GitHub
acassis commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2688845368 @cederom sorry, but I think he fixed it -- 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

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-27 Thread via GitHub
cederom commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2688658051 @acassis you have merged PR with "changes requested" and unresolved discussion :-( :-( -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-27 Thread via GitHub
acassis merged PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000 -- 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.a

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-26 Thread via GitHub
zdebanos commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2686045791 I've managed to prepare the documentation for the cyclictest utility, I've created a [pull request here](https://github.com/apache/nuttx/pull/15913). Concerning the semaphore systemti

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-22 Thread via GitHub
zdebanos commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2676362175 Thank you very much for the review! I've been going through the code and there are still some bugs I did not think about. But as I have time, I'll prepare the documentation. -- Thi

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-22 Thread via GitHub
acassis commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2676137788 Nice work @zdebanos !!! Please submit also a Documentation/ to the NuttX repository show usages examples, expected results, how to spot issues using the cyclictest, etc. -- This is

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-17 Thread via GitHub
xiaoxiang781216 commented on code in PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#discussion_r1958106038 ## benchmarks/cyclictest/cyclictest.c: ## @@ -0,0 +1,1066 @@ +/ + * apps/benchmar

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-17 Thread via GitHub
zdebanos commented on code in PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#discussion_r1958183568 ## benchmarks/cyclictest/cyclictest.c: ## @@ -0,0 +1,1066 @@ +/ + * apps/benchmarks/cycl

Re: [PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-17 Thread via GitHub
nuttxpr commented on PR #3000: URL: https://github.com/apache/nuttx-apps/pull/3000#issuecomment-2662775885 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR description, while a good start, doesn't ful

[PR] benchmarks/cyclictest: the rt-tests cyclictest NuttX Port [nuttx-apps]

2025-02-17 Thread via GitHub
zdebanos opened a new pull request, #3000: URL: https://github.com/apache/nuttx-apps/pull/3000 Despite the existence of the patch in benchmarks/rt-tests, this commit adds the NuttX Official cyclictest utility. The main difference is the introduction of different waiting methods next t