2002-08-22 18:43:41

by Andries E. Brouwer

[permalink] [raw]
Subject: NULL_GUID

Earlier this evening I happened to compile a 2.4.20-pre4 and got

efi.c: In function `add_gpt_partitions':
efi.c:728: `NULL_GUID' undeclared (first use in this function)

Earlier it was defined in include/asm-ia64/efi.h,
but since there is no relation with ia64 it is reasonable
that it was moved, only it never arrived in partitions/efi.h.

[I'll be lazy and not supply the trivial patch. Too likely
that others already did.]

Andries


2002-08-22 18:57:43

by Matt Domsch

[permalink] [raw]
Subject: RE: NULL_GUID

> Earlier this evening I happened to compile a 2.4.20-pre4 and got
>
> efi.c: In function `add_gpt_partitions':
> efi.c:728: `NULL_GUID' undeclared (first use in this function)
>
> Earlier it was defined in include/asm-ia64/efi.h,
> but since there is no relation with ia64 it is reasonable
> that it was moved, only it never arrived in partitions/efi.h.
>
> [I'll be lazy and not supply the trivial patch. Too likely
> that others already did.]

Yep. FWIW, these are incorporated in the IA64 port patch released yesterday
too. I'd love to see these included in the standard 2.4.x trees.

Patches available in a number of places:
bk pull http://mdomsch.bkbits.net/linux-2.4-gpt

As separate patches:
1)
http://domsch.com/linux/patches/gpt/linux-2.4-gpt-efiguidt.cset
has been in the IA64 port for a while. This fixes the endianness issues
with the efi_guid_t type and adds the NULL_GUID definition needed to compile
the GPT code.

2)
http://domsch.com/linux/patches/ia64/linux-2.4-efihmove.cset
moves include/asm-ia64/efi.h to include/linux/efi.h similar to the 2.5
patches. This is needed to allow the GPT code to compile on non-IA64
platforms too, necessary for the use of really big disks.

3)
http://domsch.com/linux/patches/gpt/linux-2.4-gpt-efiguidt-unparse.cset
has been the IA64 port for a while. This fixes efi_guid_unparse for
endianness.


These need to be applied in the above order, as #1 touches efi.h, #2
touches and moves efi.h, and #3 touches it then too. All are already in the
ia64 port tree, but Marcelo and Alan don't have any of these three yet.
I've compiled this on x86 against BK-current building in GPT with no
troubles.

Thanks,
Matt

--
Matt Domsch
Sr. Software Engineer, Lead Engineer, Architect
Dell Linux Solutions http://www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com
#1 US Linux Server provider for 2001 and Q1/2002! (IDC May 2002)