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

From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Anthony Liguori <aliguori@amzn.com>
Cc: "Wei Liu" <wei.liu2@citrix.com>, "Matt Wilson" <msw@amazon.com>,
	"KarimAllah Ahmed" <karahmed@amazon.de>,
	"Andrew Cooper" <andrew.cooper3@citrix.com>,
	"Jan H. Schönherr" <jschoenh@amazon.de>,
	"Anthony Liguori" <aliguori@amazon.com>,
	xen-devel@lists.xenproject.org,
	"Roger Pau Monné" <roger.pau@citrix.com>
Subject: Re: [PATCH v2 09/23] vixen: modify the e820 table to advertise HVM special pages as RAM
Date: Sun, 7 Jan 2018 19:18:37 -0500	[thread overview]
Message-ID: <20180108001837.GZ21689@char.us.oracle.com> (raw)
In-Reply-To: <1515356919-922-10-git-send-email-aliguori@amzn.com>

On Sun, Jan 07, 2018 at 12:28:25PM -0800, Anthony Liguori wrote:
> From: Anthony Liguori <aliguori@amazon.com>

..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?

>  
>  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 :-) ?
> +    *end_pfn   = 0x100000;
> +}

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