[GitHub] [incubator-nuttx] btashton commented on a diff in pull request #7325: Revert "Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022""

btashton commented on code in PR #7325: URL: https://github.com/apache/incubator-nuttx/pull/7325#discussion_r996396060 ## tools/ci/docker/linux/Dockerfile: ## @@ -84,8 +85,10 @@ WORKDIR /tools FROM nuttx-toolchain-base AS nuttx-toolchain-arm # Download the latest ARM GCC toolc

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7325: Revert "Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022""

xiaoxiang781216 opened a new pull request, #7325: URL: https://github.com/apache/incubator-nuttx/pull/7325 Reverts apache/incubator-nuttx#7299 since the out of docker storage is fixed by https://github.com/apache/incubator-nuttx/pull/7323 -- This is an automated message from the Apache Gi

[incubator-nuttx] 01/01: Revert "Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022""

This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch revert-7299-22101301 in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 5f506bb64cc192fb66418792ee85b51ac620844f Author: Xiang Xiao AuthorDate: Sun Oct 16 14:16:54 20

[incubator-nuttx] branch revert-7299-22101301 created (now 5f506bb64c)

This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch revert-7299-22101301 in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git at 5f506bb64c Revert "Revert "tools/ci: patch for gcc-arm-none-eabi to 11.3.Rel1 August 08, 20

[GitHub] [incubator-nuttx] btashton commented on a diff in pull request #7294: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

btashton commented on code in PR #7294: URL: https://github.com/apache/incubator-nuttx/pull/7294#discussion_r996392161 ## tools/ci/docker/linux/Dockerfile: ## @@ -85,7 +86,9 @@ FROM nuttx-toolchain-base AS nuttx-toolchain-arm # Download the latest ARM GCC toolchain prebuilt by

[GitHub] [incubator-nuttx] btashton commented on pull request #7294: tools/ci: patch for docker gcc-arm-none-eabi to 11.3.Rel1 August 08, 2022

btashton commented on PR #7294: URL: https://github.com/apache/incubator-nuttx/pull/7294#issuecomment-1279896414 @anchao can you rebase this PR I think the issue should be resolved. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1351: testing: ostest: Add ASSERT in case of errors

xiaoxiang781216 merged PR #1351: URL: https://github.com/apache/incubator-nuttx-apps/pull/1351 -- 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-u

[incubator-nuttx-apps] branch master updated: testing: ostest: Add ASSERT in case of errors

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/incubator-nuttx-apps.git The following commit(s) were added to refs/heads/master by this push: new 97440889a testing: ostest: Add

[incubator-nuttx] branch master updated: ci: Only build docker image once.

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/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 9889b4b5c6 ci: Only build docker ima

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7323: ci: Only build docker image once.

xiaoxiang781216 merged PR #7323: URL: https://github.com/apache/incubator-nuttx/pull/7323 -- 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-unsubs

[incubator-nuttx] branch master updated (432c438c76 -> 1ab8b921f6)

This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 432c438c76 Use the correct marcro name for RP2040_UART0_BASE add 1ab8b921f6 nuttx/sched/sched: fix ru

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7322: nuttx/sched/sched: fix runtime error UBSan(division by zero)

xiaoxiang781216 merged PR #7322: URL: https://github.com/apache/incubator-nuttx/pull/7322 -- 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-unsubs

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7319: fs/procfs: Change FS_PROCFS_EXCLUDE_xxx default value to DEFAULT_SMALL

xiaoxiang781216 commented on code in PR #7319: URL: https://github.com/apache/incubator-nuttx/pull/7319#discussion_r996381986 ## fs/procfs/Kconfig: ## @@ -31,15 +31,15 @@ menu "Exclude individual procfs entries" config FS_PROCFS_EXCLUDE_PROCESS bool "Exclude process i

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7324: boards: Remove the duplicated prototype of CONFIG_INIT_ENTRYPOINT

xiaoxiang781216 opened a new pull request, #7324: URL: https://github.com/apache/incubator-nuttx/pull/7324 ## Summary Remove the duplication ## Impact code refactor only ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-nuttx] btashton commented on pull request #7323: ci: Only build docker image once.

btashton commented on PR #7323: URL: https://github.com/apache/incubator-nuttx/pull/7323#issuecomment-1279882669 Looks reasonable to me. We used 20GB with 11GB remaining on the host. If we had built twice I could see why we were getting into trouble. ![Screenshot from 2022-10-15 20-25-

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7321: Use the correct marcro name for RP2040_UART0_BASE

xiaoxiang781216 merged PR #7321: URL: https://github.com/apache/incubator-nuttx/pull/7321 -- 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-unsubs

[incubator-nuttx] branch master updated (85440f5db6 -> 432c438c76)

This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 85440f5db6 net/ioctl/cellular: remove redundant logic add 432c438c76 Use the correct marcro name for

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7323: ci: Only build docker image once.

xiaoxiang781216 commented on PR #7323: URL: https://github.com/apache/incubator-nuttx/pull/7323#issuecomment-1279882320 > @xiaoxiang781216 lets see if this helps. I noticed that the failed run you tagged me in was building the containers twice and the second build was the one running out of

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7319: fs/procfs: Change FS_PROCFS_EXCLUDE_xxx default value to DEFAULT_SMALL

xiaoxiang781216 commented on code in PR #7319: URL: https://github.com/apache/incubator-nuttx/pull/7319#discussion_r996381986 ## fs/procfs/Kconfig: ## @@ -31,15 +31,15 @@ menu "Exclude individual procfs entries" config FS_PROCFS_EXCLUDE_PROCESS bool "Exclude process i

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7319: fs/procfs: Change FS_PROCFS_EXCLUDE_xxx default value to DEFAULT_SMALL

xiaoxiang781216 commented on code in PR #7319: URL: https://github.com/apache/incubator-nuttx/pull/7319#discussion_r996381986 ## fs/procfs/Kconfig: ## @@ -31,15 +31,15 @@ menu "Exclude individual procfs entries" config FS_PROCFS_EXCLUDE_PROCESS bool "Exclude process i

[GitHub] [incubator-nuttx] btashton commented on pull request #7323: ci: Only build docker image once.

btashton commented on PR #7323: URL: https://github.com/apache/incubator-nuttx/pull/7323#issuecomment-1279880232 @xiaoxiang781216 lets see if this helps. I noticed that the failed run you tagged me in was building the containers twice and the second build was the one running out of disk s

[GitHub] [incubator-nuttx] btashton opened a new pull request, #7323: ci: Only build docker image once.

btashton opened a new pull request, #7323: URL: https://github.com/apache/incubator-nuttx/pull/7323 ## Summary When the CI job was migrated to the docker/build-push-action the push stage starts also performing the build. ## Impact Should result in us not building twice and runni

[GitHub] [incubator-nuttx] Junbo-Zheng opened a new pull request, #7322: nuttx/sched/sched: fix runtime error UBSan(division by zero)

Junbo-Zheng opened a new pull request, #7322: URL: https://github.com/apache/incubator-nuttx/pull/7322 ## Summary Fix runtime error Undefined Behavior Sanitizer(division by zero) NuttShell (NSH) NuttX-10.3.0 ap> ap> ap> uptime sched/sched_sysinfo.c:69:37: runtime err

[incubator-nuttx-website] branch asf-site updated: Publishing web: d696c67810809c3f81a4dd87b09d9e940db854b4 docs: 85440f5db6a8de75159360f067456a38870b941b

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/incubator-nuttx-website.git The following commit(s) were added to refs/heads/asf-site by this push: new eb62d61 Publishing web

[GitHub] [incubator-nuttx] ticso24 opened a new pull request, #7321: Use the correct marcro name for RP2040_UART0_BASE

ticso24 opened a new pull request, #7321: URL: https://github.com/apache/incubator-nuttx/pull/7321 ## Summary The RP2040_UART0_* register definitions use the wrong name for the register base. ## Impact If the macro gets used the compiler tries to resolve the non existing wrong

[GitHub] [incubator-nuttx] acassis opened a new pull request, #7320: Kconfig: use "default y if DEFAULT_SMALL" for readability

acassis opened a new pull request, #7320: URL: https://github.com/apache/incubator-nuttx/pull/7320 ## Summary Use "default y if DEFAULT_SMALL" for readability ## Impact It is easier to realize the intention when using "if SYMBOL" ## Testing N/A -- This is an automated me

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7319: fs/procfs: Change FS_PROCFS_EXCLUDE_xxx default value to DEFAULT_SMALL

acassis commented on code in PR #7319: URL: https://github.com/apache/incubator-nuttx/pull/7319#discussion_r996350720 ## fs/procfs/Kconfig: ## @@ -31,15 +31,15 @@ menu "Exclude individual procfs entries" config FS_PROCFS_EXCLUDE_PROCESS bool "Exclude process informati

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6965: nxmutex replace nxsem when used as a lock

pkarashchenko commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1279804156 > @pkarashchenko do you have more comment for this patch? Please give me 1 more day to take a final look. Then we can move forward -- This is an automated message f

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6965: nxmutex replace nxsem when used as a lock

xiaoxiang781216 commented on PR #6965: URL: https://github.com/apache/incubator-nuttx/pull/6965#issuecomment-1279803220 @pkarashchenko do you have more comment for this patch? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7319: fs/procfs: Change FS_PROCFS_EXCLUDE_xxx default value to DEFAULT_SMALL

xiaoxiang781216 opened a new pull request, #7319: URL: https://github.com/apache/incubator-nuttx/pull/7319 ## Summary ## Impact procfs default setting ## Testing Pass CI -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7317: Refine the implementation of rename

xiaoxiang781216 commented on code in PR #7317: URL: https://github.com/apache/incubator-nuttx/pull/7317#discussion_r996337998 ## fs/vfs/fs_rename.c: ## @@ -111,26 +111,21 @@ static int pseudorename(FAR const char *oldpath, FAR struct inode *oldinode, if (newinode->u.i_op

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1340: Fix the minor style issue

xiaoxiang781216 commented on PR #1340: URL: https://github.com/apache/incubator-nuttx-apps/pull/1340#issuecomment-1279796998 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 co

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7317: Refine the implementation of rename

xiaoxiang781216 commented on code in PR #7317: URL: https://github.com/apache/incubator-nuttx/pull/7317#discussion_r996337161 ## fs/vfs/fs_rename.c: ## @@ -90,7 +90,7 @@ static int pseudorename(FAR const char *oldpath, FAR struct inode *oldinode, { inode_rel

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7318: sched/env: Skip the envirnment variable duplication of kernel thread

xiaoxiang781216 commented on PR #7318: URL: https://github.com/apache/incubator-nuttx/pull/7318#issuecomment-1279782806 @masayuki2009 thanks for testing this patch. The problem should be fixed with the update. Could you verify again? -- This is an automated message from the Apache Git Ser

[GitHub] [incubator-nuttx-apps] pkarashchenko commented on pull request #1340: Fix the minor style issue

pkarashchenko commented on PR #1340: URL: https://github.com/apache/incubator-nuttx-apps/pull/1340#issuecomment-1279775369 @xiaoxiang781216 could you please take a look on CI failures? I restarted it couple of time, but failures persist -- This is an automated message from the Apache Git

[GitHub] [incubator-nuttx] acassis commented on a diff in pull request #7317: Refine the implementation of rename

acassis commented on code in PR #7317: URL: https://github.com/apache/incubator-nuttx/pull/7317#discussion_r996324092 ## fs/vfs/fs_rename.c: ## @@ -90,7 +90,7 @@ static int pseudorename(FAR const char *oldpath, FAR struct inode *oldinode, { inode_release(new

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1350: Refine nshlib implementation

xiaoxiang781216 commented on PR #1350: URL: https://github.com/apache/incubator-nuttx-apps/pull/1350#issuecomment-1279769726 > @xiaoxiang781216 Please rebase this PR to the latest master. Done. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-nuttx-apps] masayuki2009 commented on pull request #1350: Refine nshlib implementation

masayuki2009 commented on PR #1350: URL: https://github.com/apache/incubator-nuttx-apps/pull/1350#issuecomment-1279762881 @xiaoxiang781216 Please rebase this PR to the latest master. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx-apps] masayuki2009 opened a new pull request, #1351: testing: ostest: Add ASSERT in case of errors

masayuki2009 opened a new pull request, #1351: URL: https://github.com/apache/incubator-nuttx-apps/pull/1351 ## Summary - This commit adds ASSERT in case of errors to stop the execution of ostest - Also, add some 'ERROR' messages ## Impact - None ## Testing

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7318: sched/env: Skip the envirnment variable duplication of kernel thread

masayuki2009 commented on PR #7318: URL: https://github.com/apache/incubator-nuttx/pull/7318#issuecomment-1279760160 >Hmm, I noticed that 'hello' application not found with spresense:wifi_smp Of course, the latest upstream works without errors. -- This is an automated message fr

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #7318: sched/env: Skip the envirnment variable duplication of kernel thread

masayuki2009 commented on PR #7318: URL: https://github.com/apache/incubator-nuttx/pull/7318#issuecomment-1279759096 @xiaoxiang781216 Hmm, I noticed that 'hello' application not found with spresense:wifi_smp ``` NuttShell (NSH) NuttX-3.6.1 nsh> nsh> uname -a NuttX

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7318: sched/env: Skip the envirnment variable duplication of kernel thread

xiaoxiang781216 opened a new pull request, #7318: URL: https://github.com/apache/incubator-nuttx/pull/7318 ## Summary kernel thread shouldn't copy environment from parent task ## Impact kernel thread ## Testing Pass sim:ostest -- This is an automated message from

[incubator-nuttx] 02/02: net/ioctl/cellular: remove redundant logic

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/incubator-nuttx.git commit 85440f5db6a8de75159360f067456a38870b941b Author: chao an AuthorDate: Sat Oct 15 14:20:59 2022 +0800 net

[incubator-nuttx] 01/02: Add Cellular link layer support

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/incubator-nuttx.git commit ec4af0053877a500fc2a913b02e0c7bb26ba320f Author: luojun1 AuthorDate: Thu Jul 28 15:46:49 2022 +0800 Add

[incubator-nuttx] branch master updated (18ce70c1e7 -> 85440f5db6)

This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 18ce70c1e7 boards: qemu-armv8a: Fix pthread_attr_setstacksize failed in ostest new ec4af00538 Add Ce

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7314: Add Cellular link layer support

xiaoxiang781216 merged PR #7314: URL: https://github.com/apache/incubator-nuttx/pull/7314 -- 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-unsubs

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request, #1350: Refine nshlib implementation

xiaoxiang781216 opened a new pull request, #1350: URL: https://github.com/apache/incubator-nuttx-apps/pull/1350 ## Summary - nshlib: Call nsh_sysinitscript in usb or alt console like normal one - nshlib: Call nsh_telnetstart in nsh_initialize to avoid the dupliation ## Impact

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #7317: Refine the implementation of rename

xiaoxiang781216 opened a new pull request, #7317: URL: https://github.com/apache/incubator-nuttx/pull/7317 ## Summary - fs/vfs: Free subdir before allocate new one in pseudorename - fs/vfs: Remove the redundancy file name comparison in mountptrename ## Impact Minor

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #1349: Client & Server examples to use TCP/IP socket as an IPC Channel

xiaoxiang781216 commented on PR #1349: URL: https://github.com/apache/incubator-nuttx-apps/pull/1349#issuecomment-1279680896 @pedrobertoleti style issue still exist, you can run below command before updating patch: ../nuttx/tools/checkpatch.sh -g HEAD~...HEAD -- This is an automat