[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 3/5] VMX: Add posted interrupt supporting



Jan Beulich wrote on 2013-04-17:
>>>> On 17.04.13 at 14:40, "Zhang, Yang Z" <yang.z.zhang@xxxxxxxxx> wrote:
>> Jan Beulich wrote on 2013-04-17:
>>>>>> On 17.04.13 at 08:51, Yang Zhang <yang.z.zhang@xxxxxxxxx> wrote:
>>>> +        val[group] = pi_get_pir(&v->arch.hvm_vmx.pi_desc, group);
>>> 
>>> Which will require adjustments to the types underlying this call.
>> What do you mean? Do you suggest pass pir[group] instead pi_desc?
> 
> Not necessarily. pir[] will need to become an array of unsigned long
> too, and pi_get_pir() will need to return an unsigned long. And the
> definitions should, no matter that we don't support 32-bit anymore,
> not depend on u64 == unsigned long.
Ok, I see.

> 
>>>> +
>>>> +    for_each_set_bit(i, val, MAX_VECTOR)
>>> 
>>> I'd also favor you not adding new uses of this redundant and
>> Is there better way to iterate the setting bit?
> 
> No, the iterator is fine. MAX_VECTORS is what I dislike (and what
> should go away).
Yes, you are right. 

Best regards,
Yang



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.