Bug#574863: bumping this issue up a notch or two

[sean finney]

here's an updated, but still incomplete patch.  

the main problem with the previous patch was that the extent_count variable was
being overwritten when fetching the type option value.  simply removing
that fixed that problem.  there were further problems with the lv's though, as
it seems (i'm not an lvm guru by any means) that a snapshot consists of
two entries, one named "snapshot0" (this was the one the previous patch
would skip), and another one with the snapshot name, which is notably
different from the others in that the "status" field does not contain
"VISIBLE" like the others.

i updated the patch to also skip such lv entries, which cleared further
problems.  after these two steps, running update-grub/grub-install/grub-probe
all work without reporting errors.

*HOWEVER*, the system still doesn't boot.  after reinstalling grub and
regenerating /boot/grub/grub.cfg, the user is greeted with the error:

	error: no such disk
	error: physical volume pv0 not found
	Entering rescue mode...
	grub rescue>

so it looks like there is yet another failure in grub_lvm_read causing this.
honestly the code is so full of spaghetti i'm having trouble making it much
further.  i have a test setup running under kvm so if anyone has something
they want me to try please let me know.  otherwise i'll continue to hack at
it but i'm not too optomistic.


	sean

-- 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 574863.patch.2.diff
Type: text/x-diff
Size: 2322 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-grub-devel/attachments/20100509/47515ced/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-grub-devel/attachments/20100509/47515ced/attachment.pgp>