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 v3 1/7] Vhost-vsock device: Fixed endiannes issue.

On 14/03/16 18:36, Greg Kurz wrote:
diff --git a/drivers/vhost/vsock.c b/drivers/vhost/vsock.c
index 2c5963c..9378f23 100644
--- a/drivers/vhost/vsock.c
+++ b/drivers/vhost/vsock.c
@@ -481,6 +481,8 @@ static int vhost_vsock_set_features(struct vhost_vsock *vsock, u64 features)
  		vq = &vsock->vqs[i];
  		vq->acked_features = features;
+		vq->is_le = virtio_legacy_is_little_endian();
+		vhost_init_is_le(vq);

How can you call vhost_init_is_le() from here ?

I can't, it's an error, sorry!
Also, as I wrote already earlier, I noticed that that call is not needed because that field is initialized already somewhere else. I already removed it in my branch and it won't be there in the next version of the patchset.


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