From: Samuel Thibault <samuel.thibault@eu.citrix.com>
To: Gerd Hoffmann <kraxel@redhat.com>,
Xen Development Mailing List <xen-devel@lists.xensource.com>
Subject: Re: [bug] pv-grub doesn't run on rhel5
Date: Thu, 14 Aug 2008 16:36:10 +0100 [thread overview]
Message-ID: <20080814153610.GV4590@implementation.uk.xensource.com> (raw)
In-Reply-To: <20080814151359.GT4590@implementation.uk.xensource.com>
Samuel Thibault, le Thu 14 Aug 2008 16:14:00 +0100, a écrit :
> Gerd Hoffmann, le Thu 14 Aug 2008 17:04:07 +0200, a écrit :
> > Uhm, well, no way to fix minios to run on older xen versions then I
> > guess (which I'd prefer). Except maybe limiting the 32bit version to
> > 1GB address space. Which should be fine at least for pvgrub ...
>
> That should be feasible indeed.
As seen below. I'm not sure whether we want that into mainline Xen 3.3
however?
BTW, Keir commited the 32on64 pv-grub build, it should appear after the
regression tests.
Samuel
pv-grub: limit address space usage to less than a GB so as to be
runnable on xen hypervisor revisions earlier than c/s 17061.
Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com>
diff -r 38783464a671 extras/mini-os/arch/x86/mm.c
--- a/extras/mini-os/arch/x86/mm.c Thu Aug 14 16:16:44 2008 +0100
+++ b/extras/mini-os/arch/x86/mm.c Thu Aug 14 16:30:59 2008 +0100
@@ -414,18 +414,22 @@ pgentry_t *need_pgt(unsigned long addr)
}
static unsigned long demand_map_area_start;
+#ifndef DEMAND_MAP_PAGES
#ifdef __x86_64__
#define DEMAND_MAP_PAGES ((128ULL << 30) / PAGE_SIZE)
#else
#define DEMAND_MAP_PAGES ((2ULL << 30) / PAGE_SIZE)
#endif
+#endif
#ifdef HAVE_LIBC
unsigned long heap, brk, heap_mapped, heap_end;
+#ifndef HEAP_PAGES
#ifdef __x86_64__
#define HEAP_PAGES ((128ULL << 30) / PAGE_SIZE)
#else
#define HEAP_PAGES ((1ULL << 30) / PAGE_SIZE)
+#endif
#endif
#endif
diff -r 38783464a671 stubdom/Makefile
--- a/stubdom/Makefile Thu Aug 14 16:16:44 2008 +0100
+++ b/stubdom/Makefile Thu Aug 14 16:30:59 2008 +0100
@@ -33,6 +33,7 @@ TARGET_CFLAGS=
TARGET_CFLAGS=
NEWLIB_CFLAGS+=-D_I386MACH_ALLOW_HW_INTERRUPTS
STUBDOM_SUPPORTED=1
+PVGRUB_CPPFLAGS=-DDEMAND_MAP_PAGES='((256ULL << 20) / PAGE_SIZE)' -DHEAP_PAGES='((256ULL << 20) / PAGE_SIZE)'
endif
ifeq ($(GNU_TARGET_ARCH), x86_64)
TARGET_CFLAGS=-mno-red-zone
@@ -296,7 +297,7 @@ c-stubdom: mini-os-$(XEN_TARGET_ARCH)-c
.PHONY: pv-grub
pv-grub: mini-os-$(XEN_TARGET_ARCH)-grub libxc grub
- DEF_CPPFLAGS="$(TARGET_CPPFLAGS)" DEF_CFLAGS="-DCONFIG_GRUB $(TARGET_CFLAGS)" DEF_LDFLAGS="$(TARGET_LDFLAGS)" $(MAKE) -C $(MINI_OS) OBJ_DIR=$(CURDIR)/$< APP_OBJS=$(CURDIR)/grub-$(XEN_TARGET_ARCH)/main.a
+ DEF_CPPFLAGS="$(TARGET_CPPFLAGS) $(PVGRUB_CPPFLAGS)" DEF_CFLAGS="-DCONFIG_GRUB $(TARGET_CFLAGS)" DEF_LDFLAGS="$(TARGET_LDFLAGS)" $(MAKE) -C $(MINI_OS) OBJ_DIR=$(CURDIR)/$< APP_OBJS=$(CURDIR)/grub-$(XEN_TARGET_ARCH)/main.a
#########
# install
next prev parent reply other threads:[~2008-08-14 15:36 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-14 10:38 [bug] pv-grub doesn't run on rhel5 Gerd Hoffmann
2008-08-14 11:15 ` Samuel Thibault
[not found] ` <48A4232B.6040500@redhat.com>
2008-08-14 12:29 ` Samuel Thibault
2008-08-14 14:18 ` Gerd Hoffmann
2008-08-14 14:28 ` Samuel Thibault
2008-08-14 15:04 ` Gerd Hoffmann
2008-08-14 15:14 ` Samuel Thibault
2008-08-14 15:36 ` Samuel Thibault [this message]
2008-08-14 16:01 ` Gerd Hoffmann
2008-08-14 18:02 ` [PATCH] xenfb: make restartable [Was: pv-grub doesn't run on rhel5] Samuel Thibault
2008-08-15 7:02 ` Gerd Hoffmann
2008-08-15 12:39 ` [PATCH] xenfb: make restartable Markus Armbruster
2008-08-15 22:23 ` Samuel Thibault
2008-08-15 22:53 ` Markus Armbruster
2008-08-16 22:33 ` Samuel Thibault
2008-08-14 15:55 ` [bug] pv-grub doesn't run on rhel5 Jeremy Fitzhardinge
2008-08-14 16:04 ` Gerd Hoffmann
2008-08-14 12:33 ` [PATCH] compiled a 32bit pv-grub on x86_64 xen target [Was: pv-grub doesn't run on rhel5] Samuel Thibault
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=20080814153610.GV4590@implementation.uk.xensource.com \
--to=samuel.thibault@eu.citrix.com \
--cc=kraxel@redhat.com \
--cc=xen-devel@lists.xensource.com \
/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.