[PR] fs/spiffs: correct mutex lock cycle of spiffs [nuttx]

2023-11-21 Thread via GitHub
anchao opened a new pull request, #11248: URL: https://github.com/apache/nuttx/pull/11248 ## Summary fs/spiffs: correct mutex lock cycle of spiffs This PR will fix the below issues: 1. double lock() on dup 2. use after free on unbind Signed-off-by: chao an

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
masayuki2009 commented on PR #11207: URL: https://github.com/apache/nuttx/pull/11207#issuecomment-1822237944 > @anjiahao1 please drop the second patch Did you test the latest PR on M1 Mac? -- This is an automated message from the Apache Git Service. To respond to the message,

[PR] risc-v/mpfs: ihc: cleanup DEBUGASSERTs and irq enabling [nuttx]

2023-11-21 Thread via GitHub
eenurkka opened a new pull request, #11247: URL: https://github.com/apache/nuttx/pull/11247 Replace DEBUGASSERTs with sanity checks. DEBUGASSERT()s are not necessarily enabled at all, thus risking the functionality especially in that case. Remove PANICs as well. Don't enable the ihc

[PR] cmake: add compilation support for matter [nuttx-apps]

2023-11-21 Thread via GitHub
zhhyu7 opened a new pull request, #2209: URL: https://github.com/apache/nuttx-apps/pull/2209 ## Summary A nuttx with the matter command can be compiled using cmake and enable CONFIG_MATTER option. The matter compiled by the current script is just a lighting example of a limited

Re: [PR] PoC: allow apps to customize NuttX build time variables [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11202: URL: https://github.com/apache/nuttx/pull/11202#discussion_r1401540178 ## arch/arm/src/Makefile: ## @@ -18,7 +18,7 @@ # -include $(TOPDIR)/Make.defs

Re: [PR] PoC: allow apps to customize NuttX build time variables [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11202: URL: https://github.com/apache/nuttx/pull/11202#discussion_r1401540178 ## arch/arm/src/Makefile: ## @@ -18,7 +18,7 @@ # -include $(TOPDIR)/Make.defs

Re: [PR] arch/sim: sim support 16bbp [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11246: URL: https://github.com/apache/nuttx/pull/11246 -- 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:

(nuttx) branch master updated: arch/sim: sim support 16bbp

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new e5eabbb411 arch/sim: sim support 16bbp

(nuttx) branch master updated: tcbinfo:remove total_num form tcbinfo.

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 749655d785 tcbinfo:remove total_num form

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11207: URL: https://github.com/apache/nuttx/pull/11207 -- 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:

(nuttx) branch master updated: sched_lock refine: remove sched_[un]lock in xxx_waitsample

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 1a65f5ed88 sched_lock refine: remove

Re: [PR] sched_lock refine: remove sched_[un]lock in xxx_waitsample [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11231: URL: https://github.com/apache/nuttx/pull/11231 -- 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:

(nuttx) branch master updated: fs/tmpfs: fix an integer overflow

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new b2e6d7b9d7 fs/tmpfs: fix an integer overflow

Re: [PR] fs/tmpfs: fix an integer overflow [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11232: URL: https://github.com/apache/nuttx/pull/11232 -- 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:

Re: [PR] ostest: Add initial support for CONFIG_BUILD_KERNEL [nuttx-apps]

2023-11-21 Thread via GitHub
fxysunshine commented on code in PR #2205: URL: https://github.com/apache/nuttx-apps/pull/2205#discussion_r1401460622 ## testing/ostest/ostest_main.c: ## @@ -706,6 +719,9 @@ int main(int argc, FAR char **argv) } printf("ostest_main: Exiting with status %d\n",

Re: [PR] ostest: Add initial support for CONFIG_BUILD_KERNEL [nuttx-apps]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #2205: URL: https://github.com/apache/nuttx-apps/pull/2205#discussion_r1401456158 ## testing/ostest/ostest_main.c: ## @@ -581,8 +585,16 @@ static int user_main(int argc, char *argv[]) #endif #if defined(CONFIG_ARCH_HAVE_FORK) &&

Re: [PR] ostest: Add initial support for CONFIG_BUILD_KERNEL [nuttx-apps]

2023-11-21 Thread via GitHub
fxysunshine commented on code in PR #2205: URL: https://github.com/apache/nuttx-apps/pull/2205#discussion_r1401455747 ## testing/ostest/ostest_main.c: ## @@ -581,8 +585,16 @@ static int user_main(int argc, char *argv[]) #endif #if defined(CONFIG_ARCH_HAVE_FORK) &&

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
SPRESENSE commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1821988026 @pussuw I could confirm that this issue was fixed by https://github.com/apache/nuttx/pull/11241. -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] arch/sim: sim support 16bbp [nuttx]

2023-11-21 Thread via GitHub
qinshijing commented on code in PR #11246: URL: https://github.com/apache/nuttx/pull/11246#discussion_r1401393713 ## tools/nxstyle.c: ## @@ -563,6 +563,7 @@ static const char *g_white_content_list[] = "unzGetCurrentFileInfo64", "unzGoToNextFile", "unzGoToFirstFile", +

(nuttx) branch master updated: simwifi: Escapes the special characters of ssid in the scan results.

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new be32247e73 simwifi: Escapes the special

Re: [PR] simwifi: Escapes the special characters of ssid in the scan results. [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11235: URL: https://github.com/apache/nuttx/pull/11235 -- 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:

(nuttx-website) branch asf-site updated: Publishing web: 36ddd7c60a8230335eda886c909bc938739114a2 docs: 7c13116007ec1258a6d909ab8c1eaf0f4cfc4169

2023-11-21 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 8fd1b1ea Publishing web:

Re: [PR] [Don't submit]: add test for SMP bug [nuttx-apps]

2023-11-21 Thread via GitHub
hartmannathan commented on PR #2208: URL: https://github.com/apache/nuttx-apps/pull/2208#issuecomment-1821837719 This PR is labeled `[Don't submit]`. Do you want to mark it as Draft to avoid accidental merge? -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
hartmannathan commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1401315128 ## include/nuttx/tee.h: ## @@ -0,0 +1,479 @@ +/ + * include/nuttx/tee.h + * + *

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
hartmannathan commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1401312935 ## drivers/misc/optee.c: ## @@ -35,6 +35,14 @@ #include "optee_msg.h" +/ + *

Re: [PR] assert:read content of undefinedinsn address [nuttx]

2023-11-21 Thread via GitHub
hartmannathan commented on code in PR #11244: URL: https://github.com/apache/nuttx/pull/11244#discussion_r1401307885 ## arch/arm/src/arm/arm_undefinedinsn.c: ## @@ -42,7 +42,16 @@ void arm_undefinedinsn(uint32_t *regs) { - _alert("Undefined instruction at 0x%" PRIx32 "\n",

Re: [PR] arch/sim: sim support 16bbp [nuttx]

2023-11-21 Thread via GitHub
hartmannathan commented on code in PR #11246: URL: https://github.com/apache/nuttx/pull/11246#discussion_r1401303107 ## tools/nxstyle.c: ## @@ -563,6 +563,7 @@ static const char *g_white_content_list[] = "unzGetCurrentFileInfo64", "unzGoToNextFile", "unzGoToFirstFile",

Re: [I] gdb nsh:tcpblaster cannot access network [nuttx]

2023-11-21 Thread via GitHub
maxmbed commented on issue #11223: URL: https://github.com/apache/nuttx/issues/11223#issuecomment-1821816720 @alandeassis, No, the host cannot ping Nuttx in this case. Actually, the `sim:tcpblaster` example has already ICMP support enabled. Network is accessible if I run

Re: [PR] Rpmsgblk function optimization [nuttx]

2023-11-21 Thread via GitHub
alandeassis commented on PR #11240: URL: https://github.com/apache/nuttx/pull/11240#issuecomment-1821749316 @leo11261 please remove the chinese characters from commit message -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] task/pthread_cancelpt: Fix nxtask_delete from another task group [nuttx]

2023-11-21 Thread via GitHub
pkarashchenko commented on code in PR #11241: URL: https://github.com/apache/nuttx/pull/11241#discussion_r1400895374 ## sched/sched/sched_get_tls.c: ## @@ -0,0 +1,55 @@ +/ + *

Re: [PR] PoC: allow apps to customize NuttX build time variables [nuttx]

2023-11-21 Thread via GitHub
casaroli commented on code in PR #11202: URL: https://github.com/apache/nuttx/pull/11202#discussion_r1400851199 ## arch/arm/src/Makefile: ## @@ -18,7 +18,7 @@ # -include $(TOPDIR)/Make.defs

(nuttx-apps) branch master updated: wamr: Add support for custom section loading

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 097411de4 wamr: Add support for custom

Re: [PR] wamr: Add support for custom section loading [nuttx-apps]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #2206: URL: https://github.com/apache/nuttx-apps/pull/2206 -- 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:

Re: [PR] ostest: Add initial support for CONFIG_BUILD_KERNEL [nuttx-apps]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #2205: URL: https://github.com/apache/nuttx-apps/pull/2205#discussion_r1400833319 ## testing/ostest/ostest_main.c: ## @@ -347,13 +349,15 @@ static int user_main(int argc, char *argv[]) check_test_memory_usage(); #endif +#ifndef

Re: [PR] libtest: add PRIVATE to CMakeLists.txt [nuttx-apps]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #2207: URL: https://github.com/apache/nuttx-apps/pull/2207 -- 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:

(nuttx-apps) branch master updated: libtest: add PRIVATE to CMakeLists.txt

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new ba43911a7 libtest: add PRIVATE to

Re: [PR] task/pthread_cancelpt: Fix nxtask_delete from another task group [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11241: URL: https://github.com/apache/nuttx/pull/11241#discussion_r1400826919 ## include/nuttx/sched.h: ## @@ -1424,6 +1424,22 @@ int nxsched_set_affinity(pid_t pid, size_t cpusetsize, int nxsched_get_stackinfo(pid_t pid, FAR struct

(nuttx) branch master updated: simwifi: Connect the wifi whose ssid contains the special charaters.

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 7c13116007 simwifi: Connect the wifi whose

Re: [PR] simwifi: Connect the wifi whose ssid contains the special charaters. [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11237: URL: https://github.com/apache/nuttx/pull/11237 -- 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:

Re: [PR] list: search prev item in reverse order [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11243: URL: https://github.com/apache/nuttx/pull/11243 -- 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:

(nuttx) branch master updated: list: search prev item in reverse order

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 4446b34b2c list: search prev item in reverse

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on PR #11207: URL: https://github.com/apache/nuttx/pull/11207#issuecomment-1821200980 @anjiahao1 please drop the second patch -- 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

(nuttx) branch master updated: Revert "libc/lib_bzero:Add bzero prototype."

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new f12c4fb887 Revert "libc/lib_bzero:Add bzero

Re: [PR] Revert "libc/lib_bzero:Add bzero prototype." [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11242: URL: https://github.com/apache/nuttx/pull/11242 -- 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:

Re: [I] Simulator on M1/M2 MacOS seems not working [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 closed issue #11087: Simulator on M1/M2 MacOS seems not working URL: https://github.com/apache/nuttx/issues/11087 -- 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

(nuttx) branch master updated: arm64_vector: no need to save x0 to sp

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 6e604741f6 arm64_vector: no need to save x0

Re: [PR] arm64_vector: no need to save x0 to sp [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11245: URL: https://github.com/apache/nuttx/pull/11245 -- 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:

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1400814880 ## include/nuttx/tee.h: ## @@ -0,0 +1,479 @@ +/ + * include/nuttx/tee.h + * + *

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1400814880 ## include/nuttx/tee.h: ## @@ -0,0 +1,479 @@ +/ + * include/nuttx/tee.h + * + *

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
hartmannathan commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1400804103 ## include/nuttx/tee.h: ## @@ -0,0 +1,479 @@ +/ + * include/nuttx/tee.h + * + *

[PR] arch/sim: sim support 16bbp [nuttx]

2023-11-21 Thread via GitHub
qinshijing opened a new pull request, #11246: URL: https://github.com/apache/nuttx/pull/11246 ## Summary Convert depth 32 to depth 16 in sim_x11framebuffer ## Impact Display of sim ## Testing ./tools/configure.sh -l sim:lvgl_fb Color depth select 16: RGB565

Re: [PR] driver/tee: add optee client driver module [nuttx]

2023-11-21 Thread via GitHub
hujun260 commented on code in PR #11228: URL: https://github.com/apache/nuttx/pull/11228#discussion_r1400592051 ## include/nuttx/tee.h: ## @@ -0,0 +1,479 @@ +/ + * include/nuttx/tee.h + * + * Copyright

[PR] arm64_vector: no need to save x0 to sp [nuttx]

2023-11-21 Thread via GitHub
leo11261 opened a new pull request, #11245: URL: https://github.com/apache/nuttx/pull/11245 ## Summary It will cause an incorrect sp value saved in context when saving X0 to sp. ## Impact ## Testing -- This is an automated message from the Apache Git Service. To

Re: [PR] arch/sim: sim support 16bbp [nuttx]

2023-11-21 Thread via GitHub
qinshijing closed pull request #11225: arch/sim: sim support 16bbp URL: https://github.com/apache/nuttx/pull/11225 -- 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

[PR] assert:read content of undefinedinsn address [nuttx]

2023-11-21 Thread via GitHub
leo11261 opened a new pull request, #11244: URL: https://github.com/apache/nuttx/pull/11244 ## Summary read content of undefinedinsn address to catch possibly ram bit flip ## Impact for debug ## Testing -- This is an automated message from the Apache Git Service. To

[PR] list: search prev item in reverse order [nuttx]

2023-11-21 Thread via GitHub
leo11261 opened a new pull request, #11243: URL: https://github.com/apache/nuttx/pull/11243 ## Summary list: search prev item in reverse order ## Impact no ## Testing internel test -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
crafcat7 commented on PR #11207: URL: https://github.com/apache/nuttx/pull/11207#issuecomment-1820897869 [PR##11242 - Revert "libc/lib_bzero:Add bzero prototype."](https://github.com/apache/nuttx/pull/11242) I reverted the original bzero function implementation to a macro -- This is

[PR] Revert "libc/lib_bzero:Add bzero prototype." [nuttx]

2023-11-21 Thread via GitHub
crafcat7 opened a new pull request, #11242: URL: https://github.com/apache/nuttx/pull/11242 ## Summary Solve the problem in PR#11207: When macOS is compiled to the implementation of bzero, recursion will occur ## Impact Change bzero from a function implementation to a macro

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
pussuw commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820890026 @SPRESENSE This should fix your issue https://github.com/apache/nuttx/pull/11241/ Sorry about the regression, did not test it well enough.. -- This is an automated

[PR] task/pthread_cancelpt: Fix task_delete from another task group [nuttx]

2023-11-21 Thread via GitHub
pussuw opened a new pull request, #11241: URL: https://github.com/apache/nuttx/pull/11241 ## Summary PR #11165 causes an unnecessary regression; task_delete no longer works, if the deleted task is from another group. The logic that prevents this comes from:

[PR] Rpmsgblk function optimization [nuttx]

2023-11-21 Thread via GitHub
leo11261 opened a new pull request, #11240: URL: https://github.com/apache/nuttx/pull/11240 ## Summary rpmsgblk function optimization ## Impact rpmsgblk ## Testing rpserver: CONFIG_BCH=y rpproxy: CONFIG_BCH=y CONFIG_ALLOW_MIT_COMPONENTS=y CONFIG_TESTING_CMOCKA=y

(nuttx) branch master updated: add support to q-enconder

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new bc64559bac add support to q-enconder

Re: [PR] risc-v/mpfs/opensbi: update opensbi to version 1.3.1 [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11233: URL: https://github.com/apache/nuttx/pull/11233 -- 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:

Re: [PR] Add support enconder to ESP32-2432S028 [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11229: URL: https://github.com/apache/nuttx/pull/11229 -- 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:

(nuttx) branch master updated: risc-v/mpfs/opensbi: update opensbi to version 1.3.1

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 1cb879773a risc-v/mpfs/opensbi: update

Re: [PR] clock.h: use CONFIG_DEBUG_SCHED to test init ticks [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 merged PR #11238: URL: https://github.com/apache/nuttx/pull/11238 -- 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:

(nuttx) branch master updated: clock.h: use CONFIG_DEBUG_SCHED to test init ticks

2023-11-21 Thread xiaoxiang
This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new 6a6538c1b8 clock.h: use CONFIG_DEBUG_SCHED to

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11207: URL: https://github.com/apache/nuttx/pull/11207#discussion_r1400469471 ## libs/libc/string/lib_bzero.c: ## @@ -36,5 +36,21 @@ void bzero(FAR void *s, size_t n) Review Comment: but can we change bzero to macro -- This is

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
pussuw commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820774823 Once the regression is patched I must think if this "peeking to another process's stack" is a security concern which requires re-designing how nxnotify_cancellation is handled.

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
pussuw commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820770939 Hi @SPRESENSE funnily enough, I found this exact line myself just 5 seconds ago :) I will think of a patch to fix this. The security test (groups are equal) does not make

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
SPRESENSE commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820766516 @pussuw Thanks for your investigation. My conditions are: * CONFIG_CANCELLATION_POINTS=n * CONFIG_SCHED_THREAD_LOCAL=n I found that `nxsched_get_stackinfo()`

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
pussuw commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820670872 One last thing that comes to mind, my assumption was that the stack frame created for TLS is memset to 0 always ``` int tls_init_info(FAR struct tcb_s *tcb) { FAR

Re: [PR] libc/lib_bzero:Add bzero prototype. [nuttx]

2023-11-21 Thread via GitHub
anjiahao1 commented on code in PR #8651: URL: https://github.com/apache/nuttx/pull/8651#discussion_r1400371751 ## libs/libc/string/lib_bzero.c: ## @@ -0,0 +1,40 @@ +/ + * libs/libc/string/lib_bzero.c +

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
pussuw commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820654033 @SPRESENSE sorry about the regression.. Yes, I'll start looking immediately. I can not find an example config with the sensor SENSORS_CXD5610_GNSS enabled, so a few questions:

Re: [PR] libc/lib_bzero:Add bzero prototype. [nuttx]

2023-11-21 Thread via GitHub
pkarashchenko commented on code in PR #8651: URL: https://github.com/apache/nuttx/pull/8651#discussion_r1400366941 ## libs/libc/string/lib_bzero.c: ## @@ -0,0 +1,40 @@ +/ + *

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
anjiahao1 commented on PR #11207: URL: https://github.com/apache/nuttx/pull/11207#issuecomment-1820641640 caused by [pr8651](https://github.com/apache/nuttx/pull/8651) When compiling macos clang, bezero(s,n) and memset(s,0,n) are equivalent, causing recursion. -- This is an automated

Re: [PR] sim/macos:fix system bzero recursive call [nuttx]

2023-11-21 Thread via GitHub
anjiahao1 closed pull request #11234: sim/macos:fix system bzero recursive call URL: https://github.com/apache/nuttx/pull/11234 -- 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.

Re: [PR] sim/macos:fix system bzero recursive call [nuttx]

2023-11-21 Thread via GitHub
anjiahao1 commented on PR #11234: URL: https://github.com/apache/nuttx/pull/11234#issuecomment-1820636816 > @anjiahao1 Could you move the commit to #11207? Using the same PR is easy to test. ok i move it ,and close this pr -- This is an automated message from the Apache Git

Re: [PR] sim/macos:fix system bzero recursive call [nuttx]

2023-11-21 Thread via GitHub
masayuki2009 commented on PR #11234: URL: https://github.com/apache/nuttx/pull/11234#issuecomment-1820628139 @anjiahao1 Could you move the commit to https://github.com/apache/nuttx/pull/11207? Using the same PR is easy to test. -- This is an automated message from the Apache

Re: [PR] sim/macos:fix system bzero recursive call [nuttx]

2023-11-21 Thread via GitHub
pkarashchenko commented on PR #11234: URL: https://github.com/apache/nuttx/pull/11234#issuecomment-1820586271 @xiaoxiang781216 could someone from your team take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] Fail to nxtask_delete by update of cancel point logic [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on issue #11236: URL: https://github.com/apache/nuttx/issues/11236#issuecomment-1820588238 @pussuw could you take a look? -- 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

Re: [PR] PoC: allow apps to customize NuttX build time variables [nuttx]

2023-11-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #11202: URL: https://github.com/apache/nuttx/pull/11202#discussion_r1400311455 ## arch/arm/src/Makefile: ## @@ -18,7 +18,7 @@ # -include $(TOPDIR)/Make.defs

Re: [PR] tcbinfo:remove total_num form tcbinfo. [nuttx]

2023-11-21 Thread via GitHub
anjiahao1 commented on code in PR #11207: URL: https://github.com/apache/nuttx/pull/11207#discussion_r1400312024 ## arch/arm/src/armv6-m/arm_tcbinfo.c: ## @@ -51,13 +51,14 @@ static const uint16_t g_reg_offs[] = TCB_REG_OFF(REG_R14), TCB_REG_OFF(REG_R15),

Re: [PR] sched: delete check when pick nexttcb in readytorun list [nuttx]

2023-11-21 Thread via GitHub
GUIDINGLI commented on PR #11226: URL: https://github.com/apache/nuttx/pull/11226#issuecomment-1820493411 > > @GUIDINGLI > > > The error is in step 6: > > > > > > Could you tell us how we can reproduce the error? > > Compile cmd: qemu-armv8a:nsh_smp change

(nuttx) branch master updated: poll: pollsetup should notify only one fd passd by caller

2023-11-21 Thread jerpelea
This is an automated email from the ASF dual-hosted git repository. jerpelea pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git The following commit(s) were added to refs/heads/master by this push: new eddd90de78 poll: pollsetup should notify only

Re: [PR] poll: pollsetup should notify only one fd passed by caller [nuttx]

2023-11-21 Thread via GitHub
jerpelea merged PR #11221: URL: https://github.com/apache/nuttx/pull/11221 -- 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: