[GitHub] [incubator-nuttx-apps] zhhyu7 opened a new pull request #1116: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 opened a new pull request #1116: URL: https://github.com/apache/incubator-nuttx-apps/pull/1116 ## Summary This PR mainly contains some USRSOCK related changes ## Impact This submission depends on the nuttx's submission! ## Testing -- This is an automated message

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request #5939: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 opened a new pull request #5939: URL: https://github.com/apache/incubator-nuttx/pull/5939 ## Summary This PR mainly contains some USRSOCK related changes ## Impact This submission depends on the app's submission! ## Testing -- This is an automated message from t

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085513376 @Donny9 I noticed that my automatic test failed because the smp application with esp32-devkitc:smp (QEMU) runs too slow. -- This is an automated mes

[GitHub] [incubator-nuttx] Donny9 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
Donny9 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085516053 > too slow ? busy loop because this pr ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085520139 >too slow ? busy loop because this pr ? @Donny9 Yes. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-nuttx] Donny9 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
Donny9 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085526980 > > too slow ? busy loop because this pr ? > > @Donny9 Yes. It looks very unlikely, nxsig_alloc_actionblock will alloc four new sigactq_t every time. --

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5782: RISC-V: Implement option to run NuttX in supervisor mode (S-mode)

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5782: URL: https://github.com/apache/incubator-nuttx/pull/5782#discussion_r840324059 ## File path: arch/risc-v/include/syscall.h ## @@ -122,50 +124,17 @@ #define SYS_signal_handler_return (7) #endif /* !CONFIG_BUILD_FLAT */

[GitHub] [incubator-nuttx] zhhyu7 commented on pull request #5939: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 commented on pull request #5939: URL: https://github.com/apache/incubator-nuttx/pull/5939#issuecomment-1085548328 depends on https://github.com/apache/incubator-nuttx-apps/pull/1116 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-nuttx-apps] zhhyu7 commented on pull request #1116: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 commented on pull request #1116: URL: https://github.com/apache/incubator-nuttx-apps/pull/1116#issuecomment-1085548023 depends on https://github.com/apache/incubator-nuttx/pull/5939 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-nuttx] no1wudi opened a new pull request #5940: arch/risc-v: Add handler for misaligned load/store

2022-04-01 Thread GitBox
no1wudi opened a new pull request #5940: URL: https://github.com/apache/incubator-nuttx/pull/5940 ## Summary Some risc-v based chips don't support unaligned data access, it will trigger a exception and the lead to crash. In this patch, we handle the misaligned access by software

[GitHub] [incubator-nuttx-apps] pkarashchenko opened a new pull request #1117: open() options: pass file options 0 if only ioctl is used

2022-04-01 Thread GitBox
pkarashchenko opened a new pull request #1117: URL: https://github.com/apache/incubator-nuttx-apps/pull/1117 ## Summary Pass file options 0 if only ioctl is used. if write or read handler is not available then `-EACCES` is returned by `inode_checkflags` ## Impact Fix cfgdata an

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #5939: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
xiaoxiang781216 commented on pull request #5939: URL: https://github.com/apache/incubator-nuttx/pull/5939#issuecomment-1085607195 @masayuki2009 could you try this patch series with your gs2200m module? -- This is an automated message from the Apache Git Service. To respond to the message,

[incubator-nuttx] branch master updated (bc0ca51 -> 4e581f7)

2022-04-01 Thread xiaoxiang
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 bc0ca51 netdev_file_ioctl: Fix fcntl F_SETFL O_NONBLOCK regression add 4e581f7 nuttx: Add "#includ

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5932: nuttx: Add "#include " to tioctl.h

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5932: URL: https://github.com/apache/incubator-nuttx/pull/5932 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5934: fs/unionfs: fix some warning and bug

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5934: URL: https://github.com/apache/incubator-nuttx/pull/5934 -- 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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5933: netdev_file_ioctl: Fix fcntl F_SETFL O_NONBLOCK regression

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5933: URL: https://github.com/apache/incubator-nuttx/pull/5933 -- 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

[incubator-nuttx] branch master updated (37ef22b -> bc0ca51)

2022-04-01 Thread xiaoxiang
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 37ef22b sigact: get free sigact from freelist always successful add bc0ca51 netdev_file_ioctl: Fix

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5937: fs/epoll: support EPOLL full events

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5937: URL: https://github.com/apache/incubator-nuttx/pull/5937#discussion_r840361060 ## File path: fs/vfs/Kconfig ## @@ -60,3 +60,9 @@ config TIMER_FD_NPOLLWAITERS Maximum number of threads that can be waiti

[GitHub] [incubator-nuttx] xiaoxiang781216 closed issue #5843: Segger System View support on different platforms setup documentation.

2022-04-01 Thread GitBox
xiaoxiang781216 closed issue #5843: URL: https://github.com/apache/incubator-nuttx/issues/5843 -- 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: com

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5930: SEGGER_RTT: solve compile error when enable segger rtt in armv7m.

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5930: URL: https://github.com/apache/incubator-nuttx/pull/5930 -- 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

[incubator-nuttx] branch master updated (4e581f7 -> a91d143)

2022-04-01 Thread xiaoxiang
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 4e581f7 nuttx: Add "#include " to tioctl.h add 4631fac fs/unionfs: fix bug about use after free ab

[incubator-nuttx] branch master updated (a91d143 -> 1b6ce6b)

2022-04-01 Thread xiaoxiang
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 a91d143 fs/unionfs: add sanity check for parameter add 1b6ce6b SEGGER_RTT: solve compile error whe

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840367589 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dum

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840368303 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dum

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840368865 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dum

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840370220 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dum

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840370220 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dum

[GitHub] [incubator-nuttx] anchao commented on a change in pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
anchao commented on a change in pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915#discussion_r840373544 ## File path: include/nuttx/sched_note.h ## @@ -105,16 +105,38 @@ memset((s), 0, sizeof(struct note_filter_irq_s)) #endif -/* Note dump modul

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085638537 > I noticed that my automatic test failed because the smp application with esp32-devkitc:smp (QEMU) runs too slow. @Ouss4 Can you try the latest master a

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #1114: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #1114: URL: https://github.com/apache/incubator-nuttx-apps/pull/1114#discussion_r840246478 ## File path: system/trace/trace_dump.c ## @@ -140,6 +142,7 @@ static void trace_dump_init_context(FAR struct trace_dump_context_s

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5938: fs/procfstcbinfo: minor bug about print address

2022-04-01 Thread GitBox
pkarashchenko merged pull request #5938: URL: https://github.com/apache/incubator-nuttx/pull/5938 -- 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:

[incubator-nuttx] branch master updated: fs/procfstcbinfo: minor bug about print address

2022-04-01 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko 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 b5fb93a fs/procfstcbinfo: minor

[incubator-nuttx] branch master updated (b5fb93a -> 8a80a85)

2022-04-01 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git. from b5fb93a fs/procfstcbinfo: minor bug about print address add 8a80a85 fs/romfs: fix string overf

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5936: fs/romfs: fix string overflow

2022-04-01 Thread GitBox
pkarashchenko merged pull request #5936: URL: https://github.com/apache/incubator-nuttx/pull/5936 -- 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:

[incubator-nuttx] branch master updated: fs and unistd: increase OPEN_MAX by claiming a Kconfig.

2022-04-01 Thread pkarashchenko
This is an automated email from the ASF dual-hosted git repository. pkarashchenko 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 463a437 fs and unistd: increase

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #5924: fs and unistd: increase OPEN_MAX by claiming a Kconfig.

2022-04-01 Thread GitBox
pkarashchenko merged pull request #5924: URL: https://github.com/apache/incubator-nuttx/pull/5924 -- 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:

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
Ouss4 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085665524 @masayuki2009 I just looked at our pipelines, all the ESP32 configs are passing. (in somewhat unrelated note, ESP32-C3 is failing, all the new RISC-V changes must

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #5940: arch/risc-v: Add handler for misaligned load/store

2022-04-01 Thread GitBox
Ouss4 commented on pull request #5940: URL: https://github.com/apache/incubator-nuttx/pull/5940#issuecomment-1085667021 I just saw that our C3 tests are failing with `up_assert: Assertion failed at file:common/riscv_exception.c line: 8`. Does this fix it? -- This is an automated message

[GitHub] [incubator-nuttx-apps] anchao commented on a change in pull request #1114: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
anchao commented on a change in pull request #1114: URL: https://github.com/apache/incubator-nuttx-apps/pull/1114#discussion_r840396999 ## File path: system/trace/trace_dump.c ## @@ -140,6 +142,7 @@ static void trace_dump_init_context(FAR struct trace_dump_context_s *ctx,

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085671211 >@masayuki2009 I just looked at our pipelines, all the ESP32 configs are passing. How about the performance of the smp application with esp32-devkitc:smp on

[GitHub] [incubator-nuttx] no1wudi commented on pull request #5940: arch/risc-v: Add handler for misaligned load/store

2022-04-01 Thread GitBox
no1wudi commented on pull request #5940: URL: https://github.com/apache/incubator-nuttx/pull/5940#issuecomment-1085677797 > I just saw that our C3 tests are failing with `up_assert: Assertion failed at file:common/riscv_exception.c line: 8`. Does this fix it? No, that might be anothe

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5939: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5939: URL: https://github.com/apache/incubator-nuttx/pull/5939#issuecomment-1085680301 @zhhyu7 Please divide this PR into 2 PRs. The first PR contains usersock related changes. The second PR contains ICMP/UDP/TCP changes. -- This is an au

[incubator-nuttx-apps] 03/06: rptun: add rptun dump support

2022-04-01 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/incubator-nuttx-apps.git commit e0e5652de42f39d0122d03f0ddf3ecee3e8a7469 Author: ligd AuthorDate: Wed Jan 19 21:07:59 2022 +0800 r

[incubator-nuttx-apps] 02/06: apps: add rptun reset /dev/rptun/xx cmd

2022-04-01 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/incubator-nuttx-apps.git commit d5aae96523de34812934f467ef8e02cc5f3da868 Author: ligd AuthorDate: Fri Dec 24 17:18:56 2021 +0800 a

[incubator-nuttx-apps] 04/06: rptun cmd: add broadcast rptun cmd

2022-04-01 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/incubator-nuttx-apps.git commit 622784be89e658e7a3b3b326d3e876d38bd4685f Author: ligd AuthorDate: Fri Feb 11 21:05:08 2022 +0800 r

[incubator-nuttx-apps] 05/06: rptun: add rptun ping support

2022-04-01 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/incubator-nuttx-apps.git commit f5d2c645561aeb2523e069de5df56243f9bc6236 Author: ligd AuthorDate: Thu Mar 3 18:05:48 2022 +0800 rp

[incubator-nuttx-apps] branch master updated (de61706 -> 4701742)

2022-04-01 Thread xiaoxiang
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-apps.git. from de61706 nsh/memdump: support dynamic turn on/off backtrace in heap new c57677c apps: add rptu

[incubator-nuttx-apps] 06/06: rptun: update rptun cmd usage

2022-04-01 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/incubator-nuttx-apps.git commit 470174260c363ea636640a85e2a2c412de5ea1dc Author: ligd AuthorDate: Thu Mar 31 20:11:09 2022 +0800 r

[incubator-nuttx-apps] 01/06: apps: add rptun /dev/rptun/xx panic cmd

2022-04-01 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/incubator-nuttx-apps.git commit c57677c3ced5e0e0c5c299c0830aa9cabb4a383b Author: ligd AuthorDate: Wed Dec 22 19:50:48 2021 +0800 a

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1099: rptun cmd update

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #1099: URL: https://github.com/apache/incubator-nuttx-apps/pull/1099 -- 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

[GitHub] [incubator-nuttx] Ouss4 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
Ouss4 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085683656 I just tried locally as well. I didn't notice any performance issues. Is it timing out for your test? -- This is an automated message from the Apache Git Service. To

[incubator-nuttx] branch master updated: rpmsgfs: do NOT access the pointer when do remote ioctl

2022-04-01 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/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 6f21065 rpmsgfs: do NOT access the

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5844: rpmsgfs: do NOT access the pointer when do remote ioctl

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5844: URL: https://github.com/apache/incubator-nuttx/pull/5844 -- 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

[GitHub] [incubator-nuttx] Donny9 commented on a change in pull request #5937: fs/epoll: support EPOLL full events

2022-04-01 Thread GitBox
Donny9 commented on a change in pull request #5937: URL: https://github.com/apache/incubator-nuttx/pull/5937#discussion_r840421468 ## File path: fs/vfs/Kconfig ## @@ -60,3 +60,9 @@ config TIMER_FD_NPOLLWAITERS Maximum number of threads that can be waiting on pol

[GitHub] [incubator-nuttx] anchao opened a new pull request #5941: arch/xtensa: add syscall note support in the flat build

2022-04-01 Thread GitBox
anchao opened a new pull request #5941: URL: https://github.com/apache/incubator-nuttx/pull/5941 ## Summary arch/xtensa: add syscall note support in the flat build ## Impact N/A ## Testing trace syscall on HIFI4 -- This is an automated message from the A

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5927: Minor improve for mmcsd

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5927: URL: https://github.com/apache/incubator-nuttx/pull/5927#discussion_r840434040 ## File path: include/nuttx/sdio.h ## @@ -539,6 +539,24 @@ #define SDIO_CLOCK(dev,rate) ((dev)->clock(dev,rate)) +/

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5927: Minor improve for mmcsd

2022-04-01 Thread GitBox
pkarashchenko commented on pull request #5927: URL: https://github.com/apache/incubator-nuttx/pull/5927#issuecomment-1085710956 In general looks good -- 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

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5926: local socket: cancel assert about backlog exceed 255

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5926: URL: https://github.com/apache/incubator-nuttx/pull/5926#discussion_r840436563 ## File path: net/local/local_listen.c ## @@ -96,7 +96,11 @@ int local_listen(FAR struct socket *psock, int backlog) /* Set the backlog

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5935: fs/rpmsgfs: fix bug about using uninit variable "times"

2022-04-01 Thread GitBox
pkarashchenko commented on pull request #5935: URL: https://github.com/apache/incubator-nuttx/pull/5935#issuecomment-1085719183 Please fix similar issue in `arch/sim/src/sim/up_hostfs.c` as well -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #1114: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #1114: URL: https://github.com/apache/incubator-nuttx-apps/pull/1114#discussion_r840441662 ## File path: system/trace/trace.c ## @@ -142,10 +142,22 @@ static int trace_cmd_start(int index, int argc, FAR char **argv, stati

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5931: fix: pthread_rwlock_timedwrlock.

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5931: URL: https://github.com/apache/incubator-nuttx/pull/5931#discussion_r840448377 ## File path: sched/signal/sig_dispatch.c ## @@ -424,7 +424,18 @@ int nxsig_tcbdispatch(FAR struct tcb_s *stcb, siginfo_t *info) i

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5925: sigact: get free sigact from freelist always successful

2022-04-01 Thread GitBox
masayuki2009 commented on pull request #5925: URL: https://github.com/apache/incubator-nuttx/pull/5925#issuecomment-1085731051 >I just tried locally as well. I didn't notice any performance issues. Is it timing out for your test? @Ouss4 Thanks for your comments. It seems that

[GitHub] [incubator-nuttx] Donny9 commented on pull request #5935: fs/rpmsgfs: fix bug about using uninit variable "times"

2022-04-01 Thread GitBox
Donny9 commented on pull request #5935: URL: https://github.com/apache/incubator-nuttx/pull/5935#issuecomment-1085736516 > Please fix similar issue in `arch/sim/src/sim/up_hostfs.c` as well > Please fix similar issue in `arch/sim/src/sim/up_hostfs.c` as well Done! Thank

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5926: local socket: cancel assert about backlog exceed 255

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5926: URL: https://github.com/apache/incubator-nuttx/pull/5926#discussion_r840459876 ## File path: net/local/local_listen.c ## @@ -96,7 +96,11 @@ int local_listen(FAR struct socket *psock, int backlog) /* Set the backl

[GitHub] [incubator-nuttx] Ouss4 opened a new issue #5942: ESP32-C3 Broken

2022-04-01 Thread GitBox
Ouss4 opened a new issue #5942: URL: https://github.com/apache/incubator-nuttx/issues/5942 Looks like recent RISC-V changes broke ESP32-C3 (at least), getting this error in a loop: ``` up_assert: Assertion failed at file:common/riscv_exception.c line: 88 task: Idle Task riscv_exce

[GitHub] [incubator-nuttx] zhhyu7 closed pull request #5939: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 closed pull request #5939: URL: https://github.com/apache/incubator-nuttx/pull/5939 -- 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

[GitHub] [incubator-nuttx] zhhyu7 commented on pull request #5943: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 commented on pull request #5943: URL: https://github.com/apache/incubator-nuttx/pull/5943#issuecomment-1085774101 depends on https://github.com/apache/incubator-nuttx-apps/pull/1116 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request #5943: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 opened a new pull request #5943: URL: https://github.com/apache/incubator-nuttx/pull/5943 ## Summary This PR mainly contains some USRSOCK related changes ## Impact This submission depends on the app's submission! ## Testing -- This is an automated message from t

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request #5944: net: tcp/udp/icmp/icmpv6 add FIONSPACE support

2022-04-01 Thread GitBox
zhhyu7 opened a new pull request #5944: URL: https://github.com/apache/incubator-nuttx/pull/5944 Signed-off-by: zhanghongyu ## Summary net: tcp/udp/icmp/icmpv6 add FIONSPACE support ## Impact ## Testing -- This is an automated message from the Apache Git Servi

[GitHub] [incubator-nuttx] zhhyu7 commented on pull request #5943: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 commented on pull request #5943: URL: https://github.com/apache/incubator-nuttx/pull/5943#issuecomment-1085778939 depends on https://github.com/apache/incubator-nuttx/pull/5944 too. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-nuttx-apps] zhhyu7 commented on pull request #1116: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 commented on pull request #1116: URL: https://github.com/apache/incubator-nuttx-apps/pull/1116#issuecomment-1085779146 depends on https://github.com/apache/incubator-nuttx/pull/5944 too. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-nuttx-apps] zhhyu7 edited a comment on pull request #1116: This PR mainly contains some USRSOCK related changes

2022-04-01 Thread GitBox
zhhyu7 edited a comment on pull request #1116: URL: https://github.com/apache/incubator-nuttx-apps/pull/1116#issuecomment-1085548023 depends on https://github.com/apache/incubator-nuttx/pull/5943 -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-nuttx] zhhyu7 opened a new pull request #5945: fix:Remove udp send large pkt assert

2022-04-01 Thread GitBox
zhhyu7 opened a new pull request #5945: URL: https://github.com/apache/incubator-nuttx/pull/5945 Signed-off-by: zhanghongyu ## Summary ## Impact ## Testing -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [incubator-nuttx] Ouss4 opened a new pull request #5946: arch/xtensa: Add xtensa_tcbinfo struct that contains helpful offsets.

2022-04-01 Thread GitBox
Ouss4 opened a new pull request #5946: URL: https://github.com/apache/incubator-nuttx/pull/5946 ## Summary Add Xtensa's version of tcbinfo. ## Impact None. ## Testing A WIP on OpenOCD, this version only needs the first offset for registers, so I didn't pay too much attention

[GitHub] [incubator-nuttx] XinStellaris opened a new pull request #5947: unistd:rename CONFIG_OPEN_MAX to CONFIG_LIBC_OPEN_MAX

2022-04-01 Thread GitBox
XinStellaris opened a new pull request #5947: URL: https://github.com/apache/incubator-nuttx/pull/5947 Signed-off-by: 田昕 ## Summary Rename to make its meaning clear(which is used by libc/unistd) ## Impact N/A ## Testing -- This is an automated message from the

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5945: fix:Remove udp send large pkt assert

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5945: URL: https://github.com/apache/incubator-nuttx/pull/5945#discussion_r840509385 ## File path: net/devif/devif_iobsend.c ## @@ -55,7 +55,11 @@ void devif_iob_send(FAR struct net_driver_s *dev, FAR struct iob_s *iob,

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5937: fs/epoll: change type of eventset from uint8_t to uint32_t

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5937: URL: https://github.com/apache/incubator-nuttx/pull/5937#discussion_r840524829 ## File path: arch/arm/src/cxd56xx/cxd56_geofence.c ## @@ -450,7 +450,7 @@ static void cxd56_geofence_sighandler(uint32_t data, FAR void *u

[GitHub] [incubator-nuttx] anjiahao1 opened a new pull request #5948: libc/lib_localtime:fix deadcode

2022-04-01 Thread GitBox
anjiahao1 opened a new pull request #5948: URL: https://github.com/apache/incubator-nuttx/pull/5948 isdst always is false. remove it Signed-off-by: anjiahao -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-nuttx] anjiahao1 opened a new pull request #5949: lib_chdir:change ret judgment method

2022-04-01 Thread GitBox
anjiahao1 opened a new pull request #5949: URL: https://github.com/apache/incubator-nuttx/pull/5949 don't use ret!=0,just use ret Signed-off-by: anjiahao -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-nuttx] anjiahao1 opened a new pull request #5950: libs/libc/lib_err.c:add va_end to vwarn

2022-04-01 Thread GitBox
anjiahao1 opened a new pull request #5950: URL: https://github.com/apache/incubator-nuttx/pull/5950 call va_copy,after need call va_end Signed-off-by: anjiahao -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [incubator-nuttx] anjiahao1 opened a new pull request #5951: modlib/modlib_bind:fix unsigned_compare with zero

2022-04-01 Thread GitBox
anjiahao1 opened a new pull request #5951: URL: https://github.com/apache/incubator-nuttx/pull/5951 r_offset is a unsigned type Signed-off-by: anjiahao -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5901: power: unify lock sates and move pm_auto_update to outer dir

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5901: URL: https://github.com/apache/incubator-nuttx/pull/5901#discussion_r840528348 ## File path: drivers/power/pm_lock.c ## @@ -0,0 +1,83 @@ +/ + *

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5951: modlib/modlib_bind:fix unsigned_compare with zero

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5951: URL: https://github.com/apache/incubator-nuttx/pull/5951#discussion_r840529135 ## File path: libs/libc/modlib/modlib_bind.c ## @@ -297,7 +297,7 @@ static int modlib_relocate(FAR struct module_s *modp, /* Calcul

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5949: lib_chdir:change ret judgment method

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5949: URL: https://github.com/apache/incubator-nuttx/pull/5949#discussion_r840530634 ## File path: libs/libc/unistd/lib_chdir.c ## @@ -115,7 +115,7 @@ int chdir(FAR const char *path) /* Verify that 'path' refers to a direc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5901: power: unify lock sates and move pm_auto_update to outer dir

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5901: URL: https://github.com/apache/incubator-nuttx/pull/5901#discussion_r840536482 ## File path: drivers/power/pm_lock.c ## @@ -0,0 +1,83 @@ +/ +

[GitHub] [incubator-nuttx] anjiahao1 commented on a change in pull request #5949: lib_chdir:change ret judgment method

2022-04-01 Thread GitBox
anjiahao1 commented on a change in pull request #5949: URL: https://github.com/apache/incubator-nuttx/pull/5949#discussion_r840536497 ## File path: libs/libc/unistd/lib_chdir.c ## @@ -115,7 +115,7 @@ int chdir(FAR const char *path) /* Verify that 'path' refers to a directory

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5948: libc/lib_localtime:fix deadcode

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5948: URL: https://github.com/apache/incubator-nuttx/pull/5948#discussion_r840536668 ## File path: libs/libc/time/lib_localtime.c ## @@ -1585,14 +1583,7 @@ static int tzparse(FAR const char *name, FAR struct state_s *sp,

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5951: modlib/modlib_bind:fix unsigned_compare with zero

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5951: URL: https://github.com/apache/incubator-nuttx/pull/5951#discussion_r840537177 ## File path: libs/libc/modlib/modlib_bind.c ## @@ -485,7 +485,7 @@ static int modlib_relocateadd(FAR struct module_s *modp, /* Cal

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5949: lib_chdir:change ret judgment method

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5949: URL: https://github.com/apache/incubator-nuttx/pull/5949#discussion_r840537966 ## File path: libs/libc/unistd/lib_chdir.c ## @@ -115,7 +115,7 @@ int chdir(FAR const char *path) /* Verify that 'path' refers to a direc

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5927: Minor improve for mmcsd

2022-04-01 Thread GitBox
xiaoxiang781216 commented on a change in pull request #5927: URL: https://github.com/apache/incubator-nuttx/pull/5927#discussion_r840548554 ## File path: include/nuttx/sdio.h ## @@ -539,6 +539,24 @@ #define SDIO_CLOCK(dev,rate) ((dev)->clock(dev,rate)) +/**

[GitHub] [incubator-nuttx] anjiahao1 commented on a change in pull request #5948: libc/lib_localtime:fix deadcode

2022-04-01 Thread GitBox
anjiahao1 commented on a change in pull request #5948: URL: https://github.com/apache/incubator-nuttx/pull/5948#discussion_r840550535 ## File path: libs/libc/time/lib_localtime.c ## @@ -1585,14 +1583,7 @@ static int tzparse(FAR const char *name, FAR struct state_s *sp,

[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5948: libc/lib_localtime:fix deadcode

2022-04-01 Thread GitBox
pkarashchenko commented on a change in pull request #5948: URL: https://github.com/apache/incubator-nuttx/pull/5948#discussion_r840551170 ## File path: libs/libc/time/lib_localtime.c ## @@ -1585,14 +1583,7 @@ static int tzparse(FAR const char *name, FAR struct state_s *sp,

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5935: fs/rpmsgfs: fix bug about using uninit variable "times"

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5935: URL: https://github.com/apache/incubator-nuttx/pull/5935 -- 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

[incubator-nuttx] branch master updated: fs/rpmsgfs: fix bug about using uninit variable "times"

2022-04-01 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/incubator-nuttx.git The following commit(s) were added to refs/heads/master by this push: new 0a111b7 fs/rpmsgfs: fix bug about u

[GitHub] [incubator-nuttx] Donny9 commented on a change in pull request #5937: fs/epoll: change type of eventset from uint8_t to uint32_t

2022-04-01 Thread GitBox
Donny9 commented on a change in pull request #5937: URL: https://github.com/apache/incubator-nuttx/pull/5937#discussion_r840563961 ## File path: arch/arm/src/cxd56xx/cxd56_geofence.c ## @@ -450,7 +450,7 @@ static void cxd56_geofence_sighandler(uint32_t data, FAR void *userdata

[incubator-nuttx] branch master updated (0a111b7 -> 253562f)

2022-04-01 Thread xiaoxiang
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 0a111b7 fs/rpmsgfs: fix bug about using uninit variable "times" add 253562f arch/xtensa: add sysca

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5941: arch/xtensa: add syscall note support in the flat build

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5941: URL: https://github.com/apache/incubator-nuttx/pull/5941 -- 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

[GitHub] [incubator-nuttx-apps] acassis commented on pull request #1117: open() options: pass file options 0 if only ioctl is used

2022-04-01 Thread GitBox
acassis commented on pull request #1117: URL: https://github.com/apache/incubator-nuttx-apps/pull/1117#issuecomment-1085877168 Hi @pkarashchenko I saw your message in the mailing list, but I still thinking that passing 0 to open is wrong. Please read the man pages: $ man 2 open ...

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5915: sched/note: add support of trace section mark

2022-04-01 Thread GitBox
xiaoxiang781216 merged pull request #5915: URL: https://github.com/apache/incubator-nuttx/pull/5915 -- 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

[incubator-nuttx] branch master updated (253562f -> 6e0ba2b)

2022-04-01 Thread xiaoxiang
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 253562f arch/xtensa: add syscall note support in the flat build add 6c86e77 sched/note: unify the

  1   2   >