Re: [PR] sim: add assertions on hostfs copy of structures [nuttx]

2023-12-29 Thread via GitHub
yamt commented on code in PR #11458: URL: https://github.com/apache/nuttx/pull/11458#discussion_r1438495825 ## arch/sim/src/sim/sim_checknuttxtypes.c: ## @@ -0,0 +1,100 @@ +/ + * arch/sim/src/sim/sim_che

Re: [PR] sim: add assertions on hostfs copy of structures [nuttx]

2023-12-29 Thread via GitHub
yamt commented on code in PR #11458: URL: https://github.com/apache/nuttx/pull/11458#discussion_r1438495411 ## arch/sim/src/sim/sim_checknuttxtypes.c: ## @@ -0,0 +1,100 @@ +/ + * arch/sim/src/sim/sim_che

Re: [PR] sim: add assertions on hostfs copy of structures [nuttx]

2023-12-29 Thread via GitHub
yamt commented on code in PR #11458: URL: https://github.com/apache/nuttx/pull/11458#discussion_r1438494893 ## arch/sim/src/sim/sim_checknuttxtypes.c: ## @@ -0,0 +1,100 @@ +/ + * arch/sim/src/sim/sim_che

Re: [PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
yamt commented on PR #11466: URL: https://github.com/apache/nuttx/pull/11466#issuecomment-1872472492 > @yamt #10303 is needed to pass LTP test case. From https://en.wikipedia.org/wiki/Unix_time, most POSIX OS(except 32bit QNX) define time_t to signed integer, so it's better to follow others

Re: [PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
pkarashchenko commented on PR #11466: URL: https://github.com/apache/nuttx/pull/11466#issuecomment-1872470435 I mean, does the LTP test fail with unsigned 64-bit `time_t`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
pkarashchenko commented on PR #11466: URL: https://github.com/apache/nuttx/pull/11466#issuecomment-1872466067 Is the LTP test a part of the CI run? -- 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

[I] [help] confirming ostest output in KERNEL mode [nuttx-apps]

2023-12-29 Thread via GitHub
yf13 opened a new issue, #2246: URL: https://github.com/apache/nuttx-apps/issues/2246 Thanks for making `ostest` loadable in KERNEL mode NuttX! I run it with kernel build on CanMV230 and got the following dumps at the end: ``` Final memory usage:

Re: [PR] sim: add assertions on hostfs copy of structures [nuttx]

2023-12-29 Thread via GitHub
xiaoxiang781216 commented on code in PR #11458: URL: https://github.com/apache/nuttx/pull/11458#discussion_r1438470065 ## arch/sim/src/sim/sim_checknuttxtypes.c: ## @@ -0,0 +1,100 @@ +/ + * arch/sim/src/

Re: [PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
xiaoxiang781216 commented on PR #11466: URL: https://github.com/apache/nuttx/pull/11466#issuecomment-1872448504 @yamt https://github.com/apache/nuttx/pull/10303 is needed to pass LTP test case. -- This is an automated message from the Apache Git Service. To respond to the message, please

(nuttx-website) branch asf-site updated: Publishing web: 36ddd7c60a8230335eda886c909bc938739114a2 docs: 9b439a5c72b6e5bd941edd0195cacc22810ab5a6

2023-12-29 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 1dfea4fe Publishing web: 36ddd7c6

Re: [I] NSH: ps command broken between release 12.0 and 12.1 [nuttx]

2023-12-29 Thread via GitHub
tmedicci commented on issue #11463: URL: https://github.com/apache/nuttx/issues/11463#issuecomment-1872402943 Hi! PTAL at https://github.com/apache/nuttx/pull/11425 On ESP32-S2, the problem with the `ps` command was related to the NSH's task stack size. The error didn't occur e

Re: [I] NSH: ps command broken between release 12.0 and 12.1 [nuttx]

2023-12-29 Thread via GitHub
acassis commented on issue #11463: URL: https://github.com/apache/nuttx/issues/11463#issuecomment-1872374421 @maxikrie so maybe this is not the same issue we are seeing in our board. Confirming that ps on stm32f4discovery still working: ``` nsh> uname -a NuttX 12.3.0 ffc7a

Re: [PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
acassis commented on PR #11466: URL: https://github.com/apache/nuttx/pull/11466#issuecomment-1872373364 @yamt probably a mistake because for 32-bit he used uint32_t! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

(nuttx) branch master updated: add missing dependency to MM_KMAP for ARCH_KMAP_VBASE

2023-12-29 Thread acassis
This is an automated email from the ASF dual-hosted git repository. acassis 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 9b439a5c72 add missing dependency to MM_KMAP for

Re: [PR] add missing dependency to MM_KMAP for ARCH_KMAP_VBASE [nuttx]

2023-12-29 Thread via GitHub
acassis merged PR #11468: URL: https://github.com/apache/nuttx/pull/11468 -- 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.apac

Re: [I] NSH: ps command broken between release 12.0 and 12.1 [nuttx]

2023-12-29 Thread via GitHub
maxikrie commented on issue #11463: URL: https://github.com/apache/nuttx/issues/11463#issuecomment-1872303793 I tracked it down to changes introduced in nsh_console.c, nsh_console.h, nsh_proccmds.c between 12.0 and 12.1. If I revert only these files to 12.0, I don't see this issue. I

Re: [I] NSH: ps command broken between release 12.0 and 12.1 [nuttx]

2023-12-29 Thread via GitHub
acassis commented on issue #11463: URL: https://github.com/apache/nuttx/issues/11463#issuecomment-1872284788 @maxikrie I faced similar issue helping @JorgeGzm with witte board powered by stm32h753bi. I think it could be some issues related to STM32H7, because in other boards the issues does

[PR] add missing dependency to MM_KMAP for ARCH_KMAP_VBASE [nuttx]

2023-12-29 Thread via GitHub
yf13 opened a new pull request, #11468: URL: https://github.com/apache/nuttx/pull/11468 ## Summary This is to align with ARCH_KMAP_VBASE by source codes. It also fixes CI warnings/errors reported by `tools/refresh.sh`. ## Impact None ## Testing Check

Re: [PR] Add dependency for ARCH_KMAP_VBASE [nuttx]

2023-12-29 Thread via GitHub
yf13 closed pull request #11467: Add dependency for ARCH_KMAP_VBASE URL: https://github.com/apache/nuttx/pull/11467 -- 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 unsubscri

[PR] Add dependency for ARCH_KMAP_VBASE [nuttx]

2023-12-29 Thread via GitHub
yf13 opened a new pull request, #11467: URL: https://github.com/apache/nuttx/pull/11467 ## Summary adds missing dependency of ARCH_KMAP_VBASE to MM_KMAP per code review. this should also fix defconfig normalization warnings from `tools/refresh.sh` ## Impact None.

Re: [PR] adding kernel build for CanMV-K230 device [nuttx]

2023-12-29 Thread via GitHub
yf13 commented on code in PR #11465: URL: https://github.com/apache/nuttx/pull/11465#discussion_r1438161696 ## boards/risc-v/k230/canmv230/configs/knsh/defconfig: ## Review Comment: Sorry, my use of "tools/refresh.sh" must be wrong before sending the PR. just rerun it.

Re: [PR] adding kernel build for CanMV-K230 device [nuttx]

2023-12-29 Thread via GitHub
yf13 commented on code in PR #11465: URL: https://github.com/apache/nuttx/pull/11465#discussion_r1438161377 ## arch/risc-v/src/k230/k230_start.c: ## @@ -100,12 +91,23 @@ uintptr_t g_idle_topstack = K230_IDLESTACK_TOP; * Name: k230_start **

Re: [PR] adding kernel build for CanMV-K230 device [nuttx]

2023-12-29 Thread via GitHub
lupyuen commented on code in PR #11465: URL: https://github.com/apache/nuttx/pull/11465#discussion_r1438144694 ## boards/risc-v/k230/canmv230/configs/knsh/defconfig: ## Review Comment: Wonder if some of these settings might be redundant? ```text Normalize canmv2

Re: [PR] adding kernel build for CanMV-K230 device [nuttx]

2023-12-29 Thread via GitHub
lupyuen commented on code in PR #11465: URL: https://github.com/apache/nuttx/pull/11465#discussion_r1438141611 ## arch/risc-v/src/k230/k230_start.c: ## @@ -100,12 +91,23 @@ uintptr_t g_idle_topstack = K230_IDLESTACK_TOP; * Name: k230_start ***

Re: [PR] adding kernel build for CanMV-K230 device [nuttx]

2023-12-29 Thread via GitHub
lupyuen commented on PR #11465: URL: https://github.com/apache/nuttx/pull/11465#issuecomment-1871931246 Hi @yf13: We need some fixes here: ```text Configuration/Tool: canmv230/nsh Cleaning.

Re: [PR] sys/types: supporting 32-bit IDs for gid_t/uid_t [nuttx]

2023-12-29 Thread via GitHub
yamt commented on code in PR #10869: URL: https://github.com/apache/nuttx/pull/10869#discussion_r1438138798 ## include/sys/types.h: ## @@ -121,20 +121,27 @@ typedef uint16_t size_t; typedef int16_t ssize_t; typedef uint16_t rsize_t; +/* uid_t is used for user ID

[PR] make 64-bit time_t back to unsigned [nuttx]

2023-12-29 Thread via GitHub
yamt opened a new pull request, #11466: URL: https://github.com/apache/nuttx/pull/11466 ## Summary it has been changed by https://github.com/apache/nuttx/pull/10303 without any explicit mentions. i suppose it was a mistake. see https://github.com/apache/nuttx/pull/8201 and refe

(nuttx) branch master updated: log messages of binfmt are very much.

2023-12-29 Thread archer
This is an automated email from the ASF dual-hosted git repository. archer 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 675f775494 log messages of binfmt are very much.

Re: [PR] adjust log levels of a few binfmt logs [nuttx]

2023-12-29 Thread via GitHub
anchao merged PR #11464: URL: https://github.com/apache/nuttx/pull/11464 -- 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.apach

[PR] nshlib/ifconfig: Let "dns" option depends on `CONFIG_NETDB_DNSCLIENT` [nuttx-apps]

2023-12-29 Thread via GitHub
wengzhe opened a new pull request, #2245: URL: https://github.com/apache/nuttx-apps/pull/2245 ## Summary "dns" option of `ifconfig` can work just with `CONFIG_NETDB_DNSCLIENT`, no need to depend on `CONFIG_NETINIT_DNS` or `CONFIG_NETINIT_DHCPC`. ## Impact ifconfig xxx dns xxx

Re: [PR] Wasm.mk: disable by default [nuttx-apps]

2023-12-29 Thread via GitHub
no1wudi commented on code in PR #2065: URL: https://github.com/apache/nuttx-apps/pull/2065#discussion_r1438084238 ## interpreters/wamr/Kconfig: ## @@ -70,6 +70,11 @@ config INTERPRETERS_WAMR_LIBC_BUILTIN bool "Enable built-in libc" default n +config INTERPRETER