]> git.proxmox.com Git - mirror_zfs.git/commit
Make libefi-created GPT compatible with gptfdisk
authorZachary Bedell <zac@thebedells.org>
Mon, 19 Sep 2011 01:35:42 +0000 (21:35 -0400)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Mon, 26 Sep 2011 16:44:43 +0000 (09:44 -0700)
commit7a0232735d7527b08e02196762727994aa6065f9
tree068d6005443c0cd3ca82735d34586c87b0523d86
parentc39b2786ac98ab87d6dda00aa83b399ed175055a
Make libefi-created GPT compatible with gptfdisk

GPT's created by libefi set the HeaderSize attribute in the GPT
header to 512 -- size of the GPT header INCLUDING the 420 padding
bytes at the end.  Most other tools set the size to 92 -- size of
the actual header itself excluding the padding.  Most tools check
the recorded HeaderSize when verifying CRC, but gptfdisk hardcodes
92 and thus reports CRC verification problems for full-disk vdevs
created IE with `zpool create pool sdc`.

This commit changes libefi's behavior for GPT creation and also
fixes several edge cases where libefi's behavior was similar
(though in an incompatible manner) to gptfdisk.  Libefi assumed
HeaderSize was always 512 even if the GPT recorded a different
value.  Sanity checks of the GPT headersize read from disk were
added before applying checksum calculation -- this will prevent
segfault in cases of bogus on-disk values.

Zpools created with the resuling libefi are verified as correct
both by parted and gptfdisk.  Also pool have been tested to
import correctly on ZFS on Linux as well as Solaris Express 11
livecd.

Signed-off-by: Zachary Bedell <zac@thebedells.org>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #344
include/sys/efi_partition.h
lib/libefi/rdwr_efi.c