Re: [Xen-devel] [PATCH v6 02/15] xen/arm: basic PSCI support, implement cpu_on and cpu_off

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Tue, 2013-05-07 at 12:38 +0100, Stefano Stabellini wrote:
> On Tue, 7 May 2013, Ian Campbell wrote:
> > On Fri, 2013-05-03 at 11:51 +0100, Stefano Stabellini wrote:
> > > +int do_psci_cpu_on(uint32_t vcpuid, register_t entry_point)
> > > +{
> > > +    struct vcpu *v;
> > > +    struct domain *d = current->domain;
> > > +    struct vcpu_guest_context *ctxt;
> > > +    int rc;
> > > +
> > > +    if ( (vcpuid < 0) || (vcpuid >= MAX_VIRT_CPUS) )
> > > +        return PSCI_EINVAL;
> > > +
> > > + if ( vcpuid >= d->max_vcpus || (v = d->vcpu[vcpuid]) == NULL )
> > > +         return PSCI_EINVAL;
> > 
> > Is this indentation a legacy of a previous surrounding if statement? Or
> > perhaps some stray hard tabs...
> 
> Yeah, sorry I noticed the hard tab yesterday :(
> 
> Do you want me to resend the entire series or just this patch?

Assuming there isn't any other reason to resend the whole series just
this one will do.

Ian.



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