Re: [Xen-devel] [PATCH 2/4] xen/arm: support for guest SGI

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Tue, 23 Apr 2013, Ian Campbell wrote:
> > > > +            cpumask_clear(&vcpu_mask);
> > > > +            switch ( filter )
> > > > +            {
> > > > +                case GICD_SGI_TARGET_LIST:
> > > > +                    cpumask_bits(&vcpu_mask)[0] = (*r & 
> > > > GICD_SGI_TARGET_MASK) >> GICD_SGI_TARGET_SHIFT;
> > > 
> > > Is this the best available way to set a cpumask in Xen?
> > 
> > We could probably use cpumask_or to set the bits in vcpu_mask, but I
> > don't think it would be a great improvement.
> 
> It would have the advantage of not breaking the cpumask abstraction
> quite so wide open though.

Nope, I was wrong, cpumask_or only takes cpumask_t as arguments.  In
fact the issue is that *r is not a cpumask_t, so the only way to make it
into a cpumask_t is to use cpumask_bits or a loop.

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