OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

virtio-dev message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Re: [virtio-dev] [PATCH v4 1/6] virtio-vsock vhost device: fixed endiannes issue


and even if you make vhost_init_is_le() extern, this won't work if the device
is in legacy mode, and the host and guest have different endianness.

This is because vhost_init_is_le() computes vq->is_le as follows when we
have CONFIG_VHOST_CROSS_ENDIAN_LEGACY=y:

vq->is_le = vhost_has_feature(vq, VIRTIO_F_VERSION_1) || !vq->user_be;

and vq->user_be comes from the VHOST_SET_VRING_ENDIAN ioctl.

I was wondering if we can just put vq->is_le = 1 instead of vhost_init_is_le(), and call it a day. virtio-vsock is a virtio 1.0 device, so we are always little endian anyway.

cheers


Claudio




[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]