Re: [Xen-devel] [PATCH v2 09/23] vixen: modify the e820 table to advertise HVM special pages as RAM

[Anthony Liguori <anthony@xxxxxxxxxxxxx>]

On Sun, Jan 7, 2018 at 4:18 PM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
> On Sun, Jan 07, 2018 at 12:28:25PM -0800, Anthony Liguori wrote:
>> From: Anthony Liguori <aliguori@xxxxxxxxxx>
>
> ..snip.
>> diff --git a/xen/arch/x86/guest/vixen.c b/xen/arch/x86/guest/vixen.c
>> index c0a81dd..cacbe69 100644
>> --- a/xen/arch/x86/guest/vixen.c
>> +++ b/xen/arch/x86/guest/vixen.c
>> @@ -23,6 +23,7 @@
>>
>>  static int in_vixen;
>>  static int vixen_domid = 1;
>> +static uint32_t vixen_reserved_mem_pgstart = 0xfeff0000;
>
> This is interesting. Would it make sense to have this be
> auto-detected? Or a boot parameter to over-write?

This is done in a later patch (the one that sets up event channels).

It cannot be done here because we don't have enough infrastructure.

>>  integer_param("vixen_domid", vixen_domid);
>>
>> @@ -35,3 +36,11 @@ int vixen_get_domid(void)
>>  {
>>      return vixen_domid;
>>  }
>> +
>> +void vixen_get_reserved_mem(unsigned long *start_pfn, unsigned long 
>> *end_pfn)
>> +{
>> +    *start_pfn = vixen_reserved_mem_pgstart >> XEN_PAGE_SHIFT;
>> +
>> +    /* This is part of the Xen ABI */
>
> .. which one :-) ?

include/public/hvm/hvm_info_table.h.  See the comment around
reserved_mem_pgstart.

Regards,

Anthony Liguori

>> +    *end_pfn   = 0x100000;
>> +}
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel