Skip to content

Conversation

@zhhyu7
Copy link
Contributor

@zhhyu7 zhhyu7 commented Dec 24, 2025

Summary

Otherwise, we can only manually enable NET_WRITE_BUFFERS

Impact

net/can/can_sendmsg_buffered.c

Testing

Since no code logic was modified, only the compilation was verified.
sim:matter with enable CONFIG_NET_CAN and CONFIG_NET_CAN_WRITE_BUFFERS
.config

nuttx$ cat .config | grep NET_CAN
CONFIG_NET_CANPROTO_OPTIONS=y
CONFIG_NET_CAN=y
CONFIG_NET_CAN_WRITE_BUFFERS=y
# CONFIG_NET_CAN_EXTID is not set
CONFIG_NET_CAN_SOCK_OPTS=y
CONFIG_NET_CAN_RAW_FILTER_MAX=32
# CONFIG_NET_CAN_NOTIFIER is not set

build log

nuttx$ make -j12
Create version.h
LD:  nuttx
Pac SIM with dynamic libs..
'/lib/x86_64-linux-gnu/libz.so.1' -> 'sim-pac/libs/libz.so.1'
'/lib/x86_64-linux-gnu/libc.so.6' -> 'sim-pac/libs/libc.so.6'
'/lib64/ld-linux-x86-64.so.2' -> 'sim-pac/ld-linux-x86-64.so.2'
SIM elf with dynamic libs archive in nuttx.tgz
nuttx$ ls -l nuttx
-rwxrwxr-x 1 zhhyu zhhyu 4544616 12月 24 20:54 nuttx

zhhyu7 and others added 2 commits December 24, 2025 20:31
Otherwise, we can only manually enable NET_WRITE_BUFFERS

Signed-off-by: zhanghongyu <zhanghongyu@xiaomi.com>
variable "conn" not define if CONFIG_NET_SEND_BUFSIZE > 0

Signed-off-by: wangjinjing1 <wangjinjing1@xiaomi.com>
@zhhyu7 zhhyu7 requested a review from raiden00pl as a code owner December 24, 2025 12:56
@github-actions github-actions bot added Area: Networking Effects networking subsystem Size: S The size of the change in this PR is small labels Dec 24, 2025
@Donny9 Donny9 merged commit 7b528d9 into apache:master Dec 25, 2025
64 of 77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Networking Effects networking subsystem Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants