>>> On 10.03.15 at 13:51, <julien.gr...@linaro.org> wrote:
> Hi,
>
> On 27/02/15 11:33, Roger Pau Monne wrote:
>> diff --git a/xen/drivers/passthrough/iommu.c
> b/xen/drivers/passthrough/iommu.c
>> index cc12735..7fcbbb1 100644
>> --- a/xen/drivers/passthrough/iommu.c
>> +++ b/xen/drivers/passthrough/iommu.c
>> @@ -332,7 +332,8 @@ void iommu_share_p2m_table(struct domain* d)
>> {
>> const struct iommu_ops *ops = iommu_get_ops();
>>
>> - if ( iommu_enabled && is_hvm_domain(d) )
>> + ASSERT( hap_enabled(d) );
>
> This line is breaking compilation on ARM.
>
> Shouldn't it be replaced by iommu_use_hap_pt?
No, that's a different thing. But shouldn't ARM have a stub
hap_enabled() evaluating to constant true?
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
