Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-04-09 Thread via GitHub
no1wudi closed pull request #15563: arch/risc-v: Implement up_this_task using Thread Pointer (TP) URL: https://github.com/apache/nuttx/pull/15563 -- 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

Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
pussuw commented on code in PR #15563: URL: https://github.com/apache/nuttx/pull/15563#discussion_r1916605359 ## arch/risc-v/src/common/riscv_macros.S: ## @@ -58,7 +58,9 @@ #ifdef RISCV_SAVE_GP REGSTORE x3, REG_X3(\in)/* gp */ #endif +#ifdef CONFIG_SCHED_THREAD_LOCAL

Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
pussuw commented on code in PR #15563: URL: https://github.com/apache/nuttx/pull/15563#discussion_r1916606055 ## arch/risc-v/include/irq.h: ## @@ -749,6 +749,34 @@ int up_this_cpu(void); * Inline Functions *

Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
pussuw commented on code in PR #15563: URL: https://github.com/apache/nuttx/pull/15563#discussion_r1916604161 ## arch/risc-v/src/common/riscv_initialstate.c: ## @@ -153,6 +153,8 @@ void up_initial_state(struct tcb_s *tcb) #ifdef CONFIG_SCHED_THREAD_LOCAL xcp->regs[REG_TP]

Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
pussuw commented on code in PR #15563: URL: https://github.com/apache/nuttx/pull/15563#discussion_r1916597216 ## arch/risc-v/include/irq.h: ## @@ -749,6 +749,34 @@ int up_this_cpu(void); * Inline Functions *

Re: [PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
nuttxpr commented on PR #15563: URL: https://github.com/apache/nuttx/pull/15563#issuecomment-2592227778 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) Yes, this PR appears to meet the NuttX requirements, al

[PR] arch/risc-v: Implement up_this_task using Thread Pointer (TP) [nuttx]

2025-01-15 Thread via GitHub
no1wudi opened a new pull request, #15563: URL: https://github.com/apache/nuttx/pull/15563 ## Summary - Added up_this_task() and up_update_task() macros to use TP register for fast task pointer access - Modified CPU startup and initial state code to initialize TP with task pointer