[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6376: nuttx/mutex: do not use non-nx interface in kernel except libs

2022-06-08 Thread GitBox
pkarashchenko commented on PR #6376: URL: https://github.com/apache/incubator-nuttx/pull/6376#issuecomment-1150705246 @xiaoxiang781216 ping -- 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 spe

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #6394: Fix the code which use sched_lock/sched_unlock to protect the resource

2022-06-08 Thread GitBox
xiaoxiang781216 commented on issue #6394: URL: https://github.com/apache/incubator-nuttx/issues/6394#issuecomment-1150694543 The same issue exist before: https://github.com/apache/incubator-nuttx/issues/3600. Let's close this one. -- This is an automated message from the Apache Git Servi

[GitHub] [incubator-nuttx] xiaoxiang781216 closed issue #6394: Fix the code which use sched_lock/sched_unlock to protect the resource

2022-06-08 Thread GitBox
xiaoxiang781216 closed issue #6394: Fix the code which use sched_lock/sched_unlock to protect the resource URL: https://github.com/apache/incubator-nuttx/issues/6394 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue, #6394: Fix the code which use sched_lock/sched_unlock to protect the resource

2022-06-08 Thread GitBox
xiaoxiang781216 opened a new issue, #6394: URL: https://github.com/apache/incubator-nuttx/issues/6394 Since it doesn't work with SMP config, the possible replacement is the critical section, spinlock and semaphore. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
xiaoxiang781216 commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1150691869 > @anchao Did you confirm that this PR works in SMP mode without any side effects? sched_lock/sched_unlock can't protect the resource in SMP mode. All code which de

[GitHub] [incubator-nuttx] anchao opened a new pull request, #6393: mqueue: minor code tuning of message queue to improve the performance

2022-06-08 Thread GitBox
anchao opened a new pull request, #6393: URL: https://github.com/apache/incubator-nuttx/pull/6393 ## Summary 1. sched/mqueue: replace inline linklist to improve performance 2. sched/mqueue: minor code tuning of message queue ``` remove unnecessary temporary variables ad

[GitHub] [incubator-nuttx] anchao commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
anchao commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1150642300 > @anchao > > You have just pushed several new commits which are not sched_lock-related changes. What is your intention for the commits? Other patches have some minor

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
masayuki2009 commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1150643596 >Other patches have some minor code tuning of message queue, mainly to improve the performance further,maybe I should create a new PR to track ? @anchao I prefer to

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
masayuki2009 commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1150638587 @anchao You have just pushed several new commits which are not sched_lock-related changes. What is your intention for the commits? -- This is an automated me

[GitHub] [incubator-nuttx] anchao commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
anchao commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1150635694 > Is it tested with multiple threads trying to receive data from same mqueue? I mean the concurrent environment test Yes, I conducted a local concurrency test on the same mqu

[incubator-nuttx-website] branch asf-site updated: Publishing web: 42e9e27963415ec1359277228669d6b9c71c48d1 docs: adc23911b7c8a83e58c4f06b8095ea8bf8d4d86c

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

[incubator-nuttx] branch master updated: arch/sparc/bm3803 fix use of uart1 and uart2

2022-06-08 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 adc23911b7 arch/sparc/bm3803 fix

[GitHub] [incubator-nuttx] pkarashchenko merged pull request #6390: arch/sparc/bm3803: fix the using of uart1 and uart2

2022-06-08 Thread GitBox
pkarashchenko merged PR #6390: URL: https://github.com/apache/incubator-nuttx/pull/6390 -- 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

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
pkarashchenko commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1149975516 Is it tested with multiple threads trying to receive data from same mqueue? I mean the concurrent environment test -- This is an automated message from the Apache Git Serv

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6389: libc/ipc: add ftok(3) support

2022-06-08 Thread GitBox
pkarashchenko commented on code in PR #6389: URL: https://github.com/apache/incubator-nuttx/pull/6389#discussion_r892379840 ## include/sys/ipc.h: ## @@ -35,10 +35,10 @@ /* Mode bits: The lower order 9-bit bits are the standard mode bits */ -#define IPC_MODE0x01ff /

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6389: libc/ipc: add ftok(3) support

2022-06-08 Thread GitBox
pkarashchenko commented on code in PR #6389: URL: https://github.com/apache/incubator-nuttx/pull/6389#discussion_r892379840 ## include/sys/ipc.h: ## @@ -35,10 +35,10 @@ /* Mode bits: The lower order 9-bit bits are the standard mode bits */ -#define IPC_MODE0x01ff /

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6389: libc/ipc: add ftok(3) support

2022-06-08 Thread GitBox
pkarashchenko commented on code in PR #6389: URL: https://github.com/apache/incubator-nuttx/pull/6389#discussion_r892378319 ## include/sys/types.h: ## @@ -158,7 +158,7 @@ typedef int id_t; * semaphores. A key is simply an integer of type key_t */ -typedef int16_t

[GitHub] [incubator-nuttx] anchao commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
anchao commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1149878382 > Yes, I verified this PR on sabre-6quad:smp and custom cortex-a7 dual core device (ostest) -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
masayuki2009 commented on PR #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392#issuecomment-1149862635 @anchao Did you confirm that this PR works in SMP mode without any side effects? -- This is an automated message from the Apache Git Service. To respond to the messa

[incubator-nuttx-apps] branch master updated: rename DISCLAIMER-WIP

2022-06-08 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 The following commit(s) were added to refs/heads/master by this push: new f55523038 rename DISCLAIMER-WIP

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1189: rename DISCLAIMER-WIP

2022-06-08 Thread GitBox
xiaoxiang781216 merged PR #1189: URL: https://github.com/apache/incubator-nuttx-apps/pull/1189 -- 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] branch master updated: rename DISCLAIMER-WIP

2022-06-08 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 87ee22b8d8 rename DISCLAIMER-WIP 87e

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6388: rename DISCLAIMER-WIP

2022-06-08 Thread GitBox
xiaoxiang781216 merged PR #6388: URL: https://github.com/apache/incubator-nuttx/pull/6388 -- 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: esp32c3-devkit/knsh: Enable CONFIG_RAW_BINARY to convert the ELF file.

2022-06-08 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 c23699be60 esp32c3-devkit/knsh: Enab

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6391: esp32c3-devkit/knsh: Enable CONFIG_RAW_BINARY to convert the ELF file.

2022-06-08 Thread GitBox
xiaoxiang781216 merged PR #6391: URL: https://github.com/apache/incubator-nuttx/pull/6391 -- 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] anchao opened a new pull request, #6392: sched/mqueue: remove sched_lock to improve performance

2022-06-08 Thread GitBox
anchao opened a new pull request, #6392: URL: https://github.com/apache/incubator-nuttx/pull/6392 ## Summary sched/mqueue: remove sched_lock to improve performance remove the sched_lock/unlock to improve the performance by 18% ``` mq_send Flow

[incubator-nuttx] 02/02: sched/tls: Wrap up_info_size into tls_info_size

2022-06-08 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 11c8f3125dadebd183d1add32e615d5292ccaff7 Author: Xiang Xiao AuthorDate: Sun Jun 5 19:17:33 2022 +0800 sc

[incubator-nuttx] branch master updated (2707ba58d2 -> 11c8f3125d)

2022-06-08 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git from 2707ba58d2 sched/tls: Shouldn't get tls info directly from sp in kernel space new 8543df7e4c sched/in

[incubator-nuttx] 01/02: sched/init: Initialize the idle thread local storage by tls_init_info

2022-06-08 Thread masayuki
This is an automated email from the ASF dual-hosted git repository. masayuki pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git commit 8543df7e4cabff026cc95e339639198c0e58229a Author: Xiang Xiao AuthorDate: Sun Jun 5 22:18:05 2022 +0800 sc

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #6365: sched/tls: Wrap up_info_size into tls_info_size

2022-06-08 Thread GitBox
masayuki2009 merged PR #6365: URL: https://github.com/apache/incubator-nuttx/pull/6365 -- 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.

[GitHub] [incubator-nuttx] Ouss4 opened a new pull request, #6391: esp32c3-devkit/knsh: Enable CONFIG_RAW_BINARY to convert the ELF file.

2022-06-08 Thread GitBox
Ouss4 opened a new pull request, #6391: URL: https://github.com/apache/incubator-nuttx/pull/6391 ## Summary The nuttx_user.bin file is needed later to combine the binaries. ## Impact N/A ## Testing ESP32-C3 -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-nuttx] zouboan commented on a diff in pull request #6390: arch/sparc/bm3803: fix the using of uart1 and uart2

2022-06-08 Thread GitBox
zouboan commented on code in PR #6390: URL: https://github.com/apache/incubator-nuttx/pull/6390#discussion_r892039994 ## arch/sparc/src/bm3803/bm3803-serial.c: ## @@ -238,11 +238,15 @@ static uart_dev_t g_uart1port = { .recv = { +.head = 0, +.tail = 0, Revie

[GitHub] [incubator-nuttx] pkarashchenko commented on a diff in pull request #6390: arch/sparc/bm3803: fix the using of uart1 and uart2

2022-06-08 Thread GitBox
pkarashchenko commented on code in PR #6390: URL: https://github.com/apache/incubator-nuttx/pull/6390#discussion_r892024239 ## arch/sparc/src/bm3803/bm3803-serial.c: ## @@ -238,11 +238,15 @@ static uart_dev_t g_uart1port = { .recv = { +.head = 0, +.tail = 0,

[GitHub] [incubator-nuttx] zouboan opened a new pull request, #6390: arch/sparc/bm3803: fix usinf\g of uart1 and uart2

2022-06-08 Thread GitBox
zouboan opened a new pull request, #6390: URL: https://github.com/apache/incubator-nuttx/pull/6390 ## Summary The TX/RX of uart1 and uart2 of bm3803 is multiplex with gpio, when using uart1 and uart2 of bm3803, the gpio direction register should be configured. ## Impact using of uar

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6376: nuttx/mutex: do not use non-nx interface in kernel except libs

2022-06-08 Thread GitBox
pkarashchenko commented on PR #6376: URL: https://github.com/apache/incubator-nuttx/pull/6376#issuecomment-1149544224 The issue of naming conflict comes from: `#include ` -> `#include ` -> `#include ` because `struct file_struct` has member `rmutex_t fs_lock;` -- This is an automated mes

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6376: nuttx/mutex: do not use non-nx interface in kernel except libs

2022-06-08 Thread GitBox
pkarashchenko commented on PR #6376: URL: https://github.com/apache/incubator-nuttx/pull/6376#issuecomment-1149540308 @xiaoxiang781216 seems that `mutex_` notation conflicts with 3P component naming. Since `mutex_` is not a POSIX APIs I'm not sure what is the best way to solve the naming co