Skip to content

Commit 97eaf9b

Browse files
bchaliosJackThomson2
authored andcommitted
net: revert virtio-net queue size to 256
Changing the queue size from 256 to 512 increases RX throughput but seems to regress TX throughput by 3% on average. We are not sure what is causing this. Revert this change until we have enough time to investigate and understand better the mechanisms that are in play. Signed-off-by: Babis Chalios <[email protected]>
1 parent 17eac40 commit 97eaf9b

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

CHANGELOG.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,6 @@ and this project adheres to
3333

3434
### Changed
3535

36-
- [#4875](https://github.com/firecracker-microvm/firecracker/pull/4875):
37-
Increase default queue size for the `virtio-net` device from 256 to 512. This
38-
decreases wait time between guest and vmm threads for network packets
39-
processing and allows for more throughput.
4036
- [#4844](https://github.com/firecracker-microvm/firecracker/pull/4844): Upgrade
4137
`virtio-net` device to use `readv` syscall to avoid unnecessary memory copies
4238
on RX path, increasing the RX performance.

src/vmm/src/devices/virtio/net/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
use std::io;
77

88
/// Maximum size of the queue for network device.
9-
pub const NET_QUEUE_MAX_SIZE: u16 = 512;
9+
pub const NET_QUEUE_MAX_SIZE: u16 = 256;
1010
/// Maximum size of the frame buffers handled by this device.
1111
pub const MAX_BUFFER_SIZE: usize = 65562;
1212
/// The number of queues of the network device.

0 commit comments

Comments
 (0)