Re: [Xen-devel] [PATCH v5 18/21] xen/arm: Add virtual GICv3 support

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Stefano,

On 12/06/14 18:09, Stefano Stabellini wrote:
> On Thu, 12 Jun 2014, vijay.kilari@xxxxxxxxx wrote:
> > +    switch ( gicd_reg )
> > +    {
> > +    case GICD_CTLR:
> > +        if ( dabt.size != DABT_WORD ) goto bad_width;
> > +        vgic_lock(v);
> > +        *r = v->domain->arch.vgic.ctlr;
> > +        vgic_unlock(v);
> > +        return 1;
> > +    case GICD_TYPER:
> > +        if ( dabt.size != DABT_WORD ) goto bad_width;
> > +        /* No secure world support for guests. */
> > +        *r = (((v->domain->max_vcpus << 5) & GICD_TYPE_CPUS ) |
> > +              ((v->domain->arch.vgic.nr_lines / 32) & GICD_TYPE_LINES));
> > +        return 1;
> There used to be a vgic_lock(v) and vgic_unlock(v) here. Why did you
> remove them?
I don't think it's important to take the vgic lock here. Xen only access 
read-only data, max_vcpus and nr_lines doesn't change.
Regards,

--
Julien Grall

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