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

Re: [Xen-devel] [PATCH v3 01/62] Revert "xen/arm: vgic-v2: Drop cbase from arch_domain"

On 17/11/15 12:32, Shannon Zhao wrote:
> Hi Julien,
> On 2015/11/17 19:27, Julien Grall wrote:
>> Hi Shannon,
>> Why do you want to revert this patch?
> Because d->arch.vgic.cbase will be used by creating Dom0 MADT table
> later. See [PATCH v3 43/62].
> +            gicc.base_address = d->arch.vgic.cbase;
> My previous way is get this from ACPI table but someone suggest get it
> from struct domain and I think this way is better too since it uses the
> value after being parsed.

It's pointless to store the value in arch_domain for something that it's
only use during building...

We have struct kernel_info which store any information related to the
DOM0 during the building.

> And sorry that I didn't notice this patch before when you send it out to
> list.

In any case you have to explain in the commit message why you want to
revert the patch.


Julien Grall

Xen-devel mailing list



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