dasd: allow the use of a *regular* backing file

Before this change, dasd partition tables had to be associated with
very specialized block devices available only on IBM s390 systems.
This made testing especially hard.  Not only did you need access
to one of those relatively uncommon systems, but you also needed
access to a spare block device.  In addition, many of Parted's
tests may be run as a non-privileged user, using regular files,
and those tests would fail when run on an s390.

This change makes it so you may now write a DASD partition table
to a regular backing file, just as you may with any other supported
partition table type.  However, note that even now, DASD support
is conditionally compiled and hence enabled only when compiling
for an s390 target.

* libparted/arch/linux.c (_device_set_sector_size) [__s390__]:
Allow operation on a device of type PED_DEVICE_FILE, as well as
those of type PED_DEVICE_DASD.
* libparted/labels/fdasd.c: Include <parted/device.h>.
(fdasd_get_geometry): Add a new "dev" parameter, required for
dev-phys-sector_size, which is needed in order to initialize
the "anc" structure without using the device-specific ioctls.
When file descriptor "f" refers to a regular file, initialize
"anc", "dasd_info", etc. from other sources (fstat and dev->).
Update all callers to reflect added parameter.
(fdasd_check_api_version): Skip ioctl-calling tests when "f" is
a regular file.
* include/parted/fdasd.h: Update fdasd_get_geometry prototype.
* libparted/labels/dasd.c (dasd_probe): Also allow PED_DEVICE_FILE.
(dasd_alloc_metadata): Initialize arch_specific->real_sector_size,
which is used in calculating vtoc_end.

Reviewed-by: Karel Zak <kzak@redhat.com>

loop: simplify/correct loop_clobber

Before, it would clear a sector only if it recognized its own
signature.  In addition, it would continue clearing subsequent
sectors as long as loop_probe succeeded.  Now we simply clear
the first sector.
* libparted/labels/loop.c (loop_clobber): Simplify.

Reviewed-by: Karel Zak <kzak@redhat.com>

tests: indicate that we expect t3200-type-change to fail, for now

* tests/Makefile.am (TESTS): Move t3200-type-change.sh to ...
(XFAIL_TESTS): ...here.

tests: correct t3200-type-change.sh

* tests/t3200-type-change.sh: The initial version was incomplete.

tests: provide a new function: fail_

* tests/t-lib.sh (fail_): New function.

tests: t0001-tiny.sh: avoid spurious failure when run as root.

* tests/t0001-tiny.sh: Control characters were getting in the way.
Filter them out.

dasd: avoid NULL-dereference via disk->type->ops->partition_check

Commit f387fee8 added the new ops->partition_check() function,
and a <type>_partition_check function for each partition table
format _except_ dasd.  This change adds one for dasd, too.
* libparted/labels/dasd.c: Include "pt-tools.h".
(dasd_partition_check): New function.
* libparted/labels/pt-tools.c (ptt_partition_max_start_len): Add "dasd"
to the list of partition table type names for which the 32-bit
limits on starting sector and partition length apply.
Correct a comment.
Considering the 32-bit fields in "struct fdasd_hd_geometry",
it is safe to assume that larger offset or size is not possible.

build: "make stable" emitted an invalid gnupload command

* cfg.mk (gnu_ftp_host-stable): Rename from gnu_ftp_host-major.

test for partition-type-changing bug

* tests/t3200-type-change.sh: New file.
* tests/Makefile.am (TESTS): Add it.

gpt: greatly simply gpt_clobber; minor semantic change, too

* libparted/labels/gpt.c (gpt_clobber): Don't bother to read and
parse existing headers.  Instead, simply clear three sectors:
the pMBR, the primary header (LBA1), and the last sector.
There is no point in clearing what the primary header says is the
AlternateLBA, since once the primary header and the last sector
are cleared, there is no risk of any tool using it by mistake.

tests: t0000-basic.sh: minor correction

* tests/t0000-basic.sh: When zeroing out the first sector,
use dd's conv=notrunc.  Otherwise, we'd also truncate the
backing file size to 4KiB.

linux_read: give a proper diagnostic for an "end of file" error

Before today's gpt_probe bug fix, its test case would provoke
an invalid diagnostic.  This makes the diagnostic useful.
* libparted/arch/linux.c (linux_read): When hitting EOF while reading,
diagnose it properly, rather than via strerror(0) (i.e., "Success").

gpt_probe: don't attempt to read beyond end of a very small disk

* libparted/labels/gpt.c (gpt_probe): Don't try to read the
2nd sector if that's beyond the end of the disk.
* tests/t0001-tiny.sh: New test, to expose the above boundary-case bug.
Part of the msdos-partition-creation process involves probing for
other types of partition tables.  Probing for gpt would evoke a
nonsensical diagnostic.
* tests/Makefile.am (TESTS): Add t0001-tiny.sh.

dvh: replace open-coded dvh_clobber with equivalent, shorter code

* libparted/labels/dvh.c (dvh_clobber): Simply use ptt_clear_sectors.

tests: weaken t0202-gpt-pmbr to assume a preexisting GPT table

* tests/t0202-gpt-pmbr.sh: Lay down an initial GPT table before
writing to the MBR, so this test passes with the new semantics.

tests: rewrite t0202 to use new framework

* tests/t0202-gpt-pmbr.sh: Rewrite.

ped_disk_type_get_next: fix a const-correctness bug

* libparted/disk.c (ped_disk_type_get_next): Make param const.
* include/parted/disk.h (ped_disk_type_get_next): Update prototype.

NEWS: mention recent changes

maint: more const-correctness fixes

dasd: fix ped_disk_new_fresh to not read old data from disk

dasd_write(), was reading the volume label from the disk (trough
fdasd_check_volume()) and later writing it back again, this is fine for
existing dasd labels, but when creating a fresh label, this would
also cause the old volume label to be re-used, and if the old label
was corrupt, it would cause fdasd_check_volume() and thus dasd_write()
to fail.  Reported here: http://bugzilla.redhat.com/533808

* libparted/arch/linux.c: include fdasd.h.
(init_dasd): Do BIODASDINFO ioctl, and store the dasd devno in
arch_specific.
(init_dasd): Remove dead (never reached) code.
* libparted/arch/linux.h (struct _LinuxSpecific): Add devno member.
* libparted/labels/dasd.c (DasdDiskSpecific): add vlabel member.
(dasd_alloc): Init DasdDiskSpecific.vlabel for fresh disks
(dasd_read): Store read vlabel in DasdDiskSpecific.vlabel.
(dasd_write): Write DasdDiskSpecific.vlabel instead of on disk vlabel.

build: avoid s390-specific compilation failiure

* libparted/labels/vtoc.c (vtoc_volume_label_init): Don't use
strncpy to copy 84 bytes into a 4-byte field.  Instead, use
memcpy to copy "sizeof *vlabel" bytes into the "vlabel".

maint: fix s390-specific const correctness problems

* libparted/labels/vtoc.c (vtoc_error, vtoc_ebcdic_enc):
(vtoc_ebcdic_dec): Declare parameters to be const, as required.
* libparted/labels/fdasd.c (fdasd_error): Likewise.
* include/parted/vtoc.h (vtoc_ebcdic_enc, vtoc_ebcdic_enc): Update
prototypes.

gpt: fix a bug in the new PE-array CRC checks on big-endian systems

* libparted/labels/gpt.c (check_PE_array_CRC, gpt_read): Always
convert the PartitionEntryArrayCRC32 to CPU endianness before
comparing with a just-computed CRC.

build: use gnulib's canonicalize-lgpl module

Replace Parted's private version of canonicalize_file_name.
* libparted/device.c: Include <stdlib.h>.
(canonicalize_file_name) [!HAVE_CANONICALIZE_FILE_NAME]:
Remove definition.
* bootstrap.conf (gnulib_modules): Add canonicalize-lgpl.

libparted: linux: don't deref NULL upon failed malloc or realloc

* libparted/arch/linux.c (_read_fd): Handle allocation failure.

build: update gnulib submodule to latest

maint: use EXIT_SUCCESS and EXIT_FAILURE, not 0 and 1 to exit

Convert all uses automatically, via these two commands:
git grep -l '\<exit *(1)'|xargs --no-run-if-empty \
  perl -pi -e 's/\b(exit ?)\(1\)/$1(EXIT_FAILURE)/'
git grep -l '\<exit *(0)'|xargs --no-run-if-empty \
  perl -pi -e 's/\b(exit ?)\(0\)/$1(EXIT_SUCCESS)/'
* libparted/fs/fat/table.c (fat_table_get): Use symbolic exit codes.
* libparted/labels/vtoc.c (vtoc_update_format5_label_add): Likewise.
(vtoc_update_format5_label_del): Likewise.
(vtoc_update_format7_label_add): Likewise.
(vtoc_update_format7_label_del): Likewise.
* m4/parted.m4 (PARTED_CHECK_LIBPARTED): Likewise.
* parted/parted.c (do_quit): Likewise.
* parted/strlist.c (gettext_to_wchar, wchar_to_str): Likewise.
* parted/ui.c (reset_env, help_msg, non_interactive_mode): Likewise.

dasd: fix unused variable warning

* libparted/labels/dasd.c (dasd_get_partition_alignment): Fix unused
variable warning.

linux: use cached major minor instead of statting the device again

* libparted/arch/linux.c (_dm_remove_parts): Use cached major/minor
numbers.

linux: use devicemapper task name instead of device node name

Use devicemapper task name instead of device node name as basename for
devicemapper disk partitions.

This is needed for upcoming lvm changes where the /dev/mapper/foobar
files are becoming symlinks, so we will end up opening /dev/dm-#,
and naming our partitions dm-#p1, dm-#p2, instead of foobarp1,
foobarp2, etc.

linux: use major / minor to specify device mapper device in dm_task

Use major / minor to specify device mapper device instead of
dm_task_set_name. This is needed for upcoming lvm changes where the
/dev/mapper/foobar files are becoming symlinks, so we will end up
opening /dev/dm-#, and dm-# is not a valid dm_task name.

This is only needed in this one place, as in the other places where we
use dm_task_set_name, the name we use was returned by libdevicemapper
itself in an earlier function call.

tests: enable the FS-resizing (tests only HFS and FAT)

* tests/Makefile.am (TESTS): Add t3000-resize-fs.sh.

maint: remove stray "types and" in the big FS-op warning

* parted/parted.c (issue_fs_op_warning): Correct grammar in diagnostic.

tests: convert more tests to new framework

* tests/t0400-loop-clobber-infloop.sh: Convert.
* tests/t0010-script-no-ctrl-chars.sh: Likewise.
* tests/t3000-resize-fs.sh: Likewise, and convert it to use scsi_debug.
* tests/t-local.sh (require_512_byte_sector_size_): Copy this
function from test-lib.sh.

dasd: fix disk duplication

Fix dasd_duplicate and add a dasd_partition_duplicate function.
* libparted/labels/dasd.c(dasd_partition_duplicate): New function.
(dasd_duplicate): Don't set the disk_specific pointer to NULL!
(dasd_duplicate): Copy DasdDiskSpecific from old disk to new disk.

dasd: set disk_specific->format_type in dasd_alloc

Set disk_specific->format_type in dasd_alloc, so that the disk
returned by ped_disk_new_fresh is a valid dasd disk.
* libparted/labels/dasd.c (dasd_alloc): Set disk_specific->format_type.

dasd: cache real_sector_size in linux_probe

dasd_alloc was doing an ioctl to get the real sector size for the dasd
device (we are faking 512 bytes sectors), but when called directly
from ped_disk_new_fresh(), the device is not open causing this to fail.

This patch fixes things by caching the real_sector_size in the linux
arch_specific data and making the dasd label handling use this.
libparted/arch/linux.c(_device_set_sector_size): Cache real sector size.
(_device_set_sector_size, init_dasd, linux_new): Only compile dasd code
on s390 and s390x.

dasd: remove bogus part_info pointer from DasdPartitionData

The part_info pointer in DasdPartitionData was only used to
pass fdasd partition_info_t pointers from dasd_write to
the dasd_write dasd_update_type utility function. Since these
pointers are not valid outside the lifetime of dasd_write, they
should not be stored in the longer lived DasdPartitionData.

* libparted/labels/dasd.c(DasdPartitionData): Remove part_info member.
(dasd_read): Stop setting DasdPartitionData part_info.
(dasd_update_type): Add part_info argument to pass fdasd
partition_info_t pointers from dasd_write.
(dasd_write): Stop setting DasdPartitionData part_info, pass
partition_info_t pointers to dasd_update_type as a separate argument.

dasd: clean up anchor handling

The current dasd label code keeps an fdasd anchor struct in the
DasdDiskSpecific struct, and fills this during dasd_read. However this
anchor does not get updated with any future mods, until dasd_write,
at which points it gets completely re-initialized.

Since the contents of the anchor saved during read does not get used
anywhere else, this patch switches to using local anchor structs in
dasd_read and dasd_write. This will also allow writing a significantly
simpler duplicate implementation, then the one which is currently
added to the Fedora package with a patch from Joel Granados (The
current master dasd_duplicate implementation is not functional).

This patch also fixes several missing calls to fdasd_cleanup() fixing
several memory leaks.

* libparted/arch/linux.h (struct _LinuxSpecific): Drop anchor member.
* libparted/labels/dasd.c (DasdDiskSpecific): Drop anchor member.
(dasd_clobber): Add missing fdasd_cleanup().
(dasd_read, dasd_write): Use local anchor struct instead of a
dynamically allocated one in DasdDiskSpecific.
(dasd_read): Don't leak a PedConstraint in 2 error paths.
(dasd_update_type): Pass in the used anchor as argument.

tests: distribute new files, t-local.sh and t-lvm.sh

* tests/Makefile.am (EXTRA_DIST): Add t-local.sh and t-lvm.sh.

tests: appease "make syntax-check"

* tests/print-align.c: Include "progname.h".  Call set_program_name.

tests: t6000-dm.sh: update and adapt to changed LVM2

* tests/t6000-dm.sh: Rewrite to use new test framework and
to disable multipath testing, since LVM2 has changed in such
a way that it no longer works with loop devices.

tests: do not invoke scsi_debug_setup_ in a subshell

* tests/t9010-big-sector.sh: Do not invoke scsi_debug_setup_ in
a subshell, since it sets a variable that cleanup machinery expects
to see.
* tests/t9020-alignment.sh: Likewise.

tests: tweak temporary dirname prefix

* tests/t-lib.sh (t_): Use "pe-" as prefix, not "gz-".

tests: update lvm-utils.sh from lvm2

* tests/lvm-utils.sh: Merge in changes from lvm2.

tests: adapt/update lvm-related infrasture, too

* tests/t-local.sh: Source t-lvm.sh.
* tests/t-lvm.sh: Clone and update of lvm-utils.sh,
for when we retire that file in favor of this one.

tests: adapt t8000-loop.sh

* tests/t8000-loop.sh: Rewrite in new style, and adjust to
expect new diagnostic.

tests: print and check partition alignment info, too

* tests/print-align.c: Print partition alignment info, too
* tests/t9020-alignment.sh: Check partition alignment too.

tests: serialize tests that use the scsi_debug module

* tests/t-local.sh (scsi_debug_lock_file_): New global.
(scsi_debug_acquire_lock_): New function.
(scsi_debug_setup_): Call it, to acquire the lock.
(scsi_debug_cleanup_): Release the lock.

tests: test the new alignment functions

* tests/t9020-alignment.sh: New test.
* tests/Makefile.am (TESTS): Add it to the list.

tests: factor out some more

* tests/t-local.sh (require_scsi_debug_module_): New function.
* tests/t9010-big-sector.sh: Use it.

tests: perform scsi_debug cleanup automatically...

...rather than requiring each test to do it.
* tests/t-lib.sh ($cleanup_eval): New variable.  Initialize.
(remove_tmp_): Use it.
* tests/t-local.sh: Record whether modprobe succeeded.
(cleanup_eval_): Append an invocation of scsi_debug_cleanup_
so that it is always run.
(scsi_debug_cleanup_): Run rmmod only if the modprobe succeeded.
* tests/t9010-big-sector.sh (cleanup_): Remove.

tests: move scsi_debug framework into shared script

* tests/t-local.sh (-scsi_debug_cleanup_, wait_for_dev_to_appear_):
(print_sd_names_, scsi_debug_setup_): New functions, factored out of ...
* tests/t9010-big-sector.sh: ...this file.

tests: document and begin to factor out scsi_debug framework

* tests/t9010-big-sector.sh: Add comments, fix a bug in
"comm" usage, prepare to factor out scsi_debug framework.

tests: add a helper program to print alignment info for a device

* tests/print-align.c: New file.
* tests/Makefile.am (check_PROGRAMS): Define.
(AM_CPPFLAGS, LDADD): Define.
(AM_CFLAGS): Enable warnings here, too, by adding
$(WARN_CFLAGS) and $(WERROR_CFLAGS).

Implement get_partition_alignment DiskOp for all labels which need it

Implement get_partition_alignment DiskOp for all labels which require
an alignment with a grain_size != 1.

libparted: add ped_disk_get_partition_alignment() function

This functions get the alignment needed for partition boundaries on this disk.
The returned alignment describes the alignment for the start sector of the
partition, for all disklabel types which require alignment, except Sun
disklabels, the end sector must be aligned too. To get the end sector
alignment decrease the PedAlignment offset by 1.

libparted: add ped_device_get_xxx_aligned_constraint() functions

Add ped_device_get_minimal_aligned_constraint() and
ped_device_get_optimal_aligned_constraint() functions to libparted.

These functions can be used to get a device constraint taking alignment into
account.

ped_device_get_constraint: return constraint without alignment

In libparted-1.9 ped_device_get_constraint() would always return a constraint
without any start / end aligning (grain_size == 1). Now that we are starting to
support disks with a physical sector size other then 512, the old
ped_device_get_constraint() code may actually start returning constrains
with alignment info with a grain_size != 1.

Since this is a behavior change and since the old code does not take a possible
alignment info into account, this patch removes the aligning code from
ped_device_get_constraint(), so that its behavior is unchanged.

This patch also updates the functions doxygen documentation to reflect what
it has actually done all these years.

linux: add get_xxx_alignment() functions

* libparted/arch/linux.c (get_minimum_alignment): New function.
(get_optimum_alignment): New function.

linux: save blkid topology in arch_specific data for later use

This is a preparation patch for adding get_minimum_alignment
and get_optimum_alignment functions to linux's _PedDeviceArchOps.
* libparted/arch/linux.h (struct_LinuxSpecific) [probe, topology]:
Add members.
* libparted/arch/linux.c (linux_new): initialize
arch_specific->probe and arch_specific->topology to NULL,
(linux_destroy): Free arch_specific->probe.
(get_minimum_io_size): Remove function.
(get_blkid_topology): New function.
(_device_set_sector_size): Call get_blkid_topology() instead of
get_minimum_io_size() and retrieve minimum io size from the cached
topology.

libparted: add ped_device_get_xxx_alignment() functions

Add ped_device_get_minimum_alignment() and ped_device_get_optimum_alignment()
functions to libparted.

Note this is a resent of my previous patchset with a number of typos corrected:
aligment -> alignment
minimal_alignment -> minimum_aligment
optimal_alignment -> optimum_aligment

Sorry for the noise.

tests: protect t7000 from the environment

* tests/t7000-scripting.sh: Initialize fail=0, so that an
environment setting cannot perturb the test.

tests: convert to new testing framework

* tests/t0200-gpt.sh: Convert to new testing framework.
* tests/t0201-gpt.sh: Likewise.
* tests/t0300-dos-on-gpt.sh: Likewise.
* tests/t0100-print.sh: Likewise.
* tests/t0250-gpt.sh: Likewise.

dvh: document an as-yet-unplugged (but not serious) leak

* libparted/disk.c (ped_disk_add_partition): List some of the
things I've tried to plug a leak exposed by a root-only test
on dvh's partition-creation error path.  This leak is enough
of a corner case that I may even leave it for now, and add a
suppression for it, so that we're not distracted by it.

dvh: plug three constraint-sized leaks

* libparted/labels/dvh.c (dvh_read, dvh_alloc_metadata):
Free constraint upon failure, too.

ui: plug multiple command_line_get_word leaks

* parted/parted.c (do_mkpart, do_print):

check: plug an even smaller (formatted "unit") leak

* libparted/disk.c (ped_disk_check): Free fs_size also upon success.

check: plug a geometry-sized leak

* libparted/disk.c (ped_disk_check): Destroy "geom", when done with it.

loop: don't leak a sector in loop_clobber

* libparted/labels/loop.c (loop_clobber): Replace leaky open-coded
version with a use of Use ptt_clear_sectors.

tests: t7000: remove mkpartfs tests

This test tested both mkpart and mkpartfs.
* tests/t7000-scripting.sh: Remove the mkpartfs tests.

tests: remove tests that are too FS-centric

* tests/t1000-mkpartfs.sh: Remove file.
* tests/t1500-small-ext2.sh: Likewise.
* tests/t2000-mkfs.sh: Likewise.
* tests/t3100-resize-ext2-partion.sh: Likewise.
* tests/t3000-constraints.sh: Likewise.
* tests/Makefile.am (TESTS): Remove them here, too.

ui: plug a constraint leak in do_mkpart

* parted/parted.c (do_mkpart): Free constraint after using it to
add partition.

gpt: plug two gpt-header leaks in gpt_read_headers

* libparted/labels/gpt.c (gpt_read_headers): Free locals pri and/or
bak if they are not valid.

gpt: plug a 1-sector leak in gpt_read_headers

* libparted/labels/gpt.c (gpt_read_headers): Free a sector.

ui: plug a leak (partition_print_flags)

* parted/parted.c (do_print): Free flags, after printing.

libparted: plug an FS-alias-related leak

* libparted/filesys.c (ped_file_system_alias_unregister):
Free the alias we've just unregistered.

doc: update NEWS

* NEWS (Bug fixes): Describe two.

gpt: header validity: also consider whether a PE array crc matches

* libparted/labels/gpt.c (check_PE_array_CRC): New function.
(_header_is_valid): Use it to verify that partition entry array
checksums match, too.
Karel Zak noticed that this check was not being performed.
* tests/t0280-gpt-corrupt.sh: Adapt to changed diagnostic.

gpt: tweak interfaces to pass disk, not disk->dev

* libparted/labels/gpt.c: required for upcoming CRC-checking function.

gpt: don't ignore an improbable (1-in-2^32) failure

pth_crc32 mapped failure to a crc32 value of 0.  If that happened
to match the stored value, there would be no indication that there
had been a malloc failure.
* libparted/labels/gpt.c (pth_crc32): Change interface so we can
distinguish failure from a CRC value of 0.
Update all callers.
(_generate_header): Now that we recognize pth_crc32 can fail,
we must admit that its void-returning caller _generate_header
may fail, too.  Return "int" and adjust *its* callers.

gpt: factor out code that is about to be reused

* libparted/labels/gpt.c: Include <errno.h>.
(gpt_read_PE_array): New function, ...
(gpt_read): ...factored out of this one.  Use it.

gpt: don't dereference NULL upon OOM (pth_get_raw callers)

Handle each case in which pth_get_raw may return NULL.
* libparted/labels/gpt.c (pth_get_raw): Upon OOM error, return 0
rather than dereferencing NULL.
(gpt_write): Don't dereference NULL upon OOM error.
(gpt_read): Yet another.

gpt: don't dereference NULL upon OOM (in pth_get_raw)

* libparted/labels/gpt.c (pth_get_raw): Don't dereference NULL
on out of memory error.

gpt: remove dead store

* libparted/labels/gpt.c (pth_crc32): Remove useless initialization.

gpt: make an internal function "const"-correct

* libparted/labels/gpt.c (_parse_header): Make "gpt" const.

tests: convert t0280 to new test style.

* tests/t0280-gpt-corrupt.sh: Convert to new test style.
* tests/t-lib.sh: Source t-local.sh.
* tests/t-local.sh: New file.

gpt: clean up

* libparted/labels/gpt.c (gpt_probe): Remove now-useless test before
pth_free.
(gpt_read): Use ptt_clear_sectors in place of open-coded equivalent.
* cfg.mk (useless_free_options): Add --name=pth_free.

gpt: make its header check a tiny bit more strict

* libparted/labels/gpt.c (_header_is_valid): Declare a header
to be invalid if its AlternateLBA is the same as MyLBA.

2009-10-21  Jim Meyering  <meyering@redhat.com>

complete most of gpt-CRC-related rewrite
* libparted/labels/gpt.c (_read_header): Remove now-unused function.

t0280-gpt-corrupt.sh: adapt for new output, improve test

gpt: rewrite GPT header-reading code

* libparted/labels/gpt.c (read_header): Remove now-unused function.
(gpt_read_headers): New function.
(gpt_read): Rewrite to use gpt_read_headers.

gpt: require that the backup table's AlternateLBA is 1

* libparted/labels/gpt.c (_header_is_valid): When checking
the backup header, ensure that its AlternateLBA is 1.

tests: gpt: test for detection of corrupt backup partition header

* tests/t0280-gpt-corrupt.sh: Corrupt the backup partition header's
MyLBA, and require that an attempt to print the table fails.

tests: hook up new GPT-crc-corrupting script

* tests/Makefile.am (TESTS): Add t0280-gpt-corrupt.sh.
* libparted/labels/gpt.c (gpt_read): Change diagnostic to say that
the PTE CRC mismatch is in the primary table.
* tests/t0280-gpt-corrupt.sh: Tweak diagnostic to match
what the code now prints.

gpt: also verify MyLBA when checking table validity

* libparted/labels/gpt.c (_header_is_valid): Add parameter, my_lba.
(_read_header): Update caller.
(_parse_header): Comment out this use of _header_is_valid
in an assertion.

gpt: clean up formatting of static initializers

* libparted/labels/gpt.c (NULL_IF_DISCOVER_ONLY): Define.
(gpt_disk_ops): Use it to avoid in-decl #if directives.

gpt.c: filter through indent --no-tabs...

and manual fix-up to shorten long lines.
Also, manually format the static initializers in e.g., gpt_disk_ops,
since GNU indent botches that.

gpt: add a comment

* libparted/labels/gpt.c (_header_is_valid): Add a comment to explain
the new partition entry size limitation of UINT32_MAX/16.

build: update gnulib submodule to latest

tests: use scsi_debug to test-for 4k-sector-specific bug

* tests/t9010-big-sector.sh: New root-only test.
Requires 2.6.31, and the scsi_debug module, /sys/block/... support
and an upcoming version of libblkid.
* tests/Makefile.am (TESTS): Add t9010-big-sector.sh.
Thanks to Mike Snitzer for advice and for pointing me to similar
code he wrote to test LVM.