All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <jgross@suse.com>
To: grub-devel@gnu.org
Cc: Juergen Gross <jgross@suse.com>,
	phcoder@gmail.com, daniel.kiper@oracle.com,
	xen-devel@lists.xen.org, hans@knorrie.org, roger.pau@citrix.com
Subject: [PATCH v5 12/20] xen: get memory map from hypervisor for PVH
Date: Wed, 21 Nov 2018 15:28:47 +0100	[thread overview]
Message-ID: <20181121142855.13537-13-jgross@suse.com> (raw)
In-Reply-To: <20181121142855.13537-1-jgross@suse.com>

Retrieve the memory map from the hypervisor and normalize it to contain
no overlapping entries and to be sorted by address.

Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
---
V3: use grub_e820_mmap_entry instead of own struct (Daniel Kiper)
---
 grub-core/kern/i386/xen/pvh.c | 94 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 94 insertions(+)

diff --git a/grub-core/kern/i386/xen/pvh.c b/grub-core/kern/i386/xen/pvh.c
index 478cef0d1..bb90874b3 100644
--- a/grub-core/kern/i386/xen/pvh.c
+++ b/grub-core/kern/i386/xen/pvh.c
@@ -24,7 +24,12 @@
 #include <grub/i386/io.h>
 #include <grub/xen.h>
 #include <xen/hvm/start_info.h>
+#include <grub/i386/linux.h>
 #include <grub/machine/kernel.h>
+#include <grub/machine/memory.h>
+#include <xen/memory.h>
+
+#define XEN_MEMORY_MAP_SIZE   128
 
 grub_uint64_t grub_rsdp_addr;
 
@@ -32,6 +37,8 @@ static char hypercall_page[GRUB_XEN_PAGE_SIZE]
   __attribute__ ((aligned (GRUB_XEN_PAGE_SIZE)));
 
 static grub_uint32_t xen_cpuid_base;
+static struct grub_e820_mmap_entry map[XEN_MEMORY_MAP_SIZE];
+static unsigned int nr_map_entries;
 
 static void
 grub_xen_cons_msg (const char *msg)
@@ -103,11 +110,98 @@ grub_xen_hypercall (grub_uint32_t callno, grub_uint32_t a0,
   return __res;
 }
 
+static void
+grub_xen_sort_mmap (void)
+{
+  grub_uint64_t from, to;
+  unsigned int i;
+  struct grub_e820_mmap_entry tmp;
+
+  /* Align map entries to page boundaries. */
+  for (i = 0; i < nr_map_entries; i++)
+    {
+      from = map[i].addr;
+      to = from + map[i].len;
+      if (map[i].type == GRUB_MEMORY_AVAILABLE)
+	{
+	  from = ALIGN_UP (from, GRUB_XEN_PAGE_SIZE);
+	  to = ALIGN_DOWN (to, GRUB_XEN_PAGE_SIZE);
+	}
+      else
+	{
+	  from = ALIGN_DOWN (from, GRUB_XEN_PAGE_SIZE);
+	  to = ALIGN_UP (to, GRUB_XEN_PAGE_SIZE);
+	}
+      map[i].addr = from;
+      map[i].len = to - from;
+    }
+
+ again:
+  /* Sort entries by start address. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].addr >= map[i - 1].addr)
+	continue;
+      tmp = map[i];
+      map[i] = map[i - 1];
+      map[i - 1] = tmp;
+      i = 0;
+    }
+
+  /* Detect overlapping areas. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].addr >= map[i - 1].addr + map[i - 1].len)
+	continue;
+      tmp = map[i - 1];
+      map[i - 1].len = map[i].addr - map[i - 1].addr;
+      if (map[i].addr + map[i].len >= tmp.addr + tmp.len)
+	continue;
+      if (nr_map_entries < ARRAY_SIZE (map))
+	{
+	  map[nr_map_entries].addr = map[i].addr + map[i].len;
+	  map[nr_map_entries].len = tmp.addr + tmp.len - map[nr_map_entries].addr;
+	  map[nr_map_entries].type = tmp.type;
+	  nr_map_entries++;
+	  goto again;
+	}
+    }
+
+  /* Merge adjacent entries. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].type == map[i - 1].type &&
+	  map[i].addr == map[i - 1].addr + map[i - 1].len)
+	{
+	  map[i - 1].len += map[i].len;
+	  map[i] = map[nr_map_entries - 1];
+	  nr_map_entries--;
+	  goto again;
+	}
+    }
+}
+
+static void
+grub_xen_get_mmap (void)
+{
+  struct xen_memory_map memmap;
+
+  memmap.nr_entries = ARRAY_SIZE (map);
+  set_xen_guest_handle (memmap.buffer, map);
+  if (grub_xen_hypercall (__HYPERVISOR_memory_op, XENMEM_memory_map,
+			  (grub_uint32_t) (&memmap), 0, 0, 0, 0))
+    grub_xen_panic ("Could not get memory map from Xen!\n");
+  nr_map_entries = memmap.nr_entries;
+
+  grub_xen_sort_mmap ();
+}
+
 void
 grub_xen_setup_pvh (void)
 {
   grub_xen_cpuid_base ();
   grub_xen_setup_hypercall_page ();
+  grub_xen_get_mmap ();
 }
 
 grub_err_t
-- 
2.16.4


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

WARNING: multiple messages have this Message-ID (diff)
From: Juergen Gross <jgross@suse.com>
To: grub-devel@gnu.org
Cc: xen-devel@lists.xen.org, phcoder@gmail.com,
	daniel.kiper@oracle.com, roger.pau@citrix.com, hans@knorrie.org,
	Juergen Gross <jgross@suse.com>
Subject: [PATCH v5 12/20] xen: get memory map from hypervisor for PVH
Date: Wed, 21 Nov 2018 15:28:47 +0100	[thread overview]
Message-ID: <20181121142855.13537-13-jgross@suse.com> (raw)
In-Reply-To: <20181121142855.13537-1-jgross@suse.com>

Retrieve the memory map from the hypervisor and normalize it to contain
no overlapping entries and to be sorted by address.

Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
---
V3: use grub_e820_mmap_entry instead of own struct (Daniel Kiper)
---
 grub-core/kern/i386/xen/pvh.c | 94 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 94 insertions(+)

diff --git a/grub-core/kern/i386/xen/pvh.c b/grub-core/kern/i386/xen/pvh.c
index 478cef0d1..bb90874b3 100644
--- a/grub-core/kern/i386/xen/pvh.c
+++ b/grub-core/kern/i386/xen/pvh.c
@@ -24,7 +24,12 @@
 #include <grub/i386/io.h>
 #include <grub/xen.h>
 #include <xen/hvm/start_info.h>
+#include <grub/i386/linux.h>
 #include <grub/machine/kernel.h>
+#include <grub/machine/memory.h>
+#include <xen/memory.h>
+
+#define XEN_MEMORY_MAP_SIZE   128
 
 grub_uint64_t grub_rsdp_addr;
 
@@ -32,6 +37,8 @@ static char hypercall_page[GRUB_XEN_PAGE_SIZE]
   __attribute__ ((aligned (GRUB_XEN_PAGE_SIZE)));
 
 static grub_uint32_t xen_cpuid_base;
+static struct grub_e820_mmap_entry map[XEN_MEMORY_MAP_SIZE];
+static unsigned int nr_map_entries;
 
 static void
 grub_xen_cons_msg (const char *msg)
@@ -103,11 +110,98 @@ grub_xen_hypercall (grub_uint32_t callno, grub_uint32_t a0,
   return __res;
 }
 
+static void
+grub_xen_sort_mmap (void)
+{
+  grub_uint64_t from, to;
+  unsigned int i;
+  struct grub_e820_mmap_entry tmp;
+
+  /* Align map entries to page boundaries. */
+  for (i = 0; i < nr_map_entries; i++)
+    {
+      from = map[i].addr;
+      to = from + map[i].len;
+      if (map[i].type == GRUB_MEMORY_AVAILABLE)
+	{
+	  from = ALIGN_UP (from, GRUB_XEN_PAGE_SIZE);
+	  to = ALIGN_DOWN (to, GRUB_XEN_PAGE_SIZE);
+	}
+      else
+	{
+	  from = ALIGN_DOWN (from, GRUB_XEN_PAGE_SIZE);
+	  to = ALIGN_UP (to, GRUB_XEN_PAGE_SIZE);
+	}
+      map[i].addr = from;
+      map[i].len = to - from;
+    }
+
+ again:
+  /* Sort entries by start address. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].addr >= map[i - 1].addr)
+	continue;
+      tmp = map[i];
+      map[i] = map[i - 1];
+      map[i - 1] = tmp;
+      i = 0;
+    }
+
+  /* Detect overlapping areas. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].addr >= map[i - 1].addr + map[i - 1].len)
+	continue;
+      tmp = map[i - 1];
+      map[i - 1].len = map[i].addr - map[i - 1].addr;
+      if (map[i].addr + map[i].len >= tmp.addr + tmp.len)
+	continue;
+      if (nr_map_entries < ARRAY_SIZE (map))
+	{
+	  map[nr_map_entries].addr = map[i].addr + map[i].len;
+	  map[nr_map_entries].len = tmp.addr + tmp.len - map[nr_map_entries].addr;
+	  map[nr_map_entries].type = tmp.type;
+	  nr_map_entries++;
+	  goto again;
+	}
+    }
+
+  /* Merge adjacent entries. */
+  for (i = 1; i < nr_map_entries; i++)
+    {
+      if (map[i].type == map[i - 1].type &&
+	  map[i].addr == map[i - 1].addr + map[i - 1].len)
+	{
+	  map[i - 1].len += map[i].len;
+	  map[i] = map[nr_map_entries - 1];
+	  nr_map_entries--;
+	  goto again;
+	}
+    }
+}
+
+static void
+grub_xen_get_mmap (void)
+{
+  struct xen_memory_map memmap;
+
+  memmap.nr_entries = ARRAY_SIZE (map);
+  set_xen_guest_handle (memmap.buffer, map);
+  if (grub_xen_hypercall (__HYPERVISOR_memory_op, XENMEM_memory_map,
+			  (grub_uint32_t) (&memmap), 0, 0, 0, 0))
+    grub_xen_panic ("Could not get memory map from Xen!\n");
+  nr_map_entries = memmap.nr_entries;
+
+  grub_xen_sort_mmap ();
+}
+
 void
 grub_xen_setup_pvh (void)
 {
   grub_xen_cpuid_base ();
   grub_xen_setup_hypercall_page ();
+  grub_xen_get_mmap ();
 }
 
 grub_err_t
-- 
2.16.4



  parent reply	other threads:[~2018-11-21 14:28 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-21 14:28 [PATCH v5 00/20] xen: add pvh guest support Juergen Gross
2018-11-21 14:28 ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 01/20] xen: add some xen headers Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-28 11:31   ` Daniel Kiper
2018-11-28 11:31     ` Daniel Kiper
2018-11-28 11:46     ` Juergen Gross
2018-11-28 11:46     ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 02/20] loader/linux: support passing rsdp address via boot params Juergen Gross
2018-11-21 14:28 ` Juergen Gross
2018-11-27 19:52   ` Daniel Kiper
2018-11-27 19:52   ` Daniel Kiper
2018-11-21 14:28 ` [PATCH v5 03/20] xen: carve out grant tab initialization into dedicated function Juergen Gross
2018-11-21 14:28 ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 04/20] xen: prepare common code for Xen PVH support Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 05/20] xen: add some dummy headers for PVH mode Juergen Gross
2018-11-21 14:28 ` Juergen Gross
2018-11-27 19:59   ` Daniel Kiper
2018-11-27 19:59   ` Daniel Kiper
2018-11-21 14:28 ` [PATCH v5 06/20] xen: rearrange xen/init.c to prepare it for Xen " Juergen Gross
2018-11-21 14:28 ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 07/20] xen: modify grub_xen_ptr2mfn() for xen-pvh Juergen Gross
2018-11-27 20:02   ` Daniel Kiper
2018-11-27 20:02   ` Daniel Kiper
2018-11-21 14:28 ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 08/20] xen: add PVH specific defines to offset.h Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 09/20] xen: add basic hooks for PVH in current code Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-27 20:10   ` Daniel Kiper
2018-11-27 20:10   ` Daniel Kiper
2018-11-21 14:28 ` [PATCH v5 10/20] xen: add PVH boot entry code Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 11/20] xen: setup hypercall page for PVH Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-23 15:14   ` Roger Pau Monné
2018-11-23 15:14   ` Roger Pau Monné
2018-11-27 20:31   ` Daniel Kiper
2018-11-27 20:31     ` Daniel Kiper
2018-11-28 11:28     ` Daniel Kiper
2018-11-28 11:28     ` Daniel Kiper
2018-11-28 11:44       ` Juergen Gross
2018-11-28 11:44       ` Juergen Gross
2018-11-21 14:28 ` Juergen Gross [this message]
2018-11-21 14:28   ` [PATCH v5 12/20] xen: get memory map from hypervisor " Juergen Gross
2018-11-21 14:28 ` [PATCH v5 13/20] xen: setup Xen specific data " Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-23 15:04   ` Roger Pau Monné
2018-11-23 15:04     ` Roger Pau Monné
2018-11-27 21:10   ` Daniel Kiper
2018-11-27 21:10     ` Daniel Kiper
2018-11-21 14:28 ` [PATCH v5 14/20] xen: init memory regions " Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 15/20] xen_pvh: add build runes for grub-core Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-27 21:17   ` Daniel Kiper
2018-11-27 21:17     ` Daniel Kiper
2018-11-28 11:52     ` Juergen Gross
2018-11-28 11:52     ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 16/20] grub-module-verifier: Ignore all_video for xenpvh Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 17/20] xen: use elfnote defines instead of plain numbers Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 18/20] xen_pvh: support building a standalone image Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 19/20] xen_pvh: support grub-install for xen_pvh Juergen Gross
2018-11-21 14:28   ` Juergen Gross
2018-11-21 14:28 ` [PATCH v5 20/20] xen_pvh: add support to configure Juergen Gross
2018-11-21 14:28   ` Juergen Gross

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181121142855.13537-13-jgross@suse.com \
    --to=jgross@suse.com \
    --cc=daniel.kiper@oracle.com \
    --cc=grub-devel@gnu.org \
    --cc=hans@knorrie.org \
    --cc=phcoder@gmail.com \
    --cc=roger.pau@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.