> 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