Thread (4 messages) 4 messages, 3 authors, 2024-06-23

Re: [PATCH net-next v2 2/3] vsock/virtio: add SIOCOUTQ support for all virtio based transports

From: Luigi Leonardi <hidden>
Date: 2024-06-23 15:04:36
Also in: kvm, virtualization

Hi Paolo and Stefano,

Sorry for the (super) late reply! I haven't had much time to work on
it, I hope to send a v3 by the end of this month! Thanks both of you
for your comments and reviews! :)
quoted
This will add 2 atomic operations per packet, possibly on contended
cachelines. Have you considered leveraging the existing transport-level
lock to protect the counter updates?
Good point!
Maybe we can handle it together with `tx_cnt` in
virtio_transport_get_credit()/virtio_transport_put_credit().
WDYT?
I'll take a look at it! That's a very good idea.
Should virtio_transport_bytes_unsent() returns size_t?
Yes! int was because of atomic_int, size_t is more appropriate.

Thanks,
Luigi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help