>>> On 15.01.15 at 17:48, <t...@xen.org> wrote:
> At 13:26 -0800 on 09 Jan (1420806395), Ed White wrote:
>> + /* Init alternate p2m data */
>> + if ( (d->arch.altp2m_eptp = alloc_xenheap_page()) == NULL )
>> + {
>> + rv = -ENOMEM;
>> + goto out;
>> + }
>> + for (i = 0; i < 512; i++)
>> + d->arch.altp2m_eptp[i] = ~0ul;
>
> This 512 is architectural, I guess? It should have a named constant.
Perhaps even calculated rather than just defined as a plain
number constant, e.g. (PAGE_SIZE / sizeof(something)).
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
