nuttx icon indicating copy to clipboard operation
nuttx copied to clipboard

net/can: add write buffer to support nonblock send

Open zhhyu7 opened this issue 2 weeks ago • 9 comments

Summary

add write buffer to support nonblock send, and improve the processing of some code.

Impact

Partial return values of can socket bind, The sending method of can_sendmsg.

Testing

INFINEON board. After the can socket is bound to the CAN network card, data is sent and received in nonblock mode.

zhhyu7 avatar Dec 05 '25 15:12 zhhyu7

@simbit18 seems like github is having difficulties:

error: failed retrieving file 'vim-9.1.1623-1-x86_64.pkg.tar.zst.sig' from ftp2.osuosl.org : Operation too slow. Less than 1 bytes/sec transferred the last 10 seconds error: failed retrieving file 'ninja-1.13.1-1-x86_64.pkg.tar.zst' from mirror.msys2.org : Operation too slow. Less than 1 bytes/sec transferred the last 10 seconds warning: failed to retrieve some files error: failed to commit transaction (unexpected error) Errors occurred, no packages were upgraded. Error: The process 'C:\Windows\system32\cmd.exe' failed with exit code 1

acassis avatar Dec 05 '25 15:12 acassis

Hi @acassis , let's wait for all the jobs to finish running and then restart the msys2 job.

simbit18 avatar Dec 05 '25 15:12 simbit18

Hi @acassis , let's wait for all the jobs to finish running and then restart the msys2 job.

Question: is there some way to avoid download the source code packages all the time? Maybe adding it to the docker machine directly?

acassis avatar Dec 05 '25 15:12 acassis

@acassis , previously the cache was enabled for GitHub Action to setup MSYS2, but to avoid closing the NuttX repository, we had to make some cuts. :) https://github.com/apache/nuttx/pull/14618 https://github.com/apache/nuttx/pull/14909 Now it is only available on our Mirror for msys2 and macos https://github.com/NuttX/nuttx/actions/caches

simbit18 avatar Dec 05 '25 16:12 simbit18

@acassis , previously the cache was enabled for GitHub Action to setup MSYS2, but to avoid closing the NuttX repository, we had to make some cuts. :) #14618 #14909 Now it is only available on our Mirror for msys2 and macos https://github.com/NuttX/nuttx/actions/caches

Thank you @simbit18 ! I don't understand why ASF was limiting our job runners, but they keep the same amount for all projects, even projects that are dead.

acassis avatar Dec 05 '25 17:12 acassis

@zhhyu7 I noticed that the CAN documentation starts with wrong information: "NuttX supports only a very low-level CAN driver.", could you please fix it when updating the RST file?

acassis avatar Dec 06 '25 12:12 acassis

@zhhyu7 I noticed that the CAN documentation starts with wrong information: "NuttX supports only a very low-level CAN driver.", could you please fix it when updating the RST file?

OK.

zhhyu7 avatar Dec 08 '25 01:12 zhhyu7

@zhhyu7 new feature without Documentation, please add https://nuttx.apache.org/docs/latest/components/drivers/character/can.html

@acassis please review again.

xiaoxiang781216 avatar Dec 10 '25 15:12 xiaoxiang781216

Really good Documentation @zhhyu7 !!! Thank you for doing it!

acassis avatar Dec 10 '25 15:12 acassis