Hi Stefano, Thanks for the review. >We incremented it only for VIRTIO_VSOCK_OP_RW, should we check the >same here? Checking for the length of the skbuf or checking for VIRTIO_VSOCK_OP_RW AFAIK is equivalent. Since the only packet with payload is this one. I'll add a comment in v2 to make it more clear. >What about rename it in `consume`? Sounds good! Thanks, Luigi