public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
* [Linux-ia64] Re: new GPT and uuid patches
@ 2001-12-11  5:44 David Mosberger
  2001-12-11 15:40 ` [Linux-ia64] " Matt_Domsch
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: David Mosberger @ 2001-12-11  5:44 UTC (permalink / raw)
  To: linux-ia64

>>>>> On Mon, 10 Dec 2001 22:17:06 -0600, Matt_Domsch@Dell.com said:

  Matt> to be useful in more places.  I withdraw my previous patch
  Matt> implementing uuid.[ch] as libraries, letting efi_guid_t remain
  Matt> it's own unique thing.  It was hard enough getting the crc32
  Matt> library patch submitted, making other non-critical library
  Matt> cleanups is more than I can handle ATM.

Most of the patch looks good to me, but I do not understand why you
want to use a structured type for GUID.  This really does make the
whole store much more complicated than it has to be.  As per our
earlier discussion with Linus and Alan, the GUID should be just an
unstructured sequence of 16 bytes.  If you do that, all those ugly
conversion routines can go away and your head doesn't spin trying to
figure out what needs to be big-endian and what needs to be little
endian.  The *only* thing that needs to be endian aware is the
parsing/printing routines.  Those will be EFI GUID specific and will
have to make up for the mistake Intel made when defining the external
representation in the weird mixed byteorder (well, I'm assuming it was
a mistake; I don't assume someone would willfully do such a thing ;-).
The other benefit is that this will make GUIDs identical to UUIDs
(which they ought be), except for the external representation.

	--david


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2001-12-12  3:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-12-11  5:44 [Linux-ia64] Re: new GPT and uuid patches David Mosberger
2001-12-11 15:40 ` [Linux-ia64] " Matt_Domsch
2001-12-11 21:40 ` Matt Domsch
2001-12-12  3:55 ` Matt Domsch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox