diff options
author | Zachary Bedell <[email protected]> | 2011-09-18 21:35:42 -0400 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2011-09-26 09:44:43 -0700 |
commit | 7a0232735d7527b08e02196762727994aa6065f9 (patch) | |
tree | 068d6005443c0cd3ca82735d34586c87b0523d86 /cmd | |
parent | c39b2786ac98ab87d6dda00aa83b399ed175055a (diff) |
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 <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #344
Diffstat (limited to 'cmd')
0 files changed, 0 insertions, 0 deletions