Re: [Xen-devel] [PATCH] switch hypercall restart indication from -EAGAIN to -ERESTART

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Mon, 2014-05-12 at 14:50 +0100, Jan Beulich wrote:
> -EAGAIN being a return value we want to return to the actual caller in
> a couple of cases makes this unsuitable for restart indication, and x86
> already developed two cases where -EAGAIN could not be returned as
> intended due to this (which is being fixed here at once).
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx (ARM and common)

Two comments:

I notice some existing ERESTART's in xen/drivers/passthrough/iommu.c --
is that a problem or do the adjustments here take that into account?

This struck me as odd:

> @@ -590,6 +590,8 @@ int domain_kill(struct domain *d)
>          rc = domain_relinquish_resources(d);
>          if ( rc != 0 )
>          {
> +            if ( rc == -ERESTART )
> +                rc = -EAGAIN;

until I realised that you intended for ERESTART to only be used
internally within the hypervisor and not to be exposed to the guest.
Perhaps insert the word "internal" into the Subject?

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel