On Mon, Sep 7, 2015 at 7:50 PM, Julien Grall <julien.gr...@citrix.com> wrote:
> Hi Vijay,
>
> On 31/08/15 12:06, vijay.kil...@gmail.com wrote:
>> +static int vgic_v3_gits_lpi_mmio_read(struct vcpu *v, mmio_info_t *info)
>
> [...]
>
>> + /* Neglect if not LPI. */
>> + if ( offset < FIRST_GIC_LPI )
>> + {
>> + *r = 0;
>> + return 1;
>> + }
>
> [...]
>
>> +static int vgic_v3_gits_lpi_mmio_write(struct vcpu *v, mmio_info_t *info)
>> +{
>
> [...]
>
>> + /* Neglect if not LPI. */
>> + if ( offset < FIRST_GIC_LPI )
>> + return 1;
>
> Based on the spec, those 2 checks are wrong and make impossible to use
> LPIs. Please test this patch series before sending it on the ML.
Why do you think so?.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
