Re: [systemd-devel] [PATCH 2/2] configure.ac: Disable the QEMU BIOS test when cross compiling

2013-08-22 Thread Darren Hart
On Thu, 2013-08-22 at 12:00 +0200, Kay Sievers wrote: > On Thu, Aug 22, 2013 at 5:49 AM, Darren Hart wrote: > > On Thu, 2013-08-22 at 00:46 +0200, Kay Sievers wrote: > > > No problem. The check was the same as mine, but you changed the bios > > paths significantly, suc

Re: [systemd-devel] [PATCH 0/2 V2] gummiboot: Cross compile and IA32 support

2013-08-21 Thread Darren Hart
On Thu, 2013-08-22 at 00:53 +0200, Kay Sievers wrote: > On Thu, Aug 22, 2013 at 12:38 AM, Darren Hart wrote: > > Updated with feedback from Kay and Colin. I've boot tested this, then > > rebased to > > use Kay's updated CPP flags, and build tested. > >

Re: [systemd-devel] [PATCH 2/2] configure.ac: Disable the QEMU BIOS test when cross compiling

2013-08-21 Thread Darren Hart
On Thu, 2013-08-22 at 00:46 +0200, Kay Sievers wrote: > On Thu, Aug 22, 2013 at 12:38 AM, Darren Hart wrote: > > When cross-compiling, AC_CHECK_FILE aborts configure. Skip it when > > cross compiling. > > > > Signed-off-by: Darren Hart > > Cc: Colin Walter

Re: [systemd-devel] [PATCH 3/5] Makefile.am: Allow for user override of EFI include dir

2013-08-21 Thread Darren Hart
On Thu, 2013-08-22 at 00:32 +0200, Zbigniew Jędrzejewski-Szmek wrote: > On Wed, Aug 21, 2013 at 11:13:41AM -0700, Darren Hart wrote: > > Enable cross-compiling by allowing the user to specify an alternative to > > the /usr/include directory for the EFI includes. Add a

[systemd-devel] [PATCH 2/2] configure.ac: Disable the QEMU BIOS test when cross compiling

2013-08-21 Thread Darren Hart
When cross-compiling, AC_CHECK_FILE aborts configure. Skip it when cross compiling. Signed-off-by: Darren Hart Cc: Colin Walters --- configure.ac | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 137580a..a9534be 100644 --- a

[systemd-devel] [PATCH 1/2] Autodetect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
). Signed-off-by: Darren Hart --- Makefile.am | 3 ++- src/efi/gummiboot.c | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index 2678ab3..062f83e 100644 --- a/Makefile.am +++ b/Makefile.am @@ -80,7 +80,8 @@ efi_cppflags

[systemd-devel] [PATCH 0/2 V2] gummiboot: Cross compile and IA32 support

2013-08-21 Thread Darren Hart
Updated with feedback from Kay and Colin. I've boot tested this, then rebased to use Kay's updated CPP flags, and build tested. I am using a laoder/* config, so I wasn't sure where the BOOTIA32.EFI string came into play. Is that only used if no loader is provided. If gummiboot is typically BOOTIA32

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Thu, 2013-08-22 at 00:20 +0200, Kay Sievers wrote: > On Thu, Aug 22, 2013 at 12:07 AM, Darren Hart wrote: > > On Wed, 2013-08-21 at 23:37 +0200, Kay Sievers wrote: > >> On Wed, Aug 21, 2013 at 11:34 PM, Darren Hart > >> wrote: > >> > >> > Hrm

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Thu, 2013-08-22 at 00:20 +0200, Kay Sievers wrote: > On Thu, Aug 22, 2013 at 12:07 AM, Darren Hart wrote: > > On Wed, 2013-08-21 at 23:37 +0200, Kay Sievers wrote: > >> On Wed, Aug 21, 2013 at 11:34 PM, Darren Hart > >> wrote: > >> > >> > Hrm

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 23:37 +0200, Kay Sievers wrote: > On Wed, Aug 21, 2013 at 11:34 PM, Darren Hart wrote: > > > Hrm, actually no that fails finding VERSION ins setup.c. Did > > you ./autogen.sh before building? > > Oh, sorry, it probably just misses:

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 13:41 -0700, Darren Hart wrote: > On Wed, 2013-08-21 at 22:25 +0200, Kay Sievers wrote: > > On Wed, Aug 21, 2013 at 10:16 PM, Darren Hart > > wrote: > > > On Wed, 2013-08-21 at 21:31 +0200, Kay Sievers wrote: > > >> On Wed,

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 22:25 +0200, Kay Sievers wrote: > On Wed, Aug 21, 2013 at 10:16 PM, Darren Hart wrote: > > On Wed, 2013-08-21 at 21:31 +0200, Kay Sievers wrote: > >> On Wed, Aug 21, 2013 at 9:26 PM, Darren Hart > >> wrote: > >> > On Wed, 2013-

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 13:16 -0700, Darren Hart wrote: > On Wed, 2013-08-21 at 21:31 +0200, Kay Sievers wrote: > > On Wed, Aug 21, 2013 at 9:26 PM, Darren Hart wrote: > > > On Wed, 2013-08-21 at 20:59 +0200, Kay Sievers wrote: > > >> On Wed, Aug 21, 2013 at 8:13

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 21:31 +0200, Kay Sievers wrote: > On Wed, Aug 21, 2013 at 9:26 PM, Darren Hart wrote: > > On Wed, 2013-08-21 at 20:59 +0200, Kay Sievers wrote: > >> On Wed, Aug 21, 2013 at 8:13 PM, Darren Hart > >> wrote: > >> > The EFI specificat

Re: [systemd-devel] [PATCH 1/5] configure.ac: Add option to disable configuring the BIOS test

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 15:57 -0400, Colin Walters wrote: > On Wed, 2013-08-21 at 11:13 -0700, Darren Hart wrote: > > > +if test "x$enable_biostest" != "xno"; then > > Use AS_IF() please; systemd's current configure.ac is not consistent in > this respe

Re: [systemd-devel] [PATCH 1/5] configure.ac: Add option to disable configuring the BIOS test

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 21:15 +0200, Kay Sievers wrote: > On Wed, Aug 21, 2013 at 8:13 PM, Darren Hart wrote: > > When cross-compiling, AC_CHECK_FILE aborts configure. Provide a means to > > avoid even attempting to configure the BIOS test. > > > >

Re: [systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
On Wed, 2013-08-21 at 20:59 +0200, Kay Sievers wrote: > On Wed, Aug 21, 2013 at 8:13 PM, Darren Hart wrote: > > The EFI specification documents /EFI/BOOT/bootx64.efi for x86_64 > > machines and /EFI/BOOT/bootia32.efi for ia32 machines. Update the auto > > detect

[systemd-devel] [PATCH 5/5] Auto detect both x64 and ia32 boot*.efi payloads

2013-08-21 Thread Darren Hart
The EFI specification documents /EFI/BOOT/bootx64.efi for x86_64 machines and /EFI/BOOT/bootia32.efi for ia32 machines. Update the auto detection to allow for both. Signed-off-by: Darren Hart --- src/efi/gummiboot.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/efi

[systemd-devel] [PATCH 3/5] Makefile.am: Allow for user override of EFI include dir

2013-08-21 Thread Darren Hart
Enable cross-compiling by allowing the user to specify an alternative to the /usr/include directory for the EFI includes. Add a variable INCDIR defaulting to /usr/include, but still allowing the user to provide their own value. Signed-off-by: Darren Hart --- Makefile.am | 5 +++-- 1 file

[systemd-devel] [PATCH 4/5] configure.ac: Allow for more than just i686 for ia32

2013-08-21 Thread Darren Hart
Expand the ARCH_I686 to include i*86* (specifically to catch i586). Rename ARCH_I686 to ARCH_IA32 as that is more accurately what we are testing for. Signed-off-by: Darren Hart --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac

[systemd-devel] [PATCH 2/5] configure.ac: Use AC_CHECK_HEADER to detect the efi includes

2013-08-21 Thread Darren Hart
usability (conftest.c compilation), we don't lose much by just not looking for efi.h and assume it exists if we can find and use efibind.h. Signed-off-by: Darren Hart --- configure.ac | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure.a

[systemd-devel] [PATCH 1/5] configure.ac: Add option to disable configuring the BIOS test

2013-08-21 Thread Darren Hart
When cross-compiling, AC_CHECK_FILE aborts configure. Provide a means to avoid even attempting to configure the BIOS test. Signed-off-by: Darren Hart --- configure.ac | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index ca5a383

[systemd-devel] [PATCH 0/5] gummiboot: Crosscompile fixes and 32b boot target support

2013-08-21 Thread Darren Hart
I was carrying these patches in my yocto project recipe for gummiboot as well. Apologies, I should have included these along with the previous ia32 tsc patch. Thanks, Darren Hart ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http

[systemd-devel] [PATCH] gummiboot: Add 32 bit compatible rdtsc asm

2013-08-21 Thread Darren Hart
om(TM) CPU E640 @ 1.00GHz A value of 10 was detected. Signed-off-by: Darren Hart Cc: "Ylinen, Mikko" --- src/efi/gummiboot.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/efi/gummiboot.c b/src/efi/gummiboot.c index 844ce5f..9f2f31d 10064