>>> On 17.06.15 at 15:53, <david.vra...@citrix.com> wrote:
> +static int steal_maptrack_handle(struct grant_table *t,
> + const struct vcpu *curr)
> +{
> + const struct domain *currd = curr->domain;
> + unsigned int first, i;
> +
> + /* Find an initial victim. */
> + first = i = get_random() % currd->max_vcpus;
> +
> + do {
> + if ( currd->vcpu[i] )
> + {
> + unsigned int handle;
> +
> + handle = __get_maptrack_handle(t, currd->vcpu[i]);
> + if ( handle >= 0 )
This comparison is always true. With handle's type fixed
Reviewed-by: Jan Beulich <jbeul...@suse.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
