[Top][All Lists]

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

Re: [PATCH] New x86_64 EFI patch

From: Pavel Roskin
Subject: Re: [PATCH] New x86_64 EFI patch
Date: Thu, 17 Jul 2008 11:22:15 -0400

On Thu, 2008-07-17 at 17:02 +0200, Javier Martín wrote:
> El jue, 17-07-2008 a las 17:50 +0800, Bean escribió:
> > Committed.
> > 
> Seemingly, the fact that there is now separate recognition for "x86_64"
> machine types has borked the current build system in amd64 PCs:
> $ make
> ./config.status --recheck
> running CONFIG_SHELL=/bin/bash /bin/bash ../src/configure
> --prefix=/home/javier/Proyectos/grub/build/out/  --no-create
> --no-recursion
> checking build system type... x86_64-unknown-linux-gnu
> checking host system type... x86_64-unknown-linux-gnu
> checking target system type... x86_64-unknown-linux-gnu
> configure: error: unsupported machine type
> make: *** [config.status] Error 1

I've just noticed that.  I think x86_64 should default to "i386-pc"
platform.  I'll have a look how to do it nicely.

> The current system would build bootloader files as i386-pc (with the
> -m32 switch in gcc, which itself created the infamous "cannot find start
> or _start" problem when a multilib version of gcc was not installed) and
> helper files like grub-setup as amd64.

I was trying add an extra check there, but ran into more interesting
stuff.  Basically, the target checks should be better separated from the
rest.  I'll take another look.

>  This caused problems of its own,
> like grub-emu (built as amd64) being unable to load any grub modules
> (built as i386), but at least worked OOtB.

I didn't know grub-emu could load modules!  Live and learn :-)

> I was able to correctly recompile if I reconfigured GRUB with
> --target=i386-pc-linux-gnu (i386-pc alone was not recognized). Is this
> the current way to go? I feel it should not be so, because it multiboot2
> comes out with amd64 support, there should be an x86_64-pc grub2.

Perhaps we'll have it eventually.  But I agree, the default should be
restored for now.

Pavel Roskin

reply via email to

[Prev in Thread] Current Thread [Next in Thread]