On Wed, Nov 11, 2015 at 08:53:08AM +0000, Liu, Jijiang wrote:
...
> > If so, you'd better quote them here, or even to your patch commit.
>
> In the Virtual I/O Device (VIRTIO) Version 1.0 Committee Specification 02,
>
> (1) VIRTIO_NET_F_GUEST_CSUM (1) Driver handles packets with partial checksum.
>
> (2) If the VIRTIO_NET_F_GUEST_CSUM feature was negotiated, the
> VIRTIO_NET_HDR_F_NEEDS_-
> CSUM bit in flags MAY be set: if so, the checksum on the packet is incomplete
> and csum_start
> and csum_offset indicate how to calculate it (see Packet Transmission point
> 1).
>
> (3) If the VIRTIO_NET_F_GUEST_TSO4, TSO6 or UFO options were negotiated, then
> gso_type
> MAY be something other than VIRTIO_NET_HDR_GSO_NONE, and gso_size field
> indicates
> the desired MSS (see Packet Transmission point 2).
Thanks. Yes, sound like we need set them.
--yliu
>
> The information can be included in the next version of patch set.
>
> > --yliu
> >
> > > Yes, I'd like to listen other guys comments.
> > >
> > > > Hopefully the virtio expert, Michael, could shine some lights on that.
> > > >
> > > > --yliu
> > > >
> > > > >
> > > > > static uint64_t VHOST_FEATURES = VHOST_SUPPORTED_FEATURES;
> > > > >
> > > > > --
> > > > > 1.7.7.6
