[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Xen-devel] Re: Experience with netchannel2
> 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.
_______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
Lists.xenproject.org is hosted with RackSpace, monitoring our