yf13

Results 53 comments of yf13
trafficstars

@anchao, @xiaoxiang781216 looks like the CI system is okay now. Please give this patch a review when free and let me know if there is anything I can do.

@xiaoxiang781216 looks like our CI got connectivity issue: ``` Configuration/Tool: ucans32k146/se05x,CONFIG_ARM_TOOLCHAIN_GNU_EABI ccuurrll:: ((2288)) FFaaiilleedd ttoo cocnonnencet ctto tgoi tghiutbh.ucbo.mc opmo rpto r4t4 3 4a4f3t earf t1e3r5 110365 2m4s4: mCso:n nCeocntinoenc...

@xiaoxiang781216 latest push also address the issue below, but somehow I couldn't reply directly from my browser. ![Screenshot_2024-06-14_08-04-11](https://github.com/apache/nuttx/assets/5899566/97f80390-fe00-4537-b8a7-4f975df33ca8) So basically latest update should have addressed all in-place comments, please help...

> what's benefit to reclaim the signal action pool? if it's important to safe the memory, it's simpler and fast to alloc/free one action from heap. @xiaoxiang781216 thanks for the...

> But the change makes nxsig_release_action is slower than kmm_free. If so, why not remove the pool and call kmm_malloc/kmm_free direcctly? > ... > please study how icmp manage the...

@xiaoxiang781216, sorry for my late response as I didn't see inline comments from `gh` command line. would you teach a use case where `static-plus-dynamic` approach is improper but `never return`...

@acassis, it seems [this job log](https://github.com/apache/nuttx/actions/runs/9431629406/job/25980516627?pr=12406) complains about the `stm3240g-eval/knxwm` config: ``` arm-none-eabi-ld: /github/workspace/sources/nuttx/nuttx section `.bss' will not fit in region `ksram' arm-none-eabi-ld: region `ksram' overflowed by 72 bytes ```...

@xiaoxiang781216 thanks for the comments. when `g_sigfreeactoin` is merged to the private `g_sigpool`, then how to judge preallocated item from `sig_action.c`? here are a few options I see: - still...

@xiaoxiang781216, an update has been pushed, it moves `g_sigactions[]` and relevant macro and initialization to `sig_action.c` and makes them private. thus the sharing issue is gone.

@xiaoxiang781216 and @acassis Looks like [CI system breaks](https://github.com/apache/nuttx/actions/runs/9301821482/job/25600866370)? ``` Configuration/Tool: smartl-c906/sotest ------------------------------------------------------------------------------------ Cleaning... Configuring... Building NuttX... riscv-none-elf-ld: dynload.o: ABI is incompatible with that of the selected emulation: target emulation `elf64-littleriscv'...