Skip to content

Conversation

tigercosmos
Copy link
Collaborator

@tigercosmos tigercosmos commented Aug 23, 2025

Verified in 9225c43

@tigercosmos tigercosmos requested a review from seladb as a code owner August 23, 2025 14:45
@tigercosmos tigercosmos changed the base branch from master to dev August 23, 2025 14:45
@tigercosmos tigercosmos changed the title Fix bsd ci 0823 Fix BSD CI Aug 23, 2025
@tigercosmos tigercosmos marked this pull request as draft August 23, 2025 15:17
@tigercosmos
Copy link
Collaborator Author

ummm seems there are other errors...

Copy link

codecov bot commented Aug 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.44%. Comparing base (06a10a4) to head (9dee239).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1932      +/-   ##
==========================================
- Coverage   83.45%   83.44%   -0.01%     
==========================================
  Files         301      301              
  Lines       54028    54028              
  Branches    11988    11780     -208     
==========================================
- Hits        45087    45086       -1     
+ Misses       7761     7722      -39     
- Partials     1180     1220      +40     
Flag Coverage Δ
alpine320 75.42% <ø> (-0.01%) ⬇️
fedora42 75.53% <ø> (-0.03%) ⬇️
macos-13 81.65% <ø> (ø)
macos-14 81.65% <ø> (ø)
macos-15 81.65% <ø> (ø)
mingw32 70.31% <ø> (ø)
mingw64 70.31% <ø> (ø)
rhel94 75.26% <ø> (-0.03%) ⬇️
ubuntu2004 59.21% <ø> (-0.02%) ⬇️
ubuntu2004-zstd 59.33% <ø> (+<0.01%) ⬆️
ubuntu2204 75.21% <ø> (-0.02%) ⬇️
ubuntu2204-icpx 60.90% <ø> (ø)
ubuntu2404 75.45% <ø> (-0.01%) ⬇️
ubuntu2404-arm64 75.43% <ø> (-0.02%) ⬇️
unittest 83.44% <ø> (-0.01%) ⬇️
windows-2022 85.48% <ø> (ø)
windows-2025 85.50% <ø> (ø)
winpcap 85.50% <ø> (ø)
xdp 53.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tigercosmos
Copy link
Collaborator Author

A lot of [ERROR: PcapLiveDevice.cpp: sendPacketUnchecked:1087 ] Error sending packet: send: No buffer space available

@tigercosmos
Copy link
Collaborator Author

Open a issue for now: #1933

@seladb
Copy link
Owner

seladb commented Aug 23, 2025

A lot of [ERROR: PcapLiveDevice.cpp: sendPacketUnchecked:1087 ] Error sending packet: send: No buffer space available

We might be sending packets too fast in this test. Here is what ChatGPT says:

The error "No buffer space available" from pcap_sendpacket on FreeBSD typically indicates that the kernel's transmit buffer (or ring buffer) for the network interface is full and cannot accept more packets.

Common Causes:

Sending Packets Too Fast
If you're sending packets in a tight loop without any pacing or rate limiting, the buffer fills up faster than the NIC (Network Interface Card) can transmit them.

Interface Not in Promiscuous or Up State
If the interface is not properly configured (e.g. down or in an unexpected state), it might cause this kind of error.

Resource Limits (e.g., mbufs)
FreeBSD relies on a pool of memory buffers (mbufs) for networking. If the system runs out of mbufs due to high network usage, you'll get this error.

pcap_sendpacket Queues Too Many Packets
pcap_sendpacket() queues packets for transmission but does not block, so if the driver can't keep up, it returns an error.

Solutions:
✅ 1. Throttle Packet Sending

Add a small delay between packet sends to avoid overwhelming the interface.

usleep(100); // 100 microseconds

✅ 2. Increase mbuf Pool Size

You can increase the number of available mbufs in FreeBSD:

Edit /boot/loader.conf:

kern.ipc.nmbclusters="32768"
kern.ipc.nmbjumbop="2048"

Then reboot the system.

To check current usage:

vmstat -z | grep mbuf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants