[Xen-devel] Re: Experience with netchannel2

[Steven Smith <steven.smith@xxxxxxxxxx>]

> Thank you very much for your efforts and fixes.
> Please, consider some minor changes in gnttab_request_version
> I added the return code to it., because HYPERVISOR_grant_table_op could 
> return -EFAULT, -EINVAL additional to -ENOSYS.
> I attached the file.
Hmm.  I'm not convinced by this, for a couple of reasons:

a) HYPERVISOR_grant_table_op(GNTTABOP_set_version) won't return
   -EFAULT or -EINVAL for this particular call, because the parameters
   to the call are correct.

b) Even if it did, the current behaviour (of just assuming version 1)
   is pretty reasonable.

c) If gnttab_resume() fails then there isn't really anything which the
   rest of the kernel can do to recover, so the guest will almost
   certainly just crash a few instructions later.

So I think I'm going to pass on this one.

Thanks,

Steven.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel